Summary
pyproject.toml declares an optional dependency group:
memory = [
"neo4j>=5.20",
]Nothing in the package imports neo4j. pip install 'grapharc[memory]' therefore pulls a driver and gives the user no additional capability. The durable claim stores that do exist are SQLiteMemoryStore (no extra needed) and LadybugMemoryStore (the separate ladybug extra).
Why this matters
An extras table is a promise about what you get for the download. Honest dependency boundaries are part of the graph engineering case this project makes — every subsystem should be reachable from a shipped command, and an extra that installs a driver with no code behind it is the opposite of that. README already lists a "no implementation behind it" note; this issue is to make the packaging match reality.
Where in the code
pyproject.toml — the memory extra, around line 90grapharc/memory/ — the stores that actually exist (sqlite_store.py, ladybug_store.py)README.md — the Memory section's closing note about this extra
Confirm it:
grep -n -A3 "^memory" pyproject.toml
grep -rn "import neo4j\|from neo4j" grapharc/ # no hits
What to change
Pick one of these and say which in your PR:
- (a) Remove it. Delete the
memory extra, remove it from any all aggregation, and update the README sentence that mentions it. This is the smaller, safer change and is a fine first contribution.
- (b) Rename it. If the intent was "the durable stores", point the name at something real rather than a driver.
Do not implement a Neo4j store as part of this issue — that is a much larger piece of work and should be proposed separately.
How to verify
grep -rn "memory" pyproject.toml
uv sync --all-extras --group dev # must still resolve
uv run pytest -q
uv run ruff check .
Acceptance criteria
Skill level — good first issue
This is mostly a packaging and documentation change, with no runtime logic involved. It is a good way to learn how this repo keeps its install story honest. If you are unsure whether to remove or rename, comment here first and we will settle it before you open a PR.
Summary
pyproject.tomldeclares an optional dependency group:Nothing in the package imports
neo4j.pip install 'grapharc[memory]'therefore pulls a driver and gives the user no additional capability. The durable claim stores that do exist areSQLiteMemoryStore(no extra needed) andLadybugMemoryStore(the separateladybugextra).Why this matters
An extras table is a promise about what you get for the download. Honest dependency boundaries are part of the graph engineering case this project makes — every subsystem should be reachable from a shipped command, and an extra that installs a driver with no code behind it is the opposite of that. README already lists a "no implementation behind it" note; this issue is to make the packaging match reality.
Where in the code
pyproject.toml— thememoryextra, around line 90grapharc/memory/— the stores that actually exist (sqlite_store.py,ladybug_store.py)README.md— the Memory section's closing note about this extraConfirm it:
What to change
Pick one of these and say which in your PR:
memoryextra, remove it from anyallaggregation, and update the README sentence that mentions it. This is the smaller, safer change and is a fine first contribution.Do not implement a Neo4j store as part of this issue — that is a much larger piece of work and should be proposed separately.
How to verify
Acceptance criteria
uv sync --all-extras --group devstill resolves cleanlypyproject.tomluv run pytestgreenSkill level — good first issue
This is mostly a packaging and documentation change, with no runtime logic involved. It is a good way to learn how this repo keeps its install story honest. If you are unsure whether to remove or rename, comment here first and we will settle it before you open a PR.