refactor: extract AWF_INFRA_LINE_RE to log_parser_shared and make status page guidance engine-specific

- Extract INFRA_LINE_RE as AWF_INFRA_LINE_RE into log_parser_shared.cjs
  so parse_copilot_log.cjs and handle_agent_failure.cjs share the same
  constant (no more duplication risk when infra log format changes)
- Make startup-failure status page guidance engine-specific: copilot gets
  "GitHub Copilot status page", all other engines get "provider status page"
- Add test verifying provider-agnostic guidance for claude engine
- Update existing engine ID startup-failure test to also verify Copilot
  status page text

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/de02b350-d94e-4ee6-aceb-0e1f8d2b9ae8

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

Author: Copilot

actions/setup/js/handle_agent_failure.cjs

@@ -10,6 +10,7 @@ const { createExpirationLine, generateFooterWithExpiration } = require("./epheme
 const { MAX_SUB_ISSUES, getSubIssueCount } = require("./sub_issue_helpers.cjs");
 const { formatMissingData } = require("./missing_info_formatter.cjs");
 const { generateHistoryUrl } = require("./generate_history_link.cjs");
+const { AWF_INFRA_LINE_RE } = require("./log_parser_shared.cjs");
 const fs = require("fs");
 const path = require("path");
 
@@ -840,11 +841,9 @@ function buildEngineFailureContext() {
     // the engine itself. They must be filtered out of the fallback tail so the failure
     // context surfaces actual agent output rather than container lifecycle noise
     // (e.g. "Container awf-squid  Removed", "[WARN] Command completed with exit code: 1",
-    // "Process exiting with code: 1"). Uses the same pattern as parse_copilot_log.cjs.
-    // Note: INFO/WARN/SUCCESS/ERROR are uppercase (AWF wrapper convention); entrypoint and
-    // health-check are lowercase (container script convention) — mixed casing is intentional
-    // and reflects the actual log output format produced by different AWF components.
-    const INFRA_LINE_RE = /^\[(INFO|WARN|SUCCESS|ERROR|entrypoint|health-check)\]|^ (?:Container|Network|Volume) |^Process exiting with code:/;
+    // "Process exiting with code: 1"). Shared constant from log_parser_shared.cjs keeps the
+    // pattern in sync with parse_copilot_log.cjs.
+    const INFRA_LINE_RE = AWF_INFRA_LINE_RE;
 
     // Fallback: no known error patterns found — include the last non-empty lines so that
     // failures caused by timeouts or unexpected terminations still surface useful context.
@@ -860,12 +859,14 @@ function buildEngineFailureContext() {
     if (agentLines.length === 0) {
       // The log contains only AWF infrastructure lines — the engine exited before producing
       // any substantive output. This pattern is characteristic of a transient startup failure
-      // (e.g., Copilot API service unavailable, rate-limiting, token not yet provisioned).
+      // (e.g., API service unavailable, rate-limiting, token not yet provisioned).
       core.info("agent-stdio.log contains only infrastructure lines — engine likely failed at startup (possible transient failure)");
+      const recurringFailureGuidance =
+        process.env.GH_AW_ENGINE_ID === "copilot"
+          ? "If this failure recurs, check the GitHub Copilot status page and review the firewall audit logs.\n\n"
+          : "If this failure recurs, check the provider status page (if available) and review the firewall audit logs.\n\n";
       let context = `\n**⚠️ Engine Failure**: The${engineLabel} engine terminated before producing output.\n\n`;
-      context +=
-        "The engine exited immediately without producing any output. This often indicates a transient infrastructure issue (e.g., service unavailable, API rate limiting). " +
-        "If this failure recurs, check the GitHub Copilot status page and review the firewall audit logs.\n\n";
+      context += "The engine exited immediately without producing any output. This often indicates a transient infrastructure issue (e.g., service unavailable, API rate limiting). " + recurringFailureGuidance;
       return context;
     }
 

actions/setup/js/handle_agent_failure.test.cjs

@@ -662,6 +662,22 @@ describe("handle_agent_failure", () => {
       const result = buildEngineFailureContext();
       expect(result).toContain("`copilot` engine");
       expect(result).toContain("terminated before producing output");
+      // Copilot-specific status page guidance
+      expect(result).toContain("GitHub Copilot status page");
+    });
+
+    it("shows provider-agnostic status page guidance for non-copilot engines", () => {
+      process.env.GH_AW_ENGINE_ID = "claude";
+      vi.resetModules();
+      ({ buildEngineFailureContext } = require("./handle_agent_failure.cjs"));
+      const infraLines = ["[WARN] Command completed with exit code: 1", "Process exiting with code: 1"];
+      fs.writeFileSync(stdioLogPath, infraLines.join("\n") + "\n");
+      const result = buildEngineFailureContext();
+      expect(result).toContain("`claude` engine");
+      expect(result).toContain("terminated before producing output");
+      // Generic guidance for non-copilot engines
+      expect(result).toContain("provider status page");
+      expect(result).not.toContain("GitHub Copilot status page");
     });
 
     it("includes engine ID in failure message when GH_AW_ENGINE_ID is set", () => {

actions/setup/js/log_parser_shared.cjs

@@ -43,6 +43,31 @@ const MAX_AGENT_TEXT_LENGTH = 2000;
  */
 const SIZE_LIMIT_WARNING = "\n\n⚠️ *Step summary size limit reached. Additional content truncated.*\n\n";
 
+/**
+ * Matches AWF infrastructure lines written by the firewall/container wrapper.
+ * These lines are produced by the AWF infrastructure (container lifecycle, firewall proxy)
+ * rather than by the engine itself, and must be excluded when analysing agent output.
+ *
+ * Examples of matched lines:
+ *   - [INFO] API proxy logs available at: …
+ *   - [WARN] Command completed with exit code: 1
+ *   - [SUCCESS] Containers stopped successfully
+ *   - [ERROR] …
+ *   - [entrypoint] Starting firewall…       (lowercase — container script convention)
+ *   - [health-check] Proxy ready            (lowercase — container script convention)
+ *   -  Container awf-squid  Removed         (Docker Compose lifecycle output)
+ *   -  Network …  Removed
+ *   - Process exiting with code: 1          (AWF wrapper exit line)
+ *
+ * Note: INFO/WARN/SUCCESS/ERROR are uppercase (AWF wrapper convention); entrypoint and
+ * health-check are lowercase (container script convention). Mixed casing is intentional
+ * and reflects the actual output produced by different AWF components.
+ *
+ * Used by parse_copilot_log.cjs (parsePrettyPrintFormat) and handle_agent_failure.cjs
+ * (buildEngineFailureContext) to strip infrastructure noise from engine log analysis.
+ */
+const AWF_INFRA_LINE_RE = /^\[(INFO|WARN|SUCCESS|ERROR|entrypoint|health-check)\]|^ (?:Container|Network|Volume) |^Process exiting with code:/;
+
 /**
  * Tracks the size of content being added to a step summary.
  * Used to prevent exceeding GitHub Actions step summary size limits.
@@ -1650,6 +1675,7 @@ module.exports = {
   // Constants
   MAX_TOOL_OUTPUT_LENGTH,
   MAX_STEP_SUMMARY_SIZE,
+  AWF_INFRA_LINE_RE,
   // Classes
   StepSummaryTracker,
   // Functions

actions/setup/js/parse_copilot_log.cjs

@@ -1,7 +1,7 @@
 // @ts-check
 /// <reference types="@actions/github-script" />
 
-const { createEngineLogParser, generateConversationMarkdown, generateInformationSection, formatInitializationSummary, formatToolUse, parseLogEntries } = require("./log_parser_shared.cjs");
+const { createEngineLogParser, generateConversationMarkdown, generateInformationSection, formatInitializationSummary, formatToolUse, parseLogEntries, AWF_INFRA_LINE_RE } = require("./log_parser_shared.cjs");
 const { ERR_PARSE } = require("./error_codes.cjs");
 
 const main = createEngineLogParser({
@@ -153,7 +153,7 @@ function parsePrettyPrintFormat(logContent) {
     return [];
   }
 
-  const INFRA_LINE_RE = /^\[(INFO|WARN|SUCCESS|ERROR|entrypoint|health-check)\]|^ (?:Container|Network|Volume) |^Process exiting with code:/;
+  const INFRA_LINE_RE = AWF_INFRA_LINE_RE;
   const FAILED_TOOL_RE = /^✗\s+(\S+)/;
   const SUCCESS_TOOL_RE = /^(?:●|✓)\s+(\S+)/;
   const CONTINUATION_RE = /^\s+[└│]/;