Skip to content

[codex] Fix scheduled post-deploy verifier#514

Closed
HomenShum wants to merge 1 commit into
mainfrom
codex/fix-post-deploy-verify
Closed

[codex] Fix scheduled post-deploy verifier#514
HomenShum wants to merge 1 commit into
mainfrom
codex/fix-post-deploy-verify

Conversation

@HomenShum

Copy link
Copy Markdown
Owner

Summary

  • Update the live workspace smoke assertion to match the current deployed workspace shell.
  • Keep the scheduled Post-Deploy Verify signal focused on hydration, standalone workspace routing, selected tab state, and live-shell content.

Root cause

Scheduled Post-Deploy Verify was consistently red because tests/e2e/live-smoke.spec.ts still expected obsolete workspace copy: a heading named Workspace draft and text No fixture fallback. Production now renders the standalone workspace route with the requested report id as the h1 (acme-ai) and live shell content (Live artifact, Report shell loaded). The app route was healthy; the smoke assertion was stale.

Verification

  • npx playwright test tests/e2e/live-smoke.spec.ts --project=chromium --workers=1 --grep workspace/w --reporter=list
  • npm run post-deploy:verify:json

@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
nodebench-ai Ignored Ignored Jul 5, 2026 5:19am

Request Review

@HomenShum

Copy link
Copy Markdown
Owner Author

Superseded by #515, which uses the repo's required fix/... branch and Conventional Commits PR title.

@HomenShum HomenShum closed this Jul 5, 2026
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

✅ Dogfood Visual QA Gate: PASSED

Check Status
Screenshots 23 captured (pass)
Walkthrough 9 chapters (pass)
Key Frames 9 extracted (pass)
Scribe Steps 8 how-to steps (pass)
Build success
Artifacts

Download the dogfood-evidence-57643e3 artifact from the Actions tab for full screenshots, frames, and walkthrough video.


Generated by Dogfood QA Gate

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.

2 participants