Merge pull request #11 from dgaida/fix-gradio-5-crash-5935744820650444043

Fix Gradio 5.x crash and modernization

Author: dgaida

config/config_manager.py

@@ -38,7 +38,7 @@
 class ServerConfig(BaseModel):
     """Server configuration settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     host: str = Field("127.0.0.1", description="Server host address")
     port: int = Field(8000, ge=1024, le=65535, description="Server port")
@@ -59,7 +59,7 @@ def validate_log_level(cls, v: str) -> str:
 class WorkspaceBounds(BaseModel):
     """Workspace boundary coordinates."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     x_min: float = Field(..., description="Minimum X coordinate (meters)")
     x_max: float = Field(..., description="Maximum X coordinate (meters)")
@@ -86,7 +86,7 @@ def validate_y_range(cls, v: float, info) -> float:
 class WorkspaceConfig(BaseModel):
     """Workspace configuration."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     id: str = Field(..., description="Workspace identifier")
     bounds: WorkspaceBounds = Field(..., description="Workspace boundaries")
@@ -104,7 +104,7 @@ def validate_center(cls, v: list[float]) -> list[float]:
 class MotionConfig(BaseModel):
     """Robot motion parameters."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     pick_z_offset: float = Field(0.001, ge=0.0, le=0.1, description="Z-offset for picking (m)")
     place_z_offset: float = Field(0.001, ge=0.0, le=0.1, description="Z-offset for placing (m)")
@@ -118,7 +118,7 @@ class MotionConfig(BaseModel):
 class RobotConfig(BaseModel):
     """Robot configuration settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     type: str = Field("niryo", description="Robot type")
     simulation: bool = Field(True, description="Use simulation mode")
@@ -141,7 +141,7 @@ def validate_robot_type(cls, v: str) -> str:
 class SpatialConfig(BaseModel):
     """Spatial query thresholds."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     close_to_radius_m: float = Field(0.02, ge=0.001, le=0.5, description="'Close to' radius (m)")
     left_right_threshold_m: float = Field(0.01, ge=0.001, le=0.1, description="Left/right threshold (m)")
@@ -151,7 +151,7 @@ class SpatialConfig(BaseModel):
 class DetectionConfig(BaseModel):
     """Object detection settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     model: str = Field("owlv2", description="Detection model")
     device: str = Field("cuda", description="Computation device")
@@ -183,7 +183,7 @@ def validate_device(cls, v: str) -> str:
 class LLMProviderConfig(BaseModel):
     """LLM provider-specific settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     enabled: bool = Field(True, description="Enable this provider")
     default_model: str = Field(..., description="Default model name")
@@ -195,7 +195,7 @@ class LLMProviderConfig(BaseModel):
 class LLMConfig(BaseModel):
     """LLM configuration settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     default_provider: str = Field("auto", description="Default LLM provider")
     temperature: float = Field(0.7, ge=0.0, le=2.0, description="Sampling temperature")
@@ -218,7 +218,7 @@ def validate_provider(cls, v: str) -> str:
 class TTSProviderConfig(BaseModel):
     """TTS provider settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     voice_id: Optional[str] = None
     voice: Optional[str] = None
@@ -230,7 +230,7 @@ class TTSProviderConfig(BaseModel):
 class TTSConfig(BaseModel):
     """Text-to-speech settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     enabled: bool = Field(True, description="Enable TTS")
     provider: str = Field("elevenlabs", description="TTS provider")
@@ -250,7 +250,7 @@ def validate_provider(cls, v: str) -> str:
 class RedisStreamsConfig(BaseModel):
     """Redis stream names."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     camera: str = Field("robot_camera", description="Camera stream name")
     detected_objects: str = Field("detected_objects", description="Detected objects stream")
@@ -260,7 +260,7 @@ class RedisStreamsConfig(BaseModel):
 class RedisConfig(BaseModel):
     """Redis connection settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     host: str = Field("localhost", description="Redis host")
     port: int = Field(6379, ge=1, le=65535, description="Redis port")
@@ -272,7 +272,7 @@ class RedisConfig(BaseModel):
 class GUIConfig(BaseModel):
     """GUI settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     host: str = Field("127.0.0.1", description="GUI host")
     port: int = Field(7860, ge=1024, le=65535, description="GUI port")
