Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis PR redesigns the patch application output system by introducing structured result types (PatchGroup enum, PatchResult interface, ApplyCustomizationResult), centralizing patch application logic into a unified patches array in applyCustomization, updating return types to include detailed patch results, and modifying display consumers to render and handle these results. Changes
Sequence DiagramsequenceDiagram
participant CLI as CLI/App
participant applyC as applyCustomization()
participant patches as applyPatches()
participant patch as Patch Execution
participant result as Results Aggregator
participant display as printPatchResults()
CLI->>applyC: invoke with config
applyC->>applyC: precompute common values
applyC->>patches: call with patches array
loop For each patch
patches->>patch: execute patch function
patch-->>result: record PatchResult<br/>(id, name, group, applied, details)
result->>result: accumulate results
end
patches-->>applyC: return all results
applyC->>applyC: update config with patches
applyC-->>CLI: return ApplyCustomizationResult<br/>(config + results)
CLI->>display: call printPatchResults(results)
display->>display: group by PatchGroup
display->>display: filter by isShowUnchanged
display-->>CLI: render patch outcome
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes Possibly related PRs
Poem
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
--applynow shows a detailed list of all the patches that were applied.Summary by CodeRabbit
Release Notes
New Features
--applyoutput to display patch application results organized by category--show-unchangedflag to filter which patches appear in results displayBug Fixes
✏️ Tip: You can customize this high-level summary in your review settings.