Skip to content

docs: register knowledge organize workflow#1828

Merged
YH-1600 merged 1 commit into
mainfrom
docs/register-knowledge-organize-workflow
Jul 9, 2026
Merged

docs: register knowledge organize workflow#1828
YH-1600 merged 1 commit into
mainfrom
docs/register-knowledge-organize-workflow

Conversation

@YH-1600

@YH-1600 YH-1600 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Register the existing knowledge_organize workflow under the shared lark-drive workflow framework so Drive and Wiki routing go through the same registry-based entry path as permission_governance.

Changes

  • Route Drive knowledge-organization requests through lark-drive-workflow.md before entering knowledge_organize.
  • Route Wiki knowledge-organization requests through the same shared workflow registry.
  • Add knowledge_organize to the lark-drive Workflow Registry as an R2-R3 / S3 workflow.
  • Mark lark-drive-workflow-knowledge-organize.md as a registered workflow entry and add Protocol Step mapping to its state machine.

Test Plan

  • Manual documentation review confirms Drive and Wiki entries now point to lark-drive-workflow.md.
  • git diff --check

Related Issues

  • None

Summary by CodeRabbit

Summary

  • Documentation
    • Updated knowledge-organization guidance to lead users through a single, unified workflow entry point, then proceed to the knowledge-organization process.
    • Expanded the knowledge-organization workflow documentation with clearer structure, strengthened runtime guidance, and added an explicit protocol-step breakdown (including confirm/execute/verify and recovery/cleanup stages).
    • Refined the “default generation” and “requires separate confirmation” constraints for directory creation, moving resources, and access requests.

@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
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d3f047b0-a621-4ad4-81e7-194559618edb

📥 Commits

Reviewing files that changed from the base of the PR and between 6640fa7 and dcefe07.

📒 Files selected for processing (4)
  • skills/lark-drive/SKILL.md
  • skills/lark-drive/references/lark-drive-workflow-knowledge-organize.md
  • skills/lark-drive/references/lark-drive-workflow.md
  • skills/lark-wiki/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/lark-wiki/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (2)
  • skills/lark-drive/references/lark-drive-workflow.md
  • skills/lark-drive/references/lark-drive-workflow-knowledge-organize.md

📝 Walkthrough

Walkthrough

Documentation changes register knowledge_organize in the shared workflow registry, expand its workflow metadata and state machine, and update SKILL.md routing in lark-drive and lark-wiki to use the registry-first entry path.

Changes

Workflow registry integration

Layer / File(s) Summary
Register knowledge_organize in shared workflow registry
skills/lark-drive/references/lark-drive-workflow.md
Adds a knowledge_organize registry row and updates the Risk/Structure Gate wording, removing the prior exclusion language.
Workflow metadata and state machine detail
skills/lark-drive/references/lark-drive-workflow-knowledge-organize.md
Adds explicit workflow metadata, updates the Runtime State introduction, and adds a Protocol Step column to the execution state machine.
Update SKILL.md entry points
skills/lark-drive/SKILL.md, skills/lark-wiki/SKILL.md
Changes the decision-routing text so users read lark-drive-workflow.md first and then enter knowledge_organize via its Workflow Registry.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • larksuite/cli#1028: Updates the same knowledge_organize routing in skills/lark-drive/SKILL.md and related workflow reference docs.
  • larksuite/cli#1253: Also updates the knowledge_organize workflow documentation and its skills/lark-drive/SKILL.md entry point.
  • larksuite/cli#1292: Uses the same registry-first workflow loading pattern in skills/lark-drive/SKILL.md.

Suggested labels: documentation

Suggested reviewers: fangshuyu-768

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the main change: registering the knowledge organize workflow.
Description check ✅ Passed The description matches the required template and includes summary, changes, test plan, and related issues.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/register-knowledge-organize-workflow

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.

@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.44%. Comparing base (74d8458) to head (dcefe07).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1828   +/-   ##
=======================================
  Coverage   74.44%   74.44%           
=======================================
  Files         860      860           
  Lines       89859    89859           
=======================================
  Hits        66897    66897           
  Misses      17778    17778           
  Partials     5184     5184           

☔ 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@dcefe074dc04a4da210f544a1f830f156f2161da

🧩 Skill update

npx skills add larksuite/cli#docs/register-knowledge-organize-workflow -y -g

@YH-1600
YH-1600 force-pushed the docs/register-knowledge-organize-workflow branch from 6640fa7 to dcefe07 Compare July 9, 2026 08:20
@jiangguozhou
jiangguozhou self-requested a review July 9, 2026 10:13
@YH-1600
YH-1600 merged commit 519a600 into main Jul 9, 2026
37 checks passed
@YH-1600
YH-1600 deleted the docs/register-knowledge-organize-workflow branch July 9, 2026 10:15
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 9, 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