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
liuxinyanglxy pushed a commit that referenced this pull request Aug 5, 2026
)

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