@q-agent/agent 0.1.1 → 0.1.7

package/dist/src/version.js

@@ -0,0 +1,61 @@
+"use strict";
+/**
+ * The running agent's version.
+ *
+ * Read from the package manifest at runtime (rather than a compile-time import)
+ * so both the `npx @q-agent/agent` package and the packaged desktop bundle report
+ * the *actual* installed build — the number one signal when diagnosing a stale
+ * `npx` cache serving an old agent. Compiled output lives at `dist/src/`, so the
+ * manifest is two levels up. Any failure falls back to `"unknown"` — a missing
+ * version must never crash the CLI.
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.agentVersion = agentVersion;
+const path = __importStar(require("path"));
+let cached = null;
+/** The agent's semver (e.g. "0.1.6"), or "unknown" if the manifest can't be read. */
+function agentVersion() {
+    if (cached !== null)
+        return cached;
+    try {
+        const manifest = require(path.join(__dirname, "..", "..", "package.json"));
+        cached = manifest.version || "unknown";
+    }
+    catch {
+        cached = "unknown";
+    }
+    return cached;
+}

package/dist/test/api.test.js

@@ -135,6 +135,44 @@ function mockFetch(handler) {
     const body = JSON.parse(calls[0].init.body);
     strict_1.default.deepEqual(body, { passed: 3, failed: 1, log: "tail" });
 });
