Skip to content

Dev - #24

Merged
rekabytes merged 2 commits into
mainfrom
dev
Mar 10, 2026
Merged

Dev#24
rekabytes merged 2 commits into
mainfrom
dev

Conversation

@rekabytes

@rekabytes rekabytes commented Mar 10, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

Release Notes v1.3.2

  • Bug Fixes

    • Fixed stats display formatting and calculations for security, testing, code review, and documentation metrics
    • Corrected smart merge logic to properly handle user data synchronization
    • Fixed XP progression calculation to use correct field references
  • Chores

    • Version updated to 1.3.2

- 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
@coderabbitai

coderabbitai Bot commented Mar 10, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1ac165dc-02e6-4580-87f5-c232f583b1b4

📥 Commits

Reviewing files that changed from the base of the PR and between 8c3512a and d38e809.

📒 Files selected for processing (4)
  • package.json
  • src/display.js
  • src/merge.js
  • templates/.ocs-version

📝 Walkthrough

Walkthrough

The 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

Cohort / File(s) Summary
Version Metadata
package.json, templates/.ocs-version
Updated version from 1.3.1 to 1.3.2; added changelog entry describing stats and smart merge fixes.
Stats Display Refactoring
src/display.js
Restructured security stats (removed Audits Done/Patterns Added), testing stats (replaced with Unit/Integration/E2E Tests), code review stats (consolidated to per-severity entries), and documentation stats (simplified to Sections/Tutorials/API Docs); changed XP progression to read from xpToNextLevel instead of xpRequired.
Smart Merge Logic
src/merge.js
Narrowed preserved XP data fields, replaced xpRequired with xpToNextLevel for threshold calculations, removed legacy fields (totalTests, totalAudits, patternsAdded, completedSuites, levelHistory), and updated preservation logic to compute totals for testsWritten, issuesFixed, and docsWritten.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A hop and a skip, version bumps with delight,
Fields realigned to shine oh so bright,
Stats now simplified, no more confusion,
XP thresholds fixed—no more intrusion!
Clean merge, clean display, the code's looking right! 🎉

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rekabytes
rekabytes merged commit 6f92e16 into main Mar 10, 2026
4 of 6 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Mar 11, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant