Skip to content

fix(chat): hide the bottom pipeline Stop once the transcript has messages#877

Merged
matt2e merged 1 commit into
mainfrom
double-stop
Jul 13, 2026
Merged

fix(chat): hide the bottom pipeline Stop once the transcript has messages#877
matt2e merged 1 commit into
mainfrom
double-stop

Conversation

@matt2e

@matt2e matt2e commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

While a pipeline runs, the chat composer was unconditionally replaced by a large labeled Stop button — even after the transcript had messages, at which point Stop already lives on the Thinking row (#874). This left two Stop controls on screen and blocked queueing follow-ups.

  • Only show the bottom pipeline Stop area when the pipeline is running and the transcript is still empty (grouped.length === 0)
  • Once messages exist, Stop stays on the Thinking row and the composer remains available to queue follow-ups as usual
  • Updated the component doc comment to describe the new behavior

🤖 Generated with Claude Code

…ages

Pipeline sessions showed two Stop buttons once the AI step started
producing messages: the thinking-row Stop (added in #874) and the
pipeline-stop-area Stop that replaces the composer whenever a pipeline
is running.

Gate the bottom stop area on the transcript being empty
(grouped.length === 0) in addition to the pipeline running. During
command-only steps there is no thinking row, so the labeled Stop stays;
once messages exist, Stop lives on the thinking row and the normal
composer returns in queue mode, letting follow-ups be queued against a
running pipeline (the backend already drains them on completion). This
also keeps the composer visible on later turns of a completed pipeline
session, which previously re-entered the stop-only state.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e matt2e requested review from baxen and wesbillman as code owners July 13, 2026 02:23
@matt2e matt2e merged commit 431819f into main Jul 13, 2026
4 checks passed
@matt2e matt2e deleted the double-stop branch July 13, 2026 02:26
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.

1 participant