Skip to content

feat(feishu): add opt-in ACK reaction (已读标记)#319

Closed
everettjf wants to merge 2 commits intomainfrom
feat/feishu-ack-reaction-v2
Closed

feat(feishu): add opt-in ACK reaction (已读标记)#319
everettjf wants to merge 2 commits intomainfrom
feat/feishu-ack-reaction-v2

Conversation

@everettjf
Copy link
Copy Markdown
Contributor

Summary

Add automatic ACK reaction (已读标记) feature to Feishu channel, disabled by default.

When enabled via ack_reaction: true in feishu config, the bot sends an emoji reaction on every received message as a read receipt.

Based on #290 by @lch541, with the following improvements:

  • Opt-in config: added ack_reaction field to FeishuChannelConfig (default false)
  • Simplified emoji selection: removed locale detection system (4 reaction pools, 5 char classification functions), replaced with a single unified pool of 6 universal emojis
  • Unit tests: added tests for deterministic index selection and valid emoji output

Supersedes #290. Closes #198.

Config example

feishu:
  ack_reaction: true

Test plan

  • cargo check passes
  • cargo test ack_reaction_tests passes
  • Manual: enable ack_reaction: true, send message in Feishu, verify emoji reaction appears

🤖 Generated with Claude Code

CH L and others added 2 commits April 1, 2026 12:10
- Add locale-aware ACK reactions (Chinese Simplified/Traditional, English, Japanese)
- Detect text locale and select appropriate emoji
- Send reaction automatically when receiving messages
- Ported from ZeroClaw's lark.rs implementation

Closes: #198
Add ack_reaction config field (default false) to gate the feature,
replace locale-based emoji pools with a single unified pool, and
add unit tests for the reaction picker.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@everettjf
Copy link
Copy Markdown
Contributor Author

Changes landed directly on main via 613eb7b. Closing this PR.

@everettjf everettjf closed this Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feishu 支持表情回复,例如 点赞

1 participant