fix: add vc-domain-boundaries and enrich vc +notes#1172
Open
hugang-lark wants to merge 1 commit into
Open
Conversation
hugang-lark
added
documentation
|
|
github-actions
Bot
added
the
size/L
📝 WalkthroughWalkthroughThis PR extends the ChangesVC Meeting Note Artifact Retrieval
Search Shortcut Filter Description
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
skills/lark-vc/references/vc-domain-boundaries.md (1)
58-68: ⚡ Quick winAdd language identifiers to code blocks for better rendering.
The code blocks at lines 58-68 (transcript format examples) and lines 142-155 (domain relationship diagram) are missing language identifiers, which can cause inconsistent rendering across markdown processors.
📝 Proposed fix
-``` +```text 发言人名称 相对时间戳 <发言内容>Apply the same fix to the example at lines 65-68 and the diagram at lines 142-155: ```diff -``` +```text Calendar (日程) ──── 发起预约 ────► VC (会议) ...</details> Also applies to: 142-155 <details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@skills/lark-vc/references/vc-domain-boundaries.mdaround lines 58 - 68, The
shown Markdown code blocks for the transcript example ("发言人名称 相对时间戳\n<发言内容>" /
example "张三 00:00:00.195\n我们接下来讨论一下项目进度。") and the domain relationship diagram
("Calendar (日程) ──── 发起预约 ────► VC (会议) ...") lack language identifiers; update
their fences fromtotext for both the transcript block and the diagram
block so they render consistently across processors (locate the blocks
containing the exact strings "发言人名称 相对时间戳" and "Calendar (日程) ──── 发起预约 ────► VC
(会议)" and replace the opening triple backticks with ```text).</details> </blockquote></details> </blockquote></details> <details> <summary>🤖 Prompt for all review comments with AI agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.Nitpick comments:
In@skills/lark-vc/references/vc-domain-boundaries.md:
- Around line 58-68: The shown Markdown code blocks for the transcript example
("发言人名称 相对时间戳\n<发言内容>" / example "张三 00:00:00.195\n我们接下来讨论一下项目进度。") and the
domain relationship diagram ("Calendar (日程) ──── 发起预约 ────► VC (会议) ...") lack
language identifiers; update their fences fromtotext for both the
transcript block and the diagram block so they render consistently across
processors (locate the blocks containing the exact strings "发言人名称 相对时间戳" and
"Calendar (日程) ──── 发起预约 ────► VC (会议)" and replace the opening triple backticks
with ```text).</details> --- <details> <summary>ℹ️ Review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro **Run ID**: `e81d6c81-595f-4e99-9a29-77935e08a415` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between e18ea9a2e824472c41c279ee80933a5d559a3d86 and a1d446d4df26e534f59e35cd85fd6e2a8596ba46. </details> <details> <summary>📒 Files selected for processing (8)</summary> * `shortcuts/vc/vc_notes.go` * `shortcuts/vc/vc_search.go` * `skills/lark-minutes/SKILL.md` * `skills/lark-vc/SKILL.md` * `skills/lark-vc/references/lark-vc-notes.md` * `skills/lark-vc/references/lark-vc-search.md` * `skills/lark-vc/references/vc-domain-boundaries.md` * `skills/lark-workflow-meeting-summary/SKILL.md` </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@6398a9ea9d88a546334fd835285f4e5a9e73994e🧩 Skill updatenpx skills add larksuite/cli#feat/vc_event -y -g |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1172 +/- ##
==========================================
+ Coverage 68.92% 68.95% +0.02%
==========================================
Files 629 629
Lines 58762 58832 +70
==========================================
+ Hits 40503 40567 +64
- Misses 14952 14960 +8
+ Partials 3307 3305 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
hugang-lark
changed the title
hugang-lark
force-pushed
the
feat/vc_event
branch
from
a1d446d to
6dc95c7
Compare
hugang-lark
changed the title
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
skills/lark-vc/references/vc-domain-boundaries.md (2)
58-68: ⚡ Quick winAdd language identifier to fenced code blocks.
The code blocks showing transcript format examples should specify a language identifier for proper Markdown rendering.
📝 Proposed fix to add language identifiers
-``` +```text 发言人名称 相对时间戳 <发言内容>示例：
-
+text
张三 00:00:00.195
我们接下来讨论一下项目进度。🤖 Prompt for 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. In `@skills/lark-vc/references/vc-domain-boundaries.md` around lines 58 - 68, The fenced code blocks showing the transcript format (the block starting with "发言人名称 相对时间戳" and the example block starting with "张三 00:00:00.195") need a language identifier for proper Markdown rendering; update both triple-backtick fences to use "text" (i.e., change ``` to ```text) so the blocks explicitly declare the language and render correctly.
142-155: ⚡ Quick winAdd language identifier to ASCII diagram code block.
The code block containing the ASCII diagram should specify a language identifier for proper Markdown rendering.
📝 Proposed fix to add language identifier
-``` +```text Calendar (日程) ──── 发起预约 ────► VC (会议) │ ┌──────────────────┤🤖 Prompt for 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. In `@skills/lark-vc/references/vc-domain-boundaries.md` around lines 142 - 155, The ASCII diagram code block in vc-domain-boundaries.md lacks a language identifier; update the fenced code block that starts with the diagram (the triple-backtick block containing "Calendar (日程) ──── 发起预约") to include a language tag such as "text" (i.e., change ``` to ```text) so Markdown renderers treat it as preformatted text and preserve the diagram formatting.
🤖 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 `@shortcuts/vc/vc_notes.go`:
- Around line 778-787: The success counting incorrectly treats echoed
minute_token entries as successes because hasNotesPayload sees a non-empty
"minute_token"; modify the successCount logic in Execute to be mode-aware: when
running the --minute-tokens path (the mode enforced by Execute/ExactlyOne),
count a result as successful only if the result map from fetchNoteByMinuteToken
contains no non-nil "error" (or error == nil), otherwise for the other paths
keep the existing hasNotesPayload check; update the loop over results (where
successCount is computed) to branch on the mode and check
r.(map[string]any)["error"] for minute-token mode so the all-failed exit and
summary match the per-row FAIL statuses, and re-run hasNotesPayload/minute-token
tests.
---
Nitpick comments:
In `@skills/lark-vc/references/vc-domain-boundaries.md`:
- Around line 58-68: The fenced code blocks showing the transcript format (the
block starting with "发言人名称 相对时间戳" and the example block starting with "张三
00:00:00.195") need a language identifier for proper Markdown rendering; update
both triple-backtick fences to use "text" (i.e., change ``` to ```text) so the
blocks explicitly declare the language and render correctly.
- Around line 142-155: The ASCII diagram code block in vc-domain-boundaries.md
lacks a language identifier; update the fenced code block that starts with the
diagram (the triple-backtick block containing "Calendar (日程) ──── 发起预约") to
include a language tag such as "text" (i.e., change ``` to ```text) so Markdown
renderers treat it as preformatted text and preserve the diagram formatting.
🪄 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: e8ce7a08-f8ac-41fd-af75-f6c29bdf9e58
📒 Files selected for processing (8)
shortcuts/vc/vc_notes.goshortcuts/vc/vc_notes_test.goshortcuts/vc/vc_search.goskills/lark-minutes/SKILL.mdskills/lark-vc/SKILL.mdskills/lark-vc/references/lark-vc-notes.mdskills/lark-vc/references/vc-domain-boundaries.mdskills/lark-workflow-meeting-summary/SKILL.md
✅ Files skipped from review due to trivial changes (3)
- shortcuts/vc/vc_search.go
- skills/lark-vc/SKILL.md
- skills/lark-vc/references/lark-vc-notes.md
🚧 Files skipped from review as they are similar to previous changes (2)
- skills/lark-minutes/SKILL.md
- skills/lark-workflow-meeting-summary/SKILL.md
hugang-lark
force-pushed
the
feat/vc_event
branch
from
6dc95c7 to
6398a9e
Compare
There was a problem hiding this comment.
🧹 Nitpick comments (2)
skills/lark-vc/references/vc-domain-boundaries.md (2)
58-62: ⚡ Quick winAdd language specifiers to code blocks.
The code blocks showing transcript format examples should include a language specifier for proper syntax highlighting and linting compliance.
📝 Proposed fix
-``` +```text 发言人名称 相对时间戳 <发言内容>-
+text
张三 00:00:00.195
我们接下来讨论一下项目进度。Also applies to: 65-68
🤖 Prompt for 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. In `@skills/lark-vc/references/vc-domain-boundaries.md` around lines 58 - 62, Add a language specifier to the transcript example code fences (change triple-backtick blocks from ``` to ```text) so they use "text" for proper highlighting; update the example blocks showing "发言人名称 相对时间戳\n<发言内容>" and the sample with "张三 00:00:00.195\n我们接下来讨论一下项目进度。" (and the other similar block later) to begin with ```text instead of ```, ensuring both code fences are updated.
142-155: ⚡ Quick winAdd language specifier to ASCII diagram code block.
The ASCII art diagram should include a language specifier for markdown linting compliance.
📝 Proposed fix
-``` +```text Calendar (日程) ──── 发起预约 ────► VC (会议) │ ┌──────────────────┤🤖 Prompt for 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. In `@skills/lark-vc/references/vc-domain-boundaries.md` around lines 142 - 155, The markdown ASCII diagram block in vc-domain-boundaries.md lacks a language specifier; update the opening code fence (the triple backticks that begin the ASCII art diagram) to include a language tag (e.g., "text") so the block becomes ```text to satisfy linting; ensure only the opening fence is changed and the closing fence remains ``` so the diagram content (the VC diagram) is unchanged.
🤖 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.
Nitpick comments:
In `@skills/lark-vc/references/vc-domain-boundaries.md`:
- Around line 58-62: Add a language specifier to the transcript example code
fences (change triple-backtick blocks from ``` to ```text) so they use "text"
for proper highlighting; update the example blocks showing "发言人名称 相对时间戳\n<发言内容>"
and the sample with "张三 00:00:00.195\n我们接下来讨论一下项目进度。" (and the other similar
block later) to begin with ```text instead of ```, ensuring both code fences are
updated.
- Around line 142-155: The markdown ASCII diagram block in
vc-domain-boundaries.md lacks a language specifier; update the opening code
fence (the triple backticks that begin the ASCII art diagram) to include a
language tag (e.g., "text") so the block becomes ```text to satisfy linting;
ensure only the opening fence is changed and the closing fence remains ``` so
the diagram content (the VC diagram) is unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: aad06df8-5a51-4e07-9041-1dd073e7f047
📒 Files selected for processing (8)
shortcuts/vc/vc_notes.goshortcuts/vc/vc_notes_test.goshortcuts/vc/vc_search.goskills/lark-minutes/SKILL.mdskills/lark-vc/SKILL.mdskills/lark-vc/references/lark-vc-notes.mdskills/lark-vc/references/vc-domain-boundaries.mdskills/lark-workflow-meeting-summary/SKILL.md
✅ Files skipped from review due to trivial changes (3)
- shortcuts/vc/vc_search.go
- skills/lark-minutes/SKILL.md
- skills/lark-vc/references/lark-vc-notes.md
🚧 Files skipped from review as they are similar to previous changes (3)
- skills/lark-workflow-meeting-summary/SKILL.md
- shortcuts/vc/vc_notes.go
- shortcuts/vc/vc_notes_test.go
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
Add a Calendar/VC/Doc cross-domain boundaries document and consequence-driven CRITICAL references across the
lark-vc/lark-minutes/lark-workflow-meeting-summaryskills, and harden thevc +notesshortcut so meeting-id queries also surface theminute_tokenand translate common API error codes into actionable hints. This prevents AI agents from skipping domain knowledge or making wrong artifact selections, and gives callers a single, structured response that covers both the AI-summary and recording pipelines in one shot.Changes
skills/lark-vc/references/vc-domain-boundaries.md: Calendar/VC/Doc relationships, the two independent meeting artifact pipelines, artifact selection rules, transcript format, and the 4-step meeting summary workflow.lark-vc/SKILL.md,lark-minutes/SKILL.md,lark-workflow-meeting-summary/SKILL.md,lark-vc-notes.md, andlark-vc-search.mdso key decision points point to the boundaries doc.[2091005] permission denyhandling tip forminutes.getinlark-minutes/SKILL.md.vc +notes(shortcuts/vc/vc_notes.go): reverse-lookupminute_tokenvia the recording API on the--meeting-ids/--calendar-event-idspaths, merge note-side and minute-side failures into a single;-joinederrorfield, and translate notes 121005 / recording 121004·121005·124002 / minutes 2091005 into friendly hints; update the docs and dry-run output accordingly.Test Plan
make unit-testpasseslark-cli vc +search,lark-cli vc +notes, andlark-cli minutes minutes getbehave as expected (including the newminute_tokenfield and the friendly error hints)Related Issues
Summary by CodeRabbit
New Features
Improvements
Documentation