Skip to content

feat(drive): enhance lark-drive skill with permission auth, title rename guidance, and batch_query template#1618

Open
yballul-bytedance wants to merge 1 commit into
mainfrom
auto-research-sync/01KVZXJCG8HDY3V3WPW7BRQF29/mr-659-75bcbe6b
Open

feat(drive): enhance lark-drive skill with permission auth, title rename guidance, and batch_query template#1618
yballul-bytedance wants to merge 1 commit into
mainfrom
auto-research-sync/01KVZXJCG8HDY3V3WPW7BRQF29/mr-659-75bcbe6b

Conversation

@yballul-bytedance

@yballul-bytedance yballul-bytedance commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Enhances the lark-drive skill description and quick-decision guidance to make authorization checks and title rename operations more explicit and actionable for agents.

Changes

  • Updated description frontmatter to clarify that title rename must use Drive files patch (not lark-doc body edit) and permission checks use Drive permission.members auth
  • Added a quick-decision table under ## 快速决策 covering four high-frequency Drive tasks: title rename/rename, permission check, export-download with file_token, and folder mirror to local
  • Refined ## 修改标题 section to explicitly state title is Drive metadata, not lark-doc body
  • Added batch_query template in the API Resources section for bulk metadata + URL fetch, with doc_type reference values
  • Updated the 重要 note to allow skipping schema lookup when a ready-made template is already provided in the section

Test Plan

  • Verify skills/lark-drive/SKILL.md diff matches the upstream Codebase MR net change
  • git diff --check passes (no whitespace errors)

Related Issues

Auto research task: 01KVZXJCG8HDY3V3WPW7BRQF29

Summary by CodeRabbit

  • Documentation
    • 更新了飞书云空间技能说明,补充标题修改、权限判断、导出下载和文件夹拉取的操作指引。
    • 新增快速决策表,帮助更快匹配常见任务到对应命令。
    • 增加元数据批量查询示例,并强化原生命令使用前的准备要求。
  • Bug Fixes
    • 明确部分对象的标题/重命名必须按指定方式处理,避免误用其他更新路径。
    • 补充导出刷新与目录冲突处理建议,提升失败场景下的操作一致性。

…itle rename guidance, and batch_query template

- Update description to clarify title rename via Drive files patch and permission check via permission.members auth
- Add quick-decision table for high-frequency Drive tasks (title rename, permission check, export download, folder pull)
- Clarify 修改标题 section: title is Drive metadata, not lark-doc body edit
- Add batch_query template in API Resources section for bulk metadata fetch with URL
- Refine 重要 note to allow skipping schema lookup when template is already provided
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Updates the Lark Drive skill guide with tighter title-edit and permission boundaries, a quick-decision mapping for common drive commands, and added native API examples plus schema guidance.

Changes

Lark Drive skill guide updates

Layer / File(s) Summary
Boundary guidance and quick decision table
skills/lark-drive/SKILL.md
Updates the opening description and adds a quick-decision table mapping common Drive intents to specific CLI commands, including title changes, permission checks, export refresh/download, and folder pull conflict handling.
Title change rule
skills/lark-drive/SKILL.md
Clarifies that supported title edits use drive files patch with new_title and do not use lark-doc body updates.
Native API examples
skills/lark-drive/SKILL.md
Adds a drive metas batch_query example and expands the note that schema should be run before using native APIs.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • larksuite/cli#444: Also updates the Drive files patch / new_title guidance for title changes in skills/lark-drive/SKILL.md.
  • larksuite/cli#1028: Also changes the 快速决策 routing table in skills/lark-drive/SKILL.md.
  • larksuite/cli#863: Shares the Drive permission guidance area, including auth-related command usage in skills/lark-drive/SKILL.md.

Suggested labels

size/M, domain/ccm, documentation

Suggested reviewers

  • fangshuyu-768
  • caojie0621

Poem

A bunny hopped through Drive docs bright,
With new_title tucked in neatly right.
Batch queries bloom, commands align,
One small hop, and paths are fine 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the main drive skill updates around permission auth, rename guidance, and batch_query.
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 PR includes the required summary, changes, test plan, and related issues sections, with only a minor heading-name mismatch from the template.
✨ 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 auto-research-sync/01KVZXJCG8HDY3V3WPW7BRQF29/mr-659-75bcbe6b

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 added the size/M Single-domain feat or fix with limited business impact label Jun 26, 2026
@yballul-bytedance yballul-bytedance changed the title feat(authorization): enhance lark-drive skill with permission auth, title rename guidance, and batch_query template feat(drive): enhance lark-drive skill with permission auth, title rename guidance, and batch_query template Jun 26, 2026
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.64%. Comparing base (39d60cb) to head (407d6e9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1618   +/-   ##
=======================================
  Coverage   74.64%   74.64%           
=======================================
  Files         806      806           
  Lines       81386    81386           
=======================================
  Hits        60752    60752           
  Misses      16101    16101           
  Partials     4533     4533           

☔ 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

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#auto-research-sync/01KVZXJCG8HDY3V3WPW7BRQF29/mr-659-75bcbe6b -y -g

@github-actions

Copy link
Copy Markdown

PR Quality Summary

CI did not complete successfully. Use the failed check links below to decide whether this PR needs a code change or a rerun.

Failed checks

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

Labels

size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant