Skip to content

docs: optimize calendar vc and minutes skill docs#1269

Open
hugang-lark wants to merge 1 commit into
mainfrom
feat/vc_event
Open

docs: optimize calendar vc and minutes skill docs#1269
hugang-lark wants to merge 1 commit into
mainfrom
feat/vc_event

Conversation

@hugang-lark
Copy link
Copy Markdown
Collaborator

@hugang-lark hugang-lark commented Jun 4, 2026

Summary

Optimized the documentation for lark-calendar, lark-vc, and lark-minutes skills. The updates refine the skill descriptions, restructure the content layout, and clarify the routing rules between different domains.

Changes

  • Refined the description fields to concisely state the supported and unsupported scenarios.
  • Restructured the documents to place identity declarations and shortcuts at the beginning.
  • Removed outdated permission mapping tables and duplicated schema instructions.
  • Clarified cross-domain intention routing, such as directing meeting-related minute queries directly to the lark-vc skill.

Test Plan

  • None

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Updated skill guidance for calendar scheduling with refined routing rules and identity handling.
    • Revised meeting minutes documentation to clarify scope and local file workflows.
    • Enhanced video conference task documentation with improved meeting artifact handling and routing clarity.

@hugang-lark hugang-lark added the domain/doc Docs domain label Jun 4, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 4, 2026

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

Run ID: 8a5fca51-5e8e-4e14-b504-aa0e76480230

📥 Commits

Reviewing files that changed from the base of the PR and between abc0553 and 17f9f45.

📒 Files selected for processing (3)
  • skills/lark-calendar/SKILL.md
  • skills/lark-minutes/SKILL.md
  • skills/lark-vc/SKILL.md

📝 Walkthrough

Walkthrough

Three skill specification files (lark-calendar, lark-minutes, lark-vc) are comprehensively updated with revised scope descriptions, new operational guidance sections (identity, routing, concepts), and standardized API resource documentation that establish clearer functional boundaries and routing rules between the skills.

Changes

Skill specification and routing documentation update

Layer / File(s) Summary
Calendar skill scope and operational guidance
skills/lark-calendar/SKILL.md
Calendar skill description now emphasizes scheduling and room management scope, with a major rewrite of the "calendar (v4)" guidance section covering pre-read requirements, identity handling (--as user vs --as bot), shortcuts, pre-condition routing, core concepts (instances/all-day/time-block vs time-range), time inference rules, and meeting-room rules. API invocation format is simplified and out-of-scope section is reformatted.
Minutes skill scope, routing, and download capability
skills/lark-minutes/SKILL.md
Minutes skill description is expanded to explicitly cover search, metadata, download/upload-to-generate, title updates, and speaker replacement. New sections define default identity (--as user), shortcuts with required reference docs, intent routing (delegating meeting-related minutes to lark-vc), core concepts for minute_token extraction, and clarified scope boundaries. API resources section is reorganized with standardized CLI format and an explicit out-of-scope list.
VC skill boundaries, artifact querying, and meeting routing
skills/lark-vc/SKILL.md
VC skill description is revised to emphasize historical meeting search, meeting artifact queries (minutes/notes/recordings), and participant snapshots, explicitly excluding real-time in-meeting events (lark-vc-agent). New guidance sections define identity defaults, shortcuts, intent routing rules (boundaries vs lark-calendar and lark-vc-agent), and core concepts for product chain selection (AI summary vs recording). Pre-check note guides selection of appropriate chain for meeting notes processing. API resources are reorganized with standardized CLI format and updated out-of-scope list.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • larksuite/cli#403: Implements the new calendar +room-find workflow and time-block branching that the main PR's rewritten lark-calendar operational rules document and guide users through.
  • larksuite/cli#1036: Adds minutes +update and minutes +speaker-replace shortcuts whose scope, routing, and required permissions are documented in the main PR's revised lark-minutes specification.
  • larksuite/cli#824: Implements the +meeting-join, +meeting-leave, and +meeting-events shortcuts that the main PR's updated lark-vc documentation routes to lark-vc-agent.

Suggested labels

documentation, domain/vc, size/M

Suggested reviewers

  • zhaoleibd
  • zhicong666-bytedance

Poem

🐰 Three skills now dance in harmony clear,
With boundaries drawn and routing shear,
Calendar schedules, minutes transcript flows,
VC searches deep where the meeting goes,
Through docs refined, the path now glows! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: documentation optimization for three skill files (calendar, vc, and minutes).
Description check ✅ Passed The description follows the template structure with Summary and Changes sections clearly documented, though Test Plan shows 'None' which is acceptable for documentation-only changes.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/vc_event

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 and usage tips.

@github-actions github-actions Bot added domain/calendar PR touches the calendar domain domain/vc PR touches the vc domain size/L Large or sensitive change across domains or core paths labels Jun 4, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions Bot removed the domain/doc Docs domain label Jun 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#feat/vc_event -y -g

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.84%. Comparing base (abc0553) to head (17f9f45).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1269   +/-   ##
=======================================
  Coverage   69.84%   69.84%           
=======================================
  Files         666      666           
  Lines       64451    64451           
=======================================
  Hits        45019    45019           
  Misses      15812    15812           
  Partials     3620     3620           

☔ 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.

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 domain/vc PR touches the vc domain size/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants