Skip to content

docs: clarify mindnote routing in lark-doc skill#1581

Merged
zhanglei-1103 merged 1 commit into
mainfrom
feat/mindnote_cli
Jul 3, 2026
Merged

docs: clarify mindnote routing in lark-doc skill#1581
zhanglei-1103 merged 1 commit into
mainfrom
feat/mindnote_cli

Conversation

@zhanglei-1103

@zhanglei-1103 zhanglei-1103 commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR clarifies the routing boundary for Mindnote handling in the lark-doc skill.
It distinguishes existing Mindnote operations from new Mindnote creation and documents the correct path for each case.

Changes

  • Update skills/lark-doc/SKILL.md to route existing Mindnote requests through lark-doc-mindnote.md
  • Clarify that creating a new Mindnote should follow the lark-doc-whiteboard.md path
  • Update skills/lark-doc/references/lark-doc-mindnote.md to document the supported scope and workflow for existing Mindnotes
  • Add explicit guidance to validate that the provided URL or token is a Mindnote token before using mindnote_id

Test Plan

  • Unit tests pass
  • Manual local verification confirms the documentation changes match the intended Mindnote routing behavior
  • Verified the diff only updates:
    • skills/lark-doc/SKILL.md
    • skills/lark-doc/references/lark-doc-mindnote.md

Related Issues

  • None

@CLAassistant

CLAassistant commented Jun 25, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

skills/lark-doc/SKILL.md adds mindnote support to the skill description, CLI help, and routing rule. A new reference document explains the Mindnote command path, parameter shapes, request body fields, workflow steps, and related links.

Lark-doc Mindnote routing and reference

Layer / File(s) Summary
SKILL.md mindnote routing update
skills/lark-doc/SKILL.md
Front-matter description and cliHelp extend to mindnote operations; quick-decision routing sends mindnote requests through lark-doc, using lark-cli mindnotes nodes list/create for existing mindnotes and lark-doc-whiteboard for new ones.
Mindnote reference document
skills/lark-doc/references/lark-doc-mindnote.md
New documentation covers the entry point, nodes list/nodes create parameters, --data request body fields, workflow guidance, write-operation caution, and related links.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • larksuite/cli#890: Updates the same skills/lark-doc/SKILL.md front-matter description and CLI help guidance.
  • larksuite/cli#1283: Modifies skills/lark-doc/SKILL.md routing and scope guidance in the same quick-decision section.
  • larksuite/cli#1508: Refines the same quick-decision routing matrix in skills/lark-doc/SKILL.md.

Suggested labels

size/M

Suggested reviewers

  • SunPeiYang996
  • fangshuyu-768

🐇 A mindnote hops in with a leafy grin,
nodes list and nodes create are ready to begin.
Lark-doc points the way, clear and bright,
Whiteboard waits for brand-new notes in sight.
Hop hop hooray, the path is neat! 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title accurately summarizes the documentation update about Mindnote routing in the lark-doc skill.
Description check ✅ Passed The PR description matches the required template and includes summary, changes, test plan, and related issues.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mindnote_cli

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.

@github-actions github-actions Bot added domain/ccm PR touches the ccm domain size/M Single-domain feat or fix with limited business impact labels Jun 25, 2026
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.51%. Comparing base (462358a) to head (09394a8).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1581      +/-   ##
==========================================
- Coverage   74.52%   74.51%   -0.02%     
==========================================
  Files         851      852       +1     
  Lines       87155    87655     +500     
==========================================
+ Hits        64952    65312     +360     
- Misses      17231    17315      +84     
- Partials     4972     5028      +56     

☔ 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 Jun 25, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#feat/mindnote_cli -y -g

@github-actions github-actions Bot added size/XL Architecture-level or global-impact change and removed size/M Single-domain feat or fix with limited business impact labels Jun 29, 2026
@zhanglei-1103 zhanglei-1103 force-pushed the feat/mindnote_cli branch 2 times, most recently from 0d69f27 to 1ffeed1 Compare June 29, 2026 02:57
@github-actions github-actions Bot added size/M Single-domain feat or fix with limited business impact and removed size/XL Architecture-level or global-impact change labels Jun 29, 2026
@zhanglei-1103 zhanglei-1103 changed the title Feat/mindnote cli docs: clarify mindnote routing in lark-doc skill Jun 29, 2026
@zhanglei-1103 zhanglei-1103 force-pushed the feat/mindnote_cli branch 5 times, most recently from 7f11324 to 01b394b Compare July 2, 2026 03:39
@zhanglei-1103 zhanglei-1103 merged commit 7db899d into main Jul 3, 2026
24 checks passed
@zhanglei-1103 zhanglei-1103 deleted the feat/mindnote_cli branch July 3, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/ccm PR touches the ccm domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants