fix(sync): use configured reload interval for empty-project sleep

The guard added in b38823b slept a hardcoded 30 s when no projects were
selected, which broke test_run_handles_no_projects and three sibling
reload tests:

- test_run_handles_no_projects expects the sleep to match
  config.watch_project_reload_interval (the existing project-reload
  cadence), not a new magic number.
- test_run_reloads_projects_each_cycle, test_run_continues_after_cycle_error,
  and test_timer_task_cancelled_properly hung because they created Projects
  whose names weren't registered in app_config.projects. get_project_mode()
  treats unknown names as CLOUD, so _select_projects_to_watch() filtered
  them all out, the guard's sleep+continue fired every iteration, and the
  stubbed _watch_projects_cycle never got called.

Two fixes:

1. Sleep watch_project_reload_interval instead of 30 s, matching the
   existing reload-loop cadence and honoring the existing test contract.

2. Register the test projects as local-mode in app_config.projects so
   they survive the cloud filter. This aligns the reload tests with
   test_run_filters_cloud_only_projects_each_cycle, which already does
   this.

Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

src/basic_memory/sync/watch_service.py

@@ -216,14 +216,14 @@ async def run(self):  # pragma: no cover
                 # Why: watchfiles.awatch() requires at least one path. Calling it
                 #      with an empty list raises ValueError, which the outer handler
                 #      catches with a 5s sleep — producing a tight error-log loop.
-                # Outcome: sleep longer and retry, giving the project list a chance
-                #          to populate (e.g. after a project is added).
+                # Outcome: sleep the configured reload interval before retrying, so
+                #          newly added projects get picked up on the next cycle.
                 if not projects:
                     logger.warning(
                         "No projects to watch; sleeping before retry "
                         f"(constrained_project={self.constrained_project!r})"
                     )
-                    await asyncio.sleep(30)
+                    await asyncio.sleep(self.app_config.watch_project_reload_interval)
                     continue
 
                 project_paths = [project.path for project in projects]

tests/sync/test_watch_service_reload.py

@@ -117,7 +117,16 @@ async def fake_write_status():
 
 @pytest.mark.asyncio
 async def test_run_reloads_projects_each_cycle(monkeypatch, tmp_path):
-    config = BasicMemoryConfig(watch_project_reload_interval=1)
+    # Projects must be registered in app_config.projects as local-mode, otherwise
+    # _select_projects_to_watch() treats unknown names as CLOUD and filters them
+    # out, which would short-circuit through the empty-projects guard in run().
+    config = BasicMemoryConfig(
+        watch_project_reload_interval=1,
+        projects={
+            "project1": {"path": str(tmp_path / "p1"), "mode": "local"},
+            "project2": {"path": str(tmp_path / "p2"), "mode": "local"},
+        },
+    )
     repo = _Repo(
         projects_side_effect=[
             [Project(id=1, name="project1", path=str(tmp_path / "p1"), permalink="project1")],
@@ -229,7 +238,9 @@ async def fake_write_status():
 
 @pytest.mark.asyncio
 async def test_run_continues_after_cycle_error(monkeypatch, tmp_path):
-    config = BasicMemoryConfig()
+    config = BasicMemoryConfig(
+        projects={"test": {"path": str(tmp_path / "test"), "mode": "local"}},
+    )
     repo = _Repo(
         projects_return=[Project(id=1, name="test", path=str(tmp_path / "test"), permalink="test")]
     )
@@ -264,7 +275,9 @@ async def fake_write_status():
 
 @pytest.mark.asyncio
 async def test_timer_task_cancelled_properly(monkeypatch, tmp_path):
-    config = BasicMemoryConfig()
+    config = BasicMemoryConfig(
+        projects={"test": {"path": str(tmp_path / "test"), "mode": "local"}},
+    )
     repo = _Repo(
         projects_return=[Project(id=1, name="test", path=str(tmp_path / "test"), permalink="test")]
     )