Skip to content

fix(util-dynamodb): add alternate type-correct APIs for marshalling - #8198

Draft
kuhe wants to merge 2 commits into
mainfrom
kuhe/fix/ddb
Draft

fix(util-dynamodb): add alternate type-correct APIs for marshalling#8198
kuhe wants to merge 2 commits into
mainfrom
kuhe/fix/ddb

Conversation

@kuhe

@kuhe kuhe commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Issue

#8154

Description

Creates a new codec API to replace the marshall/unmarshall API in util-dynamodb.
The old API is too entangled with broken type interfaces.

This avoids making a breaking-change fix in util-dynamodb.

However, lib-dynamodb changes to use the new codec API, and includes some breakfix changes, narrowing some types to only those that work with DynamoDB.

Testing

new unit tests and existing lib-dynamodb e2e tests.

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts) or E2E tests.
    • It's not a feature.
  • My E2E tests are resilient to concurrent i/o.
    • I didn't write any E2E tests.
  • I added access level annotations e.g. @public, @internal tags and enabled doc generation on the package. Remember that access level annotations go below the description, not above.
    • I didn't add any public functions.
  • Streams - how do they work?? My WebStream readers/locks are properly lifecycled. Node.js stream backpressure is handled. Error handling.
    • No streams here.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@kuhe
kuhe requested a review from a team as a code owner July 15, 2026 19:39
@kuhe

kuhe commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author
  • todo: codegen updates to lib-dynamodb

@kuhe
kuhe force-pushed the kuhe/fix/ddb branch 2 times, most recently from 278732a to 28105fe Compare July 16, 2026 14:44
@kuhe
kuhe marked this pull request as draft July 16, 2026 18:15
@kuhe

kuhe commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author
  • needs opt-in to new behavior

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