refactor: simplify app — deduplicate streaming and model resolution

ppmlx/engine.py

@@ -61,18 +61,12 @@ def _strip_thinking(text: str) -> tuple[str, str | None]:
 
 
 def _resolve_model_path(repo_id: str) -> str:
-    """
-    Resolve a repo_id to a local path if available, otherwise return the repo_id
-    for direct HuggingFace loading.
-    """
+    """Resolve a repo_id to a local path if available, otherwise return the repo_id."""
     try:
-        from ppmlx.models import get_model_path
-        local = get_model_path(repo_id)
-        if local:
-            return str(local)
+        from ppmlx.models import resolve_model_path
+        return resolve_model_path(repo_id)
     except ImportError:
-        pass
-    return repo_id
+        return repo_id
 
 
 class TextEngine:

ppmlx/engine_embed.py

@@ -6,13 +6,10 @@
 
 def _resolve_model_path(repo_id: str) -> str:
     try:
-        from ppmlx.models import get_model_path
-        local = get_model_path(repo_id)
-        if local:
-            return str(local)
+        from ppmlx.models import resolve_model_path
+        return resolve_model_path(repo_id)
     except ImportError:
-        pass
-    return repo_id
+        return repo_id
 
 
 def _l2_normalize(vec: list[float]) -> list[float]:

ppmlx/engine_vlm.py

@@ -9,13 +9,10 @@
 def _resolve_model_path(repo_id: str) -> str:
     """Resolve alias to local path if available."""
     try:
-        from ppmlx.models import get_model_path
-        local = get_model_path(repo_id)
-        if local:
-            return str(local)
+        from ppmlx.models import resolve_model_path
+        return resolve_model_path(repo_id)
     except ImportError:
-        pass
-    return repo_id
+        return repo_id
 
 
 class VisionEngine:

ppmlx/models.py

@@ -395,6 +395,15 @@ def _bg_download() -> None:
     return local_path
 
 
+def resolve_model_path(repo_id: str) -> str:
+    """Resolve a repo_id to a local path if available, otherwise return the
+    repo_id for direct HuggingFace loading."""
+    local = get_model_path(repo_id)
+    if local:
+        return str(local)
+    return repo_id
+
+
 def get_model_path(alias_or_repo: str) -> Path | None:
     """Return local path if model exists, else None."""
     try: