ENG-1953: Remove duplicate node alert feature#1201
Conversation
Removes VectorDuplicateMatches, its observer wiring, the feature flag (settings schema, keys, admin panel), and the dead vector-only search wrapper in hyde.ts. The match_content_embeddings RPC stays: it is still used by discourseNodeSearchProviders.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
PR size/scope checkThis PR is over our review-size guideline.
Please split this into smaller PRs unless there is a clear reason the changes need to land together. If keeping it as one PR, please add a brief justification covering:
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
The FEATURE_FLAG_KEYS gap for "Advanced node search enabled" is real but pre-existing (as the review notes) and unrelated to this removal, so it's not fixed here — filed ENG-2001 to track it. |
1 participant
Removes the duplicate-node alert on DG pages end to end: the VectorDuplicateMatches component, its page-title observer wiring, the feature flag (schema, setting keys, admin panel), and the dead vector-only search wrapper in hyde.ts. The match_content_embeddings Supabase RPC stays - it is still used by discourseNodeSearchProviders. isSyncEnabled() now depends only on the suggestive mode overlay flag.
Fixes ENG-1953