Skip to content

fix(whiteboard): Deprecate old lark-whiteboard-cli skill#547

Merged
zkh-bytedance merged 1 commit intomainfrom
fix/deprecate-lark-whiteboard-cli
Apr 17, 2026
Merged

fix(whiteboard): Deprecate old lark-whiteboard-cli skill#547
zkh-bytedance merged 1 commit intomainfrom
fix/deprecate-lark-whiteboard-cli

Conversation

@zkh-bytedance
Copy link
Copy Markdown
Collaborator

@zkh-bytedance zkh-bytedance commented Apr 17, 2026

Summary

Changes

  • manual deprecated lark-whiteboard-cli

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark xxx command works as expected

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Added a deprecation notice in the skill description: lark-whiteboard-cli has been merged into lark-whiteboard. If you still have the deprecated skill installed, remove it with npx skills remove lark-whiteboard-cli -g and use the consolidated lark-whiteboard skill. This change only updates the documentation/description; no behavioral or public API changes were made.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 246bd79a-def2-44e0-b9da-cd46b2543114

📥 Commits

Reviewing files that changed from the base of the PR and between b7f3658 and c7ba345.

📒 Files selected for processing (1)
  • skills/lark-whiteboard/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/lark-whiteboard/SKILL.md

📝 Walkthrough

Walkthrough

Updated skills/lark-whiteboard/SKILL.md to add a deprecation/merge notice instructing users that lark-whiteboard-cli has been merged into lark-whiteboard and how to remove the old skill if present.

Changes

Cohort / File(s) Summary
Documentation Update
skills/lark-whiteboard/SKILL.md
Added a warning/deprecation note stating lark-whiteboard-cli was merged into lark-whiteboard and provided removal instruction: npx skills remove lark-whiteboard-cli -g. No code or public API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

domain/base

Poem

🐰 I hopped through docs where changes hide,
Two skills merged now, side by side.
The old one waves a gentle goodbye,
Remove its traces with a single try. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description lacks a Summary section (required by template) and provides only a vague Changes entry without explaining the motivation or scope of the deprecation. Add a Summary section explaining why lark-whiteboard-cli is being deprecated and what users should do. Expand Changes to clearly describe the deprecation notice added.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(whiteboard): Deprecate old lark-whiteboard-cli skill' clearly and concisely describes the main change—deprecating the lark-whiteboard-cli skill.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/deprecate-lark-whiteboard-cli

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 and usage tips.

@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Apr 17, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.91%. Comparing base (09e60ee) to head (c7ba345).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #547   +/-   ##
=======================================
  Coverage   59.91%   59.91%           
=======================================
  Files         388      388           
  Lines       33147    33147           
=======================================
  Hits        19859    19859           
  Misses      11420    11420           
  Partials     1868     1868           

☔ View full report in Codecov by Sentry.
📢 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.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 17, 2026

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#fix/deprecate-lark-whiteboard-cli -y -g

@zkh-bytedance zkh-bytedance force-pushed the fix/deprecate-lark-whiteboard-cli branch from b7f3658 to c7ba345 Compare April 17, 2026 16:25
@zkh-bytedance zkh-bytedance requested a review from zhangzq0 April 17, 2026 16:32
@zkh-bytedance zkh-bytedance merged commit 5a0e1d3 into main Apr 17, 2026
22 checks passed
@zkh-bytedance zkh-bytedance deleted the fix/deprecate-lark-whiteboard-cli branch April 17, 2026 16:36
@liangshuo-1 liangshuo-1 mentioned this pull request Apr 20, 2026
2 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.

3 participants