Cover the pre-stream gap with the writing-report spinner #7

Merged

schaefera merged 1 commit from worktree-progress-emoji into master

2026-08-17 11:26:00 +00:00

claude-bot commented

2026-08-17 11:21:42 +00:00

Collaborator

Summary

Follow-up to #6: the "✍️ Writing report..." spinner was dropped as soon as stream_prompt returned the stream object, not when the writer model actually started generating — leaving the terminal blank for the gap in between.
Spinner gained a stop() method so it can be cleared at a precise moment rather than only on drop.
write_text_stream now takes ownership of the spinner and stops it right as the stream's first chunk arrives, so the "Writing report..." line stays up through the whole wait.

Test plan

cargo build -p deep_research
cargo test -p deep_research
cargo clippy -p deep_research (clean)

## Summary - Follow-up to #6: the "✍️ Writing report..." spinner was dropped as soon as `stream_prompt` returned the stream object, not when the writer model actually started generating — leaving the terminal blank for the gap in between. - `Spinner` gained a `stop()` method so it can be cleared at a precise moment rather than only on drop. - `write_text_stream` now takes ownership of the spinner and stops it right as the stream's first chunk arrives, so the "Writing report..." line stays up through the whole wait. ## Test plan - [x] `cargo build -p deep_research` - [x] `cargo test -p deep_research` - [x] `cargo clippy -p deep_research` (clean)

claude-bot added 1 commit 2026-08-17 11:21:43 +00:00

Keep the writing-report spinner up until the stream's first chunk e1d6a20b3a

Between sending the report prompt and the writer actually starting to
generate, the spinner was already dropped, so the terminal went blank
for however long that gap was. Thread the spinner into
write_text_stream instead and stop it right as the first chunk
arrives, covering the wait with the same "Writing report..." line
rather than clearing it early.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

schaefera changed title from ~~WIP: Cover the pre-stream gap with the writing-report spinner~~ to Cover the pre-stream gap with the writing-report spinner

2026-08-17 11:25:56 +00:00

schaefera merged commit 351cc79e57 into master

2026-08-17 11:26:00 +00:00

schaefera referenced this pull request from a commit

2026-08-17 11:26:01 +00:00

Merge pull request 'Cover the pre-stream gap with the writing-report spinner' (#7) from worktree-progress-emoji into master

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: schaefera/doubleo7#7

No description provided.