fix(postgres-tests): avoid schema init conflicts; harden race handling

- Ensure Postgres test fixture populates db module globals so app lifespan doesn't rerun migrations\n- Tighten PostgresSearchRepository tsquery error detection (don't swallow missing-table errors)\n- Handle Postgres unique constraint message for file_path race condition

Signed-off-by: phernandez <[email protected]>

Author: phernandez

src/basic_memory/repository/postgres_search_repository.py

@@ -329,15 +329,21 @@ async def search(
                 result = await session.execute(text(sql), params)
                 rows = result.fetchall()
         except Exception as e:
-            # Handle tsquery syntax errors
-            if "tsquery" in str(e).lower() or "syntax error" in str(e).lower():  # pragma: no cover
+            # Handle tsquery syntax errors (and only those).
+            #
+            # Important: Postgres errors for other failures (e.g. missing table) will still mention
+            # `to_tsquery(...)` in the SQL text, so checking for the substring "tsquery" is too broad.
+            msg = str(e).lower()
+            if (
+                "syntax error in tsquery" in msg
+                or "invalid input syntax for type tsquery" in msg
+            ):  # pragma: no cover
                 logger.warning(f"tsquery syntax error for search term: {search_text}, error: {e}")
-                # Return empty results rather than crashing
                 return []
-            else:
-                # Re-raise other database errors
-                logger.error(f"Database error during search: {e}")
-                raise
+
+            # Re-raise other database errors
+            logger.error(f"Database error during search: {e}")
+            raise
 
         results = [
             SearchIndexRow(

src/basic_memory/sync/sync_service.py

@@ -766,7 +766,13 @@ async def sync_regular_file(self, path: str, new: bool = True) -> Tuple[Optional
                 return entity, checksum
             except IntegrityError as e:
                 # Handle race condition where entity was created by another process
-                if "UNIQUE constraint failed: entity.file_path" in str(e):
+                msg = str(e)
+                if (
+                    "UNIQUE constraint failed: entity.file_path" in msg
+                    or "uix_entity_file_path_project" in msg
+                    or "duplicate key value violates unique constraint" in msg
+                    and "file_path" in msg
+                ):
                     logger.info(
                         f"Entity already exists for file_path={path}, updating instead of creating"
                     )

tests/conftest.py

@@ -206,6 +206,13 @@ async def engine_factory(
             autoflush=False,
         )
 
+        # Important: wire the engine/session into the global db module state.
+        # Some codepaths (e.g. app initialization / MCP lifespan) call db.get_or_create_db(),
+        # which would otherwise create a separate engine and run migrations, conflicting with
+        # our test-created schema (and causing DuplicateTableError).
+        db._engine = engine
+        db._session_maker = session_maker
+
         from basic_memory.models.search import (
             CREATE_POSTGRES_SEARCH_INDEX_TABLE,
             CREATE_POSTGRES_SEARCH_INDEX_FTS,
@@ -229,6 +236,8 @@ async def engine_factory(
         yield engine, session_maker
 
         await engine.dispose()
+        db._engine = None
+        db._session_maker = None
     else:
         # SQLite mode
         db_type = DatabaseType.MEMORY