Skip to content

fix(batch): fixed the build issue with Batch processor due to missing dependencies #4498

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Sep 17, 2025
Merged

fix(batch): fixed the build issue with Batch processor due to missing dependencies #4498

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
fad8713
feat(batch): make parser dependencies optional with Standard Schema s…
dreamorosi Sep 16, 2025
1a37d87
chore: push tests
dreamorosi Sep 16, 2025
010d06b
Fixed the tests for DynamoDB and added a note for using innerSchema w…
sdangol Sep 16, 2025
751214e
fixed merge conflicts and added increased test coverage
sdangol Sep 16, 2025
e1a2ea9
added the transformer in the docstring
sdangol Sep 16, 2025
779e2da
Merge branch 'main' into fix/batch_deps
sdangol Sep 16, 2025
4b7f8ab
Changed the RuntimeZodType from any to unknown
sdangol Sep 16, 2025
fb9ebec
Merge branch 'fix/batch_deps' of github.com:aws-powertools/powertools…
sdangol Sep 16, 2025
dcc78d7
changed the runtime zodschema check to use direct checking instead of…
sdangol Sep 16, 2025
65033a2
Removed the RunTimeZodType
sdangol Sep 16, 2025
62b79ce
Merge branch 'main' into fix/batch_deps
dreamorosi Sep 16, 2025
0baecd2
Apply suggestion from @dreamorosi
sdangol Sep 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion packages/batch/src/parser.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import type {
*
* @param schema - The schema to check
*/
const isZodSchema = (schema: StandardSchemaV1): schema is RuntimeZodType =>
const isZodSchema = (schema: StandardSchemaV1) =>
schema['~standard'].vendor === SchemaVendor.Zod;

/**
Expand All @@ -40,20 +40,23 @@ const createExtendedSchema = async (options: {
const { JSONStringified } = await import(
'@aws-lambda-powertools/parser/helpers'
);
// @ts-expect-error - we know it's a Zod schema due to the runtime check earlier
schema = JSONStringified(innerSchema);
break;
}
case 'base64': {
const { Base64Encoded } = await import(
'@aws-lambda-powertools/parser/helpers'
);
// @ts-expect-error - we know it's a Zod schema due to the runtime check earlier
schema = Base64Encoded(innerSchema);
break;
}
case 'unmarshall': {
const { DynamoDBMarshalled } = await import(
'@aws-lambda-powertools/parser/helpers/dynamodb'
);
// @ts-expect-error - we know it's a Zod schema due to the runtime check earlier
schema = DynamoDBMarshalled(innerSchema);
break;
}
Expand Down
6 changes: 2 additions & 4 deletions packages/batch/src/types.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -248,13 +248,11 @@ type ParsedRecord<TRecord, TPayload, TOldPayload = TPayload> = TRecord extends {
* Parts of the parser integration within BatchProcessor rely on Zod for schema transformations,
* however some other parts also support other Standard Schema-compatible libraries.
*
* To avoid forcing a direct dependency on Zod, we use `any` here, which is not ideal but necessary.
* To avoid forcing a direct dependency on Zod, we use `unknown` here, which is not ideal but necessary.
*
* The vendor is checked at runtime to ensure Zod is being used when required using `StandardSchemaV1['~standard'].vendor`.
*
* biome-ignore lint/suspicious/noExplicitAny: using `any` to avoid direct dependency on 'zod'
*/
type RuntimeZodType = any;
type RuntimeZodType = unknown;

export type {
BatchProcessingOptions,
Expand Down
Loading