Skip to content

docs(skills): add 云盘 alias alongside 云空间 for agent clarity#1071

Closed
fangshuyu-768 wants to merge 4 commits into
mainfrom
fix/skill-add-云盘-alias

Hidden character warning

The head ref may contain hidden characters: "fix/skill-add-\u4e91\u76d8-alias"
Closed

docs(skills): add 云盘 alias alongside 云空间 for agent clarity#1071
fangshuyu-768 wants to merge 4 commits into
mainfrom
fix/skill-add-云盘-alias

Conversation

@fangshuyu-768

@fangshuyu-768 fangshuyu-768 commented May 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

"云空间" and "云盘" refer to the same concept (Lark Drive), but only "云空间" was used across all skill docs. AI agents that encounter "云盘" or "网盘" from user input cannot map these terms to the lark-drive skill, leading to routing failures.

This PR adds "(云盘)" or "/云盘" annotation next to every "云空间" occurrence so agents understand the equivalence.

Changes

  • lark-drive/SKILL.md: Updated description to 飞书云空间(云盘), added explicit routing note for "云盘"/"网盘"/"我的空间"
  • lark-drive/references/ (19 files): Annotated all "云空间" occurrences in search, upload, download, move, delete, import, export, pull, push, status, create-folder, create-shortcut, add-comment, version-*, reactions, task-result docs
  • lark-shared/SKILL.md: Annotated "云空间" in identity/resource descriptions
  • lark-wiki/SKILL.md + references: Annotated "云空间根目录" and "云空间文件夹"
  • lark-sheets/SKILL.md + references: Annotated "云空间" in description and cell-data doc
  • lark-markdown/SKILL.md: Annotated "云空间操作"
  • lark-minutes/SKILL.md + references: Annotated "云空间" in upload flow docs
  • lark-apps/SKILL.md: Annotated "云空间" in exclusion description

Total: 30 files across 8 skills, 59 insertions / 59 deletions (annotation only, no logic change).

Test

  • Verified zero remaining unannotated "云空间" occurrences: grep -rn "云空间" skills/ --include="*.md" | grep -v "云盘" returns no output
  • No code changes, no unit-test impact

Summary by CodeRabbit

  • Documentation
    • Standardized cloud storage terminology, equating "cloud space", "cloud drive", and "cloud storage" across skills
    • Clarified which skills handle file actions (move/rename/delete/search/permissions/comments) vs. markdown handling
    • Expanded import/export and command-selection guidance, including decision rules for mapping local file types to target document types
    • Added prerequisite and workflow notes (e.g., upload-before-use) and improved consistency across references

Review Change Stack

"云空间" and "云盘" refer to the same concept (Lark Drive), but only
"云空间" was used across all skill docs. AI agents that encounter
"云盘" or "网盘" from user input cannot map these terms to the
lark-drive skill. Add "(云盘)" or "/云盘" annotation next to every
"云空间" occurrence so agents understand the equivalence.

Covers 30 files across 8 skills: lark-drive, lark-shared, lark-wiki,
lark-doc, lark-sheets, lark-markdown, lark-minutes, lark-apps.
@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a56c493d-4072-4e66-b206-58fb4ee148ca

📥 Commits

Reviewing files that changed from the base of the PR and between 8bfa8db and ecd9ad6.

📒 Files selected for processing (4)
  • skills/lark-drive/SKILL.md
  • skills/lark-drive/references/lark-drive-create-folder.md
  • skills/lark-drive/references/lark-drive-search.md
  • skills/lark-sheets/SKILL.md

📝 Walkthrough

Walkthrough

This PR standardizes cloud space terminology across the Larksuite CLI skill documentation by systematically clarifying that "云空间" (cloud space), "云盘" (cloud disk), and "云存储" (cloud storage) are equivalent, using the notation "云空间(云盘/云存储)" throughout. Changes span the lark-drive skill core documentation, all 20 command references, and six related skills (lark-apps, lark-markdown, lark-minutes, lark-shared, lark-sheets, lark-wiki).

