feat: support semantic recurring calendar operations#1723
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 (3)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughCalendar skill guidance now routes recurring edits and deletions through a scope-based reference. The new recurring operations document defines instance and series ChangesLark calendar recurring schedule guidance
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1723 +/- ##
==========================================
- Coverage 74.51% 74.42% -0.09%
==========================================
Files 852 854 +2
Lines 87655 88477 +822
==========================================
+ Hits 65312 65852 +540
- Misses 17315 17555 +240
- Partials 5028 5070 +42 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@b3e727daed8b99f772a0f1dcd19ef4a69b9c3e54🧩 Skill updatenpx skills add larksuite/cli#feat/calendar-recurring-operations -y -g |
4920361 to
8189636
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/lark-calendar/references/lark-calendar-recurring.md`:
- Around line 22-25: The recurring-calendar guide incorrectly tells users to
delete all exceptions after shifting the master series, which would discard
user-specific edits. Update the instructions in lark-calendar-recurring.md to
avoid unconditional deletion: either describe how to preserve or remap
exceptions when calling lark-cli calendar +update for the master event, or
clearly mark deletion as an explicit, user-confirmed destructive step. Keep the
guidance aligned with the recurring update flow and the delete command only if
it is truly optional and intentional.
- Around line 49-53: Clarify the UNTIL rule in the recurring calendar docs by
updating the wording in the references around the split/edit flow to distinguish
timed versus all-day recurrences. In the recurring update guidance near the
`lark-cli calendar +update`, `events delete`, and `+create` steps, replace the
vague “第 N-1 次实例的日期/截止日期” phrasing with exact previous occurrence date-time for
timed events, and reserve date-only UNTIL wording for all-day events. Keep the
same terminology consistent across the related sections (including the later
rule explanation) so readers can apply the `UNTIL` calculation unambiguously.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 080ab282-0db9-4d71-9c7a-e77c92e6a75f
📒 Files selected for processing (3)
skills/lark-calendar/SKILL.mdskills/lark-calendar/references/lark-calendar-recurring.mdskills/lark-calendar/references/lark-calendar-update.md
8189636 to
b3e727d
Compare
Summary
This PR adds semantic guidance for operating on recurring calendar events so agents can choose the correct workflow for single-instance, full-series, and this-and-following edits or deletions.
Changes
skills/lark-calendar/references/lark-calendar-recurring.mdwith operation flows for recurring event scopes: only this instance, all instances, and this-and-following.event_idfor each operation.skills/lark-calendar/SKILL.mdandlark-calendar-update.mdto the new recurring operation spec.Test Plan
origin/main:git log --oneline origin/main..HEAD.origin/mainto confirm the PR only updates calendar skill documentation.lark-cli calendarrecurring event flow works as expected. Not run because this PR updates agent operation guidance rather than CLI command behavior.Related Issues
Summary by CodeRabbit
New Features
Bug Fixes