Skip to content

docs: clarify minutes file-to-notes routing - #732

Merged
calendar-assistant merged 1 commit into
mainfrom
docs/minutes-file-to-notes-routing
Apr 30, 2026
Merged

docs: clarify minutes file-to-notes routing#732
calendar-assistant merged 1 commit into
mainfrom
docs/minutes-file-to-notes-routing

Conversation

@calendar-assistant

@calendar-assistant calendar-assistant commented Apr 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Clarify the skill routing for local audio/video files when the user wants meeting notes, transcripts, written text, summaries, tasks, or chapters. The documented path is now drive +upload -> minutes +upload -> vc +notes --minute-tokens, rather than local transcription tools or starting directly from vc +notes.

Changes

  • Expand lark-minutes trigger and routing guidance for local media files that should become notes or transcript-like outputs.
  • Document the follow-up vc +notes --minute-tokens step after minutes +upload returns a minute_url.
  • Add a matching lark-vc boundary note so file-based requests start from the Minutes upload flow.

Test Plan

  • Unit tests pass (not run; docs-only change)
  • Manual local verification confirms the lark xxx command works as expected (not run; docs-only change)

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Expanded guidance on converting local audio/video files into various document types (meeting notes, transcripts, summaries).
    • Clarified workflow steps for retrieving transcripts and AI-generated outputs.
    • Updated routing rules for file conversion and post-conversion product retrieval.

Change-Id: If768200b329c5e255b13c1992b8c57d1fd8ec518
@CLAassistant

CLAassistant commented Apr 30, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added domain/vc PR touches the vc domain size/L Large or sensitive change across domains or core paths labels Apr 30, 2026
@coderabbitai

coderabbitai Bot commented Apr 30, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f5efdd89-b0b8-4d39-aa64-3b65d9aa0cb7

📥 Commits

Reviewing files that changed from the base of the PR and between f27b8fd and 923656a.

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

📝 Walkthrough

Walkthrough

The PR updates documentation across three skill files to formalize the workflow for converting local audio/video files into minutes-derived artifacts (transcripts, notes, summaries). It specifies that conversion requests must first use the minutes upload entry point to produce minute_url, then extract minute_token and retrieve desired outputs via vc +notes.

Changes

Cohort / File(s) Summary
Minutes Skill Documentation
skills/lark-minutes/SKILL.md, skills/lark-minutes/references/lark-minutes-upload.md
Expands documented scope and routing to cover converting local audio/video into multiple artifact types (transcripts, notes, summaries, todos, chapters). Adds workflow steps for extracting minute_token and retrieving outputs via vc +notes.
VC Skill Documentation
skills/lark-vc/SKILL.md
Adds guidance specifying that local audio/video conversion must first complete the minutes upload flow to produce minute_token before invoking vc +notes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

size/M

Poem

🐰 Through twisted paths, we now see clear,
A workflow true, so bright and dear,
Upload the files, extract the key,
Then fetch the notes for all to see!
Routes now organized, no more roam,
Our documentation's found its home. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: clarifying the routing documentation for converting local audio/video files into meeting notes through the minutes skill.
Description check ✅ Passed The description covers all main sections from the template with appropriate detail: summary explains the change, changes lists the specific modifications, test plan acknowledges the docs-only nature, and related issues is included.
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 docs/minutes-file-to-notes-routing

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
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

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

@codecov

codecov Bot commented Apr 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.38%. Comparing base (f27b8fd) to head (923656a).
⚠️ Report is 387 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #732   +/-   ##
=======================================
  Coverage   64.38%   64.38%           
=======================================
  Files         512      512           
  Lines       45426    45426           
=======================================
  Hits        29249    29249           
  Misses      13604    13604           
  Partials     2573     2573           

☔ 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

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#docs/minutes-file-to-notes-routing -y -g

@calendar-assistant
calendar-assistant merged commit 4fab062 into main Apr 30, 2026
20 of 22 checks passed
@calendar-assistant
calendar-assistant deleted the docs/minutes-file-to-notes-routing branch April 30, 2026 08:59
HomyeeKing pushed a commit to HomyeeKing/cli that referenced this pull request May 6, 2026
Change-Id: If768200b329c5e255b13c1992b8c57d1fd8ec518
tuxedomm pushed a commit to zhumiaoxin/cli that referenced this pull request Jun 6, 2026
Change-Id: If768200b329c5e255b13c1992b8c57d1fd8ec518
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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