Skip to content

docs: add media-preview reference#990

Merged
fangshuyu-768 merged 1 commit into
mainfrom
docs/add-media-preview-reference
May 20, 2026
Merged

docs: add media-preview reference#990
fangshuyu-768 merged 1 commit into
mainfrom
docs/add-media-preview-reference

Conversation

@wittam-01
Copy link
Copy Markdown
Collaborator

@wittam-01 wittam-01 commented May 20, 2026

Summary

Add the missing +media-preview shortcut reference to the lark-doc skill documentation so agents can discover the media preview workflow from the shortcut table.

Changes

  • Added +media-preview to skills/lark-doc/SKILL.md
  • Linked it to references/lark-doc-media-preview.md with a brief description

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark xxx command works as expected
  • Not run; documentation-only change

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Added documentation for the media preview functionality shortcut entry.

Review Change Stack

Change-Id: I5ba1991874e262fb98f3421e61503b58bb71d861
@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 May 20, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 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: d5d5264f-ee5a-4d9f-80fd-e9eb47ec032b

📥 Commits

Reviewing files that changed from the base of the PR and between 9272b9d and 675db2a.

📒 Files selected for processing (1)
  • skills/lark-doc/SKILL.md

📝 Walkthrough

Walkthrough

A single documentation entry is added to the Shortcuts table in skills/lark-doc/SKILL.md to register the +media-preview shortcut and link it to the corresponding media preview reference documentation.

Changes

Documentation Updates

Layer / File(s) Summary
Media preview shortcut entry
skills/lark-doc/SKILL.md
A new +media-preview shortcut row is added to the Shortcuts table, pointing to references/lark-doc-media-preview.md.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • larksuite/cli#334: Introduces the docs +media-preview feature that this PR registers as a shortcut in the skill descriptor.

Suggested labels

domain/ccm

Suggested reviewers

  • liujinkun2025
  • fangshuyu-768

Poem

A shortcut springs to life today,
Media preview shows the way,
One humble line in docs so true,
The table grows—our skills renew! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding a media-preview reference to documentation.
Description check ✅ Passed The description covers all required template sections (Summary, Changes, Test Plan, Related Issues) with sufficient detail about the documentation update.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/add-media-preview-reference

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


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.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.58%. Comparing base (27a5eed) to head (675db2a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #990   +/-   ##
=======================================
  Coverage   67.58%   67.58%           
=======================================
  Files         575      575           
  Lines       54269    54269           
=======================================
  Hits        36679    36679           
  Misses      14548    14548           
  Partials     3042     3042           

☔ 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@675db2a39923f7b282a6660f5dc486980010be8a

🧩 Skill update

npx skills add larksuite/cli#docs/add-media-preview-reference -y -g

@fangshuyu-768 fangshuyu-768 merged commit 8c73f49 into main May 20, 2026
24 checks passed
@fangshuyu-768 fangshuyu-768 deleted the docs/add-media-preview-reference branch May 20, 2026 07:59
@liangshuo-1 liangshuo-1 mentioned this pull request May 20, 2026
2 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