Skip to content

feat(slides): add layout density lint for sparse/empty containers #1992

Merged
BD-ZERO merged 4 commits into
update-lark-slides-skill-0715-v10from
fix/slides_lint_empty
Jul 21, 2026
Merged

feat(slides): add layout density lint for sparse/empty containers #1992
BD-ZERO merged 4 commits into
update-lark-slides-skill-0715-v10from
fix/slides_lint_empty

Conversation

@BD-ZERO

@BD-ZERO BD-ZERO commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Description:

Summary

  • 新增 xml_layout_density_lint.py 静态检查脚本,识别大型 rect
    容器内可见内容面积偏低的候选区域(大卡片配少量文字、空图片占位、单字符伪主视觉、目录卡内容过空等场景),输出可复算的几何事实(target/rule/measurement/elements),不做视觉主观判断
  • 补充对应单元测试 xml_layout_density_lint_test.py
  • 更新 validation-checklist.md,明确脚本路径解析方式(相对已加载 SKILL.md 的父目录,不猜测全局安装路径)及调用顺序:xml-get → xml_text_overlap_lint(结构) → xml_layout_density_lint(密度) → 截图视觉验收

Test plan

  • python3 skills/lark-slides/scripts/xml_layout_density_lint_test.py 通过
  • 对已知稀疏容器样例 XML 跑 xml_layout_density_lint.py,确认 sparse_container_content 命中
  • 对正常密度页面跑同一脚本,确认无误报

BD-ZERO added 4 commits July 21, 2026 21:14
  - add XML-based sparse container detection with coverage metrics
  - return structured, actionable lint JSON
  - document lint usage and visual-review workflow
  - add regression tests for density-lint behavior
  - add XML-based sparse container detection with coverage metrics
  - return structured, actionable lint JSON
  - document lint usage and visual-review workflow
  - add regression tests for density-lint behavior
@BD-ZERO
BD-ZERO requested a review from R0bynZhu July 21, 2026 13:49
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: 6279ac0a-5cd5-4086-983a-d054a9312d1d

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 fix/slides_lint_empty

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.

@BD-ZERO
BD-ZERO requested review from R0bynZhu and ethan-zhx and removed request for R0bynZhu July 21, 2026 13:49
@github-actions github-actions Bot added the size/L Large or sensitive change across domains or core paths label Jul 21, 2026
@BD-ZERO
BD-ZERO merged commit 703d7f3 into update-lark-slides-skill-0715-v10 Jul 21, 2026
3 checks passed
@BD-ZERO
BD-ZERO deleted the fix/slides_lint_empty branch July 21, 2026 13:52
R0bynZhu pushed a commit that referenced this pull request Jul 22, 2026
)

* feat: add layout density lint for sparse slide containers
  - add XML-based sparse container detection with coverage metrics
  - return structured, actionable lint JSON
  - document lint usage and visual-review workflow
  - add regression tests for density-lint behavior

* feat: add layout density lint for sparse slide containers
  - add XML-based sparse container detection with coverage metrics
  - return structured, actionable lint JSON
  - document lint usage and visual-review workflow
  - add regression tests for density-lint behavior

* feat(slides): detect sparse short cards and slide-level density

* docs: resolve slide lint scripts from loaded skill directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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