docs(calendar): add +rsvp usage to skill - #743
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR adds documentation for the existing ChangesCalendar RSVP Shortcut Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
|
Thanks for the contribution. I took another look at the current calendar skill, and I don't think we should merge this as-is. The main skill already lists We intentionally keep shortcut-specific command details in the reference docs so the main skill stays focused and does not duplicate every shortcut's full usage. Adding the same details inline here would make the main skill longer and create another place that has to be kept in sync. Could you share what behavior you observed that made this extra inline section necessary? If the issue was caused by a stale installed skill, updating/installing the latest skill may be the better fix. Thanks again for looking into |
|
It only update skill.md doc. Do you want everyone read your code ? |
Summary
This PR makes the
calendar +rsvpshortcut directly discoverable fromskills/lark-calendar/SKILL.md.It adds inline usage guidance, concrete accept/decline/tentative examples, and the required scope so agents do not need to dig into references before finding the correct command.
Changes
+rsvpusage section toskills/lark-calendar/SKILL.mdevent_idfirst and addaccept/decline/tentativeexamplescalendar:calendar.event:replyto the permissions tableTest Plan
lark xxxcommand works as expectedskills/lark-calendar/SKILL.mdRelated Issues
Summary by CodeRabbit
+rsvpshortcut with usage instructions, prerequisite requirements (event_idretrieval via+agenda), and required permission scope details. Updated permissions reference table.