chore(deps-dev): bump lint-staged from 16.4.0 to 17.0.5#60
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
📝 WalkthroughWalkthroughThe ChangesDependency Version Update
🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 73: The Node engine constraint is too permissive for lint-staged@17;
update the package.json "engines" entry for "node" to ">=22.22.1" (matching
lint-staged@17 minimum) and ensure any CI workflow node-version settings are
tightened similarly: change the node-version entries in the CI workflow(s) (the
two occurrences referenced around the workflow job setup and matrix) and in the
Gemini test workflow node-version to use "22.22.1" or ">=22.22.1" so CI and
local install enforce the same minimum Node version as the upgraded
"lint-staged" dependency.
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 4af94ff4-d76b-4690-8c88-b56ae80120d8
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
package.json
| "@types/node": "^20", | ||
| "husky": "^9.1.7", | ||
| "lint-staged": "^16.4.0", | ||
| "lint-staged": "^17.0.5", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify Node/Git compatibility signals in repo config (read-only).
set -euo pipefail
echo "== package.json engines =="
cat package.json | jq -r '.engines'
echo
echo "== lint-staged version =="
cat package.json | jq -r '.devDependencies["lint-staged"]'
echo
echo "== CI/setup-node versions (if present) =="
fd -i '.*(workflow|workflows).*\.ya?ml$' .github 2>/dev/null | while read -r f; do
echo "-- $f"
rg -n 'setup-node|node-version|node-version-file' "$f" || true
done
echo
echo "== Git version constraints mentioned in repo files (if any) =="
rg -n 'git.*2\.32|minimum git|requires git|git version' README.md .github package.json 2>/dev/null || trueRepository: AIX-Format/PiWorker-OS
Length of output: 258
🏁 Script executed:
# Check if .github directory exists and list its contents
ls -la .github 2>/dev/null || echo ".github directory not found"Repository: AIX-Format/PiWorker-OS
Length of output: 641
🌐 Web query:
lint-staged v17 minimum Node.js version requirements
💡 Result:
As of May 2026, lint-staged version 17 requires a minimum Node.js version of 22.22.1 [1][2][3]. With the release of v17.0.0, support for Node.js v20 was dropped [1][2]. Users are advised to upgrade their Node.js environment to at least v22.22.1, which is an active Long Term Support (LTS) version, to ensure compatibility with lint-staged v17 [1][3]. Additionally, lint-staged v17 requires a Git version of at least 2.32.0 [1][2].
Citations:
- 1: https://newreleases.io/project/github/lint-staged/lint-staged/release/v17.0.0
- 2: https://github.com/lint-staged/lint-staged/releases
- 3: https://newreleases.io/project/npm/lint-staged/release/17.0.0
🏁 Script executed:
# List workflow files
ls -la .github/workflows/
echo
echo "=== Checking for Node version pinning in workflows ==="
rg -n 'node-version|setup-node' .github/workflows/ --type yaml || echo "No Node version constraints found in workflows"Repository: AIX-Format/PiWorker-OS
Length of output: 871
Align Node engine floor with lint-staged@17 minimum requirement in both package.json and CI workflows.
Bumping to lint-staged@^17.0.5 while keeping "node": ">=22.x" in both package.json and CI workflows allows unsupported Node 22 patch versions (below 22.22.1) to run. Tighten the constraint to >=22.22.1 in both places to ensure compatibility.
Required changes
Update package.json:
- "engines": {
- "node": ">=22.x"
- },
+ "engines": {
+ "node": ">=22.22.1"
+ },Update .github/workflows/ci.yml (lines 26 and 132):
- node-version: "22.x"
+ node-version: "22.22.1"Update .github/workflows/gemini-test.yml (line 24):
- node-version: '22.x'
+ node-version: "22.22.1"🤖 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 73, The Node engine constraint is too permissive for
lint-staged@17; update the package.json "engines" entry for "node" to
">=22.22.1" (matching lint-staged@17 minimum) and ensure any CI workflow
node-version settings are tightened similarly: change the node-version entries
in the CI workflow(s) (the two occurrences referenced around the workflow job
setup and matrix) and in the Gemini test workflow node-version to use "22.22.1"
or ">=22.22.1" so CI and local install enforce the same minimum Node version as
the upgraded "lint-staged" dependency.
7a5fd0e to
0faaab5
Compare
Bumps [lint-staged](https://github.com/lint-staged/lint-staged) from 16.4.0 to 17.0.5. - [Release notes](https://github.com/lint-staged/lint-staged/releases) - [Changelog](https://github.com/lint-staged/lint-staged/blob/main/CHANGELOG.md) - [Commits](lint-staged/lint-staged@v16.4.0...v17.0.5) --- updated-dependencies: - dependency-name: lint-staged dependency-version: 17.0.5 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
0faaab5 to
bda6b39
Compare
There was a problem hiding this comment.
dependabot[bot] has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.
Bumps lint-staged from 16.4.0 to 17.0.5.
Release notes
Sourced from lint-staged's releases.
... (truncated)
Changelog
Sourced from lint-staged's changelog.
... (truncated)
Commits
cbd822eMerge pull request #1793 from lint-staged/changeset-release/main1911244chore(changeset): release7339d7dMerge pull request #1792 from lint-staged/fix-max-arg-length-default1f67271fix: defaultmaxArgLengthtoundefinedinstead ofNaNwhen parsing cli ...2862964Merge pull request #1789 from lint-staged/changeset-release/mainc9ecd54chore(changeset): releasecc6b51aMerge pull request #1788 from lint-staged/fix-update-index-againf95c1f8fix: update both default index.lock and non-standard lock when latter existsf44ee68Merge pull request #1786 from lint-staged/update-repo-urla61cf18build(deps): update dependencies