Skip to content

docs(lark-im): clarify message activity search - #865

Merged
BytedanceSearch merged 2 commits into
mainfrom
eval-search-im-routing-docs-a970
May 19, 2026
Merged

docs(lark-im): clarify message activity search#865
BytedanceSearch merged 2 commits into
mainfrom
eval-search-im-routing-docs-a970

Conversation

@BytedanceSearch

@BytedanceSearch BytedanceSearch commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Clarify that message activity review should keep --query "" and rely on sender/chat/time filters.
  • Replace fixed relative-date examples with runtime placeholders.
  • Add message_id, sender, chat, and create-time evidence validation guidance.
  • Keep this PR scoped to skills/lark-im/** only.

Eval Signal

Runs:

  • tests/eval-search/runs/2026-05-12T06-14Z-multientity-sandbox-a970
  • tests/eval-search/runs/2026-05-13Tfull-critical-info-sandbox

Observed:

  • The IM bot-interaction case needs empty-query activity review plus message_id/chat-context evidence validation.
  • Source data may contain concrete om_... message IDs or ou_... user IDs; those should be treated as strong recall targets.

Validation

  • git diff --check
  • PR diff is scoped to skills/lark-im/**.

Docs-only change; no unit tests were run.

Summary by CodeRabbit

  • Documentation
    • Added guidance for using bot identity when listing historical messages in group chats, including pagination instructions.
    • Added best practices for activity review searches with filter recommendations and identity constraint clarifications.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 74fb19a7-6744-492c-bef2-7f65e39f2f79

📥 Commits

Reviewing files that changed from the base of the PR and between e511404 and 39cc159.

📒 Files selected for processing (2)
  • skills/lark-im/references/lark-im-chat-messages-list.md
  • skills/lark-im/references/lark-im-messages-search.md

📝 Walkthrough

Walkthrough

This PR adds AI usage guidance to two Lark IM reference documentation files. The changes explain how to use bot identity to access historical messages in named groups and how to apply structured query filtering for activity review scenarios, including workarounds for bot-based message lookups.

Changes

Lark IM AI Usage Guidance for Bot Message Access

Layer / File(s) Summary
Bot identity message listing guidance
skills/lark-im/references/lark-im-chat-messages-list.md
Adds step 7 guidance for bot identity requests to list historical messages in named groups, specifying the +chat-search --as bot and +chat-messages-list --as bot command sequence, constraints against using +messages-search (user-only), and pagination with --page-token when has_more=true.
Activity review query boundary guidance
skills/lark-im/references/lark-im-messages-search.md
Adds "Query boundary for activity review" subsection defining when to keep --query empty and use structured filters (sender type, chat/person, time range) instead, clarifying user-only identity constraints, documenting the bot-interaction workaround via chat search and message listing, and specifying validation requirements using message_id, sender, chat, and create time evidence.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • YangJunzhou-01

Poem

🐰 A skill to search and list with grace,
Bot identity finds its rightful place,
Through chat and message, queries flow,
Activity reviewed from high to low,
Guidance shines where bots once wandered—
Now documented and well-pondered! 📚✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch eval-search-im-routing-docs-a970

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 May 13, 2026
@CLAassistant

CLAassistant commented May 13, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov

codecov Bot commented May 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.03%. Comparing base (e511404) to head (39cc159).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #865      +/-   ##
==========================================
+ Coverage   67.02%   67.03%   +0.01%     
==========================================
  Files         569      569              
  Lines       53508    53508              
==========================================
+ Hits        35864    35870       +6     
+ Misses      14649    14644       -5     
+ Partials     2995     2994       -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.

@BytedanceSearch
BytedanceSearch force-pushed the eval-search-im-routing-docs-a970 branch from ff76973 to 30dcaec Compare May 18, 2026 12:30
Change-Id: I2a9a928aab2354dfaf103cdf53add435088ff9e2
Change-Id: I6d89610db9f9d1488f207dcc6b92f7aada839f8b
@BytedanceSearch
BytedanceSearch force-pushed the eval-search-im-routing-docs-a970 branch from 30dcaec to 39cc159 Compare May 19, 2026 06:23
@BytedanceSearch BytedanceSearch self-assigned this May 19, 2026
@BytedanceSearch
BytedanceSearch marked this pull request as ready for review May 19, 2026 06:37
@BytedanceSearch
BytedanceSearch merged commit 95ffff4 into main May 19, 2026
21 of 22 checks passed
@BytedanceSearch
BytedanceSearch deleted the eval-search-im-routing-docs-a970 branch May 19, 2026 06:37
@github-actions

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@39cc159d66d53833ab3fddf01d757ffe85a1ab34

🧩 Skill update

npx skills add larksuite/cli#eval-search-im-routing-docs-a970 -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.

3 participants