@@ -285,7 +285,7 @@ class GUIConfig(BaseModel):
 class LogRotationConfig(BaseModel):
     """Log rotation settings."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     max_bytes: int = Field(10485760, ge=1024, description="Max log file size (bytes)")
     backup_count: int = Field(5, ge=1, le=100, description="Number of backup files")
@@ -294,7 +294,7 @@ class LogRotationConfig(BaseModel):
 class LoggingConfig(BaseModel):
     """Logging configuration."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     format: str = Field(..., description="Log message format")
     date_format: str = Field("%Y-%m-%d %H:%M:%S", description="Date format")
@@ -305,17 +305,22 @@ class LoggingConfig(BaseModel):
 class EnvironmentOverrides(BaseModel):
     """Environment-specific configuration overrides."""
 
-    model_config = ConfigDict(extra="allow")
+    model_config = ConfigDict(extra="ignore")
 
-    server: Optional[Dict[str, Any]] = None
-    robot: Optional[Dict[str, Any]] = None
-    llm: Optional[Dict[str, Any]] = None
+    server: Optional[ServerConfig] = None
+    robot: Optional[RobotConfig] = None
+    llm: Optional[LLMConfig] = None
+    detection: Optional[DetectionConfig] = None
+    tts: Optional[TTSConfig] = None
+    redis: Optional[RedisConfig] = None
+    gui: Optional[GUIConfig] = None
+    logging: Optional[LoggingConfig] = None
 
 
 class RobotMCPConfig(BaseModel):
     """Root configuration model."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     server: ServerConfig
     robot: RobotConfig

docs/en/configuration.md

@@ -521,7 +521,7 @@ Add custom validators in `config_manager.py`:
 
 ```python
 class ServerConfig(BaseModel):
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
     port: int = Field(8000, ge=1024, le=65535)
 
     @field_validator("port")

docs/en/mcp_api_reference.md

@@ -987,11 +987,12 @@ class PickPlaceInput(BaseModel):
         if isinstance(v, Location):
             return v
         if isinstance(v, str):
-            valid_locations = [loc.value for loc in Location if loc
+            valid_locations = [loc.value for loc in Location if loc.value is not None]
+            if v.lower() not in [loc.lower() for loc in valid_locations]:
+                raise ValueError(f"Invalid location '{v}'")
+            return v
 ```
 
-# TODO: example above uncomplete
-
 ---
 
 ## Integration Guide

pyproject.toml

@@ -39,7 +39,7 @@ dependencies = [
 [project.optional-dependencies]
 # GUI dependencies
 gui = [
-    "gradio>=4.0.0",
+    "gradio>=5.0.0",
     "redis_robot_comm @ git+https://github.com/dgaida/redis_robot_comm.git",
 ]
 

requirements_gradio.txt

@@ -13,7 +13,7 @@ git+https://github.com/dgaida/speech2text.git
 git+https://github.com/dgaida/redis_robot_comm.git
 
 # GUI dependencies
-gradio>=4.0.0,<5.0.0
+gradio>=5.0.0
 
 # Utility packages
 python-dotenv>=1.0.0

robot_gui/mcp_app.py

@@ -9,7 +9,7 @@
 - FastMCP client integration
 - Real-time object detection display
 
-Compatible with Gradio 3.x, 4.x, 5.x, and 6.x
+Compatible with Gradio 5.x and 6.x
 """
 
 import asyncio
@@ -170,7 +170,7 @@ async def process_chat(self, message: str, history: list):
 
         Args:
             message: User message
-            history: Chat history (list of tuples)
+            history: Chat history (list of dicts for type="messages")
 
         Yields:
             Updated chat history
@@ -180,29 +180,31 @@ async def process_chat(self, message: str, history: list):
             return
 
         if not self.mcp_connected:
-            history.append((message, "⚠️ MCP server not connected. Please connect first."))
+            history.append({"role": "user", "content": message})
+            history.append({"role": "assistant", "content": "⚠️ MCP server not connected. Please connect first."})
             yield history
             return
 
-        # Add user message (initially with no response)
-        history.append((message, ""))
+        # Add user message and initial assistant message
+        history.append({"role": "user", "content": message})
+        history.append({"role": "assistant", "content": ""})
         yield history
 
         try:
             # Add "thinking" indicator
-            history[-1] = (message, "🤔 Processing...")
+            history[-1]["content"] = "🤔 Processing..."
             yield history
 
             # Process through MCP client
             response = await self.mcp_client.chat(message)
 
             # Update with actual response
-            history[-1] = (message, response)
+            history[-1]["content"] = response
             yield history
 
         except Exception as e:
             error_msg = f"❌ Error: {str(e)}"
