Skip to content

docs(im): clarify audio message opus requirement#1271

Open
chenxingtong-bytedance wants to merge 1 commit into
larksuite:mainfrom
chenxingtong-bytedance:fix/audio_message_only_opus
Open

docs(im): clarify audio message opus requirement#1271
chenxingtong-bytedance wants to merge 1 commit into
larksuite:mainfrom
chenxingtong-bytedance:fix/audio_message_only_opus

Conversation

@chenxingtong-bytedance
Copy link
Copy Markdown
Contributor

@chenxingtong-bytedance chenxingtong-bytedance commented Jun 4, 2026

Change-Id: I48efb309b6222b16f39f18bbb8be039d06c0087c

Summary

Clarify the IM skill guidance for audio messages so agents know that --audio requires Opus-format files when sending or replying with audio messages.

Changes

  • Added an Audio Messages note to the IM skill documentation.
  • Documented that --audio supports only Opus-format files for audio messages.

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark-cli <domain> <command> flow works as expected
  • Ran git diff --check
  • Documentation-only change; no runtime behavior changed

Related Issues

Summary by CodeRabbit

  • Documentation
    • Added guidance for Audio Messages: sending and replying with audio requires Opus-format files (for example, .opus or .ogg).
    • Clarified expected behavior and compatibility to help users prepare audio files that will be accepted by the messaging feature.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 4, 2026

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: b6583080-75ca-4c18-be1b-8fea2ab9202c

📥 Commits

Reviewing files that changed from the base of the PR and between 7cfb2b5 and 4416f80.

📒 Files selected for processing (1)
  • skills/lark-im/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • skills/lark-im/SKILL.md

📝 Walkthrough

Walkthrough

This PR adds an "Audio Messages" section to the Lark IM skill guide, stating that --audio supports only Opus-format files for sending and replying with audio messages.

Changes

Audio Messages documentation

Layer / File(s) Summary
Audio Messages section
skills/lark-im/SKILL.md
Adds an "Audio Messages" subsection that limits --audio to Opus-format files for send and reply operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • haozhenghua-code
  • liangshuo-1

Poem

🐰 I found a note in skillful trails,
Where audio hops with Opus tales.
.opus and .ogg now softly sing,
A tiny doc change with a tidy spring.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: clarifying documentation about audio message format requirements in the IM skill.
Description check ✅ Passed The description follows the required template with complete sections: Summary, Changes, Test Plan (with all items checked), 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.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 domain/im PR touches the im domain size/M Single-domain feat or fix with limited business impact labels Jun 4, 2026
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/lark-im/SKILL.md`:
- Line 60: Update the `--audio` flag documentation to state that it accepts
Opus-format files and explicitly list supported extensions by name (e.g.,
".opus" and ".ogg") to match runtime behavior; reference the `--audio` flag text
in SKILL.md and replace the sentence "only Opus-format files" with a clarified
line like "Opus-format files (e.g., .opus and .ogg) are supported" so users know
.ogg is mapped to Opus at runtime.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ff3f36db-a871-4396-8e06-7357f0c2e183

📥 Commits

Reviewing files that changed from the base of the PR and between ac116e7 and 7cfb2b5.

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

Comment thread skills/lark-im/SKILL.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add chenxingtong-bytedance/cli#fix/audio_message_only_opus -y -g

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.18%. Comparing base (abc0553) to head (7cfb2b5).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1271      +/-   ##
==========================================
+ Coverage   69.84%   70.18%   +0.33%     
==========================================
  Files         666      671       +5     
  Lines       64451    65240     +789     
==========================================
+ Hits        45019    45786     +767     
+ Misses      15812    15783      -29     
- Partials     3620     3671      +51     

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

Change-Id: I48efb309b6222b16f39f18bbb8be039d06c0087c
@chenxingtong-bytedance chenxingtong-bytedance force-pushed the fix/audio_message_only_opus branch from 7cfb2b5 to 4416f80 Compare June 4, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/im PR touches the im 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.

1 participant