Skip to content

test(web/blog): refresh visual baseline for new blog post#2487

Merged
bpamiri merged 1 commit intodevelopfrom
claude/cranky-kilby-cca0e7
May 8, 2026
Merged

test(web/blog): refresh visual baseline for new blog post#2487
bpamiri merged 1 commit intodevelopfrom
claude/cranky-kilby-cca0e7

Conversation

@bpamiri
Copy link
Copy Markdown
Collaborator

@bpamiri bpamiri commented May 8, 2026

Summary

  • The blog visual-regression canary at / (post index) failed on develop after the Security Hardening in Wheels 4.0 post landed — 45,727 pixels differ vs. a 200-pixel threshold.
  • Visually, the diff is purely the new post entry pushing the existing entries down. Header, typography, spacing, navigation, layout — all pixel-identical.
  • Baseline replaced with the CI runner's own blog.actual.png artifact (from run 25528406944) so renderer parity with the GitHub Actions Chromium environment is preserved.

Failing runs this fixes:

Test plan

  • CI Visual regression job passes on this PR
  • After merge: confirm the next push to develop is no longer blocked

The blog index canary in the visual-regression smoke test pins `/`,
which renders the post list. Adding the "Security Hardening in
Wheels 4.0" post in 290adef shifted the existing entries down,
producing 45,727 pixel diffs (threshold: 200) on every push to
develop since.

Layout, typography, header, and spacing are unchanged — the diff is
purely the new post entry. Baseline replaced with the CI runner's
own actual.png so renderer parity is preserved.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — straightforward visual baseline refresh for the new blog post entry.

Extended reasoning...

Overview

This PR updates a single binary asset, web/tests/visual-baselines/blog.png, used by the visual-regression test for the post index page. The baseline needed a refresh because commit 290adef ("Security Hardening in Wheels 4.0") added a new blog entry, shifting existing entries downward and producing 45,727 pixel diffs against a 200-pixel threshold.

Security risks

None. The file is a PNG used only as a comparison fixture in the visual regression test suite. It is not executed, served to users, or parsed by any production path.

Level of scrutiny

Minimal. This is a mechanical baseline update — a recognized maintenance pattern in visual-regression workflows. The author also took care to source the new baseline from the CI runner's own artifact to preserve renderer parity with the GitHub Actions Chromium environment, which is the correct approach.

Other factors

The PR description is clear about the cause, the scope, and the rationale. The only risk-free way to verify the new baseline is to let the visual-regression CI job run on this PR, which is already in the test plan.

@bpamiri bpamiri merged commit 68f8dab into develop May 8, 2026
9 checks passed
@bpamiri bpamiri deleted the claude/cranky-kilby-cca0e7 branch May 8, 2026 00: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