feat(slides): add slides chart demo reference

[ethan-zhx]

Summary

Add a lark-slides chart demo reference and point the XML format guide to it so agents have a concrete chart XML example to follow.

Changes

• Add slides_chart_demo.xml covering native chart examples and SVG-backed whiteboard chart examples.
• Update xml-format-guide.md with a link to the full demo and a compact column chart snippet.

Test Plan

• Unit tests pass
• Manual local verification confirms the lark-cli <domain> <command> flow works as expected

Not run. This is a documentation/reference-only change, and no build or test command was requested for this turn.

Related Issues

• None

Summary by CodeRabbit

• Documentation
  • Added a comprehensive chart demo reference (covering common chart types and advanced variants like combo, radar, and special “others” visualizations) to help compare chart styles in one place.
  • Expanded the XML format guide with a full native combination-chart example.
  • Refined whiteboard guidance to better distinguish when to use native charts versus diagrams, and tightened SVG-specific quality and coordinate advice.
  • Updated schema and quick-reference materials to align with the revised whiteboard/SVG expectations.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR adds a new SML slide deck demonstrating multiple chart types and updates slide XML reference docs for chart examples and whiteboard/Mermaid rendering guidance.

Changes

Chart demo and rendering guidance

Layer / File(s)	Summary
Chart demo deck skills/lark-slides/references/slides_chart_demo.xml	Creates a themed presentation XML covering multiple chart families and variants, with native chart elements and whiteboard-based placeholders.
Chart XML example docs skills/lark-slides/references/xml-format-guide.md	Adds a reference to the demo deck and includes a full combo chart XML example, while removing the prior chart-DSL note.
Whiteboard routing and schema notes skills/lark-slides/references/lark-slides-whiteboard.md, skills/lark-slides/references/slides_xml_schema_definition.xml, skills/lark-slides/references/xml-schema-quick-ref.md	Refines chart-versus-whiteboard guidance, updates Mermaid applicability notes in schema docs, and adds an SVG-mode limitation note in the quick reference.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• larksuite/cli#389: Updates the same skills/lark-slides chart and whiteboard reference surface with adjacent documentation and example changes.
• larksuite/cli#1029: Introduces related <whiteboard> documentation and rendering-rule updates that this PR continues.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: adding a slides chart demo reference.
Description check	✅ Passed	The description follows the template and covers summary, changes, test plan, and related issues.
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/slides_chart_demo

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/lark-slides/references/slides_chart_demo.xml`:
- Line 1: The whiteboard demo slots are empty self-closing placeholders, so the
scatter, bubble, funnel, pareto, waterfall, and combo examples render as blank
cards. Update the `<whiteboard>` instances in slides_chart_demo.xml to include
the actual SVG/Mermaid payloads for those charts, using the existing whiteboard
elements as the locators. For the combo example, either populate it with a real
whiteboard payload or replace it with a native `<chart>` example, since combo is
already supported by the chart contract.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7c8517b1-c16a-4601-a4ef-2b86c5b67dbb

📥 Commits

Reviewing files that changed from the base of the PR and between 91d785f and 1b7f01f.

📒 Files selected for processing (2)

• skills/lark-slides/references/slides_chart_demo.xml
• skills/lark-slides/references/xml-format-guide.md

[coderabbitai]

🎯 Functional Correctness | 🟠 Major | 🏗️ Heavy lift

Populate the whiteboard examples instead of leaving them blank.

These slides currently advertise scatter, bubble, funnel, pareto, waterfall, and combo coverage, but every <whiteboard> here is self-closing. That means the demo renders empty placeholders, and the combo slide is especially misleading because combo is already a supported native <chart> type in this repo’s chart contract. Please embed the actual SVG/Mermaid payloads or switch combo back to a native chart example.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/lark-slides/references/slides_chart_demo.xml` at line 1, The
whiteboard demo slots are empty self-closing placeholders, so the scatter,
bubble, funnel, pareto, waterfall, and combo examples render as blank cards.
Update the `<whiteboard>` instances in slides_chart_demo.xml to include the
actual SVG/Mermaid payloads for those charts, using the existing whiteboard
elements as the locators. For the combo example, either populate it with a real
whiteboard payload or replace it with a native `<chart>` example, since combo is
already supported by the chart contract.

