Render Copilot CLI conversation from events.jsonl, not raw debug logs

- Revert extractWireRequestToolResults (raw [DEBUG] Wire request: parsing)
- Fix convertCopilotEventsToLegacyLogEntries to extract tool output from
  data.result.content (native Copilot CLI events.jsonl format)
- Fix error message extraction to use data.error.message (not String(obj))
- Update copilot_sdk_session.cjs to include result in tool.execution_complete
  events so SDK-based sessions also get tool output previews in events.jsonl
- Add test: renders tool output preview from result.content in events.jsonl

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Author: Copilot

actions/setup/js/copilot_sdk_session.cjs

@@ -10,7 +10,7 @@
  * Event mapping:
  *   SDK "user.message"            → JSONL "user.message"
  *   SDK "tool.execution_start"    → JSONL "tool.execution_start"  (toolName, mcpServerName)
- *   SDK "tool.execution_complete" → JSONL "tool.execution_complete" (toolName, mcpServerName, success)
+ *   SDK "tool.execution_complete" → JSONL "tool.execution_complete" (toolName, mcpServerName, success, result)
  *   SDK "assistant.message"       → JSONL "assistant.message"     (content)
  *
  * The JSONL file is written to:
@@ -265,9 +265,12 @@ async function runWithCopilotSDK({ sdkUri, prompt, logger, attempt = 0, model, c
           const mcpServerName = pending?.mcpServerName ?? "";
           if (toolCallId) pendingToolCalls.delete(toolCallId);
           const success = event.data?.success ?? !event.data?.error;
+          // Include result.content (concise LLM-facing output) so that the log
+          // parser can render tool output previews from events.jsonl directly.
+          const result = event.data?.result ?? undefined;
           // max-tool-denials intentionally tracks permission denials only.
           // Tool execution failures are still logged, but do not increment the guardrail counter.
-          writeEvent("tool.execution_complete", { toolName, mcpServerName, success }, event.timestamp);
+          writeEvent("tool.execution_complete", { toolName, mcpServerName, success, result }, event.timestamp);
           break;
         }
 

actions/setup/js/log_parser_shared.cjs

@@ -928,8 +928,12 @@ function convertCopilotEventsToLegacyLogEntries(logEntries) {
           output = data.output;
         } else if (typeof data.result === "string") {
           output = data.result;
+        } else if (data.result && typeof data.result.content === "string") {
+          // Native Copilot CLI events.jsonl format: result.content is the concise
+          // tool result text sent to the LLM (may be truncated for token efficiency).
+          output = data.result.content;
         } else if (data.error) {
-          output = String(data.error);
+          output = typeof data.error === "object" && typeof data.error.message === "string" ? data.error.message : String(data.error);
         } else if (success) {
           output = "success";
         } else {

actions/setup/js/parse_copilot_log.cjs

@@ -495,97 +495,6 @@ function scanForToolErrors(logContent) {
   return toolErrors;
 }
 
-/**
- * Extracts actual tool output content from Wire request blocks in Copilot CLI debug logs.
- * Each Wire request contains the full conversation history up to that point, including
- * tool call results from prior turns. Parsing these allows us to populate tool result
- * previews instead of showing empty content.
- * @param {string[]} lines - Log lines (pre-split)
- * @returns {Map<string, string>} Map from tool_call_id to tool output content string
- */
-function extractWireRequestToolResults(lines) {
-  // Maximum number of lines to look ahead within a tool entry for tool_call_id / content
-  const MAX_TOOL_RESULT_SCAN_LINES = 5;
-
-  const toolResultMap = new Map();
-  let inWireBlock = false;
-
-  for (let i = 0; i < lines.length; i++) {
-    const line = lines[i];
-
-    // Wire request starts: a timestamped line containing "[DEBUG] Wire request:"
-    if (/^\d{4}-\d{2}-\d{2}T[\d:.]+Z \[DEBUG\] Wire request:/.test(line)) {
-      inWireBlock = true;
-      continue;
-    }
-
-    if (inWireBlock) {
-      // Wire request body lines have no timestamp prefix; a new timestamped line ends the block
-      if (/^\d{4}-\d{2}-\d{2}T[\d:.]+Z /.test(line)) {
-        inWireBlock = false;
-        continue;
-      }
-
-      // Detect tool result entries by role
-      if (line.includes('"role": "tool"')) {
-        let toolCallId = null;
-        let contentValue = null;
-
-        // The next few lines contain tool_call_id and content in order
-        for (let j = i + 1; j < Math.min(i + MAX_TOOL_RESULT_SCAN_LINES, lines.length); j++) {
-          const nextLine = lines[j];
-
-          // Stop if we hit a timestamp line (end of wire block)
-          if (/^\d{4}-\d{2}-\d{2}T[\d:.]+Z /.test(nextLine)) break;
-
-          if (toolCallId === null) {
-            const toolCallIdIdx = nextLine.indexOf('"tool_call_id":');
-            if (toolCallIdIdx >= 0) {
-              // Use JSON.parse to correctly handle any escape sequences in the ID value
-              const rest = nextLine
-                .slice(toolCallIdIdx + '"tool_call_id":'.length)
-                .trim()
-                .replace(/,\s*$/, "");
-              try {
-                const parsed = JSON.parse(rest);
-                if (typeof parsed === "string") toolCallId = parsed;
-              } catch (e) {
-                // Skip unparseable ID
-              }
-            }
-          }
-
-          if (contentValue === null) {
-            const contentIdx = nextLine.indexOf('"content":');
-            if (contentIdx >= 0) {
-              // Parse the JSON string value to correctly handle escape sequences
-              const rest = nextLine
-                .slice(contentIdx + '"content":'.length)
-                .trim()
-                .replace(/,\s*$/, "");
-              try {
-                const parsed = JSON.parse(rest);
-                if (typeof parsed === "string") contentValue = parsed;
-              } catch (e) {
-                // Skip non-string content (e.g. null or array)
-              }
-            }
-          }
-
-          if (toolCallId !== null && contentValue !== null) break;
-        }
-
-        // Wire requests repeat history across turns; only store the first occurrence of each ID
-        if (toolCallId !== null && contentValue !== null && !toolResultMap.has(toolCallId)) {
-          toolResultMap.set(toolCallId, contentValue);
-        }
-      }
-    }
-  }
-
-  return toolResultMap;
-}
-
 /**
  * Parses Copilot CLI debug log format and reconstructs the conversation flow
  * @param {string} logContent - Raw debug log content
@@ -1053,22 +962,6 @@ function parseDebugLogFormat(logContent) {
     }
   }
 
-  // Populate tool result content from Wire request blocks.
-  // Wire requests contain the full conversation history including actual tool outputs,
-  // which we use to fill in the preview lines for each tool call.
-  const wireToolResults = extractWireRequestToolResults(lines);
-  if (wireToolResults.size > 0) {
-    for (const entry of entries) {
-      if (entry.type === "user" && entry.message?.content) {
-        for (const content of entry.message.content) {
-          if (content.type === "tool_result" && !content.is_error && wireToolResults.has(content.tool_use_id)) {
-            content.content = wireToolResults.get(content.tool_use_id);
-          }
-        }
-      }
-    }
-  }
-
   // Add system init entry at the beginning if we have entries
   if (entries.length > 0) {
     const initEntry = {
@@ -1105,6 +998,5 @@ if (typeof module !== "undefined" && module.exports) {
     main,
     parseCopilotLog,
     parsePrettyPrintFormat,
-    extractWireRequestToolResults,
   };
 }

actions/setup/js/parse_copilot_log.test.cjs

@@ -4,7 +4,7 @@ import path from "path";
 
 describe("parse_copilot_log.cjs", () => {
   let mockCore, originalConsole, originalProcess;
-  let main, parseCopilotLog, extractWireRequestToolResults;
+  let main, parseCopilotLog;
 
   beforeEach(async () => {
     originalConsole = global.console;
@@ -45,7 +45,6 @@ describe("parse_copilot_log.cjs", () => {
     const module = await import("./parse_copilot_log.cjs?" + Date.now());
     main = module.main;
     parseCopilotLog = module.parseCopilotLog;
-    extractWireRequestToolResults = module.extractWireRequestToolResults;
   });
 
   afterEach(() => {
@@ -125,6 +124,20 @@ describe("parse_copilot_log.cjs", () => {
       expect(resultData?.numTurns).toBe(1);
     });
 
+    it("renders tool output preview from result.content in Copilot CLI events.jsonl", () => {
+      const eventsLog = [
+        '{"type":"user.message","timestamp":"2026-06-05T00:44:01.367Z","data":{}}',
+        '{"type":"tool.execution_start","timestamp":"2026-06-05T00:44:04.520Z","data":{"toolName":"bash","mcpServerName":""}}',
+        '{"type":"tool.execution_complete","timestamp":"2026-06-05T00:44:04.700Z","data":{"toolName":"bash","mcpServerName":"","success":true,"result":{"content":"file1.txt\\nfile2.txt\\nfile3.txt"}}}',
+        '{"type":"assistant.message","timestamp":"2026-06-05T00:44:59.769Z","data":{"content":"Done"}}',
+      ].join("\n");
+
+      const result = parseCopilotLog(eventsLog);
+
+      expect(result.markdown).toContain("bash");
+      expect(result.markdown).toContain("file1.txt");
+    });
+
     it("should handle tool calls with details in HTML format", () => {
       const logWithHtmlDetails = JSON.stringify([
         { type: "system", subtype: "init", session_id: "html-test", tools: ["Bash"], model: "gpt-5" },
@@ -503,126 +516,4 @@ describe("parse_copilot_log.cjs", () => {
       expect(result.markdown).toContain("safe_outputs::create_issue");
     });
   });
-
-  describe("extractWireRequestToolResults function", () => {
-    it("should extract tool results from Wire request blocks", () => {
-      const lines = [
-        "2026-06-18T16:00:00.000Z [DEBUG] Wire request: {",
-        '  "messages": [',
-        '    { "role": "user", "content": "list files" },',
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_abc123",',
-        '      "content": "file1.txt\\nfile2.txt\\nfile3.txt"',
-        "    }",
-        "  ]",
-        "}",
-        "2026-06-18T16:00:01.000Z [INFO] Done",
-      ];
-      const result = extractWireRequestToolResults(lines);
-      expect(result.get("tooluse_abc123")).toBe("file1.txt\nfile2.txt\nfile3.txt");
-    });
-
-    it("should skip non-string content values (e.g. null)", () => {
-      const lines = [
-        "2026-06-18T16:00:00.000Z [DEBUG] Wire request: {",
-        '  "messages": [',
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_nullcontent",',
-        '      "content": null',
-        "    }",
-        "  ]",
-        "}",
-        "2026-06-18T16:00:01.000Z [INFO] Done",
-      ];
-      const result = extractWireRequestToolResults(lines);
-      expect(result.has("tooluse_nullcontent")).toBe(false);
-    });
-
-    it("should only store the first occurrence of a tool_call_id (from earlier Wire request)", () => {
-      const lines = [
-        // First Wire request: only tool A
-        "2026-06-18T16:00:00.000Z [DEBUG] Wire request: {",
-        '  "messages": [',
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_A",',
-        '      "content": "result A"',
-        "    }",
-        "  ]",
-        "}",
-        // Second Wire request: tool A repeated plus tool B
-        "2026-06-18T16:00:01.000Z [DEBUG] Wire request: {",
-        '  "messages": [',
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_A",',
-        '      "content": "result A again"',
-        "    },",
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_B",',
-        '      "content": "result B"',
-        "    }",
-        "  ]",
-        "}",
-        "2026-06-18T16:00:02.000Z [INFO] Done",
-      ];
-      const result = extractWireRequestToolResults(lines);
-      expect(result.get("tooluse_A")).toBe("result A");
-      expect(result.get("tooluse_B")).toBe("result B");
-    });
-
-    it("should return an empty map when no Wire request blocks are present", () => {
-      const lines = ["2026-06-18T16:00:00.000Z [DEBUG] data:", "2026-06-18T16:00:00.001Z [DEBUG] {", '  "choices": []', "}", "2026-06-18T16:00:01.000Z [INFO] Done"];
-      const result = extractWireRequestToolResults(lines);
-      expect(result.size).toBe(0);
-    });
-  });
-
-  describe("debug log format tool result previews", () => {
-    it("should show tool output preview lines from Wire request blocks", () => {
-      const debugLog = [
-        "2026-06-18T16:00:00.000Z [INFO] Starting Copilot CLI: 0.0.412",
-        "2026-06-18T16:00:01.000Z [DEBUG] data:",
-        "2026-06-18T16:00:01.001Z [DEBUG] {",
-        '  "model": "claude-sonnet-4.6",',
-        '  "usage": { "prompt_tokens": 100, "completion_tokens": 50 },',
-        '  "choices": [',
-        "    {",
-        '      "message": {',
-        '        "content": null,',
-        '        "tool_calls": [',
-        "          {",
-        '            "id": "tooluse_abc123",',
-        '            "type": "function",',
-        '            "function": { "name": "bash", "arguments": "{\\"command\\": \\"ls /tmp\\"}" }',
-        "          }",
-        "        ]",
-        "      }",
-        "    }",
-        "  ]",
-        "}",
-        "2026-06-18T16:00:02.000Z [DEBUG] Wire request: {",
-        '  "messages": [',
-        "    {",
-        '      "role": "tool",',
-        '      "tool_call_id": "tooluse_abc123",',
-        '      "content": "file1.txt\\nfile2.txt\\nfile3.txt"',
-        "    }",
-        "  ]",
-        "}",
-        "2026-06-18T16:00:03.000Z [INFO] Done",
-      ].join("\n");
-
-      const result = parseCopilotLog(debugLog);
-
-      // Tool call should be shown
-      expect(result.markdown).toContain("ls /tmp");
-      // Preview lines from Wire request should be shown
-      expect(result.markdown).toContain("file1.txt");
-      expect(result.markdown).toContain("file2.txt");
-    });
-  });
 });