Skip to content

feat(lark-im): add chat.members.bots to skill#616

Open
shifengjuan-dev wants to merge 1 commit intolarksuite:mainfrom
shifengjuan-dev:feat/member_list_with_bot
Open

feat(lark-im): add chat.members.bots to skill#616
shifengjuan-dev wants to merge 1 commit intolarksuite:mainfrom
shifengjuan-dev:feat/member_list_with_bot

Conversation

@shifengjuan-dev
Copy link
Copy Markdown
Contributor

@shifengjuan-dev shifengjuan-dev commented Apr 22, 2026

  • Add chat.members.bots entry under chat.members API resources
  • Add chat.members.bots -> im:chat.members:read scope mapping

Change-Id: I57039a9a8649d794bbda84a1e41fae9cc31d570a

Summary

The chat.members.bots API is already registered in lark-cli but was missing
from the lark-im skill docs, so agents could not discover it when asked to
list bots in a chat. This PR adds the entry and its required scope mapping so
the skill stays in sync with the registered API surface.

Changes

  • Add chat.members.bots entry under the chat.members API resources section in skills/lark-im/SKILL.md
  • Add chat.members.botsim:chat.members:read row to the scope table

Test Plan

  • Docs-only change — no Go source touched, so make unit-test, go vet, gofmt -l ., go mod tidy, and golangci-lint are not affected by this diff
  • Manual verification in BOE: lark-cli im chat.members bots --params '{"chat_id":"oc_aa3a3f1a6150397580df93d0f59d92c0"}' --as user returns the bot list

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Added documentation for a new API method to retrieve the list of bots within a chat, including scope requirements and permission details.

  - Add chat.members.bots entry under chat.members API resources
  - Add chat.members.bots -> im:chat.members:read scope mapping

Change-Id: I57039a9a8649d794bbda84a1e41fae9cc31d570a
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 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: 055eb70a-8ce0-4569-8f68-947efffc40f7

📥 Commits

Reviewing files that changed from the base of the PR and between e6f3fa2 and 44449d7.

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

📝 Walkthrough

Walkthrough

Added documentation for the chat.members.bots API method in the Lark IM skill guide, including identity support specifications, caller conditions, and required permissions scope.

Changes

Cohort / File(s) Summary
Documentation Addition
skills/lark-im/SKILL.md
Added API method documentation for chat.members.bots with identity support details (user, bot), caller requirements, and scope permissions (im:chat.members:read).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

domain/im, size/M

Suggested reviewers

  • MaxHuang22
  • fangshuyu-768

Poem

🐰 A little note, so neat and clean,
For bots within the chat machine,
With scopes and rules all tucked away,
The docs shine bright in every way!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat(lark-im): add chat.members.bots to skill' directly and clearly summarizes the main change: adding the chat.members.bots API to the lark-im skill documentation.
Description check ✅ Passed The description covers all required template sections (Summary, Changes, Test Plan, Related Issues) with clear, detailed information about the documentation updates and verification approach.
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 Apr 22, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.01%. Comparing base (04e3a28) to head (44449d7).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #616      +/-   ##
==========================================
- Coverage   61.05%   60.01%   -1.05%     
==========================================
  Files         400      405       +5     
  Lines       34130    42830    +8700     
==========================================
+ Hits        20838    25703    +4865     
- Misses      11368    15108    +3740     
- Partials     1924     2019      +95     

☔ 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@44449d755e887753b0012c4b589fe3778e849639

🧩 Skill update

npx skills add shifengjuan-dev/lark-cli#feat/member_list_with_bot -y -g

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