Skip to content

Project dependencies update#1084

Merged
kentcdodds merged 14 commits intomainfrom
cursor/project-dependencies-update-f13a
Feb 1, 2026
Merged

Project dependencies update#1084
kentcdodds merged 14 commits intomainfrom
cursor/project-dependencies-update-f13a

Conversation

@kentcdodds
Copy link
Member

@kentcdodds kentcdodds commented Feb 1, 2026

Summary

Updated root package.json dev dependencies including Playwright, Vitest, and various @types packages. prisma, @prisma/client, @prisma/instrumentation, and zod were intentionally kept on their current major versions due to typecheck breakage on newer majors.

Also updated remix.init dependencies (execa, inquirer, open, parse-github-url). All lockfiles have been refreshed.

Test Plan

  • DATABASE_URL="file:./data.db?connection_limit=1" npx prisma migrate deploy
  • DATABASE_URL="file:./data.db?connection_limit=1" npx prisma generate --sql
  • npm run lint
  • npm run typecheck

Checklist

  • Tests updated
  • Docs updated

Screenshots


Open in Cursor Open in Web


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-v8 to 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 redirect cache.server.ts imports to the stub when VITEST is set, avoiding node:sqlite bundling/CACHE_DATABASE_PATH requirements in unit tests. Also documents this decision (docs/decisions/047-*) and updates one GitHub provider import to use the #app/* alias.

Updates remix.init dependencies (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.

cursoragent and others added 3 commits February 1, 2026 14:10
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
@cursor
Copy link

cursor bot commented Feb 1, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@kentcdodds kentcdodds marked this pull request as ready for review February 1, 2026 14:19
@kentcdodds
Copy link
Member Author

@cursoragent Please review and fix CI failures on this PR.

@cursor
Copy link

cursor bot commented Feb 1, 2026

Summary

  • Mitigated the Vitest CI failure where Vite tried to bundle node:sqlite from cache.server.ts by aliasing that module to a test-only stub in test mode.【c7b474†L50-L60】【F:vite.config.ts†L41-L79】
  • Added an in-memory cache stub for tests to satisfy cache exports without sqlite dependencies.【F:tests/mocks/cache-server.ts†L1-L56】

Testing

  • npm run lint (warnings only in remix.init/index.mjs import order).【2afcf1†L1-L11】
  • npm run typecheck.【5131d6†L1-L4】
  • ⚠️ npm run test -- --coverage (blocked: Prisma CLI requires explicit consent to run npx prisma migrate reset --force --skip-seed --skip-generate during global setup; it would reset the local test DB at tests/prisma/base.db).【55971a†L1-L34】
    Testing rationale: ran lint/typecheck to validate tooling changes; no manual testing because no UI changes.

Consent needed to finish CI verification: please confirm you want me to rerun npm run test -- --coverage (invokes npx prisma migrate reset --force --skip-seed --skip-generate, which irreversibly deletes the local test database; safe only for non-production).【55971a†L1-L34】

Open in Cursor Open in Web

cursoragent and others added 2 commits February 1, 2026 14:31
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Copy link

@cursor cursor bot left a 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.

@cursor
Copy link

cursor bot commented Feb 1, 2026

Bugbot Autofix resolved the bug found in the latest run.

  • ✅ Fixed: Mock cachified omits required context parameter for getFreshValue
    • Updated the mock to pass a minimal context with metadata into getFreshValue, matching production behavior and preventing undefined access.

@kentcdodds
Copy link
Member Author

@cursoragent investigate CI failures with the tests

@cursor
Copy link

cursor bot commented Feb 1, 2026

Bugbot Autofix resolved the bug found in the latest run.

  • ✅ Fixed: Mock cachified omits required context parameter for getFreshValue
    • Updated the mock to pass a minimal context with metadata into getFreshValue, matching production behavior and preventing undefined access.

Create PR
Open in Cursor Open in Web

cursoragent and others added 8 commits February 1, 2026 14:53
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>
@kentcdodds kentcdodds merged commit d1c3dde into main Feb 1, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants