Update dependencies#58
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Summary by CodeRabbit
WalkthroughDependency versions are bumped across root, example projects, and the CLI package; tsdown build configuration is extended; test mock helpers are extracted to reduce duplication in three test suites; and a test environment setup step is added for symlink escape testing. ChangesDependency Updates and Test Infrastructure Improvements
🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
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 `@packages/cli/package.json`:
- Line 57: The package is using `@types/node` "^24.12.4" while engines.node is
">=20", which allows TypeScript to compile against Node 24-only APIs; update the
dependency entry for "`@types/node`" in packages/cli/package.json to pin it to the
Node 20 major (for example a ^20.x version) so typings match the declared
runtime floor and prevent accidental usage of Node 21+/24-only APIs; ensure the
package.json dependency for "`@types/node`" is changed and run a quick TypeScript
build to validate.
🪄 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: ASSERTIVE
Plan: Pro
Run ID: f3405666-5279-435f-aff2-2e0975241a89
⛔ Files ignored due to path filters (3)
examples/hello-world/pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlexamples/next-smoke/pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlpnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (9)
examples/hello-world/package.jsonexamples/next-smoke/package.jsonpackage.jsonpackages/cli/package.jsonpackages/cli/tests/app-build.test.tspackages/cli/tests/app-bun-compat.test.tspackages/cli/tests/app-provider.test.tspackages/cli/tests/auth-ops.test.tspackages/cli/tsdown.config.ts
Updates the CLI workspace and examples to current compatible dependency lines while preserving the published Node 20 runtime contract. This also folds in security-related updates for Vitest, Next.js, and the Next smoke example transitive PostCSS dependency.
Changes
@prisma/compute-sdk,c12,magicast,commander,tsdown,typescript,vitest, and shared root dev tooling.examples/hello-worldandexamples/next-smokedependencies, including TypeScript 6, React patch releases, Next.js 16.2.6, and apostcssoverride for the Next.js transitive advisory.packages/cli/tsdown.config.tsemittingdist/cli.jsafter the tsdown upgrade so the existing packagebincontract remains valid.Updated Packages
skills^1.5.7->^1.5.9;tsx^4.19.2->^4.22.4.@clack/prompts^1.2.0->^1.5.0;@prisma/compute-sdk^0.19.0->^0.20.0;@prisma/credentials-store^7.7.0->^7.8.0;@prisma/management-api-sdk^1.34.0->^1.35.0;c124.0.0-beta.4->4.0.0-beta.5;commander^12.1.0->^14.0.3;magicast^0.3.5->^0.5.3;string-width^8.2.0->^8.2.1.@types/node^24.0.0->^24.12.4;tsdown^0.12.5->^0.21.10;tsx^4.19.2->^4.22.4;typescript^5.8.3->^6.0.3;vitest^3.2.4->^4.1.8.@types/bun^1.2.0->^1.3.14;tsx^4.19.2->^4.22.4;typescript^5.9.3->^6.0.3.next16.2.3->16.2.6;react19.2.4->19.2.6;react-dom19.2.4->19.2.6.@types/node^20->^20.19.41;@types/react^19->^19.2.15;@types/react-dom^19->^19.2.3;tsx^4.19.2->^4.22.4;typescript^5->^6.0.3.postcssoverride at8.5.10for the transitive Next.js advisory.Why
The update takes compatible dependency lines as far as possible without raising
@prisma/clibeyond its declared Node>=20support. Security-related updates were prioritized, while dependencies requiring newer Node runtimes were intentionally held back.Skipped Dependencies
commander@15: requires Node>=22.12.0;@prisma/clicurrently supports Node>=20.tsdown@0.22.x: requires Node>=22.18.0;@prisma/clicurrently supports Node>=20.@types/node@25in@prisma/cli: would expose Node 25 APIs while the CLI still supports lower Node versions.@types/node@25inexamples/next-smoke: left on the Node 20 type line to match common supported deployment runtimes.Verification
pnpm testpnpm build:clipnpm lint:skillspnpm test:skillspnpm auditexamples/hello-world:pnpm exec tsc --noEmitexamples/hello-world:pnpm auditexamples/next-smoke:pnpm buildexamples/next-smoke:pnpm audit