Skip to content

docs: surface minutes permission application in skill description - #1890

Merged
calendar-assistant merged 1 commit into
mainfrom
docs/minutes-apply-permission-description
Jul 14, 2026
Merged

docs: surface minutes permission application in skill description#1890
calendar-assistant merged 1 commit into
mainfrom
docs/minutes-apply-permission-description

Conversation

@calendar-assistant

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

Copy link
Copy Markdown
Collaborator

Summary

Fixes the issue where the "apply for minutes permission" intent could not route to the lark-minutes skill.

The +apply-permission shortcut was already documented in the skill body, but the front-matter description didn't mention it — so requests to apply for minutes permission failed to match the minutes commands. This adds the capability and its trigger to the description.

Summary by CodeRabbit

  • Documentation
    • Clarified the entry conditions for the minutes skill, including when a minute_token or local audio/video input is provided.
    • Updated guidance on routing: local audio/video should be converted via the minutes skill workflow (upload + minutes + detail) rather than local transcription tools.
    • Reworded the skill description to explicitly cover requesting permission before proceeding.

@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 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

Run ID: 8d4c8f9e-152d-4c2d-ad79-63fcd6c5e0a1

📥 Commits

Reviewing files that changed from the base of the PR and between 2a60f62 and f189163.

📒 Files selected for processing (1)
  • skills/lark-minutes/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • skills/lark-minutes/SKILL.md

📝 Walkthrough

Walkthrough

The lark-minutes skill description clarifies activation conditions and routes local audio/video transcription through the minutes upload and detail flow instead of local tools.

Changes

Minutes skill routing

Layer / File(s) Summary
Clarify minutes skill entry and routing
skills/lark-minutes/SKILL.md
The frontmatter description explicitly covers minute_token, local audio/video, permission requests, and routing local conversion through the minutes skill flow.

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

Possibly related PRs

  • larksuite/cli#591: Updates the same minutes skill’s routing guidance.
  • larksuite/cli#732: Documents local media routing through the minutes upload and detail flow.
  • larksuite/cli#793: Updates documentation for the same minutes skill’s artifact and upload flow.

Suggested labels: domain/ccm

Suggested reviewers: zhaoleibd, hugang-lark, fangshuyu-768, caojie0621

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes a Summary, but it omits required Changes, Test Plan, and Related Issues sections. Add the missing Changes, Test Plan, and Related Issues sections, and note how the change was verified.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the docs update to the minutes skill description.
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/minutes-apply-permission-description

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.

@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.66%. Comparing base (ac2508d) to head (f189163).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1890   +/-   ##
=======================================
  Coverage   74.66%   74.66%           
=======================================
  Files         878      878           
  Lines       91764    91764           
=======================================
  Hits        68520    68520           
  Misses      17930    17930           
  Partials     5314     5314           

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

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#docs/minutes-apply-permission-description -y -g

The lark-minutes SKILL.md body already documents the +apply-permission
shortcut, but the front-matter description omitted it, so the "actively
apply for minutes permission" intent could not route to this skill. Add
the capability and its trigger condition to the description.
@calendar-assistant
calendar-assistant force-pushed the docs/minutes-apply-permission-description branch from 2a60f62 to f189163 Compare July 14, 2026 13:12
@calendar-assistant
calendar-assistant merged commit 8acd55e into main Jul 14, 2026
39 checks passed
@calendar-assistant
calendar-assistant deleted the docs/minutes-apply-permission-description branch July 14, 2026 13:27
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 15, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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