Skip to content

feat(drive): surface secure label management in lark-drive skill#1913

Merged
yballul-bytedance merged 1 commit into
larksuite:mainfrom
yballul-bytedance:auto-research-sync/01KXJW71G2N1F2MJ8B9GCBH28Z/mr-1113-263cd1f2
Jul 16, 2026
Merged

feat(drive): surface secure label management in lark-drive skill#1913
yballul-bytedance merged 1 commit into
larksuite:mainfrom
yballul-bytedance:auto-research-sync/01KXJW71G2N1F2MJ8B9GCBH28Z/mr-1113-263cd1f2

Conversation

@yballul-bytedance

@yballul-bytedance yballul-bytedance commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Surface Feishu document secure label management in the lark-drive skill so agents can discover the capability from SKILL.md.

Changes

  • Mention secure labels in the skill description alongside metadata/version/import capabilities.
  • Add a decision-rule entry routing "set/modify secure label" and "list available secure labels" requests to references/lark-drive-secure-label.md.

Related

  • Auto research task: 01KXJW71G2N1F2MJ8B9GCBH28Z (Drive 命令优化)

Summary by CodeRabbit

  • Documentation
    • Expanded the Lark Drive skill description to clarify its scope and highlight secure-label capabilities.
    • Added instructions to read the secure-label reference to set/modify and review currently available secure labels for Drive documents.

@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 16, 2026
@coderabbitai

coderabbitai Bot commented Jul 16, 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: 9d691e1a-ef14-4f14-ba51-4f291f05371f

📥 Commits

Reviewing files that changed from the base of the PR and between 111e7d3 and 95ae1a3.

📒 Files selected for processing (1)
  • skills/lark-drive/SKILL.md

📝 Walkthrough

Walkthrough

The Lark Drive skill description now includes secure-label scope and clarified responsibilities. Quick-decision guidance directs users to the secure-label reference for modifying document labels or querying available labels.

Changes

Lark Drive guidance

Layer / File(s) Summary
Secure label metadata and routing
skills/lark-drive/SKILL.md
The frontmatter description mentions secure labels and clarified scope, while quick-decision guidance routes secure-label operations to references/lark-drive-secure-label.md.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • larksuite/cli#985: Introduces the secure-label shortcuts and reference documentation referenced by this update.
  • larksuite/cli#1284: Updates related Lark Drive skill scope and quick-decision routing guidance.

Suggested reviewers: fangshuyu-768, wittam-01

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers summary and changes, but it omits the required Test Plan and Related Issues sections. Add the Test Plan section and a Related Issues section to match the template, even if the test plan is brief or notes no tests were run.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the secure label management discoverability change.
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

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.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add yballul-bytedance/cli#auto-research-sync/01KXJW71G2N1F2MJ8B9GCBH28Z/mr-1113-263cd1f2 -y -g

@yballul-bytedance
yballul-bytedance force-pushed the auto-research-sync/01KXJW71G2N1F2MJ8B9GCBH28Z/mr-1113-263cd1f2 branch from 06b73cc to 111e7d3 Compare July 16, 2026 09:22

@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 27-28: Update the first routing rule in the skill instructions to
remove “密级标签” from its permission_governance trigger, so secure-label setting,
modification, and querying consistently route through
lark-drive-secure-label.md. Keep the existing routing for other permission and
ownership governance operations unchanged.
🪄 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: 52825c24-4940-431e-bdcf-42210d78c557

📥 Commits

Reviewing files that changed from the base of the PR and between 06b73cc and 111e7d3.

📒 Files selected for processing (1)
  • skills/lark-drive/SKILL.md

Comment thread skills/lark-drive/SKILL.md
@yballul-bytedance
yballul-bytedance force-pushed the auto-research-sync/01KXJW71G2N1F2MJ8B9GCBH28Z/mr-1113-263cd1f2 branch from 111e7d3 to 95ae1a3 Compare July 16, 2026 09:49
@yballul-bytedance
yballul-bytedance merged commit 6558657 into larksuite:main Jul 16, 2026
37 checks passed
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 16, 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