Skip to content

Commit f23fdc1

Browse files
committed
fix(aws): add conditionExpression migration fallback for DynamoDB delete, fix SES pageSize min
1 parent 14f14c7 commit f23fdc1

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

apps/sim/app/api/tools/ses/list-identities/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const ListIdentitiesSchema = z.object({
1212
region: z.string().min(1, 'AWS region is required'),
1313
accessKeyId: z.string().min(1, 'AWS access key ID is required'),
1414
secretAccessKey: z.string().min(1, 'AWS secret access key is required'),
15-
pageSize: z.number().int().min(0).max(1000).nullish(),
15+
pageSize: z.number().int().min(1).max(1000).nullish(),
1616
nextToken: z.string().nullish(),
1717
})
1818

apps/sim/blocks/blocks/dynamodb.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -707,8 +707,10 @@ Return ONLY the expression - no explanations.`,
707707
// key: fall back to getKey (legacy migration target for shared 'key' subblock)
708708
const key = parseJson(params.deleteKey || params.getKey, 'key')
709709
if (key !== undefined) result.key = key
710-
if (params.deleteConditionExpression)
711-
result.conditionExpression = params.deleteConditionExpression
710+
// conditionExpression: fall back to updateConditionExpression (legacy migration target for shared 'conditionExpression' subblock)
711+
const deleteCondExpr =
712+
params.deleteConditionExpression || params.updateConditionExpression
713+
if (deleteCondExpr) result.conditionExpression = deleteCondExpr
712714
// expressionAttributeNames: fall back to queryExpressionAttributeNames (legacy migration target)
713715
const names = parseJson(
714716
params.deleteExpressionAttributeNames || params.queryExpressionAttributeNames,

0 commit comments

Comments
 (0)