Skip to content

docs: add lark drive knowledge organization workflow#1028

Open
YH-1600 wants to merge 1 commit into
mainfrom
feat/workflow-knowledgeOrganize
Open

docs: add lark drive knowledge organization workflow#1028
YH-1600 wants to merge 1 commit into
mainfrom
feat/workflow-knowledgeOrganize

Conversation

@YH-1600
Copy link
Copy Markdown
Collaborator

@YH-1600 YH-1600 commented May 22, 2026

Change-Id: I2343fcdf26ceefb898cc8d4faeae4b17384cfea8

Summary

Add a lark-drive knowledge organization workflow reference for Drive/Wiki/document-library organization scenarios.

Changes

  • Add a quick-decision rule in skills/lark-drive/SKILL.md to route cloud storage / knowledge-base organization requests to the workflow.
  • Add skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md covering entry parsing, inventory, deduplication, planning, write confirmation, permission handling, and verification.

Test Plan

  • Documentation-only change; no Go behavior changed.
  • Verified referenced skill/shortcut commands and workflow links.
  • CI checks pass.

Related Issues

  • None

Summary by CodeRabbit

  • New Features

    • Added a quick-decision rule requiring users to consult the organization workflow guide before organizing cloud storage contents, including drives, folders, and document libraries.
  • Documentation

    • New comprehensive workflow documentation for Drive/Wiki/document library organization, detailing discovery processes, planning procedures, and confirmation requirements for write operations.

Review Change Stack

Change-Id: I2343fcdf26ceefb898cc8d4faeae4b17384cfea8
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 22, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

📝 Walkthrough

Walkthrough

Added a skill decision rule routing cloud storage organization requests to a new comprehensive workflow document covering inventory analysis, problem identification, organization planning, and execution with detailed validation rules, permission controls, and output specifications.

Changes

Knowledge Organization Workflow for Drive/Wiki

Layer / File(s) Summary
Workflow Foundation & Decision Rule
skills/lark-drive/SKILL.md, skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md
SKILL.md adds quick-decision routing for organization/inventory/plan requests. Workflow document establishes pre-work authorization, permission rules, workflow goals and boundaries, success criteria, and capability-to-skill mapping across phases.
Inventory & Analysis Steps (1–4)
skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md
Entry parsing branches by source (Drive, Wiki, folder token, search, single resource) with scale controls. Recursive inventory implements depth/threshold truncation with partial=true markers. ResourceItem standardization defines field constraints and deduplication/merge rules across sources. Current-state statistics specify minimum metrics and optional dimensions.
Organization Planning Steps (5–9)
skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md
Content reading strategy applies outline/keyword/section reads only for low-confidence items with needs_review=true on failure. Problem identification and rule generation classify items by confidence and uncertainty into pending/temporary buckets. Target directory tree requires 2–3 levels with consistency to move plan. Move/create plan defines action, reason, evidence, confidence, and needs_review columns with example rows.
Execution, Verification & Output
skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md
Step 10 defines write-operation whitelist, target_path to token mapping, create/move ordering, and Drive/Wiki async task commands. Step 11 verifies plan execution with result table (expected/actual path, token, status, reason). Final output format segments scope/analysis/problems/rules/tree/plan/confirmation/results; results shown only after user confirmation. Data rules enforce path priority, local read preference, conservative execution, and batch limits. Permission scope table maps commands to required scopes. References link to related skills and workflows.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested labels

domain/base, size/L

Suggested reviewers

  • kongenpei
  • zhouyue-bytedance

Poem

🐰 A rabbit hops through folders deep,
Organizing knowledge, secrets to keep—
Drive and Wiki, now sorted clean,
Workflow rules ensure pristine!
Eleven steps to plan it right,
Then execute with user's might! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: adding documentation for a lark drive knowledge organization workflow.
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.
Description check ✅ Passed The pull request description includes all required template sections: Summary (clear 1-sentence scope), Changes (two concrete items detailing the modifications), Test Plan (documented verification approach), and Related Issues (explicitly stated as None).

✏️ 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 feat/workflow-knowledgeOrganize

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 the size/L Large or sensitive change across domains or core paths label May 22, 2026
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/references/lark-drive-workflow-knowledgeOrganize.md`:
- Around line 81-105: The fenced workflow block (the triple-backtick code fence
that wraps the ASCII diagram under the workflow header) lacks a language tag,
causing MD040 lint failures; update the opening fence from ``` to ```text so the
block is explicitly annotated (e.g., change "```" to "```text") and leave the
closing fence as is to resolve the lint issue while preserving the diagram
content.
🪄 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: 32b25765-591a-4bca-88cb-b32d3434d45e

📥 Commits

Reviewing files that changed from the base of the PR and between fbe4cc6 and 08936db.

📒 Files selected for processing (2)
  • skills/lark-drive/SKILL.md
  • skills/lark-drive/references/lark-drive-workflow-knowledgeOrganize.md

Comment on lines +81 to +105
```
{用户输入的整理目标:URL / 名称 / token / 范围描述}
入口解析(识别资源类型并分流到 Drive / Wiki / 单资源链路)
递归盘点资源 ──► 标准化 ResourceItem 清单
现状统计 ──► 轻量内容读取(仅低置信度资源)
问题识别 ──► 整理规则生成
目标目录树 ──► 移动 / 创建计划
用户确认
├─► 不执行:输出整理方案
└─► 执行:创建目录 / 移动资源 / 重新扫描验证
```
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

Add a language tag to the fenced workflow block.

This block is missing a fence language and triggers MD040; please annotate it (e.g., text) for lint compliance.

Suggested diff
-```
+```text
 {用户输入的整理目标:URL / 名称 / token / 范围描述}
         │
         ▼
   入口解析(识别资源类型并分流到 Drive / Wiki / 单资源链路)
@@
-```
+```
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)

[warning] 81-81: Fenced code blocks should have a language specified

(MD040, fenced-code-language)

🤖 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/references/lark-drive-workflow-knowledgeOrganize.md` around
lines 81 - 105, The fenced workflow block (the triple-backtick code fence that
wraps the ASCII diagram under the workflow header) lacks a language tag, causing
MD040 lint failures; update the opening fence from ``` to ```text so the block
is explicitly annotated (e.g., change "```" to "```text") and leave the closing
fence as is to resolve the lint issue while preserving the diagram content.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.75%. Comparing base (ac85c3e) to head (08936db).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1028      +/-   ##
==========================================
- Coverage   67.77%   67.75%   -0.03%     
==========================================
  Files         590      590              
  Lines       55291    55188     -103     
==========================================
- Hits        37475    37392      -83     
+ Misses      14696    14684      -12     
+ Partials     3120     3112       -8     

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

@github-actions
Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#feat/workflow-knowledgeOrganize -y -g

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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