Update all dependencies - #69
Conversation
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_51e07a48-b479-4e25-8a12-7d1b2b5dbc6f) |
Greptile SummaryThe PR refreshes direct and transitive dependencies, updates the Vite middleware integration to load the Vite 8 configuration file, and adjusts tests for deterministic behavior under the upgraded toolchain.
Confidence Score: 5/5The PR appears safe to merge, with no concrete changed-code failure established in the supported build, test, development, or runtime paths. The dependency updates align with the existing APIs inspected, the test changes preserve their assertions, and the Vite migration is reached through documented repository-root development workflows without an established routing or startup regression.
|
| Filename | Overview |
|---|---|
| server/vite.ts | Migrates middleware-mode startup to Vite’s config-file loading path; no concrete failure was established in supported development workflows. |
| package.json | Updates all direct dependencies, including the connect-redis major version, while existing session code already matches its upgraded API. |
| yarn.lock | Refreshes resolved transitive dependencies consistently with the manifest updates. |
| server/tests/anilistOAuth.test.ts | Mocks background snapshot prefetching so OAuth persistence assertions remain isolated and deterministic. |
| client/src/components/tests/layout.test.tsx | Replaces nullable DOM querying with an accessible role query without changing the tested layout contract. |
Reviews (1): Last reviewed commit: "Update all dependencies" | Re-trigger Greptile
Summary
Validation
yarn run checkyarn test(21 files, 104 tests)yarn buildyarn outdated --json(no outdated direct dependencies)yarn audit --json(0 vulnerabilities)Supersedes #39, #40, #41, #42, #43, #44, #45, #47, #57, #61, #63, #66, #67, and #68.
Note
Medium Risk
Broad dependency upgrades (including
connect-redismajor, Vite 8.2, and session/redis-related packages) increase regression risk in dev server, auth, and caching paths despite limited code changes.Overview
Bumps every direct dependency in
package.json(Radix UI, Sentry, TanStack Query, Vite 8.2.1, Vitest/jsdom,connect-redisv10, Redis client, etc.) and aligns the Vite resolution to match.For Vite 8.2, the Express dev integration in
server/vite.tsno longer imports and spreadsvite.config; it starts Vite withconfigFilepointing atvite.config.ts, which matches the updated Vite server API/types.Test-only hardening for the new toolchain:
layout.test.tsxresolves<main>viagetByRole("main")instead ofquerySelector, andanilistOAuth.test.tsmocksprefetchListSnapshotsso OAuth completion tests stay isolated from background list warm-up.Reviewed by Cursor Bugbot for commit 5ad0bf0. Bugbot is set up for automated code reviews on this repo. Configure here.