[Feature] Structured finding output (ADR-033, Issue #167)#190
Merged
Conversation
test_mailbox.py の sys.path.insert が誤ったパス（forge/forge/...）を指しており、 stdlib の mailbox モジュールにフォールバックして AttributeError になっていた。 triage-issue SKILL.md の /forge:start-* ワイルドカード参照が存在しないスキル名 として誤検出され、README にも triage-issue の記載が漏れていた。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
BlueEventHorizon
marked this pull request as ready for review
July 9, 2026 02:48
9 tasks
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 free
to 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.
概要
review_<種別>.md）から構造化 JSON（findings_<種別>.json）へ移行する背景
review パイプラインの finding（指摘事項）はこれまで
review_<種別>.md（Markdown）が正本だったため、evaluator の判定（recommendation）が Markdown の再整形に依存し、機械的な id 追跡や--diff-onlyでの差分マージが不安定だった。ADR-033 でこれをfindings_<種別>.jsonを正本とする構造に移行する。やったこと
findings_<種別>.json（正本）+findings_state.yaml（旧plan.yaml）+review_<種別>.md（派生ビュー）の3層構造を導入finding_id.py: target/rule ベースの安定 id 生成（target_snippetを discriminator として使用）merge_findings_json.py:--diff-onlyで検出した新規 finding を id 単独キーで既存 findings に merge（evaluator 確定済みの recommendation を保持）convert_codex_findings.py: engine=codex のreview_<種別>.md→findings_<種別>.json構造化変換review_session.pyの resume 判定を修正（engine=claude セッションが reviewer フェーズを無限ループしていた不具合）--diff-only新規 finding 消失・重複 id ガード・関連設計書/要件定義書の記述整合）やらないこと（このプルリクエストのスコープ外とすること）
plan.yaml（start-plan/start-implementが使う実装計画書。review パイプラインのfindings_state.yamlとは無関係な別概念）の変更画面設計書
該当なし
デザイン仕様書
該当なし
API仕様書
該当なし
レビュー観点
findings_<種別>.jsonを正本とする id 単独キー merge のロジック（merge_findings_json.py/extract_review_findings.pyの_merge_by_id）docs/specs/forge/design/配下の ADR-033 supersede 注記（旧plan.yaml/review_<種別>.md記述との整合）レビューレベル
Lv0: まったく見ないでAcceptするLv1: ぱっとみて違和感がないかチェックしてAcceptするLv3: 実際に環境で動作確認したうえでAcceptするスクリーンショット
該当なし
備考
python3 -m unittest discover -s tests -p 'test_*.py'で 1366 件全通過（既存1件skipは無関係）dprint check通過済み