@q-agent/agent 0.1.27 → 0.1.29

package/dist/src/playwrightConfig.js

@@ -54,7 +54,7 @@ export default defineConfig({
   reporter: __REPORTERS__,
   use: {
     headless: __HEADLESS__,
-    screenshot: 'only-on-failure',
+    screenshot: 'on',
     video: '__VIDEO__',
     trace: '__TRACE__',
 __EXTRA_USE__  },
@@ -72,7 +72,7 @@ __EXTRA_USE__  },
  * @param opts Heal-tuning knobs (timeouts + evidence); defaults match a normal run.
  */
 function writeConfig(specDir, workers, headless, baseUrl = "", storageState = "", opts = {}) {
-    const { testTimeoutMs = 30000, actionTimeoutMs, heavyEvidence = true, liveReporterPath } = opts;
+    const { testTimeoutMs = 30000, actionTimeoutMs, heavyEvidence = true, captureVideo = false, liveReporterPath } = opts;
     // JSON reporter (drives report.json for evidence + reconcile) + optionally the
     // live reporter (streams per-test results). Paths JSON-escaped for Windows.
     const reporters = ["['json', { outputFile: 'report.json' }]"];
@@ -89,13 +89,16 @@ function writeConfig(specDir, workers, headless, baseUrl = "", storageState = ""
         extraLines.push(`    navigationTimeout: ${Math.trunc(actionTimeoutMs)},`);
     }
     const extraUse = extraLines.length ? extraLines.join("\n") + "\n" : "";
-    const retain = heavyEvidence ? "retain-on-failure" : "off";
+    const trace = heavyEvidence ? "retain-on-failure" : "off";
+    // Video follows the "Capture video" setting (always-on when enabled), not the
+    // failure-only trace policy — but intermediate heal attempts still skip it.
+    const video = captureVideo && heavyEvidence ? "on" : "off";
     const content = PLAYWRIGHT_CONFIG_TEMPLATE.replace("__TIMEOUT__", String(Math.trunc(testTimeoutMs)))
         .replace("__WORKERS__", String(workers))
         .replace("__HEADLESS__", headless ? "true" : "false")
         .replace("__REPORTERS__", reportersStr)
-        .replace("__VIDEO__", retain)
-        .replace("__TRACE__", retain)
+        .replace("__VIDEO__", video)
+        .replace("__TRACE__", trace)
         .replace("__EXTRA_USE__", extraUse);
     fs.writeFileSync(path.join(specDir, "playwright.config.ts"), content, "utf-8");
 }
@@ -151,6 +154,15 @@ function fixturesTs(sessionFile, replaySession, captureRaw = true) {
         "  // Always-on DOM capture: after each test, snapshot the live page so the\n" +
         "  // runner (evidence) and self-heal loop (real selectors) can use it.\n" +
         "  _domCapture: [async ({ page }, use, testInfo) => {\n" +
+        "    // Console + network capture (#456): collect for the whole test, pass or\n" +
+        "    // fail. Listeners registered BEFORE use() so nothing is missed; the JSON\n" +
+        "    // is attached after and parsed into console_logs/network_logs columns.\n" +
+        "    const __net: any[] = []; const __con: any[] = []; const __started = new Map();\n" +
+        "    page.on('request', (r) => { try { __started.set(r, Date.now()); } catch {} });\n" +
+        "    page.on('response', (resp) => { try { if (__net.length < 300) { const req = resp.request(); const t0 = __started.get(req); __net.push({ method: req.method(), url: req.url(), status: resp.status(), durationMs: t0 ? Date.now() - t0 : 0 }); } } catch {} });\n" +
+        "    page.on('requestfailed', (r) => { try { if (__net.length < 300) { const t0 = __started.get(r); __net.push({ method: r.method(), url: r.url(), status: 0, durationMs: t0 ? Date.now() - t0 : 0, failed: true }); } } catch {} });\n" +
+        "    page.on('console', (msg) => { try { if (__con.length < 500) __con.push({ level: msg.type(), text: String(msg.text()).slice(0, 2000) }); } catch {} });\n" +
+        "    page.on('pageerror', (err) => { try { if (__con.length < 500) __con.push({ level: 'error', text: String((err && err.message) || err).slice(0, 2000) }); } catch {} });\n" +
         "    await use();\n" +
         "    // Distilled inventory FIRST (what self-heal needs) — retry once after a\n" +
         "    // short settle so a transiently-busy failure page still yields real\n" +
@@ -188,6 +200,16 @@ function fixturesTs(sessionFile, replaySession, captureRaw = true) {
         "      } catch {}\n" +
         "    }\n" +
         rawBlock +
+        "    try {\n" +
+        "      const netPath = testInfo.outputPath('qagent-network.json');\n" +
+        "      fs.writeFileSync(netPath, JSON.stringify(__net), 'utf-8');\n" +
+        "      await testInfo.attach('qagent-network', { path: netPath, contentType: 'application/json' });\n" +
+        "    } catch {}\n" +
+        "    try {\n" +
+        "      const conPath = testInfo.outputPath('qagent-console.json');\n" +
+        "      fs.writeFileSync(conPath, JSON.stringify(__con), 'utf-8');\n" +
+        "      await testInfo.attach('qagent-console', { path: conPath, contentType: 'application/json' });\n" +
+        "    } catch {}\n" +
         "  }, { auto: true }],\n" +
         "});\n" +
         "\n" +

package/dist/src/report.js

@@ -32,6 +32,10 @@ const ATTACHMENT_KIND_MAP = {
     // raw page HTML + a distilled interactable-element inventory.
     "qagent-dom-raw": "dom",
     "qagent-dom-distilled": "dom-distilled",
+    // Console + network captured by the fixtures (#456): uploaded via the evidence
+    // path but decoded server-side into console_logs/network_logs (not media rows).
+    "qagent-console": "console",
+    "qagent-network": "network",
 };
 /* eslint-enable @typescript-eslint/no-explicit-any */
 /**

package/dist/src/runner.js

@@ -258,6 +258,7 @@ async function processJob(cfg, job) {
         const { storageState, sessionStoragePath } = auth;
         (0, playwrightConfig_1.writeConfig)(workDir, job.workers, job.headless, job.baseUrl, storageState, {
             liveReporterPath: (0, paths_1.vendorLiveReporter)(),
+            captureVideo: Boolean(job.captureVideo),
         });
         // Always inject the generated fixtures.ts (DOM capture on every run) + rewrite
         // spec imports to it. sessionStorage replay (MSAL/SPA tokens) is additionally
@@ -398,6 +399,13 @@ async function processJob(cfg, job) {
  * the workDir is always removed, even if some uploads fail or time out.
  */
 async function uploadEvidenceThenCleanup(cfg, executionId, pendingEvidence, workDir) {
+    // Tell the UI evidence is still incoming so it can show a loader instead of an
+    // empty panel while these deferred uploads land (results are already visible).
+    if (pendingEvidence.length > 0) {
+        await api
+            .postEvent(cfg, executionId, "exec.evidence.uploading", { count: pendingEvidence.length })
+            .catch(() => { });
+    }
     try {
         for (const ev of pendingEvidence) {
             await api
@@ -413,6 +421,13 @@ async function uploadEvidenceThenCleanup(cfg, executionId, pendingEvidence, work
     }
     finally {
         fs.rmSync(workDir, { recursive: true, force: true });
+        // Signal completion so the UI drops the loader and refetches the now-uploaded
+        // evidence — always fire (even on partial failure) so the loader never sticks.
+        if (pendingEvidence.length > 0) {
+            await api
+                .postEvent(cfg, executionId, "exec.evidence.done", {})
+                .catch(() => { });
+        }
     }
 }
 /**
@@ -507,6 +522,7 @@ async function processHealJob(cfg, job, heal) {
                     testTimeoutMs: heal.testTimeoutMs,
                     actionTimeoutMs: heal.actionTimeoutMs,
                     heavyEvidence: isFinalAttempt,
+                    captureVideo: Boolean(job.captureVideo),
                 });
                 const replaySession = Boolean(job.manualAuth && storageState && sessionStoragePath && fs.statSync(sessionStoragePath).size > 0);
                 (0, playwrightConfig_1.applyFixtures)(workDir, [spec.filename], sessionStoragePath || path.join(workDir, "sessionStorage.json"), replaySession, isFinalAttempt);
@@ -978,7 +994,7 @@ async function processAuthoringJob(cfg, job) {
     }
     const sess = origin ? (0, session_1.sessionPathsForOrigin)(origin) : null;
     const profileDir = sess ? path.join(sess.dir, "browser-profile") : "";
-    const finalize = async (code, discovered, summary, ok, costUsd = 0) => {
+    const finalize = async (code, discovered, summary, ok, costUsd = 0, refreshedCredentials = "") => {
         await api
             .postAuthoringFinalize(cfg, job.sessionId, {
             code,
@@ -986,6 +1002,7 @@ async function processAuthoringJob(cfg, job) {
             summary,
             ok,
             costUsd,
+            refreshedCredentials,
         })
             .catch((err) => console.error("postAuthoringFinalize failed:", err));
     };
@@ -1195,7 +1212,19 @@ async function processAuthoringJob(cfg, job) {
             case: job.caseId, phase: ok ? "done" : "failed", message: (summary || "").slice(0, 300), costUsd,
         })
             .catch(() => { });
-        await finalize(code, discovered, summary, ok, costUsd);
+        // Auto-rotation (#cred-rotate): if we ran with the app's saved credential and
+        // the CLI refreshed the OAuth token in our temp config dir, post the rotated
+        // .credentials.json back so the server captures it before workDir is deleted.
+        let refreshedCredentials = "";
+        if (claudeConfigDir) {
+            try {
+                const credFile = path.join(claudeConfigDir, ".credentials.json");
+                if (fs.existsSync(credFile))
+                    refreshedCredentials = fs.readFileSync(credFile, "utf-8");
+            }
+            catch { /* best-effort — a missing/unreadable file just skips rotation */ }
+        }
+        await finalize(code, discovered, summary, ok, costUsd, refreshedCredentials);
     }
     finally {
         try {

package/dist/test/playwrightConfig.test.js

@@ -60,6 +60,32 @@ const playwrightConfig_1 = require("../src/playwrightConfig");
     strict_1.default.ok(noReplay.includes("testInfo.attach('qagent-dom-distilled'"));
     strict_1.default.ok(!noReplay.includes("addInitScript"), "replay=false drops the session init script");
 });
+(0, node_test_1.test)("fixturesTs captures console + network on every test (#456)", () => {
+    const fx = (0, playwrightConfig_1.fixturesTs)("/tmp/sessionStorage.json", false);
+    strict_1.default.ok(fx.includes("testInfo.attach('qagent-network'"));
+    strict_1.default.ok(fx.includes("testInfo.attach('qagent-console'"));
+    strict_1.default.ok(fx.includes("page.on('response'"), "network responses are collected");
+    strict_1.default.ok(fx.includes("page.on('console'"), "console messages are collected");
+});
+(0, node_test_1.test)("writeConfig: screenshot always-on; video honors the setting (#456)", () => {
+    const dir = fs.mkdtempSync(path.join(os.tmpdir(), "qa-cfg-"));
+    const read = () => fs.readFileSync(path.join(dir, "playwright.config.ts"), "utf-8");
+    // Screenshot is always 'on' now (captured pass or fail).
+    (0, playwrightConfig_1.writeConfig)(dir, 1, true);
+    strict_1.default.ok(read().includes("screenshot: 'on'"));
+    // Capture-video ON → video 'on' regardless of pass/fail; trace stays failure-only.
+    (0, playwrightConfig_1.writeConfig)(dir, 1, true, "", "", { captureVideo: true });
+    strict_1.default.ok(read().includes("video: 'on'"));
+    strict_1.default.ok(read().includes("trace: 'retain-on-failure'"));
+    // Capture-video OFF → no video.
+    (0, playwrightConfig_1.writeConfig)(dir, 1, true, "", "", { captureVideo: false });
+    strict_1.default.ok(read().includes("video: 'off'"));
+    // Intermediate heal attempts (heavyEvidence=false) never record video even when enabled.
+    (0, playwrightConfig_1.writeConfig)(dir, 1, true, "", "", { captureVideo: true, heavyEvidence: false });
+    strict_1.default.ok(read().includes("video: 'off'"));
+    strict_1.default.ok(read().includes("trace: 'off'"));
+    fs.rmSync(dir, { recursive: true, force: true });
+});
 (0, node_test_1.test)("applyFixtures always rewrites imports to './fixtures' + writes fixtures.ts", () => {
     const dir = fs.mkdtempSync(path.join(os.tmpdir(), "qa-fx-"));
     const specName = "1428-TC-01.spec.ts";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@q-agent/agent",
-  "version": "0.1.27",
+  "version": "0.1.29",
   "description": "Q-Agent Local Agent — claims execution jobs from the Q-Agent server and runs Playwright locally, so manual login/MFA happens on the user's own machine and session credentials never leave it.",
   "license": "UNLICENSED",
   "bin": {