CCM-11228: Implement basic schema and datastore package, integrating with existing lambdas (#95)

* CCM-11228: Implement basic schema and datastore package

* CCM-11228: Add logging functionality and update letter repository methods

* CCM-11228: Add test for letter list warning on invalid data

* CCM-11228: Update letter schema to include specificationId and groupId

* CCM-11228: Enhance letter repository tests and error handling

* CCM-11228: Resolve merge

* CCM-11228: Resolve merge

* CCM-11228: Add TTL fields to ddb repository

* Update internal/datastore/src/__test__/letter-repository.test.ts

Fix typo in test name

Co-authored-by: Francisco Videira <[email protected]>

* Update internal/datastore/src/letter-repository.ts

Update to more modern syntax

Co-authored-by: Francisco Videira <[email protected]>

* CCM-11228: Update letter repository tests and add MI schema and diagram generation

* Wire getLetters and ddb

* Wire patch letter and ddb

* Resolve merge conflicts

* Add delivered status

* Fix patch lambda permissions

---------

Co-authored-by: Francisco Videira <[email protected]>
Co-authored-by: Francisco Videira <[email protected]>

Author: m-houston

eslint.config.mjs

@@ -12,7 +12,7 @@ import unicorn from 'eslint-plugin-unicorn';
 import { defineConfig, globalIgnores } from 'eslint/config';
 import js from '@eslint/js';
 import html from 'eslint-plugin-html';
-import tseslint from 'typescript-eslint';
+import tseslint from '@typescript-eslint/parser';
 import sortDestructureKeys from 'eslint-plugin-sort-destructure-keys';
 import {
   configs as airbnbConfigs,

infrastructure/terraform/components/api/module_lambda_get_letters.tf

@@ -36,7 +36,8 @@ module "get_letters" {
   log_subscription_role_arn = local.acct.log_subscription_role_arn
 
   lambda_env_vars = {
-    LETTERS_TABLE_NAME = aws_dynamodb_table.letters.name
+    LETTERS_TABLE_NAME = aws_dynamodb_table.letters.name,
+    LETTER_TTL_HOURS = 24
   }
 }
 
@@ -61,13 +62,9 @@ data "aws_iam_policy_document" "get_letters_lambda" {
 
     actions = [
       "dynamodb:BatchGetItem",
-      "dynamodb:BatchWriteItem",
-      "dynamodb:DeleteItem",
       "dynamodb:GetItem",
-      "dynamodb:PutItem",
       "dynamodb:Query",
       "dynamodb:Scan",
-      "dynamodb:UpdateItem",
     ]
 
     resources = [

infrastructure/terraform/components/api/module_lambda_patch_letters.tf

@@ -9,7 +9,6 @@ module "patch_letters" {
   environment    = var.environment
   project        = var.project
   region         = var.region
-
   group          = var.group
 
   log_retention_in_days = var.log_retention_in_days
@@ -37,6 +36,8 @@ module "patch_letters" {
   log_subscription_role_arn = local.acct.log_subscription_role_arn
 
   lambda_env_vars = {
+    LETTERS_TABLE_NAME = aws_dynamodb_table.letters.name,
+    LETTER_TTL_HOURS = 24
   }
 }
 
@@ -54,4 +55,23 @@ data "aws_iam_policy_document" "patch_letters_lambda" {
       module.kms.key_arn, ## Requires shared kms module
     ]
   }
+
+    statement {
+    sid    = "AllowDynamoDBAccess"
+    effect = "Allow"
+
+    actions = [
+      "dynamodb:BatchGetItem",
+      "dynamodb:BatchWriteItem",
+      "dynamodb:GetItem",
+      "dynamodb:PutItem",
+      "dynamodb:Query",
+      "dynamodb:Scan",
+      "dynamodb:UpdateItem",
+    ]
+
+    resources = [
+      aws_dynamodb_table.letters.arn,
+    ]
+  }
 }

internal/datastore/.eslintignore

@@ -0,0 +1 @@
+dist

internal/datastore/.gitignore

@@ -0,0 +1,4 @@
+coverage
+node_modules
+dist
+.reports

internal/datastore/jest.config.ts

@@ -0,0 +1,60 @@
+import type { Config } from 'jest';
+
+export const baseJestConfig: Config = {
+  preset: 'ts-jest',
+
+  // Automatically clear mock calls, instances, contexts and results before every test
+  clearMocks: true,
+
+  // Indicates whether the coverage information should be collected while executing the test
+  collectCoverage: true,
+
+  // The directory where Jest should output its coverage files
+  coverageDirectory: './.reports/unit/coverage',
+
+  // Indicates which provider should be used to instrument code for coverage
+  coverageProvider: 'babel',
+
+  coverageThreshold: {
+    global: {
+      branches: 100,
+      functions: 100,
+      lines: 100,
+      statements: -10,
+    },
+  },
+
+  coveragePathIgnorePatterns: ['/__tests__/'],
+  transform: { '^.+\\.ts$': 'ts-jest' },
+  testPathIgnorePatterns: ['.build'],
+  testMatch: ['**/?(*.)+(spec|test).[jt]s?(x)'],
+
+  // Use this configuration option to add custom reporters to Jest
+  reporters: [
+    'default',
+    [
+      'jest-html-reporter',
+      {
+        pageTitle: 'Test Report',
+        outputPath: './.reports/unit/test-report.html',
+        includeFailureMsg: true,
+      },
+    ],
+  ],
+
+  // The test environment that will be used for testing
+  testEnvironment: 'jsdom',
+};
+
+const utilsJestConfig = {
+  ...baseJestConfig,
+
+  testEnvironment: 'node',
+
+  coveragePathIgnorePatterns: [
+    ...(baseJestConfig.coveragePathIgnorePatterns ?? []),
+    'zod-validators.ts',
+  ],
+};
+
+export default utilsJestConfig;