@liustack/modlens 3.16.4 → 3.16.5

package/CHANGELOG.md

@@ -1,8 +1,13 @@
 # Changelog
 
+## 3.16.5 - 2026-08-15
+
+- **dsh: pasting into a plain text-only model works again ([#36](https://github.com/liustack/modlens/issues/36)).** The paste takeover shipped in 3.14.0 has been dead in every default install since 3.16.0 moved the verdict server-side. The verdict refuses when any model matching the selector label declares image input, which is right for a real vision model and wrong for the one case it could not see: this plugin's own `(modlens vision)` wrapper reuses the upstream model id verbatim and declares image input, because that declaration is exactly how the wrapper unlocks admission. So selecting plain `DeepSeek-V4-Pro` matched the real text-only model and the plugin's synthetic twin of it, the twin vetoed, and the paste fell through to dsh's own gate and its `MODEL_DOES_NOT_SUPPORT_IMAGES`. The verdict now skips a twin two ways, both requiring proof rather than a name: the provider ids this instance actually registered, tracked as each wrapper lands, and a model that carries the `(modlens vision)` marker on a provider id minted by the rule this plugin uses, which is how a sibling instance in the same process is recognized. A real vision provider still vetoes even if it borrows the marker, and an id someone else already holds is never trusted as ours. The verdict had no test at all, which is how a regression this total shipped and stayed for six releases; it now has an integration suite driving the real route against a registry shaped like a live install. Thanks to @Taz-dingo for a report that arrived with the conflicting rules already quoted side by side.
+- **The docs cannot print a stale install command any more.** Every file the repo tracks is scanned, and any install of this package that is not pinned to an exact version has to carry `--config.minimumReleaseAge=0` as an argument of that same command. Review found the first version of that check passing five different ways of writing an unpinned install, including a command split across lines and a spec like `3.16.4+local` that only looks pinned.
+
 ## 3.16.4 - 2026-08-15
 
-- **How to update is written down, and the explanation it replaces was wrong.** 3.10.0 claimed that pnpm's release-age gate has a 10-day window and that an explicit version or dist-tag skips it, so every install command in this repo carried `@latest` as the fix. Measured on a machine with no gate configured at all: pnpm 11 turns `minimumReleaseAge` on by default at 24 hours (`24 * 60` in its config reader; `pnpm config get` does not surface that particular default), and `add @liustack/modlens@latest` installed 3.9.1 while 3.16.3 was the published latest. The gate filters the candidate versions before the tag is resolved, so the tag lands on an older one, and a day of held-back wall-clock time can be several releases on a fast week. The pnpm issue behind the original claim describes a bug in pnpm 10.16.1 that was fixed, and says in its own text that `@latest` was subject to the gate. What does work is naming the version, a deliberate request rather than a resolution: pnpm 11 installs it and records that one version as an approved exception, leaving everything else behind the window. Where a stricter policy is configured it refuses instead, and `--config.minimumReleaseAge=0` lifts the gate for one command, for every package that command resolves. `harness-setup` now has an Updating section covering both install shapes, including why `update` cannot cross a major (it stays inside the recorded semver range) and how to check what actually landed.
+- **How to update is written down, and the explanation it replaces was wrong.** 3.10.0 claimed that pnpm's release-age gate has a 10-day window and that an explicit version or dist-tag skips it, so every install command in this repo carried `@latest` as the fix. Measured on a machine with no gate configured at all: pnpm 11 turns `minimumReleaseAge` on by default at 24 hours (`24 * 60` in its config reader; `pnpm config get` does not surface that particular default), and asking for `@latest` installed 3.9.1 while 3.16.3 was the published latest. The gate filters the candidate versions before the tag is resolved, so the tag lands on an older one, and a day of held-back wall-clock time can be several releases on a fast week. The pnpm issue behind the original claim describes a bug in pnpm 10.16.1 that was fixed, and says in its own text that `@latest` was subject to the gate. What does work is naming the version, a deliberate request rather than a resolution: pnpm 11 installs it and records that one version as an approved exception, leaving everything else behind the window. Where a stricter policy is configured it refuses instead, and `--config.minimumReleaseAge=0` lifts the gate for one command, for every package that command resolves. `harness-setup` now has an Updating section covering both install shapes, including why `update` cannot cross a major (it stays inside the recorded semver range) and how to check what actually landed.
 
 ## 3.16.3 - 2026-08-15
 
@@ -74,7 +79,7 @@
 
 ## 3.9.0 - 2026-08-13
 
-- **The first plug-in vision plugin for DeepSeek Harness (dsh).** The npm package is now also a dsh bundle: `dsh plugin --profile <name> add @liustack/modlens` is the whole install. It registers a native `read_image` tool (schema in every model request, so there is no trigger heuristic at all) that spawns the modlens CLI shipped in the same package, declares the vision schema as its canonical output contract, and renders evidence text for the model. Phase 2 rides `agent/pre-step`: images pasted or dropped into the dsh Web UI are read automatically and enter the step as modlens evidence blocks, with failed reads degrading to an explanatory note instead of rejecting the step (`autoRead: false` in the plugin row turns this off). The plugin imports no dsh packages (raw JSON-Schema tool registration, node builtins only), which is also the smallest possible surface against developer-preview churn. Verified end to end on a real dsh headless profile: the DeepSeek model called `read_image` and quoted the exact transcription back.
+- **The first plug-in vision plugin for DeepSeek Harness (dsh).** The npm package is now also a dsh bundle, so a single `dsh plugin --profile <name> add` is the whole install (the current command, with its version named, is in [INSTALL.md](INSTALL.md)). It registers a native `read_image` tool (schema in every model request, so there is no trigger heuristic at all) that spawns the modlens CLI shipped in the same package, declares the vision schema as its canonical output contract, and renders evidence text for the model. Phase 2 rides `agent/pre-step`: images pasted or dropped into the dsh Web UI are read automatically and enter the step as modlens evidence blocks, with failed reads degrading to an explanatory note instead of rejecting the step (`autoRead: false` in the plugin row turns this off). The plugin imports no dsh packages (raw JSON-Schema tool registration, node builtins only), which is also the smallest possible surface against developer-preview churn. Verified end to end on a real dsh headless profile: the DeepSeek model called `read_image` and quoted the exact transcription back.
 - **Grok Build joins as the fifth reusable harness.** `reuse.grok` grants the local Grok CLI login as an engine: discovery reads `~/.grok` (OAuth evidence in auth.json, model ids from models_cache.json judged by the builtin vision table), and the route drives headless `grok -p` with `--json-schema` (which accepts this project's schema unmodified; the structuredOutput field carries the conforming answer) and `--allow Read`, following the claude-cli template since headless grok has no image-attach flag. Verified live: an exact OCR read through a real SuperGrok login. The agent region order becomes antigravity, codex, opencode, grok, pi-cli, claude-cli.
 
 ## 3.8.0 - 2026-08-13

package/README.md

@@ -34,7 +34,7 @@ Issues are welcome any time: [open one](https://github.com/liustack/modlens/issu
 
 ## Highlights
 
-**🥇 The first vision plugin for DeepSeek Harness (dsh):** one command, `npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@latest`, and the text-only DeepSeek model behind dsh reads images through a native `modlens_read_image` tool. Updating is the same command again, with one wrinkle worth knowing: pnpm 11 holds back releases published in the last 24 hours, and `@latest` does not skip that, so a same-day fix needs its version named ([how to update](docs/harness-setup.md#keeping-it-up-to-date)).
+**🥇 The first vision plugin for DeepSeek Harness (dsh):** one command, `npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@3.16.5`, and the text-only DeepSeek model behind dsh reads images through a native `modlens_read_image` tool. Updating is the same command again. The version is named rather than `@latest` on purpose: pnpm 11 holds back releases published in the last 24 hours and resolves the tag against what survives, so `@latest` would install whatever shipped a day ago ([details](docs/harness-setup.md#keeping-it-up-to-date)).
 
 Pasting an image works two ways. **① Just paste.** On a text-only model the pasted image lands as a private temp file and its path enters the composer — the same interaction OpenCode and Pi ship — and the `modlens_read_image` tool takes it from there. **② Pick a `(modlens vision)` entry** in the model selector (it remembers your choice, so once is enough), then paste: the thumbnail stays visible in your message, closer to the Codex app feel, and the image is converted to structured evidence at request time, answered by the same underlying route. The plugin auto-discovers every provider route carrying text-only DeepSeek or GLM models and adds a wrapped entry per route (a stock install gets **`DeepSeek-V4-Flash (modlens vision)`** and **`DeepSeek-V4-Pro (modlens vision)`**; extra routes like opencode-go or zai get their own); the two families' own vision models are excluded automatically. Which paste route applies is the host's per-model call: only a model its metadata positively confirms text-only is taken over, anything unconfirmed is left alone, so vision models keep their native paste ([details](docs/harness-setup.md)).
 

package/README.zh-CN.md

@@ -34,7 +34,7 @@ DeepSeek 和 GLM 的主力对话模型是纯文本的，无法进行图片识别
 
 ## 亮点
 
-**🥇 全网第一个支持 DeepSeek Harness（dsh）的外挂视觉识别插件：**一条命令 `npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@latest`，dsh 背后的纯文本 DeepSeek 模型即可通过原生 `modlens_read_image` 工具读图。更新就是再跑一遍同一条命令，有一个坑值得知道：pnpm 11 会扣住最近 24 小时内发布的版本，而 `@latest` 跳不过去，所以想要当天的修复得点名版本号（[怎么更新](docs/harness-setup.zh-CN.md#保持更新)）。
+**🥇 全网第一个支持 DeepSeek Harness（dsh）的外挂视觉识别插件：**一条命令 `npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@3.16.5`，dsh 背后的纯文本 DeepSeek 模型即可通过原生 `modlens_read_image` 工具读图。更新就是再跑一遍同一条命令。这里点名版本号而不用 `@latest` 是有意的：pnpm 11 会扣住最近 24 小时内发布的版本，dist-tag 只在剩下的里面解析，用 `@latest` 装到的会是一天前发布的那个（[细节](docs/harness-setup.zh-CN.md#保持更新)）。
 
 DeepSeek Harness 粘贴识图有两种玩法。
 
@@ -69,7 +69,7 @@ agy                                                           # 浏览器完成
 **DeepSeek Harness（dsh）用户不走 skill 流程**，本包就是原生 dsh 插件：
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@latest
+npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@3.16.5
 ```
 
 装完即有 `modlens_read_image` 工具，选「(modlens vision)」模型变体即可直接粘贴识图。引擎配置同样在 `~/.modlens`，详见[宿主接入](docs/harness-setup.zh-CN.md)。

package/dist/main.js

@@ -4069,7 +4069,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.4");
+program.name("modlens").description("Plug-in vision for text-only LLMs: image in, structured JSON evidence out").version("3.16.5");
 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"}}'`
@@ -4179,7 +4179,7 @@ program.command("doctor").description(
       configPath: CONFIG_PATH,
       // Lets doctor name an installed skill copy that is older than
       // the CLI reporting on it (issue #33).
-      version: "3.16.4"
+      version: "3.16.5"
     });
     const output = options.json ? JSON.stringify(report, null, 2) : renderDoctorReport(report);
     process.stdout.write(`${output}

package/docs/harness-setup.md

@@ -55,7 +55,7 @@ OpenCode with DeepSeek: `opencode auth login`, pick DeepSeek and paste the key (
 dsh is different from the other harnesses: modlens plugs in as a native tool, not a prompt-triggered skill. The package itself is a dsh bundle, so one command installs it into a profile:
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@latest
+npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@3.16.5
 ```
 
 This registers a `modlens_read_image` tool whose schema reaches the model on every request (no trigger heuristics), runs the modlens CLI shipped inside the same package, and returns the structured evidence as the tool's canonical JSON output. Engines, reuse grants, and guard rules stay in `~/.modlens/config.json`, shared with every other harness. dsh is in developer preview and its plugin surface may change; the plugin keeps its touch small (raw tool registration, the llm adapter surface for the vision variants, the attachment reader, and one agent pre-step hook) and degrades loudly if any of them moves.
@@ -63,35 +63,35 @@ This registers a `modlens_read_image` tool whose schema reaches the model on eve
 ### Keeping it up to date
 
 modlens ships often, and both install shapes freeze at whatever version they
-got. On dsh, re-run the install. `add` is the command, not `update`: `update`
-stays inside the semver range already recorded, and the range a plain install
-writes is a caret one, so a profile that once landed on 2.7.1 updates to 2.8.0
-and never crosses into 3.x.
+got. On dsh, re-run the install with the version named:
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@latest
+npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.5
 ```
 
-Restart dsh, then check what actually landed with
-`npx -y @deepseek-ai/dsh plugin --profile <name> list`. That check is worth
-running, because what you get may not be the newest release: pnpm 11 holds back
-anything published in the last 24 hours (`minimumReleaseAge`, on by default) and
-resolves silently to the newest release older than that. `@latest` does not
-change it: the gate filters the candidates before the tag is resolved, so the
-tag lands on an older one. What that costs is a day of wall-clock time, not one
-version, so on a fast-moving week it can be several releases back.
+`npm view @liustack/modlens version` prints the current one, and this page is
+stamped with it at release time.
 
-When it is not, name the version instead of asking for a tag:
+Two things in that command are deliberate. `add`, not `update`, because
+`update` stays inside the semver range already recorded, and a plain install
+records a caret range, so a profile that once landed on 2.7.1 updates to 2.8.0
+and never crosses into 3.x. And a named version rather than `@latest`, because
+pnpm 11 holds back anything published in the last 24 hours
+(`minimumReleaseAge`, on by default) and resolves the tag against what survives
+that filter: `@latest` lands on an older release instead of skipping the gate.
+What it costs is a day of wall-clock time rather than one version, which on a
+fast-moving week is several releases. A named version is a deliberate request,
+so pnpm installs it, and since 11.1.3 records that one version as an approved
+exception in the profile's `pnpm-workspace.yaml`, leaving everything else
+behind the window.
+
+Restart dsh, then confirm what actually landed:
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.4
+npx -y @deepseek-ai/dsh plugin --profile <name> list
 ```
 
-`npm view @liustack/modlens version` prints the current one. A named version is
-a deliberate request rather than a resolution, so pnpm 11 installs it, and
-since 11.1.3 records that one version as an approved exception in the profile's
-`pnpm-workspace.yaml`; everything else stays behind the window. The
-[troubleshooting page](troubleshooting.md#dsh-says-declares-no-dshbundle--installed-as-a-plain-dependency)
+The [troubleshooting page](troubleshooting.md#dsh-says-declares-no-dshbundle--installed-as-a-plain-dependency)
 covers the stricter case, where you configured `minimumReleaseAge` yourself and
 pnpm refuses rather than approves.
 

package/docs/harness-setup.zh-CN.md

@@ -55,28 +55,30 @@ OpenCode 接 DeepSeek：执行 `opencode auth login`，选择 DeepSeek 并粘贴
 dsh 与其他 harness 不同：modlens 以原生工具的形式接入，而不是靠提示词触发的 skill。本包自身就是一个 dsh bundle，一条命令即可装进某个 profile：
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@latest
+npx -y @deepseek-ai/dsh plugin --profile web add @liustack/modlens@3.16.5
 ```
 
 这会注册一个 `modlens_read_image` 工具，它的 schema 随每次请求抵达模型（不靠触发启发式），运行同一个包里自带的 modlens CLI，并把结构化证据作为工具的标准 JSON 输出返回。引擎、复用授权和 guard 规则仍在 `~/.modlens/config.json` 里，与其他所有 harness 共享。dsh 还在开发者预览阶段，插件接口可能变化。这个插件刻意保持很小的接触面（原生工具注册、视觉变体所用的 llm 适配层、附件读取器，以及一个 agent 执行前钩子），其中任何一处变动，它都会大声报错而不是无声退化。
 
 ### 保持更新
 
-modlens 发布很频繁，而两种安装形态都会冻结在装进来的那个版本上。dsh 上重跑一遍安装即可。命令是 `add` 而不是 `update`：`update` 只在 package.json 里已记录的 semver 范围内挪动，而普通安装写进去的是 caret 范围，所以一个当初装到 2.7.1 的 profile 只会更新到 2.8.0，永远进不了 3.x。
+modlens 发布很频繁，而两种安装形态都会冻结在装进来的那个版本上。dsh 上重跑一遍安装即可，版本号要点名：
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@latest
+npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.5
 ```
 
-重启 dsh，然后用 `npx -y @deepseek-ai/dsh plugin --profile <name> list` 看看实际装到了什么。这一步值得做，因为拿到的未必是最新版：pnpm 11 会扣住最近 24 小时内发布的版本（`minimumReleaseAge`，默认开启），静默解析到比它更早的那个最新版本。`@latest` 改变不了这一点：冷静期先把候选版本过滤掉，dist-tag 才在剩下的里面解析，于是它落到了更旧的那个上。代价是一天的自然时间，不是一个版本，发布密集的一周里可能落后好几个版本。
+`npm view @liustack/modlens version` 可以查到当前版本号，本页的版本号则由发布流程自动写入。
 
-需要今天的版本时，改成点名版本号，而不是要一个 tag：
+这条命令里有两处是刻意的。用 `add` 而不是 `update`，因为 `update` 只在已记录的 semver 范围内挪动，而普通安装写进去的是 caret 范围，所以一个当初装到 2.7.1 的 profile 只会更新到 2.8.0，永远进不了 3.x。点名版本号而不用 `@latest`，则是因为 pnpm 11 会扣住最近 24 小时内发布的版本（`minimumReleaseAge`，默认开启），dist-tag 只在通过过滤的候选里解析：`@latest` 会落到更旧的版本上，而不是跳过冷静期。代价是一天的自然时间，不是一个版本，发布密集的一周里就是好几个版本。点名版本是一次明确的指定，所以 pnpm 会装上它，11.1.3 起还会把这一个版本作为已批准的例外写进该 profile 的 `pnpm-workspace.yaml`，其余一切仍留在窗口后面。
+
+重启 dsh，然后确认实际装到了什么：
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.4
+npx -y @deepseek-ai/dsh plugin --profile <name> list
 ```
 
-`npm view @liustack/modlens version` 可以查到当前版本号。点名版本是一次明确指定而不是一次解析，所以 pnpm 11 会装上它，11.1.3 起还会把这一个版本作为已批准的例外写进该 profile 的 `pnpm-workspace.yaml`，其余一切仍留在窗口后面。更严格的情况（你自己配过 `minimumReleaseAge`，pnpm 会拒绝而不是批准）见[故障排查](troubleshooting.zh-CN.md#dsh-提示-declares-no-dshbundle--installed-as-a-plain-dependency)。
+更严格的情况（你自己配过 `minimumReleaseAge`，pnpm 会拒绝而不是批准）见[故障排查](troubleshooting.zh-CN.md#dsh-提示-declares-no-dshbundle--installed-as-a-plain-dependency)。
 
 skill 类 harness 上，skill 是一个拷贝出来的文件夹，拷贝会保留安装时的版本，重跑安装原地覆盖即可。`modlens doctor` 会读出它能找到的每一份拷贝里钉住的版本，并标出落后于当前 CLI 的那些，让版本漂移在坑到人之前就先暴露出来。
 

package/docs/troubleshooting.md

@@ -137,7 +137,7 @@ simply lands on an older one. Name the exact version instead, which pnpm treats
 as a deliberate request rather than a resolution:
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.4
+npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.5
 ```
 
 `npm view @liustack/modlens version` prints the current one. pnpm 11 installs a named
@@ -152,7 +152,7 @@ file:
 
 ```yaml
 minimumReleaseAgeExclude:
-  - '@liustack/modlens@3.16.4'
+  - '@liustack/modlens@3.16.5'
 ```
 
 Or lift the gate for a single command, which lifts it for everything that

package/docs/troubleshooting.zh-CN.md

@@ -128,7 +128,7 @@ dsh profile 装到的是旧版 modlens。`dsh.bundle` 声明从 3.9.0 起才存
 `@latest` 绕不开这一层，本页早先的说法是错的。冷静期先把候选版本过滤掉，dist-tag 才在剩下的里面解析，于是它直接落到了更旧的那个上。改成写死精确版本号，pnpm 会把它当作一次明确的指定，而不是一次解析：
 
 ```sh
-npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.4
+npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.5
 ```
 
 `npm view @liustack/modlens version` 可以查到当前版本号。pnpm 11 会装上被点名的版本，11.1.3 起还会把它作为一条已批准的例外写进该 profile 的 `pnpm-workspace.yaml`，其余所有包和 modlens 以后的版本仍然留在窗口后面。
@@ -137,7 +137,7 @@ npx -y @deepseek-ai/dsh plugin --profile <name> add @liustack/modlens@3.16.4
 
 ```yaml
 minimumReleaseAgeExclude:
-  - '@liustack/modlens@3.16.4'
+  - '@liustack/modlens@3.16.5'
 ```
 
 或者只为这一条命令解除冷静期，注意它解除的是这条命令解析到的所有包，不只 modlens：

package/dsh/index.js

@@ -40,8 +40,15 @@ export function apply(ctx, config = {}) {
   if (config.autoRead === true) {
     registerAutoRead(ctx)
   }
+  // The provider ids this plugin registered itself. The takeover verdict has
+  // to skip them: our wrapper models are synthetic twins of upstream ones,
+  // carrying the upstream id and declaring image input, so a plain text-only
+  // label matches the twin and the twin's declaration vetoes the takeover
+  // that label deserved (issue #36). Filled by registerVisionProvider as
+  // wrappers land, including the later sweeps, and read by the verdict.
+  const ownProviders = new Set()
   if (config.visionProvider !== false) {
-    registerVisionProvider(ctx, config)
+    registerVisionProvider(ctx, config, ownProviders)
   }
   // Paste-to-path: the browser half (dsh/client.js) intercepts image pastes
   // and POSTs the bytes here; the file lands in a private temp dir and the
@@ -57,7 +64,7 @@ export function apply(ctx, config = {}) {
       try {
         // scope carries webServer; the plugin's own ctx carries llm for the
         // takeover verdicts.
-        registerPasteRoute(scope, ctx)
+        registerPasteRoute(scope, ctx, ownProviders)
       } catch (error) {
         console.error(`[modlens] paste-to-path route skipped: ${error}`)
       }
@@ -215,7 +222,14 @@ const PASTE_MAX_BYTES = 25 * 1024 * 1024
  * unresolvable answers false: the native path is the safe default, and a
  * text-only model merely keeps its old error message.
  */
-async function pasteTakeoverVerdict(host, label) {
+// The provider ids registerVisionProvider mints: the legacy deepseek wrap and
+// the `modlens-<upstream>` form auto-discovery uses. A sibling instance of
+// this plugin derives its ids the same way, which is what makes the pair of
+// checks below meaningful. A custom `config.providerId` is outside the
+// convention on purpose and is covered by the registered-id set instead.
+const OWN_PROVIDER_ID = /^(deepseek-modlens$|modlens-)/
+
+async function pasteTakeoverVerdict(host, label, ownProviders) {
   if (typeof label !== 'string' || label.trim() === '') return false
   // Our own wrappers convert pastes at request time with the thumbnail
   // preserved; taking their paste over would defeat the better path.
@@ -229,6 +243,14 @@ async function pasteTakeoverVerdict(host, label) {
   for (const info of llm.listProviders()) {
     const providerId = info?.id
     if (!providerId) continue
+    // Our own wrapper: every model in it is a synthetic twin of an upstream
+    // one, carrying that upstream id and declaring image input because that
+    // is how the wrapper unlocks admission. Scanning it means a plain
+    // text-only label matches the twin by id and the twin vetoes the
+    // takeover the real model deserved (issue #36). Only ids this plugin
+    // registered itself are skipped, so a real vision provider, including
+    // one that happens to be named like ours, still votes.
+    if (ownProviders?.has(providerId)) continue
     let models = []
     try {
       models = await llm.listModels(providerId)
@@ -236,6 +258,21 @@ async function pasteTakeoverVerdict(host, label) {
       return false
     }
     for (const model of models) {
+      // A twin from another instance of this plugin, which the set above
+      // cannot know about: a second apply() in the same process hits the
+      // duplicate branch, does not claim the id, and would otherwise be
+      // vetoed by the first instance's wrapper. Both halves are required.
+      // The name marker alone proves nothing, since any provider can put that
+      // string in a model name and would then slip past a veto it deserves;
+      // the id is what makes it ours, because a sibling instance derives its
+      // provider id from the same rule this one does.
+      if (
+        OWN_PROVIDER_ID.test(providerId) &&
+        typeof model?.name === 'string' &&
+        /\(modlens vision\)/i.test(model.name)
+      ) {
+        continue
+      }
       for (const candidate of [model?.name, model?.id]) {
         if (typeof candidate !== 'string' || candidate.length === 0) continue
         if (!lowered.includes(candidate.toLowerCase())) continue
@@ -272,7 +309,7 @@ const PASTE_VERDICT_CAP = 32
  * stands down instead of swallowing pastes into a 404. Bound to the dsh web
  * server, which listens on loopback by default.
  */
-function registerPasteRoute(ctx, host) {
+function registerPasteRoute(ctx, host, ownProviders) {
   const verdicts = new Map()
   // The cache key is only the selector label, which cannot tell two
   // same-named models on different routes apart. A route mounting mid-TTL
@@ -310,7 +347,7 @@ function registerPasteRoute(ctx, host) {
             let attempts = 0
             for (;;) {
               const startedEpoch = topologyEpoch
-              takeover = await pasteTakeoverVerdict(host, label)
+              takeover = await pasteTakeoverVerdict(host, label, ownProviders)
               if (topologyEpoch === startedEpoch) {
                 verdicts.delete(label)
                 verdicts.set(label, { takeover, at: Date.now() })
@@ -398,7 +435,7 @@ function registerPasteRoute(ctx, host) {
  *   deepseek-official wrap keeps its historical `deepseek-modlens` id, so a
  *   selector remembering that provider survives the upgrade.
  */
-function registerVisionProvider(ctx, config) {
+function registerVisionProvider(ctx, config, ownProviders) {
   // Wrap only the text-only members of these families. Their own vision
   // models (present or future: deepseek-vl/ocr/janus, glm-4.5v, glm-5v-...)
   // need no bridge and are excluded by name and by declared modality.
@@ -464,6 +501,11 @@ function registerVisionProvider(ctx, config) {
         },
         evidenceCache: new Map(),
       })
+      // Trusted as ours only on a registration this call actually made. A
+      // duplicate below means someone else holds that id, and skipping a
+      // provider we do not own would let a real vision model's paste be
+      // taken over, which is the bug the verdict exists to prevent.
+      ownProviders?.add(providerId)
       return true
     } catch (error) {
       // A duplicate means a concurrent or earlier registration already won:

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@liustack/modlens",
-    "version": "3.16.4",
+    "version": "3.16.5",
     "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.4):
+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.5):
 
-1. A `modlens` on `PATH` whose major version is 3 and is at least 3.16.4: `modlens <args>`.
-2. Otherwise, if `npx` exists: `npx --yes --package @liustack/modlens@3.16.4 modlens <args>`.
-3. Otherwise, if `bunx` exists: `bunx --bun @liustack/modlens@3.16.4 <args>`.
+1. A `modlens` on `PATH` whose major version is 3 and is at least 3.16.5: `modlens <args>`.
+2. Otherwise, if `npx` exists: `npx --yes --package @liustack/modlens@3.16.5 modlens <args>`.
+3. Otherwise, if `bunx` exists: `bunx --bun @liustack/modlens@3.16.5 <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.4
+- Pinned CLI version: 3.16.5
 - 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.4'
+$Pinned = '3.16.5'
 # -------------------------------------------------------------------------------
 
 $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.4"
+PINNED="3.16.5"
 # -------------------------------------------------------------------------------
 
 NATIVE_NOTE="no native artifact is published for this tool yet; phase A ships npm launch paths only"