test: fix CI flakes (postgres migrations, tsquery errors, windows init)

- Stamp alembic head when Postgres schema is created via ORM in tests\n- Treat additional tsquery parse errors (e.g. 'no operand in tsquery') as safe empty results\n- Isolate ConfigManager home + extend timeout in subprocess init test for Windows

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

Author: phernandez

src/basic_memory/repository/postgres_search_repository.py

@@ -337,7 +337,9 @@ async def search(
             if (
                 "syntax error in tsquery" in msg
                 or "invalid input syntax for type tsquery" in msg
-            ):  # pragma: no cover
+                or "no operand in tsquery" in msg
+                or "no operator in tsquery" in msg
+            ):
                 logger.warning(f"tsquery syntax error for search term: {search_text}, error: {e}")
                 return []
 

tests/cli/test_cli_tool_exit.py

@@ -13,6 +13,7 @@
 The fix ensures db.shutdown_db() is called before asyncio.run() returns.
 """
 
+import os
 import subprocess
 import sys
 
@@ -55,7 +56,7 @@ def test_cli_command_exits_cleanly(self, command: list[str]):
                 "This indicates database connections are not being cleaned up properly."
             )
 
-    def test_ensure_initialization_exits_cleanly(self):
+    def test_ensure_initialization_exits_cleanly(self, tmp_path):
         """Test that ensure_initialization doesn't cause process hang.
 
         This test directly tests the initialization function that's called
@@ -71,11 +72,20 @@ def test_ensure_initialization_exits_cleanly(self):
 print("OK")
 """
         try:
+            # Ensure the subprocess uses an isolated home directory so ConfigManager doesn't
+            # touch the real user profile/AppData (which can be slow/flaky on CI Windows).
+            env = dict(os.environ)
+            bm_home = tmp_path / "basic-memory-home"
+            env["BASIC_MEMORY_HOME"] = str(bm_home)
+            env["HOME"] = str(tmp_path)
+            env["USERPROFILE"] = str(tmp_path)
+
             result = subprocess.run(
                 [sys.executable, "-c", code],
                 capture_output=True,
                 text=True,
-                timeout=10.0,
+                timeout=30.0,
+                env=env,
             )
             assert "OK" in result.stdout, f"Unexpected output: {result.stdout}"
         except subprocess.TimeoutExpired:

tests/conftest.py

@@ -9,6 +9,8 @@
 
 import pytest
 import pytest_asyncio
+from alembic import command
+from alembic.config import Config
 from sqlalchemy import text
 from sqlalchemy.ext.asyncio import (
     AsyncEngine,
@@ -233,6 +235,23 @@ async def engine_factory(
             await conn.execute(CREATE_POSTGRES_SEARCH_INDEX_METADATA)
             await conn.execute(CREATE_POSTGRES_SEARCH_INDEX_PERMALINK)
 
+            # Mark migrations as already applied for this test-created schema.
+            #
+            # Some codepaths (e.g. ensure_initialization()) invoke Alembic migrations.
+            # If we create tables via ORM directly, alembic_version is missing and migrations
+            # will try to create tables again, causing DuplicateTableError.
+            alembic_dir = Path(db.__file__).parent / "alembic"
+            cfg = Config()
+            cfg.set_main_option("script_location", str(alembic_dir))
+            cfg.set_main_option(
+                "file_template",
+                "%%(year)d_%%(month).2d_%%(day).2d_%%(hour).2d%%(minute).2d-%%(rev)s_%%(slug)s",
+            )
+            cfg.set_main_option("timezone", "UTC")
+            cfg.set_main_option("revision_environment", "false")
+            cfg.set_main_option("sqlalchemy.url", async_url)
+            command.stamp(cfg, "head")
+
         yield engine, session_maker
 
         await engine.dispose()