docs(vc): default transcript routing to smart notes over minutes - #1961
Merged
Conversation
github-actions
Bot
added
domain/vc
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUpdates VC documentation to clarify post-meeting authorization for AI summaries and Minutes, and formalizes artifact-selection defaults based on availability and explicit user requests. ChangesVC artifact guidance
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
hugang-lark
approved these changes
Jul 20, 2026
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/lark-vc/references/vc-domain-boundaries.md`:
- Around line 61-65: Update the “智能纪要 vs 妙记的选择规则” section to distinguish the
source used for independent “总结/提炼” operations from AI-summary output: when the
user requests an independently generated summary, derive it from the selected
transcript rather than reusing 智能纪要’s existing summary. Preserve the existing
explicit-user-selection and default source rules for transcript and other
repeated-artefact requests.
In `@skills/lark-vc/SKILL.md`:
- Line 59: Replace the universal “original meeting recording video” wording in
the Minutes authorization description with content-neutral raw audio/video
wording, while preserving the existing authorization rules. Apply the same
qualification in skills/lark-vc/SKILL.md lines 59-59 and
skills/lark-vc/references/vc-domain-boundaries.md lines 48-49.
- Around line 74-77: Make the “智能纪要 vs 妙记的选择规则” and “逐字稿路由” blockquote
contiguous by removing the blank lines between quoted paragraphs or prefixing
those continuation lines with “>”, while preserving the existing text and
Markdown structure.
- Around line 68-74: The artifact-selection rules must be operation-specific: in
skills/lark-vc/SKILL.md lines 68-74, preserve and explicitly prioritize the
transcript-only independent-analysis rule for 提炼/总结/重新总结/整理会议内容/回顾会议, while
keeping AI artifacts for direct AI-summary, todo, and chapter requests; in
skills/lark-vc/references/vc-domain-boundaries.md lines 61-65, clarify that the
generic default of 智能纪要及其逐字稿 does not authorize reusing the AI summary for
independent summarization.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 8fad0d2b-622e-4937-9a0b-ac4249c2bfed
📒 Files selected for processing (2)
skills/lark-vc/SKILL.mdskills/lark-vc/references/vc-domain-boundaries.md
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@8ecdd374c7626750977a1d55a9ac85ead953e457🧩 Skill updatenpx skills add larksuite/cli#docs/vc-minutes-transcript-routing -y -g |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1961 +/- ##
=======================================
Coverage 74.98% 74.98%
=======================================
Files 895 895
Lines 94379 94379
=======================================
Hits 70767 70767
Misses 18197 18197
Partials 5415 5415 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Clarify that smart notes (AI summary) and their verbatim docs are auto-authorized to participants, while minutes carry the raw recording and require explicit authorization. Rewrite the artifact-selection rule to cover transcripts: use whichever exists when only one is present, follow the user's explicit choice, and default to smart notes when both exist and the user is unspecified.
calendar-assistant
force-pushed
the
docs/vc-minutes-transcript-routing
branch
from
July 20, 2026 07:53
fe9a17e to
8ecdd37
Compare
hugang-lark
approved these changes
Jul 20, 2026
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Agents processing meeting content tend to route to Minutes (妙记) artifacts even when the user only wants a transcript, then hit the Minutes authorization wall. The root cause is skill wording: "smart notes vs minutes" priority was only defined for summary/todo, and transcript was listed as an equal
note OR minuteschoice. This PR encodes the correct default in thelark-vcskill docs.Changes
skills/lark-vc/SKILL.md: note the access-permission asymmetry in the Minutes concept (smart notes + verbatim docs are auto-authorized to participants; minutes carry the raw recording and require explicit authorization). In the artifact-selection table, drop the flator minutesdefault and add a selection rule.skills/lark-vc/references/vc-domain-boundaries.md: add an "authorization" note to each of the two artifact pipelines, and rewrite the pipeline-priority rule so it covers transcripts, not just summary/todo.Test Plan
go test ./internal/qualitygate/skillscan/...passesgofmt -l .produces no output (docs-only change, no Go code touched)Related Issues
Summary by CodeRabbit