Skip to content

Commit 0b340d4

Browse files
committed
fix(qa): scope message-tool-cascade no-outbound check to post-cascade cursor
ClawSweeper P2: waitForNoOutbound was scanning from index 0, so it counted the two already-observed cascade messages as violations and always failed. Capture the outbound message count after the second cascade marker and pass it as sinceIndex so the settle window only checks for messages that arrive after both expected sends.
1 parent 99fc217 commit 0b340d4

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

qa/scenarios/channels/message-tool-cascade-regression.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,15 @@ flow:
8282
params: [candidate]
8383
expr: "candidate.conversation.id === config.conversationId && candidate.conversation.kind === 'group' && (candidate.text ?? '').includes(config.expectedMarker2)"
8484
- expr: liveTurnTimeoutMs(env, 180000)
85+
- set: cascadeCursor
86+
value:
87+
expr: "state.getSnapshot().messages.filter((message) => message.direction === 'outbound').length"
8588
- call: waitForNoOutbound
8689
args:
8790
- ref: state
8891
- 15000
92+
- sinceIndex:
93+
ref: cascadeCursor
8994
- set: allOutbound
9095
value:
9196
expr: "state.getSnapshot().messages.filter((message) => message.direction === 'outbound' && message.conversation.id === config.conversationId && message.conversation.kind === 'group')"

0 commit comments

Comments
 (0)