Skip to content

docs(calendar): refine attendee guidance for bots and user-search identity - #2086

Merged
calendar-assistant merged 1 commit into
mainfrom
docs/calendar-bot-attendee-guidance
Jul 28, 2026
Merged

docs(calendar): refine attendee guidance for bots and user-search identity#2086
calendar-assistant merged 1 commit into
mainfrom
docs/calendar-bot-attendee-guidance

Conversation

@calendar-assistant

@calendar-assistant calendar-assistant commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

Refine the calendar skill docs so AI agents handle bot participants correctly. Bots are virtual identities with no free/busy semantics, no meeting-room seat, and no room preference, so they must be dropped from time/room recommendation flows while remaining valid attendees when actually creating or updating an event.

Changes

  • +suggestion: --attendee-ids note now tells agents not to pass a bot open_id, since bots have no free/busy semantics and pollute time-slot recommendations.
  • +room-find: --attendee-ids note plus a rules entry to strip bot attendees before querying — bots take no seat and carry no room preference.
  • schedule-clear-time: the pre-booking free/busy check adds a rule to skip bots (they can run parallel meetings, so checking them is meaningless).
  • +create / +update: clarify that bots are still valid attendees to add, so the exclusion rule above is not over-applied to the landing step.
  • SKILL.md: consolidate the user-search identity note into the skill entry.

Test Plan

  • Docs-only change; no code paths affected
  • Verified only calendar skill markdown files changed (git diff --name-only)

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Clarified when bot identities can be added as calendar attendees during event creation and updates.
    • Clarified that bot identities should be excluded when finding rooms or generating time suggestions.
    • Updated availability guidance to indicate that bots do not require busy/free checks.
    • Documented that user searches require user identity mode and that returned user IDs can be used for attendee-related scheduling actions.

…ntity

Consolidate the user-search identity note into SKILL.md, and clarify bot
handling across attendee flows: bots are virtual identities with no
free/busy semantics, no meeting-room seat, and no room preference, so
they must be excluded from +suggestion, +room-find, and the scheduling
free/busy check. Note in create/update that bots remain valid attendees.
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 586b6ba9-d931-436a-a489-59cb114c6644

📥 Commits

Reviewing files that changed from the base of the PR and between d2e22c5 and c99176f.

📒 Files selected for processing (6)
  • skills/lark-calendar/SKILL.md
  • skills/lark-calendar/references/lark-calendar-create.md
  • skills/lark-calendar/references/lark-calendar-room-find.md
  • skills/lark-calendar/references/lark-calendar-schedule-clear-time.md
  • skills/lark-calendar/references/lark-calendar-suggestion.md
  • skills/lark-calendar/references/lark-calendar-update.md

📝 Walkthrough

Walkthrough

Calendar documentation clarifies bot identity search behavior, valid bot attendee usage, and when bot identities must be excluded from room and busy/free calculations.

Changes

Calendar bot attendee handling

Layer / File(s) Summary
Bot identity and attendee contracts
skills/lark-calendar/SKILL.md, skills/lark-calendar/references/lark-calendar-create.md, skills/lark-calendar/references/lark-calendar-update.md
Documents that user search requires --as user, while bot identities may be valid attendees for calendar creation and updates.
Room and availability filtering
skills/lark-calendar/references/lark-calendar-room-find.md, skills/lark-calendar/references/lark-calendar-schedule-clear-time.md, skills/lark-calendar/references/lark-calendar-suggestion.md
Requires removing bot IDs from room-finding and suggestion attendee lists, and skips bot busy/free queries.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Suggested labels: documentation

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the PR’s main docs update about bot attendee handling and user-search identity guidance.
Description check ✅ Passed The description follows the template and includes summary, changes, test plan, and related issues with enough detail.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/calendar-bot-attendee-guidance

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added domain/calendar PR touches the calendar domain size/M Single-domain feat or fix with limited business impact labels Jul 28, 2026
@github-actions

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@c99176f10ff98cfe8695bb0e5b6eb9c1bc085967

🧩 Skill update

npx skills add larksuite/cli#docs/calendar-bot-attendee-guidance -y -g

@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.16%. Comparing base (d2e22c5) to head (c99176f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2086   +/-   ##
=======================================
  Coverage   75.16%   75.16%           
=======================================
  Files         912      912           
  Lines       96475    96475           
=======================================
  Hits        72517    72517           
  Misses      18381    18381           
  Partials     5577     5577           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@calendar-assistant
calendar-assistant merged commit 2e0fb9a into main Jul 28, 2026
32 of 42 checks passed
@calendar-assistant
calendar-assistant deleted the docs/calendar-bot-attendee-guidance branch July 28, 2026 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/calendar PR touches the calendar domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants