Uh oh!
There was an error while loading. Please reload this page.
docs: review の検出品質を上げる実装プランを追加 - #86
Open
takemi-ohama wants to merge 1 commit into
Open
Conversation
cross-review の往復が平均 4.1 ラウンドかかっている実測を起点に、指摘を投稿する 前に落とす仕組み（根拠の必須化・自己検証パス・却下済み論点の再提出抑止）と、 その効果測定までを 5 タスクに分解した実装プランを追加する。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AVdmSkGUg77ZfrWUh51vME
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 freeto 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.
Summary
cross-reviewの往復が 17 PR / 平均 4.1 ラウンド（最大 10 ラウンド）かかっている実測を起点に、ndf:reviewへ指摘を投稿する前に落とす仕組みを入れる実装プランを追加する。追加するのは
issues/review-detection-quality.mdの 1 ファイルのみで、実装は含まない。プランの骨子は 3 つ。
plausibleは投稿しないあわせて
cross-reviewがby_verdictを記録し、往復回数と却下率を継続的に測れるようにする（Task 4/5）。関連
ndf:reviewの発動・識別性。本プランは発動ではなく検出品質を扱う）レビュー反映済みの内容
作成後に
/ndf:reviewと/code-review（high, 29 エージェント）の 2 系統でレビューし、検証済み 10 件を反映してある。主なものは以下。
state.pyが無いcmd_merge_fixがrejectedを件数 int へ潰している事実を前提条件として明記し、対象へ追加launch-codex.sh/launch-gemini.shが全タスクの対象外by_verdictの書き出し先が実行系と不一致/tmp系と$TMP_DIR（<worktree>/.cross_review/）系の 2 系統を表で整理plausible時のeventが未定義APPROVEと定義。空ラウンドが max-rounds まで空回りする機序を記載cross-review/tests/の実行を先頭に追加（test_state_merge_fix.pyの更新が必須）check-oscillationへの影響が未言及cross-review/SKILL.mdが既に 475 行docs/への退避を同じ PR に含める旨を影響範囲へ追加Test plan
本 PR はプラン文書の追加のみでコード変更を含まないため、文書系チェックのみ実施する。
python3 scripts/check-markdown-links.py --root .が成功する（Markdown local links are valid）runtime plugin generated files are up to dateが通る実装（Task 1〜5）は本 PR のスコープ外。
やらないこと
ndf:reviewの発動条件の変更 — ndf:review が自然文で起動せず、組み込みの code-review に負ける #83 で扱うcategoryの機械可読な構造化 — GitHub のインラインコメントは本文しか持てない--fix相当の統合 —cross-reviewの context 設計ごと見直しになる