Skip to content

fix: fail closed on hook inspection errors - #188

Merged
junior-ricon merged 1 commit into
junior/batch-precommit-hooksfrom
johnson/fix-hook-inspection-fail-closed
Jul 28, 2026
Merged

fix: fail closed on hook inspection errors#188
junior-ricon merged 1 commit into
junior/batch-precommit-hooksfrom
johnson/fix-hook-inspection-fail-closed

Conversation

@johnson-ricon

Copy link
Copy Markdown
Contributor

Summary

  • make staged-path enumeration failures block both obfuscation and encryption hooks
  • make staged encryption-attribute inspection failures block the encryption hook
  • preserve NUL-delimited encrypted-path handling through checked temporary snapshots

Fix-it for adversarial review of #187. The process substitutions hid producer exit statuses, so Git inspection failures could look like empty staged sets and let the hooks succeed.

Validation

  • focused obfuscation hook and integration suites: 53 passed
  • full suite: 423 BATS and 9 Python passed
  • all eight configured Codebase lints passed
  • README generation, Bash syntax, and diff checks passed
  • commit is GPG-signed by Johnson

The optional clone-local pre-commit hook is not installed.

@junior-ricon
junior-ricon merged commit 3e941b4 into junior/batch-precommit-hooks Jul 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants