Improve file path extraction for Gradio compatibility

Enhanced file path extraction logic in both gradio_demo.py and openhands_poc/executor.py to support multiple Gradio versions (3.x and 4.x), handling both string paths and file-like objects. Added debug logging to aid in troubleshooting file object types and values.

gradio_demo.py

@@ -484,15 +484,25 @@ async def handle_user_openhands(schedule_file: gr.File, drawing_file: gr.File, s
             executor = get_executor()
             loop = asyncio.get_event_loop()
 
-            print(f"=== Calling executor with files: {schedule_file.name}, {drawing_file.name}, {spec_file.name} ===")
+            # Debug: Log file object details
+            print(f"=== schedule_file type: {type(schedule_file)}, value: {schedule_file} ===")
+            print(f"=== drawing_file type: {type(drawing_file)}, value: {drawing_file} ===")
+            print(f"=== spec_file type: {type(spec_file)}, value: {spec_file} ===")
+
+            # Extract paths - handle both string paths (Gradio 4.x+) and file objects (Gradio 3.x)
+            schedule_path = schedule_file if isinstance(schedule_file, str) else schedule_file.name
+            drawing_path = drawing_file if isinstance(drawing_file, str) else drawing_file.name
+            spec_path = spec_file if isinstance(spec_file, str) else spec_file.name
+
+            print(f"=== Calling executor with files: {schedule_path}, {drawing_path}, {spec_path} ===")
 
             # Wrap in lambda to ensure proper argument binding (SDK pattern)
             comparison_table, metadata = await loop.run_in_executor(
                 None,
                 lambda: executor.classify_and_compare_with_sdk(
-                    schedule_file.name,
-                    drawing_file.name,
-                    spec_file.name,
+                    schedule_path,
+                    drawing_path,
+                    spec_path,
                 )
             )
             print(f"=== Executor returned ===")

openhands_poc/executor.py

@@ -307,14 +307,27 @@ class AgentExecutor:
         Returns:
             str: File path
         """
+        # Debug: Log what we received
+        print(f"[_get_file_path] Received type: {type(file_obj)}, value: {file_obj}")
+        if hasattr(file_obj, "__dict__"):
+            print(f"[_get_file_path] Object attrs: {file_obj.__dict__}")
+
+        # Gradio 4.x+ returns file path as string directly
         if isinstance(file_obj, str):
             return file_obj
-        elif hasattr(file_obj, "name"):
+        # Gradio 3.x returns object with .name attribute (temp file path)
+        elif hasattr(file_obj, "name") and isinstance(file_obj.name, str) and len(file_obj.name) > 4:
             return file_obj.name
-        elif hasattr(file_obj, "path"):
+        # Some versions use .path
+        elif hasattr(file_obj, "path") and isinstance(file_obj.path, str):
             return file_obj.path
+        # Handle NamedString or similar wrapper types
+        elif hasattr(file_obj, "orig_name"):
+            # Try to find the actual path
+            if hasattr(file_obj, "name"):
+                return str(file_obj.name)
         else:
-            raise ValueError(f"Cannot extract path from {type(file_obj)}")
+            raise ValueError(f"Cannot extract path from {type(file_obj)}: {file_obj}")
 
 
 # Global executor instance