@follenfang/fupload 0.0.16 → 0.0.17

package/fupload/SKILL.md

@@ -2,7 +2,7 @@
 name: fupload
 description: Explicit author-publishing workflow for World of Warcraft plugins, configuration shares, and WA/strings on NewBeeBox, NetEase DD, CurseForge, Heybox Workshop, and ModUs.Creator, including local Creator login reuse and plugin ZIP publishing. Use only when the user explicitly invokes `$fupload`, explicitly asks to use the Fupload Skill, or loads this Skill by path. Do not trigger from ordinary mentions of publishing, NewBeeBox, DD, CurseForge, Heybox, ModUs, plugins, configurations, or WA.
 metadata:
-  version: "0.0.16"
+  version: "0.0.17"
 ---
 
 # Fupload

package/fupload/scripts/fupload_cli/__init__.py

@@ -1,3 +1,3 @@
 """Fupload Python CLI."""
 
-__version__ = "0.0.16"
+__version__ = "0.0.17"

package/fupload/scripts/fupload_cli/dd.py

@@ -240,6 +240,20 @@ def state_dir() -> Path:
     return result
 
 
+def _sidecar_startup_error(error: Mapping[str, Any]) -> FuploadError:
+    message = str(error.get("message") or "DD sidecar failed to start")
+    return FuploadError(
+        message,
+        kind=str(error.get("kind") or "authentication_error"),
+        stage=str(error.get("stage") or "session"),
+        endpoint=error.get("endpoint"),
+        http_status=error.get("http_status"),
+        business_code=error.get("business_code"),
+        verification_required=bool(error.get("verification_required")),
+        details=error.get("details") if isinstance(error.get("details"), dict) else None,
+    )
+
+
 class Sidecar:
     def __init__(self) -> None:
         self.dd_dir, self.signature = discover_dd_info()
@@ -272,11 +286,8 @@ class Sidecar:
             raise
         if not ready.get("ready"):
             self.close()
-            raise FuploadError(
-                str((ready.get("error") or {}).get("message") or "DD sidecar failed to start"),
-                kind="authentication_error",
-                stage="session",
-            )
+            error = ready.get("error") if isinstance(ready.get("error"), dict) else {}
+            raise _sidecar_startup_error(error)
         credential_kind = ready.get("credential_kind")
         if credential_kind not in ("email", "mobile"):
             self.close()

package/fupload/scripts/fupload_cli/dd_broker.py

@@ -13,7 +13,7 @@ import time
 import uuid
 from pathlib import Path
 from types import SimpleNamespace
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List, Mapping, Optional
 
 from .errors import FuploadError, redact
 from .trust import verify_dd_executable
@@ -406,6 +406,17 @@ def _send(value: Dict[str, Any], timeout: float = 300) -> Dict[str, Any]:
     return response.get("data")
 
 
+def _startup_error(pending: Mapping[str, Any]) -> FuploadError:
+    error = pending.get("error")
+    if isinstance(error, dict):
+        return FuploadError.from_dict(error)
+    return FuploadError(
+        str(error or "DD task session failed to start"),
+        kind="session_start_failed",
+        stage="session",
+    )
+
+
 def start(confirm_close_gui: bool) -> Dict[str, Any]:
     existing = _load_live_state()
     if existing:
@@ -447,7 +458,7 @@ def start(confirm_close_gui: bool) -> Dict[str, Any]:
         creationflags=flags,
     )
     deadline = time.time() + 90
-    last_error = ""
+    last_error: Any = ""
     while time.time() < deadline:
         state = _load_live_state()
         if state and state.get("startup_id") == startup_id:
@@ -465,7 +476,7 @@ def start(confirm_close_gui: bool) -> Dict[str, Any]:
         if startup.exists():
             try:
                 pending = _read_json(startup)
-                last_error = str(pending.get("error") or "")
+                last_error = pending.get("error") or ""
             except FuploadError:
                 pass
         if process.poll() is not None:
@@ -487,10 +498,11 @@ def start(confirm_close_gui: bool) -> Dict[str, Any]:
     try:
         pending = _read_json(startup)
         if pending.get("startup_id") == startup_id:
