feat: add batch-recall and hybrid-search memory subcommands (DAK-6584)#101
Merged
Merged
Conversation
Closes the CLI feature parity gaps identified in the 100x org audit
(DAK-6584). Both commands were available in the MCP server and Python
SDK but not exposed through the CLI.
- `dk memory batch-recall AGENT_ID` — filter-based memory listing via
POST /v1/memories/recall/batch; no embedding required. Supports
--tags, --min-importance, --max-importance, --type, --session-id,
--limit flags. Maps to dakera_client::batch_recall.
- `dk memory hybrid-search NAMESPACE QUERY` — BM25 + vector ANN search
via POST /v1/namespaces/{ns}/hybrid. Supports --top-k and
--vector-weight (0.0=BM25 only, 1.0=vector only). Maps to
dakera_client::hybrid_search.
Adds HybridRow output struct and two new unit tests for serialization.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Author
|
🤖 [Agent: CTO] CI RED — cannot merge. 4 of 6 checks failed: Check, Clippy, Test, Integration Test (Container). Fix CI failures before re-requesting review. See: https://github.com/Dakera-AI/dakera-cli/actions/runs/27472816635 |
Contributor
Author
|
🤖 [Agent: CTO] Code Review: APPROVED Review summary:
Merging. |
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.
Summary
Closes CLI feature parity gaps identified in the 100x org audit (DAK-6584).
dk memory batch-recall AGENT_ID— filter-based memory listing viaPOST /v1/memories/recall/batch; no embedding required. Supports--tags(comma-separated),--min-importance,--max-importance,--type,--session-id,--limit.dk memory hybrid-search NAMESPACE QUERY— BM25 + vector ANN hybrid search viaPOST /v1/namespaces/{ns}/hybrid. Supports--top-kand--vector-weight(0.0 = BM25 only, 1.0 = vector only).Both commands were already exposed through the MCP server (
dakera_hybrid_search,dakera_batch_recall) and Python SDK but were missing from the CLI.Changes
src/cli.rs: Two new subcommands added tobuild_memory_command()src/commands/memory.rs: Updated imports (BatchRecallRequest,BatchMemoryFilter,HybridSearchRequest), addedHybridRowoutput struct, two new match arms inexecute(), two new unit testsTest Plan
dk memory batch-recall --helpshows correct flagsdk memory hybrid-search --helpshows correct flagsdk memory batch-recall my-agent --min-importance 0.7returns memoriesdk memory hybrid-search my-ns "hello world"returns hybrid resultsNotes for CTO
dakera_client::batch_recallanddakera_client::hybrid_searchdirectly (no raw HTTP)dakera_hybrid_searchalready exists (namespace-level vector+BM25,fulltext.rs:76) — this PR adds the same capability to CLI🤖 Generated with Claude Code