gdharness 1.0.2 → 1.0.3

package/build/cli.js

@@ -16155,7 +16155,7 @@ var init_tool_definitions = __esm(() => {
         property: {
           type: "string",
           ops: ["property", "find"],
-          description: 'property: which one to read. find: read this one off every node matched, so a panel of labels is one call rather than one per label. Colons read through what a node holds, "_game:clock:speed", which is where a game keeps what is worth asking about. A number or a key steps into a list or a map, "_game:run:roster:0:traits", which is how the lists a game keeps its state in are walked: a roster, a board, an in-tray. A negative number counts from the end. A step that is not there is named, and says what was there instead.'
+          description: 'property: which one to read. find: read this one off every node matched, so a panel of labels is one call rather than one per label. Colons read through what a node holds, "_game:clock:speed", which is where a game keeps what is worth asking about. A number or a key steps into a list or a map, "_game:run:roster:0:traits", which is how the lists a game keeps its state in are walked: a roster, a board, an in-tray. A negative number counts from the end. A step written as a call, "get_viewport():gui_get_focus_owner()", calls a method that takes no arguments and walks into what it returned, which is how a question only a method answers is read in one call: which control holds the focus in the viewport this one is in. A step that is not there is named, and says what was there instead.'
         },
         depth: { type: "number", ops: ["tree"], description: "tree: levels to descend. Default 3." },
         includeProperties: {
@@ -16236,7 +16236,7 @@ var init_tool_definitions = __esm(() => {
         property: {
           type: "string",
           ops: ["set"],
-          description: 'set: which one to write. Colons write through what a node holds, "_game:run:day", and a number or a key steps into a list or a map on the way, "_game:run:roster:0:name", a negative number counting from the end. The answer reads back off the same holder, so a write a typed container refused shows as an unchanged value.'
+          description: 'set: which one to write. Colons write through what a node holds, "_game:run:day", and a number or a key steps into a list or a map on the way, "_game:run:roster:0:name", a negative number counting from the end, and a step written as a call, "get_viewport():gui_embed_subwindows", walks through what a method taking no arguments returned. The answer reads back off the same holder, so a write a typed container refused shows as an unchanged value.'
         },
         value: {
           ops: ["set"],
@@ -16246,7 +16246,7 @@ var init_tool_definitions = __esm(() => {
         method: {
           type: "string",
           ops: ["call"],
-          description: 'call: which one to call. Colons call through what a node holds, "_game:run:advance", the same way a property is written through them, and a list or a map is stepped into by index or key, "_game:run:roster:0:retire".'
+          description: 'call: which one to call. Colons call through what a node holds, "_game:run:advance", the same way a property is written through them, a list or a map is stepped into by index or key, "_game:run:roster:0:retire", and a step written as a call walks through what a method taking no arguments returned, "get_viewport():gui_get_focus_owner".'
         },
         args: {
           type: "array",
@@ -16306,7 +16306,7 @@ var init_tool_definitions = __esm(() => {
         pressed: {
           type: "boolean",
           ops: ["action", "key", "mouse_click"],
-          description: "action, key: leave it out and the press is a whole one, down and up a frame apart. true holds it down, false lets go of one being held. mouse_click is one raw event, so it is down unless you say false."
+          description: "action, key: leave it out and the press is a whole one, down and up a frame apart. true holds it down, false lets go of one being held. mouse_click is one raw event, so it is down unless you say false; a wheel button down is a whole step, since a wheel is never held."
         },
         strength: { type: "number", ops: ["action"], description: "action: 0 to 1. Default 1." },
         keycode: {
@@ -16384,7 +16384,7 @@ var init_tool_definitions = __esm(() => {
           requires: ["text"]
         },
         mouse_click: {
-          summary: "one mouse button event at a position, carrying the buttons held once it has happened. A drag is a press held with pressed true, motions, and a release with pressed false",
+          summary: "one mouse button event at a position, carrying the buttons held once it has happened. A drag is a press held with pressed true, motions, and a release with pressed false. A wheel button is a whole step, press and release together as a mouse sends one, and the answer says released: a lone wheel press holds the viewport on the control that took it, so every click after it lands there",
           requires: ["x", "y"]
         },
         mouse_motion: {
@@ -16406,7 +16406,11 @@ var init_tool_definitions = __esm(() => {
         },
         nodePath: { type: "string", ops: ["signal", "until"], description: "signal, until: the node." },
         signal: { type: "string", ops: ["signal"], description: "signal: the signal name." },
-        property: { type: "string", ops: ["until"], description: "until: the property name." },
+        property: {
+          type: "string",
+          ops: ["until"],
+          description: 'until: the property name, or a colon path through what the node holds, "_game:run:day", read the way runtime_inspect property reads one, a step written as a call included. Walked again on every frame, so a holder the game replaces while the wait is on is followed.'
+        },
         value: { ops: ["until"], description: "until: the value to wait for, fitted to the property's type." },
         says: {
           type: "string",
@@ -36462,18 +36466,59 @@ var init_lsp_client = __esm(() => {
 });
 
 // src/process-children.ts
-import { execFile as execFile5 } from "node:child_process";
+import { execFile as execFile5, execFileSync, spawnSync as spawnSync2 } from "node:child_process";
 import { basename as basename2 } from "node:path";
 import { promisify as promisify5 } from "node:util";
+function startTimesOf(pids) {
+  const began = new Map;
+  if (pids.length === 0) {
+    return began;
+  }
+  try {
+    if (process.platform === "win32") {
+      const filter = pids.map((pid) => `ProcessId=${pid}`).join(" OR ");
+      const said = execFileSync("powershell", [
+        "-NoProfile",
+        "-NonInteractive",
+        "-Command",
+        `Get-CimInstance Win32_Process -Filter "${filter}" | ForEach-Object { "$($_.ProcessId) $(([DateTimeOffset]$_.CreationDate).ToUnixTimeMilliseconds())" }`
+      ], { encoding: "utf8", timeout: ASK_TIMEOUT_MS, windowsHide: true });
+      for (const line of said.split(`
+`)) {
+        const found = /^\s*(\d+)\s+(\d+)\s*$/.exec(line);
+        if (found?.[1] !== undefined && found[2] !== undefined) {
+          began.set(Number(found[1]), Number(found[2]));
+        }
+      }
+      return began;
+    }
+    const said = spawnSync2("ps", ["-p", pids.join(","), "-o", "pid=,lstart="], {
+      encoding: "utf8",
+      timeout: ASK_TIMEOUT_MS,
+      env: { ...process.env, LC_ALL: "C" }
+    }).stdout;
+    for (const line of said.split(`
+`)) {
+      const found = /^\s*(\d+)\s+(.+)$/.exec(line);
+      const at = Date.parse(found?.[2]?.trim() ?? "");
+      if (found?.[1] !== undefined && !Number.isNaN(at)) {
+        began.set(Number(found[1]), at);
+      }
+    }
+  } catch {}
+  return began;
+}
 async function processTree() {
   try {
-    const printed = process.platform === "win32" ? await askWindows() : await askPosix();
-    return parseProcessTable(printed);
+    if (process.platform === "win32") {
+      return parseProcessTable(await askWindows(), true);
+    }
+    return parseProcessTable(await askPosix());
   } catch {
     return;
   }
 }
-function parseProcessTable(printed) {
+function parseProcessTable(printed, started = false) {
   const tree = new Map;
   for (const line of printed.split(/\r?\n/)) {
     const fields = /^\s*(\d+)\s+(\d+)(?:\s+(.*))?$/.exec(line);
@@ -36482,16 +36527,38 @@ function parseProcessTable(printed) {
     }
     const pid = Number(fields[1]);
     const parent = Number(fields[2]);
-    if (pid > 0) {
-      tree.set(pid, { parent, command: (fields[3] ?? "").trim() });
+    const rest = (fields[3] ?? "").trim();
+    if (pid <= 0) {
+      continue;
+    }
+    if (!started) {
+      tree.set(pid, { parent, command: rest });
+      continue;
+    }
+    const timed = /^(\d+)(?:\s+(.*))?$/.exec(rest);
+    if (timed === null) {
+      continue;
     }
+    const began = Number(timed[1]);
+    tree.set(pid, { parent, command: (timed[2] ?? "").trim(), ...began > 0 ? { startedAt: began } : {} });
   }
   return tree;
 }
+function linked(tree, child, parent) {
+  const from = tree.get(child);
+  const to = tree.get(parent);
+  if (from === undefined || from.parent !== parent) {
+    return false;
+  }
+  if (from.startedAt === undefined || to?.startedAt === undefined) {
+    return true;
+  }
+  return from.startedAt >= to.startedAt;
+}
 function childrenIn(tree, pid) {
   const children = [];
-  for (const [child, listed] of tree) {
-    if (listed.parent === pid) {
+  for (const child of tree.keys()) {
+    if (linked(tree, child, pid)) {
       children.push(child);
     }
   }
@@ -36519,13 +36586,15 @@ function descendantsIn(tree, pid) {
 function ancestorsIn(tree, pid, root) {
   const chain = [];
   const seen = new Set([pid]);
+  let child = pid;
   let parent = tree.get(pid)?.parent;
-  while (parent !== undefined && !seen.has(parent)) {
+  while (parent !== undefined && !seen.has(parent) && linked(tree, child, parent)) {
     chain.push(parent);
     if (parent === root) {
       break;
     }
     seen.add(parent);
+    child = parent;
     parent = tree.get(parent)?.parent;
   }
   return chain;
@@ -36581,7 +36650,7 @@ async function askWindows() {
     "-NoProfile",
     "-NonInteractive",
     "-Command",
-    'Get-CimInstance Win32_Process | ForEach-Object { "$($_.ProcessId) $($_.ParentProcessId) $($_.CommandLine)" }'
+    'Get-CimInstance Win32_Process | ForEach-Object { $began = 0; if ($_.CreationDate) { $began = ([DateTimeOffset]$_.CreationDate).ToUnixTimeMilliseconds() }; "$($_.ProcessId) $($_.ParentProcessId) $began $($_.CommandLine)" }'
   ], { timeout: ASK_TIMEOUT_MS, windowsHide: true, maxBuffer: 16 * 1024 * 1024 });
   return stdout;
 }
@@ -36813,7 +36882,7 @@ var init_restart_note = __esm(() => {
 import { existsSync as existsSync13, readdirSync as readdirSync5, readFileSync as readFileSync15, statSync as statSync3, unlinkSync } from "node:fs";
 import { createConnection as createConnection3 } from "node:net";
 import { tmpdir as tmpdir4 } from "node:os";
-import { join as join17, resolve as resolve4 } from "node:path";
+import { dirname as dirname11, join as join17, resolve as resolve4 } from "node:path";
 import { setTimeout as delay2 } from "node:timers/promises";
 function runtimeDirectory(variables = process.env) {
   const explicit = envValue("GDHARNESS_RUNTIME_DIR", variables);
@@ -36922,11 +36991,31 @@ function announcedAndGone(projectPath) {
   const wanted = resolve4(projectPath);
   return gone.filter((one) => one.noticedAt >= since && resolve4(one.project) === wanted);
 }
+function strangersAmong(candidates, now, startTimes = startTimesOf) {
+  const toJudge = candidates.filter((one) => {
+    if (now - one.writtenAt < JUDGED_AFTER_MS) {
+      return false;
+    }
+    const known = verdicts.get(one.file);
+    return known === undefined || known.writtenAt !== one.writtenAt || !known.stranger && now - known.judgedAt > CONFIRMED_FOR_MS;
+  });
+  if (toJudge.length > 0) {
+    const began = startTimes(toJudge.map((one) => one.pid));
+    for (const one of toJudge) {
+      const at = began.get(one.pid);
+      const stranger = at !== undefined && at - one.writtenAt > STARTED_AFTER_ANNOUNCING_MS;
+      verdicts.set(one.file, { writtenAt: one.writtenAt, judgedAt: now, stranger });
+    }
+  }
+  return new Set(candidates.filter((one) => verdicts.get(one.file)?.stranger === true).map((one) => one.file));
+}
 function announcedIn(directory) {
   if (!existsSync13(directory)) {
     return [];
   }
   const found = [];
+  const candidates = [];
+  const entries = [];
   for (const entry of readdirSync5(directory)) {
     const report = ERROR_REPORT_PATTERN.exec(entry);
     if (report) {
@@ -36940,20 +37029,59 @@ function announcedIn(directory) {
     const file = join17(directory, entry);
     const pid = Number.parseInt(match[1] ?? "", 10);
     const alive = processAlive(pid);
-    const announced = alive ? parseAnnouncement(file, pid) : { kind: "rubbish" };
+    entries.push({ file, pid, alive });
+    if (alive) {
+      try {
+        candidates.push({ file, pid, writtenAt: statSync3(file).mtimeMs });
+      } catch {}
+    }
+  }
+  const strangers = strangersAmong(candidates, Date.now());
+  for (const { file, pid, alive } of entries) {
+    const announced = alive && !strangers.has(file) ? parseAnnouncement(file, pid) : { kind: "rubbish" };
     if (announced.kind === "rubbish") {
-      if (!alive) {
+      if (!alive || strangers.has(file)) {
         wentAway(file, pid);
       }
       try {
         unlinkSync(file);
       } catch {}
+      verdicts.delete(file);
       continue;
     }
     found.push(announced);
   }
+  const held = new Set(entries.map((one) => one.file));
+  const here = join17(directory, ".");
+  for (const file of verdicts.keys()) {
+    if (dirname11(file) === here && !held.has(file)) {
+      verdicts.delete(file);
+    }
+  }
   return found;
 }
+async function announcementEnded(pid, directories = runtimeDirectories()) {
+  const files = directories.map((directory) => join17(directory, `runtime-${pid}.json`)).filter((file) => existsSync13(file));
+  const first = files[0];
+  if (first === undefined) {
+    return;
+  }
+  const until = Date.now() + ENDED_WITHIN_MS;
+  while (processAlive(pid)) {
+    const left = until - Date.now();
+    if (left <= 0) {
+      return;
+    }
+    await delay2(Math.min(LOOK_EVERY_MS, left));
+  }
+  wentAway(first, pid);
+  for (const file of files) {
+    try {
+      unlinkSync(file);
+    } catch {}
+    verdicts.delete(file);
+  }
+}
 function sweepErrorReport(path, pid, now = Date.now()) {
   if (processAlive(pid)) {
     return;
@@ -37134,16 +37262,18 @@ function runtimeRequest(endpoint, command, params, timeoutMs) {
     });
   });
 }
-var RUNTIME_PROTOCOL = 2, ANNOUNCEMENT_PATTERN, ERROR_REPORT_PATTERN, ERROR_REPORT_KEEP_MS, GONE_WINDOW_MS = 120000, GONE_KEPT = 32, gone, LOOK_EVERY_MS = 50, ANNOUNCE_BUDGET_MS = 5000, nextRequestId = 1;
+var RUNTIME_PROTOCOL = 2, ANNOUNCEMENT_PATTERN, ERROR_REPORT_PATTERN, ERROR_REPORT_KEEP_MS, GONE_WINDOW_MS = 120000, GONE_KEPT = 32, gone, STARTED_AFTER_ANNOUNCING_MS = 2000, CONFIRMED_FOR_MS = 60000, JUDGED_AFTER_MS = 60000, verdicts, ENDED_WITHIN_MS = 3000, LOOK_EVERY_MS = 50, ANNOUNCE_BUDGET_MS = 5000, nextRequestId = 1;
 var init_runtime_client = __esm(() => {
+  init_process_children();
   ANNOUNCEMENT_PATTERN = /^runtime-(\d+)\.json$/;
   ERROR_REPORT_PATTERN = /^runtime-(\d+)\.log$/;
   ERROR_REPORT_KEEP_MS = 60 * 60 * 1000;
   gone = [];
+  verdicts = new Map;
 });
 
 // src/run-record.ts
-import { execFileSync } from "node:child_process";
+import { execFileSync as execFileSync2 } from "node:child_process";
 import { mkdirSync as mkdirSync9, openSync, readdirSync as readdirSync6, readFileSync as readFileSync16, rmSync as rmSync9, statSync as statSync4, writeFileSync as writeFileSync11 } from "node:fs";
 import { basename as basename3, join as join18 } from "node:path";
 function runsDirectory() {
@@ -37269,7 +37399,7 @@ function runningAs(pid) {
       const raw = readFileSync16(`/proc/${pid}/cmdline`, "utf8").replaceAll("\x00", " ").trim();
       return raw === "" ? null : { kind: "commandLine", text: raw, ...when };
     }
-    const args = execFileSync("ps", ["-p", String(pid), "-o", "args="], {
+    const args = execFileSync2("ps", ["-p", String(pid), "-o", "args="], {
       encoding: "utf8",
       timeout: 15000
     }).trim();
@@ -37281,20 +37411,14 @@ function runningAs(pid) {
 function listeningPid(port) {
   try {
     if (process.platform === "win32") {
-      const said = execFileSync("powershell", [
-        "-NoProfile",
-        "-NonInteractive",
-        "-Command",
-        `(Get-NetTCPConnection -State Listen -LocalPort ${port} -ErrorAction SilentlyContinue | Select-Object -First 1).OwningProcess`
-      ], { encoding: "utf8", timeout: 15000 }).trim();
-      return /^\d+$/.test(said) ? Number(said) : null;
+      return listeningPidInNetstat(execFileSync2("netstat", ["-ano"], { encoding: "utf8", timeout: 15000 }), port);
     }
     for (const [command, args, pattern] of [
       ["lsof", ["-nP", `-iTCP:${port}`, "-sTCP:LISTEN", "-t"], /^(\d+)/],
       ["ss", ["-ltnpH", `sport = :${port}`], /pid=(\d+)/]
     ]) {
       try {
-        const said = execFileSync(command, [...args], { encoding: "utf8", timeout: 15000 }).trim();
+        const said = execFileSync2(command, [...args], { encoding: "utf8", timeout: 15000 }).trim();
         const found = pattern.exec(said);
         if (found?.[1] !== undefined) {
           return Number(found[1]);
@@ -37306,9 +37430,22 @@ function listeningPid(port) {
     return null;
   }
 }
+function listeningPidInNetstat(printed, port) {
+  for (const line of printed.split(/\r?\n/)) {
+    const fields = /^\s*TCP\s+(\S+)\s+\S+\s+LISTENING\s+(\d+)\s*$/i.exec(line);
+    if (fields === null) {
+      continue;
+    }
+    const local = fields[1] ?? "";
+    if (Number(local.slice(local.lastIndexOf(":") + 1)) === port) {
+      return Number(fields[2]);
+    }
+  }
+  return null;
+}
 function windowsCommandLine(pid) {
   try {
-    const answer = execFileSync("powershell", [
+    const answer = execFileSync2("powershell", [
       "-NoProfile",
       "-NonInteractive",
       "-Command",
@@ -37326,7 +37463,7 @@ function startedAt(pid, windowsAnswer) {
     return Number.isNaN(when) ? null : when;
   }
   try {
-    const said = execFileSync("ps", ["-p", String(pid), "-o", "lstart="], {
+    const said = execFileSync2("ps", ["-p", String(pid), "-o", "lstart="], {
       encoding: "utf8",
       timeout: 15000
     }).trim();
@@ -37338,7 +37475,7 @@ function startedAt(pid, windowsAnswer) {
 }
 function windowsImage(pid) {
   try {
-    const csv = execFileSync("tasklist", ["/FI", `PID eq ${pid}`, "/FO", "CSV", "/NH"], {
+    const csv = execFileSync2("tasklist", ["/FI", `PID eq ${pid}`, "/FO", "CSV", "/NH"], {
       encoding: "utf8",
       timeout: 15000,
       windowsHide: true
@@ -37441,7 +37578,7 @@ import {
   readSync
 } from "node:fs";
 import { tmpdir as tmpdir5 } from "node:os";
-import { basename as basename4, dirname as dirname11, join as join19, normalize as normalize3, relative as relative3 } from "node:path";
+import { basename as basename4, dirname as dirname12, join as join19, normalize as normalize3, relative as relative3 } from "node:path";
 import { setTimeout as delay3 } from "node:timers/promises";
 import { fileURLToPath as fileURLToPath4 } from "node:url";
 import { promisify as promisify7 } from "node:util";
@@ -38659,11 +38796,11 @@ class GodotServer {
     if (!engine.ok) {
       return engine.response;
     }
-    const folder = dirname11(output.absolutePath);
+    const folder = dirname12(output.absolutePath);
     try {
       mkdirSync10(folder, { recursive: true });
     } catch (error) {
-      return this.createErrorResponse(`Cannot create ${dirname11(output.relativePath)} to export into: ${errorMessage(error)}`);
+      return this.createErrorResponse(`Cannot create ${dirname12(output.relativePath)} to export into: ${errorMessage(error)}`);
     }
     const debug = readBoolean(args, "debug") ?? false;
     const exportLogs = mkdtempSync3(join19(tmpdir5(), "gdharness-export-"));
@@ -39117,11 +39254,14 @@ class GodotServer {
       return null;
     }
     try {
-      const answer = await Promise.race([
-        this.godotBridge.invokeTool("playing_status", {}),
-        delay3(PLAYING_STATUS_MS).then(() => null)
-      ]);
+      const askedAt = Date.now();
+      const asked = this.godotBridge.invokeTool("playing_status", {});
+      const answer = await Promise.race([asked, delay3(PLAYING_STATUS_MS).then(() => null)]);
       if (answer === null) {
+        this.logDebug(`playing_status did not answer within ${PLAYING_STATUS_MS}ms`);
+        asked.then(() => {
+          this.logDebug(`playing_status answered after ${Date.now() - askedAt}ms`);
+        }).catch(() => {});
         return null;
       }
       return {
@@ -39441,6 +39581,9 @@ class GodotServer {
     let asking = null;
     let editorAskedAt = 0;
     let editorSaysGoing = true;
+    let editorAsks = 0;
+    const waitBegan = Date.now();
+    const lastSaid = () => editorSaysGoing ? "playing" : "not playing";
     const endpoint = await announcedSince(projectPath, before, {
       budgetMs,
       accept: (one) => this.isTheGameOf(this.currentRun(), one, projectPath),
@@ -39454,6 +39597,7 @@ class GodotServer {
         }
         if (asking === null && Date.now() - editorAskedAt >= RUN_POLL_MS) {
           editorAskedAt = Date.now();
+          editorAsks += 1;
           asking = this.runStillGoing(run).then((going) => {
             editorSaysGoing = going;
           }).finally(() => {
@@ -39463,6 +39607,7 @@ class GodotServer {
         return !editorSaysGoing;
       }
     });
+    this.logDebug(`announce wait ended after ${Date.now() - waitBegan}ms: ${endpoint === null ? "nothing announced" : `pid ${endpoint.pid} announced`}, editor asked ${editorAsks} times, last said ${lastSaid()}`);
     const going = this.currentRun();
     if (endpoint !== null && going !== null && going.announcedPid === undefined) {
       going.announcedPid = endpoint.pid;
@@ -40139,11 +40284,19 @@ class GodotServer {
     this.drainEditorOutput(stopped);
     this.drainTranscript(stopped);
     const wasRunning = await this.runStillGoing(stopped);
-    const endedPid = stopped.pid ?? this.announcedPidOf(stopped) ?? null;
+    const announcedPid = this.announcedPidOf(stopped) ?? null;
+    const endedPid = stopped.pid ?? announcedPid;
     const children = readBoolean(args, "andChildren") === true && wasRunning ? await this.whatTheRunStarted(stopped) : null;
     this.logDebug("Stopping the running game");
     await this.endActiveGame("editor_run stop");
     const ended = children === null ? null : endChildrenAmong(children);
+    if (wasRunning) {
+      for (const pid of new Set([endedPid, announcedPid])) {
+        if (pid !== null) {
+          await announcementEnded(pid);
+        }
+      }
+    }
     return this.jsonTextResponse({
       stopped: true,
       through: stopped.throughEditor ? "editor" : "gdharness",
@@ -40591,7 +40744,7 @@ var init_server3 = __esm(() => {
   init_tool_definitions();
   init_update_check();
   run7 = promisify7(execFile7);
-  __dirname2 = dirname11(fileURLToPath4(import.meta.url));
+  __dirname2 = dirname12(fileURLToPath4(import.meta.url));
   CONSOLE_WRAPPER = /_console(\.exe)?$/i;
   VALUE_NAMES = {
     string: "a string",
@@ -42142,6 +42295,7 @@ that is running, and the project on disk. ${TOOL_SPECS.length} tools, named \`do
 | Where a control is, and whether it is visible | \`runtime_inspect\` \`find\`, \`rect\` |
 | What a property reads right now, on a node whose path you already hold | \`runtime_inspect\` \`property\`, \`runtime_invoke\` |
 | What a property on an object a node holds reads | the same two, with colons: \`_game:clock:speed\` |
+| Which control holds the focus, or anything only a method answers | the same path with a step written as a call, \`get_viewport():gui_get_focus_owner\`, which calls a method taking no arguments and walks into what it returned |
 | Press a button | \`runtime_input click\`, which says what was under the pointer |
 | Press the next thing after a click that changed screen | \`runtime_wait until\` with \`says\` and a word the new screen shows, then the click. A click sent straight after a screen change lands on a control the new screen is about to free, answers \`landed: true\`, and does nothing; a frame count is a number nobody can pick correctly across a cold start and a warm one, and waiting for the words answered in about 130ms where twenty frames had not been enough |
 | Fill in a field | \`runtime_input click\` on it, then \`runtime_input text\` with \`replace\`; a submitted field has to be clicked again |

package/build/godot/addons/gdharness_runtime/runtime_input.gd

@@ -352,8 +352,15 @@ func inject_mouse_click(params: Dictionary) -> Dictionary:
 	# its drag from the press, which is where a real pointer would be.
 	var _moved: Vector2 = _arrive(position)
 	Input.parse_input_event(_button(position, button, pressed, double))
+	# A wheel step is a press and a release together, as a mouse sends one: there is no holding a
+	# wheel. A lone wheel press left the viewport's mouse focus on the control that took it, with
+	# the wheel's bit in the focus mask, and every click after it landed on that control rather
+	# than under the pointer, landing true and doing nothing, until a release was sent by hand.
+	var notch: bool = pressed and _mask_of(button) == 0
+	if notch:
+		Input.parse_input_event(_button(position, button, false, false))
 
-	return {
+	var answer: Dictionary = {
 		"type": "input_injected",
 		"input_type": "mouse_click",
 		"position": [position.x, position.y],
@@ -361,6 +368,9 @@ func inject_mouse_click(params: Dictionary) -> Dictionary:
 		"pressed": pressed,
 		"double": double
 	}
+	if notch:
+		answer["released"] = true
+	return answer
 
 
 ## Moves the pointer to a position, with the movement the event carries taken from where the
@@ -895,4 +905,7 @@ func _resolve_mouse_button(raw: Variant) -> int:
 				return MOUSE_BUTTON_WHEEL_DOWN
 			_:
 				return -1
-	return Read.as_int(raw, -1)
+	# The engine's buttons run from 1 to 9: 0 is MOUSE_BUTTON_NONE, which no event carries, and a
+	# number past the two extra buttons names nothing a mask bit can be made for.
+	var number: int = Read.as_int(raw, -1)
+	return number if number >= MOUSE_BUTTON_LEFT and number <= MOUSE_BUTTON_XBUTTON2 else -1