+            last_error = pending.get("error") or last_error
             startup.unlink()
     except (FuploadError, OSError):
         pass
-    raise FuploadError(last_error or "DD task session failed to start", kind="session_start_failed", stage="session")
+    raise _startup_error({"error": last_error})
 
 
 def status(session_id: Optional[str] = None) -> Dict[str, Any]:
@@ -655,9 +667,17 @@ def _serve(startup_id: str) -> int:
                 connection.sendall((json.dumps(response, ensure_ascii=False, separators=(",", ":")) + "\n").encode("utf-8"))
         return 0
     except Exception as exc:
+        if isinstance(exc, FuploadError):
+            error = exc.as_dict()
+        else:
+            error = FuploadError(
+                redact(str(exc))[:400] or "DD task session failed to start",
+                kind="session_start_failed",
+                stage="session",
+            ).as_dict()
         _atomic_json(startup_path, {
             "startup_id": startup_id,
-            "error": redact(str(exc))[:400],
+            "error": error,
         })
         return 1
     finally:

package/fupload/scripts/fupload_cli/dd_sidecar.py

@@ -362,12 +362,13 @@ def _validation_details(probe):
     return result
 
 
-def install_response_probe(client):
+def install_response_probe(client, capture_success=False):
     """Capture response objects and HTTPError bodies without changing DD behavior."""
     session = getattr(client, "_session", None)
     restore = getattr(client, "_fupload_restore_response_probe", None)
     if callable(restore):
         restore()
+    client._fupload_capture_success = bool(capture_success)
     original_opener_open = urllib.request.OpenerDirector.open
 
     def opener_open(opener, *args, **kwargs):
@@ -400,6 +401,7 @@ def install_response_probe(client):
         if getattr(urllib.request.OpenerDirector, "open", None) is opener_open:
             urllib.request.OpenerDirector.open = original_opener_open
         client._fupload_restore_response_probe = None
+        client._fupload_capture_success = False
 
     client._fupload_restore_response_probe = restore_probe
     for method_name in ("get", "post"):
@@ -415,7 +417,7 @@ def install_response_probe(client):
                     status = int(getattr(response, "status_code", 0) or 0)
                 except (TypeError, ValueError):
                     status = 0
-                if status >= 400:
+                if status >= 400 or getattr(client, "_fupload_capture_success", False):
                     try:
                         body = str(getattr(response, "text", "") or "")
                     except Exception:
@@ -507,6 +509,59 @@ def failure_from_exception(exc, stage, response=None):
     )
 
 
+def author_login_failure(client):
+    probe = getattr(client, "_fupload_last_response_error", None)
+    return failure_from_exception(
+        RuntimeError("DD author API login failed"), "session", probe,
+    )
+
+
+def _last_response_payload(client):
+    probe = getattr(client, "_fupload_last_response_error", None)
+    if not isinstance(probe, dict) or not probe.get("body"):
+        return None
+    try:
+        payload = json.loads(probe["body"])
+    except (TypeError, ValueError):
+        return None
+    return payload if isinstance(payload, dict) else None
+
+
+def login_author_client(client):
+    try:
+        if client.login():
+            return
+        if _retryable_get_signature_rejection(_last_response_payload(client)):
+            client._fupload_last_response_error = None
+            if client.login():
+                return
+        raise author_login_failure(client)
+    finally:
+        client._fupload_capture_success = False
+
+
+def author_session_or_cleanup(qt, container, flow, jwt_helper, client):
+    session = (qt, container, flow, jwt_helper, client)
+    try:
+        login_author_client(client)
+    except Exception:
+        try:
+            close_session(session)
+        except Exception:
+            pass
+        raise
+    return session
+
+
+def startup_error_payload(exc):
+    if isinstance(exc, SidecarFailure):
+        return exc.as_dict()
+    return {
+        "type": type(exc).__name__,
+        "message": safe_exception_message(exc)[:400],
+    }
+
+
 def bootstrap():
     resource = os.path.join(DD_DIR, "ccvoicehub.res")
     pyqt_dir = os.path.join(DD_DIR, "ccsub64", "PyQt5")