Changes

Cloud space/disk terminology standardization across skill documentation

Layer / File(s) Summary
lark-drive core: terminology foundation and decision routing
skills/lark-drive/SKILL.md
Establishes "云空间(云盘/云存储)" terminology in the skill description, adds new "术语说明" and "导入分流规则" sections, and rewrites "快速决策" command-routing bullets to use consistent terminology for search, import, markdown, and folder-creation guidance. Renames "Drive Folder" heading to include "云盘/云存储" wording.
lark-drive commands: substantive documentation refinements
skills/lark-drive/references/lark-drive-*.md
Adds prerequisite notes to download, move, and upload to direct readers to shared SKILL.md. Expands search intro with unified discovery entry point concept. Clarifies status default SHA-256 vs. --quick modified_time comparison. Refines create-folder, create-shortcut, delete descriptions and constraints. Clarifies pull and push file-level mirroring semantics.
lark-drive commands: systematic reference terminology updates
skills/lark-drive/references/lark-drive-*.md (20 files)
Bulk updates across add-comment, create-folder, create-shortcut, delete, download, export, export-download, import, move, pull, push, reactions, status, task-result, upload, version-delete, version-get, version-history, version-revert standardizing "参考" section reference text to "云空间(云盘/云存储)全部命令" notation.
Related skills: cross-skill terminology and routing alignment
skills/lark-apps/SKILL.md, skills/lark-markdown/SKILL.md, skills/lark-minutes/SKILL.md, skills/lark-minutes/references/lark-minutes-upload.md, skills/lark-shared/SKILL.md, skills/lark-sheets/SKILL.md, skills/lark-sheets/references/lark-sheets-cell-data.md, skills/lark-wiki/SKILL.md, skills/lark-wiki/references/lark-wiki-move.md
lark-apps extends scope to "云空间/云盘/云存储"; lark-markdown routes cloud-space operations to lark-drive; lark-minutes expands file_token workflow documentation (upload → file_token → minutes +upload → vc +notes); lark-shared expands identity examples; lark-sheets updates heading and description; lark-sheets clarifies +find is not cloud-space search; lark-wiki adds explicit Drive-folder/storage-root routing; lark-wiki-move clarifies drive +move for cloud-space folder organization.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • larksuite/cli#870: Documents the same drive +status exact-vs---quick detection semantics referenced in this PR.
  • larksuite/cli#864: Edits drive +search guidance; related to this PR's search intro wording and scope changes.
  • larksuite/cli#704: Adds lark-markdown shortcuts; this PR routes native .md workflows to that skill.

Suggested reviewers

  • wittam-01

Poem

🐰 I nibble through docs, tidy and bright,
Unifying cloud terms with delight.
云空间、云盘、云存储 — one name,
Now every skill sings the same refrain.
Hooray for clarity, hop hop — good night!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding terminology aliases (云盘) alongside 云空间 to improve agent understanding.
Description check ✅ Passed The PR description is comprehensive and well-structured, covering motivation, detailed changes across all 30 files and 8 skills, and verification approach, though lacking explicit 'Test Plan' section checkbox format.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/skill-add-云盘-alias

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 and usage tips.

@github-actions github-actions Bot added domain/ccm PR touches the ccm domain size/L Large or sensitive change across domains or core paths labels May 25, 2026
"云存储" is another common alias for the same concept (Lark Drive).
Add it to all existing annotations so agents can route "云存储" to
lark-drive just like "云盘" and "网盘".
@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

Unavailable for this PR because source repo/branch metadata is missing.

@codecov

