fix(tests): make CI-environment-independent

Two test fixes that were silently passing locally because of dev-machine
artifacts but failing on clean CI runners:

1. tests/test_project_manager.py used a hardcoded "/tmp/proj" path. On
   clean Linux/macOS/Windows CI runners this path doesn't exist, so
   `Path("/tmp/proj").is_dir()` returned False and the live-session and
   last-session-derivation code paths in `list_projects` were silently
   skipped, causing the assertions to compare against the wrong values.
   Locally the path happened to exist (e.g. U:\tmp\proj on the dev box)
   making the test pass for the wrong reason. Fixed by using a real
   `tempfile.TemporaryDirectory()` in setUp / tearDown.

2. test_ollama_embedding_path_used_when_available depends on numpy,
   which oracle/services/federated_graph.py lazy-imports. numpy was in
   the [vector] extra but not in [dev], so CI couldn't exercise the
   embedding similarity path. Added numpy + scikit-learn to [dev] so
   the CI matrix tests the production code paths end-to-end.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: drknowhow

pyproject.toml

@@ -68,6 +68,10 @@ dev = [
     "ruff>=0.6.0",
     "build>=1.0.0",
     "twine>=5.0.0",
+    # Vector deps so CI exercises the embedding/similarity code paths
+    # (federated_graph lazy-imports numpy; tests for that path need it).
+    "numpy>=1.24.0",
+    "scikit-learn>=1.3.0",
 ]
 
 [project.urls]

tests/test_project_manager.py

@@ -1,3 +1,4 @@
+import tempfile
 import unittest
 from datetime import datetime, timedelta, timezone
 from unittest.mock import patch
@@ -22,6 +23,16 @@ def get_recent(self, limit=1, event_type=None, since=None):
 class TestProjectManager(unittest.TestCase):
     def setUp(self):
         self.pm = ProjectManager()
+        # Use a real existing dir so Path(p["path"]).is_dir() is True. The
+        # previous hardcoded self.proj_path was accidentally truthy on dev
+        # machines where the path happened to exist (e.g. U:\tmp\proj on
+        # Windows) but False on clean CI runners — making the live-session
+        # and last-session-derivation paths silently skip.
+        self._tmp = tempfile.TemporaryDirectory()
+        self.proj_path = self._tmp.name
+
+    def tearDown(self):
+        self._tmp.cleanup()
 
     def test_live_session_info_ignores_stale_activity(self):
         old = (datetime.now(timezone.utc) - timedelta(hours=2)).isoformat()
@@ -47,8 +58,8 @@ def test_live_session_info_keeps_recent_activity(self):
 
     def test_list_projects_does_not_mark_null_port_project_active_without_recent_session(self):
         now = datetime.now(timezone.utc).isoformat()
-        with patch.object(self.pm, "_read_projects", return_value=[{"name": "Proj", "path": "/tmp/proj", "added_at": now}]), \
-             patch.object(self.pm, "_read_registry", return_value=[{"project_path": "/tmp/proj", "port": None, "started_at": now}]), \
+        with patch.object(self.pm, "_read_projects", return_value=[{"name": "Proj", "path": self.proj_path, "added_at": now}]), \
+             patch.object(self.pm, "_read_registry", return_value=[{"project_path": self.proj_path, "port": None, "started_at": now}]), \
              patch.object(self.pm, "_port_alive", return_value=False), \
              patch.object(self.pm, "_get_live_session_info", return_value=None), \
              patch.object(self.pm, "_read_project_config", return_value={}):
@@ -64,7 +75,7 @@ def test_list_projects_derives_last_session_from_activity_log(self):
         recent = (datetime.now(timezone.utc) - timedelta(hours=6)).isoformat()
         with patch.object(self.pm, "_read_projects", return_value=[{
             "name": "Proj",
-            "path": "/tmp/proj",
+            "path": self.proj_path,
             "added_at": old,
             "last_session": old,
         }]), \
@@ -86,7 +97,7 @@ def test_list_projects_marks_live_session_active_without_ui_port(self):
             "last_activity": now,
             "description": "recent",
         }
-        with patch.object(self.pm, "_read_projects", return_value=[{"name": "Proj", "path": "/tmp/proj", "added_at": now}]), \
+        with patch.object(self.pm, "_read_projects", return_value=[{"name": "Proj", "path": self.proj_path, "added_at": now}]), \
              patch.object(self.pm, "_read_registry", return_value=[]), \
              patch.object(self.pm, "_get_live_session_info", return_value=live_session), \
              patch.object(self.pm, "_read_project_config", return_value={}), \
@@ -103,15 +114,15 @@ def test_get_active_sessions_includes_live_session_without_ui_port(self):
         with patch.object(self.pm, "_read_registry", return_value=[]), \
              patch.object(self.pm, "list_projects", return_value=[{
                  "name": "Proj",
-                 "path": "/tmp/proj",
+                 "path": self.proj_path,
                  "session_active": True,
                  "ui_active": False,
                  "started_at": now,
                  "live_session_id": "sess-2",
              }]):
             sessions = self.pm.get_active_sessions()
         self.assertEqual(sessions, [{
-            "project_path": "/tmp/proj",
+            "project_path": self.proj_path,
             "project_name": "Proj",
             "port": None,
             "started_at": now,