docs(chunk): fix a mode name the rename missed - #886
Conversation
A comment in `composite-tx-boundary-source.test.ts` still refers to `CHUNK_PEER_ORIGIN_MODE=shadow`. That value was renamed to `audit` in #880 before merge and no longer exists, so the comment points a reader at a mode the config rejects at startup. Comment only; no behavior change. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Pu1QiLLfN2Kk8JA91TzSWt
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe change updates one test comment from ChangesSource Attribution
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change corrects documentation in a test comment to use the current audit mode name. It introduces no runtime behavior change and is ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #886 +/- ##
===========================================
+ Coverage 80.86% 81.05% +0.19%
===========================================
Files 145 145
Lines 58383 58383
Branches 4567 4570 +3
===========================================
+ Hits 47210 47325 +115
+ Misses 11115 11007 -108
+ Partials 58 51 -7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
What changed
One comment.
composite-tx-boundary-source.test.tsstill referred toCHUNK_PEER_ORIGIN_MODE=shadow, a value renamed toauditin #880 before it merged.config.tsrejectsshadowat startup, so the comment pointed a reader at a mode that does not exist.Caught by a post-merge grep for rename leftovers on
develop. No behavior change; tests and lint pass.🤖 Generated with Claude Code