chore: Add tsgo, run vite fmt. (#454)

This PR:
* Adds a `fmt` command, and runs it on CI so we no longer check in unformatted files.
* Formats a few files that weren't formatted properly before.
* Adds tsgo which is faster at telling us we have 143 errors.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Adds automated formatting and typecheck tooling and wires them into CI.
> 
> - New `fmt` script (`vite fmt`) with config in `vite.config.ts` (ignore patterns, package.json/import sorting); printed in CI help and enforced in CI (`vite fmt --check`, `pnpm fmt` in upgrade workflow)
> - package.json: switch `typecheck` to `tsgo`, add `tsgo` script; add `@typescript/native-preview` to dev deps
> - Workflows: run formatter checks in CLI E2E; upgrade-deps now formats code after syncing and dedupe
> - pnpm workspace/catalog/lockfile updated to include TypeScript native preview artifacts
> - Minor code formatting/ordering cleanups across CLI/global files (no functional changes)
> 
> <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 56bf307. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: cpojer

.github/workflows/ci.yml

@@ -194,6 +194,10 @@ jobs:
           vite --version
           vite -h
 
+      - name: Run CLI fmt
+        if: ${{ matrix.os != 'windows-latest' }}
+        run: vite fmt --check
+      
       - name: Run CLI lint
         run: vite run lint
 

.github/workflows/upgrade-deps.yml

@@ -57,6 +57,9 @@ jobs:
         run: |
           pnpm install --no-frozen-lockfile
           pnpm dedupe
+      
+      - name: Format code
+        run: pnpm fmt
 
       - name: Close and delete previous PR
         env:

package.json

@@ -12,9 +12,11 @@
     "bootstrap-cli": "pnpm build && pnpm install-global-cli",
     "bootstrap-cli:ci": "pnpm install-global-cli",
     "install-global-cli": "pnpm --filter=vite-plus-cli copy-binding && npm install -g ./packages/global --force",
-    "typecheck": "tsc -b tsconfig.json",
+    "typecheck": "tsgo -b tsconfig.json",
+    "tsgo": "tsgo -b tsconfig.json",
     "lint": "vite lint --type-aware --threads 4",
     "test": "vite test run && pnpm -r snap-test",
+    "fmt": "vite fmt",
     "test:unit": "vite test run",
     "docs:dev": "pnpm --filter=./docs dev",
     "docs:build": "pnpm --filter=./docs build",
@@ -24,7 +26,7 @@
     "@oxc-node/cli": "catalog:",
     "@oxc-node/core": "catalog:",
     "@types/node": "catalog:",
-    "vite-plus": "workspace:*",
+    "@typescript/native-preview": "catalog:",
     "@voidzero-dev/vite-plus-tools": "workspace:*",
     "husky": "catalog:",
     "lint-staged": "catalog:",
@@ -33,6 +35,7 @@
     "playwright": "catalog:",
     "typescript": "catalog:",
     "vite": "catalog:",
+    "vite-plus": "workspace:*",
     "vitest": "catalog:"
   },
   "lint-staged": {

packages/cli/build.ts

@@ -38,7 +38,9 @@ const CORE_PACKAGE_NAME = '@voidzero-dev/vite-plus-core';
 const skipNative = process.argv.includes('--skip-native');
 const skipTs = process.argv.includes('--skip-ts');
 // Filter out custom flags before passing to NAPI CLI
-const napiArgs = process.argv.slice(2).filter((arg) => arg !== '--skip-native' && arg !== '--skip-ts');
+const napiArgs = process.argv
+  .slice(2)
+  .filter((arg) => arg !== '--skip-native' && arg !== '--skip-ts');
 
 // Build native first - TypeScript may depend on the generated binding types
 if (!skipNative) {

packages/global/src/migration/bin.ts

@@ -17,13 +17,13 @@ import {
   selectAgentTargetPath,
   writeAgentInstructions,
 } from '../utils/index.js';
+import { getVitePlusHeader } from '../utils/terminal.js';
 import {
   checkVitestVersion,
   checkViteVersion,
   rewriteMonorepo,
   rewriteStandaloneProject,
 } from './migrator.js';
-import { getVitePlusHeader } from '../utils/terminal.js';
 
 const { green, gray } = colors;
 
@@ -103,8 +103,7 @@ async function main() {
 
   const workspaceInfoOptional = await detectWorkspace(projectPath);
   const packageManager =
-    workspaceInfoOptional.packageManager ??
-    (await selectPackageManager(options.interactive));
+    workspaceInfoOptional.packageManager ?? (await selectPackageManager(options.interactive));
 
   // ensure the package manager is installed by vite-plus
   const downloadResult = await downloadPackageManager(

packages/global/src/new/bin.ts

@@ -22,21 +22,17 @@ import {
   detectExistingAgentTargetPath,
   writeAgentInstructions,
 } from '../utils/index.js';
+import { getVitePlusHeader } from '../utils/terminal.js';
 import type { ExecutionResult } from './command.js';
 import { discoverTemplate, inferParentDir } from './discovery.js';
-import {
-  cancelAndExit,
-  checkProjectDirExists,
-  promptPackageNameAndTargetDir,
-} from './prompts.js';
+import { cancelAndExit, checkProjectDirExists, promptPackageNameAndTargetDir } from './prompts.js';
 import {
   executeBuiltinTemplate,
   executeMonorepoTemplate,
   executeRemoteTemplate,
 } from './templates/index.js';
 import { BuiltinTemplate, TemplateType } from './templates/types.js';
 import { formatTargetDir } from './utils.js';
-import { getVitePlusHeader } from '../utils/terminal.js';
 
 const { blue, cyan, green, gray } = colors;
 
@@ -115,8 +111,7 @@ function parseArgs() {
   const viteArgs = separatorIndex >= 0 ? args.slice(0, separatorIndex) : args;
 
   // Arguments after -- are template options
-  const templateArgs =
-    separatorIndex >= 0 ? args.slice(separatorIndex + 1) : [];
+  const templateArgs = separatorIndex >= 0 ? args.slice(separatorIndex + 1) : [];
 
   const parsed = mri<{
     directory?: string;
@@ -171,9 +166,7 @@ Template name is required when running in non-interactive mode
 Usage: vite new [TEMPLATE] [OPTIONS] [-- TEMPLATE_OPTIONS]
 
 Example: 
-  ${gray(
-    '# Create a new application in non-interactive mode with a custom target directory',
-  )}
+  ${gray('# Create a new application in non-interactive mode with a custom target directory')}
   vite new vite:application --no-interactive --directory=apps/my-app
 
 Use \`vite new --list\` to list all available templates, or run \`vite new --help\` for more information.
@@ -292,10 +285,7 @@ Use \`vite new --list\` to list all available templates, or run \`vite new --hel
 
   const isBuiltinTemplate = selectedTemplateName.startsWith('vite:');
   if (targetDir && !isBuiltinTemplate) {
-    cancelAndExit(
-      'The --directory option is only available for builtin templates',
-      1,
-    );
+    cancelAndExit('The --directory option is only available for builtin templates', 1);
   }
   if (selectedTemplateName === BuiltinTemplate.monorepo && isMonorepo) {
     prompts.log.info(
@@ -371,16 +361,11 @@ Use \`vite new --list\` to list all available templates, or run \`vite new --hel
       packageName = selected.packageName;
       targetDir = selected.targetDir;
     } else {
-      let defaultPackageName = `vite-plus-${
-        selectedTemplateName.split(':')[1]
-      }`;
+      let defaultPackageName = `vite-plus-${selectedTemplateName.split(':')[1]}`;
       if (workspaceInfoOptional.monorepoScope) {
         defaultPackageName = `${workspaceInfoOptional.monorepoScope}/${defaultPackageName}`;
       }
-      const selected = await promptPackageNameAndTargetDir(
-        defaultPackageName,
-        options.interactive,
-      );
+      const selected = await promptPackageNameAndTargetDir(defaultPackageName, options.interactive);
       packageName = selected.packageName;
       targetDir = selectedParentDir
         ? path.join(selectedParentDir, selected.targetDir)
@@ -390,8 +375,7 @@ Use \`vite new --list\` to list all available templates, or run \`vite new --hel
 
   // Prompt for package manager or use default
   const packageManager =
-    workspaceInfoOptional.packageManager ??
-    (await selectPackageManager(options.interactive));
+    workspaceInfoOptional.packageManager ?? (await selectPackageManager(options.interactive));
   // ensure the package manager is installed by vite-plus
   const downloadResult = await downloadPackageManager(
     packageManager,
@@ -439,20 +423,14 @@ Use \`vite new --list\` to list all available templates, or run \`vite new --hel
   // #region Handle monorepo template
   if (templateInfo.command === BuiltinTemplate.monorepo) {
     prompts.log.info(`Target directory: ${cyan(targetDir)}`);
-    await checkProjectDirExists(
-      path.join(workspaceInfo.rootDir, targetDir),
-      options.interactive,
-    );
+    await checkProjectDirExists(path.join(workspaceInfo.rootDir, targetDir), options.interactive);
     const result = await executeMonorepoTemplate(
       workspaceInfo,
       { ...templateInfo, packageName, targetDir },
       options.interactive,
     );
     if (result.exitCode !== 0) {
-      cancelAndExit(
-        `Failed to create monorepo, exit code: ${result.exitCode}`,
-        result.exitCode,
-      );
+      cancelAndExit(`Failed to create monorepo, exit code: ${result.exitCode}`, result.exitCode);
     }
 
     // rewrite monorepo to add vite-plus dependencies
@@ -477,16 +455,11 @@ Use \`vite new --list\` to list all available templates, or run \`vite new --hel
   if (templateInfo.type === TemplateType.builtin) {
     // prompt for package name if not provided
     if (!targetDir) {
-      let defaultPackageName = `vite-plus-${
-        templateInfo.command.split(':')[1]
-      }`;
+      let defaultPackageName = `vite-plus-${templateInfo.command.split(':')[1]}`;
       if (workspaceInfo.monorepoScope) {
         defaultPackageName = `${workspaceInfo.monorepoScope}/${defaultPackageName}`;
       }
-      const selected = await promptPackageNameAndTargetDir(
-        defaultPackageName,
-        options.interactive,
-      );
+      const selected = await promptPackageNameAndTargetDir(defaultPackageName, options.interactive);
       packageName = selected.packageName;
       targetDir = templateInfo.parentDir
         ? path.join(templateInfo.parentDir, selected.targetDir)
@@ -617,48 +590,23 @@ function showAvailableTemplates() {
   console.log('');
 
   console.log(blue('Vite+ Built-in Templates:'));
-  console.log(
-    '  • vite:monorepo                 ' + gray('- Create a new monorepo'),
-  );
-  console.log(
-    '  • vite:application              ' + gray('- Create a new application'),
-  );
-  console.log(
-    '  • vite:library                  ' + gray('- Create a new library'),
-  );
-  console.log(
-    '  • vite:generator                ' +
-      gray('- Scaffold a new code generator'),
-  );
+  console.log('  • vite:monorepo                 ' + gray('- Create a new monorepo'));
+  console.log('  • vite:application              ' + gray('- Create a new application'));
+  console.log('  • vite:library                  ' + gray('- Create a new library'));
+  console.log('  • vite:generator                ' + gray('- Scaffold a new code generator'));
   console.log('');
 
   console.log(green('Popular Remote Templates:'));
-  console.log(
-    '  • create-vite                   ' + gray('- Official Vite templates'),
-  );
-  console.log(
-    '  • @tanstack/create-start        ' + gray('- TanStack applications'),
-  );
-  console.log(
-    '  • create-next-app               ' + gray('- Next.js application'),
-  );
-  console.log(
-    '  • create-nuxt                   ' + gray('- Nuxt application'),
-  );
-  console.log(
-    '  • create-typescript-app         ' + gray('- TypeScript application'),
-  );
-  console.log(
-    '  • create-react-router           ' + gray('- React Router application'),
-  );
+  console.log('  • create-vite                   ' + gray('- Official Vite templates'));
+  console.log('  • @tanstack/create-start        ' + gray('- TanStack applications'));
+  console.log('  • create-next-app               ' + gray('- Next.js application'));
+  console.log('  • create-nuxt                   ' + gray('- Nuxt application'));
+  console.log('  • create-typescript-app         ' + gray('- TypeScript application'));
+  console.log('  • create-react-router           ' + gray('- React Router application'));
   console.log('  • create-vue                    ' + gray('- Vue application'));
 
-  console.log(
-    '\n' + gray('Run ') + cyan('vite new') + gray(' for interactive mode'),
-  );
-  console.log(
-    gray('Run ') + cyan('vite new <template>') + gray(' to use any template'),
-  );
+  console.log('\n' + gray('Run ') + cyan('vite new') + gray(' for interactive mode'));
+  console.log(gray('Run ') + cyan('vite new <template>') + gray(' to use any template'));
   console.log(
     gray('Run ') +
       cyan('vite new <template> -- <options>') +

packages/global/src/utils/terminal.ts

@@ -9,16 +9,14 @@ const RESET = `${CSI}0m`;
 const fgRgb = (r: number, g: number, b: number) => `${CSI}38;2;${r};${g};${b}m`;
 
 const shouldColorize = (stream = process.stdout) =>
-  stream?.isTTY &&
-  (typeof stream.hasColors === 'function' ? stream.hasColors() : true);
+  stream?.isTTY && (typeof stream.hasColors === 'function' ? stream.hasColors() : true);
 
 function supportsTrueColor(stream = process.stdout) {
   if (!stream?.isTTY) {
     return false;
   }
 
-  const depth =
-    typeof stream.getColorDepth === 'function' ? stream.getColorDepth() : 1;
+  const depth = typeof stream.getColorDepth === 'function' ? stream.getColorDepth() : 1;
   return depth >= 24;
 }
 
@@ -192,16 +190,11 @@ export async function getVitePlusHeader() {
 
   if (!colors) {
     const fg = await getForegroundColor();
-    colors = fg
-      ? gradient(text.length, fg, purple)
-      : fadeToColor(text.length, purple);
+    colors = fg ? gradient(text.length, fg, purple) : fadeToColor(text.length, purple);
   }
 
   return `${styleText(
     'bold',
-    `VITE+(${styleText(
-      'blueBright',
-      '⚡︎',
-    )}) - The Unified Toolchain ${colorize(text, colors)}`,
+    `VITE+(${styleText('blueBright', '⚡︎')}) - The Unified Toolchain ${colorize(text, colors)}`,
   )}`;
 }

packages/global/src/version.ts

@@ -52,10 +52,7 @@ function readPackageVersionFromPath(packageJsonPath: string): string | null {
   return readPackageJsonFromPath(packageJsonPath)?.version ?? null;
 }
 
-function resolvePackageJson(
-  packageName: string,
-  baseDir: string,
-): PackageJson | null {
+function resolvePackageJson(packageName: string, baseDir: string): PackageJson | null {
   try {
     const packageJsonPath = require.resolve(`${packageName}/package.json`, {
       paths: [baseDir],
@@ -66,13 +63,10 @@ function resolvePackageJson(
   }
 }
 
-function resolveToolVersion(
-  tool: ToolVersionSpec,
-  localPackagePath: string,
-): string | null {
+function resolveToolVersion(tool: ToolVersionSpec, localPackagePath: string): string | null {
   const pkg = resolvePackageJson(tool.packageName, localPackagePath);
   const bundledVersion = tool.bundledVersionKey
-    ? pkg?.bundledVersions?.[tool.bundledVersionKey] ?? null
+    ? (pkg?.bundledVersions?.[tool.bundledVersionKey] ?? null)
     : null;
   if (bundledVersion) {
     return bundledVersion;
@@ -86,18 +80,14 @@ function resolveToolVersion(
   return null;
 }
 
-function formatToolVersion(
-  tool: ToolVersionSpec,
-  version: string | null,
-): string {
+function formatToolVersion(tool: ToolVersionSpec, version: string | null): string {
   return `${tool.displayName} ${version ? `v${version}` : `Not found`}`;
 }
 
 const cliLabel = 'vite-plus-cli';
 const localLabel = 'vite-plus';
 const columnWidth = cliLabel.length + 1;
-const getColumnWidth = (label: string) =>
-  Math.max(0, columnWidth - label.length);
+const getColumnWidth = (label: string) => Math.max(0, columnWidth - label.length);
 
 /**
  * Print version information for both local and global CLI
@@ -172,11 +162,7 @@ export async function printVersion(cwd: string) {
     tool,
     version: resolveToolVersion(tool, localMetadata.path),
   }));
-  if (
-    resolvedTools.some(
-      ({ tool, version }) => tool.bundledVersionKey && !version,
-    )
-  ) {
+  if (resolvedTools.some(({ tool, version }) => tool.bundledVersionKey && !version)) {
     return;
   }
 

packages/test/build.ts

@@ -1192,7 +1192,9 @@ async function patchVitestCoreResolver() {
 
   if (!content.includes(oldPattern)) {
     throw new Error(
-      'Could not find VitestCoreResolver pattern to patch in ' + cliApiChunk + '. ' +
+      'Could not find VitestCoreResolver pattern to patch in ' +
+        cliApiChunk +
+        '. ' +
         'This likely means vitest code has changed and the patch needs to be updated.',
     );
   }