整備: ハーネス設計の穴を塞ぐ(サブエージェント契約の一本化ほか)#10
Merged
Merged
Conversation
レビューで見つかったハーネスエンジニアリング上の不整合を修正: - dango-review の allowed-tools に Agent を追加。本文が並列サブエージェントを 指示しているのにツール未宣言で指示倒れになっていたのを解消 - サブエージェント契約を「親が section を Read→本文をプロンプト展開」に一本化。 jp-reviewer 側の「パス推測」分岐を削除し、対象プロジェクトでの空振りを防止 - dango-run のゲート判断の前に決定ログを読み戻す導線を追加(状態→判断のループ化) - 実装⇄レビューの往復上限(最大3回)とエスカレーションを追加し無限ループを防止 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
何を変えたか
ハーネスエンジニアリング観点のレビューで見つかった、実行時の実害につながる不整合を修正:
allowed-toolsにAgentを追加 — 本文が並列サブエージェントを指示しているのにツール未宣言で、実行時に Agent を呼べず指示倒れになっていたsections/{観点}.mdを Read → 本文をプロンプトに展開して渡す」に固定。jp-reviewer側の「パス推測」分岐を削除し、対象プロジェクト側で skills パスが解決できず空振りする失敗を防止dango-runのゲート判断の 前に 決定ログを読み戻す導線を追加(状態→判断のループにする。書きっぱなし防止)なぜ(背景・課題)
プログレッシブ・ディスクロージャやコンテキスト分離は取り込めていたが、(1) ツール宣言と本文の不一致、(2) サブエージェントへの受け渡し契約の曖昧さ、(3) 決定ログが書きっぱなしでゲートに戻らない、(4) 往復の上限なし、というハーネス設計上の穴があった。特に (1)(2) は実行時に動かない/空振りする実害がある。
動作確認
node bin/validate.mjs→ 検証OK(7役割)レビュー観点(日本語)
🤖 Generated with Claude Code