@q-agent/agent 0.1.26 → 0.1.28

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
@@ -507,6 +508,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);
@@ -1100,10 +1102,17 @@ async function processAuthoringJob(cfg, job) {
         // (#420), so abort the local Claude run immediately instead of burning the
         // rest of the budget on work whose result will be rejected anyway.
         let aborted = false;
+        // AGENT LOG mirrors the web trail's Settings verbosity (#438): in "concise"
+        // (default) skip the raw tool/Bash step lines (▷ …) so the local log shows
+        // only Claude's readable narration. The server WS still gets every line — the
+        // web filters its own stream — and abort-on-stop keys on those posts.
+        const conciseLog = (job.logVerbosity ?? "concise") === "concise";
         const emitStep = (line) => {
             const trimmed = line.length > 300 ? line.slice(0, 300) + "…" : line;
             console.log(`[authoring ${job.caseId}] ${trimmed}`);
-            (0, bus_1.emit)("authoring-step", { caseId: job.caseId, line: trimmed });
+            if (!(conciseLog && trimmed.trimStart().startsWith("▷"))) {
+                (0, bus_1.emit)("authoring-step", { caseId: job.caseId, line: trimmed });
+            }
             void api
                 .postAuthoringEventAlive(cfg, job.sessionId, "authoring.progress", {
                 case: job.caseId, phase: "step", message: trimmed,

package/dist/src/ui.js

@@ -497,6 +497,7 @@ function renderCells(){
 }
 
 function timeStr(ts){ return new Date(ts).toTimeString().slice(0,8); }
+var LOG_EMPTY_HTML = '<div id="logEmptyPh" style="color:#5c5c6e;padding:2px 0;line-height:1.7">Ready \\u00b7 waiting for activity\\u2026<br><span style="color:#4c4c5a;font-size:10.5px">Runs, live-authoring and self-heal will stream here.</span></div>';
 function logColor(t){
   if (t === "error" || t === "auth-error") return "#ff9ea1";
   if (t === "auth-waiting") return "#fbbf24";
@@ -524,6 +525,7 @@ function logMsg(ev){
 function addLog(ev){
   if (ev.type === "agent-status") return; // status events aren't log lines; pill is driven by view state
   var msg = logMsg(ev); if (!msg) return;
+  var ph = document.getElementById("logEmptyPh"); if (ph) ph.remove(); // first real line clears the placeholder
   var row = document.createElement("div");
   row.style.cssText = "display:flex;gap:10px;animation:logIn .4s ease both";
   var t = document.createElement("span"); t.style.cssText = "color:#5c5c6e;flex-shrink:0"; t.textContent = timeStr(ev.ts);
@@ -582,7 +584,9 @@ function showConnected(s){
   el("sessionId").textContent = "device #" + (s.deviceId==null?"?":s.deviceId);
   // The SSE stream replays the recent-events buffer on connect, so seed the log
   // ONLY from the stream (seeding from /api/state too would double every line).
-  el("log").innerHTML = ""; openStream();
+  // Show a placeholder until the first line arrives so an idle/just-reconnected
+  // agent reads as "ready & waiting", not a broken empty panel (#agent-log-empty).
+  el("log").innerHTML = LOG_EMPTY_HTML; openStream();
 }
 function refresh(){
   fetch("/api/state").then(function(r){ return r.json(); }).then(function(s){

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.26",
+  "version": "0.1.28",
   "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": {