build: split vite plus core and test

Author: Brooooooklyn

.github/actions/build-upstream/action.yml

@@ -21,6 +21,8 @@ runs:
         fi
         pnpm --filter rolldown build-node
         pnpm --filter rolldown-vite build-types
+        pnpm --filter=@voidzero-dev/vite-plus-core build
+        pnpm --filter=@voidzero-dev/vite-plus-test build
         CC=clang pnpm --filter=@voidzero-dev/vite-plus build --target ${{ inputs.target }} --use-napi-cross
       env:
         TARGET_CC: clang
@@ -35,6 +37,8 @@ runs:
         fi
         pnpm --filter rolldown build-node
         pnpm --filter rolldown-vite build-types
+        pnpm --filter=@voidzero-dev/vite-plus-core build
+        pnpm --filter=@voidzero-dev/vite-plus-test build
         TARGET_CFLAGS="-D_BSD_SOURCE" pnpm --filter=@voidzero-dev/vite-plus build --target ${{ inputs.target }} --use-napi-cross
       env:
         TARGET_CC: clang
@@ -49,4 +53,6 @@ runs:
         fi
         pnpm --filter rolldown build-node
         pnpm --filter rolldown-vite build-types
+        pnpm --filter=@voidzero-dev/vite-plus-core build
+        pnpm --filter=@voidzero-dev/vite-plus-test build
         pnpm --filter=@voidzero-dev/vite-plus build --target ${{ inputs.target }}

.oxfmtrc.json

@@ -0,0 +1,8 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "ignorePatterns": [
+    "packages/cli/binding/index.d.ts",
+    "packages/cli/binding/index.js"
+  ],
+  "singleQuote": true
+}

bench/generate-monorepo.ts

