Fix pixelmatch type error breaking CI builds

[amikofalvy]

Summary

• Fixed TypeScript type error in agents-manage-ui/vitest.config.ts:133-141 that was failing all three CI workflows (CI, Cypress, Publish) on main
• The pixelmatch function's third parameter accepts Uint8Array | undefined but was being passed Uint8Array | null — changed to use undefined for the call and ?? null for the return type which expects TypedArray | null

Test plan

• pnpm typecheck passes for agents-manage-ui
• CI, Cypress, and Publish workflows go green

🤖 Generated with Claude Code

[changeset-bot]

⚠️ No Changeset found

Latest commit: dc6071e

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

[vercel]

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

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
agents-docs	Skipped		Feb 20, 2026 1:21am

[claude]

PR Review Summary

(0) Total Issues | Risk: Low

This is a clean, targeted fix for a TypeScript type error in the vitest screenshot comparator configuration. No issues identified.

Analysis

The Problem:
The pixelmatch library's third parameter (output) accepts Uint8Array | undefined, but the code was passing null. This caused TypeScript to fail type checking, breaking CI.

The Fix:

1. Line 133: Changed null → undefined for the diffBuf variable when no diff image is needed, matching pixelmatch's expected parameter type
2. Line 141: Added ?? null to coerce undefined back to null for the return value's diff property, matching vitest's expected return type (TypedArray | null)

Verification:

• The fix is type-correct: undefined satisfies pixelmatch's parameter type, and ?? null produces the null that vitest expects
• Consistent with existing code: the early return at line 116 already uses diff: null
• No behavioral change: the runtime behavior is identical (pixelmatch treats undefined and null equivalently for the output parameter)

Scope

• Surface: Internal test infrastructure only (vitest config)
• Customer impact: None
• Risk: Minimal — type-only change with no runtime behavior difference

---

✅ APPROVE

Summary: Clean type fix that unblocks CI. The change correctly adapts to pixelmatch's type signature while preserving vitest's expected return contract. Ship it! 🚀

Reviewers (1)

Reviewer	Returned	Main Findings	Consider	While You're Here	Inline Comments	Pending Recs	Discarded
orchestrator (direct)	0	0	0	0	0	0	0
Total	0	0	0	0	0	0	0

Note: Subagent dispatch was skipped due to minimal scope (2-line type fix in test config).