Skip to content

feat(im): cli support feed group#1102

Open
zhumiaoxin wants to merge 1 commit into
larksuite:mainfrom
zhumiaoxin:feat/feed-group
Open

feat(im): cli support feed group#1102
zhumiaoxin wants to merge 1 commit into
larksuite:mainfrom
zhumiaoxin:feat/feed-group

Conversation

@zhumiaoxin
Copy link
Copy Markdown
Collaborator

@zhumiaoxin zhumiaoxin commented May 26, 2026

Summary

Add IM feed group support documentation for lark-cli, making the raw im feed.groups.* APIs discoverable and easier for agents to use correctly.

Changes

  • Register feed.groups methods in the lark-im skill index with user-only identity requirements and required OAuth scopes.
  • Add a feed group reference covering create, update, delete, list, batch query, item add/remove, item query, and item list flows.
  • Document request/response shapes, pagination behavior, soft-deleted results, enum values, permissions, and example commands for feed group operations.

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark-cli im feed.groups <command> flow works as expected

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Clarified flag cancel command behavior documentation to describe best-effort double-cancel operations.
    • Clarified flag create command behavior documentation for feed-layer flag creation and automatic item type detection.
    • Added comprehensive feed groups API documentation with endpoints, request/response examples, parameters, enumerations, and required permissions.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 26, 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: 913dbc34-f38b-4d16-9a0c-167ec4836abc

📥 Commits

Reviewing files that changed from the base of the PR and between f12d279 and 0ec7254.

📒 Files selected for processing (4)
  • shortcuts/im/im_flag_cancel.go
  • shortcuts/im/im_flag_create.go
  • skills/lark-im/SKILL.md
  • skills/lark-im/references/lark-im-feed-groups.md

📝 Walkthrough

Walkthrough

This PR updates IM feature documentation by clarifying flag shortcut behavior and introducing comprehensive documentation for the new feed.groups API. It includes minor description refinements in shortcut metadata, a new API index section in the main docs, and a detailed reference page covering all feed group and feed card operations.

Changes

IM Documentation and API References

Layer / File(s) Summary
Flag shortcut description updates
shortcuts/im/im_flag_cancel.go, shortcuts/im/im_flag_create.go, skills/lark-im/SKILL.md
Updated ImFlagCancel and ImFlagCreate shortcut descriptions to clarify best-effort double-cancel semantics and feed-layer flag handling with auto-detected item type from chat mode.
Feed groups API introduction in SKILL.md
skills/lark-im/SKILL.md
Added new feed.groups API resource subsection listing feed group and feed card operations (create, update, delete, batch query/add/remove/query/list items, list groups) with required identities and token types. Extended permissions table with im:feed_group_v1 read/write scope mappings for all feed group methods.
Comprehensive feed.groups API reference documentation
skills/lark-im/references/lark-im-feed-groups.md
New documentation page covering prerequisites, common behavioral notes (user-only auth, soft-deleted lists, timestamp units), complete per-method request/response schemas and examples, enum definitions (group types, feed card types, rule types and operators), shared feed_group_rules shape with per-condition required fields, and permission scope requirements for each method.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • larksuite/cli#770: Prior PR introducing the +flag-cancel and +flag-create shortcut implementations; this PR refines their descriptions and adds the related feed.groups API documentation.

Suggested labels

domain/im, size/L

Suggested reviewers

  • YangJunzhou-01
  • haozhenghua-code

Poem

🐰 Feed groups now bloom in well-worn docs,
Flag shortcuts find their clearer voice,
With methods mapped and schemas locked,
The IM API makes its choice—
A rabbit's joy in thorough prose! 📚✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat(im): cli support feed group' is concise and directly describes the main change—adding CLI support for IM feed groups, which aligns with the PR's primary objective of exposing feed.groups APIs.
Description check ✅ Passed The PR description follows the template structure with Summary, Changes, Test Plan, and Related Issues sections. All required sections are present and appropriately filled, though one test checkbox remains 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.

@github-actions github-actions Bot added domain/im PR touches the im domain size/L Large or sensitive change across domains or core paths labels May 26, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.92%. Comparing base (ee9d090) to head (0ec7254).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1102      +/-   ##
==========================================
- Coverage   67.92%   67.92%   -0.01%     
==========================================
  Files         601      603       +2     
  Lines       55766    55779      +13     
==========================================
+ Hits        37880    37888       +8     
- Misses      14751    14755       +4     
- Partials     3135     3136       +1     

☔ 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@0ec7254a54b2df50854e6383eb4004a9ee1f9e88

🧩 Skill update

npx skills add zhumiaoxin/cli#feat/feed-group -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/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant