Skip to content

chore(slides): update lark-slides skill to 0715 snapshot#1933

Merged
R0bynZhu merged 7 commits into
mainfrom
update-lark-slides-skill-0715
Jul 20, 2026
Merged

chore(slides): update lark-slides skill to 0715 snapshot#1933
R0bynZhu merged 7 commits into
mainfrom
update-lark-slides-skill-0715

Conversation

@R0bynZhu

@R0bynZhu R0bynZhu commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

chore(slides): update lark-slides skill to 0715 release bundle
Overwrite skills/lark-slides with the lark-slides-0715 release bundle.

Summary

Refresh the lark-slides skill to the 0715 release snapshot. This is a docs/skill-only change (no Go code touched): it rewrites SKILL.md and the reference/script set to the latest authoring guidance, drops materials that were retired in 0715, and adds the per-page create reference.

Changes

  • Refresh SKILL.md, references/* to the lark-slides-0715 snapshot (tighter visual/layout, XML construction, asset-fallback, chart/icon, and lint/validation guidance).
  • Add references/lark-slides-xml-presentation-slide-create.md (per-page create) and update the full-presentation XML retrieval docs, shortcuts, and error guidance.
  • Remove retired materials: examples.md, slide-templates.md, slides_demo.xml, xml-format-guide.md, visual-styles.md, lark-slides-whiteboard.md, lark-slides-xml-get.md.
  • Fix cross-links to renamed/removed docs (lark-slides-xml-get.mdlark-slides-xml-presentations-get.md, add -slide-create).
  • Preserve the <table> width/height schema attributes already on main (not reverted by the bundle).

Related Issues

  • None

Summary by CodeRabbit

  • Documentation

    • Strengthened slide creation and planning guidance with stricter visual/layout, image/cover, typography, XML construction, asset-fallback, and read-back/validation workflows.
    • Improved troubleshooting and verification (including overlap linting and screenshot QA) plus clearer template-editing and icon rules.
    • Added/expanded references for creating individual slides and updated full-presentation XML retrieval shortcuts.
  • Changes

    • Removed whiteboard element support and related docs.
    • Refreshed the native chart demo deck and removed outdated templates and end-to-end examples.

@github-actions github-actions Bot added the size/XL Architecture-level or global-impact change label Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The PR updates Lark Slides generation guidance, asset fallback rules, XML schema references, command documentation, validation workflows, and chart examples. It removes whiteboard XML support and adds stricter planning, readback, linting, and delivery requirements.

Changes

Lark Slides workflow and visual rules

Layer / File(s) Summary
Planning and visual-generation rules
skills/lark-slides/SKILL.md, skills/lark-slides/references/{asset-planning,planning-layer,visual-planning,iconpark,lark-slides-pptx-template-workflows}.md
Generation guidance now requires full-document reads, plan-driven XML generation, explicit asset fallbacks, stricter layout rules, and local XML validation.
XML schema and element contracts
skills/lark-slides/references/slides_xml_schema_definition.xml, skills/lark-slides/references/xml-schema-quick-ref.md
XML references clarify content, shape, image, icon, table, chart, and slide constraints while removing whiteboard from the schema.
Native chart demonstration deck
skills/lark-slides/references/slides_chart_demo.xml
The demo deck now contains native column, line, pie, bar, area, combo, and radar chart examples with embedded configurations.
Slide command documentation
skills/lark-slides/references/lark-slides-*.md
Command references add xml_presentation.slide create, revise XML retrieval shortcuts and responses, update related links, and document screenshot fallback behavior.
Validation and troubleshooting pipeline
skills/lark-slides/references/{troubleshooting,validation-checklist}.md
Troubleshooting adds XML preflight and recovery guidance; validation now requires saved XML, overlap linting with zero errors, screenshot QA, and layout-risk review.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Planner
  participant XMLGenerator
  participant SlideAPI
  participant XMLReadback
  participant Validator
  Planner->>XMLGenerator: provide slide_plan.json and fallback_if_missing
  XMLGenerator->>SlideAPI: submit generated slide XML
  SlideAPI-->>XMLReadback: expose presentation XML
  XMLReadback->>Validator: save XML and run overlap/static checks
Loading

Possibly related PRs

Suggested labels: domain/ccm

Suggested reviewers: fangshuyu-768

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description matches the template well, but it omits the required Test Plan section. Add a Test Plan section with the verification steps used for this docs-only update, even if it only notes manual validation.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: updating the lark-slides skill to the 0715 snapshot.
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 update-lark-slides-skill-0715

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.

@CLAassistant

CLAassistant commented Jul 17, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.98%. Comparing base (7b98994) to head (3d49950).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1933   +/-   ##
=======================================
  Coverage   74.98%   74.98%           
=======================================
  Files         895      896    +1     
  Lines       94379    94403   +24     
=======================================
+ Hits        70767    70789   +22     
- Misses      18197    18198    +1     
- Partials     5415     5416    +1     

☔ 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

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#update-lark-slides-skill-0715 -y -g

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 11

🤖 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/asset-planning.md`:
- Around line 133-137: Restrict synthetic fallbacks to illustrative assets and
prevent them from representing identity or evidence. In
skills/lark-slides/references/asset-planning.md:133-137, branch fallback
behavior by asset class, using labeled schematic placeholders for logos,
screenshots, and paper figures while retaining appropriate generated
approximations for illustrations. In
skills/lark-slides/references/planning-layer.md:91-91, replace the
generated-logo fallback with a neutral text badge; at :214-214 document this
identity/evidence exception in the field contract; at :222-222 update the logo
example to a labeled placeholder. In
skills/lark-slides/references/visual-planning.md:109-109, limit generated
approximations to illustrative types, and at :232-232 use labeled schematic or
crop/zoom fallbacks for screenshots and paper figures.

In `@skills/lark-slides/references/iconpark.md`:
- Line 29: Update the icon fallback guidance in the high-frequency examples
section to prohibit random substitutions when no semantically appropriate icon
exists. Require using a relevant keyword or number, or omitting the icon region,
while preserving semantic accuracy for concepts such as success, failure, and
security.

In `@skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md`:
- Line 169: Standardize executable command references to the registered form
“xml_presentation.slide create”: update
skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md lines
169-169 and skills/lark-slides/references/troubleshooting.md lines 3-3, 12-12,
23-24, 37-37, and 63-63, expanding “slide.create” where it denotes a command
while preserving non-command terminology.

In `@skills/lark-slides/references/slides_chart_demo.xml`:
- Around line 221-229: Update the slide footer page-number text in each visible
footer shape, including the current “03 / 12” and the additional locations, to
number the seven slides sequentially as 01 / 07 through 07 / 07 without skipping
08 or retaining the stale / 12 denominator.
- Line 2: Update the presentation root element’s xmlns declaration in the
canonical chart demo to use the full namespace URI defined by
slides_xml_schema_definition.xml instead of the shorthand /sml/2.0 value, while
preserving the existing width and height attributes.

In `@skills/lark-slides/references/xml-schema-quick-ref.md`:
- Around line 153-158: Update the canonical XML examples in the shape/content
sections, including the title example and the examples around the additional
referenced range, so every content element includes the mandatory explicit
fontSize attribute. Add the documented wrapping and auto-fit attributes where
title or body text may span multiple lines, while preserving the examples’
existing structure and semantics.
- Around line 203-205: Update the image XML example to remove the unsupported
URL option from the src placeholder, using only the supported file-token format.
Keep the existing positioning and sizing attributes unchanged, and ensure the
example aligns with the supported image source format described near the
referenced schema guidance.

In `@skills/lark-slides/scripts/xml_text_overlap_lint.py`:
- Line 153: Update the text extraction used by the overlap height estimation
around strip_xml(content) so paragraph boundaries remain represented as line
breaks rather than being collapsed. Ensure multiple short <p> elements
contribute separate rendered lines when calculating the visual bounding box, and
add a regression test covering later paragraph lines overlapping another text
box.
- Around line 112-114: Update lint_xml’s invalid-root validation around
xml_local_name so it returns the normal JSON-shaped summary with a structured
issue instead of raising XmlTextOverlapLintError. Preserve the existing
validation message, and ensure CLI handling continues to expose actionable,
specific details through the stable summary contract.

In `@skills/lark-slides/SKILL.md`:
- Line 21: Update the image requirement in the skill instructions to require
meaningful visual elements rather than大量图片 in every deliverable. Allow asset
planning to select asset_type: "none" when images do not clarify the
key_message, and prohibit irrelevant, repetitive, or fabricated imagery while
preserving the requirement for semantically useful images.
- Line 218: Wrap the Chinese placeholder text near the element-placement section
in a valid XML comment so it is ignored when the template is copied and remains
directly executable. Preserve the existing guidance content while ensuring it is
no longer emitted as a raw child of the <data> 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: dea92811-aa22-4ca9-8cf0-7e963ba1d0a7

📥 Commits

Reviewing files that changed from the base of the PR and between 27df16d and 1bdfa17.

📒 Files selected for processing (28)
  • skills/lark-slides/SKILL.md
  • skills/lark-slides/references/asset-planning.md
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/iconpark.md
  • skills/lark-slides/references/lark-slides-create.md
  • skills/lark-slides/references/lark-slides-media-upload.md
  • skills/lark-slides/references/lark-slides-pptx-template-workflows.md
  • skills/lark-slides/references/lark-slides-replace-slide.md
  • skills/lark-slides/references/lark-slides-screenshot.md
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-delete.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md
  • skills/lark-slides/references/lark-slides-xml-presentations-get.md
  • skills/lark-slides/references/planning-layer.md
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/slides_chart_demo.xml
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/slides_xml_schema_definition.xml
  • skills/lark-slides/references/troubleshooting.md
  • skills/lark-slides/references/validation-checklist.md
  • skills/lark-slides/references/visual-planning.md
  • skills/lark-slides/references/xml-format-guide.md
  • skills/lark-slides/references/xml-schema-quick-ref.md
  • skills/lark-slides/scripts/xml_text_overlap_lint.py
  • skills/lark-slides/scripts/xml_text_overlap_lint_test.py
💤 Files with no reviewable changes (7)
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/xml-format-guide.md
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/lark-slides-replace-slide.md

Comment thread skills/lark-slides/references/asset-planning.md
Comment thread skills/lark-slides/references/iconpark.md
Comment thread skills/lark-slides/references/slides_chart_demo.xml
Comment thread skills/lark-slides/references/slides_chart_demo.xml
Comment thread skills/lark-slides/references/xml-schema-quick-ref.md
Comment thread skills/lark-slides/scripts/xml_text_overlap_lint.py
Comment thread skills/lark-slides/scripts/xml_text_overlap_lint.py Outdated
Comment thread skills/lark-slides/SKILL.md
Comment thread skills/lark-slides/SKILL.md
Comment thread skills/lark-slides/references/lark-slides-screenshot.md
Comment thread skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md Outdated
Comment thread skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md Outdated
Comment thread skills/lark-slides/references/lark-slides-xml-presentations-get.md
Comment thread skills/lark-slides/references/validation-checklist.md
@R0bynZhu
R0bynZhu force-pushed the update-lark-slides-skill-0715 branch from 1bdfa17 to 7e53492 Compare July 17, 2026 10:04

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

♻️ Duplicate comments (2)
skills/lark-slides/SKILL.md (2)

238-238: 🎯 Functional Correctness | 🔴 Critical

Keep the XML template schema-valid.

The raw placeholder text inside <data> is not a valid SML element and can cause the copied command payload to be rejected. Wrap it in an XML comment or replace it with a valid element placeholder. As per coding guidelines, command templates must remain directly usable and specific.

🤖 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/SKILL.md` at line 238, Update the XML template placeholder
for shape, line, table, and chart elements so the content inside <data> remains
schema-valid, using an XML comment or valid element placeholder while preserving
a directly usable, specific command template.

Source: Coding guidelines


21-21: 🎯 Functional Correctness | 🟠 Major

Keep visual fallbacks semantically optional and accurate.

Both rules force decorative assets when a meaningful asset may not exist, which can produce irrelevant imagery or misleading icons.

  • skills/lark-slides/SKILL.md#L21-L21: replace the mandatory image quota with a requirement for meaningful visual elements and allow no-image plans when appropriate.
  • skills/lark-slides/references/iconpark.md#L29-L29: replace random icon substitution with a relevant keyword/number or omission of the icon region.
🤖 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/SKILL.md` at line 21, Update the visual guidance in
skills/lark-slides/SKILL.md at line 21 to require meaningful visual elements
rather than a mandatory image quota, and explicitly allow plans with no images
when appropriate. Also update skills/lark-slides/references/iconpark.md at line
29 to replace random icon substitution with a relevant keyword or number, or
omit the icon region when no suitable icon exists.

Source: Coding guidelines

🤖 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/visual-planning.md`:
- Line 232: Update the visual-planning guidance for dense real assets to
explicitly prohibit regenerating factual evidence with image generation; require
fidelity-preserving alternatives such as cropping, zooming, splitting across
slides, or deterministic redrawing, while retaining the existing readability
guidance.

In `@skills/lark-slides/SKILL.md`:
- Line 25: Resolve the conflict between the global edge-strip prohibition on
line 25 and the optional card edge-strip rule on line 61. Update the relevant
guidance so narrow strips are consistently prohibited, or explicitly scope the
exception to a clearly defined card context while preserving the broader
page-edge restriction.

---

Duplicate comments:
In `@skills/lark-slides/SKILL.md`:
- Line 238: Update the XML template placeholder for shape, line, table, and
chart elements so the content inside <data> remains schema-valid, using an XML
comment or valid element placeholder while preserving a directly usable,
specific command template.
- Line 21: Update the visual guidance in skills/lark-slides/SKILL.md at line 21
to require meaningful visual elements rather than a mandatory image quota, and
explicitly allow plans with no images when appropriate. Also update
skills/lark-slides/references/iconpark.md at line 29 to replace random icon
substitution with a relevant keyword or number, or omit the icon region when no
suitable icon exists.
🪄 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: 7d6efba9-bd45-47ac-83c0-1a06db74de9e

📥 Commits

Reviewing files that changed from the base of the PR and between 1bdfa17 and 40ea282.

📒 Files selected for processing (26)
  • skills/lark-slides/SKILL.md
  • skills/lark-slides/references/asset-planning.md
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/iconpark.md
  • skills/lark-slides/references/lark-slides-create.md
  • skills/lark-slides/references/lark-slides-media-upload.md
  • skills/lark-slides/references/lark-slides-pptx-template-workflows.md
  • skills/lark-slides/references/lark-slides-replace-slide.md
  • skills/lark-slides/references/lark-slides-screenshot.md
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-delete.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md
  • skills/lark-slides/references/lark-slides-xml-presentations-get.md
  • skills/lark-slides/references/planning-layer.md
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/slides_chart_demo.xml
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/slides_xml_schema_definition.xml
  • skills/lark-slides/references/troubleshooting.md
  • skills/lark-slides/references/validation-checklist.md
  • skills/lark-slides/references/visual-planning.md
  • skills/lark-slides/references/xml-format-guide.md
  • skills/lark-slides/references/xml-schema-quick-ref.md
💤 Files with no reviewable changes (7)
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/lark-slides-replace-slide.md
  • skills/lark-slides/references/xml-format-guide.md
🚧 Files skipped from review as they are similar to previous changes (7)
  • skills/lark-slides/references/slides_xml_schema_definition.xml
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-delete.md
  • skills/lark-slides/references/troubleshooting.md
  • skills/lark-slides/references/xml-schema-quick-ref.md
  • skills/lark-slides/references/asset-planning.md
  • skills/lark-slides/references/slides_chart_demo.xml
  • skills/lark-slides/references/planning-layer.md

Comment thread skills/lark-slides/references/visual-planning.md
Comment thread skills/lark-slides/SKILL.md
Comment thread skills/lark-slides/SKILL.md Outdated
@R0bynZhu
R0bynZhu force-pushed the update-lark-slides-skill-0715 branch from a5fedd8 to 9881acf Compare July 20, 2026 06:59

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

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/SKILL.md (1)

331-331: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Standardize executable slide commands as xml_presentation.slide create.

  • skills/lark-slides/SKILL.md#L331-L331: replace xml_presentation.slide.create in the creation workflow.
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md#L169-L169: replace the dotted form when referring to the executable CLI command.

Verify the registered form against meta_data.json; retain dotted notation only for schema/API identifiers.

🤖 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/SKILL.md` at line 331, Standardize the executable command
references by replacing the dotted form with `xml_presentation.slide create` in
skills/lark-slides/SKILL.md lines 331-331 and
skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md lines
169-169. Verify the command spelling against meta_data.json, while retaining
dotted notation only when referring to schema or API identifiers.

Sources: Coding guidelines, Learnings

♻️ Duplicate comments (3)
skills/lark-slides/references/visual-planning.md (1)

232-232: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Do not regenerate factual evidence with image generation.

For screenshots, charts, paper figures, and product captures, use cropping, zooming, splitting across slides, or deterministic redrawing. Generative recreation can change labels or data and make the presentation factually misleading.

🤖 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/visual-planning.md` at line 232, Update the
visual-planning guidance to prohibit regenerating factual evidence with
image-generation tools. For screenshots, charts, paper figures, and product
captures, require cropping, zooming, splitting across slides, or deterministic
redrawing instead, while preserving the existing readability guidance.
skills/lark-slides/SKILL.md (2)

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

Make the XML placeholder a comment.

The raw Chinese text becomes an invalid child of <data> if the template is copied literally.

Proposed fix
-    在这里放置 shape、line、table、chart 等元素
+    <!-- 在这里放置 shape、line、table、chart 等元素 -->
🤖 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/SKILL.md` at line 240, Replace the raw Chinese placeholder
text under the <data> template section with a valid XML comment, preserving its
guidance about placing shape, line, table, and chart elements without leaving
text as a direct child of <data>.

Source: Coding guidelines


21-21: 🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Do not require images on every deliverable.

This can force irrelevant or fabricated imagery and conflicts with metadata-only asset planning and fallback_if_missing. Require meaningful visual elements, allowing no image when imagery does not clarify the key message.

🤖 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/SKILL.md` at line 21, Update the visual-content
requirement in the skill guidance to avoid requiring images for every
deliverable. Require meaningful visual elements only when they clarify the key
message, and allow metadata-only planning or no image when appropriate,
consistent with fallback_if_missing; remove the prohibition on image-free output
and duplicate-image restriction.
🤖 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/planning-layer.md`:
- Around line 207-222: Update the asset contract in the asset_need guidance to
explicitly allow supported XML-native primitives such as <shape> and <line> for
non-data diagram fallbacks, including architecture diagrams. Retain image
generation as an option and require native <chart> for supported data charts,
keeping the existing asset_type and chart_contract rules consistent with the
architecture_diagram example.

In `@skills/lark-slides/SKILL.md`:
- Line 41: 更新 SKILL.md 中关于交付 PPT 链接的要求:不要强制依赖 NotifyHuman;优先通过正常响应或输出提供 Slides
URL,并仅在 NotifyHuman 工具可用时使用它。

---

Outside diff comments:
In `@skills/lark-slides/SKILL.md`:
- Line 331: Standardize the executable command references by replacing the
dotted form with `xml_presentation.slide create` in skills/lark-slides/SKILL.md
lines 331-331 and
skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md lines
169-169. Verify the command spelling against meta_data.json, while retaining
dotted notation only when referring to schema or API identifiers.

---

Duplicate comments:
In `@skills/lark-slides/references/visual-planning.md`:
- Line 232: Update the visual-planning guidance to prohibit regenerating factual
evidence with image-generation tools. For screenshots, charts, paper figures,
and product captures, require cropping, zooming, splitting across slides, or
deterministic redrawing instead, while preserving the existing readability
guidance.

In `@skills/lark-slides/SKILL.md`:
- Line 240: Replace the raw Chinese placeholder text under the <data> template
section with a valid XML comment, preserving its guidance about placing shape,
line, table, and chart elements without leaving text as a direct child of
<data>.
- Line 21: Update the visual-content requirement in the skill guidance to avoid
requiring images for every deliverable. Require meaningful visual elements only
when they clarify the key message, and allow metadata-only planning or no image
when appropriate, consistent with fallback_if_missing; remove the prohibition on
image-free output and duplicate-image restriction.
🪄 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: 425abfb6-21b6-4844-aff9-d40ab016d8d7

📥 Commits

Reviewing files that changed from the base of the PR and between a5fedd8 and 9881acf.

📒 Files selected for processing (25)
  • skills/lark-slides/SKILL.md
  • skills/lark-slides/references/asset-planning.md
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/iconpark.md
  • skills/lark-slides/references/lark-slides-create.md
  • skills/lark-slides/references/lark-slides-media-upload.md
  • skills/lark-slides/references/lark-slides-pptx-template-workflows.md
  • skills/lark-slides/references/lark-slides-replace-slide.md
  • skills/lark-slides/references/lark-slides-screenshot.md
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-create.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-delete.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-get.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md
  • skills/lark-slides/references/lark-slides-xml-presentations-get.md
  • skills/lark-slides/references/planning-layer.md
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/slides_chart_demo.xml
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/slides_xml_schema_definition.xml
  • skills/lark-slides/references/troubleshooting.md
  • skills/lark-slides/references/validation-checklist.md
  • skills/lark-slides/references/visual-planning.md
  • skills/lark-slides/references/xml-schema-quick-ref.md
💤 Files with no reviewable changes (6)
  • skills/lark-slides/references/lark-slides-whiteboard.md
  • skills/lark-slides/references/slides_demo.xml
  • skills/lark-slides/references/lark-slides-xml-get.md
  • skills/lark-slides/references/slide-templates.md
  • skills/lark-slides/references/examples.md
  • skills/lark-slides/references/lark-slides-replace-slide.md
🚧 Files skipped from review as they are similar to previous changes (13)
  • skills/lark-slides/references/lark-slides-media-upload.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-get.md
  • skills/lark-slides/references/lark-slides-screenshot.md
  • skills/lark-slides/references/lark-slides-create.md
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-delete.md
  • skills/lark-slides/references/validation-checklist.md
  • skills/lark-slides/references/lark-slides-xml-presentations-get.md
  • skills/lark-slides/references/slides_chart_demo.xml
  • skills/lark-slides/references/lark-slides-xml-presentation-slide-replace.md
  • skills/lark-slides/references/asset-planning.md
  • skills/lark-slides/references/troubleshooting.md
  • skills/lark-slides/references/xml-schema-quick-ref.md
  • skills/lark-slides/references/slides_xml_schema_definition.xml

Comment thread skills/lark-slides/references/planning-layer.md
Comment thread skills/lark-slides/SKILL.md Outdated
Comment thread skills/lark-slides/SKILL.md
fangshuyu-768
fangshuyu-768 previously approved these changes Jul 20, 2026
@R0bynZhu
R0bynZhu force-pushed the update-lark-slides-skill-0715 branch from 0c1b4d1 to 3d49950 Compare July 20, 2026 11:01
@R0bynZhu
R0bynZhu merged commit 78390f8 into main Jul 20, 2026
26 checks passed
@R0bynZhu
R0bynZhu deleted the update-lark-slides-skill-0715 branch July 20, 2026 11:23
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 20, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XL Architecture-level or global-impact change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants