Expand the searchable Verified Feed experience#47
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
origin/mainat424a865without editing their analyzer-owned componentsTDD evidence
First RED:
npx vitest run src/components/feed/feed-explorer.test.tsxfailed because the publicFeedExplorermodule did not exist.Subsequent vertical RED/GREEN slices covered category/reset behavior, coherent 6-10 fixture/category coverage, no-match/stale/unavailable states, and the persisted UUID contract. The live seed first failed safely because readable fixture IDs did not satisfy the Supabase UUID column; deterministic UUID fixture identities made the same runtime contract persistable.
Focused results:
e2e/feed.spec.ts: 6 passedFinal Node 22.19.0 gate
npm run lint— passednpm run typecheck— passednpm run test:unit— 318 passed, 6 credential-gated skipsnpm run build— passednpm run test:e2e— 38 passed, 2 expected skips across desktop/mobile ChromiumFixture and browser verification
yt-dlpmetadata before inclusion; unverified candidates were excluded.dev-personalSupabase catalog was seeded with all eight items. A second seed updated the same rows, confirming idempotency.Closes #29