fix(skill): persist result_urls to local for offline recovery

- Extend _save_session to store result_urls and completed_at
- Save results locally on completion in wait-and-deliver, query-session, tasks, recover
- Recover reads completed sessions from local (no API call), only checks pending via API
- Reduces hallucination: agent can always find video URLs from local file

Author: alchemistklk

apps/desktop/static/bundled-skills/libtv-video/scripts/libtv_video.py

@@ -67,22 +67,32 @@ def _load_sessions():
     with open(path, "r") as f:
         return json.load(f)
 
-def _save_session(session_id, project_uuid="", status="active", text=""):
+def _save_session(session_id, project_uuid="", status="active", text="",
+                   result_urls=None, completed_at=""):
     sessions = _load_sessions()
     for s in sessions:
         if s["session_id"] == session_id:
             s["status"] = status
             if project_uuid:
                 s["project_uuid"] = project_uuid
+            if result_urls:
+                s["result_urls"] = result_urls
+            if completed_at:
+                s["completed_at"] = completed_at
             break
     else:
-        sessions.append({
+        entry = {
             "session_id": session_id,
             "project_uuid": project_uuid,
             "status": status,
             "text": text[:80],
             "created_at": datetime.now().isoformat(),
-        })
+        }
+        if result_urls:
+            entry["result_urls"] = result_urls
+        if completed_at:
+            entry["completed_at"] = completed_at
+        sessions.append(entry)
     sessions = sessions[-50:]
     path = _sessions_file_path()
     os.makedirs(os.path.dirname(path), exist_ok=True)
@@ -396,6 +406,11 @@ def cmd_query_session(args):
     # Extract result URLs
     urls = extract_result_urls(messages)
 
+    # Persist results to local so recover can read without API
+    if urls:
+        _save_session(session_id, project_uuid=args.project_id or "", status="completed",
+                      result_urls=urls, completed_at=datetime.now().isoformat())
+
     out = {"messages": messages}
     if args.project_id:
         out["projectUrl"] = f"{PROJECT_CANVAS_BASE}{args.project_id}"
@@ -463,7 +478,8 @@ def cmd_wait_and_deliver(args):
         urls = extract_result_urls(messages)
 
         if urls:
-            _save_session(session_id, project_uuid=project_uuid, status="completed")
+            _save_session(session_id, project_uuid=project_uuid, status="completed",
+                          result_urls=urls, completed_at=datetime.now().isoformat())
             deliver_results(urls, project_uuid=project_uuid)
 
             # Auto-download
@@ -496,12 +512,58 @@ def cmd_wait_and_deliver(args):
 
 
 def cmd_recover(args):
-    pending = _get_pending_sessions()
+    sessions = _load_sessions()
+    if not sessions:
+        print("No sessions found.")
+        return
+
+    # Show completed sessions from local (no API call needed)
+    completed = [s for s in sessions if s["status"] in ("completed", "failed", "timeout")]
+    pending = [s for s in sessions if s["status"] not in ("completed", "failed", "timeout")]
+
+    if completed:
+        print(f"📋 {len(completed)} completed session(s) (from local):")
+        for s in completed[-10:]:
+            sid = s["session_id"]
+            project_uuid = s.get("project_uuid", "")
+            local_urls = s.get("result_urls", [])
+            status = s["status"]
+            text = s.get("text", "")
+
+            if status == "completed" and local_urls:
+                print(f"  ✅ {sid[:16]}... Completed ({text})")
+                for url in local_urls:
+                    print(f"     🎬 {url}")
+                if project_uuid:
+                    print(f"     🎨 Canvas: {PROJECT_CANVAS_BASE}{project_uuid}")
+            elif status == "failed":
+                print(f"  ❌ {sid[:16]}... Failed ({text})")
+            elif status == "timeout":
+                print(f"  ⏰ {sid[:16]}... Timeout ({text})")
+            else:
+                # Completed but no local URLs — re-fetch once
+                try:
+                    result = call_gateway("GET", f"/libtv/v1/session/{sid}")
+                    messages = result.get("messages") or []
+                    urls = extract_result_urls(messages)
+                    if urls:
+                        _save_session(sid, project_uuid=project_uuid, status="completed",
+                                      result_urls=urls, completed_at=datetime.now().isoformat())
+                        print(f"  ✅ {sid[:16]}... Completed ({text})")
+                        for url in urls:
+                            print(f"     🎬 {url}")
+                    else:
+                        print(f"  ✅ {sid[:16]}... Completed, no result URLs ({text})")
+                except Exception as e:
+                    print(f"  ✅ {sid[:16]}... Completed, fetch error: {e}")
+
     if not pending:
-        print("No pending sessions found.")
+        if not completed:
+            print("No sessions found.")
         return
 
-    print(f"Found {len(pending)} pending session(s):")
+    # Check pending sessions via API
+    print(f"\n🔍 {len(pending)} pending session(s) (checking API...):")
     for s in pending:
         sid = s["session_id"]
         project_uuid = s.get("project_uuid", "")
@@ -513,7 +575,8 @@ def cmd_recover(args):
             urls = extract_result_urls(messages)
 
             if urls:
-                _save_session(sid, project_uuid=project_uuid, status="completed")
+                _save_session(sid, project_uuid=project_uuid, status="completed",
+                              result_urls=urls, completed_at=datetime.now().isoformat())
                 print(f"  ✅ {sid[:16]}... Completed!")
                 for url in urls:
                     print(f"     🎬 {url}")
@@ -544,6 +607,11 @@ def cmd_tasks(args):
         video_url = t.get("video_url")
         error = t.get("error_message")
 
+        # Persist completed task results to local for offline recovery
+        if status == "completed" and video_url:
+            _save_session(tid, status="completed",
+                          result_urls=[video_url], completed_at=completed or "")
+
         status_icon = {
             "completed": "✅", "failed": "❌", "composing": "⏳",
             "rendering": "⏳", "pending": "🔄",