Skip to content

fix: remove legacy shortcut#1997

Merged
syh-cpdsss merged 1 commit into
mainfrom
fix/celar_wb_legacy_shortcut
Jul 22, 2026
Merged

fix: remove legacy shortcut#1997
syh-cpdsss merged 1 commit into
mainfrom
fix/celar_wb_legacy_shortcut

Conversation

@syh-cpdsss

@syh-cpdsss syh-cpdsss commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

After whiteboard +query shortcut renamed whiteboard +export, change remaining +query quotes in lark-doc and lark-drive skill.

Changes

whiteboard +query shortcut quotes in lark-doc-whiteboard.md and lark-drive-comment-location.md

Test Plan

  • Unit tests pass

Related Issues

  • None

Summary by CodeRabbit

  • 文档
    • 更新画板 SVG 处理示例,改用导出命令生成预览图片。
    • 更新画板内部评论定位示例,采用新的 raw 节点导出参数。
    • 统一相关命令示例格式,提升操作指引的准确性与可用性。

@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 22, 2026
@coderabbitai

coderabbitai Bot commented Jul 22, 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: d6a621f7-d9e0-4ed7-84f1-7c3d4b7d2a14

📥 Commits

Reviewing files that changed from the base of the PR and between bb246b5 and 58a1012.

📒 Files selected for processing (2)
  • skills/lark-doc/references/lark-doc-whiteboard.md
  • skills/lark-drive/references/lark-drive-comment-location.md

📝 Walkthrough

Walkthrough

更新两个白板文档示例,将 whiteboard +query 替换为 whiteboard +export,并分别使用 previewraw 输出类型参数。

Changes

Whiteboard command documentation

Layer / File(s) Summary
Update whiteboard export examples
skills/lark-doc/references/lark-doc-whiteboard.md, skills/lark-drive/references/lark-drive-comment-location.md
预览示例改用 --output-type preview,评论定位示例改用 --output-type raw,并保留对应的输出路径和白板令牌参数。

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

Possibly related PRs

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the main change: removing a legacy shortcut reference.
Description check ✅ Passed The description follows the template and covers summary, changes, test plan, and related issues.
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 fix/celar_wb_legacy_shortcut

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

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@58a1012fd4cf31f7a49166361cf10902ff185d92

🧩 Skill update

npx skills add larksuite/cli#fix/celar_wb_legacy_shortcut -y -g

@syh-cpdsss
syh-cpdsss merged commit 54ddcf4 into main Jul 22, 2026
25 of 39 checks passed
@syh-cpdsss
syh-cpdsss deleted the fix/celar_wb_legacy_shortcut branch July 22, 2026 07:33
@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.06%. Comparing base (bb246b5) to head (58a1012).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1997   +/-   ##
=======================================
  Coverage   75.06%   75.06%           
=======================================
  Files         902      902           
  Lines       95944    95944           
=======================================
  Hits        72025    72025           
  Misses      18380    18380           
  Partials     5539     5539           

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

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