Skip to content

Commit 86594a7

Browse files
committed
fix(ses): correct json input types for tags/dkimSigningAttributes
The SES block declared the tags and dkimSigningAttributes block inputs as 'string' instead of 'json', so the generic block executor never parsed the JSON code-editor value before forwarding it — workflow runs sent a raw JSON string where the contract now expects a structured object/array, failing validation. Also corrected the corresponding tool param TypeScript types, which were still typed as string | null.
1 parent 62137c3 commit 86594a7

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

apps/sim/blocks/blocks/ses.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -705,8 +705,8 @@ export const SESBlock: BlockConfig<ToolResponse> = {
705705
startDate: { type: 'string', description: 'Suppression list filter start date (ISO 8601)' },
706706
endDate: { type: 'string', description: 'Suppression list filter end date (ISO 8601)' },
707707
emailIdentity: { type: 'string', description: 'Email address or domain identity' },
708-
dkimSigningAttributes: { type: 'string', description: 'JSON BYODKIM signing attributes' },
709-
tags: { type: 'string', description: 'JSON array of key/value tags' },
708+
dkimSigningAttributes: { type: 'json', description: 'JSON BYODKIM signing attributes' },
709+
tags: { type: 'json', description: 'JSON array of key/value tags' },
710710
newConfigurationSetName: { type: 'string', description: 'Name for a new configuration set' },
711711
customRedirectDomain: { type: 'string', description: 'Custom domain for open/click tracking' },
712712
httpsPolicy: { type: 'string', description: 'HTTPS policy for tracking links' },

apps/sim/tools/ses/types.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -198,8 +198,12 @@ export interface SESListSuppressedDestinationsResponse extends ToolResponse {
198198

199199
export interface SESCreateEmailIdentityParams extends SESConnectionConfig {
200200
emailIdentity: string
201-
dkimSigningAttributes?: string | null
202-
tags?: string | null
201+
dkimSigningAttributes?: {
202+
domainSigningSelector?: string
203+
domainSigningPrivateKey?: string
204+
nextSigningKeyLength?: 'RSA_1024_BIT' | 'RSA_2048_BIT'
205+
} | null
206+
tags?: Array<{ key: string; value: string }> | null
203207
configurationSetName?: string | null
204208
}
205209

@@ -288,7 +292,7 @@ export interface SESCreateConfigurationSetParams extends SESConnectionConfig {
288292
reputationMetricsEnabled?: boolean | null
289293
sendingEnabled?: boolean | null
290294
suppressedReasons?: string | null
291-
tags?: string | null
295+
tags?: Array<{ key: string; value: string }> | null
292296
}
293297

294298
export interface SESCreateConfigurationSetResponse extends ToolResponse {

0 commit comments

Comments
 (0)