fix(magentic-one): Enhance error handling in orchestrate_step to manage invalid ledger formats (microsoft#4845)

* fix(magentic-one): Enhance error handling in orchestrate_step to manage invalid ledger formats

* formatting

---------

Co-authored-by: Roy Belio <[email protected]>
Co-authored-by: Jack Gerrits <[email protected]>
Co-authored-by: Jack Gerrits <[email protected]>

Author: 3 people

python/packages/autogen-agentchat/src/autogen_agentchat/teams/_group_chat/_magentic_one/_magentic_one_orchestrator.py

@@ -297,6 +297,7 @@ async def _orchestrate_step(self, cancellation_token: CancellationToken) -> None
                 for key in required_keys:
                     if (
                         key not in progress_ledger
+                        or not isinstance(progress_ledger[key], dict)
                         or "answer" not in progress_ledger[key]
                         or "reason" not in progress_ledger[key]
                     ):
@@ -305,7 +306,9 @@ async def _orchestrate_step(self, cancellation_token: CancellationToken) -> None
                 if not key_error:
                     break
                 await self._log_message(f"Failed to parse ledger information, retrying: {ledger_str}")
-            except json.JSONDecodeError:
+            except (json.JSONDecodeError, TypeError):
+                key_error = True
+                await self._log_message("Invalid ledger format encountered, retrying...")
                 continue
         if key_error:
             raise ValueError("Failed to parse ledger information after multiple retries.")