Skip to content

feat: support webhook payload passthrough and dynamic notification recipients#4667

Open
huanghongbo-hhb wants to merge 6 commits into
koderover:mainfrom
huanghongbo-hhb:feat/workflow-payload-pass-through-v2
Open

feat: support webhook payload passthrough and dynamic notification recipients#4667
huanghongbo-hhb wants to merge 6 commits into
koderover:mainfrom
huanghongbo-hhb:feat/workflow-payload-pass-through-v2

Conversation

@huanghongbo-hhb
Copy link
Copy Markdown
Contributor

@huanghongbo-hhb huanghongbo-hhb commented May 7, 2026

Background

Workflow needs to pass through code-related context from webhook triggers and support dynamic recipients in IM notifications based on payload / author fields.

Changes

  • Pass through TARGET_BRANCH, COMMITTER, COMMIT_ID, COMMIT_MESSAGE, and PR for webhook-triggered workflows
  • Flatten webhook payload into runtime variables so it can be referenced directly in notification templates
  • Support using payload fields as notification recipient values via dynamic_recipients
  • Fix runtime context overwrite in retry / manual execution scenarios that caused build output data loss
  • Reuse the legacy Feishu webhook sending path and align URL encoding / proxy handling

Verification

  • GitHub PR webhook triggers workflow successfully
  • TARGET_BRANCH / COMMITTER / COMMIT_ID are passed through correctly
  • {{.payload.pull_request.user.login}} and other payload fields can be resolved
  • Mail / Feishu dynamic recipients can be sent successfully
  • Build image name and output variables are preserved in retry / manual execution scenarios

Note

  • GitLab was not fully validated in this chain.

This change is Reviewable

Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant