review(mcp): clarify mirror_pose dst-overwrite contract#600
Conversation
Codex P2 on PR #599 (merged): the schema description for `dst` implied "overwrites in place if same as src", but PoseLibrary actually does an unconditional `byName.insert(dst, ...)` — any existing pose at `dst` is silently clobbered regardless of `src`. Fix: update the schema description so agents know `dst` is an unconditional overwrite and should be chosen carefully (or `list_poses` called first to avoid stomping unrelated poses). Code behaviour unchanged; this is a doc-only fix. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ 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 |
|
1 participant
Codex P2 on PR #599 (merged): the schema description for
dstimplied "overwrites in place if same assrc", butPoseLibrary::mirrorPoseactually does an unconditionalbyName.insert(dst, ...)— any existing pose atdstis silently clobbered regardless ofsrc.Fix
Update the schema description so agents know
dstis an unconditional overwrite and should be chosen carefully (orlist_posescalled first to avoid stomping unrelated poses).Code behaviour unchanged; this is a doc-only fix.
🤖 Generated with Claude Code