Normalize 'under' path handling and add remote embedding support#209
Normalize 'under' path handling and add remote embedding support#209
Conversation
Refactors all uses of the 'under' parameter to normalize as a suffix and use MatchText for substring matching, improving path filtering consistency. Adds support for remote embedding services in memory.py and mcp_memory_server.py, routing embedding calls to remote or local as appropriate. Updates documentation and skill usage tips to reflect new search strategies, parallel execution, and token efficiency defaults.
🤖 Augment PR SummarySummary: This PR standardizes how the Changes:
Technical Notes: Remote embedding is performed via the existing 🤖 Was this summary useful? React with 👍 or 👎 |
| norm_under = "/".join([p for p in norm_under.split("/") if p]) | ||
| if norm_under: | ||
| norm_under = "/" + norm_under | ||
| must.append(models.FieldCondition(key="metadata.path_prefix", match=models.MatchText(text=norm_under))) |
There was a problem hiding this comment.
metadata.path_prefix filtering is now using MatchText, but the indexer currently creates a KEYWORD payload index for metadata.path_prefix (see ensure_payload_indexes in scripts/ingest/qdrant.py), which may cause Qdrant to reject the filter or change matching semantics. Consider aligning the payload schema/index type (and any migration path) so under filtering stays reliable.
Other Locations
scripts/hybrid_search.py:1111scripts/hybrid/expand.py:811scripts/rerank_tools/local.py:372scripts/rerank_tools/local.py:453
🤖 Was this useful? React with 👍 or 👎
Refactors all uses of the 'under' parameter to normalize as a suffix and use MatchText for substring matching, improving path filtering consistency. Adds support for remote embedding services in memory.py and mcp_memory_server.py, routing embedding calls to remote or local as appropriate. Updates documentation and skill usage tips to reflect new search strategies, parallel execution, and token efficiency defaults.