Skip to content

fix(feishu): keep bridge reactions best-effort#217

Merged
RussellLuo merged 1 commit into
OpenCSGs:mainfrom
GatewayJ:fix/feishu-bridge-processing-reactions
Jun 18, 2026
Merged

fix(feishu): keep bridge reactions best-effort#217
RussellLuo merged 1 commit into
OpenCSGs:mainfrom
GatewayJ:fix/feishu-bridge-processing-reactions

Conversation

@GatewayJ

Copy link
Copy Markdown
Collaborator

Summary

  • Make Feishu processing reactions asynchronous best-effort with bounded cleanup.
  • Add context-aware Feishu update/reaction service paths and require room_id for local update consistency.
  • Cover update fallback and reaction add/delete failure paths with regression tests.

Validation

  • make fmt
  • git diff --check
  • go test -count=1 ./internal/channel/feishu ./internal/channelbridge/codexbridge
  • go test -count=10 ./internal/channelbridge/codexbridge -run 'FeishuProcessingPin|GeneratedRootUpdate'

Notes

  • Full go test ./... still has existing internal/api failures unrelated to this change: TestDirectoryPickerCanceled and TestHandleAgentsPatchFieldMaskClearsRuntimeOptions.

@GatewayJ GatewayJ marked this pull request as ready for review June 18, 2026 03:16
@GatewayJ GatewayJ requested a review from RussellLuo June 18, 2026 03:16
@RussellLuo RussellLuo merged commit f710b71 into OpenCSGs:main Jun 18, 2026
1 check passed
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.

2 participants