Skip to content

docs(skills): add 云盘/云存储 alias alongside 云空间 for agent clarity#1073

Merged
fangshuyu-768 merged 1 commit into
mainfrom
fix/skill-add-drive-alias
May 25, 2026
Merged

docs(skills): add 云盘/云存储 alias alongside 云空间 for agent clarity#1073
fangshuyu-768 merged 1 commit into
mainfrom
fix/skill-add-drive-alias

Conversation

@fangshuyu-768
Copy link
Copy Markdown
Collaborator

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

Summary

"云空间" (Cloud Space), "云盘" (Cloud Drive), and "云存储" (Cloud Storage) are different names for the same product — Lark's cloud file storage and management center. However, only "云空间" was used across all skill docs, causing AI agents to fail routing when users say "云盘", "云存储", or "网盘".

This PR adds "(云盘/云存储)" or "/云盘/云存储" annotations next to every "云空间" occurrence, and adds a terminology callout at the top of lark-drive SKILL.md, so agents understand the equivalence.

8 skills, 30 files, documentation-only, no logic changes.

Summary by CodeRabbit

  • Documentation
    • Standardized cloud-storage terminology: many skill docs now include "cloud drive / cloud storage" alongside "cloud space".
    • Expanded and clarified routing & usage guidance across drive-related actions (upload/download/import/move/search/versioning/versions/comments/permissions).
    • Added terminology callout and refined quick-decision guidance for Markdown/Drive handling.
    • Updated workflows and notes (e.g., upload → obtain file token → convert) and polished identity/formatting tables.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

📝 Walkthrough

Walkthrough

Docs updated across multiple skills to standardize "云空间/云盘/云存储" phrasing, expand lark-drive SKILL.md routing/decision guidance, and route cloud-storage file/folder operations to lark-drive; minutes upload docs now require pre-obtained file_token.

Changes

Cloud Storage Terminology and Skill Routing

Layer / File(s) Summary
lark-drive skill definition and routing rules
skills/lark-drive/SKILL.md
Expanded skill description to clarify "云空间/云盘/云存储" equivalence, added terminology note, and refined "快速决策" routing for drive +search, drive +import --type mapping, native .md delegation to lark-markdown, and Drive Folder heading.
Drive references terminology expansion
skills/lark-drive/references/lark-drive-*.md (many files)
Updated "参考" entries and headings across drive reference docs to expand "云空间" wording to include "云盘/云存储" and refined a few command intro lines (upload/download/push/pull/status/import/move/delete/create-*).
Cross-skill routing and integration
skills/lark-markdown/SKILL.md, skills/lark-wiki/SKILL.md, skills/lark-wiki/references/lark-wiki-move.md, skills/lark-sheets/SKILL.md, skills/lark-sheets/references/lark-sheets-cell-data.md, skills/lark-apps/SKILL.md, skills/lark-minutes/SKILL.md, skills/lark-minutes/references/lark-minutes-upload.md, skills/lark-shared/SKILL.md
Aligned terminology and clarified that cloud-storage operations (move/rename/delete/search/permissions/comments) should be handled by lark-drive; clarified minutes upload prerequisite (must obtain file_token from Drive upload first).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • larksuite/cli#725: Related to minutes upload shortcut and --file-token usage in minutes flow.
  • larksuite/cli#704: Related lark-markdown docs/shortcuts for native .md handling referenced by this PR.
  • larksuite/cli#444: Related drive "title change" docs and files.patch capability referenced in drive docs.

Suggested labels

size/M

Suggested reviewers

  • wittam-01
  • SunPeiYang996

"🐰
云盘与云空间并肩行,
文档轻跳着把词对齐。
技能互指路共清明,
小兔欢跃把草啃起。 🥕"

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding 云盘/云存储 aliases alongside 云空间 across skill documentation for improved agent routing clarity.
Linked Issues check ✅ Passed The PR fully addresses all coding and documentation objectives from issue #1071: annotating all 云空间 occurrences with 云盘/云存储 aliases across 30 files in 8 skills.
Out of Scope Changes check ✅ Passed All changes are documentation annotations adding 云盘/云存储 terminology across skill docs; no unrelated out-of-scope modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description covers all required template sections: Summary clearly explains motivation, Changes implicitly listed through PR objectives, and context provided. All critical information is present.

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

✨ 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 fix/skill-add-drive-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
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 25, 2026

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#fix/skill-add-drive-alias -y -g

@fangshuyu-768 fangshuyu-768 force-pushed the fix/skill-add-drive-alias branch from 564dd40 to d2b5b2c Compare May 25, 2026 08:52
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1073   +/-   ##
=======================================
  Coverage   67.82%   67.82%           
=======================================
  Files         592      592           
  Lines       55307    55307           
=======================================
  Hits        37512    37512           
  Misses      14683    14683           
  Partials     3112     3112           

☔ 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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

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 containing the Chinese sentence "**术语说明:**
飞书云空间也常被称为\"云盘\"或\"云存储\",三者指的是同一个产品,是飞书官方的云端文件存储与管理中心。" has an extra blank line
that breaks the blockquote (MD028); remove the blank line or prefix the
blank/continuation line with ">" so the blockquote remains continuous and
conforms to markdownlint rules.
🪄 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: 71004f09-6b14-493a-b816-f931bfa934b8

📥 Commits

Reviewing files that changed from the base of the PR and between 3338410 and d2b5b2c.

📒 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
✅ Files skipped from review due to trivial changes (24)
  • skills/lark-sheets/references/lark-sheets-cell-data.md
  • skills/lark-drive/references/lark-drive-add-comment.md
  • skills/lark-sheets/SKILL.md
  • skills/lark-drive/references/lark-drive-export.md
  • skills/lark-drive/references/lark-drive-move.md
  • skills/lark-drive/references/lark-drive-export-download.md
  • skills/lark-drive/references/lark-drive-version-revert.md
  • skills/lark-drive/references/lark-drive-create-shortcut.md
  • skills/lark-drive/references/lark-drive-import.md
  • skills/lark-drive/references/lark-drive-status.md
  • skills/lark-drive/references/lark-drive-download.md
  • skills/lark-drive/references/lark-drive-upload.md
  • skills/lark-drive/references/lark-drive-version-history.md
  • skills/lark-markdown/SKILL.md
  • skills/lark-minutes/SKILL.md
  • skills/lark-wiki/SKILL.md
  • skills/lark-drive/references/lark-drive-push.md
  • skills/lark-drive/references/lark-drive-pull.md
  • skills/lark-shared/SKILL.md
  • skills/lark-wiki/references/lark-wiki-move.md
  • skills/lark-drive/references/lark-drive-reactions.md
  • skills/lark-minutes/references/lark-minutes-upload.md
  • skills/lark-drive/references/lark-drive-search.md
  • skills/lark-drive/references/lark-drive-version-get.md

Comment thread skills/lark-drive/SKILL.md
@fangshuyu-768 fangshuyu-768 merged commit 282c277 into main May 25, 2026
32 of 34 checks passed
@fangshuyu-768 fangshuyu-768 deleted the fix/skill-add-drive-alias branch May 25, 2026 09:15
@liangshuo-1 liangshuo-1 mentioned this pull request May 25, 2026
2 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/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants