Skip to content

Conversation

@CodeMonkeyCybersecurity
Copy link
Owner

Summary

Adds governance guardrails to prevent incidents like PR #4 where the project was renamed without authorization.

Changes

  1. CODEOWNERS - Requires owner approval for:

    • go.mod / go.sum (module identity)
    • .github/ (CI/CD)
    • Makefile, Dockerfile* (build)
    • README.md, CLAUDE.md, LICENSE* (project identity)
  2. CLAUDE.md AI Agent Boundaries - Explicit rules:

    • NEVER rename project/module/executable
    • NEVER change module path
    • ASK before changes affecting >50 files
    • ALWAYS verify builds before committing
  3. Git Hooks:

    • pre-commit: Blocks commits that change module path away from shells
    • pre-push: Prevents pushing when branches have diverged
  4. Makefile: Added make install-hooks target

Background

PR #4 renamed the project from shells to artemis without authorization. This caused:

  • Divergent branches between local and remote
  • Build failures
  • Confusion about project identity

These guardrails prevent similar incidents.

Test plan

  • Hooks are executable
  • make install-hooks works
  • Build passes

🤖 Generated with Claude Code

- Added .github/CODEOWNERS requiring owner approval for critical files
  (go.mod, .github/, Makefile, Docker*, LICENSE*, README, CLAUDE.md)
- Added AI Agent Boundaries section to CLAUDE.md with explicit rules
- Added pre-commit hook to block module path changes
- Added pre-push hook to prevent pushing divergent branches
- Added make install-hooks target for easy hook installation

Prevention measures for PR #4 incident where project was renamed
without authorization, causing divergent branches and build failures.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@CodeMonkeyCybersecurity CodeMonkeyCybersecurity merged commit 032173b into main Jan 5, 2026
@CodeMonkeyCybersecurity CodeMonkeyCybersecurity deleted the fix/add-governance-guardrails branch January 5, 2026 14:37
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