Skip to content

Small follow-ups from the audit: one clock, no stuck loading, /where stays quiet - #144

Merged
hamzafer merged 2 commits into
mainfrom
fix/small-follow-ups
Oct 23, 2026
Merged

hamzafer merged 2 commits into
mainfrom
fix/small-follow-ups

Conversation

@hamzafer

@hamzafer hamzafer commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

The follow-ups left over from today's audit and reviews.

  • mission-control (0.1.5): keeps one 1-second clock even if session.start fires again (the same fix One timer per session, a reels test for today's Claude Code, and caught background errors #142 gave three other mods). It also catches its background redraws, keeps only the last 5 edits per file for the "why" line, and survives a failed TMPDIR read.
  • heatmap (0.1.3): the pane resets its loading state whether or not the refresh succeeds, so it can't stay on "Reading your session history…".
  • where-am-i (0.1.7): a slash command like /where leaves the band alone. Before, it set "now" to "reading your message" until the next real turn. A slash command that does start a turn is picked up at turn.start, which now also records its text.

Tests: 2 new, and each fails on the old code: a second session.start keeps one clock, not two (mission-control) and a slash command that starts no turn leaves now as it was (where-am-i). claude plugin test: mission-control 11, heatmap 28, where-am-i 11, all pass; validate passes for all three; node --test test/*.test.mjs 168 pass.

Note: this takes mission-control to 0.1.5, so #133 moves to 0.1.6 when it merges in main.

@hamzafer

hamzafer commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner Author

Review outcomes (one fix batch):

  • Codex: no findings.
  • Opus: turn.start replaced the prompt on every main turn, so a turn Claude Code starts itself (like a background agent finishing) would hand Haiku the notification text instead of the person's message, and log = [] there could wipe early tool calls. Fixed: where-am-i only takes turn.start's text, and resets the log, right after a slash command. New test: a turn Claude Code starts itself keeps the person's message as the prompt, which fails on the reviewed commit.
  • Opus: the 5-edit cap kept only the latest edits, so a big first edit could fall out of the "why". Fixed: it keeps the first edit plus the latest four.
  • Kept as is: a message starting with / (like "/tmp/x is broken") skips the "reading your message" line until the first tool call. The band still ends up right.

Tests: where-am-i 12 pass, mission-control 11 pass, heatmap 28 pass; validate passes; node --test test/*.test.mjs 168 pass.

@hamzafer
hamzafer merged commit 736b26d into main Oct 23, 2026
2 checks passed
@hamzafer
hamzafer deleted the fix/small-follow-ups branch October 23, 2026 22:03
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