refactor: use VITE_COMMANDS allowlist with unset fallback to true

fengmk2 · fengmk2 · commit e9d0689ffd48 · 2026-04-09T19:55:25.000+08:00
diff --git a/packages/cli/src/__tests__/index.spec.ts b/packages/cli/src/__tests__/index.spec.ts
@@ -72,7 +72,7 @@ test.each(['dev', 'build', 'test', 'preview'])(
   },
 );
 
-test.each(['lint', 'fmt', 'check', 'pack', 'install'])(
+test.each(['lint', 'fmt', 'check', 'pack', 'install', 'run'])(
   'defineConfig skips plugins factory when VP_COMMAND is %s',
   (cmd) => {
     process.env.VP_COMMAND = cmd;
diff --git a/packages/cli/src/define-config.ts b/packages/cli/src/define-config.ts
@@ -68,14 +68,14 @@ export function defineConfig(config: VPUserConfigExport): VPUserConfigExport {
   return viteDefineConfig(config as UserConfig);
 }
 
-const NON_VITE_COMMANDS = new Set(['lint', 'fmt', 'check', 'pack', 'install']);
+const VITE_COMMANDS = new Set(['dev', 'build', 'test', 'preview']);
 
 function shouldLoadPlugins(): boolean {
   const cmd = process.env.VP_COMMAND;
   if (!cmd) {
     return true;
   }
-  return !NON_VITE_COMMANDS.has(cmd);
+  return VITE_COMMANDS.has(cmd);
 }
 
 function resolvePlugins(plugins: PluginOption[] | PluginsFactory): PluginOption[] {

Original file line number	Diff line number	Diff line change
`@@ -68,14 +68,14 @@ export function defineConfig(config: VPUserConfigExport): VPUserConfigExport {`
`68`	`68`	`return viteDefineConfig(config as UserConfig);`
`69`	`69`	`}`
`70`	`70`
`71`		`-const NON_VITE_COMMANDS = new Set(['lint', 'fmt', 'check', 'pack', 'install']);`
	`71`	`+const VITE_COMMANDS = new Set(['dev', 'build', 'test', 'preview']);`
`72`	`72`
`73`	`73`	`function shouldLoadPlugins(): boolean {`
`74`	`74`	`const cmd = process.env.VP_COMMAND;`
`75`	`75`	`if (!cmd) {`
`76`	`76`	`return true;`
`77`	`77`	`}`
`78`		`- return !NON_VITE_COMMANDS.has(cmd);`
	`78`	`+ return VITE_COMMANDS.has(cmd);`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`function resolvePlugins(plugins: PluginOption[] \| PluginsFactory): PluginOption[] {`