evolve(handoff/pickup): handle args and explicit next-step instructions - #676
Conversation
When the user passes an absolute path after /pickup, use it directly without re-asking. When they pass a keyword, match and confirm once.
When the user passes text after /handoff, include it verbatim at the top of Next Steps labeled "(from user at handoff time)" so it isn't lost among inferred steps.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR updates the ChangesSkill Argument Enhancements
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
✨ 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 |
Summary
/pickup, use it directly (no AskUserQuestion). When a keyword/phrase is passed, list candidates, match the best one, and confirm with a single question./handoff, include it verbatim at the top of## Next Stepslabeled(from user at handoff time).Changes
plugins/me/skills/pickup/SKILL.md— new Step 0 for args handlingplugins/me/skills/handoff/SKILL.md— new Arguments sectionEvidence
Observed in 3 sessions where
/pickupwas called with an explicit file path; in one session (ccd0e9e4) AskUserQuestion was rejected because the path was already provided. Observed in 1 session (6e0a9211) where/handoffwas called with an explicit next-step instruction that had no documented handling.Summary by CodeRabbit
/handoffcommand now accepts additional text immediately after invocation, which is inserted as an explicit next-step instruction./pickupcommand now supports optional arguments—provide a file path directly or use keywords to search and match handoff candidates.