[github-actions]

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#feat/slides_chart_demo -y -g

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.45%. Comparing base (869a259) to head (1b7f01f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1769      +/-   ##
==========================================
+ Coverage   74.43%   74.45%   +0.02%     
==========================================
  Files         854      855       +1     
  Lines       88490    88564      +74     
==========================================
+ Hits        65867    65944      +77     
+ Misses      17554    17548       -6     
- Partials     5069     5072       +3     

☔ 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

• coverage — failure — details
• unit-test — failure — details
• results — failure — details

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

skills/lark-slides/references/slides_chart_demo.xml (1)

2-2: 🎯 Functional Correctness | 🔴 Critical | ⚡ Quick win

Remove the stray trailing 2.
A literal 2 after </presentation> makes the XML malformed and will break parsing/loading.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/lark-slides/references/slides_chart_demo.xml` at line 2, Remove the
stray trailing literal after the closing presentation tag in the
slides_chart_demo XML so the document remains well-formed and parsable. Update
the presentation markup to end cleanly at the closing tag, and verify there are
no extra characters appended after the root element.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/lark-slides/references/lark-slides-whiteboard.md`:
- Around line 64-71: The model-path table in the whiteboard reference is too
broad: scatter and funnel visuals must stay on the SVG route and should not be
redirected to Mermaid for any model. Update the guidance around the third step
and the model mapping so that scatter/funnel remain explicitly included in the
SVG path, while only truly Mermaid-friendly decorative/diagram cases use Mermaid
for lower-capability models. Keep the fix anchored in the
“第三步：非原生图表与装饰元素按模型身份选路径” section and the model identity table.

---

Outside diff comments:
In `@skills/lark-slides/references/slides_chart_demo.xml`:
- Line 2: Remove the stray trailing literal after the closing presentation tag
in the slides_chart_demo XML so the document remains well-formed and parsable.
Update the presentation markup to end cleanly at the closing tag, and verify
there are no extra characters appended after the root element.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 93d6ba1c-98c1-4e82-9fc9-0d39ed3ac2eb

📥 Commits

Reviewing files that changed from the base of the PR and between 1b7f01f and c5eb6e7.

📒 Files selected for processing (5)

• skills/lark-slides/references/lark-slides-whiteboard.md
• skills/lark-slides/references/slides_chart_demo.xml
• skills/lark-slides/references/slides_xml_schema_definition.xml
• skills/lark-slides/references/xml-format-guide.md
• skills/lark-slides/references/xml-schema-quick-ref.md

✅ Files skipped from review due to trivial changes (3)

• skills/lark-slides/references/xml-schema-quick-ref.md
• skills/lark-slides/references/slides_xml_schema_definition.xml
• skills/lark-slides/references/xml-format-guide.md

[coderabbitai]

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Keep scatter/funnel on the SVG path.
The table now sends unsupported data visuals to Mermaid for some models, but the repo contract explicitly maps scatter/funnel to <whiteboard> SVG. That mismatch can produce the wrong renderer choice.

Suggested adjustment

-| 上表以外的场景（散点图、漏斗图、进度条、时间线、波浪背景、星点纹理等）需要精确控制坐标和配色，SVG 表达力更强，但各模型生成 SVG 的能力有差异：
+| 上表以外的场景（散点图、漏斗图、进度条、时间线、波浪背景、星点纹理等）统一走 `<whiteboard>` SVG；如果需要降级，请简化图形本身，而不是切换到 Mermaid：
@@
-| Doubao / Seed / Other | **Mermaid** — 用 `gantt`、`flowchart` 等近似表达；确实无法用 Mermaid 表达时才回退到简单 SVG 矩形/线条
+| Doubao / Seed / Other | **SVG** — 用更简单的坐标/标注表达降级；不要把散点/漏斗这类数据视觉切到 Mermaid

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	### 第三步：非原生图表与装饰元素按模型身份选路径
	上表以外的场景（散点图、漏斗图、进度条、时间线、波浪背景、星点纹理等）需要精确控制坐标和配色，SVG 表达力更强，但各模型生成 SVG 的能力有差异：
	| 模型身份 | 路径 |
	|----------|------|
	| Claude / Gemini / GPT / GLM | **SVG** — 精确控制坐标、颜色、透明度 |
	| Doubao / Seed / Other | **Mermaid** — 用 `pie`、`gantt` 等近似表达；确实无法用 Mermaid 表达时才回退到简单 SVG 矩形/线条 |
	| Doubao / Seed / Other | **Mermaid** — 用 `gantt`、`flowchart` 等近似表达；确实无法用 Mermaid 表达时才回退到简单 SVG 矩形/线条 |
	### 第三步：非原生图表与装饰元素按模型身份选路径
	上表以外的场景（散点图、漏斗图、进度条、时间线、波浪背景、星点纹理等）统一走 `<whiteboard>` SVG；如果需要降级，请简化图形本身，而不是切换到 Mermaid：
	| 模型身份 | 路径 |
	|----------|------|
	| Claude / Gemini / GPT / GLM | **SVG** — 精确控制坐标、颜色、透明度 |
	| Doubao / Seed / Other | **SVG** — 用更简单的坐标/标注表达降级；不要把散点/漏斗这类数据视觉切到 Mermaid |

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/lark-slides/references/lark-slides-whiteboard.md` around lines 64 -
71, The model-path table in the whiteboard reference is too broad: scatter and
funnel visuals must stay on the SVG route and should not be redirected to
Mermaid for any model. Update the guidance around the third step and the model
mapping so that scatter/funnel remain explicitly included in the SVG path, while
only truly Mermaid-friendly decorative/diagram cases use Mermaid for
lower-capability models. Keep the fix anchored in the “第三步：非原生图表与装饰元素按模型身份选路径”
section and the model identity table.