docs: clarify native mindnote support boundary

[xu91102]

Summary

Clarify that native Mind Note URLs are not supported by the docs/whiteboard content shortcuts, and guide agents toward Drive-only operations for supported metadata, permission, and organization tasks.

Changes

• Add a specific docs +fetch validation message for /mindnote/ URLs instead of the generic unsupported document URL text.
• Cover the Mind Note URL case in parseDocumentRef tests.
• Update lark-doc, lark-drive, and lark-whiteboard skill boundaries so agents do not route native Mind Note content requests to Docx or whiteboard flows.

Test Plan

• GOPROXY=https://goproxy.cn,direct go test ./shortcuts/doc -run TestParseDocumentRef -count=1
• GOPROXY=https://goproxy.cn,direct go test ./shortcuts/doc -count=1
• git diff --check
• Manual local verification confirms the lark-cli <domain> <command> flow works as expected (not run; this change is limited to parser validation text and skill documentation)

Related Issues

• Fixes docs: clarify native mindnote editing is not currently supported #1494

Summary by CodeRabbit

• Bug Fixes
  • Native Mind Note links (/mindnote/…) are now detected correctly and prevented from being routed as whiteboard content.
  • Unsupported native Mind Note usages now return a clear, parameter-specific validation error.
• Documentation
  • Updated lark-doc, lark-drive, and lark-whiteboard guidance to explicitly exclude native Mind Note content reading/node editing and clarify correct /mindnote/ handling.
• Tests
  • Extended parsing tests to cover native Mind Note URLs and verify structured validation error details.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 00b2f6a7-d6d0-4cef-8f60-f9f468965f73

📥 Commits

Reviewing files that changed from the base of the PR and between 0ad5106 and 92bf149.

📒 Files selected for processing (3)

• skills/lark-doc/SKILL.md
• skills/lark-drive/SKILL.md
• skills/lark-whiteboard/SKILL.md

💤 Files with no reviewable changes (3)

• skills/lark-drive/SKILL.md
• skills/lark-whiteboard/SKILL.md
• skills/lark-doc/SKILL.md

---

📝 Walkthrough

Walkthrough

The PR adds native Mind Note /mindnote/ rejection in document parsing, extends parser test coverage, and updates Docs, Drive, and Whiteboard skill guidance to mark native Mind Note content and node editing as unsupported.

Changes

Mind Note routing and unsupported handling

Layer / File(s)	Summary
Native Mind Note validation shortcuts/doc/helpers.go, shortcuts/doc/helpers_test.go	parseDocumentRef now rejects /mindnote/ doc values with a validation error, and TestParseDocumentRef adds a table case plus structured error assertions for that input.
Drive skill scope and routing skills/lark-drive/SKILL.md	The Drive skill description, quick decision guidance, URL/token table, and scope note now state that /mindnote/ maps to Drive-layer operations only and excludes content reading or node editing.
Docs and whiteboard scope notes skills/lark-doc/SKILL.md, skills/lark-whiteboard/SKILL.md	The Docs and Whiteboard skill descriptions and scope bullets now say /mindnote/ is native Mind Note, not whiteboard, and that native Mind Note content or node editing is unsupported.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• larksuite/cli#1466: Updates validateFetchV2 to validate --doc through parseDocumentRef, so the new /mindnote/ handling affects the same validation path.

Suggested labels

documentation

Suggested reviewers

• SunPeiYang996
• fangshuyu-768
• zhangzq0

Poem

A bunny hopped by in the dawn's soft glow,
“/mindnote/ isn’t whiteboard,” the signs now show.
Drive keeps the paths for metadata bright,
Docs and Whiteboard route with clearer sight.
🐇✨ Hops of clarity through the night.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: clarifying the native Mind Note support boundary.
Description check	✅ Passed	The description follows the template with Summary, Changes, Test Plan, and Related Issues sections.
Linked Issues check	✅ Passed	The PR addresses #1494 by clarifying native Mind Note is unsupported in parser errors and skill routing docs.
Out of Scope Changes check	✅ Passed	The changes stay within documentation and parser validation needed to clarify Mind Note boundaries.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 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/doc/helpers_test.go`:
- Around line 51-55: The negative case in the helpers test currently only checks
the validation message string, so update the assertion in the relevant test case
to verify the typed validation error shape instead. Use the existing error
helpers and `errors.As`/`errs.ProblemOf` in `helpers_test.go` to assert the
expected category, subtype, and `Param` for the native Mind Note URL case, while
still preserving the cause.

🪄 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: 6d50b9d6-6972-49cd-9db1-c76be3d94e05

📥 Commits

Reviewing files that changed from the base of the PR and between 898e6d4 and 8f8638d.

📒 Files selected for processing (5)

• shortcuts/doc/helpers.go
• shortcuts/doc/helpers_test.go
• skills/lark-doc/SKILL.md
• skills/lark-drive/SKILL.md
• skills/lark-whiteboard/SKILL.md