Skip to content

chore(e2e): review screenshots for #2384 - #2387

Closed
copse-release-bot[bot] wants to merge 2 commits into
codex/fix-linked-worktree-startupfrom
screenshots/pr-2384/1abfa913fa00
Closed

copse-release-bot[bot] wants to merge 2 commits into
codex/fix-linked-worktree-startupfrom
screenshots/pr-2384/1abfa913fa00

Conversation

@copse-release-bot

Copy link
Copy Markdown
Contributor

Screenshot candidates rendered for parent PR
#2384 at
1abfa913fa00ac4771bcd6c110ba082c3ac7eb5e by
CI run 33989401958.

Review GitHub's image diffs, then merge this PR (or enable auto-merge) to apply the
accepted references to codex/fix-linked-worktree-startup. This branch contains
only PNG candidates from the immutable
reference-screenshot-candidates-33989401958 artifact and never targets main.

If the parent branch has advanced beyond the source SHA above, do not merge this PR;
the successful CI run for the new head will replace it.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

🖥️ PR preview

@jonathanKingston

Copy link
Copy Markdown
Collaborator

Reviewed while validating parent #2384. The parent changes main-process startup/sandbox metadata only and has no intended DOM or visual output change; its focused Electron run also preserved the checked-in reference locally. These eight screenshot updates are therefore unrelated capture/baseline churn and should not be merged into the parent.

@jonathanKingston
jonathanKingston force-pushed the codex/fix-linked-worktree-startup branch from 1abfa91 to 520c593 Compare September 6, 2026 09:19

Copy link
Copy Markdown
Collaborator

Closing: these candidates predate #2398, which pinned every reference capture at a 2x device-pixel ratio. All 8 PNGs here were rendered at 1x (mcp-tool-labels.png is 1200x800; main's is 2400x1600), so merging this would push 1x baselines back onto codex/fix-linked-worktree-startup and undo the normalization on main.

Parent #2384 has been rebased onto current main, so its head has moved past the 1abfa913fa00 this branch is pinned to. CI on the new head re-renders the references at the pinned 2x and publishes a fresh review PR.


Generated by Claude Code

This branch was previously deployed

1 inactive deployment
github-pages — 8126b78c Deployed Sep 5, 2026 by copse-release-bot[bot] via deploy / deploy #3579
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