Centralize shared command error extraction for async shell-outs #435

New issue

Closed

opened 2026-04-26 23:33:04 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-04-26 23:33:04 +00:00

Problem

cmd_error(...) style logic is currently duplicated across modules like ops and review. That makes error formatting drift easier and complicates follow-up work to preserve richer stderr/stdout details consistently.

Proposed solution

Move shared command error extraction into one helper with a single contract for stderr/stdout/fallback handling, then update async callers to use it consistently.

Notes

root tracker: #170

## Problem `cmd_error(...)` style logic is currently duplicated across modules like `ops` and `review`. That makes error formatting drift easier and complicates follow-up work to preserve richer stderr/stdout details consistently. ## Proposed solution Move shared command error extraction into one helper with a single contract for stderr/stdout/fallback handling, then update async callers to use it consistently. ## Notes - root tracker: #170