feat(progress): a waiting turn says how long, in the shape Kai drew #741

Merged

coilyco-ops merged 1 commit from feat/a-waiting-turn-says-how-long into main

2026-08-13 21:56:25 +00:00

coilyco-ops commented

2026-08-13 21:55:06 +00:00

Member

Slice of issue 370. Kai drew three lines; one shipped.

> 🤔 `thinking...`          <- live since sirens-echo#373
> 🕐 `still thinking... (9 seconds)`     <- never built
> 🕐 `still thinking... (99 seconds)`    <- never built

Why the other two never appeared

turnprogress.go, refresh:

if p.finished || p.lastStage == "" || p.messageID != "" {
    return
}

Watch ticks every beat and calls refresh. refresh returns immediately once a line is posted, so a turn that stays in one stage for 99 seconds shows one static line for 99 seconds. The sink's Edit was used only when the stage changed.

An earlier seat closed the first half, added consult for three unnamed icons, and did not record that this half was undelivered.

What ships

A beat that finds a line already up appends an elapsed line instead of returning:

> 🤔 `thinking...`
> 🕐 `still thinking 9 seconds...`
> 🕐 `still thinking 99 seconds...`

Bounded at maxProgressWaitLines = 12, and a stage change clears the waits because the new line is describing something else.

Two faithfulness notes

Punctuation differs and the alphabet is why. noticeAllowed is [a-z0-9 ,./-], so parentheses are stripped, and stageNotice appends the ellipsis at the end. still thinking... (9 seconds) therefore renders still thinking 9 seconds.... I did not widen the alphabet for punctuation. That shape is what lets a member tell a harness line from model output; sirens-echo#373 widened it once for the emoji and said so, and doing it again for brackets is not worth the same cost. Asserted by TestEveryWaitLineKeepsTheNoticeShape.

One clock, not a rotation. Kai wrote "if we wanna be real cute" for rotating clock faces, which is an invitation rather than a specification. One word and it is a one-line change.

Verification

ward exec gate PASS on all six steps.

Two mutations, both compiling:

restoring the original early return: fails TestAWaitingTurnSaysHowLong, TestEachBeatAddsALine, TestTheWaitColumnIsBounded
removing the bound: fails TestTheWaitColumnIsBounded

closes #740

Slice of [issue 370](https://forgejo.coilysiren.me/coilyco-gaming/sirens-echo/issues/370). Kai drew three lines; one shipped. ``` > 🤔 `thinking...` <- live since sirens-echo#373 > 🕐 `still thinking... (9 seconds)` <- never built > 🕐 `still thinking... (99 seconds)` <- never built ``` ## Why the other two never appeared `turnprogress.go`, `refresh`: ```go if p.finished || p.lastStage == "" || p.messageID != "" { return } ``` `Watch` ticks every beat and calls `refresh`. **`refresh` returns immediately once a line is posted**, so a turn that stays in one stage for 99 seconds shows one static line for 99 seconds. The sink's `Edit` was used only when the stage *changed*. An earlier seat closed the first half, added `consult` for three unnamed icons, and did not record that this half was undelivered. ## What ships A beat that finds a line already up appends an elapsed line instead of returning: ``` > 🤔 `thinking...` > 🕐 `still thinking 9 seconds...` > 🕐 `still thinking 99 seconds...` ``` Bounded at `maxProgressWaitLines = 12`, and a stage change clears the waits because the new line is describing something else. ## Two faithfulness notes **Punctuation differs and the alphabet is why.** `noticeAllowed` is `[a-z0-9 ,./-]`, so parentheses are stripped, and `stageNotice` appends the ellipsis at the end. `still thinking... (9 seconds)` therefore renders `still thinking 9 seconds...`. **I did not widen the alphabet for punctuation.** That shape is what lets a member tell a harness line from model output; sirens-echo#373 widened it once for the emoji and said so, and doing it again for brackets is not worth the same cost. Asserted by `TestEveryWaitLineKeepsTheNoticeShape`. **One clock, not a rotation.** Kai wrote *"if we wanna be real cute"* for rotating clock faces, which is an invitation rather than a specification. One word and it is a one-line change. ## Verification `ward exec gate` PASS on all six steps. Two mutations, both compiling: - restoring the original early return: fails `TestAWaitingTurnSaysHowLong`, `TestEachBeatAddsALine`, `TestTheWaitColumnIsBounded` - removing the bound: fails `TestTheWaitColumnIsBounded` closes #740

coilyco-ops added 1 commit

2026-08-13 21:55:06 +00:00

feat(progress): a waiting turn says how long, in the shape Kai drew

ci / image-build (pull_request) Successful in 21s

Details

ci / test (pull_request) Successful in 36s

Details

ci / publish-echo-image (pull_request) Has been skipped

Details

ci / publish-observed (pull_request) Has been skipped

Details

f62ae55394

refresh returned as soon as a line was posted, so a turn that stayed in one
stage for 99 seconds showed one static line for 99 seconds. The sink's Edit was
used only on a stage change.

A beat that finds a line already up now appends an elapsed line, bounded at 12,
cleared when the stage changes because the new line describes something else.

The alphabet strips parentheses and stageNotice appends the ellipsis, so it
renders 'still thinking 9 seconds...' rather than his exact punctuation. I did
not widen noticeShape for brackets: 373 widened it once for the emoji and said
what that cost, and punctuation is not worth the same.

One clock rather than a rotation. 'if we wanna be real cute' is an invitation.

closes #740

Co-authored-by: Kai Siren <coilysiren@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>

coilyco-ops requested review from coilysiren

2026-08-13 21:55:06 +00:00

coilyco-ops referenced this pull request

2026-08-13 21:55:27 +00:00

thinking message could be more interesting #370

coilyco-ops merged commit 65860589f5 into main

2026-08-13 21:56:25 +00:00

coilyco-ops referenced this pull request from a commit

2026-08-13 21:56:27 +00:00

Merge pull request 'feat(progress): a waiting turn says how long, in the shape Kai drew' (#741) from feat/a-waiting-turn-says-how-long into main