Conversation
- display.js: fix xpToNextLevel (was xpRequired), align all 4 stat blocks to actual fields - display.js: security/code-review use issuesFixed, docs uses docsWritten, testing uses testsWritten - display.js: remove totalReviews, totalTests, totalAudits, patternsAdded (not in our xp.json) - merge.js: fix preserveFields list to match our xp.json fields - merge.js: remove xpTable assignment (field removed from xp.json) - merge.js: fix level recalculation to use xpToNextLevel (was xpRequired) - merge.js: fix getSmartMergePreserveInfo labels to match our fields
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe PR bumps the package version to 1.3.2 and refactors the stats display and smart merge logic to use correct field names from xp.json, simplifying stats categories and consolidating preserved data fields across the codebase. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
✨ Finishing Touches
🧪 Generate unit tests (beta)
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 by CodeRabbit
Release Notes v1.3.2
Bug Fixes
Chores