Add semantic task retrieval with scoped embeddings and RAG context - #677
Merged
Conversation
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.
Merge decision: ready — Von can retrieve related tasks by meaning through its existing task-search tool and API, with current visibility checks, citation context and explicit lexical fallback.
User outcome
Native task
#V#task_agent_9a8deead9e6cb572a422aa411a7c2897requests semantic and embedding indexing of tasks with RAG support.search_mode=semanticfinds related task documents while retaining existing filters and lexical search as the default.Material changes
Evidence
116 targeted tests pass:
The semantic suite covers the real task service, canonical repository visibility filters, SQLite cache, Flask route and MCP handler with isolated mock storage and a fixture embedder: paraphrase ranking, filters, pagination, current text after edits, model-version changes, corruption, partial-batch recovery, actor/organisation/public boundaries, revocation, deletion and deletion during embedding. Syntax compilation and
git diff --checkpass.A broader test,
test_get_task_route_adds_local_hrefs_for_external_resource_actions, returns 404 instead of 200. It reproduces on an untouched worktree at starting revisiond9ac13e09; it is outside semantic search and does not change the merge decision.Ship boundary
No secrets or runtime data committed; the JSON fixture contains synthetic tasks only.