Skip to content

fix: filter Neptune query params#1568

Open
cnYui wants to merge 1 commit into
getzep:mainfrom
cnYui:codex/neptune-query-param-filter-followup-forkbase
Open

fix: filter Neptune query params#1568
cnYui wants to merge 1 commit into
getzep:mainfrom
cnYui:codex/neptune-query-param-filter-followup-forkbase

Conversation

@cnYui

@cnYui cnYui commented Jun 9, 2026

Copy link
Copy Markdown

Summary

  • Filter Neptune execute_query() kwargs down to non-None parameters actually referenced by the Cypher query after flattening nested params.
  • Keep referenced params such as LIMIT $limit, while dropping live-path extras like search_vector, min_score, database_, and routing_ before they reach Neptune openCypher.
  • Alias fallback community AOSS writes from communities to the Neptune community_name index.
  • Add focused NeptuneDriver regression coverage for both follow-up paths.

Context

This is a small follow-up for the live Neptune/AOSS path findings discussed in #1529 and #1539. It intentionally does not include the other #1539 fixes (clone() handling or AOSS bulk action _id changes), so it can stay focused on query parameter filtering and community index naming.

Testing

  • uv run --frozen --extra dev --extra neptune pytest tests/driver/test_neptune_driver.py -q
  • uv run --frozen --extra dev --extra neptune ruff check graphiti_core/driver/neptune_driver.py tests/driver/test_neptune_driver.py
  • uv run --frozen --extra dev --extra neptune ruff format --check graphiti_core/driver/neptune_driver.py tests/driver/test_neptune_driver.py
  • uv run --frozen --extra dev --extra neptune pyright graphiti_core/driver/neptune_driver.py

@zep-cla-assistant

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. For privacy information, see our Privacy Notice. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: [email protected]

or

I have read the CLA Document and I hereby sign the CLA behalf of my company, e-mail: [email protected]

Signature is valid for 6 months.


This bot will be retriggered when the Contributor License Agreement comment has been provided. Posted by the CLA Assistant Lite bot.

@cnYui

cnYui commented Jun 9, 2026

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: [email protected]

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