dayhoff-tools 1.6.9__tar.gz → 1.6.10__tar.gz

{dayhoff_tools-1.6.9 → dayhoff_tools-1.6.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: dayhoff-tools
-Version: 1.6.9
+Version: 1.6.10
 Summary: Common tools for all the repos at Dayhoff Labs
 Author: Daniel Martin-Alarcon
 Author-email: dma@dayhofflabs.com

{dayhoff_tools-1.6.9 → dayhoff_tools-1.6.10}/dayhoff_tools/cli/engine_commands.py

@@ -1862,33 +1862,29 @@ def _attempt_studio_attach(studio, engine, target_user, public_key):
 
     # --- determine if we should retry ---
     recoverable = False
+    error_text = response.json().get("error", "Unknown error")
+    err_msg = error_text.lower()
+    
+    # Check for "Studio is not available (status: in-use)" which means it's already attached
+    if response.status_code == 400 and "not available" in err_msg and "in-use" in err_msg:
+        # Studio is already attached somewhere - check if it's to THIS engine
+        if _is_studio_attached(studio["studio_id"], engine["instance_id"]):
+            return True, None  # It's attached to our target engine - success!
+        else:
+            return False, error_text  # It's attached elsewhere - fatal error
+    
     if response.status_code in (409, 503):
-        # Special-case: 409 because studio is already in-use. If it's attached to
-        # *this* engine we can safely treat it as success.
-        if response.status_code == 409:
-            try:
-                err_text = response.json().get("error", "").lower()
-            except Exception:
-                err_text = ""
-            # Fast path – avoid another API call when we know why we got 409.
-            if "in-use" in err_text or "already attached" in err_text:
-                if _is_studio_attached(studio["studio_id"], engine["instance_id"]):
-                    return True, None
         recoverable = True
     else:
-        err_msg = response.json().get("error", "").lower()
         RECOVERABLE_PATTERNS = [
             "not ready",
             "still starting",
             "initializing",
             "failed to mount",
             "device busy",
-            "not available",
             "pending",  # VM state pending
         ]
         FATAL_PATTERNS = [
-            "in-use",
-            "already attached",
             "permission",
         ]
         if any(p in err_msg for p in FATAL_PATTERNS):
@@ -1896,8 +1892,6 @@ def _attempt_studio_attach(studio, engine, target_user, public_key):
         elif any(p in err_msg for p in RECOVERABLE_PATTERNS):
             recoverable = True
 
-    error_text = response.json().get("error", "Unknown error")
-
     if not recoverable:
         # fatal – abort immediately
         return False, error_text

{dayhoff_tools-1.6.9 → dayhoff_tools-1.6.10}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "poetry.core.masonry.api"
 
 [project]
 name = "dayhoff-tools"
-version = "1.6.9"
+version = "1.6.10"
 description = "Common tools for all the repos at Dayhoff Labs"
 authors = [
     {name = "Daniel Martin-Alarcon", email = "dma@dayhofflabs.com"}