feat: show progress during owner-wide planning#6
Conversation
6baea2c to
2a2f3ac
Compare
|
Validation report for commit Implemented a dependency-free progress reporter plus bounded planning/repository-detail concurrency. The CLI keeps progress on Checks run:
Smoke test: node dist/src/cli/main.js plan osolmaz --all --progress --token "$(gh auth token)" >/tmp/github-sane-defaults-progress-plan.out 2>/tmp/github-sane-defaults-progress-plan.errResult: |
|
Final validation report for commit Implemented progress for the long owner-wide phases:
Checks run:
Smoke tests: node dist/src/cli/main.js plan osolmaz --all --progress --token "$(gh auth token)" >/tmp/github-sane-defaults-progress-plan.out 2>/tmp/github-sane-defaults-progress-plan.errResult: node dist/src/cli/main.js apply osolmaz/acpx --yes --progress --token "$(gh auth token)" >/tmp/github-sane-defaults-progress-apply-clean.out 2>/tmp/github-sane-defaults-progress-apply-clean.errResult: clean no-change apply path passed without mutating GitHub state. The apply mutation progress path is covered by unit tests. |
Opened on behalf of Onur Solmaz (
osolmaz).Summary
Owner-wide runs could look frozen while the CLI fetched, planned, and applied work across many repositories.
This change adds one quiet progress line for the long phases while keeping the final command output clean.
It also bounds the network concurrency so large accounts finish faster without firing every request at once.
What Changed
The CLI now reports progress on
stderrand still prints the final plan and apply summaries onstdout.The status line covers repository detail loading, planning, and the mutation phase of
apply.--progressand--no-progress.stderroutside CI.src/sharedlayer for the concurrency helper and updated Slophammer boundaries.Testing
I tested the local unit path, the repo quality gates, and live non-mutating CLI flows.
The live plan run confirmed that progress stays on
stderrand the final plan stays onstdout.The live apply smoke used a clean repository, so it verified the apply command path without mutating GitHub state; the mutation progress path is covered by unit tests.
npm run checknpm run drynpm run mutatenpm run slophammernpx -y @simpledoc/simpledoc checkgit diff --checknode dist/src/cli/main.js plan osolmaz --all --progress --token "$(gh auth token)" >/tmp/github-sane-defaults-progress-plan.out 2>/tmp/github-sane-defaults-progress-plan.errnode dist/src/cli/main.js apply osolmaz/acpx --yes --progress --token "$(gh auth token)" >/tmp/github-sane-defaults-progress-apply-clean.out 2>/tmp/github-sane-defaults-progress-apply-clean.errRisks
The progress bar is disabled by default in non-interactive output and CI, so automation behavior stays stable.
The apply phase remains sequential; only planning and repository-detail reads use bounded concurrency.
stdout.