Address code review: remove redundant import, document request_id regex

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

Author: Copilot

actions/setup/js/parse_mcp_gateway_log.cjs

@@ -222,7 +222,9 @@ function writeStepSummaryWithTokenUsage(coreObj) {
     for (const rawLine of raw.split("\n")) {
       const line = rawLine.trim();
       if (!line) continue;
-      // Lightweight request_id extraction for deduplication.
+      // Lightweight request_id extraction for deduplication — mirrors extractRequestId() in
+      // parse_token_usage.cjs. The pattern covers standard JSON escaping (\\, \") but not
+      // exotic cases; api-proxy request IDs are UUIDs so this is sufficient in practice.
       const idMatch = line.match(/"request_id"\s*:\s*"((?:\\.|[^"\\])*)"/);
       const dedupeKey = idMatch ? `request_id:${idMatch[1]}` : line;
       if (seenRequestIds.has(dedupeKey)) continue;

actions/setup/js/parse_token_usage.test.cjs

@@ -54,7 +54,6 @@ describe("parse_token_usage", () => {
     });
 
     test("AGENT_USAGE_JSONL_PATH points to agent_usage.jsonl", () => {
-      const { AGENT_USAGE_JSONL_PATH } = require("./parse_token_usage.cjs");
       expect(AGENT_USAGE_JSONL_PATH).toBe("/tmp/gh-aw/agent_usage.jsonl");
     });