Source: PR #60 review (Important, non-blocking).
Duplicate hiding works across FTS5, recall semantic, recall hybrid, and the MCP vector path via one shared SQL fragment — but only the FTS5 path is test-covered. A regression in the shared fragment's application to the other three paths would ship silently.
Fix sketch: one test per path asserting a marked duplicate is hidden by default and revealed with --include-duplicates (or MCP equivalent). Overlaps usefully with #67's hybrid fix.
Source: PR #60 review (Important, non-blocking).
Duplicate hiding works across FTS5,
recall semantic,recall hybrid, and the MCP vector path via one shared SQL fragment — but only the FTS5 path is test-covered. A regression in the shared fragment's application to the other three paths would ship silently.Fix sketch: one test per path asserting a marked duplicate is hidden by default and revealed with
--include-duplicates(or MCP equivalent). Overlaps usefully with #67's hybrid fix.