chore(deps): bump commander from 12.1.0 to 15.0.0#239
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
WalkthroughThe CLI’s ChangesCommander dependency upgrade
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 52: Update the commander dependency declaration in package.json from
version 15 to the latest compatible Commander 14 range, preserving the
repository’s Node 20.19.0 engines and CI support.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 5cd487e3-5b50-4d4b-9d5b-74557a9b76de
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json,!package-lock.json
📒 Files selected for processing (1)
package.json
| "license": "Apache-2.0", | ||
| "dependencies": { | ||
| "commander": "^12.1.0", | ||
| "commander": "^15.0.0", |
There was a problem hiding this comment.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -eu
jq '{engines, type, scripts, dependencies}' package.json
fd -H -t f 'package-lock.json|npm-shrinkwrap.json|yarn.lock|pnpm-lock.yaml|tsconfig.*|Dockerfile.*' .
rg -n 'node-version|setup-node|FROM node|engines|module|moduleResolution|commander|exitOverride|configureOutput' \
package.json src .github Dockerfile* tsconfig*.json 2>/dev/null || trueRepository: TestSprite/testsprite-cli
Length of output: 7139
Commander 15 conflicts with the current Node support floor. This repo still declares and tests Node 20.19.0, but Commander 15 requires Node >=22.12. Keep Commander 14 unless you also drop Node 20 from engines and CI.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@package.json` at line 52, Update the commander dependency declaration in
package.json from version 15 to the latest compatible Commander 14 range,
preserving the repository’s Node 20.19.0 engines and CI support.
Bumps [commander](https://github.com/tj/commander.js) from 12.1.0 to 15.0.0. - [Release notes](https://github.com/tj/commander.js/releases) - [Changelog](https://github.com/tj/commander.js/blob/master/CHANGELOG.md) - [Commits](tj/commander.js@v12.1.0...v15.0.0) --- updated-dependencies: - dependency-name: commander dependency-version: 15.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
5cf455a to
d3ab42e
Compare
|
commander 15 requires Node >=22.12, which breaks our documented engines range (^20.19 || ^22.13 || >=24) — Build (Node 20) fails at install, and the ESM rewrite changes rendered --help text (11 snapshot failures). Consistent with our policy for majors that break CI: @dependabot ignore this major version |
|
OK, I won't notify you about version 15.x.x again, unless you re-open this PR. |
Bumps commander from 12.1.0 to 15.0.0.
Release notes
Sourced from commander's releases.
... (truncated)
Changelog
Sourced from commander's changelog.
... (truncated)
Commits
ba6d13dFix release dates in changelog (#2523)a752ed9Pin GitHub actions with hash (#2521)74d5dfeDrop EOL node 20 from test matrix, and add node 26 (#2520)6df9b68Update details for 15.0.0 release (#2519)01ce5d0Remove jest esm examples (#2517)d785d8bUpdate dependencies (#2518)9098b48Update dependencies (#2506)373f660Use node:util stripVTControlCharacters instead of own code (#2486)987f289Use simple match in test (to avoid warning about expensive regex) (#2485)0ea3bb3Update dependecies and lint (#2489)