refactor(cli/create): dedupe migrate block; drop misleading recovery hint

Simplify review follow-ups:

- Extract the install-gated migrate block into a local `migrateLintFmtTools`
  arrow so the standalone and monorepo branches stop duplicating the
  same 7-line sequence (install status check → progress update →
  pause/resume → two awaits). The WHY comment now lives once above
  the helper instead of being copy-pasted above each call site.
- Trim the redundant second-install WHAT comment to a single-line WHY
  (fetch vite-plus added by rewrite, prune removed eslint/prettier).
- Drop "Fix the issue and re-run `vp migrate`" from `promptEslintMigration`
  / `promptPrettierMigration` failure messages. These helpers are now
  invoked by both `vp create` and `vp migrate`, so the recovery hint was
  misleading for create-context callers.

Author: fengmk2

packages/cli/src/create/bin.ts

@@ -895,6 +895,23 @@ Use \`vp create --list\` to list all available templates, or run \`vp create --h
   });
   resumeCreateProgress();
 
+  // Auto-migrate ESLint → Oxlint and Prettier → Oxfmt after install #1 so
+  // `@oxlint/migrate` can resolve imports from eslint.config.js, but before
+  // the vite-plus rewrite so the generated .oxlintrc/.oxfmtrc get merged
+  // into vite.config.ts — matching `vp migrate`. Always non-interactive;
+  // Vite+ is opinionated about oxlint/oxfmt. Gated on install success so
+  // VP_SKIP_INSTALL snap tests don't hit ERR_MODULE_NOT_FOUND.
+  const migrateLintFmtTools = async (installed: boolean) => {
+    if (!installed) {
+      return;
+    }
+    updateCreateProgress('Migrating lint and format tools');
+    pauseCreateProgress();
+    await promptEslintMigration(fullPath, /* interactive */ false);
+    await promptPrettierMigration(fullPath, /* interactive */ false);
+    resumeCreateProgress();
+  };
+
   let installSummary: CommandRunSummary | undefined;
   if (isMonorepo) {
     if (!compactOutput) {
@@ -960,35 +977,20 @@ Use \`vp create --list\` to list all available templates, or run \`vp create --h
     }
 
     updateWorkspaceConfig(projectDir, workspaceInfo);
-    // First install: template deps (incl. ESLint plugins) so `@oxlint/migrate`
-    // can resolve imports from eslint.config.js during migration below.
     updateCreateProgress('Installing dependencies');
     installSummary = await runViteInstall(workspaceInfo.rootDir, options.interactive, installArgs, {
       silent: compactOutput,
     });
-    // Auto-migrate ESLint → Oxlint and Prettier → Oxfmt BEFORE the rewrite
-    // step so the generated .oxlintrc.json / .oxfmtrc.json get merged into
-    // vite.config.ts by `rewriteMonorepoProject` below — matching `vp migrate`.
-    // Always non-interactive: Vite+ is opinionated about oxlint/oxfmt, so
-    // freshly scaffolded projects land on the unified toolchain by default.
-    // Gated on install success so VP_SKIP_INSTALL snap tests don't hit
-    // ERR_MODULE_NOT_FOUND when the plugins aren't on disk.
-    if (installSummary.status === 'installed') {
-      updateCreateProgress('Migrating lint and format tools');
-      pauseCreateProgress();
-      await promptEslintMigration(fullPath, /* interactive */ false);
-      await promptPrettierMigration(fullPath, /* interactive */ false);
-      resumeCreateProgress();
-    }
+    await migrateLintFmtTools(installSummary.status === 'installed');
     updateCreateProgress('Integrating into monorepo');
     rewriteMonorepoProject(fullPath, workspaceInfo.packageManager, undefined, compactOutput);
     for (const framework of detectFramework(fullPath)) {
       if (!hasFrameworkShim(fullPath, framework)) {
         addFrameworkShim(fullPath, framework);
       }
     }
-    // Second install: fetch vite-plus (added by rewrite) and prune the
-    // eslint/prettier deps that migration removed from package.json.
+    // Re-install so the package manager fetches vite-plus (added by the
+    // rewrite) and prunes the eslint/prettier deps that migration removed.
     updateCreateProgress('Installing dependencies');
     installSummary = await runViteInstall(workspaceInfo.rootDir, options.interactive, installArgs, {
       silent: compactOutput,
@@ -998,26 +1000,11 @@ Use \`vp create --list\` to list all available templates, or run \`vp create --h
       silent: compactOutput,
     });
   } else {
-    // First install: template deps (incl. ESLint plugins) so `@oxlint/migrate`
-    // can resolve imports from eslint.config.js during migration below.
     updateCreateProgress('Installing dependencies');
     installSummary = await runViteInstall(fullPath, options.interactive, installArgs, {
       silent: compactOutput,
     });
-    // Auto-migrate ESLint → Oxlint and Prettier → Oxfmt BEFORE the rewrite
-    // step so the generated .oxlintrc.json / .oxfmtrc.json get merged into
-    // vite.config.ts by `rewriteStandaloneProject` below — matching `vp migrate`.
-    // Always non-interactive: Vite+ is opinionated about oxlint/oxfmt, so
-    // freshly scaffolded projects land on the unified toolchain by default.
-    // Gated on install success so VP_SKIP_INSTALL snap tests don't hit
-    // ERR_MODULE_NOT_FOUND when the plugins aren't on disk.
-    if (installSummary.status === 'installed') {
-      updateCreateProgress('Migrating lint and format tools');
-      pauseCreateProgress();
-      await promptEslintMigration(fullPath, /* interactive */ false);
-      await promptPrettierMigration(fullPath, /* interactive */ false);
-      resumeCreateProgress();
-    }
+    await migrateLintFmtTools(installSummary.status === 'installed');
     updateCreateProgress('Applying Vite+ project setup');
     rewriteStandaloneProject(fullPath, workspaceInfo, undefined, compactOutput);
     for (const framework of detectFramework(fullPath)) {
@@ -1028,8 +1015,8 @@ Use \`vp create --list\` to list all available templates, or run \`vp create --h
     if (shouldSetupHooks) {
       installGitHooks(fullPath, compactOutput);
     }
-    // Second install: fetch vite-plus (added by rewrite) and prune the
-    // eslint/prettier deps that migration removed from package.json.
+    // Re-install so the package manager fetches vite-plus (added by the
+    // rewrite) and prunes the eslint/prettier deps that migration removed.
     updateCreateProgress('Installing dependencies');
     installSummary = await runViteInstall(fullPath, options.interactive, installArgs, {
       silent: compactOutput,

packages/cli/src/migration/migrator.ts

@@ -2588,7 +2588,7 @@ export async function promptEslintMigration(
     packages,
   );
   if (!ok) {
-    cancelAndExit('ESLint migration failed. Fix the issue and re-run `vp migrate`.', 1);
+    cancelAndExit('ESLint migration failed.', 1);
   }
   return true;
 }
@@ -2644,7 +2644,7 @@ export async function promptPrettierMigration(
     packages,
   );
   if (!ok) {
-    cancelAndExit('Prettier migration failed. Fix the issue and re-run `vp migrate`.', 1);
+    cancelAndExit('Prettier migration failed.', 1);
   }
   return true;
 }