Skip to content

docs(skills): clarify callout child rules#2048

Merged
SunPeiYang996 merged 1 commit into
mainfrom
docs/callout-child-rules
Jul 24, 2026
Merged

docs(skills): clarify callout child rules#2048
SunPeiYang996 merged 1 commit into
mainfrom
docs/callout-child-rules

Conversation

@fangshuyu-768

@fangshuyu-768 fangshuyu-768 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Clarify the Lark Doc XML rules for <callout> so agents use text blocks and avoid nesting unsupported block and resource types.

Changes

  • Expand the existing callout table row to prohibit raw text and unsupported block or resource types.

Test Plan

  • node scripts/skill-format-check/index.js skills
  • make unit-test
  • go vet ./...
  • gofmt -l . produces no output
  • go mod tidy produces no changes
  • go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6 run --new-from-rev=origin/main
  • Manual live verification covered standalone callout writes and nested table round-tripping.

Related Issues

  • None

@coderabbitai

coderabbitai Bot commented Jul 24, 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

更新 <callout> 的内容约束,禁止裸文本及指定块级或资源标签直接嵌入。

Changes

Callout XML 规则

Layer / File(s) Summary
Callout 直接子节点规则
skills/lark-doc/references/lark-doc-xml.md
明确文本必须位于段落、标题、列表、待办或引用等文本块内,禁止裸文本及 table/img/pre/hr/grid/whiteboard/sheet 等块级或资源标签直接嵌入。

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

Possibly related PRs

Suggested labels: size/M, domain/ccm

🚥 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 is concise and accurately summarizes the main change to Lark Doc callout child rules.
Description check ✅ Passed The description follows the required template and includes summary, changes, test plan, and related issues.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/callout-child-rules

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.

@github-actions github-actions Bot added domain/ccm PR touches the ccm domain size/M Single-domain feat or fix with limited business impact labels Jul 24, 2026
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#docs/callout-child-rules -y -g

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.14%. Comparing base (e7d5ecd) to head (6fc33b3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2048   +/-   ##
=======================================
  Coverage   75.14%   75.14%           
=======================================
  Files         911      911           
  Lines       96322    96322           
=======================================
  Hits        72381    72381           
  Misses      18372    18372           
  Partials     5569     5569           

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

@fangshuyu-768
fangshuyu-768 force-pushed the docs/callout-child-rules branch from 49efa8c to 6fc33b3 Compare July 24, 2026 09:52
@SunPeiYang996
SunPeiYang996 merged commit dd7f741 into main Jul 24, 2026
26 checks passed
@SunPeiYang996
SunPeiYang996 deleted the docs/callout-child-rules branch July 24, 2026 10:18
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 24, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/ccm PR touches the ccm domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants