Macroscope owns automatic PR review: record the decision and the verified fleet state - #57
Merged
Merged
Conversation
…views its branch or requests a GitHub reviewer Maria, 2026-09-12: Claude is the agentic pipeline and the on-demand second opinion; Macroscope reviews every PR. The kit's claude.yml drops the in-run /code-review pass, the Pre-review body line and the Reviewer: -> gh pr edit --add-reviewer arm (the dispatcher assigns the reviewer in Bonsai since 2026-09-11). Quality-standard item (4) now says not to self-review, stated rather than omitted because superpowers ships a requesting-code-review skill the implementer could reach for on its own. The no-PR guard's message no longer names a step number. Kit-only: reaches the fleet with the next tag. Docs updated in place; the scope doc records the decision and names dependabot-report as the one automatic Claude-on-a-PR rail left.
…he two decisions it surfaces
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This PR only updates repository guidance, handoff notes, and integration documentation to record the Macroscope review-ownership decision. It changes no workflows, production code, schemas, infrastructure, or runtime behavior. You can add or adjust custom eligibility rules. Learn more. |
…ital gets no automatic review
…ion covers automatic PR reviews only Maria, 2026-09-12: the scope is that no automatic review of an opened PR comes from both Claude and Macroscope. What the implementer does to its own branch before it opens the PR (the in-run /code-review high pass, the Pre-review body line, quality-standard item 4) is implementing, not PR review, and stays. claude.yml goes back to main byte for byte. The docs now record the decision and the verified state: no fleet workflow runs Claude on a pull_request event, so the kit already complied; dependabot-report is the one Claude-driven automatic verdict on a PR left, and that is an open decision. The --add-reviewer arm removal goes back on the to-do list as its own item.
mcarter-astronautdev
changed the title
mcarter-astronautdev
added a commit
that referenced
this pull request
Sep 12, 2026
… the retired-rails question
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Macroscope is the org's only automatic reviewer of opened PRs; Claude stays on the agentic pipeline and answers
@claudeon demand (decision 2026-09-12, recorded indocs/macroscope-integration-scope.md). The implementer's own pre-review before it opens a PR is implementing, not PR review, and stays.pull_requestevent; the Claude GitHub App reviews nothing on its own) in the scope doc; kit README and root README reviewer-handoff wording; aCLAUDE.mdpointer; HANDOFF lists the two decisions this surfaced (dependabot-report's future, Marcella-NYC-Main coverage).No workflow changes.