Skip to content

8477 s3 refactor to test 3#9774

Merged
AbelTaxCourt merged 49 commits intotestfrom
8477-s3-refactor-to-test-3
Feb 24, 2026
Merged

8477 s3 refactor to test 3#9774
AbelTaxCourt merged 49 commits intotestfrom
8477-s3-refactor-to-test-3

Conversation

@AbelTaxCourt
Copy link
Contributor

  • Added documentStorageId field to the Docket Entry entity instead of using docketEntryId as the S3 key so docket entries on unconsolidated cases can update their own S3 documents independently from the consolidated group
    (Supports Consolidation: Change to QC/Edit Docket Entry Process #8477)
  • This PR includes the second migration that backfills the documentStorageId column on the Docket Entry table using docketEntryId
  • This PR includes the functionality that uses the new documentStorageId field
  • See 8477 s3 first migration #9706

AbelTaxCourt and others added 30 commits February 6, 2026 14:08
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
kampofo and others added 17 commits February 20, 2026 14:26
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
@AbelTaxCourt AbelTaxCourt marked this pull request as ready for review February 24, 2026 18:59
Copilot AI review requested due to automatic review settings February 24, 2026 18:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the AWS Terraform provider, Node.js runtime, and various dependencies across the infrastructure modules. It also removes temporary code related to issue #8477 and updates field documentation for the docket entry entity.

Changes:

  • Updated AWS Terraform provider from 6.31.0 to 6.32.1 (with one file at 6.33.0)
  • Updated Node.js runtime from 24.13.0 to 24.13.1 and npm from 11.6.2 to 11.8.0
  • Removed temporary code comment and fallback logic for documentStorageId field in docket entry mapper
  • Updated AWS SDK packages from 3.985.0 to 3.994.0 and other minor dependency versions

Reviewed changes

Copilot reviewed 37 out of 40 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
web-api/terraform/modules/worker/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/waf/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/ui/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/regional-log-subscription-filters/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/rds/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/opensearch-sync/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/kms/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/kibana/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/health-alarms/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/everything-else-deprecated/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/elasticsearch/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/default-vpc/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/modules/batch/docker-image/package.json Updated AWS SDK and Node type dependencies
web-api/terraform/modules/api/providers.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/wait-for-workflow/wait-for-workflow-cron-applyable.tf Updated AWS provider version to 6.33.0
web-api/terraform/applyables/switch-colors-cron/switch-colors-cron-applyable.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/stale-cases-email-cron/stale-cases-email-cron-applyable.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/reindex-cron/reindex-cron-applyable.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/green/green.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/blue/blue.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/allColors/allColors.tf Updated AWS provider version to 6.32.1
web-api/terraform/applyables/account-specific/account-specific.tf Updated AWS provider version to 6.32.1
web-api/src/persistence/postgres/docketEntries/mapper.ts Removed temporary fallback logic for documentStorageId
web-api/runtimes/puppeteer/package.json Updated Node.js, npm, and puppeteer-core versions
web-api/runtimes/puppeteer/Dockerfile Updated Node.js base image to 24.13.1
shared/src/business/useCases/loadPDFForSigningInteractor.ts Removed unused import and type annotation
shared/src/business/useCases/loadPDFForSigningInteractor.test.ts Renamed variable for clarity
shared/src/business/useCases/externalDocument/uploadOrderDocumentInteractor.ts Removed outdated JSDoc comment
shared/src/business/useCases/externalDocument/uploadOrderDocumentInteractor.test.ts Renamed variable for clarity
shared/src/business/entities/EntityValidationConstants.ts Updated documentation for docketEntryId and added documentStorageId field
scripts/verify-terraform-version.sh Updated required Terraform version to 1.14.5
package.json Updated Node.js, npm, AWS SDK packages, and various dependencies
docs/dependency-updates.md Added documentation for dependency version updates and known issues
Dockerfile Updated Node.js image, AWS CLI, and Terraform versions
CHANGES.md Added deployment instructions for dependency updates
8477TODO.md Removed temporary TODO file
.circleci/config.yml Updated Docker image version and Node.js version
Files not reviewed (2)
  • web-api/runtimes/puppeteer/package-lock.json: Language not supported
  • web-api/terraform/modules/batch/docker-image/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

aws = {
source = "hashicorp/aws"
version = "6.31.0"
version = "6.33.0"
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file uses AWS provider version 6.33.0 while all other Terraform files use 6.32.1. According to the documentation added in this PR (docs/dependency-updates.md lines 303-305), version 6.33.0 caused deployment failures and should be reverted to 6.32.1 for consistency.

Suggested change
version = "6.33.0"
version = "6.32.1"

Copilot uses AI. Check for mistakes.
AbelTaxCourt and others added 2 commits February 24, 2026 14:33
Co-authored-by: Christie Laferriere <christie.laferriere.ctr@ustaxcourt.gov>
@AbelTaxCourt AbelTaxCourt merged commit e2643a1 into test Feb 24, 2026
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants