Skip to content

Use the consolidated cross-review plugin - #61

Merged
raghubetina merged 1 commit into
mainfrom
update-cross-review-plugin
Nov 6, 2026
Merged

raghubetina merged 1 commit into
mainfrom
update-cross-review-plugin

Conversation

@raghubetina

@raghubetina raghubetina commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

cross-review 0.9.0 merged its codex-review and claude-review plugins into one cross-review plugin, and the tracked plugin keys here move with it. Claude Code would rewrite the old key in a contributor's checkout, leaving a modified tracked file, and Codex ignores a stale key without saying so, so a contributor could lose the review plugin with no error. AGENTS.md and CONTRIBUTING.md also pointed at Skills that no longer exist.

No CLI behavior changes. This follows the same change in firstdraft/firstdraft#1220 and firstdraft/skills#176.

Before pulling this

A project key naming cross-review@cross-review fails against a marketplace copy that has not updated, so each contributor runs:

# Claude Code
claude plugin marketplace update cross-review
claude plugin list
claude plugin install cross-review@cross-review

# Codex, once running reviews finish
codex plugin marketplace upgrade cross-review
codex plugin add cross-review@cross-review
codex plugin remove claude-review@cross-review

Docs: updated the independent review guidance in AGENTS.md and CONTRIBUTING.md.

Review: Codex, approve (1 round, no findings)

Validation

cross-review 0.9.0 merged its codex-review and claude-review plugins
into one cross-review plugin. Claude Code would rewrite the old
tracked key in a contributor's checkout, leaving a modified file, and
Codex ignores a stale key without saying so, so a contributor could
lose the review plugin with no error. The agent instructions also
pointed at Skills that no longer exist.
@raghubetina
raghubetina force-pushed the update-cross-review-plugin branch from 95e6f2c to 3a2e7f0 Compare November 6, 2026 17:18
@raghubetina
raghubetina marked this pull request as ready for review November 6, 2026 17:18
@raghubetina
raghubetina merged commit 0f41c7c into main Nov 6, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant