Skip to content

fix: close 8 defects from bug-hunt sweep - #127

Merged
jkyberneees merged 3 commits into
mainfrom
fix/bughunt-sweep-0918
Sep 18, 2026
Merged

jkyberneees merged 3 commits into
mainfrom
fix/bughunt-sweep-0918

Conversation

@jkyberneees

Copy link
Copy Markdown
Contributor

RED-first sweep from a 3-hunter bug hunt (networking/lifecycle, TUI, rendering/persistence).

  • workspace: torn concurrent persist (per-call temp file) + .corrupt quarantine
  • server: zombie-child fast-fail via background reaper; stderr tail in waitSpawned errors
  • client: bounded WS dial + handshake deadline
  • tui: approval ctrl+enter newline; per-entry expiry bell; queue click bounds
  • update: typed status error replaces substring 401/403/429 matching

All fixes validated failing-test-first; full race suite green.

- workspace: per-call staged temp file fixes torn concurrent persist;
  corrupt workspaces.json now quarantined as .corrupt instead of a
  silent reset (mirrors tokens.go)
- server: background reaper detects exited-but-unreaped children so a
  crashed odek serve fails fast instead of burning the ready timeout;
  waitSpawned errors carry a bounded stderr tail
- client: bounded WS dial + handshake deadline (wsDialTimeout)
- tui: ctrl+enter inserts a newline in the approval composer; approval
  expiry bell latches per queue entry; queue-strip click controls
  bounded to their own cells
- update: typed status error replaces substring 401/403/429 matching
  so transport errors no longer trigger the HTML asset fallback
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 18, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bodek 5dc5127 Commit Preview URL

Branch Preview URL
Sep 18 2026, 11:24 AM

A token-print-and-exit server can flush its banner after the loop's
top-of-loop token check; the reaper's fast liveness signal then failed
Connect even though the token was captured. The token wins.
@jkyberneees
jkyberneees merged commit a5d03dc into main Sep 18, 2026
9 checks passed
@jkyberneees
jkyberneees deleted the fix/bughunt-sweep-0918 branch September 18, 2026 11:27
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