chore: reconcile pi-tools release state - #14
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change documents Changespi-tools release updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR reconciles the package release metadata, refines documentation, and repairs test typing without changing production behavior or runtime dependencies. No actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
- docs/compatibility.md: tool surface, config precedence, runtime matrix, local/no-telemetry boundary, versioning policy, benchmark reproducibility - scripts/: version-sync, no-telemetry, breaking-major gates + benchmark-compare - test/compat-surface.test.ts: doc-vs-code surface consistency (5 tests) - CI: gates wired into external-tests.yml + release.yaml pre-publish guard - README: performance claims now link the reproducible benchmark script Version drift (repo 0.10.5 vs npm 0.10.8) was already reconciled by #14; this adds the gate that keeps it closed.
CHE-234 portfolio convergence.
What changed:
test.eachreadonly typing issue found while reproducing main.Verification: