feat(lark-im): add chat.members.bots to skill#616
feat(lark-im): add chat.members.bots to skill#616shifengjuan-dev wants to merge 1 commit intolarksuite:mainfrom
Conversation
- Add chat.members.bots entry under chat.members API resources - Add chat.members.bots -> im:chat.members:read scope mapping Change-Id: I57039a9a8649d794bbda84a1e41fae9cc31d570a
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdded documentation for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@44449d755e887753b0012c4b589fe3778e849639🧩 Skill updatenpx skills add shifengjuan-dev/lark-cli#feat/member_list_with_bot -y -g |
Change-Id: I57039a9a8649d794bbda84a1e41fae9cc31d570a
Summary
The
chat.members.botsAPI is already registered in lark-cli but was missingfrom the
lark-imskill docs, so agents could not discover it when asked tolist 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
chat.members.botsentry under thechat.membersAPI resources section inskills/lark-im/SKILL.mdchat.members.bots→im:chat.members:readrow to the scope tableTest Plan
make unit-test,go vet,gofmt -l .,go mod tidy, and golangci-lint are not affected by this difflark-cli im chat.members bots --params '{"chat_id":"oc_aa3a3f1a6150397580df93d0f59d92c0"}' --as userreturns the bot listRelated Issues
Summary by CodeRabbit