Skip to content

Align S3 prefixes with APHL eCR pipeline#419

Open
nickclyde wants to merge 1 commit intomainfrom
nickclyde/more-prefix-changes
Open

Align S3 prefixes with APHL eCR pipeline#419
nickclyde wants to merge 1 commit intomainfrom
nickclyde/more-prefix-changes

Conversation

@nickclyde
Copy link
Copy Markdown
Member

@nickclyde nickclyde commented Apr 2, 2026

Summary

  • Update 5 S3 prefix defaults to match APHL AIMS eCR pipeline conventions (ValidationResponseV2/, TextToCodeSubmissionV2/, TTCAugmentationMetadataV2/, TTCMetadataV2/, AugmentationMetadataV2/)
  • Add source-bucket routing to TTC Lambda: extracts bucket name from S3 event with fallback to S3_BUCKET env var, enabling the training pipeline to share the same Lambda

Closes #374

Test plan

  • All TTC Lambda tests pass (7/7)
  • All Augmentation Lambda tests pass (7/7)
  • Grep confirms no remaining references to old prefix strings
  • Verify Terraform validates in CI
  • Confirm prefix names with APHL

Follow-up

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.93%. Comparing base (1a8e8dd) to head (19e5bdf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #419   +/-   ##
=======================================
  Coverage   93.93%   93.93%           
=======================================
  Files          42       42           
  Lines        2160     2161    +1     
=======================================
+ Hits         2029     2030    +1     
  Misses        131      131           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align S3 prefixes with APHL eCR pipeline

2 participants