refactor(sheets): sync spec — chart position/offset/size + sparkline split#962
Conversation
…split
同步 sheet-skill-spec 仓的 refactor 改动(对应 spec MR ee/sheet-skill-spec!3)。
## 本次范围(仅 reference 文档)
- skills/lark-sheets/references/lark-sheets-chart.md
+chart-create / +chart-update 拎 6 个 scalar:
--position-col / --position-row / --offset-col / --offset-row /
--size-width / --size-height;`--properties` 收窄为只装 snapshot
- skills/lark-sheets/references/lark-sheets-sparkline.md
+sparkline-create / +sparkline-update 把 --properties 拆成 --config
+ --sparklines;描述对齐 mcp-tools 新 schema(`{config, sparklines[]}`)
## ⚠️ Go 实现层尚未跟进
reference 文档已先行同步,但 shortcuts/sheets/lark_sheet_object_crud.go
里的 chartSpec / sparklineSpec 还没拆 flag——后续会单独提一个 PR 补 Go 代码:
- chart: 加 createExtraFlags + enhanceCreateInput(模仿 pivotSpec 写法)
- sparkline: --properties → --config + --sparklines(factory 扩展或全 hoist)
在 Go PR 合并前,新的 flag(--position-col 等 / --config / --sparklines)
**实际不可用**;文档已先描绘"目标形态"用于模型 prompt 对齐。
## 同步方式
在 spec 仓跑 `npm run sync:consumers --only larksuite-cli`,按
`consumers/larksuite-cli.json` mirror 到 skills/lark-sheets/。
本次提交仅保留与 spec MR!3 直接相关的 2 个 reference;其他文件即便
sync 时被 mirror,diff 也 reverted 回 origin/feat/lark-sheets-refactor。
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
|
|
Caution Review failedFailed to post review comments 📝 WalkthroughWalkthroughAdds a new Sheets v2 tool-based shortcut suite (workbook, structure, read/write, range ops, search/replace, object list/CRUD, batch update), replaces legacy implementations, reorganizes registry, and updates docs. Extensive new unit/e2e tests validate dry-run/execute payloads, error propagation, and legacy flows. ChangesSheets shortcuts v2 and docs
Sequence Diagram(s)sequenceDiagram
autonumber
participant User
participant CLI
participant ToolGateway
participant OpenAPI
participant Sheets
User->>CLI: sheets +cells-set ... --dry-run/execute
CLI->>ToolGateway: buildToolBody(tool_name, input)
ToolGateway->>OpenAPI: POST invoke_write
OpenAPI->>Sheets: route to set_cell_range
Sheets-->>OpenAPI: envelope {code=0, data.output="<json>"}
OpenAPI-->>ToolGateway: envelope
ToolGateway-->>CLI: decode output / error
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
|
同步 sheet-skill-spec 仓的 refactor 改动(对应 spec MR ee/sheet-skill-spec!3)。
本次范围(仅 reference 文档)
skills/lark-sheets/references/lark-sheets-chart.md
+chart-create / +chart-update 拎 6 个 scalar:
--position-col / --position-row / --offset-col / --offset-row /
--size-width / --size-height;
--properties收窄为只装 snapshotskills/lark-sheets/references/lark-sheets-sparkline.md
+sparkline-create / +sparkline-update 把 --properties 拆成 --config
{config, sparklines[]})reference 文档已先行同步,但 shortcuts/sheets/lark_sheet_object_crud.go
里的 chartSpec / sparklineSpec 还没拆 flag——后续会单独提一个 PR 补 Go 代码:
在 Go PR 合并前,新的 flag(--position-col 等 / --config / --sparklines)
实际不可用;文档已先描绘"目标形态"用于模型 prompt 对齐。
同步方式
在 spec 仓跑
npm run sync:consumers --only larksuite-cli,按consumers/larksuite-cli.jsonmirror 到 skills/lark-sheets/。本次提交仅保留与 spec MR!3 直接相关的 2 个 reference;其他文件即便
sync 时被 mirror,diff 也 reverted 回 origin/feat/lark-sheets-refactor。
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com
Summary by CodeRabbit
New Features
Documentation