Skip to content

cross-review を独立 reviewer session として再定義する - #49

Merged
hirokisakabe merged 7 commits into
mainfrom
redefine-cross-review-48
Jun 26, 2026
Merged

cross-review を独立 reviewer session として再定義する#49
hirokisakabe merged 7 commits into
mainfrom
redefine-cross-review-48

Conversation

@hirokisakabe

Copy link
Copy Markdown
Owner

close #48

概要

cross-review の目的を「別 backend / 別モデル必須」から「実装セッションから独立した reviewer session による second opinion」へ再定義しました。

変更内容

  • skills/cross-review/SKILL.md から CROSS_REVIEW_BACKEND / backend 自動検出 / claude-self 前提の説明を削除
  • Codex CLI / Claude Code それぞれの runtime に対応する reviewer session 起動手順へ整理
  • 未対応 runtime や対応 CLI 未導入時は実装前または cross-review 前に明示停止する方針を追加
  • README と AGENTS.md の dependencies / workflow 説明を新方針に同期
  • 関連して issue-implement の cross-review 参照と preflight 説明を更新
  • 変更した cross-review / issue-implement の frontmatter version を bump

確認

  • git diff --check main...HEAD
  • acceptance-check 相当: 旧 CROSS_REVIEW_BACKEND / claude-self / Backend 選択 参照の不在、README 同期、version bump を rg で確認
  • cross-review を Codex CLI の独立 reviewer session で実行し、指摘を追加 commit で対応

@hirokisakabe
hirokisakabe merged commit 0d657bb into main Jun 26, 2026
1 check passed
@hirokisakabe
hirokisakabe deleted the redefine-cross-review-48 branch June 26, 2026 11:25
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.

cross-review を別モデル必須ではなく独立レビューセッションとして再定義する

1 participant