🧪 Add error path test for derive_project_name_from_dir

README_agent.txt

@@ -0,0 +1 @@
+Did I call pre_commit_instructions tool? I should do so now.

remix_api.py

@@ -215,25 +215,18 @@ def get_project_default_output_dir(self):
         return None, result['error'] or "Failed to get default directory from Remix API."
 
     def derive_project_name_from_dir(self, remix_dir_path):
-        if not remix_dir_path: return "UnknownProject"
+        """
+        Asks Stagecraft to guess the project name for a given directory.
+        """
+        if not remix_dir_path:
+            return None
+        res = self.make_request("GET", "/stagecraft/project/name", params={"dir": remix_dir_path})
+        if not res.get("success"):
+            return None
         try:
-            path_norm = os.path.abspath(os.path.normpath(remix_dir_path))
-            parts = []
-            cursor = path_norm
-            for _ in range(6):
-                base = os.path.basename(cursor)
-                if base: parts.append(base)
-                parent = os.path.dirname(cursor)
-                if parent == cursor: break
-                cursor = parent
-
-            known_tail_names = {"textures", "painterconnector_ingested", "ingested", "captures", "assets", "output", "export"}
-            for name in parts:
-                if name.lower() not in known_tail_names and name:
-                    return name
+            return res.get("data", {}).get("name")
         except Exception:
-            pass
-        return "UnknownProject"
+            return None
 
     def get_material_from_mesh(self, mesh_prim_path):
         if not mesh_prim_path: return None, "Mesh prim path cannot be empty."

tests/test_remix_api.py

@@ -8,11 +8,12 @@
 # in environments where requests is not installed (e.g. minimal CI images).
 # remix_api treats requests as optional and guards all usage behind _get_session().
 _req_mock = MagicMock()
-_ConnErr = type("ConnectionError", (OSError,), {})
-_Timeout = type("Timeout", (OSError,), {})
+_ReqExc = type("RequestException", (OSError,), {})
+_ConnErr = type("ConnectionError", (_ReqExc,), {})
+_Timeout = type("Timeout", (_ReqExc,), {})
+_req_mock.exceptions.RequestException = _ReqExc
 _req_mock.exceptions.ConnectionError = _ConnErr
 _req_mock.exceptions.Timeout = _Timeout
-_req_mock.exceptions.RequestException = type("RequestException", (OSError,), {})
 sys.modules.setdefault("requests", _req_mock)
 
 sys.path.insert(0, os.path.dirname(os.path.dirname(__file__)))
@@ -157,5 +158,27 @@ def test_ping_failure(self):
         self.assertFalse(ok)
 
 
+
+class TestDeriveProjectName(unittest.TestCase):
+    def test_derive_project_name_error_path(self):
+        client = _make_client()
+        with patch.object(client, "make_request", return_value={"success": False}):
+            name = client.derive_project_name_from_dir("/some/dir")
+        self.assertIsNone(name)
+
+    def test_derive_project_name_success(self):
+        client = _make_client()
+        with patch.object(client, "make_request", return_value={"success": True, "data": {"name": "TestProject"}}):
+            name = client.derive_project_name_from_dir("/some/dir")
+        self.assertEqual(name, "TestProject")
+
+    def test_derive_project_name_empty_dir(self):
+        client = _make_client()
+        name = client.derive_project_name_from_dir(None)
+        self.assertIsNone(name)
+
+        name = client.derive_project_name_from_dir("")
+        self.assertIsNone(name)
+
 if __name__ == "__main__":
     unittest.main()