Skip to content

feat(pr-review-fix-loop): progress banner with iteration stats and ETA - #27

Merged
dapi merged 2 commits into
masterfrom
feature/pr-review-progress-banner
Mar 1, 2026
Merged

feat(pr-review-fix-loop): progress banner with iteration stats and ETA#27
dapi merged 2 commits into
masterfrom
feature/pr-review-progress-banner

Conversation

@dapi

@dapi dapi commented Feb 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add ASCII progress banner rendered by stop-hook between iterations showing: progress bar, issues trend, elapsed time, ETA
  • Add check-gitignore.sh to warn when loop artifacts are not in .gitignore
  • Add record-iteration.sh to track iteration stats (issues count, duration) in JSON
  • Add show-progress.sh to render the banner with ANSI colors and trend analysis
  • Modify setup-loop.sh to create JSON stats file alongside YAML state
  • Modify stop-hook.sh to call show-progress.sh on exit and continue
  • Add Step 2.5 to command prompt for recording iteration stats
  • 16 new tests (40 total), all passing

Test plan

  • All 40 tests pass (bash pr-review-fix-loop/tests/test-loop-scripts.sh)
  • No emoji violations in pr-review-fix-loop plugin
  • JSON valid in plugin.json
  • All new scripts executable
  • No absolute paths in new files

🤖 Generated with Claude Code

…gitignore check)

Rebase progress tracking features onto master (v1.9.0 base):
- Add show-progress.sh: ASCII progress banner with trend, ETA, elapsed time
- Add record-iteration.sh: append iteration stats to JSON
- Add check-gitignore.sh: validate artifacts are in .gitignore
- Edit stop-hook.sh: invoke progress banner at iteration boundaries
- Edit setup-loop.sh: create stats JSON, add jq check, rm -f stats cleanup
- Edit assemble-prompt.sh: add step 2.5 for recording iteration stats
- Edit pr-review-fix-loop.md: add .gitignore check before other pre-checks
- Bump version 1.9.0 -> 1.10.0 (minor: new features)
- 120 tests passing (29 existing + 91 new)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dapi
dapi force-pushed the feature/pr-review-progress-banner branch from 460d25a to a6b224e Compare March 1, 2026 19:48
- Fix always-pass tests 61/67/102 to assert expected set -euo behavior
- Preserve jq stderr in record-iteration.sh for diagnostics
- Add *) default case to show-progress.sh minimal banner
- Complete rm -f migration in stop-hook.sh (lines 117, 124)
- Add debug log to check-gitignore.sh FILES_TO_CHECK
- Fix pre-existing emoji in spec-reviewer/commands/spec-review.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dapi
dapi merged commit a487238 into master Mar 1, 2026
1 check passed
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