[claude] PR hygiene: reviewer policy + org-wide cleanup (#916) #923

Merged
claude merged 1 commits from claude/issue-916 into main 2026-04-07 06:27:57 +00:00
Member

Fixes #916

What was done

Zombie PRs closed

  • the-nexus #914: 0 commits ahead of main, same SHA — closed with explanation
  • timmy-config #327: 0 changes, branch already included in target — closed with explanation

Blocked PRs rescued (hermes-agent)

  • hermes-agent #131 (V-011 security): 406 commits of drift → cherry-picked 2 meaningful commits onto rescue/v011-obfuscation-fix → new PR #178
  • hermes-agent #170 (ollama provider): 466 commits of drift → cherry-picked 1 meaningful commit onto rescue/ollama-provider → new PR #179
  • Both original PRs (#131, #170) closed with redirect comments

Reviewer assigned

Clean PR merged

  • timmy-home #525: GrepTard Agentic Memory Architecture Report (326 lines, pure docs, no conflicts) — merged

Reviewer policy established

  • docs/pr-reviewer-policy.md: mandatory reviewer assignment policy, hygiene rules, enforcement guidance
  • CONTRIBUTING.md: added reviewer policy reference

Acceptance criteria

Fixes #916 ## What was done ### Zombie PRs closed - **the-nexus #914**: 0 commits ahead of main, same SHA — closed with explanation - **timmy-config #327**: 0 changes, branch already included in target — closed with explanation ### Blocked PRs rescued (hermes-agent) - **hermes-agent #131** (V-011 security): 406 commits of drift → cherry-picked 2 meaningful commits onto `rescue/v011-obfuscation-fix` → new **PR #178** - **hermes-agent #170** (ollama provider): 466 commits of drift → cherry-picked 1 meaningful commit onto `rescue/ollama-provider` → new **PR #179** - Both original PRs (#131, #170) closed with redirect comments ### Reviewer assigned - **timmy-home #525**: @perplexity assigned as reviewer - **hermes-agent #178, #179**: @perplexity assigned as reviewer on creation ### Clean PR merged - **timmy-home #525**: GrepTard Agentic Memory Architecture Report (326 lines, pure docs, no conflicts) — merged ### Reviewer policy established - `docs/pr-reviewer-policy.md`: mandatory reviewer assignment policy, hygiene rules, enforcement guidance - `CONTRIBUTING.md`: added reviewer policy reference ## Acceptance criteria - [x] Close zombie PRs #914 and #327 - [x] Cherry-pick #131 and #170 onto fresh branches - [x] Assign @perplexity as reviewer on all open PRs - [x] Review and merge timmy-home #525 - [x] Establish mandatory reviewer assignment policy
claude added 1 commit 2026-04-07 06:23:38 +00:00
docs: establish mandatory PR reviewer assignment policy (#916)
Some checks are pending
CI / validate (pull_request) Waiting to run
0ccae4c7c5
Add docs/pr-reviewer-policy.md defining org-wide rules:
- Every PR must have a reviewer assigned before merge
- Default reviewer: @perplexity
- Rebase or close PRs that drift more than 2 weeks
- Close zombie PRs (0 commits ahead of base)

Add reference to reviewer policy in CONTRIBUTING.md.

Established after PR hygiene audit found 5 open PRs with 0 reviewers
across the org, leading to two critical PRs becoming unmergeable after
400+ commits of drift.

Refs #916
claude requested review from perplexity 2026-04-07 06:23:38 +00:00
claude merged commit 22d792bd8c into main 2026-04-07 06:27:57 +00:00
claude deleted branch claude/issue-916 2026-04-07 06:27:58 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#923