@liustack/modlens 3.16.0 → 3.16.1

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 3.16.1 - 2026-08-14
+
+- **OpenChamber (OpenCode's desktop UI) is detected, and Windows detection got quieter and sharper ([#30](https://github.com/liustack/modlens/issues/30)).** Three stacked Windows gaps from one runtime-confirmed report. The env-fingerprint fallback never checked the markers opencode servers inject (`OPENCODE`, `OPENCODE_PID`, `OPENCODE_BINARY`), so OpenChamber read as "none detected" and `recover-paste` never auto-ran, while the pasted bytes sat recoverable in the opencode database the whole time; the fingerprint now resolves to `opencode`, placed before Claude Code's so nested setups pick the innermost input box. The `ps` ancestry probe now runs only off Windows: MSYS machines carry a `ps` that exists but rejects `-Ao`, and a failed child's stderr printed into every `doctor` run (the docs always said Windows skips ancestry, now the code agrees). And `findOnPath` tries the PATHEXT extensions before the bare name, so the POSIX `sh` shim npm installs next to `opencode.cmd` no longer shadows the executable into a `spawnSync ENOENT`. Thanks to @IA20201 for a report with the evidence already attached: observed env markers, doctor output, and the exact database row.
+
 ## 3.16.0 - 2026-08-14
 
 - **dsh: whether a paste is taken over is now the host's call, made from real model metadata.** The browser half used to guess with a name regex, which read every vision model it did not recognize (Qwen2.5-VL, GPT-4o, ...) as text-only and hijacked its native paste. And turning `pasteToPath` off only removed the host route while the client kept capturing pastes into a 404. The client now asks `GET /modlens/paste?model=<selector label>` and the host answers from the provider registry's declared `inputModalities`, with every unknown answered conservatively: the label carries no provider id, so EVERY model whose name or id appears in it must be confirmed text-only. One image-capable match anywhere vetoes, an unreadable provider catalog vetoes (the vision twin could live there), and missing modality metadata counts as unknown, never as text-only. Verdicts are re-asked on every composer focus and every paste, the host empties its own cache on every provider-topology change (a same-named vision route mounting mid-session is seen within one round-trip), a 60-second hard age bound backstops both, and a route that vanishes mid-session costs only the pastes inside the one round-trip it takes the failure to come back, after which the client forgets its verdicts and stands down. Until a model is positively confirmed text-only, pastes stay native. The paste route's magic-byte table now matches the CLI's signature for signature (a plain `ftypmp42` video is refused instead of saved as `paste.heic`, real heic/heif brands pass), and the browser half gained its first test suite, loaded straight from `dsh/client.js`.

package/dist/main.js

@@ -1630,7 +1630,7 @@ const PROVIDER_DESCRIPTORS = [
 ];
 function findOnPath(bin, env) {
   const dirs = (env.PATH ?? "").split(path.delimiter).filter(Boolean);
-  const suffixes = process.platform === "win32" ? ["", ...(env.PATHEXT ?? ".COM;.EXE;.BAT;.CMD").split(";").filter(Boolean)] : [""];
+  const suffixes = process.platform === "win32" ? [...(env.PATHEXT ?? ".COM;.EXE;.BAT;.CMD").split(";").filter(Boolean), ""] : [""];
   for (const dir of dirs) {
     for (const suffix of suffixes) {
       const full = path.join(dir, bin + suffix);
@@ -2745,27 +2745,40 @@ function detectHarnessDetailed() {
   if (override) {
     return { harness: override === "none" ? null : override, source: "override" };
   }
-  try {
-    const ps = childProcess.execFileSync("ps", ["-Ao", "pid=,ppid=,command="], {
-      encoding: "utf-8",
-      maxBuffer: 16 * 1024 * 1024
-    });
-    const found = harnessFromPsTable(ps, process.pid);
-    if (found) {
-      return { harness: found, source: "ancestry" };
+  if (process.platform !== "win32") {
+    try {
+      const ps = childProcess.execFileSync("ps", ["-Ao", "pid=,ppid=,command="], {
+        encoding: "utf-8",
+        maxBuffer: 16 * 1024 * 1024,
+        stdio: ["ignore", "pipe", "pipe"]
+      });
+      const found = harnessFromPsTable(ps, process.pid);
+      if (found) {
+        return { harness: found, source: "ancestry" };
+      }
+    } catch {
     }
-  } catch {
   }
-  if (process.env.PI_CODING_AGENT) {
-    return { harness: "pi", source: "env" };
+  const fromEnv = harnessFromEnv(process.env);
+  if (fromEnv) {
+    return { harness: fromEnv, source: "env" };
   }
-  if (process.env.CODEX_THREAD_ID || process.env.CODEX_SANDBOX) {
-    return { harness: "codex", source: "env" };
+  return { harness: null, source: "none" };
+}
+function harnessFromEnv(env) {
+  if (env.PI_CODING_AGENT) {
+    return "pi";
   }
-  if (process.env.CLAUDECODE || process.env.CLAUDE_CODE_SESSION_ID) {
-    return { harness: "claude-code", source: "env" };
+  if (env.CODEX_THREAD_ID || env.CODEX_SANDBOX) {
+    return "codex";
   }
-  return { harness: null, source: "none" };
+  if (env.OPENCODE || env.OPENCODE_PID || env.OPENCODE_BINARY) {
+    return "opencode";
+  }
+  if (env.CLAUDECODE || env.CLAUDE_CODE_SESSION_ID) {
+    return "claude-code";
+  }
+  return null;
 }
 function detectHarness() {
   return detectHarnessDetailed().harness;
@@ -3781,7 +3794,7 @@ function parsePositiveInt(raw, flag) {
   }
   return Number.parseInt(raw, 10);
 }
-program.name("modlens").description("Plug-in vision for text-only LLMs: image in, structured JSON evidence out").version("3.16.0");
+program.name("modlens").description("Plug-in vision for text-only LLMs: image in, structured JSON evidence out").version("3.16.1");
 program.command("analyze", { isDefault: true }).description("Analyze an image into structured JSON evidence (default command)").requiredOption("-i, --input <path|url>", "Input image path or https URL").option("-o, --output <path>", "Write result JSON to a file").option("-m, --model <name>", "Provider model name").option("-p, --provider <name>", `Vision provider (${listProviders().join(", ")})`).option("--prompt <text>", "Extra focus for this image").option("--timeout <ms>", "Provider timeout in milliseconds", "180000").option("--provider-bin <path>", "Provider binary path (default: agy)").option("--workdir <path>", "Working directory for the provider").option(
   "--extra-body <json>",
   `JSON merged into the API request body, e.g. '{"thinking":{"type":"disabled"}}'`

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@liustack/modlens",
-    "version": "3.16.0",
+    "version": "3.16.1",
     "description": "Plug-in vision for text-only LLMs, powered by the free Antigravity CLI",
     "type": "module",
     "bin": {

package/skills/modlens/SKILL.md

@@ -20,11 +20,11 @@ powershell -ExecutionPolicy Bypass -File <skill-dir>\scripts\run.ps1 <args>
 
 It resolves a working runtime (PATH `modlens`, then `npx`, then `bunx`) and forwards your arguments unchanged. Exit 78 means no runtime: relay the `nextSteps` from its stderr JSON instead of retrying.
 
-If your harness forbids running scripts, reason through the same order by hand and run the first line that works (the pinned version is 3.16.0):
+If your harness forbids running scripts, reason through the same order by hand and run the first line that works (the pinned version is 3.16.1):
 
-1. A `modlens` on `PATH` whose major version is 3 and is at least 3.16.0: `modlens <args>`.
-2. Otherwise, if `npx` exists: `npx --yes --package @liustack/modlens@3.16.0 modlens <args>`.
-3. Otherwise, if `bunx` exists: `bunx --bun @liustack/modlens@3.16.0 <args>`.
+1. A `modlens` on `PATH` whose major version is 3 and is at least 3.16.1: `modlens <args>`.
+2. Otherwise, if `npx` exists: `npx --yes --package @liustack/modlens@3.16.1 modlens <args>`.
+3. Otherwise, if `bunx` exists: `bunx --bun @liustack/modlens@3.16.1 <args>`.
 4. Otherwise tell the user no JavaScript runtime was found and that installing Node 22.19+ (https://nodejs.org) or Bun (https://bun.sh) is the next step. Do not claim modlens itself failed.
 
 `references/runtime.md` documents the pin and the diagnostic fields.

package/skills/modlens/references/runtime.md

@@ -8,7 +8,7 @@ shell syntax.
 
 ## Pinned version
 
-- Pinned CLI version: 3.16.0
+- Pinned CLI version: 3.16.1
 - npm package: `@liustack/modlens`
 - CLI binary name: `modlens`
 

package/skills/modlens/scripts/run.ps1

@@ -24,7 +24,7 @@ $ErrorActionPreference = 'Stop'
 # package.json version, and the release script rewrites it on every bump.
 $Package = '@liustack/modlens'
 $Bin = 'modlens'
-$Pinned = '3.16.0'
+$Pinned = '3.16.1'
 # -------------------------------------------------------------------------------
 
 $NativeNote = 'no native artifact is published for this tool yet; phase A ships npm launch paths only'

package/skills/modlens/scripts/run.sh

@@ -22,7 +22,7 @@ set -eu
 # package.json version, and the release script rewrites it on every bump.
 PKG="@liustack/modlens"
 BIN="modlens"
-PINNED="3.16.0"
+PINNED="3.16.1"
 # -------------------------------------------------------------------------------
 
 NATIVE_NOTE="no native artifact is published for this tool yet; phase A ships npm launch paths only"