Fix: Kiro版NDFエージェントにtools宣言を追加 (v4.20.1) - #60
Merged
Conversation
Kiro CLI は tools を宣言していない agent config を、ツールを1つも持たない エージェントとして読み込む。installer が生成する default エージェントには tools が無く、skill が SKILL.md を読むことも git / gh を実行することも できない状態だった。skill 一覧の表示は動くため、一見正常に見えていた。 - agents/default.json.template に "tools": ["*"] を追加 - runtime smoke test に tools 宣言の検査を追加。従来はファイルの生成有無 しか見ておらず、この欠落を検出できなかった 検証: - tools を外した状態で smoke test が exit 1 になることを確認 - 修正版 installer で生成したエージェントが 13 ツールを持ち、symlink 越しの SKILL.md 読み込みと git コマンド実行が動作することを kiro-cli 2.16.1 で確認 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wa3ZT8WCqEDTyybpZ6Zncw
13 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
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
Kiro CLI は
toolsを宣言していない agent config を、ツールを1つも持たないエージェントとして読み込みます。installer が生成するdefaultエージェントにはtoolsが無く、skill が SKILL.md を読むことも git / gh を実行することもできない状態でした。skill 一覧の表示は動くため一見正常に見えており、既存の smoke test も
.kiro/agents/default.jsonの生成有無しか検査していなかったため検出できていませんでした。kiro-cli 2.16.1 での実測:
kiro_defaultdefaultdefault変更内容
plugins/ndf-kiro/agents/default.json.templateに"tools": ["*"]を追加tests/runtime-smoke/assertions/assert-plugin-files.shに、生成された agent config がtoolsを宣言しているかの検査を追加toolsの値を["*"]としたのは、Claude 版の NDF エージェントが全ツール利用可であることに揃えるためです。allowedToolsは宣言していないため、ツール実行時の承認は従来どおり都度求められます。Test plan
bash scripts/validate-runtime-plugins.shが passbash scripts/runtime-smoke-test.sh --runtime kiroが passtoolsを外した状態で smoke test が exit 1 になることを確認(追加した検査が実際に不具合を捕まえる).kiro/skills/review-branch/SKILL.mdを読み、内容を要約できることを確認git branch --show-currentを実行できることを確認(/ndf:pr等が必要とするシェル実行系)bash plugins/ndf-kiro/install.sh後にkiro-cli chatが skill を読めることやらないこと
allowedToolsによる自動承認設定の追加(挙動が変わるため別途判断)