codecov Bot commented May 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.83%. Comparing base (f2a4c95) to head (ecd9ad6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1071      +/-   ##
==========================================
+ Coverage   67.82%   67.83%   +0.01%     
==========================================
  Files         592      592              
  Lines       55307    55307              
==========================================
+ Hits        37512    37518       +6     
+ Misses      14683    14678       -5     
+ Partials     3112     3111       -1     

☔ View full report in Codecov by Sentry.
📢 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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 `@skills/lark-drive/SKILL.md`:
- Around line 15-16: The blockquote in SKILL.md contains an internal blank line
triggering markdownlint MD028; remove the empty line inside the `>` block so the
entire quote ("术语说明: 飞书云空间也常被称为\"云盘\"或\"云存储\",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。") is
continuous on consecutive `>`-prefixed lines to satisfy the no-blanks-blockquote
rule.
🪄 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: d48d0c2d-4e3e-42db-834c-b8ecb369bbc6

📥 Commits

Reviewing files that changed from the base of the PR and between f2a4c95 and 8bfa8db.

📒 Files selected for processing (30)
  • skills/lark-apps/SKILL.md
  • skills/lark-drive/SKILL.md
  • skills/lark-drive/references/lark-drive-add-comment.md
  • skills/lark-drive/references/lark-drive-create-folder.md
  • skills/lark-drive/references/lark-drive-create-shortcut.md
  • skills/lark-drive/references/lark-drive-delete.md
  • skills/lark-drive/references/lark-drive-download.md
  • skills/lark-drive/references/lark-drive-export-download.md
  • skills/lark-drive/references/lark-drive-export.md
  • skills/lark-drive/references/lark-drive-import.md
  • skills/lark-drive/references/lark-drive-move.md
  • skills/lark-drive/references/lark-drive-pull.md
  • skills/lark-drive/references/lark-drive-push.md
  • skills/lark-drive/references/lark-drive-reactions.md
  • skills/lark-drive/references/lark-drive-search.md
  • skills/lark-drive/references/lark-drive-status.md
  • skills/lark-drive/references/lark-drive-task-result.md
  • skills/lark-drive/references/lark-drive-upload.md
  • skills/lark-drive/references/lark-drive-version-delete.md
  • skills/lark-drive/references/lark-drive-version-get.md
  • skills/lark-drive/references/lark-drive-version-history.md
  • skills/lark-drive/references/lark-drive-version-revert.md
  • skills/lark-markdown/SKILL.md
  • skills/lark-minutes/SKILL.md
  • skills/lark-minutes/references/lark-minutes-upload.md
  • skills/lark-shared/SKILL.md
  • skills/lark-sheets/SKILL.md
  • skills/lark-sheets/references/lark-sheets-cell-data.md
  • skills/lark-wiki/SKILL.md
  • skills/lark-wiki/references/lark-wiki-move.md

Comment on lines +15 to +16
> **术语说明:** 飞书云空间也常被称为"云盘"或"云存储",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Remove blank line inside blockquote to satisfy markdownlint MD028.

Line 16 creates a blank line within the > blockquote section and triggers no-blanks-blockquote.

Suggested fix
 > **术语说明:** 飞书云空间也常被称为"云盘"或"云存储",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。
- 
 > **导入分流规则:** 如果用户要把本地 Excel / CSV / `.base` 快照导入成 Base / 多维表格 / bitable,必须优先使用 `lark-cli drive +import --type bitable`。不要先切到 `lark-base`;`lark-base` 只负责导入完成后的表内操作。
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
> **术语说明:** 飞书云空间也常被称为"云盘"或"云存储",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。
> **术语说明:** 飞书云空间也常被称为"云盘"或"云存储",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。
> **导入分流规则:** 如果用户要把本地 Excel / CSV / `.base` 快照导入成 Base / 多维表格 / bitable,必须优先使用 `lark-cli drive +import --type bitable`。不要先切到 `lark-base``lark-base` 只负责导入完成后的表内操作。
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)

[warning] 16-16: Blank line inside blockquote

(MD028, no-blanks-blockquote)

🤖 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-drive/SKILL.md` around lines 15 - 16, The blockquote in SKILL.md
contains an internal blank line triggering markdownlint MD028; remove the empty
line inside the `>` block so the entire quote ("术语说明:
飞书云空间也常被称为\"云盘\"或\"云存储\",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。") is continuous on
consecutive `>`-prefixed lines to satisfy the no-blanks-blockquote rule.

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/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant