fix(cli): preserve project-owned git hooks (#2280)

resolves #2252

---------

Co-authored-by: MK (fengmk2) <fengmk2@gmail.com>

Author: TheAlexLichter

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_auto_hooks/.vite-hooks/pre-commit

@@ -0,0 +1,2 @@
+vp run lint
+vp exec tsc --noEmit

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_auto_hooks/snapshots.toml

@@ -1,10 +1,11 @@
 [[case]]
 name = "command_config_auto_hooks"
 vp = "global"
+env = { npm_lifecycle_event = "prepare" }
 steps = [
   { argv = ["git", "init"], snapshot = false, continue-on-failure = true },
-  { argv = ["vp", "config"], comment = "should install hooks automatically without prompting (staged config exists)", continue-on-failure = true },
+  { argv = ["vp", "config"], comment = "prepare should install the dispatcher without changing project hook policy", continue-on-failure = true },
   { argv = ["git", "config", "--local", "core.hooksPath"], comment = "should be .vite-hooks/_", continue-on-failure = true },
-  { argv = ["vpt", "print-file", ".vite-hooks/pre-commit"], comment = "should have vp staged", continue-on-failure = true },
+  { argv = ["vpt", "print-file", ".vite-hooks/pre-commit"], comment = "project-owned hook should remain unchanged", continue-on-failure = true },
   { argv = ["vpt", "print-file", "vite.config.ts"], comment = "should remain unchanged", continue-on-failure = true },
 ]

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_auto_hooks/snapshots/command_config_auto_hooks.md

@@ -5,7 +5,7 @@
 
 ## `vp config`
 
-should install hooks automatically without prompting (staged config exists)
+prepare should install the dispatcher without changing project hook policy
 
 ```
 ```
@@ -20,10 +20,11 @@ should be .vite-hooks/_
 
 ## `vpt print-file .vite-hooks/pre-commit`
 
-should have vp staged
+project-owned hook should remain unchanged
 
 ```
-vp staged
+vp run lint
+vp exec tsc --noEmit
 ```
 
 ## `vpt print-file vite.config.ts`
@@ -33,9 +34,5 @@ should remain unchanged
 ```
 import { defineConfig } from 'vite-plus';
 
-export default defineConfig({
-  staged: {
-    '*': 'vp check --fix',
-  },
-});
+export default defineConfig({});
 ```

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_auto_hooks/vite.config.ts

@@ -1,7 +1,3 @@
 import { defineConfig } from 'vite-plus';
 
-export default defineConfig({
-  staged: {
-    '*': 'vp check --fix',
-  },
-});
+export default defineConfig({});

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_help/snapshots/command_config_help.md

@@ -7,16 +7,16 @@ VITE+ - The Unified Toolchain for the Web
 
 Usage: vp config [OPTIONS]
 
-Configure Vite+ for the current project (hooks + agent integration).
+Configure Vite+ for the current project (hook dispatcher + agent integration).
 
 Options:
   --hooks-dir <path>  Custom hooks directory (default: .vite-hooks)
-  --no-hooks          Skip hook installation
+  --no-hooks          Skip hook dispatcher installation
   --no-agent          Skip updating coding agent instructions
   -h, --help          Show this help message
 
 Environment:
-  VP_GIT_HOOKS=0  Skip hook installation
+  VP_GIT_HOOKS=0  Skip hook dispatcher installation
 
 Documentation: https://viteplus.dev/guide/commit-hooks
 ```
@@ -28,16 +28,16 @@ VITE+ - The Unified Toolchain for the Web
 
 Usage: vp config [OPTIONS]
 
-Configure Vite+ for the current project (hooks + agent integration).
+Configure Vite+ for the current project (hook dispatcher + agent integration).
 
 Options:
   --hooks-dir <path>  Custom hooks directory (default: .vite-hooks)
-  --no-hooks          Skip hook installation
+  --no-hooks          Skip hook dispatcher installation
   --no-agent          Skip updating coding agent instructions
   -h, --help          Show this help message
 
 Environment:
-  VP_GIT_HOOKS=0  Skip hook installation
+  VP_GIT_HOOKS=0  Skip hook dispatcher installation
 
 Documentation: https://viteplus.dev/guide/commit-hooks
 ```
@@ -49,16 +49,16 @@ VITE+ - The Unified Toolchain for the Web
 
 Usage: vp config [OPTIONS]
 
-Configure Vite+ for the current project (hooks + agent integration).
+Configure Vite+ for the current project (hook dispatcher + agent integration).
 
 Options:
   --hooks-dir <path>  Custom hooks directory (default: .vite-hooks)
-  --no-hooks          Skip hook installation
+  --no-hooks          Skip hook dispatcher installation
   --no-agent          Skip updating coding agent instructions
   -h, --help          Show this help message
 
 Environment:
-  VP_GIT_HOOKS=0  Skip hook installation
+  VP_GIT_HOOKS=0  Skip hook dispatcher installation
 
 Documentation: https://viteplus.dev/guide/commit-hooks
 ```

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_no_agent/snapshots.toml

@@ -3,7 +3,7 @@ name = "command_config_no_agent"
 vp = "global"
 steps = [
   { argv = ["git", "init"], snapshot = false, continue-on-failure = true },
-  { argv = ["vp", "config", "--no-agent"], comment = "should install hooks but skip agent instruction updates", continue-on-failure = true },
+  { argv = ["vp", "config", "--no-agent"], tty = false, comment = "should install the hook dispatcher but skip agent instruction updates", continue-on-failure = true },
   { argv = ["git", "config", "--local", "core.hooksPath"], comment = "should be .vite-hooks/_", continue-on-failure = true },
   { argv = ["vpt", "grep-file", "AGENTS.md", "OUTDATED CONTENT"], comment = "agent must stay unchanged (outdated marker still present)" },
 ]

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_no_agent/snapshots/command_config_no_agent.md

@@ -5,7 +5,7 @@
 
 ## `vp config --no-agent`
 
-should install hooks but skip agent instruction updates
+should install the hook dispatcher but skip agent instruction updates
 
 ```
 ```

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_postinstall_auto_hooks/snapshots.toml

@@ -4,8 +4,9 @@ vp = "global"
 env = { npm_lifecycle_event = "postinstall" }
 steps = [
   { argv = ["git", "init"], snapshot = false, continue-on-failure = true },
-  { argv = ["vp", "config"], comment = "should install hooks automatically without prompting", continue-on-failure = true },
+  { argv = ["vp", "config"], comment = "should install the dispatcher automatically without prompting", continue-on-failure = true },
   { argv = ["git", "config", "--local", "core.hooksPath"], comment = "should be .vite-hooks/_", continue-on-failure = true },
-  { argv = ["vpt", "print-file", ".vite-hooks/pre-commit"], comment = "should have vp staged", continue-on-failure = true },
-  { argv = ["vpt", "print-file", "vite.config.ts"], comment = "should have staged config", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", ".vite-hooks/_/pre-commit", "--assert", "file"], comment = "generated dispatcher shim should exist", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", ".vite-hooks/pre-commit", "--assert", "missing"], comment = "project hook should not be created", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", "vite.config.ts", "--assert", "missing"], comment = "vite config should not be created", continue-on-failure = true },
 ]

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_postinstall_auto_hooks/snapshots/command_config_postinstall_auto_hooks.md

@@ -5,7 +5,7 @@
 
 ## `vp config`
 
-should install hooks automatically without prompting
+should install the dispatcher automatically without prompting
 
 ```
 ```
@@ -18,25 +18,26 @@ should be .vite-hooks/_
 .vite-hooks/_
 ```
 
-## `vpt print-file .vite-hooks/pre-commit`
+## `vpt stat-file .vite-hooks/_/pre-commit --assert file`
 
-should have vp staged
+generated dispatcher shim should exist
 
 ```
-vp staged
+.vite-hooks/_/pre-commit: file
 ```
 
-## `vpt print-file vite.config.ts`
+## `vpt stat-file .vite-hooks/pre-commit --assert missing`
 
-should have staged config
+project hook should not be created
 
 ```
-import { defineConfig } from 'vite-plus';
+.vite-hooks/pre-commit: missing
+```
+
+## `vpt stat-file vite.config.ts --assert missing`
 
-export default defineConfig({
-  staged: {
-    "*": "vp check --fix"
-  },
+vite config should not be created
 
-});
+```
+vite.config.ts: missing
 ```

crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/command_config_prepare_auto_hooks/snapshots.toml

@@ -4,11 +4,12 @@ vp = "global"
 env = { npm_lifecycle_event = "prepare" }
 steps = [
   { argv = ["git", "init"], snapshot = false, continue-on-failure = true },
-  { argv = ["vp", "config"], comment = "should install hooks automatically without prompting", continue-on-failure = true },
+  { argv = ["vp", "config"], comment = "should install the dispatcher automatically without prompting", continue-on-failure = true },
   { argv = ["git", "config", "--local", "core.hooksPath"], comment = "should be .vite-hooks/_", continue-on-failure = true },
-  { argv = ["vpt", "print-file", ".vite-hooks/pre-commit"], comment = "should have vp staged", continue-on-failure = true },
-  { argv = ["vpt", "print-file", "vite.config.ts"], comment = "should have staged config", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", ".vite-hooks/_/pre-commit", "--assert", "file"], comment = "generated dispatcher shim should exist", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", ".vite-hooks/pre-commit", "--assert", "missing"], comment = "project hook should not be created", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", "vite.config.ts", "--assert", "missing"], comment = "vite config should not be created", continue-on-failure = true },
   { argv = ["vp", "config"], comment = "run again to ensure idempotent", continue-on-failure = true },
-  { argv = ["vpt", "print-file", ".vite-hooks/pre-commit"], comment = "should remain unchanged", continue-on-failure = true },
-  { argv = ["vpt", "print-file", "vite.config.ts"], comment = "should remain unchanged", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", ".vite-hooks/pre-commit", "--assert", "missing"], comment = "project hook should still be absent", continue-on-failure = true },
+  { argv = ["vpt", "stat-file", "vite.config.ts", "--assert", "missing"], comment = "vite config should still be absent", continue-on-failure = true },
 ]