-
Notifications
You must be signed in to change notification settings - Fork 463
Project dependencies update #1084
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
|
Cursor Agent can help with this pull request. Just |
|
@cursoragent Please review and fix CI failures on this PR. |
Summary
Testing
Consent needed to finish CI verification: please confirm you want me to rerun |
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is ON. A Cloud Agent has been kicked off to fix the reported issue.
|
Bugbot Autofix resolved the bug found in the latest run.
|
|
@cursoragent investigate CI failures with the tests |
|
Bugbot Autofix resolved the bug found in the latest run.
|
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>



Summary
Updated root
package.jsondev dependencies including Playwright, Vitest, and various@typespackages.prisma,@prisma/client,@prisma/instrumentation, andzodwere intentionally kept on their current major versions due to typecheck breakage on newer majors.Also updated
remix.initdependencies (execa,inquirer,open,parse-github-url). All lockfiles have been refreshed.Test Plan
DATABASE_URL="file:./data.db?connection_limit=1" npx prisma migrate deployDATABASE_URL="file:./data.db?connection_limit=1" npx prisma generate --sqlnpm run lintnpm run typecheckChecklist
Screenshots
Note
Medium Risk
Main risk is build/test tooling behavior changes (Vitest v4 and Vite module resolution override for
cache.server.ts), which could affect CI and test semantics but should not impact production runtime.Overview
Updates tooling dependencies and stabilizes Vitest runs. Dev dependencies are bumped (notably
vitest/@vitest/coverage-v8to v4,@playwright/test, and several@types/*packages) with lockfiles refreshed.Adds a Vitest-only stub for
cache.server.ts(tests/mocks/cache-server.ts) and a Vite pre-plugin to redirectcache.server.tsimports to the stub whenVITESTis set, avoidingnode:sqlitebundling/CACHE_DATABASE_PATHrequirements in unit tests. Also documents this decision (docs/decisions/047-*) and updates one GitHub provider import to use the#app/*alias.Updates
remix.initdependencies (execa,inquirer,open,parse-github-url) and refreshes its lockfile.Written by Cursor Bugbot for commit a602687. This will update automatically on new commits. Configure here.