Fixes

stevebux · stevebux · commit cf5f7ee2f4e4 · 2026-04-22T13:27:58.000+01:00
diff --git a/infrastructure/terraform/components/api/iam_role_upsert_idempotency.tf b/infrastructure/terraform/components/api/iam_role_upsert_idempotency.tf
diff --git a/infrastructure/terraform/components/api/module_lambda_upsert_letter.tf b/infrastructure/terraform/components/api/module_lambda_upsert_letter.tf
@@ -68,6 +68,7 @@ data "aws_iam_policy_document" "upsert_letter_lambda" {
 
     resources = [
       aws_dynamodb_table.letters.arn,
+      aws_dynamodb_table.upsert_idempotency.arn,
       "${aws_dynamodb_table.letters.arn}/index/supplierStatus-index"
     ]
   }
diff --git a/lambdas/upsert-letter/src/handler/upsert-handler.ts b/lambdas/upsert-letter/src/handler/upsert-handler.ts
@@ -252,10 +252,10 @@ export default function createUpsertLetterHandler(deps: Deps): SQSHandler {
 }
 
 async function processRecord(
-  deps: Deps,
   letterEvent: LetterStatusChangeEvent | PreparedEvents,
   supplierSpec: SupplierSpec | undefined,
   perSupplierSuccess: Map<string, number>,
+  deps: Deps,
 ) {
   const supplier =
     !supplierSpec || !supplierSpec.supplierId
@@ -283,7 +283,6 @@ async function processRecord(
 function makeIdempotentOnId(fn: AnyFunction, deps: Deps) {
   return makeIdempotent(fn, {
     persistenceStore: deps.idempotencyLayer,
-    dataIndexArgument: 1,
     config: new IdempotencyConfig({
       eventKeyJmesPath: "id",
     }),

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,7 @@ data "aws_iam_policy_document" "upsert_letter_lambda" {`
`68`	`68`
`69`	`69`	`resources = [`
`70`	`70`	`aws_dynamodb_table.letters.arn,`
	`71`	`+ aws_dynamodb_table.upsert_idempotency.arn,`
`71`	`72`	`"${aws_dynamodb_table.letters.arn}/index/supplierStatus-index"`
`72`	`73`	`]`
`73`	`74`	`}`