Skip to content

ci: dispatch instanode-web auth-contract Playwright on api PR#204

Merged
mastermanas805 merged 1 commit into
masterfrom
ci/auth-contract-e2e-dispatch
May 30, 2026
Merged

ci: dispatch instanode-web auth-contract Playwright on api PR#204
mastermanas805 merged 1 commit into
masterfrom
ci/auth-contract-e2e-dispatch

Conversation

@mastermanas805

Copy link
Copy Markdown
Member

Cross-repo Layer-1 trigger paired with instanode-web #153.

Why

The api owns the CORS allowlist + /auth/exchange + /auth/email/start. An api-side change that drops access-control-allow-credentials wouldn't trigger instanode-web's CI on its own, so today's regression could ship despite green api unit tests.

What

New dispatch-auth-contract-e2e job in ci.yml that fires repository_dispatch on instanode-web (type auth-contract-e2e-from-api). The web side already has the listener (PR #153).

Safety

  • Only fires on pull_request or push to master.
  • All untrusted github.event.* values go through env: (security-guidance compliant).
  • PR_NUMBER is validated as numeric before JSON payload construction.
  • Soft-skips if REPO_ACCESS_TOKEN secret missing → api CI stays green during rollout.

Coverage (rule 17)

  • Symptom: api-side CORS / auth-contract regression ships without instanode-web's Playwright firing
  • Sites found: 1 (ci.yml)
  • Live verified: YAML parses; dispatch step will fire on this very PR

Cross-repo Layer-1 trigger. The api owns the CORS allowlist + /auth/exchange
+ /auth/email/start endpoints — an api-side change that drops
access-control-allow-credentials wouldn't trigger instanode-web's CI on its
own, so today's 2026-05-29 → 2026-05-30 login regression could ship
despite green api unit tests.

This job fires a repository_dispatch on instanode-web; it listens for
'auth-contract-e2e-from-api' and runs the Chromium smoke against prod
(see instanode-web PR #153). Soft-skips if REPO_ACCESS_TOKEN secret is
missing so api CI stays green during initial rollout.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mastermanas805 mastermanas805 merged commit 66f4d85 into master May 30, 2026
15 checks passed
@mastermanas805 mastermanas805 deleted the ci/auth-contract-e2e-dispatch branch May 30, 2026 15:48
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