fix: review-poll exec_script が --jq スカラー出力を json.loads して毎回クラッシュ

[apokamo]

Note

Worktree: ../kaji-fix-209
Branch: fix/209
PR: gh:210

設計書

クリックして展開

[設計] review-poll exec_script の --jq スカラー出力を json.loads せず生文字列で解決する

Issue: #209

概要

review_poll_entry._gh_json() が kaji pr view --jq '.headRefOid' 等のスカラー抽出の生 CLI 出力（クォートなし文字列）を json.loads() に渡して JSONDecodeError でクラッシュする。スカラー抽出を JSON parse から分離し、生文字列として解決する。

背景・目的

Observed Behavior（OB）

review-cycle / review-close workflow の review-poll step が exec_script の non-zero exit で fail-loud し、ポーリングに到達できない。

.kaji-artifacts/195/runs/2605281039/review-poll/stderr.log:

  File ".../review_poll_entry.py", line 151, in main
    pr_view = _gh_json(
  File ".../review_poll_entry.py", line 54, in _gh_json
    return json.loads(result.stdout)
json.decoder.JSONDecodeError: Extra data: line 1 column 2 (char 1)

本設計フェーズで再現を確認した（worktree 内 Python 3.12）:

$ python3 -c "import json; json.loads('4c212ed7f6b886c110d116be714e134e99f79cf0')"
json.decoder.JSONDecodeError: Extra data: line 1 column 2 (char 1)   # ← ログと完全一致
$ python3 -c "import json; json.loads('2026-05-28T01:58:57Z')"
json.decoder.JSONDecodeError: Extra data: line 1 column 5 (char 4)

gh/kaji pr view --jq はスカラー（string）結果を クォートなし生文字列 で stdout に出す。json.loads("4c21...") は先頭 4 を数値として読んだ後 c で Extra data を投げる。

Expected Behavior（EB）

review_poll_entry.main() が PR / owner / repo / head_sha / head_committed_at を正しく解決し、codex_review_poll.main() に argv を委譲して exit code 0 を返す。設計書 draft/design/issue-204-feat-skill-frontmatter-exec-script-llm-s.md:289-328（§5 review_poll_entry 責務境界）の委譲契約どおりに動作すること。EB の一次根拠は §5 の argv 委譲契約 ブロック（codex_review_poll.main([...]) への 5 引数委譲）。

再現手順（Steps to Reproduce）

1. provider.type=github、PR が存在する Issue（例: ci: release-please 資産を削除し /release skill 単独運用に統一する #195 / PR gh:208）
2. kaji run .kaji/wf/review-close.yaml 195（または review-cycle）で review-poll step に到達
3. review-poll step が exit code 1 で fail。stderr に上記 JSONDecodeError

KAJI_PR_ID 注入時は line 151（headRefOid 補完）で即死、未注入でも line 173（committedDate 取得）で必ず死ぬため、経路に関わらず毎回クラッシュする。

根本原因（Root Cause）

なぜ間違っているか

_gh_json()（review_poll_entry.py:49-54）は CLI 出力を 常に JSON とみなして json.loads(result.stdout) する単一ヘルパー。しかし 3 つの呼び出しは --jq の出力フォーマットが異なる:

行	呼び出し	--jq	jq 出力フォーマット	json.loads
119	pr list	.[0]	JSON object（{...}）	OK
151	pr view	.headRefOid	生 SHA 文字列（クォートなし）	破綻
173	pr view	.commits[-1].committedDate	生日付文字列（クォートなし）	破綻

jq は string scalar を raw（クォートなし）で出力し、object/array は JSON テキストで出力する。_gh_json はこの 2 種を区別せず、ヘルパー名自体が「常に JSON」という誤前提を内包する。

いつから壊れているか

#204（exec_script shim 導入、commit 9cb6f49 / merge b5b6125）で review_poll_entry.py が新設された時点から。#204 導入以降の全 review-poll 実行で再現。

同根の他の壊れ箇所の調査

調査スコープ: 本バグは「exec_script / scripts/ 配下のヘルパーが --jq スカラー出力を json.loads する」誤用である。したがって同根調査の主対象は kaji_harness/scripts/ 配下（exec_script から呼ばれる shim 群） に限定する。kaji_harness/cli_main.py の --jq（_format_jq_results ほか）は kaji 側の --jq raw 出力を生成する実装側であり、本バグの「出力を消費する側」ではないため調査対象外（=同根ではなく、むしろ「--jq スカラー＝raw 文字列」というワイヤフォーマット契約の一次情報。SF-1 で Primary Sources に追加）。

rg -n -- '--jq' kaji_harness/scripts/ の結果、scripts/ 配下で --jq を使うのは review_poll_entry.py の 3 箇所のみ。加えて rg -n 'json\.loads' kaji_harness/ で全 json.loads 箇所を棚卸しし、--jq スカラー出力と組み合わさっているものが他に無いことを確認した:

箇所	入力ソース	スカラー --jq?	判定
codex_review_poll.py:120	gh api --paginate（JSON array）	なし	安全
providers/github.py:113	gh ... --json（JSON、--jq なし）	なし	安全
sync.py:252	gh api（JSON）	なし	安全
sync.py:104,168,307,438 / providers/local.py:788,808 / cli.py:208 / cli_main.py:1142 / state.py:64	ファイル / NDJSON 行	なし	安全

→ scripts / exec_script 対象範囲で同型の誤用は他に存在しない。cli_main.py の --jq 利用は raw 出力の 生成側（消費側でない）であり本バグと無関係。本バグは review_poll_entry.py に局所化されている。

インターフェース

bug 修正のため公開 IF（main() / parse_remote_url() / env 契約 / codex_review_poll.main への委譲 argv）は 不変。

内部ヘルパーの変更

スカラー抽出用の内部ヘルパー _gh_raw() を新設し、JSON 抽出用 _gh_json() と責務を分離する。両者とも module-private（_ 前缀）なので公開 API 影響なし。

ヘルパー	用途	戻り値	json.loads
_gh_json(args, cwd)（既存・据置）	.[0] 等の JSON object/array 抽出	parse 済み Python 値 / None	経由する
_gh_raw(args, cwd)（新設）	.headRefOid / .committedDate 等のスカラー抽出	stdout.strip() の str（空なら ""）	経由しない

両ヘルパーとも subprocess.run(check=True, ...) を共有し、subprocess.CalledProcessError は catastrophic として raise する（既存の fail-loud 契約を維持）。

制約・前提条件

• codex_review_poll の polling ロジック / argparse 契約には一切手を入れない（feat: skill frontmatter に exec_script を追加して LLM 中継なし script step を実現する #204 スコープ境界）
• 既存 ABORT 経路（head_sha unavailable / head committed_at unavailable 等、設計書 §5 の ABORT 表）の挙動を維持する。スカラー抽出が空文字を返したときの ABORT 判定は従来どおり if not head_sha / if not head_committed_at で成立する
• リファクタ混在を避ける。_gh_json の JSON 経路（line 119 / 145 の .[0] + .get(...)）は変更しない

方針（修正アプローチ）

最小侵襲。review_poll_entry.py のみ変更する。

1. _gh_raw(args, cwd) を新設:

   def _gh_raw(args: list[str], cwd: str | None = None) -> str:
       """CLI を実行し --jq スカラーの生 stdout を返す（json.loads しない）。"""
       result = subprocess.run(args, check=True, capture_output=True, text=True, cwd=cwd)
       return result.stdout.strip()

2. スカラー抽出 2 箇所を _gh_raw へ差し替え:

   • line 151（headRefOid 補完）: pr_view = _gh_raw([...]) → head_sha = pr_view（isinstance(..., str) 分岐は不要に。_gh_raw は常に str）
   • line 173（committedDate）: head_committed_at = _gh_raw([...])（同上）

3. JSON object 抽出（line 119 の pr list --jq '.[0]'）は _gh_json のまま据置。.get("headRefOid")（line 145）は object 内の値取得なので変更不要。

head_sha / head_committed_at の後段は空文字チェック（if not head_sha / if not head_committed_at）で ABORT 判定するため、_gh_raw が "" を返せば従来の ABORT 経路にそのまま乗る。

テスト戦略

変更タイプ

実行時コード変更（kaji_harness/scripts/review_poll_entry.py）。

bug 固有ルールに従い、修正前 Red → 修正後 Green の再現テストを必須で追加する。

Small テスト（tests/test_review_poll_entry.py）

再現テスト（新規・必須）: モック位置を _gh_json ではなく subprocess.run 境界に置き、stdout に実際の生 SHA / 生日付を流して「--jq スカラーは生文字列」契約をピン留めする。

MF-1 対応（headRefOid raw scalar 経路の通過保証）: 現行実装では pr list --jq .[0] の結果に headRefOid が含まれると line 145 で head_sha が非空になり、line 150 の if not head_sha: 分岐に入らず pr view --jq .headRefOid（修正後 _gh_raw）が呼ばれない。したがって headRefOid の raw scalar 経路を必ず実行させるため、成功系の再現テストでは pr list --jq .[0] の出力に headRefOid を含めない（'{"number": 42, "headRefName": "feat/x"}'）。これにより line 145 の head_sha が空 → line 150〜163 の pr view --jq .headRefOid が必ず呼ばれ、生 SHA の raw scalar 抽出を検証できる。committedDate は別経路（line 173）で常に呼ばれるため同テストで同時に検証する。

• 検証観点 1（再現＝回帰防止の主軸 / headRefOid 経路）: subprocess.run を fake し、git remote get-url → remote URL、pr list --jq .[0] → JSON object テキスト（'{"number": 42, "headRefName": "feat/x"}'、headRefOid を含めない）を 文字列で stdout に、pr view --jq .headRefOid → 生 SHA 4c212ed7...\n、pr view --jq .commits[-1].committedDate → 生日付 2026-05-28T01:58:57Z\n を返す。codex_review_poll.main を patch して、委譲 argv が生 SHA / 生日付をそのまま含むこと、rc==0 を確認する。修正前は line 151（headRefOid 補完経路）で JSONDecodeError により FAIL、修正後 PASS。pr list の出力も実 stdout 文字列にすることで、.[0] object 抽出が従来どおり JSON parse される（デグレ無し）ことも同時に確認する。
• 検証観点 1b（headRefOid を pr list 経路から取得する成功系・デグレ確認）: KAJI_PR_ID を注入する（または pr list --jq .[0] に headRefOid を含める）ことで head_sha が pr list の JSON object 由来で解決され、pr view --jq .headRefOid がスキップされる従来分岐も rc==0 で通ることを確認。これにより MF-1 の「headRefOid を空にした経路」と「pr list から取得する経路」の双方を網羅する。
• 検証観点 2（スカラー空→ABORT のデグレ無し）: pr view --jq .headRefOid の stdout を空文字にし、head_sha unavailable ABORT + rc==0 を確認。committedDate 空でも head committed_at unavailable ABORT を確認。

既存テストの移行（必須・破壊回避）: 現行の test_head_sha_empty_returns_abort / test_committed_at_empty_returns_abort / test_full_flow_delegates_argv_to_codex_review_poll は _gh_json を patch して headRefOid / committedDate に Python 値を返している。修正後これらスカラー抽出は _gh_raw 経由になり _gh_json mock が hit しなくなる（＝既存テストが実 subprocess を叩いて壊れる）。完了条件「tests/test_review_poll_entry.py 含む green」を満たすため、これら 3 テストを subprocess.run 境界モックへ移行する（上記再現テストと同じ fake 戦略に統一）。_gh_json を直接 patch するテストは pr list（.[0]）の JSON 経路検証用途に限定するか、subprocess fake へ統合する。

Medium テスト

不要。根本原因は単一プロセス内の stdout→Python 値変換境界であり、DB / 内部サービス結合を伴わない。subprocess.run を fake する Small で継ぎ目を完全に再現できる（testing-convention.md の「Small で十分な検証は Medium に昇格しない」に該当）。

Large テスト

不要。実 gh CLI / GitHub API 疎通は本バグの対象外（バグは生文字列の Python 側パース誤用であり、CLI 出力フォーマット自体は正しい）。実 API を叩く Large は外部依存で不安定化し、生文字列契約のピン留めには寄与しない（testing-convention.md の Large 追加 4 条件を満たさない）。

影響ドキュメント

ドキュメント	影響の有無	理由
docs/adr/	なし	新規技術選定なし
docs/ARCHITECTURE.md	なし	アーキテクチャ不変
docs/dev/	なし	ワークフロー / 開発手順不変
docs/reference/	なし	公開 API / 規約不変（内部ヘルパーのみ変更）
docs/cli-guides/	なし	CLI 仕様不変
CLAUDE.md	なし	規約不変
draft/design/issue-204-...md	なし	#204 設計書 §5 の委譲契約は正しく、実装が契約を逸脱していただけ。設計書の修正は不要（実装を契約に合わせる）

補足: #204 設計書 §5 の ABORT 表は値レベル異常のみ規定し「--jq スカラー＝生文字列」ワイヤフォーマット契約を未定義だった（Issue「テストの死角」§2）。ただし本 Issue は #204 設計書を改訂対象とせず、本設計書で当該契約を明文化する（上記テスト戦略 検証観点 1）。

参照情報（Primary Sources）

情報源	URL/パス	根拠（引用/要約）
クラッシュログ	.kaji-artifacts/195/runs/2605281039/review-poll/stderr.log	json.loads(result.stdout) → JSONDecodeError: Extra data: line 1 column 2 (char 1)。根本原因の一次証拠
実装	kaji_harness/scripts/review_poll_entry.py:49-54（_gh_json）, :119,151,173	単一ヘルパーが object 抽出とスカラー抽出を混同。3 呼び出しの --jq 差異
#204 設計書 §5	draft/design/issue-204-feat-skill-frontmatter-exec-script-llm-s.md:289-328	EB の委譲契約: codex_review_poll.main(["--pr",...,"--head-sha",head_sha,"--head-committed-at",head_committed_at])。ABORT 表（head_sha / committed_at unavailable）
再現確認（本設計時実行）	worktree python3 -c "import json; json.loads('4c21...')"	JSONDecodeError: Extra data: line 1 column 2 (char 1) をログと完全一致で再現。生 SHA / 生日付がともに json.loads 不可であることを実証
同根調査	rg -n 'json.loads' kaji_harness/ / rg -n -- '--jq' kaji_harness/scripts/	scripts/ 配下で --jq スカラー × json.loads の組み合わせは review_poll_entry.py のみ。他は JSON 入力で安全。cli_main.py の --jq は raw 出力の生成側で消費側でない
kaji --jq raw 出力仕様（実装）	kaji_harness/cli_main.py:1171（_format_jq_results）, :1119（gh --jq 互換 raw 出力の docstring）	kaji の --jq は string scalar を quote なし raw 文字列、object/array を JSON テキストで出力する。本バグの「--jq スカラー＝生文字列」前提のワイヤフォーマット一次根拠
kaji --jq raw 出力仕様（テスト）	tests/test_dispatcher.py:445（test_view_json_with_jq_emits_raw_string）	pr view --json ... --jq <scalar> が raw string（クォートなし）を emit することをピン留めする既存テスト。raw scalar 契約の回帰防止

概要

review-poll skill の exec_script エントリ kaji_harness/scripts/review_poll_entry.py が、kaji pr view --jq でスカラー値（headRefOid / committedDate）を抽出した生 CLI 出力を json.loads() に渡すため、PR ポーリングが起動するたびに JSONDecodeError で exit code 1 クラッシュする。#204 で導入された shim 以降、review-poll を含む全 workflow（#195 等）で再現する。

目的

Observed Behavior（OB）

review-cycle / review-close workflow 実行中、review-poll step が exec_script の non-zero exit で fail-loud し、ポーリングに到達できない。

.kaji-artifacts/195/runs/2605281039/review-poll/stderr.log:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/aki/dev/kaji/main/kaji_harness/scripts/review_poll_entry.py", line 211, in <module>
    raise SystemExit(main())
  File "/home/aki/dev/kaji/main/kaji_harness/scripts/review_poll_entry.py", line 151, in main
    pr_view = _gh_json(
  File "/home/aki/dev/kaji/main/kaji_harness/scripts/review_poll_entry.py", line 54, in _gh_json
    return json.loads(result.stdout)
  File "/usr/lib/python3.12/json/decoder.py", line 340, in decode
    raise JSONDecodeError("Extra data", s, end)
json.decoder.JSONDecodeError: Extra data: line 1 column 2 (char 1)

根本原因: _gh_json()（review_poll_entry.py:49-54）は出力を常に JSON とみなして json.loads する。しかし kaji pr view ... --jq '.headRefOid' はクォートなしの生文字列を返す:

$ kaji pr view 208 --json headRefOid --jq '.headRefOid'
4c212ed7f6b886c110d116be714e134e99f79cf0
$ kaji pr view 208 --json commits --jq '.commits[-1].committedDate'
2026-05-28T01:58:57Z

json.loads("4c212ed7...") は先頭 4 を数値として読んだ後 c で Extra data: line 1 column 2 (char 1) を投げる。トレースバックと完全一致。

3つの CLI 呼び出しのうち、スカラー抽出する2箇所が破綻する:

行	呼び出し	--jq	出力	json.loads
119	pr list	.[0]	JSON object	OK
151	pr view	.headRefOid	生 SHA 文字列	破綻
173	pr view	.commits[-1].committedDate	生日付文字列	破綻

KAJI_PR_ID 注入時は 151 行目で即死、未注入でも 173 行目で必ず死ぬため、経路に関わらず review-poll は毎回クラッシュする。

Expected Behavior（EB）

review_poll_entry.main() が PR / owner / repo / head_sha / head_committed_at を正しく解決し、codex_review_poll.main() に argv を委譲して exit code 0 を返す。設計書 draft/design/issue-204-feat-skill-frontmatter-exec-script-llm-s.md:289-328（§5 review_poll_entry 責務境界）が定める委譲契約どおりに動作すること。

再現手順（Steps to Reproduce）

1. provider.type=github、PR が存在する Issue（例: ci: release-please 資産を削除し /release skill 単独運用に統一する #195 / PR gh:208）
2. kaji run .kaji/wf/review-close.yaml 195（または review-cycle）で review-poll step に到達
3. review-poll step が exit code 1 で fail。stderr に上記 JSONDecodeError

なぜ #204 のテストで補足できなかったか（テストの死角）

修正実装時に正しいモック位置へ誘導するため、見逃しの構造的原因を記録する。

1. 全テスト層がバグの継ぎ目をモックで切除: バグの実体は _gh_json 内の json.loads(result.stdout)（生 CLI 出力 → Python 値の変換境界）。しかし Small (test_review_poll_entry.py) は _gh_json を丸ごと patch、Medium (test_runner_exec_script_dispatch.py) は execute_script を patch、Large (test_exec_script_subprocess_large.py) は dummy fixture script で dispatch 機構のみ検証。S/M/L のどの層も実 CLI 出力を json.loads に通す経路を一度も実行していない。
2. 設計書が CLI のワイヤフォーマット契約を未定義: 設計書 309-310 行目の ABORT 表は値レベル異常（headRefOid が空 / committedDate が空）のみ規定し、「--jq スカラーは生文字列＝json.loads 不可」というフォーマット契約を言語化していない。テストは ABORT 表を忠実に再現したため、表外の「パース失敗」が現れなかった。
3. _gh_json がオブジェクト抽出とスカラー抽出を混同: 単一ヘルパーで .[0]（JSON object）と .headRefOid（生スカラー）を処理。ヘルパー名自体が「常に JSON」という誤前提を内包。mock が native Python 値を返すため呼び出し種別の分岐が実行時にしか露呈しない。

完了条件

• 設計書で根本原因（_gh_json の json.loads がスカラー生出力に非対応）が特定されている
• 再現テストが 1 本以上追加され、修正前 FAIL・修正後 PASS を確認。モック位置は _gh_json ではなく subprocess.run 境界とし、stdout に実際の生 SHA / 生日付を流して「--jq スカラーは生文字列」契約をピン留めする
• スカラー抽出（headRefOid / committedDate）が json.loads を経由せず生文字列として解決される
• .[0] オブジェクト抽出（pr list）は従来どおり JSON parse されることを確認（デグレ無し）
• 同根の他の壊れ箇所の調査結果が設計書に記載されている（他 exec_script / scripts 配下で同型の json.loads 誤用が無いか）
• 影響モジュール全体のテストが green（tests/test_review_poll_entry.py 含む）
• make check 通過

影響範囲（初期評価）

• 影響するモジュール / コマンド: review-poll skill を含む全 workflow（review-cycle.yaml / review-close.yaml）。PR レビュー収束サイクルの自動ポーリングが完全に機能停止
• 深刻度: 操作不能（review-poll が一度も成功しない / feat: skill frontmatter に exec_script を追加して LLM 中継なし script step を実現する #204 導入以降の全 review-poll 実行で再現）
• 回避策の有無: 手動で /review-poll 相当の判定を行うか、review skill を直接呼ぶ fallback に頼るしかない

参考

• 関連 Issue/PR: feat: skill frontmatter に exec_script を追加して LLM 中継なし script step を実現する #204（exec_script shim 導入元）、ci: release-please 資産を削除し /release skill 単独運用に統一する #195（本バグが顕在化した workflow 実行）
• 設計書: draft/design/issue-204-feat-skill-frontmatter-exec-script-llm-s.md:289-328（§5 review_poll_entry 責務境界・委譲契約）
• 実装: kaji_harness/scripts/review_poll_entry.py:49-54（_gh_json）, :151, :173
• テスト: tests/test_review_poll_entry.py（_gh_json を全面 mock している死角）
• ログ: .kaji-artifacts/195/runs/2605281039/review-poll/stderr.log

[apokamo]

レディネスレビュー

全観点クリア。作業着手に進行可。

[apokamo]

設計書作成完了

設計書を作成しました。

成果物

• ファイル: draft/design/issue-209-fix-review-poll-exec-script-jq-json-load.md
• コミット: 29a0d80

設計の要点

1. What: _gh_json() がスカラー --jq 出力（生 SHA / 生日付）を json.loads してクラッシュする問題を、スカラー抽出専用ヘルパー _gh_raw()（stdout.strip() を返し json.loads しない）を新設して分離する。
2. Why: jq は string scalar をクォートなし生文字列で出力するため .headRefOid / .commits[-1].committedDate は json.loads 不可。.[0] の JSON object 抽出だけが parse 可能。単一ヘルパーが両者を混同していた根本原因に対し、責務を分離する最小侵襲修正。
3. Constraints: 公開 IF（main / env 契約 / codex_review_poll への委譲 argv）不変。.[0] の JSON 経路は _gh_json のまま据置（デグレ無し）。リファクタ混在なし。

同根調査

grep -rn "json.loads" + --jq 使用箇所を worktree 全体で調査。--jq スカラー × json.loads の組み合わせは review_poll_entry.py のみ。codex_review_poll.py:120 / github.py:113 / sync.py 他は JSON 入力で安全 → バグは局所化。

テスト戦略

• 再現テスト（Small・必須）: モック位置を _gh_json ではなく subprocess.run 境界に置き、stdout に実際の生 SHA 4c212ed7... / 生日付 2026-05-28T01:58:57Z を流して「--jq スカラーは生文字列」契約をピン留め。修正前 FAIL（line 151 で JSONDecodeError）→ 修正後 PASS。
• 同テストで pr list --jq .[0] の JSON 経路が従来どおり parse される（デグレ無し）ことも確認。
• 既存 3 テスト（_gh_json mock 依存）は subprocess 境界モックへ移行（修正後 scalar 抽出が _gh_raw 経由になり既存 mock が hit しなくなるため）。
• Medium / Large は不要（単一プロセス内の stdout→Python 値変換境界であり、Small で継ぎ目を完全再現）。

Self-Check 結果（design pre-handoff）

• 経路: main-session self-check
• 対象 commit: 29a0d80
• 参照 rubric: /issue-review-design SKILL.md Step 1.5 / Step 2 § type 重み付け / Step 2 § レビュー基準 1〜5

Step 1.5: Gate Check（一次情報）

• 判定: ✅
• 根拠: 「参照情報（Primary Sources）」にクラッシュログ・実装行・feat: skill frontmatter に exec_script を追加して LLM 中継なし script step を実現する #204 設計書 §5・本設計時の再現実行・同根調査コマンドを URL/パス付きで記載。各行に根拠引用あり。

Step 2 § type 重み付け（type: bug）

• 判定: ✅
• 根拠: OB/EB を一次情報（stderr.log / §5 委譲契約）で裏付け、再現手順最小、根本原因「なぜ」（jq の raw scalar 出力と単一ヘルパーの誤前提）、同根の他の壊れ箇所を調査表で網羅、再現テスト必須を明記。

Step 2 § レビュー基準 1〜5

• 1. 抽象化と責務の分離: ✅（_gh_raw / _gh_json の責務分離表）
• 2. インターフェース設計: ✅（公開 IF 不変を明示、内部ヘルパーのみ変更）
• 3. 信頼性とエッジケース: ✅（スカラー空 → 既存 ABORT 経路維持、catastrophic は raise 維持）
• 4. 検証可能性: ✅（subprocess 境界での再現テスト、S/M/L の採否理由を testing-convention に紐付け）
• 5. 影響ドキュメント: ✅（全行「なし」を理由付きで評価。feat: skill frontmatter に exec_script を追加して LLM 中継なし script step を実現する #204 設計書は改訂不要と判断根拠を記載）

補完した項目

• 無し（初回作成時に全観点を充足）

Self-Check Verdict

• Yes — handoff 可

完了条件の段階確認

設計段階で確認可能な完了条件:

• 設計書で根本原因が特定されている: ✅ 「根本原因」セクションで _gh_json の json.loads がスカラー生出力に非対応である点を 3 呼び出しの --jq 差異表で特定
• 同根の他の壊れ箇所の調査結果が設計書に記載: ✅ 「同根の他の壊れ箇所の調査」表
• 再現テスト追加（修正前 FAIL・修正後 PASS、モック位置は subprocess.run 境界）: テスト戦略で方針確定、実装段階で実装
• スカラー抽出が json.loads を経由せず生文字列で解決: 方針確定、実装段階で実装
• .[0] オブジェクト抽出は従来どおり JSON parse（デグレ無し）: 方針確定、再現テストで確認予定
• 影響モジュール全体のテストが green / make check 通過: 実装段階で確認予定

次のステップ

/issue-review-design 209 でレビューをお願いします。

[apokamo]

設計レビュー結果

参照した一次情報

情報源	確認結果
.kaji-artifacts/195/runs/2605281039/review-poll/stderr.log	✅ review_poll_entry.py:151 → _gh_json() → json.loads(result.stdout) → JSONDecodeError: Extra data: line 1 column 2 のクラッシュを確認
kaji_harness/scripts/review_poll_entry.py	✅ pr list --jq '.[0]'、pr view --jq '.headRefOid'、pr view --jq '.commits[-1].committedDate' の 3 経路が同じ _gh_json() を使っていることを確認
draft/design/issue-204-feat-skill-frontmatter-exec-script-llm-s.md:289-328	✅ review_poll_entry は env→argv 変換と PR/head 解決のみ担当し、codex_review_poll.main([...]) に --head-sha / --head-committed-at を委譲する契約であることを確認
kaji_harness/cli_main.py:_format_jq_results / tests/test_dispatcher.py	✅ kaji の --jq 互換出力は string scalar を quote なし raw 文字列、object/array を JSON として出す仕様であることを確認
rg -n -- '--jq' kaji_harness tests .kaji .claude / rg -n 'json\\.loads' kaji_harness tests	⚠️ kaji_harness/ 全体には cli_main.py 等の --jq 利用が存在する。scripts 配下では review_poll_entry.py に局所化しているが、設計書の「worktree 内全体」「kaji_harness/ 全体」の表現とは差異あり

概要

設計の修正方針（JSON object 抽出用 _gh_json() と raw scalar 抽出用 _gh_raw() を分離し、公開 IF と codex_review_poll 契約は不変）は妥当です。OB/EB、根本原因、ABORT 経路維持も概ね一次情報と整合しています。

ただし、テスト戦略の主軸が Issue の完了条件を満たせない形になっています。type:bug では再現テストと同根調査の精度を高く見るため、実装着手前に修正が必要です。

指摘事項 (Must Fix)

• MF-1: 再現テスト案が headRefOid の raw scalar 経路を実行しない

  • 設計書の Small 再現テスト案では、pr list --jq .[0] の stdout に {"number": 42, "headRefOid": "..."} を返すとしています。
  • しかし現行実装では pr_list.get("headRefOid") が非空なら if not head_sha: に入らず、pr view --jq .headRefOid は呼ばれません。
  • そのため、このテストは commits[-1].committedDate の raw scalar は検証しても、Issue 完了条件の「スカラー抽出（headRefOid / committedDate）が json.loads を経由せず生文字列として解決される」のうち headRefOid 側をピン留めできません。
  • 一次情報との関連: review_poll_entry.py の pr_list 経路では head_sha = (pr_list.get("headRefOid") or "").strip() 後、空のときだけ pr view --jq .headRefOid を呼ぶ。
  • 修正案: 成功系の再現テストは KAJI_PR_ID を注入する、または pr list の headRefOid を空にして pr view --jq .headRefOid を必ず通す形にする。加えて committedDate raw scalar も同じテストまたは別テストで通す。

• MF-2: 同根調査のスコープ表現が実際の検索結果と矛盾している

  • 設計書は「worktree 内全体」「grep -rn "json.loads" kaji_harness/ と --jq 使用箇所」「--jq を使うのは review_poll_entry.py の 3 箇所のみ」と読める記述になっています。
  • 実際には kaji_harness/cli_main.py などにも --jq 利用があります。Issue 完了条件が求めているのは「他 exec_script / scripts 配下で同型の json.loads 誤用が無いか」なので、結論自体は scripts 配下に限定すれば妥当ですが、現在の記述は調査範囲を過大に書いています。
  • 一次情報との関連: rg -n -- '--jq' kaji_harness で cli_main.py の --jq 実装が複数ヒットする。一方、kaji_harness/scripts/ 配下の --jq は review_poll_entry.py のみ。
  • 修正案: 「scripts / exec_script 対象に限定した同根調査」と明記し、cli_main.py の --jq は raw 出力仕様の一次情報として扱う、または全 kaji_harness/ 調査を主張するなら cli_main.py を除外できる理由を明記する。

改善提案 (Should Fix)

• SF-1: raw scalar 出力仕様の一次情報を Primary Sources に追加する
  • 設計書は raw scalar を前提にしていますが、Primary Sources には kaji_harness/cli_main.py:_format_jq_results や tests/test_dispatcher.py::test_view_json_with_jq_emits_raw_string が入っていません。
  • 今回の根本原因は「CLI の wire format と Python parse 境界」の不整合なので、kaji 側 --jq raw 出力仕様を一次情報に加えると実装者が迷いません。

完了条件の段階確認

• 根本原因の特定: ✅ _gh_json() が object と scalar の --jq 出力を混同している点は設計書で説明済み
• 再現テスト追加方針: ❌ MF-1 により、現案では headRefOid raw scalar 成功経路を通らない
• スカラー抽出を json.loads から分離する方針: ✅ _gh_raw() 新設方針は妥当
• .[0] object 抽出のデグレ防止方針: ✅ _gh_json() 据置と stdout JSON object テスト方針あり
• 同根調査結果の記載: ❌ MF-2 により、調査対象の表現修正が必要
• テスト green / make check: 実装段階で確認

判定

[ ] Approve (実装着手可)
[x] Changes Requested (設計修正が必要)

[apokamo]

設計修正報告

レビュー指摘 MF-1 / MF-2 / SF-1 に対応しました。コミット: f1f3772

対応済み

• MF-1: 再現テスト案が headRefOid の raw scalar 経路を実行しない

  • 修正: 成功系の再現テスト（検証観点 1）で pr list --jq .[0] の出力から headRefOid を除外（'{"number": 42, "headRefName": "feat/x"}'）し、line 145 で head_sha を空にして line 150〜163 の pr view --jq .headRefOid（修正後 _gh_raw）を必ず通すよう明記。修正前 FAIL 行を「line 151（headRefOid 補完経路）」に特定。
  • 加えて検証観点 1b を新設し、KAJI_PR_ID 注入（または pr list に headRefOid 含有）で headRefOid を pr list 経路から取得する従来分岐も網羅。committedDate raw scalar は line 173 で常に呼ばれるため同テストで同時検証。

• MF-2: 同根調査のスコープ表現が実検索結果と矛盾

  • 修正: 同根調査を kaji_harness/scripts/ 配下（exec_script shim 群）に限定すると明記。cli_main.py の --jq（_format_jq_results）は raw 出力の生成側であり「出力を消費する側」でないため調査対象外、むしろワイヤフォーマット契約の一次情報である旨を追記。検索コマンドを rg 表記に統一。

• SF-1: raw scalar 出力仕様の一次情報を Primary Sources に追加

  • 修正: kaji_harness/cli_main.py:1171（_format_jq_results）/ :1119（gh --jq 互換 raw 出力 docstring）と tests/test_dispatcher.py:445（test_view_json_with_jq_emits_raw_string）を Primary Sources に追加。「--jq スカラー＝生文字列」前提の一次根拠として明示。

議論/見送り

なし（全指摘を受け入れ）。

次のステップ

/issue-verify-design 209 で修正確認をお願いします。

[apokamo]

設計修正確認結果

修正項目の確認

指摘項目	状態	理由・根拠
MF-1: 再現テスト案が headRefOid の raw scalar 経路を実行しない	✅ OK	設計書の Small テスト戦略に「pr list の出力に headRefOid を含めない」と明記され、line 145 で head_sha を空にして line 150〜163 の pr view --jq .headRefOid を必ず呼ぶ構成になっている。さらに検証観点 1 で生 SHA / 生日付 stdout を subprocess.run fake から返し、委譲 argv にそのまま入ることを確認する方針が記載されている。検証観点 1b で pr list 由来の従来分岐も別途網羅している。
MF-2: 同根調査のスコープ表現が実際の検索結果と矛盾している	✅ OK	設計書の「同根の他の壊れ箇所の調査」で、主対象を kaji_harness/scripts/ 配下（exec_script shim 群）に限定すると明記された。cli_main.py の --jq は raw 出力の生成側であり、消費側の同根調査対象外だが wire format の一次情報として扱う、という除外理由も記載されている。

反論への検討結果

見送り項目	検討結果	理由
なし	✅ 受け入れ	設計修正報告で MF-1 / MF-2 / SF-1 は全て受け入れと明記され、見送り・議論項目はない。

新規発見事項（参考情報）

注意: 以下は今回の判定には影響しません。verify の対象は前回指摘事項のみです。

発見事項	重要度	推奨対応
なし	-	-

判定

[x] Approve (実装着手可)
[ ] Changes Requested (再修正が必要)

次のステップ

/issue-implement 209 で実装を開始してください。

[apokamo]

Pre-Handoff Review

• 経路: subagent
• 起動 agent: kaji-code-reviewer
• 対象 commit: 5fc9f49

1. 設計書整合

• 判定: ✅
• 根拠: 設計書「内部ヘルパーの変更」が規定する _gh_raw(args, cwd) -> str（json.loads 非経由）の新設・差し替え 2 箇所（headRefOid 補完 / committedDate）・.[0] object 抽出（pr list）の _gh_json 据置・isinstance(str) 分岐削除・ABORT 経路維持がすべて実装と一致。影響ドキュメント全項目「なし」と変更ファイル範囲も一致。

2. テスト証跡

• 判定: ✅
• 根拠: 設計書テスト戦略どおり Small のみ（Medium/Large 不要）。検証観点 1（headRefOid raw scalar 経路強制通過・MF-1 対応）/ 1b（pr list object 由来 head_sha で pr view スキップ・デグレ無し）/ 2（スカラー空→ABORT）を実装。既存 3 テストを _gh_json patch → subprocess.run 境界 patch へ移行。フルスイート 1560 passed / 1 skipped（baseline 1558 → +2、regression 0）。Red Phase で修正前 3 テストが JSONDecodeError（Extra data: line 1 column 2/5）で FAIL することをログ一致で確認。

3. Scope 混在

• 判定: ✅
• 根拠: 変更は kaji_harness/scripts/review_poll_entry.py と tests/test_review_poll_entry.py のみ。設計書「review_poll_entry.py のみ変更」と合致。codex_review_poll / providers / cli_main への変更なし。最小侵襲 bugfix + 必須テスト追加のみ。

4. auto-close 規約

• 判定: ✅
• 根拠: 本文・実装コメント・docstring に hazard pattern（close/fix/resolve/implement の直後 #数字）なし。

指摘事項

特になし。

Pre-Handoff Review Verdict

• Yes

[apokamo]

実装完了報告 (TDD)

設計（draft/design/issue-209-fix-review-poll-exec-script-jq-json-load.md）に基づき、bug 再現テスト先行で実装しました。

実施内容

• 再現テスト先行 (Red): subprocess.run 境界モックで生 SHA / 生日付を流し、修正前は _gh_json の json.loads が JSONDecodeError（Extra data: line 1 column 2/5、クラッシュログと完全一致）で FAIL することを確認
• 実装 (Green): kaji_harness/scripts/review_poll_entry.py にスカラー抽出用 _gh_raw() を新設。headRefOid 補完（line 166）と committedDate 取得（line 187）を _gh_json → _gh_raw に差し替え、json.loads を経由せず生文字列で解決。.[0] object 抽出（pr list）は _gh_json のまま据置
• テスト移行: 既存 3 テスト（pr_list_empty / head_sha_empty / committed_at_empty）と argv 委譲テストを _gh_json patch → subprocess.run 境界 patch へ移行。新規 TestScalarRawResolution で生スカラー契約をピン留め

テスト結果

$ pytest tests/test_review_poll_entry.py
18 passed in 0.77s

$ pytest  (フルスイート)
1560 passed, 1 skipped in 181.33s

項目	結果
テスト総数	1561
passed	1560
failed	0 (baseline: 0, regression: 0)
errors	0
skipped	1
review_poll_entry テスト	18 passed

baseline（修正前 commit f1f3772）= 1558 passed / 1 skipped。+2 は新規 TestScalarRawResolution の 2 テスト。regression 0。

品質チェック結果

$ ruff check kaji_harness/ tests/
All checks passed!
$ ruff format kaji_harness/ tests/
105 files left unchanged
$ mypy kaji_harness/
Success: no issues found in 29 source files

変更ファイル

• kaji_harness/scripts/review_poll_entry.py: _gh_raw() 新設、スカラー抽出 2 箇所を差し替え、_gh_json docstring に object/array 専用である旨を追記
• tests/test_review_poll_entry.py: モック位置を subprocess.run 境界へ統一、TestScalarRawResolution 追加

Pre-Handoff Review 結果

• Pre-Handoff Review コメント数 (PHR_COUNT): 1
• 最終 verdict: Yes
• 最新の ## Pre-Handoff Review コメント: fix: review-poll exec_script が --jq スカラー出力を json.loads して毎回クラッシュ #209 (comment)

完了条件の段階確認

• 設計書で根本原因（_gh_json の json.loads がスカラー生出力に非対応）が特定されている: ✅
• 再現テスト追加、修正前 FAIL・修正後 PASS。モック位置は subprocess.run 境界、生 SHA / 生日付を流す: ✅
• スカラー抽出が json.loads を経由せず生文字列で解決される: ✅ _gh_raw
• .[0] object 抽出は従来どおり JSON parse（デグレ無し）: ✅ test_head_sha_from_pr_list_skips_pr_view
• 同根の他の壊れ箇所の調査結果が設計書に記載: ✅ 設計書「同根の他の壊れ箇所の調査」（review_poll_entry.py のみに局所化）
• 影響モジュール全体のテストが green: ✅
• make check 通過（ruff/format/mypy/pytest）: ✅

次のステップ

/issue-review-code 209 によるコードレビューをお願いします。

[apokamo]

コードレビュー結果

概要

Approve。--jq スカラー出力を JSON parse しない責務分離になっており、設計書・完了条件・bug type の追加観点に対して不足は見つかりませんでした。

独立テスト実行結果

項目	結果
ruff check / ruff format --check / mypy	PASS
pytest 総数 / passed / failed / errors / skipped	1561 / 1560 / 0 / 0 / 1
baseline failure 一致	0 件（Baseline Check コメントなし。全テスト exit 0 のため regression なし）
新規 FAILED/ERROR (regression)	0 件

実行コマンド:

source .venv/bin/activate && ruff check kaji_harness/ tests/ && ruff format --check kaji_harness/ tests/ && mypy kaji_harness/
source .venv/bin/activate && pytest

結果:

All checks passed!
105 files already formatted
Success: no issues found in 29 source files
1560 passed, 1 skipped in 181.32s (0:03:01)

type 別追加観点の判定

判定対象 type: type:bug

観点	該当	判定	根拠
A. IF 契約の忠実性	feat	—	bug issue のため対象外
B. 再現テスト Red→Green	bug	✅	実装完了報告に、修正前は _gh_json の json.loads が生 SHA / 生日付で JSONDecodeError となる Red、修正後は tests/test_review_poll_entry.py と full pytest Green の証跡あり。追加テスト TestScalarRawResolution は subprocess.run 境界で生 stdout を流している
C. 同根欠陥の波及修正	bug	✅	設計書「同根の他の壊れ箇所の調査」で kaji_harness/scripts/ 配下の --jq 使用が review_poll_entry.py に局所化されていることを確認済み。実装差分も当該 shim とテストに限定
D. 振る舞い非変更の保証	refactor	—	bug issue のため対象外
E. 改善指標の達成	refactor	—	bug issue のため対象外
F. Scope 混在禁止	feat/bug/refactor	✅	差分は kaji_harness/scripts/review_poll_entry.py、tests/test_review_poll_entry.py、設計書のみ。codex_review_poll / provider / CLI 本体への変更なし

指摘事項 (Must Fix)

なし。

改善提案 (Should Fix)

なし。

完了条件の段階確認

コードレビュー段階の完了条件に対する充足判定:

• 設計書で根本原因が特定されている: ✅ _gh_json が object/array と scalar の --jq 出力を混同していた点を設計書に記載
• 再現テスト追加・修正前 FAIL / 修正後 PASS: ✅ TestScalarRawResolution::test_raw_sha_and_date_delegate_without_json_parse が subprocess.run 境界で生 SHA / 生日付を流し、実装報告に Red→Green 証跡あり
• スカラー抽出が json.loads を経由せず生文字列で解決される: ✅ _gh_raw() を新設し、.headRefOid / .commits[-1].committedDate の 2 箇所を _gh_raw に差し替え
• .[0] object 抽出は従来どおり JSON parse: ✅ pr list --jq '.[0]' は _gh_json のまま。test_head_sha_from_pr_list_skips_pr_view で pr list object 由来の headRefOid 分岐を確認
• 同根の他の壊れ箇所の調査結果が設計書に記載: ✅ kaji_harness/scripts/ 配下を対象に確認済み
• 影響モジュール全体のテストが green: ✅ pytest tests/test_review_poll_entry.py は実装報告で 18 passed、レビュワー独立実行の full pytest も 1560 passed / 1 skipped
• make check 通過相当: ✅ ruff check / ruff format --check / mypy / pytest を独立実行して全 PASS

判定

[x] Approve (修正なしでマージ可)
[ ] Changes Requested (要修正)

[apokamo]

最終チェック結果

前段証跡の確認

ステップ	コメント有無	最新判定
issue-design	✅	設計書作成済み (29a0d80)
issue-review-design	✅	Changes Requested (MF-1/MF-2/SF-1)
issue-fix-design → issue-verify-design	✅	Approve（全指摘解消、f1f3772）
issue-implement	✅	full pytest 1560 passed / 1 skipped、ruff/format/mypy PASS
issue-review-code	✅	Approve（独立実行 1560 passed / 1 skipped、Must/Should なし）

完了条件の充足状態

条件	充足	確認元
設計書で根本原因が特定されている	✅	設計書「根本原因」+ review-design
再現テスト追加（subprocess.run 境界モック、生 SHA/生日付）	✅	TestScalarRawResolution、implement Red→Green
スカラー抽出が json.loads を経由せず生文字列で解決	✅	_gh_raw() 新設（review_poll_entry.py）
.[0] object 抽出は従来どおり JSON parse（デグレ無し）	✅	_gh_json 据置 + test_head_sha_from_pr_list_skips_pr_view
同根の他の壊れ箇所の調査結果が設計書に記載	✅	設計書「同根の他の壊れ箇所の調査」（scripts 配下に局所化）
影響モジュール全体のテストが green	✅	final-check 独立実行 1560 passed / 1 skipped
make check 通過	✅	final-check で make check 実行 PASS

品質ゲート

ゲート	結果	備考
make check	PASS	ruff / format / mypy / pytest 一括（1560 passed, 1 skipped）
変更タイプ固有検証	N/A	通常変更（コード＋テスト）、追加検証不要

docs 整合

• 設計書昇格: 不要（bug の局所修正。新規 ADR 相当の決定なし。影響ドキュメント評価は設計書で全行「なし」）
• docs 更新: 不要（公開 IF / env 契約 / 委譲 argv 不変）

Issue 本文更新

• チェックボックス更新: 実施（完了条件 7 件すべて [x]）
• 設計書添付: 実施（NOTE ブロック直下に draft/design/ 版を <details> で添付）

判定

PASS

[apokamo]

Issue クローズ完了

項目	状態
PR	マージ済み（#210）
worktree	削除済み（kaji-fix-209）
ローカルブランチ	削除済み
リモートブランチ	削除済み
main	最新化済み
Issue	クローズ済み