Skip to content

feat(orchestrator): require actionable approval reasons - #311

Merged
version-1 merged 3 commits into
mainfrom
feat/describe-approval-requests
Aug 14, 2026
Merged

feat(orchestrator): require actionable approval reasons#311
version-1 merged 3 commits into
mainfrom
feat/describe-approval-requests

Conversation

@version-1

Copy link
Copy Markdown
Owner

Summary

Require agents to provide an actionable reason before requesting approval for command execution or file changes.

主な変更点

  • 初回の task work prompt に、承認対象・対象範囲・必要性・想定効果を明記する契約を追加
  • resume および multi-turn の継続 prompt にも同じ契約を追加
  • 初回・継続 prompt の回帰テストを追加
  • system prompt の設計ドキュメントを英語・日本語で同期更新

やったこと

  • reason が null、空、または曖昧な承認要求にならないようエージェント向け指示を具体化
  • command の場合は command と working directory、file change の場合は file paths を対象範囲として要求
  • app-server の runtime validation や Tasq の approval denial policy は変更せず、prompt contract に限定

動作確認

  • go test ./internal/orchestrator/...
  • go test ./internal/orchestrator/runner
  • git diff --check origin/main...HEAD

レビュー & 動作確認 チェックリスト

  • 初回 prompt が承認理由の必須4要素を含む
  • 継続 prompt が同じ承認理由契約を含む
  • 日英ドキュメントが同期している
  • Reviewer agent による再レビューで High / Medium / Low なし

推奨テスト計画

  • 承認が必要な command execution を発生させ、blocker comment の payload に具体的な reason が含まれることを確認する
  • file change approval でも対象 file paths と expected effect が reason に含まれることを確認する
  • resume 後または multi-turn の後続 turn でも同じ形式になることを確認する

その他気になることや相談ごと

  • 今回は prompt contract のみを対象としています。app-server 側で空の reason を拒否する runtime validation は必要になった場合の別変更です。

Require agents to describe the operation, target scope, necessity, and expected effect before requesting approval. Apply the contract to initial and continuation prompts, with synchronized tests and documentation.
@version-1
version-1 merged commit de39758 into main Aug 14, 2026
5 checks passed
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.

1 participant