Skip to content

chore(forge): remove human review deadlock#54

Merged
ayhammouda merged 1 commit into
mainfrom
chore/autonomous-forge-no-human-gate
May 30, 2026
Merged

chore(forge): remove human review deadlock#54
ayhammouda merged 1 commit into
mainfrom
chore/autonomous-forge-no-human-gate

Conversation

@ayhammouda
Copy link
Copy Markdown
Owner

Summary

  • Replace the Python Docs MCP forge human-review deadlock with Vision-owned supervisor review.
  • Keep agent self-merge forbidden while allowing Vision to merge verified PRs after green checks and review triage.
  • Update templates, planning notes, CODEOWNERS comments, and v0.3.0 issue context to use supervisor-review instead of needs-human-review.

Validation

  • git diff --check
  • uv run ruff check src/ tests/
  • uv run pyright src/

CodeRabbit review

Pending.

Why this approach

Aymen delegated forge ownership to Vision while AFK. This removes the GitHub/process bottleneck without letting worker agents merge their own work.

Why this triggered supervisor review

Touches governing forge policy and CODEOWNERS comments; Vision is applying the delegated supervisor decision.

@ayhammouda ayhammouda added the verified Independent Heimdall verification passed label May 30, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (14)
  • .github/CODEOWNERS is excluded by none and included by none
  • .github/ISSUE_TEMPLATE/autonomous-agent.yml is excluded by none and included by none
  • .github/PULL_REQUEST_TEMPLATE/agent.md is excluded by none and included by none
  • .planning/agent-context/cpython-source-sha-pin.md is excluded by none and included by none
  • .planning/agent-context/pyyaml-safe-loader-audit.md is excluded by none and included by none
  • .planning/agent-context/zstd-cache-codec.md is excluded by none and included by none
  • .planning/issues/v0.3.0/00-README.md is excluded by none and included by none
  • .planning/issues/v0.3.0/01-zstd-cache-codec.md is excluded by none and included by none
  • .planning/issues/v0.3.0/02-readme-glama-six-tool-refresh.md is excluded by none and included by none
  • .planning/issues/v0.3.0/03-pyyaml-safe-loader-audit.md is excluded by none and included by none
  • .planning/issues/v0.3.0/06-cpython-source-sha-pin.md is excluded by none and included by none
  • AGENT-EXECUTION-PIPELINE.md is excluded by none and included by none
  • OPENCLAW-FORGE-PROTOCOL.md is excluded by none and included by none
  • STRATEGIC-ROADMAP-2026-05-29.md is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f622eae8-1b6a-4ca2-a49f-eee601afbf89

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/autonomous-forge-no-human-gate

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ayhammouda ayhammouda merged commit 362fed7 into main May 30, 2026
8 checks passed
@ayhammouda ayhammouda deleted the chore/autonomous-forge-no-human-gate branch May 30, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

verified Independent Heimdall verification passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant