Skip to content

feat(webhook): sends Feishu/larksuit direct messages to issue/PR participants via the Feishu Open API#38377

Open
jan2xue wants to merge 4 commits into
go-gitea:mainfrom
jan2xue:main
Open

feat(webhook): sends Feishu/larksuit direct messages to issue/PR participants via the Feishu Open API#38377
jan2xue wants to merge 4 commits into
go-gitea:mainfrom
jan2xue:main

Conversation

@jan2xue

@jan2xue jan2xue commented Jul 9, 2026

Copy link
Copy Markdown

Add Feishu app integration that sends direct messages to issue/PR
participants via the Feishu Open API, mirroring the email notifier.

Features:

  • HookTask.DeliveryRecipients for persisting DM recipients
  • feishu_app.go with token caching and message delivery
  • deliveryRecipientEmails in notifier to resolve recipients
  • app_id/app_secret fields in Feishu webhook form
  • Copy DeliveryRecipients in ReplayHookTask for redelivery
  • Exclude explicitly unwatched users, matching mailer behavior
  • Token cache expiry derived from API expire field

yanchao.zhou and others added 2 commits July 9, 2026 19:38
Add Feishu app integration that sends direct messages to issue/PR
participants via the Feishu Open API, mirroring the email notifier.

Features:
- HookTask.DeliveryRecipients for persisting DM recipients
- feishu_app.go with token caching and message delivery
- deliveryRecipientEmails in notifier to resolve recipients
- app_id/app_secret fields in Feishu webhook form
- Copy DeliveryRecipients in ReplayHookTask for redelivery
- Exclude explicitly unwatched users, matching mailer behavior
- Token cache expiry derived from API expire field
- Style consistency (modules/json, NewRequestWithContext)
- Remove unused UseRequestSecret from template

Assisted-by: CodeBuddy:GLM-5.2
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 9, 2026
@jan2xue jan2xue changed the title Add Feishu app integration that sends direct messages to issue/PR participants via the Feishu Open API feat(webhook): sends Feishu/larksuit direct messages to issue/PR participants via the Feishu Open API Jul 10, 2026
@github-actions github-actions Bot added the type/feature Completely new functionality. Can only be merged if feature freeze is not active. label Jul 10, 2026
- Use errors.New instead of fmt.Errorf for static message
- Use assert.Contains instead of assert.True with strings.Contains
- Use range over int instead of traditional for loop
- Add meta field to deliver test hookCase for Feishu credentials

Assisted-by: CodeBuddy:GLM-5.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/feature Completely new functionality. Can only be merged if feature freeze is not active.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants