Skip to content

Commit 32f63f3

Browse files
committed
refactor: merge Proposal 4 into Proposal 3 in presentation commands cleanup plan
- Merged 'Extract User Output Helper' into 'Extract Shared Command Dependencies' - Added UserOutput to CommandContext alongside repository and clock - Kept report_error() utility function for consistent error formatting - Reduced total proposals from 12 to 11 - Renumbered all subsequent proposals (5→4, 6→5, etc.) - Updated all cross-references and dependencies - Updated progress tracking and sprint plan - Added entry to active refactorings list Rationale: Including UserOutput in CommandContext eliminates the need for a separate create_default() wrapper function while keeping the valuable report_error() utility. All commands now get a consistent UserOutput instance from the context, reducing duplication and ensuring consistency.
1 parent 9b5d6a8 commit 32f63f3

File tree

3 files changed

+2022
-3
lines changed

3 files changed

+2022
-3
lines changed
Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# Active Refactoring Plans
22

3-
| Document | Status | Issue | Target | Created |
4-
| --------------------------------------------------------------------------------- | -------------- | ----- | -------------------------------------- | ----------- |
5-
| [Command Code Quality Improvements](./plans/command-code-quality-improvements.md) | 🚧 In Progress | - | `ProvisionCommand`, `ConfigureCommand` | Oct 7, 2025 |
3+
| Document | Status | Issue | Target | Created |
4+
| --------------------------------------------------------------------------------- | -------------- | ----- | -------------------------------------- | ------------ |
5+
| [Command Code Quality Improvements](./plans/command-code-quality-improvements.md) | 🚧 In Progress | - | `ProvisionCommand`, `ConfigureCommand` | Oct 7, 2025 |
6+
| [Presentation Commands Cleanup](./plans/presentation-commands-cleanup.md) | 📋 Planning | - | `src/presentation/commands` module | Oct 28, 2025 |

0 commit comments

Comments
 (0)