Skip to content

feat(tui): queued-prompt visibility + CI platform matrix - #42

Merged
jkyberneees merged 2 commits into
mainfrom
feat/queue-visibility-and-platform-ci
Aug 29, 2026
Merged

feat(tui): queued-prompt visibility + CI platform matrix#42
jkyberneees merged 2 commits into
mainfrom
feat/queue-visibility-and-platform-ci

Conversation

@jkyberneees

Copy link
Copy Markdown
Contributor

Round-2 improvement round — items A10 + A2 of the expert-panel top-10.

A10 — prompt-queue visibility

  • Mid-turn ⏎ now acknowledges the hold with a transient note (queued — it sends when the turn ends); the input clearing silently read as a lost message.
  • Queue depth rides the busy status line (🧠 thinking · 2 queued) — where the eyes already are while a turn runs. The footer indicator stays.
  • Drain semantics preserved: one turn-end pops exactly one held prompt; the count steps down until empty.
  • 2 new tests (count + acknowledgment, drain decrement) and updated queue assertions; README synced.

A2 — platform CI legs

  • Build & Test is now a matrix: ubuntu + macos run the full race suite (green locally on arm64); fail-fast: false.
  • windows gets a build+vet leg — the full suite lands there only after a green probe run, since the server/tui suites lean on tty/process behaviour; rationale in-file.
  • Coverage summary/artifact stay ubuntu-only (no artifact-name collisions).

Verification

  • go test -race -count=1 ./... — all 7 packages ok
  • go vet clean, go fmt clean, YAML parse check on ci.yml
  • golangci-lint could not run locally (tooling gate) — CI covers it on this PR

…s line

Mid-turn ⏎ queued the prompt silently — the input cleared with zero
feedback, which reads as a lost message. A transient note now
acknowledges each hold (same wording as the retry path), and the queue
depth rides the busy status line, next to the spinner where the eyes
already are. One turn-end drains exactly one held prompt; the count
steps down until empty.

The footer indicator predates this; tests cover both surfaces and the
drain decrements.
Darwin and windows binaries ship in every release while CI only ever
ran ubuntu. The test job gains a macos leg (the suite passes locally on
arm64), coverage artifacts stay ubuntu-only to avoid name collisions,
and a windows-build leg gates compile + vet per GOOS. The full suite
lands on windows only after a green probe run — the server and tui
suites lean on tty/process behaviour that may not hold there.
@jkyberneees
jkyberneees merged commit e8403c4 into main Aug 29, 2026
7 checks passed
@jkyberneees
jkyberneees deleted the feat/queue-visibility-and-platform-ci branch August 29, 2026 11:44
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