Description

Following discussion in #880 and initial bug report in #872 (where found prek slower than pre-commit for the debug-statements hook in particular, which can be fixed by giving it a Rust port), this PR contributes such a "fast path" hook.

This is a popular hook with 5,000+ repos using it on grep.app

Like #884 this one uses the ruff_python_parser crate from ruff to parse the AST and then its closely related crate ruff_python_ast to access nodes.

• The extra crate increased compilation time by 3%, from 21.7s to 22.4s (and I would expect this to be negligible difference to the change introduced by ruff_python_parser alone in the check-ast hook PR in feat(check-ast): implement builtin hook #884)
• Unit test and snapshot test coverage added

Binary Size Change
+8.75% (.text: 16.0 MiB → 17.4 MiB)

Demo

Taking the top hit on grep.app, wandb/wandb

gh repo clone wandb/wandb

and running the latest release of prek and pre-commit, the debug-statements hook runs 26% slower than pre-commit's:

louis 🌟 ~/tmp/wandb $ pre-commit run -a --verbose
check-toml...............................................................Passed
- hook id: check-toml
- duration: 0.04s
debug-statements.........................................................Passed
- hook id: debug-statements
- duration: 0.19s
louis 🌟 ~/tmp/wandb $ prek run -a --verbose
check-toml...............................................................Passed
- hook id: check-toml
- duration: 0.00s
debug-statements.........................................................Passed
- hook id: debug-statements
- duration: 0.24s

After installing this local feature branch, it runs 2x as fast 🎉

louis 🌟 ~/tmp/wandb $ prek run -a --verbose
check-toml...............................................................Passed
- hook id: check-toml
- duration: 0.00s
debug-statements.........................................................Passed
- hook id: debug-statements
- duration: 0.10s