chore(deps): update dependency get-port to v7#47
Conversation
📝 WalkthroughWalkthroughThe 🚥 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 |
861aa0a to
36b26d3
Compare
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 38: The package.json file is missing an explicit Node engine constraint
even though the newly added [email protected] dependency requires Node >=16. Add
"node": ">=16" to the engines field in package.json to declare this minimum Node
version requirement and prevent unsupported local installations.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 0e6d601b-5cdc-475c-9695-683614bd984b
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (1)
package.json
| "fastify-static": "^4.7.0", | ||
| "file-loader": "^6.2.0", | ||
| "get-port": "^5.1.1", | ||
| "get-port": "^7.2.0", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "== Node/pnpm engine declarations =="
fd -HI 'package.json|.nvmrc|.node-version|.tool-versions' | while read -r f; do
echo "--- $f"
sed -n '1,220p' "$f"
done
echo
echo "== CI/workflow node-version references =="
rg -n -C2 'node-version|setup-node|actions/setup-node|NODE_VERSION' .github/workflows || true
echo
echo "== get-port usage patterns that may be impacted by v6+ API changes =="
rg -n -C3 --type=js --type=ts --type=mjs --type=cjs 'get-port|makeRange|portNumbers|clearLockedPorts|reserve'Repository: rstackjs/rspack-plugin-ci
Length of output: 17103
🏁 Script executed:
cat .github/workflows/test.ymlRepository: rstackjs/rspack-plugin-ci
Length of output: 1466
🏁 Script executed:
rg -n 'matrix|node' .github/workflows/test.yml | head -20Repository: rstackjs/rspack-plugin-ci
Length of output: 412
Add explicit Node engine constraint to match [email protected] requirement.
[email protected] requires Node >=16, but package.json does not declare a Node engine floor—only pnpm version is constrained. While CI tests only Node 22.x (which satisfies the requirement), the missing constraint leaves a gap where local installs could use unsupported versions. Add "node": ">=16" to the engines field for clarity and to prevent version mismatches.
Note: get-port is not directly used in the codebase, so the risk of breaking changes from the v5→v7 upgrade is low.
🤖 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 38, The package.json file is missing an explicit Node
engine constraint even though the newly added [email protected] dependency requires
Node >=16. Add "node": ">=16" to the engines field in package.json to declare
this minimum Node version requirement and prevent unsupported local
installations.
36b26d3 to
ba4f868
Compare
ba4f868 to
c11f7d0
Compare
This PR contains the following updates:
^5.1.1→^7.2.0Release Notes
sindresorhus/get-port (get-port)
v7.2.0Compare Source
reserveoption to lock ports for the process lifetime8af215bv7.1.0Compare Source
clearLockedPorts()method (#70)d4fb7f8v7.0.0Compare Source
Breaking
5cc3422Improvements
bd8a403v6.1.2Compare Source
portNumbersargument validation and number generation (#64)1fb0e70v6.1.1Compare Source
d7ba424v6.1.0Compare Source
excludeoption (#53)d03c07bv6.0.0Compare Source
Breaking
c08a4aerequire('get-port').makeRange=>import {portsNumbers] from 'get-port'Improvements
c3bbed9Configuration
📅 Schedule: (in timezone Asia/Shanghai)
* 0-3 1,15 * *)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.