fix(runner): resolve PR review feedback for SDK bump

- Add messageId to reasoning cleanup end events so clients can correlate
  start/end events and avoid orphaning in-flight reasoning blocks
- Auto-migrate deprecated max_thinking_tokens to ThinkingConfigEnabled
  in forwarded_props for forward compatibility with SDK changes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jeremyeder

components/runners/ambient-runner/ag_ui_claude_sdk/adapter.py

@@ -1132,10 +1132,19 @@ def flush_pending_msg():
                 logger.debug("Cleanup: closing hanging thinking block")
                 ts = now_ms()
                 yield ReasoningMessageEndEvent(
-                    threadId=thread_id, runId=run_id, timestamp=ts
+                    threadId=thread_id,
+                    runId=run_id,
+                    messageId=current_reasoning_id,
+                    timestamp=ts,
+                )
+                yield ReasoningEndEvent(
+                    threadId=thread_id,
+                    runId=run_id,
+                    messageId=current_reasoning_id,
+                    timestamp=ts,
                 )
-                yield ReasoningEndEvent(threadId=thread_id, runId=run_id, timestamp=ts)
                 in_thinking_block = False
+                current_reasoning_id = None
 
             if has_streamed_text and current_message_id:
                 logger.debug(

components/runners/ambient-runner/ag_ui_claude_sdk/utils.py

@@ -300,6 +300,22 @@ def apply_forwarded_props(
     if not forwarded_props or not isinstance(forwarded_props, dict):
         return merged_kwargs
 
+    # Migrate deprecated max_thinking_tokens → thinking config
+    if (
+        "max_thinking_tokens" in forwarded_props
+        and "thinking" not in forwarded_props
+        and forwarded_props["max_thinking_tokens"] is not None
+    ):
+        from claude_agent_sdk.types import ThinkingConfigEnabled
+
+        budget = forwarded_props.pop("max_thinking_tokens")
+        forwarded_props["thinking"] = ThinkingConfigEnabled(budget_tokens=budget)
+        logger.warning(
+            "max_thinking_tokens is deprecated; converted to "
+            "thinking=ThinkingConfigEnabled(budget_tokens=%d)",
+            budget,
+        )
+
     applied_count = 0
     for key, value in forwarded_props.items():
         # Only apply whitelisted keys