@@ -643,10 +698,8 @@ def open_session(timeout=45):
         raise RuntimeError("DD NEP module is not initialized")
     client = UiApiClient(nep, login_cookie=getattr(controller, "_cookie", None))
     client._session.headers["User-Agent"] = USER_AGENT
-    if not client.login():
-        raise RuntimeError("DD author API login failed")
-    install_response_probe(client)
-    return qt, container, flow, jwt_helper, client
+    install_response_probe(client, capture_success=True)
+    return author_session_or_cleanup(qt, container, flow, jwt_helper, client)
 
 
 def api_result(payload):
@@ -905,10 +958,7 @@ def main():
                         "error": error})
         return 0
     except Exception as exc:
-        output({"ready": False, "error": {
-            "type": type(exc).__name__,
-            "message": safe_exception_message(exc)[:400],
-        }})
+        output({"ready": False, "error": startup_error_payload(exc)})
         return 1
     finally:
         machine_data.clientNo = original

package/npm/skill-manifest.json

@@ -1,9 +1,9 @@
 {
   "schema": "fupload.npm-skill-manifest.v1",
   "package_name": "@follenfang/fupload",
-  "package_version": "0.0.16",
-  "skill_version": "0.0.16",
-  "tree_sha256": "aba774b14679854ac01a09711aa10b2f7266af2c43e8220cc67c64b9559bfb51",
+  "package_version": "0.0.17",
+  "skill_version": "0.0.17",
+  "tree_sha256": "064aeb1a1291d240a3bea7e71458846395abe29c6e91f682d269ec0e684c4294",
   "files": [
     {
       "path": "agents/openai.yaml",
@@ -113,7 +113,7 @@
     {
       "path": "scripts/fupload_cli/__init__.py",
       "bytes": 50,
-      "sha256": "f0d2cbd76206a61d5fd78eef58ad808108f9538963da2c0ea67df79c8a41e104"
+      "sha256": "8e72acb94a18f173cf9cbbf0436b20f64e1991f54a3537b7e930b4196bab8439"
     },
     {
       "path": "scripts/fupload_cli/blackbox_web.py",
@@ -137,18 +137,18 @@
     },
     {
       "path": "scripts/fupload_cli/dd_broker.py",
-      "bytes": 27637,
-      "sha256": "1ba00dcaea80905769bb2e00a6e965f2a1e9bb46a33534d1e77a6564158605a8"
+      "bytes": 28248,
+      "sha256": "6bbae4ed7aef4feca552816d7dbefa6069dc3d1d9a39a4af35582a06db4ce2d5"
     },
     {
       "path": "scripts/fupload_cli/dd_sidecar.py",
-      "bytes": 37603,
-      "sha256": "6ad1ab9e2145757f44b410d87d554485566812d9584820180ff3bb7b5bc7763a"
+      "bytes": 39150,
+      "sha256": "42791c82e5518838aedbe209823b94deb7de09b2f3eccb4fe563364f58becc6c"
     },
     {
       "path": "scripts/fupload_cli/dd.py",
-      "bytes": 113914,
-      "sha256": "31e349cfee243c1ae8afefd74acfe3f5d79771a1c23ed1892aaf9d8e9b416922"
+      "bytes": 114434,
+      "sha256": "cf808efdd4a93d80bddb533af4ac14696e889e0fd359acdacd58a7a49ce050b0"
     },
     {
       "path": "scripts/fupload_cli/errors.py",
@@ -208,7 +208,7 @@
     {
       "path": "SKILL.md",
       "bytes": 28383,
-      "sha256": "2477e603ca826b9bd31533995f2217bfd9239c070adbaa2aa4b829fb6a89fb7f"
+      "sha256": "9aebce883e6d83f0e3c8b36f497e59eb6f4df5e1e4af231df60ca3299188fb86"
     }
   ]
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@follenfang/fupload",
-  "version": "0.0.16",
+  "version": "0.0.17",
   "description": "Install and run the Fuploader Agent Skill and Python CLI.",
   "type": "module",
   "bin": {