Skip to content

feat: add extra SerDes module and cloud E2E coverage - #684

Open
zhongkechen wants to merge 1 commit into
mainfrom
feature/extra-serdes-cloud-e2e
Open

feat: add extra SerDes module and cloud E2E coverage#684
zhongkechen wants to merge 1 commit into
mainfrom
feature/extra-serdes-cloud-e2e

Conversation

@zhongkechen

Copy link
Copy Markdown
Contributor

Summary

  • add the optional aws-durable-execution-sdk-java-extra-serdes module
  • use AWS Lambda runtime serializers for SQSEvent, SNSEvent, S3Event, and other supported event models
  • add local, replay, and cloud E2E coverage for SQS event mappings
  • wire Maven, CI, publishing, coverage, examples, and documentation

Validation

  • mvn -B -Dmaven.repo.local=/tmp/durable-m2 -DskipTests install
  • focused extra-serdes unit tests
  • LambdaEventSerDesExampleTest
  • generated SAM template includes lambda-event-ser-des-example

The cloud test runs in the existing SAM-backed E2E workflow when cloud tests are enabled.

@zhongkechen
zhongkechen requested a review from a team September 3, 2026 22:54
@zhongkechen
zhongkechen deployed to ai-pr-review-runtime September 3, 2026 22:54 — with GitHub Actions Active
@zhongkechen
zhongkechen deployed to ai-pr-review-runtime September 3, 2026 23:24 — with GitHub Actions Active
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Codex AI review

No actionable findings. Residual test risk remains for Lambda event model types beyond the tested SQS, SNS, and S3 cases.

Reviewed commit 61996f7370758a0fedf13a8a9ab9a30a27ede1b1. Workflow run

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.

1 participant