Skip to content

docs(im): name --query/--member-ids in +chat-search shortcut row#812

Merged
liangshuo-1 merged 1 commit into
larksuite:mainfrom
shifengjuan-dev:docs/im-chat-search-flag-names
May 11, 2026
Merged

docs(im): name --query/--member-ids in +chat-search shortcut row#812
liangshuo-1 merged 1 commit into
larksuite:mainfrom
shifengjuan-dev:docs/im-chat-search-flag-names

Conversation

@shifengjuan-dev
Copy link
Copy Markdown
Contributor

@shifengjuan-dev shifengjuan-dev commented May 11, 2026

Summary

  • Replace the natural-language phrase "by keyword and/or member open_ids" with the actual flag names --query / --member-ids in the +chat-search row of skills/lark-im/SKILL.md.
  • LLM agents reading only the shortcut table were inferring a non-existent --keyword flag from the prose. Naming the real flags inline matches the style already used by +chat-messages-list and removes the wrong-guess path.

Test plan

  • git diff main shows the intended single-line change in skills/lark-im/SKILL.md.
  • In a fresh Claude Code session with the updated skill installed, asking "搜群 X" causes the agent to call lark-cli im +chat-search --query "X" directly, without first guessing --keyword.

Summary by CodeRabbit

  • Documentation
    • Updated chat search shortcut documentation to improve clarity regarding member filtering parameters.

Review Change Stack

The +chat-search row in lark-im SKILL.md described the search as
"by keyword and/or member open_ids", which doesn't match the real
flag names (--query, --member-ids). Naming them inline avoids
agents guessing --keyword from the prose, matching the style
already used by +chat-messages-list.

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

coderabbitai Bot commented May 11, 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: 6e282962-e22d-4efd-8e1a-4468c6badd90

📥 Commits

Reviewing files that changed from the base of the PR and between 16f1a0f and 266a7c9.

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

📝 Walkthrough

Walkthrough

Updated the +chat-search Lark IM skill documentation to correct the member filtering parameter name from member open_ids to --member-ids, ensuring the documented interface matches the actual implementation.

Changes

Skill Documentation

Layer / File(s) Summary
Skill Reference
skills/lark-im/SKILL.md
The +chat-search shortcut documentation is updated to specify --member-ids as the correct flag for filtering by member identifiers, replacing the previous reference to member open_ids.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

documentation, domain/im, size/M

Suggested reviewers

  • YangJunzhou-01
  • MaxHuang22

Poem

A rabbit hops through docs so neat, 🐰
Fixing flags to make them sweet,
member-ids now shines so bright,
The search command reads just right! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating the documentation to name the actual CLI flags (--query and --member-ids) in the +chat-search shortcut row, which directly matches the file change in SKILL.md.
Description check ✅ Passed The description covers the summary and rationale well, but the Test Plan section is incomplete: one checkbox is marked done (git diff verification), but the more important Claude Code session verification is unchecked.
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.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.60%. Comparing base (29a9896) to head (266a7c9).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #812      +/-   ##
==========================================
+ Coverage   65.44%   65.60%   +0.16%     
==========================================
  Files         509      513       +4     
  Lines       47069    47644     +575     
==========================================
+ Hits        30806    31259     +453     
- Misses      13596    13685      +89     
- Partials     2667     2700      +33     

☔ 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@266a7c9ff71a14f33cc5717137c1308b6d0abd95

🧩 Skill update

npx skills add shifengjuan-dev/lark-cli#docs/im-chat-search-flag-names -y -g

@liangshuo-1 liangshuo-1 merged commit 25c72ce into larksuite:main May 11, 2026
18 checks passed
@liangshuo-1 liangshuo-1 mentioned this pull request May 11, 2026
2 tasks
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.

2 participants