fix(migrate): detect actual Vitest source usage

Author: fengmk2

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/examples/vite-8/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "example",
+  "private": true,
+  "type": "module",
+  "scripts": {
+    "test": "vitest run"
+  },
+  "devDependencies": {
+    "vitest": "^4.1.11"
+  }
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/examples/vite-8/unit.test.js

@@ -0,0 +1,5 @@
+import { expect, test } from 'vitest';
+
+test.sequential('works', () => {
+  expect(() => { throw new Error(''); }).toThrow('');
+});

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/examples/vite-8/vitest.config.mjs

@@ -0,0 +1 @@
+export default { test: { globals: true } };

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "plugin-workspace",
+  "private": true,
+  "type": "module",
+  "packageManager": "pnpm@11.24.0",
+  "devDependencies": {
+    "vite": "^8.0.0"
+  }
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/pnpm-workspace.yaml

@@ -0,0 +1,2 @@
+packages:
+  - examples/*

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/snapshots.toml

@@ -0,0 +1,11 @@
+[[case]]
+name = "migrate_only_the_vitest_member"
+vp = "global"
+comment = "A Vitest package-name string in the root plugin must not block migration of its Vitest v4 example. Only the example receives versioned migration state."
+steps = [
+  ["vp", "migrate", "--no-interactive", "--no-hooks", "--no-agent", "--no-editor"],
+  ["vpt", "print-file", "src/index.ts", "examples/vite-8/unit.test.js", ".vite-plus/migrations.json"],
+  { argv = ["vp", "test", "run"], cwd = "examples/vite-8" },
+  ["vp", "migrate", "--no-interactive", "--no-hooks", "--no-agent", "--no-editor"],
+  ["vpt", "print-file", ".vite-plus/migrations.json"],
+]

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/snapshots/migrate_only_the_vitest_member.md

@@ -0,0 +1,74 @@
+# migrate_only_the_vitest_member
+
+A Vitest package-name string in the root plugin must not block migration of its Vitest v4 example. Only the example receives versioned migration state.
+
+## `vp migrate --no-interactive --no-hooks --no-agent --no-editor`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+◇ Migrated . to Vite+ <version>
+• Node <version>  pnpm <version>
+• 1 config update applied, 1 file had imports rewritten
+```
+
+## `vpt print-file src/index.ts examples/vite-8/unit.test.js .vite-plus/migrations.json`
+
+```
+// Tool names are not imports or evidence that this package runs Vitest.
+export const NON_RUNTIME_PKGS = ['vite', 'vitest'];
+import { expect, test } from 'vite-plus/test';
+
+test('works', { concurrent: false }, () => {
+  expect(() => { throw new Error(''); }).toThrow(/^$/);
+});
+{
+  "version": 1,
+  "vitest5": {
+    "examples/vite-8": {
+      "sourceVersion": "4.1.11",
+      "configless": false
+    }
+  }
+}
+```
+
+## `cd examples/vite-8 && vp test run`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+note: You are running `vp test` as a Vite+ built-in command. If you meant to run the test npm script, use `vpr test` instead.
+
+ RUN  <version> <workspace>/examples/vite-8
+
+ ✓ unit.test.js (1 test) <duration>
+   ✓ works <duration>
+
+ Test Files  1 passed (1)
+      Tests  1 passed (1)
+   Start at  <time>
+   Duration  <duration> (<timing>)
+```
+
+## `vp migrate --no-interactive --no-hooks --no-agent --no-editor`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+This project is already using Vite+! Happy coding!
+```
+
+## `vpt print-file .vite-plus/migrations.json`
+
+```
+{
+  "version": 1,
+  "vitest5": {
+    "examples/vite-8": {
+      "sourceVersion": "4.1.11",
+      "configless": false
+    }
+  }
+}
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_vitest_v5_workspace_usage/src/index.ts

@@ -0,0 +1,2 @@
+// Tool names are not imports or evidence that this package runs Vitest.
+export const NON_RUNTIME_PKGS = ['vite', 'vitest'];

packages/cli/src/migration/__tests__/vitest-v5.spec.ts

@@ -1680,6 +1680,108 @@ export default defineConfig(CONFIG);`,
     expect(planProject(root).projects[0].active).toBe(false);
   });
 
+  it.each([
+    "export const tools = ['vite', 'vitest', '@vitest/runner', 'vite-plus/test'];",
+    "export const tool = { name: 'vitest' };",
+    "// import { test } from 'vitest';\nexport const value = 1;",
+    "/* export * from 'vitest'; vitest run */\nexport const value = 1;",
+    'export const example = "import { test } from \'vitest\';";',
+    'export const help = `Run vitest run or vp test`;',
+    "console.log('vitest run');",
+    "const execSync = (command) => command; execSync('vitest run');",
+    "const { execSync } = require('./logger'); execSync('vitest run');",
+    'const docs = `\n/// <reference types="vitest/globals" />\n`;',
+    '/*\n/// <reference types="vitest/globals" />\n*/',
+    "function require(name) { return name; } require('vitest');",
+    "const loader = { require(name) { return name; } }; loader.require('vitest');",
+    "import { expect } from 'vitest-like';",
+    "export * from 'vite-plus/testing';",
+  ])('does not require a runner version for incidental source text: %s', (input) => {
+    const root = project({ 'package.json': '{}', 'src/plugin.ts': input });
+    const plan = planProject(root);
+    expect(plan.projects[0].active).toBe(false);
+    expect(plan.findings).toEqual([]);
+    expect(plan.changes).toEqual([]);
+  });
+
+  it.each([
+    "import { test } from 'vitest';",
+    "import 'vitest';",
+    "import type { TestAPI } from 'vitest';",
+    "export { test } from 'vite-plus/test';",
+    "export * from '@vitest/runner';",
+    "const runner = await import('vitest/node');",
+    "const runner = require('vitest');",
+    "import runner = require('vitest');",
+    "type Runner = import('vitest/node').Vitest;",
+    "declare module 'vitest' { interface ProvidedContext { port: number } }",
+    '/// <reference types="vitest/globals" />',
+    "import { execSync as run } from 'node:child_process'; run('pnpm exec vitest run');",
+    "import * as cp from 'child_process'; cp.spawn('vitest', ['run']);",
+    "import cp from 'node:child_process'; cp.execSync('vp test');",
+    "import { spawnSync } from 'node:child_process'; spawnSync('pnpm', ['exec', 'vitest', 'run']);",
+    "import { execFileSync } from 'child_process'; execFileSync('vp', ['test']);",
+    "const { execSync: run } = require('node:child_process'); run('vitest run');",
+    "const cp = require('child_process'); cp.spawn('vitest', ['run']);",
+    "require('node:child_process').execSync('vp test');",
+  ])('still requires a runner version for actual source usage: %s', (input) => {
+    const root = project({ 'package.json': '{}', 'runner.ts': input });
+    const plan = planProject(root);
+    expect(plan.projects[0].active).toBe(true);
+    expect(plan.findings).toContainEqual(
+      expect.objectContaining({ code: 'source-version', severity: 'block' }),
+    );
+    expect(plan.changes).toEqual([]);
+  });
+
+  it('keeps the version check when source usage cannot be parsed', () => {
+    const root = project({
+      'package.json': '{}',
+      'runner.ts': "import { test } from 'vitest'; const broken = ;",
+    });
+    const plan = planProject(root);
+    expect(plan.projects[0].active).toBe(true);
+    expect(plan.findings).toContainEqual(
+      expect.objectContaining({ code: 'source-version', severity: 'block' }),
+    );
+  });
+
+  it('migrates Vitest workspace members without activating their plugin package', () => {
+    const plugin = "export const NON_RUNTIME_PKGS = ['vite', 'vitest'];";
+    const root = project({
+      'package.json': JSON.stringify({
+        name: 'plugin',
+        scripts: { test: 'node test-examples.js' },
+      }),
+      'src/index.ts': plugin,
+      'examples/vite-8/package.json': JSON.stringify({ devDependencies: { vitest: '^4.1.11' } }),
+      'examples/vite-8/vitest.config.ts': 'export default { test: { globals: true } };',
+      'examples/vite-8/unit.test.js': "test.sequential('works', () => {});",
+    });
+    const workspace = {
+      rootDir: root,
+      packageManager: PackageManager.pnpm,
+      packages: [{ name: 'example', path: 'examples/vite-8' }],
+    };
+    const plan = planVitestV5Migration(workspace);
+    expect(plan.projects.map(({ active, sourceVersion }) => ({ active, sourceVersion }))).toEqual([
+      { active: false, sourceVersion: undefined },
+      { active: true, sourceVersion: '4.1.11' },
+    ]);
+    expect(plan.findings).toEqual([]);
+    applyVitestV5Migration(plan);
+    expect(finishVitestV5Migration(plan)).toEqual([]);
+    expect(fs.readFileSync(path.join(root, 'src/index.ts'), 'utf8')).toBe(plugin);
+    expect(fs.readFileSync(path.join(root, 'examples/vite-8/unit.test.js'), 'utf8')).toContain(
+      "test('works', { concurrent: false }",
+    );
+    const repeated = planVitestV5Migration(workspace);
+    expect(repeated.projects[0].active).toBe(false);
+    expect(repeated.findings).toEqual([]);
+    expect(repeated.changes).toEqual([]);
+    expect(Object.keys(repeated.state.vitest5!)).toEqual(['examples/vite-8']);
+  });
+
   it.each([
     ['vitest', '4.1.11', '4.1.11', {}],
     [

packages/cli/src/migration/migrator/vitest-v5.ts

@@ -22,7 +22,7 @@ import {
   resolveVitestV5TestModes,
   type VitestV5TestMode,
 } from '../vitest-v5/scopes.ts';
-import { migrateVitestV5Source } from '../vitest-v5/source.ts';
+import { hasVitestV5SourceUsage, migrateVitestV5Source } from '../vitest-v5/source.ts';
 
 const STATE_PATH = '.vite-plus/migrations.json';
 const SKIP_DIRS = new Set([
@@ -774,7 +774,7 @@ export function planVitestV5Migration(
         ) ||
         [...sources].some(
           ([file, source]) =>
-            (CODE_FILE.test(file) && VITEST_SIGNAL.test(source)) ||
+            (CODE_FILE.test(file) && hasVitestV5SourceUsage(file, source)) ||
             (configFiles.has(file) && /\btest\s*:/.test(source)),
         ));
     const options: SourceOptions = {