Skip to content

docs: refine doubao whiteboard workflow routing#1841

Merged
cl900811 merged 1 commit into
mainfrom
feat/whiteboard-doubao-workflow
Jul 10, 2026
Merged

docs: refine doubao whiteboard workflow routing#1841
cl900811 merged 1 commit into
mainfrom
feat/whiteboard-doubao-workflow

Conversation

@cl900811

@cl900811 cl900811 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR refine doubao whiteboard workflow, use svg first

Changes

  • refine doubao whiteboard workflow, svg first

Test Plan

  • Unit tests pass

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Clarified that rendering routes are evaluated from top to bottom and stop at the first match.
    • Updated routing guidance for other chart types: Doubao and Seed identities now use SVG routes, while only Other uses DSL routes.
    • Preserved existing routing behavior for Mermaid class diagrams and specified chart types under Doubao and Seed.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5ba7cc98-468a-4dfb-98b5-ee7171281640

📥 Commits

Reviewing files that changed from the base of the PR and between 3d80e06 and 59a4bb5.

📒 Files selected for processing (1)
  • skills/lark-whiteboard/references/lark-whiteboard-workflow.md

📝 Walkthrough

Walkthrough

This PR updates the whiteboard rendering routing documentation with top-to-bottom matching and revised identity-based routing for “其他图表”.

Changes

Routing rule documentation

Layer / File(s) Summary
Rendering route rule update
skills/lark-whiteboard/references/lark-whiteboard-workflow.md
Documents first-match routing and changes Doubao/Seed identities to use the SVG route, while only Other uses the DSL route for “其他图表”.

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

Possibly related PRs

  • larksuite/cli#1371: Updates related rendering route and identity-routing rules in the same whiteboard workflow documentation.

Suggested labels: domain/ccm

Suggested reviewers: syh-cpdsss, zhangzq0

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the PR’s main change: refining the whiteboard routing to prefer SVG.
Description check ✅ Passed The description follows the required template with Summary, Changes, Test Plan, and Related Issues filled in.
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/whiteboard-doubao-workflow

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 the size/M Single-domain feat or fix with limited business impact label Jul 9, 2026
zkh-bytedance
zkh-bytedance previously approved these changes Jul 9, 2026
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#feat/whiteboard-doubao-workflow -y -g

@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.44%. Comparing base (519a600) to head (59a4bb5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1841   +/-   ##
=======================================
  Coverage   74.44%   74.44%           
=======================================
  Files         860      860           
  Lines       89859    89859           
=======================================
  Hits        66897    66897           
  Misses      17778    17778           
  Partials     5184     5184           

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

@cl900811
cl900811 merged commit 4a4cc1e into main Jul 10, 2026
47 of 49 checks passed
@cl900811
cl900811 deleted the feat/whiteboard-doubao-workflow branch July 10, 2026 03:33
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 13, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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