Skip to content

feat: --enrich CLI flag for cross-entity recall (#689 PR4)#704

Merged
ajianaz merged 1 commit into
developfrom
feat/cli-enrich-flag
Jul 16, 2026
Merged

feat: --enrich CLI flag for cross-entity recall (#689 PR4)#704
ajianaz merged 1 commit into
developfrom
feat/cli-enrich-flag

Conversation

@ajianaz

@ajianaz ajianaz commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Wire up the enrich parameter added by PR702 so users can opt into cross-entity link resolution via uteke recall --enrich.

Changes

  • cli.rs: Add --enrich bool flag to the Recall subcommand
  • commands/mod.rs: Thread enrich from CLI dispatch to run_recall()
  • commands/recall.rs: Pass enrich to recall_unified() instead of hardcoded false
  • output.rs: Display cross-entity links in human output:
    • Memory results: πŸ“„ Docs: <slugs> when linked_doc_slugs is populated
    • Document results: πŸ”— Memories: <count> when linked_memory_ids is populated

MCP

Left enrich=false for MCP recall β€” MCP enrichment can be a separate enhancement with its own tool schema update.

References

Closes part of #689 (PR4 of the enrichment series).

@github-actions

Copy link
Copy Markdown

πŸ” Cora AI Code Review

βœ… No issues found. Code looks good!


Review powered by cora-cli Β· BYOK Β· MIT

Wire up the enrich parameter added by PR702 so users can opt into
cross-entity link resolution via uteke recall --enrich.

- Add --enrich bool flag to Recall subcommand in cli.rs
- Thread enrich from CLI dispatch β†’ run_recall β†’ recall_unified()
- Display linked_doc_slugs (πŸ“„ Docs) on memory results in human output
- Display linked_memory_ids count (πŸ”— Memories) on document results
- MCP recall left at enrich=false for now (separate enhancement)
@ajianaz
ajianaz force-pushed the feat/cli-enrich-flag branch from 5bf7fc1 to d34269f Compare July 16, 2026 11:56
@ajianaz
ajianaz merged commit 20078ab into develop Jul 16, 2026
8 checks passed
@ajianaz
ajianaz deleted the feat/cli-enrich-flag branch July 16, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant