feat(sheets): combine chart workflows and special chart types

[zhengzhijiej-tech]

Problem

Chart workflow improvements and bubble/waterfall/pareto support were developed on separate branches. Keeping them separate leaves the semantic shortcuts, batch paths, generated schemas, and bundled Sheets skill out of sync.

Changes

• add semantic chart create/config/data-update shortcuts and dedicated chart batch entry points
• preserve partial chart updates, detached headers, cross-sheet ranges, color themes, and batch error recovery
• support bubble, waterfall, and pareto charts across examples, schemas, readback, and generated flags
• add semantic bubble role flags for key/x/y/group/size on create and data update
• sync the bundled lark-sheets skill from the combined sheet-skill-spec output, including the chart layout check script
• keep generic +batch-update compatible with chart operations while guiding pure chart batches to the dedicated shortcuts

Validation

• go test -count=1 ./shortcuts/sheets/...
• make unit-test
• make vet
• make fmt-check
• node scripts/skill-format-check/index.js skills
• make quality-gate QUALITY_GATE_CHANGED_FROM=origin/main

[coderabbitai]

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 Plus

Run ID: 0275c3f4-c6c2-4ebf-97b4-ba1db781a0f8

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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

❌ Patch coverage is 64.08840% with 520 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.28%. Comparing base (b6d0473) to head (8affba5).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
shortcuts/sheets/lark_sheet_chart.go	63.04%	255 Missing and 129 partials ⚠️
shortcuts/sheets/lark_sheet_batch_update.go	63.87%	88 Missing and 37 partials ⚠️
shortcuts/sheets/flag_schema.go	28.57%	4 Missing and 1 partial ⚠️
shortcuts/sheets/batch_op_dispatch.go	91.11%	4 Missing ⚠️
shortcuts/sheets/flag_view.go	66.66%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2374      +/-   ##
==========================================
- Coverage   76.43%   76.28%   -0.16%     
==========================================
  Files        1046     1047       +1     
  Lines      115162   116570    +1408     
==========================================
+ Hits        88022    88921     +899     
- Misses      20377    20717     +340     
- Partials     6763     6932     +169     

☔ 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]

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

• deadcode — failure — details
• lint — failure — details
• results — failure — details