Repository navigation
task/kiro awaiting - #284
Merged
Merged
Conversation
…iro-cli has no waiting-on-user hook Probed kiro-cli 2.32.0. Its agent-config `hooks` field accepts only agentSpawn, userPromptSubmit, preToolUse, postToolUse and stop. It has no ask-the-user tool; its one confirm tool, switch_to_execution, lives in the built-in kiro_planner agent, which no task-force agent includes. No hook payload field marks a tool as waiting for approval. So there is nothing to wire `radio awaiting` to, and a waiting kiro worker never shows the question-mark tab or trips worker-awaiting. Docs: a "No awaiting state on kiro" section and a kiro hook-table row in the README, plus the monitor Attention paragraph, the schema doc, .claude/gh-workflow.md and all seven steering templates. A doc test in tests/task_monitor_snapshot.bats pins the wording and the version. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… the doc test Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Owner
Author
Spec complianceIssue #278 asked to investigate whether kiro-cli exposes a "waiting on the user" hook. If it does, wire it. If it doesn't, document the gap wherever
Code-review findingsNo code changes, so there is nothing to find on correctness or security.
Verdictclean. The investigation is documented with evidence and a version, the docs are consistent across all loadouts, and the doc test pins them. Nothing to fix in this PR. Merge is PM's call. |
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 workers can't show the
❓"waiting on you" tab or trip the monitor'sworker-awaitingsignal, because kiro-cli 2.32.0 has no hook that fires while it waits on the user. There is nothing to wire, so this PR documents the gap with the evidence and the version checked, wherever the docs describe❓/worker-awaiting.Closes #278
Findings (kiro-cli 2.32.0)
hooksfield accepts exactlyagentSpawn,userPromptSubmit,preToolUse,postToolUse,stop(the trigger enum in the binary). No permission or notification trigger, unlike claude'sPermissionRequest.switch_to_execution("Ready to exit [plan] agent…"), belongs to kiro's built-inkiro_planneragent. No task-force kiro agent lists it intools, and radio's hooks don't run insidekiro_planner.hook_event_name,cwd,session_id,tool_name,tool_input,tool_response,assistant_response, with no approval-pending marker.~/.kiro/sessions/cli/*.jsonlrecord onlyPrompt/AssistantMessage/ToolResults/Clear/Compaction. The V3 (KAS) engine has anawaiting_approvaltool-call status in its session files, but it's unverified whether that's written while the prompt is up, and using it would mean polling rather than a hook.How I probed
"/Applications/Kiro CLI.app/Contents/MacOS/kiro-cli-chat" --version→kiro-cli-chat 2.32.0.stringsover that binary (trigger enum, hook payload keys, tool names, status enums). Read-onlyls/jqover~/.kiro/sessionsto see the session-log shapes.kiro-cli-chatdirectly withenv -u TASK_FORCE_ROLE -u TASK_FORCE_PM_ROLE -u ZELLIJ -u ZELLIJ_SESSION_NAME HOME=<scratch> TASK_FORCE_HOME=<scratch>/tf, in a scratch workspace with a throwaway.kiro/agents/probe.jsonwhose five hooks each appended the trigger name to a log file.execute_bashwas left out ofallowedToolsto force an approval prompt. The onlysettingswrite (chat.enableAutoAgentUpgrade false) went to the scratch HOME.~/Library/Application Support/kiro-cli/data.sqlite3into the scratch HOME. I deleted it right after the probe. Afind ~/.kiro -newercheck afterwards showed nothing in the real~/.kirochanged.stopfire before the prompt?) could not be observed. The docs say "most likelybusy" and call it unverified. A side observation: on that failed request,userPromptSubmitfired andstopdid not, so a kiro tab may staybusyafter an API error. That's out of scope, but it may be worth a ticket.Changes
docs/task-monitor-schema.md,.claude/gh-workflow.mdand all seven steering templates: "claude-only for now" becomes the gap plus the kiro-cli version checked.tests/task_monitor_snapshot.bats: a doc test pins the wording, the version and the README anchor. It passes on the tree, and it went red when I restored the old kiro-local doc and when I broke the anchor.Green
./run_tests.sh: 1461/1461 ok. The new test was also run on its own: ok.tools/check-drift.sh: 15 groups OK. No shell files changed.🤖 Generated with Claude Code