Report terminal spawn and exit failures for live CI views #641

New issue

Closed

opened 2026-05-06 19:55:43 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-05-06 19:55:43 +00:00

Owner

Parent: #634

Problem

Terminal-backed CI watch/live-tail paths call termopen() and do not report spawn failure or terminal job exit status. If the CLI is missing, auth fails before output, or the process exits immediately, Forge may leave the user with an unhelpful terminal buffer.

Scope

Improve lua/forge/term.lua around termopen() return values and exit callbacks.
Apply to CI watch, live tail, and terminal summary views.
Preserve normal TTY behavior for successful processes.
Add tests around terminal spawn failure if feasible with the existing test harness.

Non-goals

Do not replace terminal views with parsed buffers.
Do not parse TTY output.

Parent: #634 ## Problem Terminal-backed CI watch/live-tail paths call `termopen()` and do not report spawn failure or terminal job exit status. If the CLI is missing, auth fails before output, or the process exits immediately, Forge may leave the user with an unhelpful terminal buffer. ## Scope - Improve `lua/forge/term.lua` around `termopen()` return values and exit callbacks. - Apply to CI watch, live tail, and terminal summary views. - Preserve normal TTY behavior for successful processes. - Add tests around terminal spawn failure if feasible with the existing test harness. ## Non-goals - Do not replace terminal views with parsed buffers. - Do not parse TTY output.

barrettruth added the

bug

v0.2.0

labels

2026-05-06 19:55:43 +00:00

barrettruth referenced this issue

2026-05-06 19:56:06 +00:00

Tracker: error handling and edge cases #634

barrettruth referenced this issue from a pull request that will close it,

2026-05-06 22:08:54 +00:00

Report terminal command failures #654

barrettruth closed this issue