Mathias Fredriksson 2a48c8721e fix: stop treating pending chat notification as cancel signal ...

Stale "pending" pubsub notifications from SendMessage can arrive at the
worker control subscriber after close(controlArmed), causing a spurious
ErrInterrupted that aborts the LLM call before the assistant message is
committed.

Remove ChatStatusPending from the cancel-trigger set in
shouldCancelChatFromControlNotification. A "pending" notification
reaching an active worker is always a stale echo: SendMessage (trigger),
EditMessage (handled by persistStep ownership guard), or auto-promote
(runs after processing). None warrant canceling the active run.

Unskip TestSubscribeRelayEstablishedMidStream (CODAGT-353). The only
stale notification that could cancel this test was "pending" from
SendMessage, which the fix now ignores. Verified with a deterministic
red/green/revert cycle using delayedPendingPubsub: 5/5 FAIL without
fix (chat interrupted / Condition never satisfied), 5/5 PASS with fix,
5/5 FAIL on revert.

Fixes CODAGT-383.
Refs CODAGT-179, CODAGT-353.

2026-06-01 18:56:53 +00:00

..

chatd

fix: stop treating pending chat notification as cancel signal

2026-06-01 18:56:53 +00:00

chatfiles

fix: align chat attachment picker with allowed file types (#24917)

2026-05-05 12:25:13 +10:00

gitsync

feat: add GitLab support to coderd/externalauth/gitprovider

2026-05-25 17:41:02 +01:00

nats

feat: add initial NATS implementation (#25602)

2026-05-27 12:57:20 -05:00

skills

fix: parse skill frontmatter as YAML (#25610)

2026-05-22 15:09:30 +02:00