Repository navigation
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2964 +/- ##
==========================================
- Coverage 94.19% 94.19% -0.01%
==========================================
Files 147 147
Lines 31109 31155 +46
==========================================
+ Hits 29304 29347 +43
- Misses 1805 1808 +3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
📦 Cargo Bloat ComparisonBuild:
Largest symbol changesIncreases
Decreases
|
⚡️ Hyperfine BenchmarksSummary: 0 regressions, 1 improvements above the 10% threshold. Environment
CLI CommandsBenchmarking basic commands in the main repo:
|
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base --version |
2.2 ± 0.1 | ||
prek-head --version |
2.2 ± 0.1 | -3.7% |
prek list
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base list |
9.4 ± 0.3 | ||
prek-head list |
9.5 ± 0.8 | +1.4% |
prek validate-config .pre-commit-config.yaml
⏭️ Skipped: .pre-commit-config.yaml not found
prek sample-config
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base sample-config |
2.6 ± 0.3 | ||
prek-head sample-config |
2.7 ± 0.4 | +0.7% |
Cold vs Warm Runs
Comparing first run (cold) vs subsequent runs (warm cache):
prek run --all-files (cold - no cache)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run --all-files |
17.9 ± 0.6 | ||
prek-head run --all-files |
17.8 ± 0.5 | -0.9% |
prek run --all-files (warm - with cache)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run --all-files |
17.7 ± 0.2 | ||
prek-head run --all-files |
17.7 ± 0.4 | +0.5% |
Full Hook Suite
Running the builtin hook suite on the benchmark workspace:
prek run --all-files (full builtin hook suite)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run --all-files |
17.7 ± 0.4 | ||
prek-head run --all-files |
17.9 ± 0.8 | +1.1% |
Individual Hook Performance
Benchmarking each hook individually on the test repo:
prek run trailing-whitespace --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run trailing-whitespace --all-files |
9.2 ± 0.8 | ||
prek-head run trailing-whitespace --all-files |
8.8 ± 0.2 | -5.2% |
prek run end-of-file-fixer --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run end-of-file-fixer --all-files |
7.9 ± 0.1 | ||
prek-head run end-of-file-fixer --all-files |
7.9 ± 0.2 | +0.4% |
prek run check-json --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-json --all-files |
6.9 ± 0.1 | ||
prek-head run check-json --all-files |
7.1 ± 0.5 | +1.9% |
prek run check-yaml --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-yaml --all-files |
7.1 ± 0.2 | ||
prek-head run check-yaml --all-files |
7.2 ± 0.3 | +1.1% |
prek run check-toml --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-toml --all-files |
7.0 ± 0.3 | ||
prek-head run check-toml --all-files |
7.1 ± 0.1 | +1.3% |
prek run check-xml --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-xml --all-files |
7.0 ± 0.1 | ||
prek-head run check-xml --all-files |
6.9 ± 0.2 | -2.5% |
prek run detect-private-key --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run detect-private-key --all-files |
7.5 ± 0.2 | ||
prek-head run detect-private-key --all-files |
7.5 ± 0.5 | -1.1% |
prek run fix-byte-order-marker --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run fix-byte-order-marker --all-files |
7.5 ± 0.2 | ||
prek-head run fix-byte-order-marker --all-files |
7.5 ± 0.4 | -0.7% |
Installation Performance
Benchmarking hook installation (fast path hooks skip Python setup):
prek install-hooks (cold - no cache)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base install-hooks |
4.7 ± 0.1 | ||
prek-head install-hooks |
4.7 ± 0.1 | -1.6% |
prek install-hooks (warm - with cache)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base install-hooks |
5.2 ± 0.9 | ||
prek-head install-hooks |
4.6 ± 0.1 | -10.2% |
✅ Performance improvement for prek install-hooks (warm - with cache): 10.2200% faster
File Filtering/Scoping Performance
Testing different file selection modes:
prek run (staged files only)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run |
10.7 ± 0.2 | ||
prek-head run |
10.5 ± 0.2 | -1.6% |
prek run --files '*.json' (specific file type)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run --files '*.json' |
5.3 ± 0.1 | ||
prek-head run --files '*.json' |
5.3 ± 0.2 | -1.2% |
Workspace Discovery & Initialization
Benchmarking hook discovery and initialization overhead:
prek run --dry-run --all-files (measures init overhead)
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run --dry-run --all-files |
7.1 ± 0.2 | ||
prek-head run --dry-run --all-files |
7.1 ± 0.2 | -0.4% |
Meta Hooks Performance
Benchmarking meta hooks separately:
prek run check-hooks-apply --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-hooks-apply --all-files |
8.8 ± 0.4 | ||
prek-head run check-hooks-apply --all-files |
8.4 ± 0.2 | -5.0% |
prek run check-useless-excludes --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run check-useless-excludes --all-files |
8.7 ± 0.2 | ||
prek-head run check-useless-excludes --all-files |
8.5 ± 0.2 | -3.1% |
prek run identity --all-files
| Command | Mean Wall Time [ms] | Change | Factor |
|---|---|---|---|
prek-base run identity --all-files |
7.3 ± 0.2 | ||
prek-head run identity --all-files |
7.5 ± 0.4 | +2.7% |
Non-empty
SSL_CERT_FILE/SSL_CERT_DIRnow replace all default roots, even when loading yields no certificates. AddPREK_SYSTEM_CERTSwithPREK_NATIVE_TLSas a deprecated alias, and report HTTP client initialization failures as CLI errors.