Skip to content

refactor(chore): remove Node js 20 support#164

Open
Sourav-kashyap wants to merge 1 commit into
masterfrom
GH-163
Open

refactor(chore): remove Node js 20 support#164
Sourav-kashyap wants to merge 1 commit into
masterfrom
GH-163

Conversation

@Sourav-kashyap

Copy link
Copy Markdown

remove Node js 20 support

BREAKING CHANGE:
yes

GH-163

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Checklist:

  • Performed a self-review of my own code
  • npm test passes on your machine

Build:

Screenshot 2026-06-11 at 2 59 10 PM

Test:

Screenshot 2026-06-11 at 2 59 23 PM

remove Node js 20 support

BREAKING CHANGE:
yes

GH-163
@Sourav-kashyap Sourav-kashyap self-assigned this Jun 11, 2026
@Sourav-kashyap Sourav-kashyap linked an issue Jun 11, 2026 that may be closed by this pull request
@sonarqubecloud

Copy link
Copy Markdown

SonarQube reviewer guide

Summary: Drop support for Node.js 20 and update minimum required version to Node.js 22, along with dependency updates.

Review Focus:

  • Minimum Node.js version change from 20 to 22 in package.json engine requirements - verify this aligns with project needs and downstream consumer expectations
  • Extensive dependency updates throughout package-lock.json, including major version bumps for some packages (agent-base 7→9, http-proxy-agent 7→9, https-proxy-agent 7→9, type-is 2.0→2.1) - ensure compatibility with these changes
  • Breaking changes in proxy agent dependencies now require Node.js >= 20 (but we're requiring >=22, so this is safe)
  • CI workflow updated to test only Node.js 22 and 24 instead of 20, 22, and 24

Start review at: package.json. This is the policy change that drives the entire PR—the minimum Node.js requirement shift from 20 to 22. Understanding this constraint is essential before evaluating the lock file changes, as some dependency updates were likely made to support or be compatible with Node.js 22+.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove Node.js 20 support

1 participant