pgvector-haystack 6.6.1__tar.gz → 7.0.0__tar.gz

{pgvector_haystack-6.6.1 → pgvector_haystack-7.0.0}/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [integrations/pgvector-v6.6.1] - 2026-09-25
+
+### 🐛 Bug Fixes
+
+- *(pgvector)* Raise FilterError on empty logical conditions (#3946)
+- *(pgvector)* Count every written document in write_documents_async (#3964)
+
+### ⚙️ CI
+
+- Improve changelog generation; fix existing changelogs (#3883)
+
+
 ## [integrations/pgvector-v6.6.0] - 2026-08-26
 
 ### 🐛 Bug Fixes

{pgvector_haystack-6.6.1 → pgvector_haystack-7.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pgvector-haystack
-Version: 6.6.1
+Version: 7.0.0
 Summary: An integration of pgvector (vector search extension for Postgres) with Haystack
 Project-URL: Source, https://github.com/deepset-ai/haystack-core-integrations
 Project-URL: Documentation, https://github.com/deepset-ai/haystack-core-integrations/blob/main/integrations/pgvector/README.md

{pgvector_haystack-6.6.1 → pgvector_haystack-7.0.0}/src/haystack_integrations/document_stores/pgvector/document_store.py

@@ -9,6 +9,7 @@ from haystack import default_from_dict, default_to_dict, logging
 from haystack.dataclasses.document import Document
 from haystack.document_stores.errors import DocumentStoreError, DuplicateDocumentError
 from haystack.document_stores.types import DuplicatePolicy
+from haystack.errors import FilterError
 from haystack.utils.auth import Secret, deserialize_secrets_inplace
 from psycopg import AsyncConnection, Connection, Cursor, Error, IntegrityError
 from psycopg.cursor_async import AsyncCursor
@@ -1078,18 +1079,21 @@ class PgvectorDocumentStore:
         :param filters: The filters to apply to select documents for deletion.
             For filter syntax, see [Haystack metadata filtering](https://docs.haystack.deepset.ai/docs/metadata-filtering)
         :returns: The number of documents deleted.
+        :raises FilterError: If `filters` is empty. Use `delete_all_documents()` to delete everything.
         """
         _validate_filters(filters)
+        if not filters:
+            # An empty filter would compile to an unqualified DELETE and empty the table
+            msg = "delete_by_filter requires a non-empty filter. Use delete_all_documents() to delete every document."
+            raise FilterError(msg)
 
         delete_sql = SQL("DELETE FROM {schema_name}.{table_name}").format(
             schema_name=Identifier(self.schema_name),
             table_name=Identifier(self.table_name),
         )
 
-        params = ()
-        if filters:
-            sql_where_clause, params = _convert_filters_to_where_clause_and_params(filters)
-            delete_sql += sql_where_clause
+        sql_where_clause, params = _convert_filters_to_where_clause_and_params(filters)
+        delete_sql += sql_where_clause
 
         self._ensure_db_setup()
         assert self._cursor is not None
@@ -1120,18 +1124,24 @@ class PgvectorDocumentStore:
         :param filters: The filters to apply to select documents for deletion.
             For filter syntax, see [Haystack metadata filtering](https://docs.haystack.deepset.ai/docs/metadata-filtering)
         :returns: The number of documents deleted.
+        :raises FilterError: If `filters` is empty. Use `delete_all_documents_async()` to delete everything.
         """
         _validate_filters(filters)
+        if not filters:
+            # An empty filter would compile to an unqualified DELETE and empty the table
+            msg = (
+                "delete_by_filter_async requires a non-empty filter. "
+                "Use delete_all_documents_async() to delete every document."
+            )
+            raise FilterError(msg)
 
         delete_sql = SQL("DELETE FROM {schema_name}.{table_name}").format(
             schema_name=Identifier(self.schema_name),
             table_name=Identifier(self.table_name),
         )
 
-        params = ()
-        if filters:
-            sql_where_clause, params = _convert_filters_to_where_clause_and_params(filters)
-            delete_sql += sql_where_clause
+        sql_where_clause, params = _convert_filters_to_where_clause_and_params(filters)
+        delete_sql += sql_where_clause
 
         await self._ensure_db_setup_async()
         assert self._async_cursor is not None

{pgvector_haystack-6.6.1 → pgvector_haystack-7.0.0}/tests/test_document_store.py

@@ -9,6 +9,7 @@ import pytest
 from haystack.dataclasses.document import ByteStream, Document
 from haystack.document_stores.errors import DocumentStoreError, DuplicateDocumentError
 from haystack.document_stores.types import DuplicatePolicy
+from haystack.errors import FilterError
 from haystack.testing.document_store import (
     CountDocumentsByFilterTest,
     CountDocumentsTest,
@@ -165,6 +166,12 @@ class TestDocumentStore(
         assert document_store._connection is not None
 
 
+def test_delete_by_filter_rejects_an_empty_filter(mock_store):
+    """An empty filter would compile to an unqualified DELETE and empty the table."""
+    with pytest.raises(FilterError, match="non-empty filter"):
+        mock_store.delete_by_filter({})
+
+
 @pytest.mark.usefixtures("patches_for_unit_tests")
 def test_init(monkeypatch):
     monkeypatch.setenv("PG_CONN_STR", "some_connection_string")

{pgvector_haystack-6.6.1 → pgvector_haystack-7.0.0}/tests/test_document_store_async.py

@@ -9,6 +9,7 @@ import pytest
 from haystack.dataclasses.document import ByteStream, Document
 from haystack.document_stores.errors import DocumentStoreError, DuplicateDocumentError
 from haystack.document_stores.types import DuplicatePolicy
+from haystack.errors import FilterError
 from haystack.testing.document_store_async import (
     CountDocumentsAsyncTest,
     CountDocumentsByFilterAsyncTest,
@@ -367,6 +368,13 @@ async def test_write_documents_async_rejects_non_document_items(mock_store):
         await mock_store.write_documents_async([{"not": "a document"}])
 
 
+@pytest.mark.asyncio
+async def test_delete_by_filter_async_rejects_an_empty_filter(mock_store):
+    """Same contract as the sync path, and the message must name the async methods."""
+    with pytest.raises(FilterError, match=r"delete_by_filter_async requires .* delete_all_documents_async\(\)"):
+        await mock_store.delete_by_filter_async({})
+
+
 @pytest.mark.asyncio
 async def test_count_unique_metadata_by_filter_async_rejects_empty_fields(mock_store):
     with pytest.raises(ValueError, match="metadata_fields must be a non-empty list"):