docs(slides): generalize whole-page validation guidance

[tianyouskrrr]

Summary

Follow-up to #2227's restoration of the +update-slide route. The whole-page validation gate is a command-agnostic quality gate keyed on the "whole-page write-back" action, not on whichever shortcut currently performs it. #2227 hardcoded +update-slide into the validation CRITICAL in SKILL.md and the migrated workflow/validation-xml.md, reintroducing the special-casing that #2143 avoided. This PR generalizes the wording back to the behavior so the gate stays bound to the action and survives future default-command rollouts without further doc edits.

Changes

• skills/lark-slides/SKILL.md: drop the +update-slide special-case from the validation CRITICAL, keying it on whole-page write-back instead.
• skills/lark-slides/references/workflow/validation-xml.md: apply the same generalization.
• The lint CRITICAL's explicit command list is left unchanged — it has enumerated the whole-page XML entry points since the replace-pages era, so it is not a special-case introduced by fix(slides): restore update-slide skill guidance #2227.

Test Plan

• make quality-gate passes locally (EXIT=0), no REJECT; only a pre-existing skill_critical_noise WARNING unrelated to this PR remains.
• Documentation only; no behavior change.

Related Issues

• None

Summary by CodeRabbit

• Documentation
  • Clarified that explicit post-creation validation is required after creating slides, making major rewrites, or writing back an entire page.
  • Updated validation guidance to apply broadly to full-page changes rather than a specific update operation.

[coderabbitai]

📝 Walkthrough

Walkthrough

The change updates Lark Slides documentation so explicit validation applies after creation, major rewrites, and full-page write-backs. It removes the specific requirement to validate after every slides +update-slide operation.

Changes

Slides documentation

Layer / File(s)	Summary
Validation trigger guidance skills/lark-slides/SKILL.md, skills/lark-slides/references/workflow/validation-xml.md	The validation requirement now covers creation, major rewrites, and full-page write-backs. References to every slides +update-slide operation were removed.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• larksuite/cli#2143: Introduced the slides +update-slide behavior referenced by the updated validation guidance.
• larksuite/cli#2187: Modified the same slide skill and workflow guidance.
• larksuite/cli#2207: Introduced related post-edit validation guidance in validation-xml.md.

Suggested labels: documentation

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title clearly and concisely describes the main documentation change: generalizing whole-page validation guidance.
Description check	✅ Passed	The description follows the required template and clearly explains the scope, changes, verification, and related issues.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/slides-generalize-validation-guidance

---

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

[github-actions]

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#fix/slides-generalize-validation-guidance -y -g

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.36%. Comparing base (2016120) to head (3761a23).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2255   +/-   ##
=======================================
  Coverage   76.36%   76.36%           
=======================================
  Files        1011     1011           
  Lines      111269   111269           
=======================================
  Hits        84970    84970           
  Misses      19815    19815           
  Partials     6484     6484           

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

[tianyouskrrr]

Folded into #2225. The whole-page validation-gating generalization here (SKILL.md and workflow/validation-xml.md — "整页写回后" instead of hardcoding +update-slide) is now part of #2225 (commit 1bf712d), alongside pointing page_replace/slide_replace recovery at +update-slide and completing the #2227 +replace-pages doc deprecation. Closing to avoid duplication.