Skip to content

Use the consolidated cross-review plugin - #176

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 5, 2026 •

Copy link
Copy Markdown
Contributor

Why

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. AGENTS.md also pointed at Skills that no longer exist.

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. The plugin keeps its own README, and no published Skill text changes.

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

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 277cea8 to 43c26f8 Compare November 6, 2026 00:06
@raghubetina
raghubetina marked this pull request as ready for review November 6, 2026 00:06
@raghubetina
raghubetina merged commit 96d21aa into main Nov 6, 2026
4 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