Skip to content

fix: restore visual regression test baselines#2202

Merged
amikofalvy merged 1 commit intomainfrom
fix/restore-screenshot-baselines
Feb 20, 2026
Merged

fix: restore visual regression test baselines#2202
amikofalvy merged 1 commit intomainfrom
fix/restore-screenshot-baselines

Conversation

@amikofalvy
Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • @inkeep/agents-manage-ui#test passes in CI

🤖 Generated with Claude Code

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 20, 2026

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

Project Deployment Actions Updated (UTC)
agents-api Building Building Preview, Comment Feb 20, 2026 1:38am
agents-manage-ui Building Building Preview, Comment Feb 20, 2026 1:38am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
agents-docs Skipped Skipped Feb 20, 2026 1:38am

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 20, 2026

⚠️ No Changeset found

Latest commit: 0405e06

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@amikofalvy amikofalvy merged commit 6dd0e8d into main Feb 20, 2026
5 of 9 checks passed
@amikofalvy amikofalvy deleted the fix/restore-screenshot-baselines branch February 20, 2026 01:38
Copy link
Copy Markdown
Contributor

@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.

PR Review Summary

(0) Total Issues | Risk: Low

This is a straightforward infrastructure fix that restores accidentally deleted visual regression test baselines.

Verification Completed ✅

Check Status
Files match expected test baselines ✅ Verified — names correspond to tests in form.browser.test.tsx (lines 105, 115) and nodes.browser.test.tsx (line 77)
Deletion was accidental ✅ Verified — PR #2033 modified vitest.config.ts and these files were removed alongside unrelated changes
No code changes ✅ Confirmed — 0 insertions, 0 deletions, only binary PNG files
File naming convention ✅ Follows pattern: {TestSuite}-{test-name}-{screenshot-number}-{browser}.png

Files Restored

File Associated Test
Form-should-properly-highlight-error-state-1-chromium.png form.browser.test.tsx:105
Form-should-properly-highlight-nested-error-state-1-chromium.png form.browser.test.tsx:115
Nodes-should-handle-long-names-with-character-limit-1-chromium.png nodes.browser.test.tsx:77

✅ APPROVE

Summary: Clean restoration of test infrastructure. The visual regression test baselines were accidentally removed in PR #2033, and this PR correctly restores them. No concerns — ship it! 🚀

Reviewers (0)

No specialized reviewers dispatched — this PR contains only binary files with no code changes.

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