Skip to content

Inbox Service#2

Merged
tuanknguyen merged 24 commits into
mainfrom
inbox
Oct 8, 2025
Merged

Inbox Service#2
tuanknguyen merged 24 commits into
mainfrom
inbox

Conversation

@tuanknguyen

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:
Add inbox service for async send message between terminals

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@tuanknguyen tuanknguyen requested a review from haofeif October 8, 2025 03:09
@tuanknguyen tuanknguyen self-assigned this Oct 8, 2025
@tuanknguyen tuanknguyen merged commit 79a0d5b into main Oct 8, 2025
2 checks passed
@tuanknguyen tuanknguyen deleted the inbox branch October 8, 2025 03:10
haofeif added a commit that referenced this pull request Mar 7, 2026
…ion (#94)

* fix: Fix regex to catch Claude Code Processing spinner (#2)

* Add custom agent store

* fix: Fix failing Q-CLI working directory tests

* unstage custom change

* fix formatting

* add pre-session cleanup fixture

---------

Co-authored-by: haofeif <56006724+haofeif@users.noreply.github.com>
aziz0220 pushed a commit to aziz0220/cli-agent-orchestrator that referenced this pull request Mar 8, 2026
…ion (awslabs#94)

* fix: Fix regex to catch Claude Code Processing spinner (awslabs#2)

* Add custom agent store

* fix: Fix failing Q-CLI working directory tests

* unstage custom change

* fix formatting

* add pre-session cleanup fixture

---------

Co-authored-by: haofeif <56006724+haofeif@users.noreply.github.com>
haofeif added a commit that referenced this pull request Apr 9, 2026
- Add "Kiro is working" ghost text as positive PROCESSING signal,
  checked before idle prompt absence (jwalaQ comment #1)
- Add TUI permission pattern "Yes No Always Allow" alongside legacy
  [y/n/t] format, requires all three options to avoid false positives
  on bare "Yes"/"No" in agent output (jwalaQ comment #2)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
haofeif added a commit that referenced this pull request Apr 10, 2026
…159) (#163)

* feat(kiro_cli): add full TUI mode support with --legacy-ui fallback (#159)

Remove hardcoded --legacy-ui from launch command and add TUI-native
status detection and message extraction. The provider now:

- Launches in TUI mode by default, falls back to --legacy-ui on timeout
- Detects COMPLETED via ▸ Credits: marker + idle prompt (TUI path)
- Extracts messages using separator (────) boundaries when no green arrows
- Retains full backward compatibility with legacy UI patterns

Also adds "aren't available" to e2e REFUSAL_KEYWORDS for Claude Code test fix.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* style: apply black formatting to kiro_cli provider and tests

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(kiro_cli): address PR #163 review comments

- Raise separator minimum from 4 to 20 chars to avoid matching short
  markdown separators in agent output (jwalaQ comment #4)
- Remove redundant ANSI cleanup in _extract_tui_message — input is
  already ANSI-stripped by caller (jwalaQ comment #5)
- Improve timeout error message wording (jwalaQ comment #3)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* feat(kiro_cli): add TUI processing and permission detection patterns

- Add "Kiro is working" ghost text as positive PROCESSING signal,
  checked before idle prompt absence (jwalaQ comment #1)
- Add TUI permission pattern "Yes No Always Allow" alongside legacy
  [y/n/t] format, requires all three options to avoid false positives
  on bare "Yes"/"No" in agent output (jwalaQ comment #2)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(kiro_cli): update TUI idle pattern to match real kiro-cli v1.29+ output

Verified against real kiro-cli v1.29.1 TUI output via tmux capture-pane:
- Idle prompt is "Ask a question or describe a task" (capital A, no comma)
- Pattern now accepts both old (lowercase, comma) and new formats
- Updated fixtures to use real TUI output format
- Updated inline test strings to match v1.29+ output

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(kiro_cli): use forward-search for TUI separator to handle agent output separators (#159)

Changed _extract_tui_message() to find the first separator after the
previous turn's Credits line instead of the last separator before the
current Credits. This prevents false matches when agent output contains
box-drawing separator characters. Also updated docs for launch command.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(test): add 'Kiro is working' ghost text to TUI processing fixture (#159)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants