build(test): bundle deps (#333)

<!-- CURSOR_SUMMARY -->
> [!NOTE]
> Bundles vitest and @vitest deps into `@voidzero-dev/vite-plus-test` with browser/Node separation, rewrites imports/exports, adds plugin shims, and updates ecosystem CI to use bundled packages.
> 
> - **@voidzero-dev/vite-plus-test**:
>   - **Build pipeline**: Copy `vitest-dev` and `@vitest/*` into `dist/`; parse and bundle only leaf deps to `dist/vendor/`; rewrite imports (map `vite` → `@voidzero-dev/vite-plus-core`, `@vitest/*`, `vitest/*`); create browser-safe `module-runner` stub; fix path resolutions; patch `@vitest/browser` and `@vitest/browser-playwright` for browser-safe usage; add vendor stubs and validate externals.
>   - **Exports**: Add Node-specific entry `dist/index-node.js`, `./browser-compat`, `./context`, `./client`, `./locators`, `./browser-playwright`, and `./plugins/*` shims mapping `@vitest/*` to `dist/plugins/*`.
>   - **Dependencies**: Move bundled deps to devDeps; add runtime deps (`sirv`, `ws`, `pixelmatch`, `pngjs`); keep `@voidzero-dev/vite-plus-core` as dep; drop `@vitest/browser-playwright` from peers.
> - **Ecosystem CI**:
>   - **vibe-dashboard**: Point workspace overrides to local tgz; remove `@vitest/*` devDeps from `apps/dashboard/package.json`; switch imports to `vitest/browser-playwright`; add `@vitest/*` version overrides in `pnpm-workspace.yaml`.
>   - **skeleton**: Change test script to `vite test`; add overrides for `vitest`, `@vitest/browser`, and `@vitest/browser-playwright`; relax Node engine; update `vite.config.ts` imports to `vitest/browser-playwright`; remove `@vitest/browser-playwright` from package devDeps; patch workspace versions and append policy rules.
> 
> <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit de87af8. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: Brooooooklyn

ecosystem-ci/patch-project.ts

@@ -29,11 +29,68 @@ async function patchVibeDashboard() {
     .replace(
       '"vitest": "npm:@voidzero-dev/vite-plus-test"',
       `"vitest": "file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz"
+  "@vitest/browser": "file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz"
+  "@vitest/browser-playwright": "file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz"
   "@voidzero-dev/vite-plus": "file:${tgzPath}/voidzero-dev-vite-plus-0.0.0.tgz"
   "@voidzero-dev/vite-plus-core": "file:${tgzPath}/voidzero-dev-vite-plus-core-0.0.0.tgz"
   "@voidzero-dev/vite-plus-test": "file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz"`,
     );
   fs.writeFileSync(pnpmWorkspacePath, pnpmWorkspaceFile);
+
+  // Remove @vitest/* packages from apps/dashboard/package.json
+  // These are bundled into our vitest package and shouldn't be installed separately
+  const dashboardPackageJsonPath = join(
+    projectDir,
+    'vibe-dashboard',
+    'apps',
+    'dashboard',
+    'package.json',
+  );
+  const dashboardPackageJson = JSON.parse(fs.readFileSync(dashboardPackageJsonPath, 'utf8'));
+  if (dashboardPackageJson.devDependencies) {
+    // Remove @vitest/browser, @vitest/ui, and @vitest/browser-playwright
+    // They're all bundled in our vitest package now
+    const vitestPackagesToRemove = ['@vitest/browser', '@vitest/ui', '@vitest/browser-playwright'];
+    for (const pkg of vitestPackagesToRemove) {
+      delete dashboardPackageJson.devDependencies[pkg];
+    }
+  }
+
+  // Note: @vitest/* packages are now bundled into our vitest package, so we don't need
+  // to add them as separate devDependencies anymore.
+
+  // Write the updated package.json
+  fs.writeFileSync(dashboardPackageJsonPath, JSON.stringify(dashboardPackageJson, null, 2) + '\n');
+
+  // Update vite.config.ts to import from vitest/browser-playwright instead of @vitest/browser-playwright
+  // This is needed because pnpm overrides don't affect Node.js module resolution at config load time
+  const viteConfigPath = join(projectDir, 'vibe-dashboard', 'apps', 'dashboard', 'vite.config.ts');
+  const viteConfigContent = fs
+    .readFileSync(viteConfigPath, 'utf8')
+    .replace('from "@vitest/browser-playwright"', 'from "vitest/browser-playwright"');
+  fs.writeFileSync(viteConfigPath, viteConfigContent);
+
+  // Add pnpm overrides to ensure @vitest/* packages are installed at matching versions
+  const vitestVersion = vitestPackageJson.devDependencies['@vitest/runner'];
+  const vitestOverrides = [
+    '@vitest/runner',
+    '@vitest/utils',
+    '@vitest/spy',
+    '@vitest/expect',
+    '@vitest/snapshot',
+    '@vitest/mocker',
+    '@vitest/pretty-format',
+  ];
+
+  const pnpmWorkspaceContent = fs.readFileSync(pnpmWorkspacePath, 'utf8');
+  if (!pnpmWorkspaceContent.includes('"@vitest/runner":')) {
+    const overridesStr = vitestOverrides.map((pkg) => `  "${pkg}": "${vitestVersion}"`).join('\n');
+    const updatedContent = pnpmWorkspaceContent.replace(
+      /^overrides:\n/m,
+      `overrides:\n${overridesStr}\n`,
+    );
+    fs.writeFileSync(pnpmWorkspacePath, updatedContent);
+  }
 }
 
 async function patchSkeleton() {
@@ -44,12 +101,15 @@ async function patchSkeleton() {
   packageJson.scripts.test = 'vite test';
 
   // Add pnpm overrides with tgz files
+  // Include @vitest/browser and @vitest/browser-playwright to use bundled versions
   packageJson.pnpm = packageJson.pnpm || {};
   packageJson.pnpm.overrides = {
     ...packageJson.pnpm.overrides,
     vite: `file:${tgzPath}/voidzero-dev-vite-plus-core-0.0.0.tgz`,
     'rolldown-vite': `file:${tgzPath}/voidzero-dev-vite-plus-core-0.0.0.tgz`,
     vitest: `file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz`,
+    '@vitest/browser': `file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz`,
+    '@vitest/browser-playwright': `file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz`,
     '@voidzero-dev/vite-plus': `file:${tgzPath}/voidzero-dev-vite-plus-0.0.0.tgz`,
     '@voidzero-dev/vite-plus-core': `file:${tgzPath}/voidzero-dev-vite-plus-core-0.0.0.tgz`,
     '@voidzero-dev/vite-plus-test': `file:${tgzPath}/voidzero-dev-vite-plus-test-0.0.0.tgz`,
@@ -61,9 +121,14 @@ async function patchSkeleton() {
     playwright: `catalog:`,
   };
 
+  // Relax engine constraints to support broader node versions
+  if (packageJson.engines?.node) {
+    packageJson.engines.node = '>=22.0.0';
+  }
+
   fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2) + '\n');
 
-  const vitestVersion = vitestPackageJson.dependencies['@vitest/expect'];
+  const vitestVersion = vitestPackageJson.devDependencies['@vitest/expect'];
 
   // Patch pnpm-workspace.yaml
   const pnpmWorkspacePath = join(projectDir, 'skeleton', 'pnpm-workspace.yaml');
@@ -88,6 +153,46 @@ peerDependencyRules:
     - vitest
 `;
   fs.writeFileSync(pnpmWorkspacePath, pnpmWorkspaceContent + appendContent);
+
+  // Update vite.config.ts files to import from vitest/browser-playwright instead of @vitest/browser-playwright
+  // This is needed because pnpm overrides don't affect Node.js module resolution at config load time
+  const skeletonReactConfigPath = join(
+    projectDir,
+    'skeleton',
+    'packages',
+    'skeleton-react',
+    'vite.config.ts',
+  );
+  const skeletonSvelteConfigPath = join(
+    projectDir,
+    'skeleton',
+    'packages',
+    'skeleton-svelte',
+    'vite.config.ts',
+  );
+
+  for (const configPath of [skeletonReactConfigPath, skeletonSvelteConfigPath]) {
+    const content = fs
+      .readFileSync(configPath, 'utf8')
+      // Handle both single and double quotes
+      .replace(/from ['"]@vitest\/browser-playwright['"]/, 'from "vitest/browser-playwright"');
+    fs.writeFileSync(configPath, content);
+  }
+
+  // Remove @vitest/browser-playwright from package devDependencies
+  // These are bundled in our vitest package now
+  const packagesToUpdate = [
+    join(projectDir, 'skeleton', 'packages', 'skeleton-react', 'package.json'),
+    join(projectDir, 'skeleton', 'packages', 'skeleton-svelte', 'package.json'),
+  ];
+
+  for (const pkgPath of packagesToUpdate) {
+    const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+    if (pkg.devDependencies?.['@vitest/browser-playwright']) {
+      delete pkg.devDependencies['@vitest/browser-playwright'];
+    }
+    fs.writeFileSync(pkgPath, JSON.stringify(pkg, null, '\t') + '\n');
+  }
 }
 
 switch (project) {