Skip to content

[codex] Harden pre-merge deployment readiness - #188

Merged
DannyDesert merged 2 commits into
mainfrom
codex/premerge-readiness-audit-20260504
May 4, 2026
Merged

DannyDesert merged 2 commits into
mainfrom
codex/premerge-readiness-audit-20260504

Conversation

@DannyDesert

Copy link
Copy Markdown
Member

Summary

  • add a public-release readiness regression test that blocks Troy-specific AWS/Discord/private-network IDs in public deploy/docs/prompt surfaces
  • remove active Strategist Builder/Deployer directive publications and update the objective workflow examples to route through Architect/AO-era ownership
  • make AO a first-class AWS Terraform service with private Cloud Map service discovery, Core -> AO URL wiring, AO -> Core callback URL wiring, AO storage/concurrency config, and fail-fast GitHub/AO auth checks
  • scrub deployment helper docs/scripts and Terraform comments/defaults so the product does not publish private deployment IDs
  • fix Terraform validation blockers in Secrets Manager for_each handling and S3 lifecycle configuration

Coverage against existing PRs

Verification

  • npx vitest run packages/core/tests/public-release-readiness.test.ts
  • npm run validate-configs --workspace=packages/core
  • terraform -chdir=deploy/aws validate
  • npm run typecheck
  • npm test
  • npm run build

Notes

  • terraform validate proves syntax/module wiring. The new GitHub/AO check blocks are plan-time guardrails and require real variable values during an AWS plan/apply.
  • The full test suite currently emits an existing Vitest warning about a nested vi.mock in packages/core/tests/discord-executor.test.ts; tests still pass.

@DannyDesert
DannyDesert marked this pull request as ready for review May 4, 2026 18:19
@DannyDesert
DannyDesert merged commit 383ddfa into main May 4, 2026
9 checks passed
@DannyDesert
DannyDesert deleted the codex/premerge-readiness-audit-20260504 branch May 4, 2026 19:51
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.

1 participant