docs: clarify bitable import routing to drive +import#368
Conversation
Change-Id: I12f86a343d79b8fb480084305ed34b54aa92fb94
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR clarifies routing rules for importing local files into Base/multi-dimensional tables (bitable) across multiple skill and domain documentation files. The core change enforces that local-file imports must initiate with Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Greptile SummaryThis docs-only PR adds explicit routing guidance so agent models know to use Confidence Score: 5/5Safe to merge — docs-only change with consistent, well-placed routing guidance across all five files. All remaining findings are P2 or lower (one suggestion to also update the lark-base description metadata). The routing guidance is correctly stated and consistent across every changed file, and no code or schema changes are involved. skills/lark-base/SKILL.md — consider updating the description metadata field to also exclude local-import routing at the skill-selection layer.
|
| Filename | Overview |
|---|---|
| skills/lark-drive/SKILL.md | Updated description metadata to claim bitable imports; added a new "快速决策" section that lists import routing rules before any other content. |
| skills/lark-base/SKILL.md | Added routing redirect at the top-level preconditions block, first item of quick-execution order, "禁止行为" list, command index, and updated examples.md link text to drop "导入"; description metadata not changed. |
| skills/lark-drive/references/lark-drive-import.md | Added [!IMPORTANT] callout at the top and below the file-type table; added concrete bitable import example; clarified that "Base/多维表格/bitable" all map to --type bitable. |
| skill-template/domains/base.md | Added routing rule #9 to core rules, a new row in the intent→command index, and a header-level routing callout; removed "导入" from examples link. |
| skill-template/domains/drive.md | Added a routing-note blockquote at the top of "核心概念" directing bitable imports to drive +import. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User request: import local file to Base/bitable] --> B{File type?}
B -->|.xlsx / .csv| C[lark-cli drive +import --type bitable]
B -->|.xlsx / .xls / .csv| D[lark-cli drive +import --type sheet]
B -->|.md / .docx / .doc / .txt / .html| E[lark-cli drive +import --type docx]
C --> F[Import complete → get token/url]
F --> G[lark-cli base +... for table operations]
D --> H[Sheet ready]
E --> I[Doc ready]
style C fill:#d4edda,stroke:#28a745
style G fill:#cce5ff,stroke:#004085
style B fill:#fff3cd,stroke:#856404
Comments Outside Diff (1)
-
skills/lark-base/SKILL.md, line 4 (link)Description metadata doesn't exclude local-import routing
The
descriptionfield is typically what a skill-router reads to decide which skill to activate before opening the SKILL.md body. Because it still lists "建表" without noting that local-file → bitable imports belong tolark-drive, an agent that scores skills by description alone could still picklark-basefor an import request and only encounter the redirect once it reads the body.Adding a short exclusion to the description (e.g. "不含本地文件导入到 Base — 该场景请使用 lark-drive") would close this gap at the selection layer, complementing the body-level routing guidance already added.
Reviews (1): Last reviewed commit: "route base import guidance to drive +imp..." | Re-trigger Greptile
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@d1926f119f8b0b8be6caa727fd1611616a52db77🧩 Skill updatenpx skills add larksuite/cli#docs/route-bitable-import-to-drive-import -y -g |
Summary
Clarify the agent guidance for Base import scenarios so local file imports to Base/bitable are routed to
lark-cli drive +import --type bitableinstead oflark-base. This reduces a recurring misread where agents jump into Base table operationsbefore completing the Drive import step.
Changes
lark-driveskill anddrive +importdocs to explicitly state that local Excel/CSV -> Base/bitable imports belong todrive +import --type bitablelark-basedocs and templates so Base skills redirect local file import requests back todrive +importTest Plan
lark xxxcommand works as expectedlark-drive,lark-base, and the corresponding skill templatesRelated Issues
Summary by CodeRabbit