Skip to content

Fix packaging metadata and add a non-embedding recall path#1

Open
0xCyberBerserker wants to merge 1 commit intoSabaTech-dev:mainfrom
0xCyberBerserker:codex/fix-packaging-and-recall
Open

Fix packaging metadata and add a non-embedding recall path#1
0xCyberBerserker wants to merge 1 commit intoSabaTech-dev:mainfrom
0xCyberBerserker:codex/fix-packaging-and-recall

Conversation

@0xCyberBerserker
Copy link
Copy Markdown

Summary

This fixes a couple of rough edges in the current package layout and recall flow.

  • make the package installable with current setuptools by declaring the exported package explicitly
  • clean up deprecated metadata/config usage in pyproject.toml and Pydantic models
  • add a lexical fallback in MemoryBank.recall() when no query embedding is provided
  • align the README examples with the current public API
  • add a unit test covering the non-embedding recall path

Why

At the moment the project fails to install through pip install . because setuptools sees both sql and xmemory at the top level and refuses automatic discovery.

Also, recall() only returns results when embeddings are passed in, which makes the basic API harder to use during early integration or local testing. A simple text/context fallback keeps the method useful before an embedding pipeline is wired in.

The README also had a few examples that no longer match the constructor or the stats model fields.

Validation

  • python -m pytest -q

Current result locally:

  • 3 passed, 14 skipped

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