Skip to content

[codex] Restore production smoke spec#386

Merged
blove merged 1 commit into
mainfrom
codex/fix-production-smoke-spec
May 17, 2026
Merged

[codex] Restore production smoke spec#386
blove merged 1 commit into
mainfrom
codex/fix-production-smoke-spec

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 17, 2026

Summary

Restores the standalone Playwright production smoke spec at apps/cockpit/e2e/production-smoke.spec.ts, which is the path invoked by the post-deploy production smoke job.

Root cause

The e2e harness cleanup removed apps/cockpit/e2e, but .github/workflows/ci.yml still invokes npx playwright test apps/cockpit/e2e/production-smoke.spec.ts --reporter=list. On push to main, that job would fail before exercising the deployed stack because the spec file is missing.

Validation

  • git diff --check
  • Confirmed .github/workflows/ci.yml references apps/cockpit/e2e/production-smoke.spec.ts
  • Confirmed the restored spec exists at that path

Full Playwright execution was not run locally because this worktree has no node_modules, and npm/npx are not available in the command environment.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 2026

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

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 17, 2026 2:59am

Request Review

@blove blove enabled auto-merge (squash) May 17, 2026 02:57
@blove blove merged commit 6ab763d into main May 17, 2026
14 checks passed
@blove blove deleted the codex/fix-production-smoke-spec branch May 17, 2026 14:50
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