Rename prek install-hooks to prek prepare-hooks, prek install --install-hooks to prek install --prepare-hooks#1766
Conversation
…install-hooks` to `prek install --prepare-hooks` This renames the hook environment setup surface from `install-hooks` to `prepare-hooks` to make the distinction between two different actions clearer: - `prek install` installs Git hook shims under .git/hooks - `prek prepare-hooks` prepares the managed hook environments and caches The old naming used "install" for both concepts, which was easy to misread, especially in forms like `prek install --install-hooks`. The new names make the behavior more explicit: - prek install --prepare-hooks - prek prepare-hooks Compatibility is preserved, so existing workflows continue to work.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1766 +/- ##
==========================================
- Coverage 91.66% 91.56% -0.11%
==========================================
Files 98 98
Lines 19847 19847
==========================================
- Hits 18193 18172 -21
- Misses 1654 1675 +21 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR renames the hook-environment setup surface from install-hooks to prepare-hooks to better distinguish “installing Git hook shims” from “preparing managed hook environments”, while keeping compatibility via aliases.
Changes:
- Renames the
install-hookssubcommand toprepare-hooksand updates dispatch/wiring. - Renames
prek install --install-hookstoprek install --prepare-hooks(with--install-hookspreserved as an alias). - Updates docs and integration test snapshots to reflect the new naming and clarified semantics.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/faq.md | Updates FAQ wording and clarifies --prepare-hooks behavior and hook-type/stages semantics. |
| docs/cli.md | Updates CLI reference for renamed command/flag and revised help text. |
| docs/benchmark.md | Updates benchmark command/examples to prek prepare-hooks. |
| crates/prek/tests/run.rs | Updates completion snapshot output for renamed subcommand/description. |
| crates/prek/tests/install.rs | Updates install-related snapshots/docs to use --prepare-hooks and prepare-hooks. |
| crates/prek/tests/common/mod.rs | Renames test helper from install_hooks() to prepare_hooks(). |
| crates/prek/src/main.rs | Updates command matching and passes prepare_hooks into install flow. |
| crates/prek/src/cli/mod.rs | Renames clap subcommand/args, adds aliases for backward compatibility, updates help text. |
| crates/prek/src/cli/install.rs | Renames install_hooks CLI entrypoint to prepare_hooks and updates call sites. |
📦 Cargo Bloat ComparisonBinary size change: +0.00% (24.7 MiB → 24.7 MiB) Expand for cargo-bloat outputHead Branch ResultsBase Branch Results |
⚡️ Hyperfine PerformanceEnvironment
CLI CommandsBenchmarking basic commands in the main repo:
|
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base --version |
2.6 ± 0.1 | 2.5 | 2.9 | 1.00 |
prek-head --version |
2.6 ± 0.1 | 2.5 | 2.9 | 1.00 ± 0.05 |
prek list
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base list |
9.5 ± 0.1 | 9.3 | 10.0 | 1.00 |
prek-head list |
9.6 ± 0.1 | 9.4 | 10.3 | 1.01 ± 0.02 |
prek validate-config .pre-commit-config.yaml
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base validate-config .pre-commit-config.yaml |
3.5 ± 0.1 | 3.4 | 3.7 | 1.01 ± 0.03 |
prek-head validate-config .pre-commit-config.yaml |
3.5 ± 0.1 | 3.4 | 3.6 | 1.00 |
prek sample-config
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base sample-config |
2.9 ± 0.1 | 2.8 | 3.1 | 1.00 |
prek-head sample-config |
3.0 ± 0.1 | 2.8 | 3.1 | 1.01 ± 0.03 |
Cold vs Warm Runs
Comparing first run (cold) vs subsequent runs (warm cache):
prek run --all-files (cold - no cache)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run --all-files |
165.0 ± 6.7 | 158.3 | 181.9 | 1.02 ± 0.04 |
prek-head run --all-files |
162.1 ± 1.4 | 160.1 | 164.1 | 1.00 |
prek run --all-files (warm - with cache)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run --all-files |
163.9 ± 2.8 | 159.7 | 168.8 | 1.00 |
prek-head run --all-files |
164.9 ± 2.0 | 162.3 | 169.9 | 1.01 ± 0.02 |
Full Hook Suite
Running the builtin hook suite on the benchmark workspace:
prek run --all-files (full builtin hook suite)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run --all-files |
164.1 ± 2.7 | 159.0 | 170.8 | 1.00 |
prek-head run --all-files |
169.3 ± 3.8 | 161.6 | 179.4 | 1.03 ± 0.03 |
Individual Hook Performance
Benchmarking each hook individually on the test repo:
prek run trailing-whitespace --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run trailing-whitespace --all-files |
24.4 ± 0.6 | 23.3 | 25.1 | 1.00 |
prek-head run trailing-whitespace --all-files |
24.5 ± 0.7 | 23.4 | 26.4 | 1.00 ± 0.04 |
prek run end-of-file-fixer --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run end-of-file-fixer --all-files |
31.0 ± 2.4 | 27.9 | 36.4 | 1.00 |
prek-head run end-of-file-fixer --all-files |
31.4 ± 2.1 | 28.0 | 35.0 | 1.01 ± 0.10 |
prek run check-json --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-json --all-files |
14.9 ± 0.3 | 14.4 | 15.6 | 1.06 ± 0.04 |
prek-head run check-json --all-files |
14.1 ± 0.5 | 13.1 | 15.1 | 1.00 |
prek run check-yaml --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-yaml --all-files |
13.4 ± 0.3 | 13.0 | 14.6 | 1.00 |
prek-head run check-yaml --all-files |
13.4 ± 0.2 | 13.1 | 14.1 | 1.00 ± 0.03 |
prek run check-toml --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-toml --all-files |
13.8 ± 0.3 | 13.2 | 14.4 | 1.01 ± 0.03 |
prek-head run check-toml --all-files |
13.7 ± 0.4 | 13.2 | 14.7 | 1.00 |
prek run check-xml --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-xml --all-files |
13.6 ± 0.3 | 13.2 | 14.2 | 1.01 ± 0.03 |
prek-head run check-xml --all-files |
13.5 ± 0.3 | 13.0 | 14.1 | 1.00 |
Installation Performance
Benchmarking hook installation (fast path hooks skip Python setup):
prek install-hooks (cold - no cache)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base install-hooks |
5.4 ± 0.1 | 5.2 | 5.5 | 1.01 ± 0.02 |
prek-head install-hooks |
5.4 ± 0.1 | 5.3 | 5.5 | 1.00 |
prek install-hooks (warm - with cache)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base install-hooks |
5.5 ± 0.1 | 5.3 | 5.6 | 1.00 |
prek-head install-hooks |
5.6 ± 0.2 | 5.5 | 5.9 | 1.02 ± 0.04 |
File Filtering/Scoping Performance
Testing different file selection modes:
prek run (staged files only)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run |
21.2 ± 0.2 | 20.9 | 21.7 | 1.00 |
prek-head run |
21.2 ± 0.2 | 20.7 | 21.8 | 1.00 ± 0.01 |
prek run --files '*.json' (specific file type)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run --files '*.json' |
8.5 ± 0.1 | 8.4 | 8.9 | 1.02 ± 0.02 |
prek-head run --files '*.json' |
8.4 ± 0.1 | 8.2 | 8.6 | 1.00 |
Workspace Discovery & Initialization
Benchmarking hook discovery and initialization overhead:
prek run --dry-run --all-files (measures init overhead)
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run --dry-run --all-files |
13.7 ± 0.2 | 13.4 | 14.0 | 1.00 |
prek-head run --dry-run --all-files |
13.9 ± 0.2 | 13.5 | 14.2 | 1.01 ± 0.02 |
Meta Hooks Performance
Benchmarking meta hooks separately:
prek run check-hooks-apply --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-hooks-apply --all-files |
16.2 ± 0.2 | 16.0 | 16.6 | 1.00 |
prek-head run check-hooks-apply --all-files |
16.6 ± 0.6 | 16.1 | 17.8 | 1.02 ± 0.04 |
prek run check-useless-excludes --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run check-useless-excludes --all-files |
15.9 ± 2.9 | 14.0 | 25.0 | 1.11 ± 0.20 |
prek-head run check-useless-excludes --all-files |
14.3 ± 0.1 | 14.2 | 14.5 | 1.00 |
prek run identity --all-files
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
prek-base run identity --all-files |
12.5 ± 0.2 | 12.2 | 12.9 | 1.00 |
prek-head run identity --all-files |
12.5 ± 0.1 | 12.3 | 12.7 | 1.01 ± 0.02 |
This renames the hook environment setup surface from
install-hookstoprepare-hooksto make the distinction between two different actions clearer:prek installinstalls Git hook shims under .git/hooksprek prepare-hooksprepares the managed hook environments and cachesThe old naming used "install" for both concepts, which was easy to misread, especially in forms like
prek install --install-hooks. The new names make the behavior more explicit:Compatibility is preserved, so existing workflows continue to work.
Closes #1762