refactor: update vercel and migrate to vitest#68
Conversation
…pendencies - Changed the test command in package.json from "jest" to "vitest". - Updated @vercel/node dependency version to 5.6.16. - Updated typescript version to 5.9.3. - Updated devDependencies: added vitest and updated @types/node, tsx, and vercel versions. - Modified vite.config.ts to support Vitest configuration and adjusted the root directory based on the environment.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
💡 Codex Reviewhttps://github.com/sargant/dfyb.run/blob/f2f592724ef74f7f7faddf9ef7a9e249d2009ac8/package-lock.json#L19675-L19678 Adding Vitest 4 also adds a nested Line 50 in f2f5927  Align the Vitest upgrade with the app's Vite major
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
sargant
changed the title
- removed @types/jest version 27.0.3 - updated vite from version ^2.7.0 to version 7.3.1
sargant
force-pushed
the
modernization-2026
branch
from
088d580 to
9326785
Compare
fix: change output file extension from .js to .ts in encrypt-certs.ts refactor: update tsconfig.json for module resolution and include paths style: update gray color to neutral in windi.config.ts feat: add TypeScript declaration file for WindiCSS in windicss-fix.d.ts
|
@codex re-run the review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8eb3d4853d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| import { join } from 'path' | ||
|
|
||
| import * as certs from './certs.enc.js' | ||
| import * as certs from './certs.enc' |
There was a problem hiding this comment.
Any non-bundled execution path for this API module (for example vercel dev or direct Node execution) now fails before the handler runs, because Node’s ESM resolver does not load ./certs.enc without an explicit extension. I verified that in this repo with node --experimental-transform-types, which raises ERR_MODULE_NOT_FOUND for this specifier. The new moduleResolution: "bundler" setting masks the problem during type-checking, so tsc can still pass while the runtime cannot import the certificates module.
Useful? React with 👍 / 👎.
1 participant
Updating and modernising some dependencies