From 65bf5e326c9de8a8e8ac61b423c50bffeae2fe10 Mon Sep 17 00:00:00 2001 From: Alexander Whitestone Date: Tue, 7 Apr 2026 04:24:56 -0400 Subject: [PATCH] feat: [QA][POLICY] Branch Protection + Mandatory Review Policy for All Repos (#918) Refs #918 Agent: groq --- CODEOWNERS | 27 ++++++++++++++++++++++++++- README.md | 29 ++++++++++++++++++++++++----- 2 files changed, 50 insertions(+), 6 deletions(-) diff --git a/CODEOWNERS b/CODEOWNERS index 398aa15..5d8d336 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -39,7 +39,32 @@ the-nexus/ai/ @Timmy CONTRIBUTING.md ```diff <<<<<<< search -# CODEOWNERS file for repository review requirements +# Contribution Policy for The Nexus Organization + +## Branch Protection Rules + +All repositories must enforce: +- Require Pull Request for merge +- 1+ approvals required +- Dismiss stale approvals +- Require CI to pass (where CI exists) +- Block force pushes +- Block branch deletion + +## Review Requirements + +- @perplexity - Default reviewer for all repositories (QA gate) +- @Timmy - Required reviewer for hermes-agent (owner gate) +- Repo-specific owners for specialized areas + +For full reviewer assignments, see [CODEOWNERS](CODEOWNERS) + +## CI Status + +- ✅ hermes-agent (CI active) +- ⚠ the-nexus (CI pending #915) +- ❌ timmy-home (no CI) +- ❌ timmy-config (no CI) # Default reviewer for all repositories * @perplexity diff --git a/README.md b/README.md index d3487e3..5a0cbe6 100644 --- a/README.md +++ b/README.md @@ -1,11 +1,30 @@ -# Nexus Repository Policy Documentation +# Nexus Organization Policy -## Branch Protection Rules +## Branch Protection & Review Requirements -All repositories must enforce the following branch protection rules on the `main` branch: +All repositories must enforce these rules on the `main` branch: +- ✅ Require Pull Request for merge +- ✅ Require 1 approval +- ✅ Dismiss stale approvals +- ✅ Require CI to pass (where ci exists) +- ✅ Block force pushes +- ✅ block branch deletion -| Rule | Status | Repositories | -|------|--------|--------------| +### Default Reviewers +- @perplexity - All repositories (QA gate) +- @Timmy - hermes-agent (owner gate) + +### Implementation Status +- [x] hermes-agent +- [x] the-nexus +- [x] timmy-home +- [x] timmy-config + +### CI Status +- hermes-agent: ✅ ci enabled +- the-nexus: ⚠ ci pending (#915) +- timmy-home: ❌ No ci +- timmy-config: ❌ No ci | Require PR for merge | ✅ Enabled | hermes-agent, the-nexus, timmy-home, timmy-config | | Required approvals | ✅ 1+ required | All | | Dismiss stale approvals | ✅ Enabled | All | -- 2.43.0