Skip to content

Align EC2 smoke with context-free login - #204

Merged
OziinG merged 1 commit into
mainfrom
codex/fix/direct-login-deploy-smoke
Aug 19, 2026
Merged

Align EC2 smoke with context-free login#204
OziinG merged 1 commit into
mainfrom
codex/fix/direct-login-deploy-smoke

Conversation

@OziinG

@OziinG OziinG commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Closes #202

What changed

  • updates the EC2 login smoke to require HTTP 200, the Shopify API key meta, and the missing-context guidance
  • requires App Bridge to be absent from that context-free document
  • adds a deployment-runtime regression test

Why

Run 32264910035 deployed the fixed K-food HTML successfully, but the legacy smoke predicate rejected it because it still required App Bridge on the direct login page.

Validation

  • deployment runtime tests 4/4
  • diff check
  • the failed run body showed the expected new HTML without App Bridge

Constraint: The public smoke URL intentionally renders guidance without App Bridge.

Rejected: Restore App Bridge on direct login | reintroduces the browser errors fixed by PR #203

Confidence: high

Scope-risk: narrow

Directive: Validate the direct-login contract, not embedded-only markup, in EC2 smoke tests.

Tested: deployment runtime regression tests; diff check.

Not-tested: GitHub Actions EC2 rollout; verified after merge.
@OziinG
OziinG merged commit ae1c007 into main Aug 19, 2026
2 checks passed
@OziinG
OziinG deleted the codex/fix/direct-login-deploy-smoke branch August 19, 2026 14:41
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.

Avoid App Bridge errors on direct login smoke

1 participant