Skip to content

Fix: Kiro版NDFエージェントにtools宣言を追加 (v4.20.1) - #60

Merged
takemi-ohama merged 1 commit into
mainfrom
fix/kiro-agent-tools
Aug 6, 2026
Merged

Fix: Kiro版NDFエージェントにtools宣言を追加 (v4.20.1)#60
takemi-ohama merged 1 commit into
mainfrom
fix/kiro-agent-tools

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

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組み込み kiro_default 13個
修正前の NDF default 0個
修正後の NDF default 13個

変更内容

  • plugins/ndf-kiro/agents/default.json.template"tools": ["*"] を追加
  • tests/runtime-smoke/assertions/assert-plugin-files.sh に、生成された agent config が tools を宣言しているかの検査を追加
  • バージョンを v4.20.0 → v4.20.1 に更新(plugin.json ×2 / marketplace.json / README.md / AGENTS.md)

tools の値を ["*"] としたのは、Claude 版の NDF エージェントが全ツール利用可であることに揃えるためです。allowedTools は宣言していないため、ツール実行時の承認は従来どおり都度求められます。

Test plan

  • bash scripts/validate-runtime-plugins.sh が pass
  • bash scripts/runtime-smoke-test.sh --runtime kiro が pass
  • template から tools を外した状態で smoke test が exit 1 になることを確認(追加した検査が実際に不具合を捕まえる)
  • 修正版 installer で生成したエージェントが 13 ツールを持つことを kiro-cli 2.16.1 で確認
  • 同エージェントが symlink 越しに .kiro/skills/review-branch/SKILL.md を読み、内容を要約できることを確認
  • 同エージェントが git branch --show-current を実行できることを確認(/ndf:pr 等が必要とするシェル実行系)
  • レビュアー環境で bash plugins/ndf-kiro/install.sh 後に kiro-cli chat が skill を読めること

やらないこと

  • allowedTools による自動承認設定の追加(挙動が変わるため別途判断)
  • Claude / Codex 版の設定変更(本件は Kiro 版のみの不具合)

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
@takemi-ohama
takemi-ohama merged commit de46d2e into main Aug 6, 2026
6 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