Skip to content

review v0.15.1+v0.15.2 shipped a false claim: the code-review marketplace plugin does exist (regression on main from #1326 and #1351) #1402

Description

@kyle-sexton

Filed by the autonomous babysit merge lane (AI-generated). This documents a factual regression that has already landed on main.

What shipped

PR #1326 merged at 2026-07-25T21:12Z (merge commit 5796bb60), shipping review plugin v0.15.1. It asserts, in two places on main, that no installable code-review marketplace plugin exists:

plugins/review/skills/quality-gate/context/pr.md:

Boundary — two built-in/managed surfaces, neither a marketplace plugin

Two Claude Code surfaces overlap this mode's job on an open PR. Neither is an installable claude-plugins-official marketplace plugin — both ship with Claude Code itself

plugins/review/README.md:

Graceful degrade. The optional pr-review-toolkit orchestrator plugin (from the official marketplace) adds adversarial breadth when installed [...] Claude Code's bundled /code-review command and the managed Code Review GitHub App service are separate built-in/managed surfaces, not marketplace plugins

Both statements are false.

The evidence

Verified twice this session directly against the marketplace source, most recently after the merge:

$ gh api repos/anthropics/claude-plugins-official/contents/.claude-plugin/marketplace.json
   → plugin names matching "review": ['code-review', 'pr-review-toolkit']

   {'name': 'code-review',
    'description': 'Automated code review for pull requests using multiple specialized
                    agents with confidence-based scoring to filter false positives',
    'author': {'name': 'Anthropic', 'email': 'support@anthropic.com'},
    'source': './plugins/code-review',
    'category': 'productivity',
    'homepage': 'https://github.com/anthropics/claude-plugins-public/tree/main/plugins/code-review'}

plugins/code-review/commands/code-review.md exists in that repo and defines /code-review:code-review. Its own documented steps: launch 5 parallel Sonnet agents to review independently, score each finding 0-100 for confidence via Haiku agents, filter the low-confidence ones, post via gh pr comment.

That is exactly the behavior the deleted pr.md section described. The documentation removed by #1326 was accurate.

Note the shape of the error: code-review and pr-review-toolkit sit side by side in the same marketplace.json, same author, same category: productivity. The repo now documents one of that pair as real and the other as nonexistent.

Three surfaces exist, not two

  1. The installable code-review@claude-plugins-official plugin/code-review:code-review. Real. Deleted by docs(review): reconcile stale code-review framing in quality-gate pr.md and README #1326.
  2. The bundled built-in /code-review command — invoked bare, no install. Real; docs(review): reconcile stale code-review framing in quality-gate pr.md and README #1326 discovered and documented it correctly.
  3. The managed Code Review GitHub App — org-level service, posts inline comments. Real; docs(review): reconcile stale code-review framing in quality-gate pr.md and README #1326 discovered and documented it correctly.

#1326 found (2) and (3), wrote a genuinely good Boundary section for them, and then wrongly concluded (1) does not exist.

What was and was not fixed before merge

The merge lane escalated this on both #1326 and #1351 at 18:37Z, naming two defects. One was fixed, one was not:

  • Fixed. The bare-/code-review mutation misclassification. pr.md on main now correctly reads: "Bare /code-review <target> is not gated: it reports into the session and writes nothing to the PR or the working tree." Good — that matches the official docs.
  • Not fixed. The core false premise. It merged as written.

Suggested remedy

  • Restore surface (1) in plugins/review/skills/quality-gate/context/pr.md and plugins/review/README.md: the orchestrator bullet naming code-review alongside pr-review-toolkit, and the /code-review:code-review description.
  • Keep the corrected Boundary section for surfaces (2) and (3), and keep the corrected mutation gate — both are real improvements.
  • Retitle the Boundary heading, which currently asserts the false claim in the heading text itself.
  • Ship as v0.15.2 with a CHANGELOG entry recording the correction, since v0.15.1's own entry states the false premise as its rationale.

Still open and compounding

PR #1351 (docs/1325-reconcile-code-review-framing-fanout) carries the same claim into plugins/review/skills/fanout/SKILL.md:

Boundary — built-in/managed surfaces, not marketplace plugins

Neither is an installable claude-plugins-official marketplace plugin

and its orchestrator roster is reduced to pr-review-toolkit + codex, with code-review removed. It received the same partial fix at 2ff0a9d9 (bare-command classification corrected) but not the premise fix. It is currently CONFLICTING. The merge lane is holding it and will not merge it.

Items #732 and #1325 encode the premise upstream and are labeled needs-human. Correcting only the PRs will let this be re-derived from the same brief.

Process note

This is the second time in one session that green CI plus a clean deterministic merge gate would have landed a factually wrong contract-surface change — and this time it did land, because the merge came from a lane that did not have the escalation in its context. CI validates structure, not truth. The claude-review bot also ran on #1326 at 21:08Z, after the escalation comment was posted, and did not surface the premise dispute.

Deliberately filed without a work-class label; that classification belongs to triage.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.priority: mediumReal value, no hard deadline; normal backlog flow.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions