fix: Fix issue where assign invocation intermittently drops the message#21
Open
patricka3125 wants to merge 5 commits intomainfrom
Open
fix: Fix issue where assign invocation intermittently drops the message#21patricka3125 wants to merge 5 commits intomainfrom
patricka3125 wants to merge 5 commits intomainfrom
Conversation
…lse positives (awslabs#142) * fix WAITING_USER_ANSWER regex * formatting --------- Co-authored-by: haofeif <56006724+haofeif@users.noreply.github.com>
…arent restrictions (awslabs#141) (awslabs#144) _resolve_child_allowed_tools() treated ["*"] the same as None (no opinion), causing restricted parents to override a child's explicit unrestricted config. Split the condition so ["*"] returns None (unrestricted) while None inherits. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
awslabs#146) * fix(launch): clarify confirmation prompt — show Role, remove Blocked, distinguish Y vs --yolo The launch confirmation prompt was misleading: it showed "Blocked: (none)" for providers that enforce restrictions natively (kiro-cli, q-cli), and the --yolo hint didn't make clear that users need to exit and re-run. Now the prompt shows Role and Allowed in CAO vocabulary, and clearly explains that [Y] keeps restrictions while [--yolo] overrides everything. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(launch): add --auto-approve flag to skip confirmation prompt Adds --auto-approve to skip the confirmation prompt while keeping tool restrictions enforced. Useful for automated flows, scripted launches, and agent-to-agent workflows where --yolo is too permissive. Addresses review feedback on PR awslabs#146 re: automated flows hanging on the Y/n prompt. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: clarify TOOL_MAPPING role and why only 3 providers need it Rewrite the "How Tool Restrictions Are Enforced" section to explain why TOOL_MAPPING only covers Claude Code, Copilot CLI, and Gemini CLI (they need translation to native tool names), while Kiro CLI, Q CLI, Kimi CLI, and Codex accept CAO vocabulary directly. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(launch): clarify prompt hints — say "relaunch with" not bracket options The prompt hints like [--auto-approve] and [--yolo] implied users could type them at the Y/n prompt. Changed to "relaunch with --auto-approve" and "relaunch with --yolo" to make clear these are CLI flags for the next launch, not options at the current prompt. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <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
2 participants
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.
Address issue #20