+(0, node_test_1.test)("postHealFix starts a job then polls /agent/heal/{caseId}/fix/{jobId} for the action", async () => {
+    // Async flow (#313): POST starts the job (returns jobId), GET polls until done.
+    mockFetch((url, init) => {
+        if (init.method === "POST")
+            return Response.json({ jobId: "job-1", status: "running" });
+        return Response.json({ status: "done", result: { action: "fixed", code: "// fixed", diff: "@@" } });
+    });
+    const out = await api.postHealFix(cfg, 99, {
+        currentCode: "// old", error: "boom", output: "tail", domDistilled: { path: "/x" }, attempt: 2,
+    });
+    // First call: POST to start with the attempt body.
+    strict_1.default.equal(calls[0].url, "http://127.0.0.1:8787/agent/heal/99/fix");
+    strict_1.default.equal(calls[0].init.method, "POST");
+    const body = JSON.parse(calls[0].init.body);
+    strict_1.default.deepEqual(body, { currentCode: "// old", error: "boom", output: "tail", domDistilled: { path: "/x" }, attempt: 2 });
+    // Second call: GET poll on the returned job id.
+    strict_1.default.equal(calls[1].url, "http://127.0.0.1:8787/agent/heal/99/fix/job-1");
+    strict_1.default.equal(calls[1].init.method, "GET");
+    strict_1.default.equal(out.action, "fixed");
+    strict_1.default.equal(out.code, "// fixed");
+});
+(0, node_test_1.test)("postHealFix surfaces a server-side error job", async () => {
+    mockFetch((url, init) => {
+        if (init.method === "POST")
+            return Response.json({ jobId: "job-2", status: "running" });
+        return Response.json({ status: "error", error: "Claude timed out" });
+    });
+    await strict_1.default.rejects(() => api.postHealFix(cfg, 99, {
+        currentCode: "// old", error: "boom", output: "", domDistilled: null, attempt: 1,
+    }), /Claude timed out/);
+});
+(0, node_test_1.test)("postHealFinalize posts the outcome to /agent/heal/{caseId}/finalize", async () => {
+    mockFetch(() => new Response(null, { status: 200 }));
+    await api.postHealFinalize(cfg, 99, { finalStatus: "pass", finalCode: "// x", attempts: [] });
+    strict_1.default.equal(calls[0].url, "http://127.0.0.1:8787/agent/heal/99/finalize");
+    const body = JSON.parse(calls[0].init.body);
+    strict_1.default.equal(body.finalStatus, "pass");
+});
 (0, node_test_1.test)("redeemDevice posts code+name with no auth header", async () => {
     mockFetch(() => Response.json({ deviceToken: "abc", deviceId: 7 }));
     const result = await api.redeemDevice("http://127.0.0.1:8787", "PAIR123", "my-laptop");
@@ -144,3 +182,15 @@ function mockFetch(handler) {
     const body = JSON.parse(calls[0].init.body);
     strict_1.default.deepEqual(body, { code: "PAIR123", name: "my-laptop" });
 });
+(0, node_test_1.test)("fetchWithTimeout aborts a stalled request once the timeout elapses", async () => {
+    // A server that never responds but honors the abort signal.
+    mockFetch((_u, init) => new Promise((_resolve, reject) => {
+        init.signal.addEventListener("abort", () => reject(new Error("aborted")));
+    }));
+    await strict_1.default.rejects(api.fetchWithTimeout("http://127.0.0.1:8787/slow", { method: "POST" }, 20));
+});
+(0, node_test_1.test)("fetchWithTimeout returns the response when it resolves before the timeout", async () => {
+    mockFetch(() => new Response(null, { status: 200 }));
+    const res = await api.fetchWithTimeout("http://127.0.0.1:8787/ok", {}, 1000);
+    strict_1.default.equal(res.status, 200);
+});

package/dist/test/playwrightConfig.test.js

@@ -0,0 +1,84 @@
+"use strict";
+/**
+ * Unit tests for the injected-fixtures port (`src/playwrightConfig.ts`).
+ * Mirrors the server's `test_fixtures_ts_contents` / `test_apply_fixtures_always_injects`
+ * (api/tests/test_execution.py) so the agent's DOM capture stays in lock-step
+ * with the server runner.
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const strict_1 = __importDefault(require("node:assert/strict"));
+const fs = __importStar(require("node:fs"));
+const os = __importStar(require("node:os"));
+const path = __importStar(require("node:path"));
+const node_test_1 = require("node:test");
+const playwrightConfig_1 = require("../src/playwrightConfig");
+(0, node_test_1.test)("fixturesTs always wires DOM capture; sessionStorage replay is gated", () => {
+    const sessionFile = "/tmp/sessionStorage.json";
+    const replay = (0, playwrightConfig_1.fixturesTs)(sessionFile, true);
+    strict_1.default.ok(replay.includes("export const test"));
+    strict_1.default.ok(replay.includes("testInfo.attach('qagent-dom-raw'"));
+    strict_1.default.ok(replay.includes("testInfo.attach('qagent-dom-distilled'"));
+    strict_1.default.ok(replay.includes(JSON.stringify(sessionFile)));
+    strict_1.default.ok(replay.includes("addInitScript"), "replay=true injects the session init script");
+    const noReplay = (0, playwrightConfig_1.fixturesTs)(sessionFile, false);
+    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)("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";
+    const specPath = path.join(dir, specName);
+    const original = "import { test, expect } from '@playwright/test';\n" +
+        "test('x', async ({ page }) => { await page.goto('/'); });\n";
+    fs.writeFileSync(specPath, original, "utf-8");
+    const sessionFile = path.join(dir, "sessionStorage.json");
+    // Even without session replay, DOM capture means fixtures are injected.
+    (0, playwrightConfig_1.applyFixtures)(dir, [specName], sessionFile, false);
+    const rewritten = fs.readFileSync(specPath, "utf-8");
+    strict_1.default.ok(rewritten.includes("'./fixtures'"));
+    strict_1.default.ok(!rewritten.includes("'@playwright/test'"));
+    const fixtures = fs.readFileSync(path.join(dir, "fixtures.ts"), "utf-8");
+    strict_1.default.ok(fixtures.includes("qagent-dom-distilled"));
+    strict_1.default.ok(!fixtures.includes("addInitScript"));
+    // With replay enabled, the init script is added; specs stay pointed at './fixtures'.
+    (0, playwrightConfig_1.applyFixtures)(dir, [specName], sessionFile, true);
+    strict_1.default.ok(fs.readFileSync(specPath, "utf-8").includes("'./fixtures'"));
+    strict_1.default.ok(fs.readFileSync(path.join(dir, "fixtures.ts"), "utf-8").includes("addInitScript"));
+    fs.rmSync(dir, { recursive: true, force: true });
+});

package/dist/test/report.test.js

@@ -139,6 +139,8 @@ const report_1 = require("../src/report");
                                             { name: "screenshot", path: "/tmp/shot.png" },
                                             { name: "video", path: "/tmp/vid.webm" },
                                             { name: "trace", path: "/tmp/trace.zip" },
+                                            { name: "qagent-dom-raw", path: "/tmp/dom.html" },
+                                            { name: "qagent-dom-distilled", path: "/tmp/dom.json" },
                                             { name: "stdout", path: "/tmp/stdout.txt" },
                                             { name: "screenshot", path: "" },
                                         ],
@@ -155,6 +157,8 @@ const report_1 = require("../src/report");
         { kind: "screenshot", path: "/tmp/shot.png" },
         { kind: "video", path: "/tmp/vid.webm" },
         { kind: "trace", path: "/tmp/trace.zip" },
+        { kind: "dom", path: "/tmp/dom.html" },
+        { kind: "dom-distilled", path: "/tmp/dom.json" },
     ]);
 });
 (0, node_test_1.test)("flattens nested describe-block suites under one file", () => {

package/dist/test/version.test.js

@@ -0,0 +1,21 @@
+"use strict";
+/**
+ * Tests for the runtime version lookup (`src/version.ts`) — the agent must be
+ * able to report its actual installed build (for `--version` and the startup
+ * banner), which is the key signal when diagnosing a stale npx cache.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const strict_1 = __importDefault(require("node:assert/strict"));
+const node_test_1 = require("node:test");
+const version_1 = require("../src/version");
+(0, node_test_1.test)("agentVersion reads a real semver from the package manifest (not 'unknown')", () => {
+    const v = (0, version_1.agentVersion)();
+    strict_1.default.notEqual(v, "unknown", "expected the manifest version to resolve at runtime");
+    strict_1.default.match(v, /^\d+\.\d+\.\d+/, `expected semver, got ${v}`);
+});
+(0, node_test_1.test)("agentVersion is stable across calls (cached)", () => {
+    strict_1.default.equal((0, version_1.agentVersion)(), (0, version_1.agentVersion)());
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@q-agent/agent",
-  "version": "0.1.1",
+  "version": "0.1.7",
   "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": {
@@ -27,17 +27,51 @@
     "test": "npm run build && node --test dist/test",
     "release": "node scripts/release.mjs",
     "prepublishOnly": "npm run build",
-    "package:win": "node scripts/package-win.mjs"
+    "package:win": "node scripts/package-win.mjs",
+    "desktop": "npm run build && electron electron/main.cjs",
+    "dist:desktop": "npm run build && electron-builder --win"
   },
   "dependencies": {
     "@playwright/test": "^1.61.1",
     "commander": "^12.1.0",
+    "electron-updater": "^6.3.9",
     "playwright": "^1.61.1"
   },
   "devDependencies": {
     "@types/node": "^20.14.0",
+    "electron": "^33.2.0",
+    "electron-builder": "^25.1.8",
     "esbuild": "^0.24.0",
     "postject": "^1.0.0-alpha.6",
     "typescript": "^5.5.4"
+  },
+  "build": {
+    "appId": "click.chuongnd.qagent-agent",
+    "productName": "Q-Agent Local Agent",
+    "extraMetadata": {
+      "main": "electron/main.cjs",
+      "qagentServer": "https://qagent.chuongnd.click/api"
+    },
+    "files": [
+      "dist/**/*",
+      "electron/**/*",
+      "vendor/**/*",
+      "package.json"
+    ],
+    "artifactName": "qagent-agent-setup.${ext}",
+    "asar": false,
+    "win": {
+      "target": "nsis",
+      "signAndEditExecutable": false
+    },
+    "publish": [
+      {
+        "provider": "generic",
+        "url": "https://qagent.chuongnd.click/downloads/"
+      }
+    ],
+    "directories": {
+      "output": "dist-bin/desktop"
+    }
   }
 }