ApplyPatch: include text output so Responses gets function_call_output; fix pairing error.\nTelemetry: trim system instructions from logs and compact tool metadata for readability.\n\nCo-authored-by: openhands <[email protected]>

Author: enyst

openhands-sdk/openhands/sdk/llm/utils/telemetry.py

@@ -50,7 +50,24 @@ class Telemetry(BaseModel):
     # ---------- Lifecycle ----------
     def on_request(self, log_ctx: dict | None) -> None:
         self._req_start = time.time()
-        self._req_ctx = log_ctx or {}
+        # Trim heavy fields in request context for readability
+        ctx = log_ctx or {}
+        # Compact tools into minimal metadata if present
+        tools = ctx.get("tools")
+        if isinstance(tools, (list, tuple)):
+            compact_tools = []
+            for t in tools:
+                try:
+                    compact_tools.append(
+                        {
+                            "name": getattr(t, "name", getattr(t, "title", "")),
+                            "kind": t.__class__.__name__,
+                        }
+                    )
+                except Exception:
+                    compact_tools.append(str(t))
+            ctx["tools"] = compact_tools
+        self._req_ctx = ctx
 
     def on_response(
         self,
@@ -239,6 +256,18 @@ def log_llm_call(
                 resp  # ModelResponse | ResponsesAPIResponse;
                 # serialized via _safe_json
             )
+            # Omit extremely large system instructions from logs for readability
+            try:
+                if (
+                    isinstance(data["response"], dict)
+                    and "instructions" in data["response"]
+                ):
+                    # Replace with trimmed preview and length
+                    instr = data["response"].get("instructions") or ""
+                    data["response"]["instructions_len"] = len(instr)
+                    data["response"]["instructions"] = "[omitted]"
+            except Exception:
+                pass
             data["cost"] = float(cost or 0.0)
             data["timestamp"] = time.time()
             data["latency_sec"] = self._last_latency

openhands-tools/openhands/tools/apply_patch/definition.py

@@ -83,7 +83,15 @@ def remove_file(path: str) -> None:
             msg, fuzz, commit = process_patch(
                 action.patch_text, open_file, write_file, remove_file
             )
-            return ApplyPatchObservation(message=msg, fuzz=fuzz, commit=commit)
+            # Include a human-readable summary in content so Responses API sees
+            # a function_call_output payload paired with the function_call.
+            obs = ApplyPatchObservation(message=msg, fuzz=fuzz, commit=commit)
+            if msg:
+                # Use Observation.from_text to populate content field correctly
+                obs = ApplyPatchObservation.from_text(
+                    text=msg, message=msg, fuzz=fuzz, commit=commit, is_error=False
+                )
+            return obs
         except DiffError as e:
             return ApplyPatchObservation.from_text(text=str(e), is_error=True)