refactor: change complete_subgoals_by_ids to use default_factory=list instead of None default

Author: plfavreau

minitap/mobile_use/agents/cortex/cortex.py

@@ -119,7 +119,7 @@ async def __call__(self, state: State):
             update={
                 "agents_thoughts": [agent_thought],
                 "structured_decisions": response.decisions,
-                "complete_subgoals_by_ids": response.complete_subgoals_by_ids or [],
+                "complete_subgoals_by_ids": response.complete_subgoals_by_ids,
                 "screen_analysis_prompt": response.screen_analysis_prompt,
                 "latest_screenshot_base64": None,
                 "latest_ui_hierarchy": None,

minitap/mobile_use/agents/cortex/types.py

@@ -10,8 +10,8 @@ class CortexOutput(BaseModel):
         default=None,
         description="The reason for the goals completion, if there are any goals to be completed.",
     )
-    complete_subgoals_by_ids: list[str] | None = Field(
-        default=None, description="List of subgoal IDs to complete"
+    complete_subgoals_by_ids: list[str] = Field(
+        default_factory=list, description="List of subgoal IDs to complete"
     )
     screen_analysis_prompt: str | None = Field(
         default=None,