Skip to content

feat(sheets): add semantic chart shortcuts#1996

Draft
zhengzhijiej-tech wants to merge 1 commit into
feat/lark-sheets-developfrom
feat/chart-partial-update
Draft

feat(sheets): add semantic chart shortcuts#1996
zhengzhijiej-tech wants to merge 1 commit into
feat/lark-sheets-developfrom
feat/chart-partial-update

Conversation

@zhengzhijiej-tech

Copy link
Copy Markdown
Collaborator

Problem

Chart creation and common configuration updates currently require callers to construct and resend a complete chart snapshot. This increases payload size and makes simple chart tasks prone to schema and retry failures.

Scope

  • Add +chart-create-basic for column, bar, line, area, pie, scatter, combo, and radar charts.
  • Add +chart-config-update for titles, axes, legend, labels, stacking, and smoothing.
  • Regenerate sheet flag definitions and update the chart skill reference.
  • Keep per-series and marker styling on the existing raw snapshot command.

Mechanism

The CLI sends semantic chart type, range, and changed configuration fields only. The sheet service owns canonical snapshot expansion, merges updates with the current snapshot, validates the resulting structure, and writes it back. Existing +chart-create and +chart-update remain available for advanced snapshot-level control.

Related backend and specification changes:

  • byted-sheet MR !3853
  • sheet-skill-spec MR !57

Verification

  • go test ./shortcuts/sheets/... ./shortcuts/common/...
  • go build ./...
  • Dry-run coverage for all eight chart types, explicit boolean false, placement and size, partial configuration updates, and invalid enum/range rejection.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e4371385-4b6b-4915-b0d3-00e8702857cf

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/chart-partial-update

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 domain/ccm PR touches the ccm domain size/L Large or sensitive change across domains or core paths labels Jul 22, 2026
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/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant