fix(ci): restore fork main test consistency

Author: tcconnally

apps/desktop/package-lock.json

@@ -246,7 +246,9 @@ def test_update_on_fork_checks_upstream_when_origin_up_to_date(
         ), patch.object(hm, "_sync_with_upstream_if_needed") as sync_mock:
             cmd_update(mock_args)
 
-        sync_mock.assert_called_once_with(["git"], PROJECT_ROOT)
+        sync_mock.assert_called_once_with(
+            ["git", "-c", "windows.appendAtomically=false"], PROJECT_ROOT
+        )
         captured = capsys.readouterr()
         assert "Already up to date!" in captured.out
 
@@ -267,6 +269,7 @@ def test_update_refreshes_repo_and_tui_node_dependencies(
         import subprocess as _subprocess
         build_ok = _subprocess.CompletedProcess([], 0, stdout="", stderr="")
         with patch.object(hm, "_is_termux_env", return_value=False), \
+             patch("hermes_constants.find_node_executable", return_value="/usr/bin/npm"), \
              patch.object(hm, "_run_with_idle_timeout", return_value=build_ok) as mock_idle:
             cmd_update(mock_args)
 

tests/cron/test_cron_profile.py

@@ -82,7 +82,10 @@ def read(self):
 
     def test_falls_back_to_static_snapshot_on_fetch_failure(self, monkeypatch):
         monkeypatch.setattr(_models_mod, "_openrouter_catalog_cache", None)
-        with patch("hermes_cli.models._urlopen_model_catalog_request", side_effect=OSError("boom")):
+        # Pin the remote manifest out too — otherwise the fallback silently
+        # depends on whatever the deployed catalog currently contains.
+        with patch("hermes_cli.model_catalog.get_curated_openrouter_models", return_value=None), \
+             patch("hermes_cli.models._urlopen_model_catalog_request", side_effect=OSError("boom")):
             models = fetch_openrouter_models(force_refresh=True)
 
         assert models == OPENROUTER_MODELS

tests/hermes_cli/test_cmd_update.py

@@ -668,54 +668,6 @@ def _build_fal_edit_payload(
     }
 
 
-def _build_fal_edit_payload(
-    model_id: str,
-    prompt: str,
-    image_urls: list,
-    aspect_ratio: str = DEFAULT_ASPECT_RATIO,
-    seed: Optional[int] = None,
-    overrides: Optional[Dict[str, Any]] = None,
-) -> Dict[str, Any]:
-    """Build a FAL *edit* request payload (image-to-image) from unified inputs.
-
-    Every FAL edit endpoint takes ``image_urls`` (a list of source/reference
-    image URLs) plus the prompt. Size handling differs from text-to-image:
-    most edit endpoints auto-infer output dimensions from the input image, so
-    we only send ``image_size`` / ``aspect_ratio`` when the edit endpoint's
-    ``edit_supports`` whitelist accepts it. Keys outside ``edit_supports`` are
-    stripped before submission.
-    """
-    meta = FAL_MODELS[model_id]
-    edit_supports = meta.get("edit_supports") or set()
-    size_style = meta["size_style"]
-    sizes = meta["sizes"]
-
-    aspect = (aspect_ratio or DEFAULT_ASPECT_RATIO).lower().strip()
-    if aspect not in sizes:
-        aspect = DEFAULT_ASPECT_RATIO
-
-    payload: Dict[str, Any] = dict(meta.get("defaults", {}))
-    payload["prompt"] = (prompt or "").strip()
-    payload["image_urls"] = list(image_urls)
-
-    # Only express output size when the edit endpoint advertises the key.
-    # gpt-image-2 edit auto-infers size from the input, so `image_size` is
-    # intentionally absent from its edit_supports whitelist.
-    if size_style in {"image_size_preset", "gpt_literal"} and "image_size" in edit_supports:
-        payload["image_size"] = sizes[aspect]
-    elif size_style == "aspect_ratio" and "aspect_ratio" in edit_supports:
-        payload["aspect_ratio"] = sizes[aspect]
-
-    if seed is not None and isinstance(seed, int):
-        payload["seed"] = seed
-
-    if overrides:
-        for k, v in overrides.items():
-            if v is not None:
-                payload[k] = v
-
-    return {k: v for k, v in payload.items() if k in edit_supports}
-
 
 # ---------------------------------------------------------------------------
 # Upscaler