Skip to content

docs: clarify Mindnote token handling#1827

Merged
fangshuyu-768 merged 4 commits into
mainfrom
codex/mindnote-token-guidance
Jul 10, 2026
Merged

docs: clarify Mindnote token handling#1827
fangshuyu-768 merged 4 commits into
mainfrom
codex/mindnote-token-guidance

Conversation

@fangshuyu-768

@fangshuyu-768 fangshuyu-768 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • clarify that mindnote_id is the Mindnote document token, not a node ID
  • document using drive +inspect to resolve Mindnote and Wiki URLs before calling mindnotes
  • update the recommended workflow to confirm a real Mindnote token before node operations

Tests

  • git diff --check

Summary by CodeRabbit

  • Documentation
    • Clarified that --mindnote-id requires the Mindnote document token, not a node ID.
    • Added instructions for retrieving the correct token from Mindnote and Wiki URLs.
    • Updated the recommended workflow to verify the document type before use.
    • Added a reference link for drive inspection.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

新增 Mindnote 文档 token 获取说明,明确 Wiki URL 的解包和类型确认规则,并将推荐工作流更新为使用解析后的真实 token。

Changes

Mindnote token resolution documentation

Layer / File(s) Summary
Mindnote token resolution procedure and workflow integration
skills/lark-doc/references/lark-doc-mindnote.md
新增通过 drive +inspect 获取 Mindnote 文档 token 的流程,更新推荐工作流中的 token 判断与使用方式,并添加 lark-drive 参考链接。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: liangshuo-1, SunPeiYang996

🚥 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 is concise and accurately summarizes the documentation change around Mindnote token handling.
Description check ✅ Passed The description covers the summary and test evidence, but it omits the template's dedicated Changes and Related Issues sections.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/mindnote-token-guidance

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 Jul 9, 2026
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.45%. Comparing base (74d8458) to head (d04b5ad).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1827   +/-   ##
=======================================
  Coverage   74.44%   74.45%           
=======================================
  Files         860      860           
  Lines       89859    89906   +47     
=======================================
+ Hits        66897    66940   +43     
- Misses      17778    17780    +2     
- Partials     5184     5186    +2     

☔ 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 9, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#codex/mindnote-token-guidance -y -g

@fangshuyu-768

fangshuyu-768 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator Author

Verified with a real Wiki-wrapped Mindnote URL.

  • lark-cli drive +inspect --url <wiki-url> --as user --format json unwraps the Wiki node to type: mindnote and returns the canonical Mindnote token.
  • lark-cli mindnotes nodes list --mindnote-id <mindnote-token> --as user --format json succeeds and returns nodes.
  • Passing the Wiki node token directly to --mindnote-id fails with 3410003 resource not found.

This confirms the doc needs to explicitly route Wiki/title lookup through Drive before calling mindnotes nodes list.

@fangshuyu-768
fangshuyu-768 force-pushed the codex/mindnote-token-guidance branch from 117628b to 4153896 Compare July 9, 2026 08:11
@fangshuyu-768 fangshuyu-768 changed the title docs: clarify Mindnote token discovery docs: clarify Mindnote token handling Jul 10, 2026
@github-actions

Copy link
Copy Markdown

PR Quality Summary

The semantic review system could not produce a fully trusted result. This is not reported as a code defect.

System status

  • semantic review decision is missing — Action: inspect semantic-review workflow logs and quality-gate artifact

@fangshuyu-768
fangshuyu-768 merged commit 452734f into main Jul 10, 2026
51 checks passed
@fangshuyu-768
fangshuyu-768 deleted the codex/mindnote-token-guidance branch July 10, 2026 10:43
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 13, 2026
3 tasks
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.

2 participants