address Tyler's feedback

.github/workflows/qa.yml

@@ -301,20 +301,22 @@ jobs:
           ref: ${{ inputs.ref }}
           show-progress: 'false'
           persist-credentials: 'false'
-      - uses: actions/setup-python@v3
+      - uses: actions/setup-python@0b93645e9fea7318ecaed2b359559ac225c90a2b # v5.3.0
         with:
-          python-version: '3.12'
-      - run: pip install mypy
+          python-version-file: 'python/pyproject.toml'
+          cache: poetry
       - name: Get Python changed files
         id: changed-py-files
-        uses: tj-actions/changed-files@v23
+        uses: tj-actions/changed-files@@4edd678ac3f81e2dc578756871e4d00c19191daf # v45.0.4
         with:
           files: |
             *.py
             **/*.py
       - name: Run if any of the listed files above is changed
         if: steps.changed-py-files.outputs.any_changed == 'true'
-        run: mypy ${{ steps.changed-py-files.outputs.all_changed_files }} --ignore-missing-imports
+        env:
+          CHANGED_PY_FILES: ${{ steps.changed-py-files.outputs.all_changed_files }}
+        run: mypy $CHANGED_PY_FILES --ignore-missing-imports
 
 
   tflint:

python/pyproject.toml

@@ -20,6 +20,7 @@ ipython = "^8.22.2"
 pytest = "^8.1.1"
 pytest-cov = "^5.0.0"
 ruff = "^0.3.4"
+mypy = "^1.13.0"
 
 [build-system]
 requires = ["poetry-core"]

python/src/functions/create_archive.py

@@ -2,7 +2,7 @@
 import os
 import tempfile
 import zipfile
-from typing import Any
+from typing import Any, Optional
 
 import boto3
 import structlog
@@ -68,7 +68,7 @@ def create_archive(
     org_id: int,
     reporting_period_id: int,
     s3_client: S3Client,
-    logger: structlog.stdlib.BoundLogger = None,
+    logger: Optional[structlog.stdlib.BoundLogger] = None,
 ) -> None:
     """Create a zip archive of CSV files in S3"""
 

python/src/functions/test_lambdas.py

@@ -5,6 +5,7 @@
 from typing import Any
 
 from aws_lambda_typing.context import Context
+
 from src.lib.logging import get_logger, reset_contextvars
 
 
@@ -15,7 +16,7 @@ def pre_create_archive(event: dict[str, Any], _context: Context) -> dict[str, An
     """
     logger = get_logger()
     logger.info("Received new invocation event from step function")
-    logger.info(event)
+    logger.info(str(event))
     return {
         "statusCode": 200,
         "Payload": {