Skip to content

chore(release): staging to production - 2026.05.20#1083

Merged
ct3685 merged 1 commit into
productionfrom
staging
May 20, 2026
Merged

chore(release): staging to production - 2026.05.20#1083
ct3685 merged 1 commit into
productionfrom
staging

Conversation

@github-actions
Copy link
Copy Markdown

🚀 Release: Staging to Production

Release Date: 2026-05-20

Changes in this release


This PR is automatically created/updated when commits are pushed to staging.
Merging this PR will trigger the release workflow to create a new GitHub release.

…nd ingest util (#1082)

## Problem

Three more files had the same `?? ''` S3 credential bug fixed in #1079 —
passes empty strings when env vars are absent, breaking IAM/Copilot/ECS
deployments.

## Files changed

| File | Issue |
|------|-------|
| `packages-answers/scripts/generateCsv.ts` | Top-level `new S3` with
`?? ''` on both keys |
| `packages-answers/scripts/initCsvRun.ts` | Top-level `new S3` with `??
''` on both keys |
| `packages-answers/utils/src/ingest/document.ts` | `new S3Client` with
explicit credentials passed unconditionally |

## Fix

Same pattern as #1079 — only attach `credentials` when both env vars are
non-empty; otherwise omit and let the AWS SDK default credential chain
handle it (IAM, ECS task roles, Copilot, Render env vars, etc.).

## Test plan
- [ ] Verify CSV standalone scripts work in IAM-based env (no explicit
S3 keys)
- [ ] Verify document ingest works in IAM-based env
- [ ] Confirm Render (explicit env vars) still works as before
@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answerai-docs Ready Ready Preview May 20, 2026 4:25pm
the-answerai Ready Ready Preview May 20, 2026 4:25pm

Request Review

@ct3685 ct3685 merged commit 9b8838c into production May 20, 2026
8 checks passed
@maxtechera maxtechera temporarily deployed to staging - aai-unified2-flowise-moonstruck May 20, 2026 16:25 — with Render Inactive
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.

2 participants