Skip to content

Comments

Refactor progress reporting#10

Merged
soumyaray merged 6 commits intomainfrom
refactor-progress-reporting
Dec 25, 2025
Merged

Refactor progress reporting#10
soumyaray merged 6 commits intomainfrom
refactor-progress-reporting

Conversation

@soumyaray
Copy link
Contributor

No description provided.

soumyaray and others added 6 commits December 24, 2025 17:16
- Simplify .gitignore to only ignore CLAUDE.local.md (allows tracking branch-specific planning docs)
- Add CLAUDE.refactor-progress-reporting.md with refactoring plan:
  - Rename ProgressPublisher → FayeServer
  - Create ProgressMapper for symbol → percentage mapping
  - Create CloneMapper for git output parsing
  - Eliminate JobReporter (merge into ProgressMapper + controller)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Rename workers/infrastructure/messaging/ → faye/
- Rename ProgressPublisher → FayeServer (pure HTTP gateway)
- Add ProgressMapper for symbol → percentage mapping (single source of truth)
- Add CloneMapper for git output → symbols parsing
- Use -ing suffix for action phases: cloning_*, appraising_*, caching_*
- Update require_worker.rb comment

All tests pass: 95 runs, 198 assertions, 90.48% coverage

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Update Service::AppraiseProject to use symbols instead of percentages
- Use CloneMapper to convert git clone output to progress symbols
- Update Worker controller to use ProgressMapper instead of JobReporter
- Delete obsolete JobReporter and progress_monitor files
- Update CLAUDE.md to reflect new architecture
- Replace scale_clone_progress tests with CloneMapper tests

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Service uses input[:progress].report(:symbol) instead of input[:progress].call(:symbol)
- Remove progress_callback method from ProgressMapper
- Cleaner interface with explicit method calls

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Moved to .claude/_archive/ (gitignored)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@soumyaray soumyaray merged commit 9a0f5d0 into main Dec 25, 2025
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