Repository navigation
Conversation
Formatting-only change, no code changes, so it can be ignored in git blame via .git-blame-ignore-revs. Uses the fmt settings added to vite.config.ts in the following Vite+ migration commit (4-space indent, no semicolons, single quotes, 160 char print width). Co-Authored-By: Claude Opus 5.5 <[email protected]>
Replace Vite 7 + ESLint + tsc with Vite+ (vite-plus 1.4.0): Vite 8 (Rolldown), Vitest 5, Oxlint (type-aware, via tsgolint) and Oxfmt, configured in one vite.config.ts. npm scripts stay the entry points for the .NET build/dev integration, so no global `vp` install is required. - vite-plus pinned with npm overrides for vite -> @voidzero-dev/vite-plus-core - lint rules ported from the ESLint config (which failed to load on main); `npm run check` replaces `tsc -b` in the build - remove ESLint config + packages, duplicate PostCSS config, and postinstall.mjs (empty DBs are migrated on app startup) - add test scripts and seed tests (utils, gateway, HelloApi) - fix rules-of-hooks findings in lib/auth.tsx without changing behaviour or the shared appAuth() API - upgrade deps to latest, incl. TypeScript 7 and @vitejs/plugin-react 6 - commit package-lock.json Co-Authored-By: Claude Opus 5.5 <[email protected]>
- build.yml: new client job (vp check, vp test run, vp 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 <[email protected]>
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 the React client from Vite 7 + ESLint +
tscto Vite+, VoidZero's unified toolchain:tsc -b && vite buildvp dev/vp build(Vite 8, Rolldown)tsc -bvp check(Oxlint type-aware, TypeScript Go)main)vp lint(Oxlint)vp fmt(Oxfmt)vp test(Vitest 5), 5 seed testsAll tool config (
lint,fmt,test) is in onevite.config.ts. The npm scripts are still the entry points, so the .NET integration (npm run buildin the publish target,RunNodeProcess, the.esproj) is unchanged, and no globalvpinstall is needed.Commits
.git-blame-ignore-revs.vite-plus1.4.0 with npmoverridespostinstall.mjsremoved (empty DBs are already migrated on app startup)package-lock.jsoncommittedclientjob inbuild.yml;setup-vpreplacessetup-nodeinbuild-container.ymldtos.tsis regenerated on .NET startupMyApp.code-workspaceaddedWorth reviewing
lib/auth.tsxgot small fixes that don't change behaviour: the HOC's inner component is renamed to PascalCase, andshouldRedirectis now declared before theuseEffectthat uses it.appAuth()is a hook without auseprefix and is shared by 7 templates, so it keeps its name and the rule is suppressed per line. A rename across all templates is a separate change.no-floating-promises,react/purity) are set to warnings.vp checkpasses with 12 warnings.main's Vite 7 setup also produces 583 kB, because newer dependency versions resolve when there's no lockfile.MyApp.code-workspacerather than the folder. The Oxc extension only resolvesvite-plusfrom a workspace root'snode_modules.Verified locally
npm ci, thennpm run check,npm run test:runandnpm run buildall pass. Builds take about 190 ms, down from about 915 ms.vp check.https://localhost:5001:/api/Hellorespondsdotnet publishrunsvp buildand copies the output towwwroot.postinstall.mjs.🤖 Generated with Claude Code