Skip to content

fix: whiteboard skill#1180

Merged
syh-cpdsss merged 1 commit into
mainfrom
fix/whiteboard_skill
May 30, 2026
Merged

fix: whiteboard skill#1180
syh-cpdsss merged 1 commit into
mainfrom
fix/whiteboard_skill

Conversation

@syh-cpdsss
Copy link
Copy Markdown
Collaborator

@syh-cpdsss syh-cpdsss commented May 30, 2026

Summary

fixed lark-doc refrences in lark-whiteboard skill

Changes

fixed lark-doc refrences in lark-whiteboard skill

Test Plan

  • Unit tests pass
  • Manual evaluated

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Updated instructions for extracting whiteboard tokens from documents to include the --api-version v2 flag in the fetch command for correct token parsing.

Review Change Stack

Change-Id: If62f9446dea1273a422567394a9e7d91b40be16e
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

📝 Walkthrough

Walkthrough

This PR updates the lark-whiteboard skill documentation to add the --api-version v2 parameter to the lark-cli docs +fetch command in Step 1 of the board_token extraction workflow, ensuring the command properly retrieves and parses the whiteboard token from the returned XML response.

Changes

Whiteboard Skill Documentation

Layer / File(s) Summary
CLI command documentation for token extraction
skills/lark-whiteboard/SKILL.md
The lark-cli docs +fetch command example in the board_token extraction workflow step now includes --api-version v2 to properly handle token extraction from the returned <whiteboard token="xxx"/> response.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • larksuite/cli#891: Both PRs update skills/lark-whiteboard/SKILL.md to change the lark-cli docs +fetch example for extracting an existing board_token by including --api-version v2.
  • larksuite/cli#1166: Both PRs modify the lark-cli docs +fetch command's --api-version v2 handling in the board_token workflow step (main PR adds it; retrieved PR removes it).

Suggested labels

size/M, domain/ccm

Suggested reviewers

  • zkh-bytedance

Poem

🐰 A whiteboard takes shape, with token in hand,
The API version guides us through this land,
v2 now flows through the fetch command clear,
Documentation blooms, the path is sincere! 📋✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and lacks detail required by the template. Missing specific descriptions of changes and incomplete test plan documentation. Provide detailed change descriptions (currently only repeats summary), specify which lark-doc references were fixed, and clearly document test results by checking the appropriate checkboxes in the test plan section.
Title check ❓ Inconclusive The title 'fix: whiteboard skill' is vague and does not clearly convey what the actual change is. It lacks specificity about what was fixed (the issue involves updating API version in a CLI command). Revise the title to be more specific, such as 'fix: add --api-version v2 flag to lark-cli docs fetch in whiteboard skill' to clearly indicate the nature of the fix.
✅ Passed checks (3 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.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/whiteboard_skill

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 May 30, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.92%. Comparing base (d126ea2) to head (2fead46).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1180   +/-   ##
=======================================
  Coverage   68.92%   68.92%           
=======================================
  Files         629      629           
  Lines       58762    58762           
=======================================
  Hits        40503    40503           
  Misses      14952    14952           
  Partials     3307     3307           

☔ 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

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

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

@syh-cpdsss syh-cpdsss merged commit b1ecf2d into main May 30, 2026
24 checks passed
@syh-cpdsss syh-cpdsss deleted the fix/whiteboard_skill branch May 30, 2026 02:35
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