Message delivery delay in Assign pattern when Supervisor receives multiple async callbacks #46
Kwak-Jihyun
started this conversation in
Ideas
Replies: 1 comment
-
|
@Kwak-Jihyun there were some bugs in the Jan version, but there were recent fixes in v1.1.0. Is it something that you could try again with the latest release from main branch to see whether your issue still exist |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Kiro CLI: Message delivery issues in Assign pattern
Category: Bug Report / Discussion
Scenario
Using the Assign (Async) Pattern where a Supervisor spawns multiple workers in parallel:
Problem
When Data Analysts complete their work and send results back to the Supervisor:
The Supervisor was clearly idle and ready to receive, but queued messages were not being delivered.
Root Cause Analysis
After debugging, I found two issues:
Issue 1: Idle Pattern Mismatch (Bug)
The
IDLE_PROMPT_PATTERN_LOGinkiro_cli.pyuses a hardcoded ANSI color code that doesn't match Kiro CLI's actual terminal output:Actual Kiro CLI output (from log file):
Differences:
13, actual is93(or other values)\x1b[39mreset, actual uses\x1b[0mThis caused
_has_idle_pattern()to always returnFalse, blocking all message delivery.Proposed Fix:
Issue 2: Event-Driven Delivery Dependency (Design Limitation)
After fixing Issue 1, I noticed another limitation. Message delivery only triggers on log file modification events:
If the receiver terminal is idle with no activity, log file doesn't change, and queued messages won't be delivered until some other activity occurs.
In practice, this is less critical because ongoing terminal activity (processing messages, user interaction) keeps triggering delivery attempts. But in edge cases where a terminal is completely idle, messages could be delayed indefinitely.
Beta Was this translation helpful? Give feedback.
All reactions