Skip to content

Add subprocess progress and output framing - #63

Merged
brdv merged 1 commit into
mainfrom
subprocess-progress
Jul 24, 2026
Merged

brdv merged 1 commit into
mainfrom
subprocess-progress

Conversation

@brdv

@brdv brdv commented Jul 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

Add TTY-only progress indicators and isolate native subprocess output from stck output.

This is stack 2/3, based on #62 (output-foundation), with #64 as its child. Long Git and GitHub phases now show transient progress in terminals, successful subprocess chatter is suppressed, and failed git/gh streams are preserved in labeled diagnostic blocks.

Changelist

  • src/ui.rs: add the dependency-free spinner, tool command labels, and TTY policy.
  • src/commands.rs: wrap preflight, discovery, rebase, push, and PR operations with progress.
  • src/gitops.rs and src/github.rs: capture mutating subprocess output and retain full failure diagnostics.
  • src/util.rs: format captured stdout/stderr into labeled blocks.
  • tests/: cover suppressed success chatter, framed failures, real-Git conflict recovery, and clean non-TTY output.

Checklist

  • Changes are minimal and focused for this milestone
  • No breaking CLI changes unless explicitly intended
  • Errors are user-facing, actionable, and prefixed with error:
  • Added/updated tests for behavior changes
  • Updated docs/plan if behavior or scope changed

@brdv brdv mentioned this pull request Jul 24, 2026
5 tasks done
@brdv brdv self-assigned this Jul 24, 2026
@brdv brdv changed the title subprocess-progress Add subprocess progress and output framing Jul 24, 2026
@brdv brdv mentioned this pull request Jul 24, 2026
5 tasks done
Base automatically changed from output-foundation to main July 24, 2026 13:46
@brdv
brdv force-pushed the subprocess-progress branch from 1685997 to ddb9faa Compare July 24, 2026 13:47
@brdv
brdv merged commit e665957 into main Jul 24, 2026
2 checks passed
@brdv
brdv deleted the subprocess-progress branch July 24, 2026 13:48
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