Skip to content

Fix deprecated npm install warnings (#26816)#26853

Closed
AaronL2005 wants to merge 1 commit intoopenclaw:mainfrom
AaronL2005:fix/26816-deprecated-npm-warnings
Closed

Fix deprecated npm install warnings (#26816)#26853
AaronL2005 wants to merge 1 commit intoopenclaw:mainfrom
AaronL2005:fix/26816-deprecated-npm-warnings

Conversation

@AaronL2005
Copy link
Copy Markdown

@AaronL2005 AaronL2005 commented Feb 25, 2026

Updates pnpm overrides to use non-deprecated transitive deps pulled in by node-pre-gyp/cmake-js (npmlog, rimraf, glob), reducing deprecated warnings on install.\n\nFixes #26816

Greptile Summary

Adds pnpm overrides to replace deprecated transitive dependencies pulled in by @discordjs/node-pre-gyp and cmake-js, upgrading npmlog (5.0.1/6.0.2 → 7.0.1), rimraf (3.0.2 → 5.0.10), glob (7.2.3 → 10.4.5), and inflight (1.0.6 → @zkochan/inflight 1.0.6). This eliminates deprecated package warnings during pnpm install.

  • Overrides use semver ranges (^) which follows existing patterns in the pnpm.overrides section
  • Uses npm:@zkochan/inflight as a maintained fork replacement for the deprecated inflight package
  • Lockfile changes show proper resolution with no deprecated warnings
  • The approach aligns with existing override patterns in package.json for other security/deprecation fixes

Confidence Score: 4/5

  • This PR is safe to merge with minimal risk - it only updates deprecated transitive dependencies to their non-deprecated versions
  • The changes follow existing patterns in the codebase for dependency overrides and address a legitimate issue (deprecated install warnings). The pnpm overrides target specific transitive dependencies with well-tested, non-deprecated versions. The lockfile shows clean resolution without introducing breaking changes. One minor consideration: CLAUDE.md line 170 mentions that dependency patching/overrides requires explicit approval, though this PR references issue [Bug]: Deprecated npm dependencies warnings on install (v2026.2.24) #26816 which likely provided that context.
  • No files require special attention - the changes are straightforward dependency version overrides

Last reviewed commit: 6b9f946

@vincentkoc
Copy link
Copy Markdown
Member

Resolved in #28318

@vincentkoc vincentkoc closed this Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants