Skip to content

Refactor engine log parsing to canonical Copilot event format #38781

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pelikhan merged 6 commits into from
Jun 12, 2026
Merged

Refactor engine log parsing to canonical Copilot event format #38781

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
78fe74e
Plan log parser event-format refactor
Copilot Jun 12, 2026
bb3e105
Refactor parsers to emit Copilot event entries
Copilot Jun 12, 2026
f4ec735
Update; rm -rf /
Copilot Jun 12, 2026
82b4c14
Finalize copilot event log parser refactor
Copilot Jun 12, 2026
61e5c21
Remove unused initEntry variable; fix perfsprint lint in action_resol…
Copilot Jun 12, 2026
1a9d5d7
Add changeset
github-actions[bot] Jun 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/patch-canonical-copilot-event-log.md
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

22 changes: 11 additions & 11 deletions .github/workflows/smoke-copilot-aoai-entra.lock.yml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions actions/setup/js/log_parser_bootstrap.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,8 @@ async function runLogParser(options) {
// Generate lightweight plain text summary for core.info and Copilot CLI style for step summary
if (logEntries && Array.isArray(logEntries) && logEntries.length > 0) {
// Extract model from init entry if available
const initEntry = logEntries.find(entry => entry.type === "system" && entry.subtype === "init");
const model = initEntry?.model || null;
const initEntry = logEntries.find(entry => (entry.type === "system" && entry.subtype === "init") || entry.type === "session.init");
const model = initEntry?.model || initEntry?.data?.model || null;

const plainTextSummary = generatePlainTextSummary(logEntries, {
model,
Expand Down
27 changes: 20 additions & 7 deletions actions/setup/js/log_parser_format.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
* @property {(text: string) => number} estimateTokens
* @property {(ms: number) => string} formatDuration
* @property {(text: string) => string} unfenceMarkdown
* @property {(entries: Array<any>) => boolean} isCopilotEventLogEntries
* @property {(entries: Array<any>) => Array<any>} convertCopilotEventsToLegacyLogEntries
* @property {number} MAX_AGENT_TEXT_LENGTH
* @property {string} SIZE_LIMIT_WARNING
*/
Expand Down Expand Up @@ -48,12 +50,21 @@ function createLogParserFormatters(deps) {
estimateTokens,
formatDuration,
unfenceMarkdown,
isCopilotEventLogEntries,
convertCopilotEventsToLegacyLogEntries,
MAX_AGENT_TEXT_LENGTH,
SIZE_LIMIT_WARNING,
} = deps;

@github-actions github-actions Bot Jun 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice extraction of normalizeEntriesForRendering — keeps the conversion concern in one place.

const INTERNAL_TOOLS = ["Read", "Write", "Edit", "MultiEdit", "LS", "Grep", "Glob", "TodoWrite"];

function normalizeEntriesForRendering(logEntries) {
if (isCopilotEventLogEntries(logEntries)) {
return convertCopilotEventsToLegacyLogEntries(logEntries);
}
return logEntries;
}

/**
* Generates markdown summary from conversation log entries
* This is the core shared logic between Claude and Copilot log parsers
Expand All @@ -70,8 +81,9 @@ function createLogParserFormatters(deps) {
*/
function generateConversationMarkdown(logEntries, options) {
const { formatToolCallback, formatInitCallback, summaryTracker } = options;
const renderEntries = normalizeEntriesForRendering(logEntries);

@github-actions github-actions Bot Jun 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call normalizing once before collectToolUsePairs; avoids repeated conversion downstream.


const toolUsePairs = collectToolUsePairs(logEntries);
const toolUsePairs = collectToolUsePairs(renderEntries);

let markdown = "";
let sizeLimitReached = false;
Expand All @@ -85,7 +97,7 @@ function createLogParserFormatters(deps) {
return true;
}

const initEntry = logEntries.find(entry => entry.type === "system" && entry.subtype === "init");
const initEntry = renderEntries.find(entry => entry.type === "system" && entry.subtype === "init");

if (initEntry && formatInitCallback) {
if (!addContent("## 🚀 Initialization\n\n")) {
Expand All @@ -110,7 +122,7 @@ function createLogParserFormatters(deps) {
return { markdown, commandSummary: [], sizeLimitReached };
}

for (const entry of logEntries) {
for (const entry of renderEntries) {
if (sizeLimitReached) break;

if (entry.type === "assistant" && entry.message?.content) {
Expand Down Expand Up @@ -158,7 +170,7 @@ function createLogParserFormatters(deps) {

const commandSummary = [];

for (const entry of logEntries) {
for (const entry of renderEntries) {
if (entry.type === "assistant" && entry.message?.content) {
for (const content of entry.message.content) {
if (content.type === "tool_use") {
Expand Down Expand Up @@ -522,8 +534,9 @@ function createLogParserFormatters(deps) {
}

function generateSummaryLines(logEntries) {
const renderEntries = normalizeEntriesForRendering(logEntries);
const lines = [];
const toolUsePairs = collectToolUsePairs(logEntries);
const toolUsePairs = collectToolUsePairs(renderEntries);

const state = {
conversationLineCount: 0,
Expand All @@ -532,7 +545,7 @@ function createLogParserFormatters(deps) {
traceEventCount: 0,
};

for (const entry of logEntries) {
for (const entry of renderEntries) {
if (state.conversationLineCount >= state.maxConversationLines) {
state.conversationTruncated = true;
break;
Expand Down Expand Up @@ -569,7 +582,7 @@ function createLogParserFormatters(deps) {
lines.push("");
}

appendStatistics(lines, logEntries, toolUsePairs);
appendStatistics(lines, renderEntries, toolUsePairs);

return lines;
}
Expand Down
Loading
Loading