Skip to content

fix(ci): preserve visual reports across tiers - #4672

Open
hellofrequencylab wants to merge 3 commits into
mainfrom
codex/visual-artifact-retention-20261104
Open

hellofrequencylab wants to merge 3 commits into
mainfrom
codex/visual-artifact-retention-20261104

Conversation

@hellofrequencylab

@hellofrequencylab hellofrequencylab commented Nov 5, 2026 •

Copy link
Copy Markdown
Owner

The final visual upload previously kept only the advisory tier because each invocation cleared the preceding tier's result and HTML directories. Give public, member and advisory runs independent directories so all three visual report sets survive.

Daniel approved exact workflow patch SHA256 05b29b302606bea428c260a5e22b617a28c4dbd95767ee04d72eaaea9aa0c7f4. The workflow matches that reviewed candidate byte for byte: six YAML leaves. Upload roots, selectors, secrets, timeouts, thresholds, baselines and blocking/advisory semantics are unchanged. The accompanying OPEN finding is assigned to W7.

Validation:

  • Workflow contract, one-list, static backlog and 193 existing focused tests pass.
  • Actual exact-head run 41962040036 uploaded artifact13387323630. Its decoded reports retain 60 public, 52 member and four advisory failures, with all 1,535 attachment references present in the archive. Settings actual/expected/diff images are recovered. Selected entries were CRC-verified through bounded range reads.
  • The initial full-test failure identified the new row's missing wave placement. Metadata-only commit15fde7947 fixes placement; its fresh exact checks are required before merge. No tests or probes changed.

The original visual and Lighthouse failures remain evidence; this PR does not claim an application visual or performance pass. The root HTML report contains the production-contract suite: that earlier invocation still overwrites smoke HTML. This inherited smoke-retention gap keeps the finding OPEN; separate two-leaf follow-up cbf1ca61 is external and unapplied pending exact approval.

No application code, baselines, SQL or operator procedures changed. #4663/#4669 and their original runs remain untouched.

@vercel

vercel Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
frequency-web Ready Ready Preview Nov 5, 2026 6:22am UTC

Request Review

This branch was successfully deployed

1 active deployment
Preview — 03eae7ea Deployed Nov 5, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants