Uh oh!
There was an error while loading. Please reload this page.
feat: migrate toolchain to Vite+ - #2
Open
fengmk2 wants to merge 3 commits into
Open
Conversation
Replace split vite/vitest/oxlint/oxfmt tooling with vite-plus 0.2.7 via vp migrate. - Bump apps/web to Vite 8.2.0, resolved through the new pnpm catalog to the Rolldown-powered @voidzero-dev/vite-plus-core build - Consolidate .oxlintrc.json into the new root vite.config.ts - Rewrite vitest imports to vite-plus/test in 56 files, switch package scripts to vp lint / vp fmt / vp test run - Reformat 319 files with the bundled Oxfmt 0.62, which now also formats Markdown - Keep husky + gitleaks pre-commit hooks; vp hook setup skipped - Leave type-aware linting off in the root config: it reports 88 pre-existing findings and its suggested rootDir fix would break the apps/api build, which compiles workspace packages via tsconfig paths
Install hooks via vp config (.vite-hooks dispatcher). The gitleaks fail-closed scans move unchanged into .vite-hooks/pre-commit and .vite-hooks/pre-push, and pre-commit additionally runs vp staged (vp check --fix on staged files).
fengmk2
marked this pull request as ready for review
August 5, 2026 08:35
thonatos
reviewed
Aug 5, 2026
thonatos
left a comment
Member
There was a problem hiding this comment.
Review: 工具链迁移本身合理,注意两点
实质改动:husky→.vite-hooks(vp config 接管)、oxlint/oxfmt→vp lint/fmt、vitest→vp test、lint 配置并入 vite.config.ts、vite/vitest catalog + peerDependencyRules、pre-commit 保留 gitleaks 暂存扫描。方向正确。
注意
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Migrates the monorepo from split tooling (Vite 7, Vitest, Oxlint, Oxfmt) to Vite+ 0.2.7 via
vp migrate.@voidzero-dev/vite-plus-corebuild, Vitest pinned at 4.1.10.oxlintrc.jsoninto the new rootvite.config.ts, now the single config for lint/fmt/testvitestimports tovite-plus/testin 56 files, switch package scripts tovp lint/vp fmt/vp test runopenspec/andwiki/docs)Type-aware linting stays off in the root config for now: it reports 88 pre-existing findings (58
no-floating-promises, plusunbound-method,no-base-to-string, one real type error inscripts/migrate-mongo-to-pg.ts) and its suggested tsconfigrootDirfix would break the apps/api build, which compiles workspace packages via tsconfig paths. Planned as a follow-up.Verified:
vp checkclean (750 files formatted, 268 lint-clean), 232 tests pass, all builds pass, typecheck clean,openspec validate --all --strict62/62.