Skip to content

avoid mutating shared search config#1655

Open
adolphinvx wants to merge 2 commits into
getzep:mainfrom
adolphinvx:codex/fix-search-config-mutation
Open

avoid mutating shared search config#1655
adolphinvx wants to merge 2 commits into
getzep:mainfrom
adolphinvx:codex/fix-search-config-mutation

Conversation

@adolphinvx

Copy link
Copy Markdown

Summary

  • copy the shared edge search recipe before applying the per-request result limit
  • prevent Graphiti.search() calls from mutating module-level search configuration
  • add regression coverage for searches with and without a center node

Testing

  • pytest tests/test_graphiti_search.py -q
  • make lint
  • unit test suite: 341 passed, 11 skipped

API / schema impact

None.

@zep-cla-assistant

zep-cla-assistant Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@adolphinvx
adolphinvx force-pushed the codex/fix-search-config-mutation branch from 7b421e1 to bc1b28d Compare July 15, 2026 09:32
@adolphinvx

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: exarunner@gmail.com

zep-cla-assistant Bot added a commit that referenced this pull request Jul 16, 2026
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