Skip to content

Match recall scopes by complete path components - #33

Open
importcpp wants to merge 1 commit into
tigerless-labs:mainfrom
importcpp:fix/recall-scope-prefix
Open

importcpp wants to merge 1 commit into
tigerless-labs:mainfrom
importcpp:fix/recall-scope-prefix

Conversation

@importcpp

Copy link
Copy Markdown

Summary

  • compare recall scopes against complete path components
  • prevent a scope such as experience from matching a sibling domain such as experience-archive
  • add a regression test for the colliding-prefix case

Closes #17

Testing

  • uv run pytest -q --cov=agent_memory --cov-report=term-missing --cov-fail-under=85 (389 passed, 90.91% coverage)
  • uv run ruff check .
  • uv run mypy

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.

Recall scope filter matches path prefixes without a directory separator

1 participant