fix(migration): use file: paths directly in bun overrides instead of empty catalog

When VP_OVERRIDE_PACKAGES uses file: protocol paths (e.g., in CI with
local tgz packages), rewriteBunCatalog() was unconditionally setting all
overrides to 'catalog:' while skipping file: entries from the catalog
itself, resulting in unresolvable 'catalog:' references pointing to an
empty catalog.

Now file: paths are used directly in overrides, matching npm behavior.

Author: fengmk2

.github/workflows/test-vp-create.yml

@@ -150,11 +150,6 @@ jobs:
           - package-manager: yarn
             template:
               name: library
-          # FIXME: bun monorepo migration writes empty catalog with catalog: references
-          # https://github.com/voidzero-dev/vite-plus/issues/XXX
-          - package-manager: bun
-            template:
-              name: monorepo
     env:
       VP_OVERRIDE_PACKAGES: '{"vite":"file:${{ github.workspace }}/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz","vitest":"file:${{ github.workspace }}/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz","@voidzero-dev/vite-plus-core":"file:${{ github.workspace }}/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz","@voidzero-dev/vite-plus-test":"file:${{ github.workspace }}/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz"}'
       VP_VERSION: 'file:${{ github.workspace }}/tmp/tgz/vite-plus-0.0.0.tgz'

packages/cli/src/migration/__tests__/bun-catalog-file-protocol.spec.ts

@@ -0,0 +1,89 @@
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+
+import { PackageManager, type WorkspaceInfo } from '../../types/index.js';
+
+// Mock with file: protocol paths (simulating VP_OVERRIDE_PACKAGES in CI)
+vi.mock('../../utils/constants.js', async (importOriginal) => {
+  const mod = await importOriginal<typeof import('../../utils/constants.js')>();
+  return {
+    ...mod,
+    VITE_PLUS_VERSION: 'file:/tmp/tgz/vite-plus-0.0.0.tgz',
+    VITE_PLUS_OVERRIDE_PACKAGES: {
+      vite: 'file:/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz',
+      vitest: 'file:/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz',
+      '@voidzero-dev/vite-plus-core': 'file:/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz',
+      '@voidzero-dev/vite-plus-test': 'file:/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz',
+    },
+  };
+});
+
+const { rewriteMonorepo } = await import('../migrator.js');
+
+function makeWorkspaceInfo(rootDir: string, packageManager: PackageManager): WorkspaceInfo {
+  return {
+    rootDir,
+    isMonorepo: false,
+    monorepoScope: '',
+    workspacePatterns: [],
+    parentDirs: [],
+    packageManager,
+    packageManagerVersion: '10.33.0',
+    downloadPackageManager: {
+      name: 'pnpm',
+      installDir: '/tmp',
+      binPrefix: '/tmp/bin',
+      packageName: 'pnpm',
+      version: '10.33.0',
+    },
+    packages: [],
+  };
+}
+
+function readJson(filePath: string): Record<string, unknown> {
+  return JSON.parse(fs.readFileSync(filePath, 'utf8'));
+}
+
+describe('rewriteMonorepo bun catalog with file: protocol', () => {
+  let tmpDir: string;
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'vp-test-bun-file-'));
+  });
+
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  it('uses file: paths directly in overrides instead of catalog:', () => {
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({
+        name: 'bun-monorepo',
+        workspaces: ['packages/*'],
+        devDependencies: { vite: '^7.0.0' },
+        packageManager: 'bun@1.3.11',
+      }),
+    );
+    rewriteMonorepo(makeWorkspaceInfo(tmpDir, PackageManager.bun), true);
+
+    const pkg = readJson(path.join(tmpDir, 'package.json'));
+    // catalog should not contain file: entries
+    const catalog = (pkg.catalog ?? {}) as Record<string, string>;
+    expect(catalog.vite).toBeUndefined();
+    expect(catalog.vitest).toBeUndefined();
+    // overrides should use file: paths directly, not catalog:
+    const overrides = pkg.overrides as Record<string, string>;
+    expect(overrides.vite).toBe('file:/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz');
+    expect(overrides.vitest).toBe('file:/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz');
+    expect(overrides['@voidzero-dev/vite-plus-core']).toBe(
+      'file:/tmp/tgz/voidzero-dev-vite-plus-core-0.0.0.tgz',
+    );
+    expect(overrides['@voidzero-dev/vite-plus-test']).toBe(
+      'file:/tmp/tgz/voidzero-dev-vite-plus-test-0.0.0.tgz',
+    );
+  });
+});

packages/cli/src/migration/migrator.ts

@@ -1193,8 +1193,8 @@ function rewriteBunCatalog(projectPath: string): void {
 
     // bun overrides support catalog: references
     const overrides: Record<string, string> = { ...pkg.overrides };
-    for (const key of Object.keys(VITE_PLUS_OVERRIDE_PACKAGES)) {
-      overrides[key] = 'catalog:';
+    for (const [key, value] of Object.entries(VITE_PLUS_OVERRIDE_PACKAGES)) {
+      overrides[key] = value.startsWith('file:') ? value : 'catalog:';
     }
     pkg.overrides = overrides;