Repair post-merge CI, search, and repository hardening#6
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the useVideoSearch hook to accept a syncToStore option, allowing multiple observers to use the hook without duplicating state syncs in the Zustand store. It also introduces deduplication of search results by identifier when appending pages. The review feedback highlights two important improvements: adding a cleanup function to the loading state effect to prevent the global loading state from getting stuck on unmount, and introducing defensive checks when mapping search results to avoid potential runtime crashes from malformed API data.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Addresses PR review feedback on use-video-search: - reset the global loading flag when the syncing observer unmounts so the UI can't get stuck loading if a request is still in flight - filter out docs and stored results without identifiers before keying the dedupe map, protecting the render tree from malformed payloads Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 participant
Summary
Repairs the post-merge regression and hardens the repository in a reviewable sequence.
@vercel/nodedependency and upgrade to Vite 7Verification
npm ciwith Node 20.20.2 / npm 10.8.2npm run checknpm run build(no CSS parser or stale Browserslist warnings)drizzle-kit checknpm audit: 0 vulnerabilitiesnpm audit: 4 moderate, dev-only findings, all from the deprecated esbuild loader bundled by the latest stabledrizzle-kit; npm's proposed fix is an unsafe downgrade/api/searchreturns a real resultPreview
Protected exact-HEAD preview: https://staticbuffet-50crmia6r-fladry-creative.vercel.app