Skip to content

fix: harden terminal and CLI behavior - #28

Merged
m1ngsama merged 3 commits into
mainfrom
agent/prompt-1-5-hardening
Aug 13, 2026
Merged

fix: harden terminal and CLI behavior#28
m1ngsama merged 3 commits into
mainfrom
agent/prompt-1-5-hardening

Conversation

@m1ngsama

@m1ngsama m1ngsama commented Aug 12, 2026

Copy link
Copy Markdown
Member

What

  • decode coalesced and split terminal input without losing navigation or pasted text
  • make plain document rendering, pager handoff, and theme mutations consistently ANSI-free
  • add width-aware compact event output and stricter CLI validation/exit semantics
  • cancel TUI view work on quit and restore the terminal on q, Ctrl-C, SIGINT, SIGTERM, and SIGHUP
  • harden schedule authentication/session ownership against late async completion
  • adapt the docs reader to current Documents HTML checkboxes/details and cancel in-flight docs requests
  • report browser launcher and update-check failures with actionable nonzero exits

Impact

Existing commands and both bin names remain compatible. Normal q exits 0; Ctrl-C/SIGINT exits 130, SIGTERM 143, and SIGHUP 129. No package version, dependency, README, release workflow, authentication protocol, or publishing change is included.

Checks

  • TZ=UTC npm run check on Node 20.12.2, Node 22.18.0, and the local Node runtime
  • 54 test files / 718 tests, CLI contract, package consumer smoke, audit 0
  • current Documents fixture: 188/188 budget statuses preserved, no residual input/details/summary HTML
  • fresh tarball install: 73 files, both bins, no legacy bin/ directory
  • deterministic status/docs/events CLI and narrow-width acceptance
  • real PTY tests for q/Ctrl-C/signals, classic docs q/j/l, split/coalesced keys, pending network cancellation, and exact terminal restoration
  • independent strict diff review: no blocker

@m1ngsama
m1ngsama merged commit 4289c38 into main Aug 13, 2026
2 checks passed
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