Skip to content

test(snapshot): verify compatibility with published 1.0.1 snapshots - #43

Open
codewarnab wants to merge 1 commit into
mainfrom
test/phase-2-golden-snapshot
Open

codewarnab wants to merge 1 commit into
mainfrom
test/phase-2-golden-snapshot

Conversation

@codewarnab

Copy link
Copy Markdown
Owner

Snapshots written by an older release were covered only by tests that generated data at runtime. A coordinated writer/reader change could therefore pass those tests while breaking saved snapshots.

Add a 1,465-byte fixture written by the published webgpu-search@1.0.1 package, with npm provenance, a SHA-256 checksum, fixed input documents, and nine frozen query expectations. Extend the existing snapshot suite to restore those bytes and check document contents, IDs, ordering, scores, and matched fields. CI already runs the suite. Document fixture preservation and record the coverage in the improvement plan; include a patch changeset.

Validation: shader checks, typecheck, build, mock tests, and snapshot tests passed. An independent subagent reproduced identical fixture bytes and query expectations using the released package. A negative control changing a frozen expected score failed in the new historical compatibility check. No runtime library code changed.

Copilot AI balanced review requested due to automatic review settings October 18, 2026 19:28
@vercel

vercel Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
webgpu-search Ignored Ignored Oct 18, 2026 7:28pm UTC

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch has not been deployed

No deployments
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.

2 participants