-            history[-1] = (message, error_msg)
+            history[-1]["content"] = error_msg
             yield history
 
     def record_voice(self):
@@ -331,8 +333,8 @@ def create_gradio_interface(gui: RobotMCPGUI):
         with gr.Row():
             # Left: Chat interface
             with gr.Column(scale=2):
-                # Simple chatbot without optional parameters for max compatibility
-                chatbot = gr.Chatbot(label="Robot Assistant", height=500)
+                # Use type="messages" for modern Gradio compatibility
+                chatbot = gr.Chatbot(label="Robot Assistant", height=500, type="messages")
 
                 with gr.Row():
                     msg_input = gr.Textbox(

server/schemas.py

@@ -18,7 +18,7 @@
 class CoordinateModel(BaseModel):
     """Validates 2D coordinate [x, y] in meters."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     coordinate: List[float] = Field(..., min_length=2, max_length=2)
 
@@ -34,7 +34,7 @@ def validate_coordinate_values(cls, v):
 class PickPlaceInput(BaseModel):
     """Input validation for pick_place_object."""
 
-    model_config = ConfigDict(arbitrary_types_allowed=True, extra="forbid")
+    model_config = ConfigDict(arbitrary_types_allowed=True, extra="ignore")
 
     object_name: str = Field(..., min_length=1, description="Name of the object to pick")
     pick_coordinate: List[float] = Field(..., min_length=2, max_length=2)
@@ -82,7 +82,7 @@ def validate_location(cls, v):
 class PickObjectInput(BaseModel):
     """Input validation for pick_object."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     object_name: str = Field(..., min_length=1)
     pick_coordinate: List[float] = Field(..., min_length=2, max_length=2)
@@ -100,7 +100,7 @@ def validate_coordinate(cls, v):
 class PlaceObjectInput(BaseModel):
     """Input validation for place_object."""
 
-    model_config = ConfigDict(arbitrary_types_allowed=True, extra="forbid")
+    model_config = ConfigDict(arbitrary_types_allowed=True, extra="ignore")
 
     place_coordinate: List[float] = Field(..., min_length=2, max_length=2)
     location: Optional[Union[Location, str]] = Field(None, description="Relative placement location")
@@ -137,7 +137,7 @@ def validate_location(cls, v):
 class PushObjectInput(BaseModel):
     """Input validation for push_object."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     object_name: str = Field(..., min_length=1)
     push_coordinate: List[float] = Field(..., min_length=2, max_length=2)
@@ -165,7 +165,7 @@ def validate_direction(cls, v):
 class WorkspacePointInput(BaseModel):
     """Input validation for get_workspace_coordinate_from_point."""
 
-    model_config = ConfigDict(extra="forbid")
+    model_config = ConfigDict(extra="ignore")
 
     workspace_id: str = Field(..., min_length=1)
     point: str = Field(...)
@@ -183,7 +183,7 @@ def validate_point(cls, v):
 class GetDetectedObjectsInput(BaseModel):
     """Input validation for get_detected_objects."""
 
-    model_config = ConfigDict(arbitrary_types_allowed=True, extra="forbid")
+    model_config = ConfigDict(arbitrary_types_allowed=True, extra="ignore")
 
     location: Optional[Union[Location, str]] = Field(None, description="Relative location")
     coordinate: Optional[List[float]] = Field(None, min_length=2, max_length=2)

tests/test_gui.py

@@ -92,8 +92,10 @@ async def test_process_chat_not_connected(self, gui):
         async for update in gui.process_chat("Hello", history):
             updated_history = update
 
-        assert len(updated_history) == 1
-        assert "not connected" in updated_history[0][1].lower()
+        assert len(updated_history) == 2
+        assert updated_history[0]["role"] == "user"
+        assert updated_history[0]["content"] == "Hello"
+        assert "not connected" in updated_history[1]["content"].lower()
 
     @pytest.mark.asyncio
     async def test_process_chat_success(self, gui, mock_dependencies):
@@ -110,7 +112,8 @@ async def test_process_chat_success(self, gui, mock_dependencies):
 
         assert len(updates) > 0
         final_history = updates[-1]
-        assert final_history[0] == ("Pick up pencil", "Robot response")
+        assert final_history[0] == {"role": "user", "content": "Pick up pencil"}
+        assert final_history[1] == {"role": "assistant", "content": "Robot response"}
         mock_client.chat.assert_called_once_with("Pick up pencil")
 
     def test_record_voice_success(self, gui, mock_dependencies):