Preserve CLI failure context in shared system errors #635

New issue

Closed

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

barrettruth commented

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

Owner

Parent: #634

Problem

forge.system.cmd_error() currently chooses stderr, else stdout, else a fallback. That loses exit code, command context, and the lower-priority stream, even when both streams contain useful information. This feeds operation notifications, compose flows, CI fetches, review fetches, and picker failures.

Scope

Define a shared command-failure formatter/shape.
Preserve action context, exit code, stderr excerpt, stdout excerpt, and command identity where appropriate.
Add focused tests for stderr-only, stdout-only, mixed-stream, blank-output, and nonzero-exit cases.

Non-goals

Do not dump full command output into notifications.
Do not change backend command semantics.

Parent: #634 ## Problem `forge.system.cmd_error()` currently chooses stderr, else stdout, else a fallback. That loses exit code, command context, and the lower-priority stream, even when both streams contain useful information. This feeds operation notifications, compose flows, CI fetches, review fetches, and picker failures. ## Scope - Define a shared command-failure formatter/shape. - Preserve action context, exit code, stderr excerpt, stdout excerpt, and command identity where appropriate. - Add focused tests for stderr-only, stdout-only, mixed-stream, blank-output, and nonzero-exit cases. ## Non-goals - Do not dump full command output into notifications. - Do not change backend command semantics.

barrettruth added the

bug

v0.2.0

labels

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

barrettruth referenced this issue

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

Standardize picker fetch and decode error rows #638

barrettruth referenced this issue

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

Show structured command failures in CI log and summary buffers #639

barrettruth referenced this issue