Refresh managed guidance during init - #54
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Warning Review limit reachedNext included review available in 11 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
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 |
Summary
Validation
High-level PR Summary
This PR releases CLI version 0.15.2 with an enhancement to the initialization process for managed guidance sections. The key change updates the
maybe_backfill_generated_filefunction to refresh existing Tree Ring managed sections (inAGENTS.md,SKILL.md, andCLI.md) during init, not just insert missing ones. The implementation preserves custom content outside managed sections and respects CRLF line endings, ensuring legacy installer text is replaced while user customizations remain intact. A new regression test validates that stale managed content is properly refreshed while custom notes are preserved.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
crates/tree-ring-memory-cli/src/agent_awareness.rscrates/tree-ring-memory-cli/Cargo.tomlcrates/tree-ring-memory-sqlite/Cargo.tomlCargo.tomlCargo.lockREADME.mddocs/press-kit.mdmarketing/README.md