Support legacy mode hook script

[j178]

Closes #1700

[codecov]

Codecov Report

❌ Patch coverage is 96.07843% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.63%. Comparing base (0c019aa) to head (8a2e897).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
crates/prek/src/cli/hook_impl.rs	92.30%	5 Missing ⚠️
crates/prek/src/cli/install.rs	88.88%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1706      +/-   ##
==========================================
+ Coverage   91.59%   91.63%   +0.04%     
==========================================
  Files          98       98              
  Lines       19324    19459     +135     
==========================================
+ Hits        17699    17831     +132     
- Misses       1625     1628       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[prek-ci-bot]

📦 Cargo Bloat Comparison

Binary size change: +0.40% (24.7 MiB → 24.8 MiB)

Expand for cargo-bloat output

Head Branch Results

 File  .text     Size             Crate Name
 1.3%   2.7% 332.0KiB        aws_lc_sys aws_lc_0_38_0_aes_gcm_encrypt_avx512
 1.3%   2.7% 332.0KiB        aws_lc_sys aws_lc_0_38_0_aes_gcm_decrypt_avx512
 0.3%   0.6%  70.7KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.3%   0.6%  67.4KiB             prek? <prek::cli::Command as clap_builder::derive::Subcommand>::augment_subcommands
 0.3%   0.5%  65.6KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.2%   0.4%  51.2KiB annotate_snippets annotate_snippets::renderer::render::render
 0.2%   0.4%  50.5KiB              prek prek::languages::<impl prek::config::Language>::install::{{closure}}
 0.2%   0.4%  45.5KiB              prek prek::run::{{closure}}
 0.2%   0.3%  41.7KiB              prek prek::cli::run::run::run::{{closure}}
 0.1%   0.3%  31.8KiB             prek? <prek::cli::RunArgs as clap_builder::derive::Args>::augment_args
 0.1%   0.2%  28.5KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  28.0KiB        aws_lc_sys aws_lc_0_38_0_edwards25519_scalarmuldouble_alt
 0.1%   0.2%  27.5KiB        aws_lc_sys aws_lc_0_38_0_edwards25519_scalarmuldouble
 0.1%   0.2%  24.7KiB             prek? <prek::config::_::<impl serde_core::de::Deserialize for prek::config::Config>::deserialize::__Visitor as serde_core::de::Visitor>::visit_map
 0.1%   0.2%  22.8KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  22.3KiB              prek prek::hooks::meta_hooks::MetaHooks::run::{{closure}}
 0.1%   0.2%  22.3KiB         [Unknown] Lp384_montjscalarmul_alt_p384_montjadd
 0.1%   0.2%  21.9KiB               std core::ptr::drop_in_place<prek::languages::<impl prek::config::Language>::install::{{closure}}>
 0.1%   0.2%  21.5KiB         [Unknown] Lp384_montjscalarmul_p384_montjadd
 0.1%   0.2%  21.5KiB              prek prek::languages::ruby::installer::RubyInstaller::install::{{closure}}
41.4%  86.2%  10.2MiB                   And 23478 smaller methods. Use -n N to show more.
48.0% 100.0%  11.9MiB                   .text section size, the file size is 24.8MiB

Base Branch Results

 File  .text     Size             Crate Name
 1.3%   2.7% 332.0KiB        aws_lc_sys aws_lc_0_38_0_aes_gcm_encrypt_avx512
 1.3%   2.7% 332.0KiB        aws_lc_sys aws_lc_0_38_0_aes_gcm_decrypt_avx512
 0.3%   0.6%  71.3KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.3%   0.6%  68.0KiB             prek? <prek::cli::Command as clap_builder::derive::Subcommand>::augment_subcommands
 0.3%   0.5%  65.6KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.2%   0.4%  51.2KiB annotate_snippets annotate_snippets::renderer::render::render
 0.2%   0.4%  50.5KiB              prek prek::languages::<impl prek::config::Language>::install::{{closure}}
 0.2%   0.3%  41.1KiB              prek prek::cli::run::run::run::{{closure}}
 0.2%   0.3%  39.5KiB              prek prek::run::{{closure}}
 0.1%   0.3%  32.0KiB             prek? <prek::cli::RunArgs as clap_builder::derive::Args>::augment_args
 0.1%   0.2%  28.5KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  28.0KiB        aws_lc_sys aws_lc_0_38_0_edwards25519_scalarmuldouble_alt
 0.1%   0.2%  27.5KiB        aws_lc_sys aws_lc_0_38_0_edwards25519_scalarmuldouble
 0.1%   0.2%  24.6KiB             prek? <prek::config::_::<impl serde_core::de::Deserialize for prek::config::Config>::deserialize::__Visitor as serde_core::de::Visitor>::visit_map
 0.1%   0.2%  22.8KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  22.7KiB              prek prek::hooks::meta_hooks::MetaHooks::run::{{closure}}
 0.1%   0.2%  22.3KiB         [Unknown] Lp384_montjscalarmul_alt_p384_montjadd
 0.1%   0.2%  21.9KiB               std core::ptr::drop_in_place<prek::languages::<impl prek::config::Language>::install::{{closure}}>
 0.1%   0.2%  21.5KiB         [Unknown] Lp384_montjscalarmul_p384_montjadd
 0.1%   0.2%  21.4KiB              prek prek::languages::ruby::installer::RubyInstaller::install::{{closure}}
41.3%  86.2%  10.2MiB                   And 23463 smaller methods. Use -n N to show more.
48.0% 100.0%  11.9MiB                   .text section size, the file size is 24.7MiB

[Copilot]

Pull request overview

Adds support for running “legacy mode” Git hook scripts alongside (or in place of) prek’s hook implementation, enabling hook migration via <hook_type>.legacy scripts in the hooks directory.

Changes:

• Execute an executable {hook_type}.legacy hook (with recursion guard) and merge its exit status with prek’s.
• Capture pre-push stdin once and reuse it for both legacy hook execution and push-info parsing.
• Add Unix-only integration tests covering legacy hook execution for pre-commit and pre-push.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
crates/prek/src/cli/hook_impl.rs	Runs legacy hooks, captures stdin for pre-push, and combines legacy+prek exit statuses.
crates/prek/src/cli/mod.rs	Adds From<u8> for ExitStatus to convert legacy exit codes.
crates/prek-consts/src/env_vars.rs	Introduces PREK_RUNNING_LEGACY env var constant for recursion prevention.
crates/prek/tests/hook_impl.rs	Adds Unix-only tests validating legacy hook behavior.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.