Repository navigation
Conversation
Next.js keeps building the app (next dev/build/start unchanged). Vite+
(vite-plus 1.4.0) takes over the rest of the tooling: Vitest 5, Oxlint
(type-aware, via tsgolint) and Oxfmt, configured in one vite.config.ts.
npm scripts stay the entry points, so no global `vp` install is required.
- vite-plus pinned with npm overrides for vite -> @voidzero-dev/vite-plus-core
- vitest.config.ts merged into vite.config.ts; tests import from vite-plus/test
- new check/lint/fmt scripts; typecheck now runs `vp check --no-fmt --no-lint`
- lint: eslint:recommended + typescript-eslint recommended equivalents, plus
react hooks and nextjs rules (the template had no ESLint before)
- fix lint findings: unused imports/vars, `extends {}`, ValidateAuth hook
naming/order in lib/auth.tsx; suppress useMetadata and appAuth() false
positives without changing the shared appAuth() API
- remove postinstall.mjs (empty DBs are migrated on app startup), autoprefixer
- drop tsconfig baseUrl (removed in TypeScript Go)
- add seed tests (gateway, HelloApi component)
- upgrade deps, incl. TypeScript 7, @vitejs/plugin-react 6, Next.js 16.4
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Formatting-only change, no code changes, so it can be ignored in git blame via .git-blame-ignore-revs. Uses the fmt settings in vite.config.ts from the previous Vite+ migration commit (4-space indent, no semicolons, single quotes, 160 char print width). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- build.yml: new client job (vp check, vp test run, npm run build) - build-container.yml: voidzero-dev/setup-vp replaces setup-node + npm install - AGENTS.md / README.md: Vite+ commands; dtos.ts is regenerated on .NET startup, so `npm run dtos` is typically never needed - MyApp.code-workspace: adds MyApp.Client as its own root so the Oxc extension resolves the local vite-plus; TypeScript 7 language server - .git-blame-ignore-revs: ignore the Oxfmt formatting commit Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Generated by vp migrate (VITE PLUS START/END markers), followed by the block next dev maintains (BEGIN/END nextjs-agent-rules). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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 free
to 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.
Moves
MyApp.Clienttooling to Vite+ (vite-plus1.4.0). Next.js still builds the app (next dev/next buildunchanged), and Vite+ provides tests, lint, format and type checking from onevite.config.ts.Changes
vite-pluspinned with npmoverrides(vite→@voidzero-dev/vite-plus-core).vitest.config.tsis merged intovite.config.ts, and tests import fromvite-plus/test.check,lint,fmt,typecheck(vp check --no-fmt --no-lint),test,test:run,test:ui, plusoutdatedandupgrade.upgraderunsnpm-check-updateswith the Vite+ pins excluded.extends {}, andValidateAuthhook naming/order. TheuseMetadataandappAuth()warnings are suppressed per line.postinstall.mjs(empty databases are now migrated on app startup),autoprefixer, andbaseUrlfromtsconfig.json.@vitejs/plugin-react6, Next.js 16.4 andlucide-react1.x.clientjob inbuild.ymlrunsvp check,vp test runandnpm run build.build-container.ymlusesvoidzero-dev/setup-vp.AGENTS.mdandREADME.mdupdated. They now saydtos.tsis regenerated on .NET startup. AddedMyApp.Client/AGENTS.md(Vite+ guidance, plus the blocknext devmaintains) andMyApp.code-workspacefor the Oxc VS Code extension.7f5b8fa) and is listed in.git-blame-ignore-revs.Verification
npm run check: 0 errors (16 warnings, kept non-blocking)npm run test:run: 8 passednpm run build: static export todist/. Next's own type check works with TypeScript 7 and catches a deliberately planted type error.vp checkpasses on a fresh tree with nodist/or.next/dotnet runwith an emptyApp_Data: "Empty database detected" bootstrap ran,/api/Helloresponded, and the home page loaded through the Next dev proxyPlease merge with a merge commit, not squash or rebase, so the formatting commit SHA in
.git-blame-ignore-revsstays valid.🤖 Generated with Claude Code