@@ -147,7 +147,8 @@ class MonorepoGenerator {
       const hasVitePlusConfig = Math.random() > 0.3;
 
       // Select dependencies from packages created before this one
-      const dependencies = i === 0 ? [] : this.selectDependencies(i, allPackageNames.slice(0, i));
+      const dependencies =
+        i === 0 ? [] : this.selectDependencies(i, allPackageNames.slice(0, i));
 
       this.packages.set(packageName, {
         name: packageName,
@@ -181,9 +182,15 @@ class MonorepoGenerator {
 
         // Create the scenario: A has build, B doesn't, C has build
         const scriptName = this.getRandomElement(this.SCRIPT_NAMES);
-        pkgA.scripts[scriptName] = this.generateScriptCommand(scriptName, nameA);
+        pkgA.scripts[scriptName] = this.generateScriptCommand(
+          scriptName,
+          nameA,
+        );
         delete pkgB.scripts[scriptName]; // B doesn't have the script
-        pkgC.scripts[scriptName] = this.generateScriptCommand(scriptName, nameC);
+        pkgC.scripts[scriptName] = this.generateScriptCommand(
+          scriptName,
+          nameC,
+        );
       }
     }
   }
@@ -201,10 +208,13 @@ class MonorepoGenerator {
       version: '1.0.0',
       main: 'src/index.js',
       scripts: pkg.scripts,
-      dependencies: pkg.dependencies.reduce((deps, dep) => {
-        deps[`@monorepo/${dep}`] = 'workspace:*';
-        return deps;
-      }, {} as Record<string, string>),
+      dependencies: pkg.dependencies.reduce(
+        (deps, dep) => {
+          deps[`@monorepo/${dep}`] = 'workspace:*';
+          return deps;
+        },
+        {} as Record<string, string>,
+      ),
     };
 
     fs.writeFileSync(
@@ -352,11 +362,9 @@ module.exports = { ${pkg.name.replace('-', '_')} };
     console.log('\nStatistics:');
     console.log(`- Total packages: ${this.packages.size}`);
     console.log(
-      `- Average dependencies per package: ${
-        (
-          totalDeps / this.packages.size
-        ).toFixed(2)
-      }`,
+      `- Average dependencies per package: ${(
+        totalDeps / this.packages.size
+      ).toFixed(2)}`,
     );
     console.log(`- Max dependencies in a package: ${maxDeps}`);
     console.log(`- Packages with vite-plus.json: ${packagesWithVitePlus}`);

docs/.vitepress/config.mts

@@ -6,9 +6,7 @@ export default defineConfig({
   description: 'Vite+',
   themeConfig: {
     // https://vitepress.dev/reference/default-theme-config
-    nav: [
-      { text: 'Home', link: '/' },
-    ],
+    nav: [{ text: 'Home', link: '/' }],
 
     sidebar: [],
 

dprint.json

@@ -13,6 +13,10 @@
     "quotes": "preferSingle"
   },
   "excludes": [
+    "**/*.ts",
+    "**/*.tsx",
+    "**/*.js",
+    "**/*.jsx",
     "crates/fspy_detours_sys/detours",
     "pnpm-lock.yaml",
     "packages/cli/binding/index.d.ts",

package.json

@@ -13,7 +13,7 @@
     "install-global-cli": "npm install -g ./packages/global",
     "typecheck": "tsc -b tsconfig.json",
     "lint": "vite lint --type-aware --threads 4",
-    "test": "vite test run && pnpm --filter=@voidzero-dev/vite-plus test && pnpm -r snap-test",
+    "test": "vite test run && pnpm -r snap-test",
     "prepare": "husky"
   },
   "devDependencies": {
@@ -27,14 +27,15 @@
     "oxfmt": "catalog:",
     "oxlint": "catalog:",
     "typescript": "catalog:",
-    "vitest": "catalog:"
+    "vitest": "workspace:@voidzero-dev/vite-plus-test@*"
   },
   "lint-staged": {
-    "*.@(js|ts|tsx|yml|yaml|md|json|html|toml)": [
+    "*.@(yml|yaml|md|json|html|toml)": [
       "dprint fmt --staged"
     ],
     "*.@(js|ts|tsx)": [
-      "oxlint -- --fix"
+      "oxlint -- --fix",
+      "oxfmt"
     ],
     "*.rs": [
       "cargo fmt --"

packages/cli/binding/__tests__/run-command.spec.ts

@@ -18,21 +18,27 @@ test('should run command successfully', async () => {
 
 // write file on the temp directory
 test('should write file on the temp directory', async () => {
-  const tempDir = await fs.realpath(await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')));
+  const tempDir = await fs.realpath(
+    await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')),
+  );
   const result = await runCommand({
     binName: 'node',
     args: ['-e', `fs.writeFileSync("test.txt", "Hello, world!")`],
     envs: {},
     cwd: tempDir,
   });
   expect(result).toMatchSnapshot();
-  expect(await fs.readFile(path.join(tempDir, 'test.txt'), 'utf-8')).toBe('Hello, world!');
+  expect(await fs.readFile(path.join(tempDir, 'test.txt'), 'utf-8')).toBe(
+    'Hello, world!',
+  );
   await fs.rm(tempDir, { recursive: true });
 });
 
 // read file on the temp directory
 test('should read file on the temp directory', async () => {
-  const tempDir = await fs.realpath(await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')));
+  const tempDir = await fs.realpath(
+    await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')),
+  );
   await fs.writeFile(path.join(tempDir, 'test.txt'), 'Hello, world!');
   const result = await runCommand({
     binName: 'node',
@@ -46,10 +52,15 @@ test('should read file on the temp directory', async () => {
 
 // write and read file on the temp directory
 test('should write and read file on the temp directory', async () => {
-  const tempDir = await fs.realpath(await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')));
+  const tempDir = await fs.realpath(
+    await fs.mkdtemp(path.join(tmpdir(), 'vite-plus-test-')),
+  );
   const result = await runCommand({
     binName: 'node',
-    args: ['-e', `fs.writeFileSync("test.txt", "Hello, world!"); fs.readFileSync("test.txt", "utf-8")`],
+    args: [
+      '-e',
+      `fs.writeFileSync("test.txt", "Hello, world!"); fs.readFileSync("test.txt", "utf-8")`,
+    ],
     envs: {},
     cwd: tempDir,
   });

packages/cli/binding/index.d.ts

@@ -11,21 +11,24 @@
  */
 export interface CliOptions {
   /** Resolver function for the lint tool (oxlint) */
-  lint: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  lint: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Resolver function for the fmt tool (oxfmt) */
-  fmt: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  fmt: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Resolver function for the vite tool (used for build/dev) */
-  vite: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  vite: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Resolver function for the test tool (vitest) */
-  test: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  test: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Resolver function for the lib tool (tsdown) */
-  lib: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  lib: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Resolver function for the doc tool (vitepress) */
-  doc: ((err: Error | null, ) => Promise<JsCommandResolvedResult>)
+  doc: (err: Error | null) => Promise<JsCommandResolvedResult>;
   /** Optional working directory override */
-  cwd?: string
+  cwd?: string;
   /** Read the vite.config.ts in the Node.js side and return the `lint` and `fmt` config JSON string back to the Rust side */
-  resolveUniversalViteConfig: ((err: Error | null, arg: string) => Promise<string>)
+  resolveUniversalViteConfig: (
+    err: Error | null,
+    arg: string,
+  ) => Promise<string>;
 }
 
 /**
@@ -53,13 +56,15 @@ export interface CliOptions {
  * console.log(`Workspace root: ${result.root}`);
  * ```
  */
-export declare function detectWorkspace(cwd: string): Promise<DetectWorkspaceResult>
+export declare function detectWorkspace(
+  cwd: string,
+): Promise<DetectWorkspaceResult>;
 
 export interface DetectWorkspaceResult {
-  packageManagerName?: string
-  packageManagerVersion?: string
-  isMonorepo: boolean
-  root?: string
+  packageManagerName?: string;
+  packageManagerVersion?: string;
+  isMonorepo: boolean;
+  root?: string;
 }
 
 /**
@@ -95,20 +100,22 @@ export interface DetectWorkspaceResult {
  * console.log(`Package manager version: ${result.version}`);
  * ```
  */
-export declare function downloadPackageManager(options: DownloadPackageManagerOptions): Promise<DownloadPackageManagerResult>
+export declare function downloadPackageManager(
+  options: DownloadPackageManagerOptions,
+): Promise<DownloadPackageManagerResult>;
 
 export interface DownloadPackageManagerOptions {
-  name: string
-  version: string
-  expectedHash?: string
+  name: string;
+  version: string;
+  expectedHash?: string;
 }
 
 export interface DownloadPackageManagerResult {
-  name: string
-  installDir: string
-  binPrefix: string
-  packageName: string
-  version: string
+  name: string;
+  installDir: string;
+  binPrefix: string;
+  packageName: string;
+  version: string;
 }
 
 /**
@@ -120,19 +127,19 @@ export interface DownloadPackageManagerResult {
  */
 export interface JsCommandResolvedResult {
   /** Absolute path to the tool's executable or script */
-  binPath: string
+  binPath: string;
   /** Environment variables to set when running the tool */
-  envs: Record<string, string>
+  envs: Record<string, string>;
 }
 
 /** Access modes for a path. */
 export interface PathAccess {
   /** Whether the path was read */
-  read: boolean
+  read: boolean;
   /** Whether the path was written */
-  write: boolean
+  write: boolean;
   /** Whether the path was read as a directory */
-  readDir: boolean
+  readDir: boolean;
 }
 
 /**
@@ -156,7 +163,7 @@ export interface PathAccess {
  * Errors from JavaScript resolvers are converted to specific error types
  * (e.g., `LintFailed`, `ViteError`) to provide better error messages.
  */
-export declare function run(options: CliOptions): Promise<number>
+export declare function run(options: CliOptions): Promise<number>;
 
 /**
  * Run a command with fspy tracking, callable from JavaScript.
@@ -192,7 +199,9 @@ export declare function run(options: CliOptions): Promise<number>
  * console.log(`Path accesses:`, result.pathAccesses);
  * ```
  */
-export declare function runCommand(options: RunCommandOptions): Promise<RunCommandResult>
+export declare function runCommand(
+  options: RunCommandOptions,
+): Promise<RunCommandResult>;
 
 /**
  * Input parameters for running a command with fspy tracking.
@@ -205,13 +214,13 @@ export declare function runCommand(options: RunCommandOptions): Promise<RunComma
  */
 export interface RunCommandOptions {
   /** The name of the binary to execute */
-  binName: string
+  binName: string;
   /** Command line arguments to pass to the binary */
-  args: Array<string>
+  args: Array<string>;
   /** Environment variables to set when executing the command */
-  envs: Record<string, string>
+  envs: Record<string, string>;
   /** The current working directory for the command */
-  cwd: string
+  cwd: string;
 }
 
 /**
@@ -223,7 +232,7 @@ export interface RunCommandOptions {
  */
 export interface RunCommandResult {
   /** The exit code of the command */
-  exitCode: number
+  exitCode: number;
   /** Map of relative paths to their access modes */
-  pathAccesses: Record<string, PathAccess>
+  pathAccesses: Record<string, PathAccess>;
 }