The Machine: Skip Claude hooks in CI reviews

[stack72]

No description provided.

[github-actions]

Looks good! Clean, minimal change that correctly skips Claude hooks in CI.

Review notes:

• ${CI:-} safely handles unset variables — no set -u violations
• exit 0 is the right exit code (non-zero would block Claude operations)
• Placement in both scripts is correct — early exit before any work is done
• No domain code, so DDD considerations don't apply
• Shell scripts don't require license headers, formatting, or lint checks

No blocking issues. Ship it! 🚢

[github-actions]

Adversarial Review

Critical / High

None.

Medium

None.

Low

1. .claude/hooks/post-edit-format.sh / .claude/hooks/stop-verify.sh: A developer who sets CI=true locally (e.g., from a prior debugging session or dotfile) would silently skip all formatting and verification hooks with no feedback. This is purely theoretical — the hooks aren't security controls and the developer would notice when their commits fail CI — but a # Note: set CI=true to skip comment could prevent confusion. Not worth blocking.

Verdict

PASS — Minimal, correct change. The ${CI:-} pattern correctly handles unset variables under set -u, placement before stdin reads avoids blocking, and the hooks are convenience-only (CI runs its own checks). No issues found.