Update app.py

app.py

@@ -49,36 +49,39 @@ def ensure_model() -> str:
 # ---------------------------------------
 # Piper: rely on the PyPI wheel (piper-tts)
 # ---------------------------------------
-def ensure_piper_bin() -> str | None:
-    """
-    Make sure a usable Piper binary is available.
+# --- keep your existing imports ---
+import os
+from huggingface_hub import hf_hub_download
 
-    Priority:
-      1) If PIPER_BIN is an absolute path and executable → use it
-      2) If 'piper' is in PATH (installed by pip: piper-tts) → use that
-      3) Otherwise, return None (TTS will fall back to 'say' on macOS or be disabled)
+def ensure_piper_voice():
+    """
+    Make sure the Piper voice model exists at PIPER_MODEL.
+    Defaults to Amy-medium English if none specified.
     """
-    desired = os.getenv("PIPER_BIN")
-    if desired:
-        # absolute path wins if it exists
-        if os.path.isabs(desired) and os.path.exists(desired):
-            st = os.stat(desired)
-            if not (st.st_mode & stat.S_IXUSR):
-                os.chmod(desired, st.st_mode | stat.S_IXUSR)
-            print(f"[PIPER] Using PIPER_BIN at {desired}")
-            return desired
-        # if user set just 'piper', let which() resolve it
-        if desired.strip() == "piper":
-            resolved = which("piper")
-            if resolved:
-                print(f"[PIPER] Found in PATH: {resolved}")
-                return resolved
-            else:
-                print("[PIPER] 'piper' not found in PATH despite PIPER_BIN=piper")
-                return None
-        # user gave a relative path that doesn't exist
-        print(f"[PIPER] PIPER_BIN set but not valid: {desired}")
-        return None
+    target_path = os.getenv("PIPER_MODEL", "models/piper/en_US-amy-medium.onnx")
+    repo_id = os.getenv("PIPER_VOICE_REPO", "rhasspy/piper-voices")
+    hf_file = os.getenv("PIPER_VOICE_FILE",
+                        "en/en_US/en_US-amy-medium.onnx")  # path inside the repo
+
+    # If already present, we're done.
+    if os.path.exists(target_path):
+        print(f"[PIPER] Voice model present: {target_path}")
+        return target_path
+
+    os.makedirs(os.path.dirname(target_path), exist_ok=True)
+    print(f"[PIPER] Downloading voice {hf_file} from {repo_id} …")
+    local = hf_hub_download(
+        repo_id=repo_id,
+        filename=hf_file,
+        local_dir=os.path.dirname(target_path),
+        local_dir_use_symlinks=False,
+    )
+    # Move/rename to the exact target filename if the name differs
+    if os.path.abspath(local) != os.path.abspath(target_path):
+        import shutil
+        shutil.copyfile(local, target_path)
+    print(f"[PIPER] Voice ready at: {target_path}")
+    return target_path
 
     # No PIPER_BIN provided → try PATH
     resolved = which("piper")
@@ -109,11 +112,7 @@ def main():
     # Ensure model exists locally
     ensure_model()
 
-    # Make sure Piper binary is discoverable (no download at runtime; rely on piper-tts)
-    piper_bin = ensure_piper_bin()
-    if piper_bin:
-        # Export to env so tts_router can pick it up
-        os.environ["PIPER_BIN"] = piper_bin
+    ensure_piper_voice()
 
     # Launch Gradio
     demo = build_demo()