@keepsnappy/agent 0.1.0-alpha.17 → 0.1.0-alpha.19

package/README.md

@@ -70,7 +70,7 @@ Annotated example: `examples/keepsnappy.yml`. Every setting:
 
 Unknown settings and wrong types are errors, all listed together.
 
-Results and exit codes: results never change the exit code by default (ruling D9, no blocking PR checks): `0` also for "More work". `--fail-on-worse` (needs `--wait`) fails the job (exit `3`) only when the verdict computed on that run set is Worse and blocking. Closing a run set never computes its verdict (decision 0031: the operator computes verdicts on demand), so with no computed verdict `--fail-on-worse` checks nothing: the CLI prints `Verdict not computed yet (computed on demand): --fail-on-worse checked nothing` and exits `0`. The CLI exits `2` for a configuration problem (bad or missing `keepsnappy.yml`, unknown or misspelt flags, a glob matching nothing, `--wait` without a branch or key) and `1` for an infrastructure problem (cannot read the device log, an API request failing, an API URL that does not answer as the KeepSnappy API, `--upload` that sent no run: "Blocked"; results stay in the out dir). Failed runs are marked invalid in `runs.ndjson` / `metrics.csv` and ignored by the statistics. The ending summary reads like:
+Results and exit codes: results never change the exit code by default (ruling D9, no blocking PR checks): `0` also for "More work". `--fail-on-worse` (needs `--wait`) fails the job (exit `3`) only when the verdict computed on that run set is Worse and blocking. Closing a run set never computes its verdict (decision 0031: the operator computes verdicts on demand), so with no computed verdict `--fail-on-worse` checks nothing: the CLI prints `Verdict not computed yet (computed on demand): --fail-on-worse checked nothing` and exits `0`. The CLI exits `2` for a configuration problem (bad or missing `keepsnappy.yml`, unknown or misspelt flags, a glob matching nothing, `--wait` without a branch or key) and `1` for an infrastructure problem (cannot read the device log, an API request failing, an API URL that does not answer as the KeepSnappy API, `--upload` that sent no run: "Blocked", or a local run with no valid run; results stay in the out dir). For a local run with no valid run, it prints `No valid run was recorded` with the invalid reasons when known. Failed runs are marked invalid in `runs.ndjson` / `metrics.csv` and ignored by the statistics. The ending summary reads like:
 
 ```
 7/8 runs valid (dropped: 1 test-failed: exit 1)
@@ -137,7 +137,7 @@ runner. It writes:
    run with data. A measured iteration sends nothing until its end-of-iteration flush, so a run with data completed and
    is never blamed; a Maestro failure stops the session, so later iterations have no data;
 3. finishes every run not yet finished (a run without a single batch is reported as "no data", never judged), closes the
-   set (`--wait`: verdicts inline; else on read) and prints the iterations and the same plain-words verdict as
+   set (`--wait`: verdicts inline; else on read). It then prints the iterations and the same plain-words verdict as
    `keepsnappy run --upload --wait`, plus `$GITHUB_STEP_SUMMARY` and `<out>/keepsnappy-result.json` (`--out`, default the
    state's directory);
 4. puts the empty stub back into the tokens file, also when a step above failed. The example workflow also deletes the
@@ -149,10 +149,10 @@ result, `3` with `--fail-on-worse` when the verdict computed on that run set is
 verdict it checks nothing and says so, exit `0`), `2` for configuration, `1` when the API fails or nothing could be
 judged ("Blocked": no iteration sent any data).
 
-Slack: set `KEEPSNAPPY_SLACK_WEBHOOK` (an incoming-webhook URL kept as a CI secret) and `farm close --wait` posts the
-result: a pull request only when something got worse (a table of the 3 biggest changes, main vs PR), main only when
-something changed (`KEEPSNAPPY_SLACK_ALWAYS=1` also posts "no change"), and a measurement that sent no data always. The
-commit's title and author come from `git log` in the checkout. A Slack failure is printed and never changes the exit code.
+Slack: set `KEEPSNAPPY_SLACK_WEBHOOK` (an incoming-webhook URL kept as a CI secret) and `farm close` posts a measurement that sent
+no data ("No result") at once. Results are posted after the verdict is computed, by the operator's recompute job (decision 0031), in
+two lines and one button (docs/mockups/2026-10-26-slack, present `resultSlackPayload`). `$GITHUB_STEP_SUMMARY` gets the same two
+lines: "No result", the computed result, or "Not judged yet · <branch> / Waiting for the judgement" with the run set's page. A Slack failure is printed and never changes the exit code.
 
 Declared steps on the farm: `maestro/keepsnappy/step.yaml` / `step.js` / `send-markers.js` (copy them into the suite).
 With a token set, `step.js` only records the marker; `send-markers.js`, run in `end-run.yaml` after each iteration,

package/dist/appium-driver.js

@@ -507,23 +507,29 @@ class Driver {
     });
     await this.cmd("DELETE", "/actions").catch(() => null);
   }
-  async drag(from, to, durationMs) {
+  async drag(from, to, durationMs, note) {
     to = { x: to.x, y: Math.min(to.y, await this.safeBottomY()) };
-    await this.cmd("POST", "/actions", {
-      actions: [
-        {
-          type: "pointer",
-          id: "finger1",
-          parameters: { pointerType: "touch" },
-          actions: [
-            { type: "pointerMove", duration: 0, x: from.x, y: from.y },
-            { type: "pointerDown", button: 0 },
-            { type: "pointerMove", duration: durationMs, x: to.x, y: to.y },
-            { type: "pointerUp", button: 0 }
-          ]
-        }
-      ]
-    });
+    const logged = this.log.length;
+    try {
+      await this.cmd("POST", "/actions", {
+        actions: [
+          {
+            type: "pointer",
+            id: "finger1",
+            parameters: { pointerType: "touch" },
+            actions: [
+              { type: "pointerMove", duration: 0, x: from.x, y: from.y },
+              { type: "pointerDown", button: 0 },
+              { type: "pointerMove", duration: durationMs, x: to.x, y: to.y },
+              { type: "pointerUp", button: 0 }
+            ]
+          }
+        ]
+      });
+    } finally {
+      if (note && this.log[logged])
+        Object.assign(this.log[logged], note);
+    }
     await this.cmd("DELETE", "/actions").catch(() => null);
   }
   async relaunch(bundleId = this.bundleId) {
@@ -726,7 +732,7 @@ import { dirname, join, relative, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 var DRIVER_NAME = "appium";
 function driverVersion() {
-  const built = "0.1.0-alpha.17";
+  const built = "0.1.0-alpha.19";
   if (built)
     return built;
   let dir = dirname(fileURLToPath(import.meta.url));
@@ -800,7 +806,7 @@ async function sendMarkers(fetchImpl, api, markers, opts = {}) {
   const url = `${base}/v1/markers`;
   const version = (opts.driverVersion ?? driverVersion)();
   const reported = opts.reported ?? new Set;
-  const driverField = { driver: { name: DRIVER_NAME, ...version ? { version } : {}, ...opts.deviceUnit ? { unit: opts.deviceUnit.slice(0, 200) } : {} } };
+  const driverField = { driver: { name: DRIVER_NAME, ...version ? { version } : {}, ...opts.deviceUnit ? { unit: opts.deviceUnit.slice(0, 200) } : {}, ...opts.capture ? { capture: opts.capture } : {} } };
   let sent = 0;
   for (const [token, list] of byToken) {
     const trips = [];
@@ -902,9 +908,11 @@ class Session {
   readyWaits = [];
   actionGc;
   endGc;
+  capture;
   gcN = 0;
   stepDepth = 0;
   stepLog = [];
+  defects = { gestureRetry: 0, items: [] };
   onStepChange = null;
   refinds = 0;
   get staleRefinds() {
@@ -917,6 +925,7 @@ class Session {
     this.api = o.api ?? "";
     this.actionGc = o.actionGc ? "step" : "off";
     this.endGc = o.endGc ?? END_GC_DEFAULT;
+    this.capture = o.capture;
   }
   async gc(atEnd = false) {
     if (this.actionGc !== "step" && !(atEnd && this.endGc && this.actionGc === "off"))
@@ -1293,7 +1302,7 @@ class Session {
   async endRun() {
     if (this.markers.some((m) => m.phase === "start"))
       await this.memRead(await this.gc(true));
-    await sendMarkers(this.fetch, this.api, this.markers, { sleep: (ms) => this.driver.sleep(ms), now: () => this.now(), deviceUnit: this.driver.deviceUnit, tokens: [...this.opened], reported: this.told, note: (line) => this.driver.log.push({ ...this.driver.context, cmd: "driver-report", t: this.now(), ms: 0, status: 1, answer: line }) });
+    await sendMarkers(this.fetch, this.api, this.markers, { sleep: (ms) => this.driver.sleep(ms), now: () => this.now(), deviceUnit: this.driver.deviceUnit, capture: this.capture, tokens: [...this.opened], reported: this.told, note: (line) => this.driver.log.push({ ...this.driver.context, cmd: "driver-report", t: this.now(), ms: 0, status: 1, answer: line }) });
     const evidence = { linkAt: this.now(), resentAt: null, ackAt: null, gaveUpAt: null };
     this.lastFlush = evidence;
     const acks = new Set;
@@ -1460,7 +1469,7 @@ async function runDriver(argv, module, deps = realRunDeps()) {
   const { api, tokens } = parseTokensJs(readFileSync2(String(a["tokens-js"]), "utf8"));
   const log = [];
   const driver = new Driver({ hub: HUB, log, headers: { Authorization: auth }, bundleId: module.app.bundleId, platform, fetch: deps.fetch, now: deps.now, sleep: deps.sleep });
-  const t = new Session({ driver, scheme: module.app.scheme, fetch: deps.fetch, api, actionGc: a["action-gc"] === "on", endGc: a["end-gc"] === "on" });
+  const t = new Session({ driver, scheme: module.app.scheme, fetch: deps.fetch, api, actionGc: a["action-gc"] === "on", endGc: a["end-gc"] === "on", capture: capture ? "on" : "off" });
   const sessionName = module.sessionName ?? "keepsnappy (appium)";
   const device = String(a.device);
   const ctx = { fetch: deps.fetch, tokens, journeys, repeat, args: a };
@@ -1503,6 +1512,7 @@ async function runDriver(argv, module, deps = realRunDeps()) {
       flushError,
       flushOff,
       finalFlush,
+      defects: t.defects,
       steps: t.stepLog,
       iterations,
       commands: log