Conversation
- Wrap all JSON.parse calls in merge.js and stats.js with try/catch to give friendly error messages when xp.json is corrupted - Replace MD5 with SHA-256 in hashFile() for file change detection - Update README to reflect all 4 agents and current v1.3.2 state
…nd skills Agents on fresh installs had no instruction to reset XP on level-up or carry overflow forward. Also corrects the Level Thresholds table in all 4 agent files from cumulative XP to per-level XP to match xp.json.
|
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 (12)
📝 WalkthroughWalkthroughThis PR updates the XP progression system across agent and skill templates with a mandatory Level-Up Rule featuring carry-over mechanics and revised thresholds. It enhances code robustness with defensive JSON parsing in merge and stats modules, upgrades file hashing from MD5 to SHA-256, expands README documentation of the OpenCode XP system, and bumps the package version to 1.3.3. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
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
New Features
Bug Fixes
Documentation