Repository navigation
docs: note editor-context Screen values and captured-variable filtering in skills - #2315
Conversation
…ng in skills execute-dynamic-code snippets see Editor Screen pixels, not Game View resolution. pause-point --captured-variables now points at --captured-variable-names when the full dump is noisy. Co-authored-by: Cursor <cursoragent@cursor.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PR updates six skill documents. Execute-dynamic-code guidance covers Editor pixels and Game View-center rays. Pause-point guidance covers full variable capture, truncation limits, and output filtering. ChangesSkill documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change clarifies editor-context screen values and captured-variable filtering; no actionable merge-blocking risk remains after normal checks and review. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
full keeps each captured entry's Value; it does not change which variables were captured, and capture caps still apply. Co-authored-by: Cursor <cursoragent@cursor.com>
Summary
execute-dynamic-codenow states thatScreen.width/Screen.heightin this tool are Editor pixels, not Game View resolution, and points toViewportPointToRayfor a Game View center ray.pause-pointnow points from--captured-variablesto--captured-variable-nameswhen the defaultfulldump is noisy.User Impact
Screen.*missed, and agents spent extra round trips assuming the raycast was broken.--captured-variables fullburied the locals of interest under unrelated InstanceField values, and--captured-variable-nameswas not discoverable from that flag's description.--captured-variablesbullet itself points at the name filter.Changes
.claude/and.agents/skill copies. Parameter tables are unchanged, sodefault-tools.jsonstayed up to date.Verification
scripts/sync-tool-docs.sh→cli/common/tools/default-tools.json is already up to date.dist/darwin-arm64/uloop skills install --claude --agents→ Updated: 2, Skipped: 18