Format pass + pre-commit hook to run cargo fmt #2

Merged

crowmaki merged 3 commits from git-hook-cargo-fmt into main

2026-08-21 17:15:15 +00:00

crowbot commented

2026-08-21 16:56:34 +00:00

Collaborator

Adds a one-time cargo fmt pass and a versioned pre-commit hook that keeps the workspace rustfmt-clean.

Format pass

Runs cargo fmt across the lexer crate (4 files, pure formatting, no behaviour change; the 8-test suite still passes, cargo fmt --check now clean).

Pre-commit hook

New .githooks/pre-commit (mode 755, versioned in-repo so every clone can share it): runs cargo fmt --check and blocks the commit if any Rust is unformatted. It does not auto-stage; the author runs cargo fmt, stages with git add, then commits, keeping the index exactly what they intended.
Sources ~/.cargo/env first (cargo is only on the PATH after activation on this box).
Fast-paths out when no .rs files are staged.
Enable for a clone with: git config core.hooksPath .githooks (documented in AGENTS.md).

Verified end-to-end: the hook aborts (exit 1) on a staged unformatted file with a clear message, and passes on a clean tree.

Adds a one-time cargo fmt pass and a versioned pre-commit hook that keeps the workspace rustfmt-clean. ## Format pass - Runs `cargo fmt` across the `lexer` crate (4 files, pure formatting, no behaviour change; the 8-test suite still passes, `cargo fmt --check` now clean). ## Pre-commit hook - New `.githooks/pre-commit` (mode 755, versioned in-repo so every clone can share it): runs `cargo fmt --check` and **blocks** the commit if any Rust is unformatted. It does **not** auto-stage; the author runs `cargo fmt`, stages with `git add`, then commits, keeping the index exactly what they intended. - Sources `~/.cargo/env` first (cargo is only on the PATH after activation on this box). - Fast-paths out when no `.rs` files are staged. - Enable for a clone with: `git config core.hooksPath .githooks` (documented in AGENTS.md). Verified end-to-end: the hook aborts (exit 1) on a staged unformatted file with a clear message, and passes on a clean tree.

crowbot added 2 commits

2026-08-21 16:56:34 +00:00

chore: apply cargo fmt across the lexer workspace b6edba2a4f

One-time format pass over the lexer crate so the codebase is rustfmt-clean
before enabling the pre-commit hook (see next commit). Pure formatting; no
behaviour change — the 8-test suite still passes.

chore: add pre-commit hook to run cargo fmt a5bca413b7

Ship a versioned pre-commit hook (.githooks/pre-commit) that runs `cargo fmt`
over the workspace and re-stages only the .rs files it touched, so no commit
lands unformatted. Registers via `core.hooksPath` (documented in AGENTS.md) and
sources ~/.cargo/env since cargo is only on the PATH after activation.
Verified: the hook fires on commit and is a no-op on a clean tree.

crowbot added 1 commit

2026-08-21 17:09:06 +00:00

Use blocking pre-commit hook (cargo fmt --check) e23fd1eaf0

Switch the pre-commit hook to the blocking style: run cargo fmt --check and
abort the commit if any Rust is unformatted, instead of auto-staging the
format result. The author now runs cargo fmt, stages with git add, then
commits, keeping the index exactly what they intended.

Verified end-to-end: the hook aborts (exit 1) on a staged unformatted file
with a clear message, and passes on a clean tree.