chore(tooling): migrate to Vite+ - #19
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b9905ca04
ℹ️ 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".
There was a problem hiding this comment.
All reported issues were addressed across 28 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
8b9905c to
206cdd4
Compare
|
@cubic-dev-ai review this PR after the rebase. |
@caio-pizzol I have started the AI code review. It will take a few minutes to complete. |
Code Review by Qodo
View medium (1)🟠 **Medium** 1. Hook failures are masked
|
Summary
Why
This gives the workspace one tool for the web build, static checks, workspace tasks, and hooks. Bun remains in place where the project depends on its runtime, especially the
bun:testsuites.The formatter ignores existing files that Biome did not manage. This avoids mixing a broad formatting rewrite into the tooling migration.
Verification
bunx --package vite-plus@0.2.9 vp install --frozen-lockfilebun run checkbun run test— 108 passed, 4 cache-dependent smoke tests skippedbun test tests/scripts/db-migrate.test.ts— 5 passedbun run buildbun run --cwd apps/mcp-server buildvp buildvp stagedin a scratch Git repositoryOne existing unused-import warning remains in
scripts/ingest-xsd/ingest.ts.