Skip to content

CCM-14200 - event driven callback delivery#33

Merged
mjewildnhs merged 87 commits intomainfrom
feature/CCM-14200-event-driven-callback-delivery
Mar 4, 2026
Merged

CCM-14200 - event driven callback delivery#33
mjewildnhs merged 87 commits intomainfrom
feature/CCM-14200-event-driven-callback-delivery

Conversation

@mjewildnhs
Copy link
Contributor

@mjewildnhs mjewildnhs commented Feb 17, 2026

Description

Implement the event-driven callback delivery pipeline. Replaces the placeholder lambda handler stub with a full implementation: an SQS-to-EventBridge Pipes enrichment Lambda (client-transform-filter-lambda) that validates, transforms, and routes CloudEvents to client webhook endpoints via EventBridge. A new mock-webhook-lambda is added for use in tests.

The integration test requires further work but is dependent on other PRs (#43 / #32)

Changes:

  • Implements the client-transform-filter-lambda handler with full validation (CloudEvents + Zod), transformation (MessageStatus/ChannelStatus), observability (metrics, structured logging), and structured error handling
  • Adds a mock-webhook-lambda that acts as a test webhook receiver, logging received callbacks to CloudWatch in a queryable format
  • Adds an integration test suite (tests/integration/) that publishes SQS messages and verifies end-to-end callback delivery, alongside infrastructure Terraform changes to support the new pipeline.

Context

Required for phase 1 of 2-way-messaging.

Type of changes

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming
  • If I have used the 'skip-trivy-package' label I have done so responsibly and in the knowledge that this is being fixed as part of a separate ticket/PR.

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

@mjewildnhs mjewildnhs force-pushed the feature/CCM-14200-event-driven-callback-delivery branch 3 times, most recently from 9447393 to c90f89d Compare February 18, 2026 11:52
@mjewildnhs mjewildnhs changed the base branch from main to feature/CCM-14199-foundation February 18, 2026 13:20
@mjewildnhs
Copy link
Contributor Author

@copilot review

Copy link

Copilot AI commented Feb 18, 2026

@mjewildnhs I've opened a new pull request, #34, to work on those changes. Once the pull request is ready, I'll request review from you.

@mjewildnhs mjewildnhs changed the base branch from feature/CCM-14199-foundation to main February 18, 2026 13:24
@mjewildnhs mjewildnhs marked this pull request as ready for review February 18, 2026 13:40
@mjewildnhs mjewildnhs requested review from a team as code owners February 18, 2026 13:40
@mjewildnhs mjewildnhs marked this pull request as draft February 18, 2026 13:41
@mjewildnhs mjewildnhs closed this Feb 18, 2026
@mjewildnhs mjewildnhs reopened this Feb 18, 2026
@mjewildnhs mjewildnhs force-pushed the feature/CCM-14200-event-driven-callback-delivery branch 2 times, most recently from c412701 to 6287115 Compare February 24, 2026 16:49
rhyscoxnhs
rhyscoxnhs previously approved these changes Feb 26, 2026
Copy link
Contributor

@rhyscoxnhs rhyscoxnhs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No real fundamental issues here. I've left some minor suggestions for improvements, but there's no need to block a merge for these.

@mjewildnhs mjewildnhs force-pushed the feature/CCM-14200-event-driven-callback-delivery branch from 848f634 to fc910ea Compare March 2, 2026 17:27
@mjewildnhs mjewildnhs force-pushed the feature/CCM-14200-event-driven-callback-delivery branch from b988105 to 48be4bf Compare March 3, 2026 15:33
@mjewildnhs mjewildnhs force-pushed the feature/CCM-14200-event-driven-callback-delivery branch from 0bb06ad to e2d0639 Compare March 3, 2026 17:05
@mjewildnhs mjewildnhs merged commit 4b14dd9 into main Mar 4, 2026
34 checks 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.

6 participants