dsh-zotero 0.8.1 → 0.8.3

package/README.en.md

@@ -10,7 +10,7 @@
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/v/dsh-zotero" alt="npm version" style="max-width:100%;"></a>
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/dm/dsh-zotero" alt="npm downloads" style="max-width:100%;"></a>
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/l/dsh-zotero" alt="license" style="max-width:100%;"></a>
-  <a href="https://github.com/deepseek-ai/deepseek-harness/releases/tag/dsh-v0.1.5-rc.1"><img src="https://img.shields.io/badge/dsh-%3E%3D0.1.5--rc.1-blue" alt="minimum dsh version" style="max-width:100%;"></a>
+  <a href="https://github.com/deepseek-ai/deepseek-harness/releases/tag/dsh-v0.1.5-rc.2"><img src="https://img.shields.io/badge/dsh-%3E%3D0.1.5--rc.2-blue" alt="minimum dsh version" style="max-width:100%;"></a>
   <a href="https://awesome-dsh-plugin.com"><img src="https://awesome-dsh-plugin.com/badge.svg" alt="Awesome DSH Plugin"></a>
 </p>
 </div>
@@ -61,7 +61,7 @@ dsh plugin --profile <name> add ./dsh-zotero-*.tgz
 
 After installing, start a new session so the agent picks up the Zotero tools.
 
-The plugin provides a settings card under **Settings → Plugins** where you can adjust the API address, concurrency limits, full-text retrieval toggle, and more. Changes take effect on save. See [Configuration](docs/configuration.md).
+The plugin provides a settings page under **Settings → Zotero** — a left-nav entry beside General, Models, and Plugins — where you can adjust the API address, concurrency limits, full-text retrieval toggle, and more. Changes take effect on save. See [Configuration](docs/configuration.md).
 
 [Installation details →](docs/getting-started.md)
 
@@ -69,7 +69,7 @@ The plugin provides a settings card under **Settings → Plugins** where you can
 
 - Zotero ≥ 7 with local API enabled: **Settings → Advanced → "Allow other applications on this computer to communicate with Zotero"**
 - Node.js ≥ 22.19 (or ≥ 24)
-- dsh 0.1.5-rc.1 host (all `@deepseek-ai/dsh-*` peer dependencies are `^0.1.5-rc.1`, and the declared compatibility range is `engines.dsh`; only the latest pre-release line is supported until upstream stabilizes, with no backward compatibility)
+- dsh 0.1.5-rc.2 host (all `@deepseek-ai/dsh-*` peer dependencies are `^0.1.5-rc.2`, and the declared compatibility range is `engines.dsh`; only the latest pre-release line is supported until upstream stabilizes, with no backward compatibility)
 - Local API at `http://127.0.0.1:23119/api`, unauthenticated, read-only
 
 ## Usage example
@@ -78,20 +78,23 @@ The agent calls tools step by step during a conversation. Each result becomes co
 
 ```text
 User: Find papers about Risk
-Agent → zotero_search(query: "Risk", itemType: "journalArticle")
+Agent → zotero_search(query: "Risk", itemTypes: ["journalArticle"])
        5 matches; user picks the first 3
 
 User: What does the first one's abstract say?
-Agent → zotero_get(ref: 1, fields: ["abstractNote"])
-       Returns the full abstract
+Agent → zotero_get(ref: "zotero://user/0/item/ABCD1234")
+       Returns the full abstract (the standard model carries it)
 
 User: Find the methodology discussion in this paper
-Agent → zotero_retrieve(query: "methodology", sources: ["fulltext", "notes"])
+Agent → zotero_retrieve(ref: "zotero://user/0/item/ABCD1234", query: "methodology",
+                        sources: ["fulltext", "note"])
        Returns relevant passages with page numbers
 
 User: Export all three as BibTeX
-Agent → zotero_export(refs: [1,2,3], format: "bibtex")
-       Generates BibTeX entries, ready to copy or download
+Agent → zotero_export(refs: ["zotero://user/0/item/ABCD1234",
+                             "zotero://user/0/item/EFGH5678",
+                             "zotero://user/0/item/IJKL9012"], format: "bibtex")
+       Generates BibTeX entries; the UI can download them, the model reads the same text
 ```
 
 More examples in [Features](docs/features.md).
@@ -101,7 +104,7 @@ More examples in [Features](docs/features.md).
 - **Read-only library**: all operations are reads; items, notes, tags, and collections are unchanged
 - **Loopback only**: network requests go only to `127.0.0.1:23119`
 - **Evidence ranking is term-based**: BM25 ranks passages by query-term frequency match
-- **Exports are static text**: returned as text, ready to copy into your target document
+- **Exports are static text**: the tool returns text, and that is what the model reads; the Zotero panel offers one-click copy or file download (`.bib`, `.ris`, `.json`), so nothing has to be retyped
 - **Full-text evidence depends on Zotero's index**: unindexed PDFs yield no full-text passages
 - **Attachment depth depends on the harness**: `zotero_attachment` returns the file location; reading the PDF further needs a matching host capability
 
@@ -109,7 +112,7 @@ More examples in [Features](docs/features.md).
 
 - **Network**: HTTP requests go only to `http://127.0.0.1:23119/api` (redirects are not followed); `resolveConfig` enforces a loopback address
 - **Filesystem**: read-only — `zotero_attachment` verifies attachment paths with `existsSync`; no file writes
-- **Persistence**: the only write comes from the settings card under Settings → Plugins, saved to the `zotero:` user layer of `$DSH_HOME/settings.yaml`
+- **Persistence**: the only write comes from the Zotero settings page in the left navigation, saved to the `zotero:` user layer of `$DSH_HOME/settings.yaml`
 - **No shell / native / background tasks**: the plugin runs no shell commands, loads no native modules, and starts no daemon
 - **Restart**: after installing or removing the plugin, restart dsh and start a new session; configuration changes hot-reload on save without a restart
 
@@ -120,7 +123,7 @@ More examples in [Features](docs/features.md).
 | [Getting Started](docs/getting-started.md) | Installation, prerequisites, first verification        |
 | [Features](docs/features.md)               | Sources panel, chat integration, evidence, exports     |
 | [Tool Reference](docs/tools.md)            | Parameters, return values, error codes for all 8 tools |
-| [Configuration](docs/configuration.md)     | 21 config fields, defaults, hot-reload                 |
+| [Configuration](docs/configuration.md)     | 22 config fields, defaults, hot-reload                 |
 | [Architecture](docs/architecture.md)       | Data flow, layer responsibilities, design boundaries   |
 | [Development](docs/development.md)         | Build, test, local development                         |
 | [Troubleshooting](docs/troubleshooting.md) | 11 common issues with symptoms and fixes               |
@@ -136,7 +139,7 @@ npm run dev                   # tsc --watch for host half hot reload
 npm run dev:client            # esbuild --watch for browser half hot reload
 ```
 
-Build output splits into `lib/` (Node side) and `lib/client.js` (browser side — settings card + Zotero tab). For full plugin development with both halves, use the `dev-lib.cordis.yml` overlay. See [Development](docs/development.md) for details.
+Build output splits into `lib/` (Node side) and `lib/client.js` (browser side — settings page + Zotero tab). For full plugin development with both halves, use the `dev-lib.cordis.yml` overlay. See [Development](docs/development.md) for details.
 
 ## License
 

package/README.md

@@ -10,7 +10,7 @@
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/v/dsh-zotero" alt="npm version" style="max-width:100%;"></a>
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/dm/dsh-zotero" alt="npm downloads" style="max-width:100%;"></a>
   <a href="https://www.npmjs.com/package/dsh-zotero"><img src="https://img.shields.io/npm/l/dsh-zotero" alt="license" style="max-width:100%;"></a>
-  <a href="https://github.com/deepseek-ai/deepseek-harness/releases/tag/dsh-v0.1.5-rc.1"><img src="https://img.shields.io/badge/dsh-%3E%3D0.1.5--rc.1-blue" alt="最低 dsh 版本" style="max-width:100%;"></a>
+  <a href="https://github.com/deepseek-ai/deepseek-harness/releases/tag/dsh-v0.1.5-rc.2"><img src="https://img.shields.io/badge/dsh-%3E%3D0.1.5--rc.2-blue" alt="最低 dsh 版本" style="max-width:100%;"></a>
   <a href="https://awesome-dsh-plugin.com"><img src="https://awesome-dsh-plugin.com/badge.svg" alt="Awesome DSH Plugin"></a>
 </p>
 </div>
@@ -61,7 +61,7 @@ dsh plugin --profile <name> add ./dsh-zotero-*.tgz
 
 安装后重启新建会话，Agent 即可使用 Zotero 工具。
 
-插件在 **Settings → Plugins** 中提供配置卡片，可调整 API 地址、并发限制、全文检索开关等参数，保存即生效。详见 [配置](docs/configuration.md)。
+插件在 **设置 → Zotero** 中提供一个配置页（与 General、Models、Plugins 并列的左侧导航项），可调整 API 地址、并发限制、全文检索开关等参数，保存即生效。详见 [配置](docs/configuration.md)。
 
 [安装详情 →](docs/getting-started.md)
 
@@ -69,7 +69,7 @@ dsh plugin --profile <name> add ./dsh-zotero-*.tgz
 
 - Zotero ≥ 7 桌面版，启用本地 API：**设置 → 高级 → "允许其他应用程序与 Zotero 通信"**
 - Node.js ≥ 22.19（或 ≥ 24）
-- 宿主 dsh 0.1.5-rc.1（`@deepseek-ai/dsh-*` peer 依赖均为 `^0.1.5-rc.1`，兼容范围另见 `engines.dsh`；上游发布稳定版之前只跟最新预发布线，不向后兼容）
+- 宿主 dsh 0.1.5-rc.2（`@deepseek-ai/dsh-*` peer 依赖均为 `^0.1.5-rc.2`，兼容范围另见 `engines.dsh`；上游发布稳定版之前只跟最新预发布线，不向后兼容）
 - 本地 API 地址 `http://127.0.0.1:23119/api`，无认证，只读
 
 ## 使用示例
@@ -78,20 +78,23 @@ Agent 在对话中根据用户需求逐步调用工具，每次调用的结果
 
 ```text
 用户：帮我找 Risk 相关的论文
-Agent → zotero_search(query: "Risk", itemType: "journalArticle")
+Agent → zotero_search(query: "Risk", itemTypes: ["journalArticle"])
        5 篇匹配结果，用户选择前 3 篇
 
 用户：第一篇的摘要说了什么？
-Agent → zotero_get(ref: 1, fields: ["abstractNote"])
-       返回摘要全文
+Agent → zotero_get(ref: "zotero://user/0/item/ABCD1234")
+       返回摘要全文（标准模型已含 abstract）
 
 用户：这篇里关于方法论的讨论，帮我找出来
-Agent → zotero_retrieve(query: "methodology", sources: ["fulltext", "notes"])
+Agent → zotero_retrieve(ref: "zotero://user/0/item/ABCD1234", query: "methodology",
+                        sources: ["fulltext", "note"])
        返回相关段落，带页码和来源
 
 用户：把这三篇导出为 BibTeX
-Agent → zotero_export(refs: [1,2,3], format: "bibtex")
-       生成 BibTeX 条目，可复制或下载
+Agent → zotero_export(refs: ["zotero://user/0/item/ABCD1234",
+                             "zotero://user/0/item/EFGH5678",
+                             "zotero://user/0/item/IJKL9012"], format: "bibtex")
+       生成 BibTeX 条目；界面可下载，模型读到同样的文本
 ```
 
 更多示例见 [功能概览](docs/features.md)。
@@ -101,7 +104,7 @@ Agent → zotero_export(refs: [1,2,3], format: "bibtex")
 - **只读文献库**：所有操作均为读取，不修改条目、笔记、标签或分类
 - **只访问本机**：网络请求仅发往 `127.0.0.1:23119`
 - **证据排序是词项相关性**：基于 BM25，按查询词与 passage 的词频匹配度排序
-- **导出是静态文本**：以文本形式返回，需要手动复制到目标位置
+- **导出是静态文本**：工具以文本形式返回，模型读到的就是它；Zotero 面板可以一键复制或下载文件（`.bib` / `.ris` / `.json` 等），不需要手动誊抄
 - **全文证据依赖 Zotero 索引**：未索引的 PDF 无法提供全文段落
 - **附件深度取决于宿主**：`zotero_attachment` 返回文件位置，继续阅读 PDF 需要宿主具备对应能力
 
@@ -109,7 +112,7 @@ Agent → zotero_export(refs: [1,2,3], format: "bibtex")
 
 - **网络**：只向 `http://127.0.0.1:23119/api` 发起 HTTP 请求（不跟随重定向），`resolveConfig` 强制 loopback 地址
 - **文件**：只读，`zotero_attachment` 用 `existsSync` 校验 Zotero 返回的附件路径，不写文件系统
-- **持久化**：唯一写入来自 Settings → Plugins 中的配置卡片，保存到 `$DSH_HOME/settings.yaml` 的 `zotero:` 用户层
+- **持久化**：唯一写入来自设置页左侧导航的 Zotero 配置页，保存到 `$DSH_HOME/settings.yaml` 的 `zotero:` 用户层
 - **无 Shell / native / 后台任务**：插件不执行 shell 命令、不加载 native 模块、不启动常驻进程
 - **重启**：安装或卸载插件后需要重启 dsh 并新建会话；配置修改保存即热更新，无需重启
 
@@ -120,7 +123,7 @@ Agent → zotero_export(refs: [1,2,3], format: "bibtex")
 | [快速上手](docs/getting-started.md) | 安装、前置条件、首次验证            |
 | [功能概览](docs/features.md)        | 来源面板、对话集成、证据提取、导出  |
 | [工具参考](docs/tools.md)           | 全部 8 个工具的参数、返回值、错误码 |
-| [配置](docs/configuration.md)       | 21 个配置字段、默认值、热更新       |
+| [配置](docs/configuration.md)       | 22 个配置字段、默认值、热更新       |
 | [架构](docs/architecture.md)        | 数据流、各层职责、设计边界          |
 | [开发指南](docs/development.md)     | 构建、测试、本地开发                |
 | [问题排查](docs/troubleshooting.md) | 11 个常见问题的症状和处理           |
@@ -136,7 +139,7 @@ npm run dev                   # tsc --watch，host half 热更新
 npm run dev:client            # esbuild --watch，浏览器部分热更新
 ```
 
-`lib/` 放 Node 侧代码，`lib/client.js` 放浏览器侧代码（settings 卡片和 Zotero tab）。你用 `dev-lib.cordis.yml` overlay 跑完整插件流程，见[开发指南](docs/development.md)。本仓库与 `../deepseek-harness` 并列，属本地暂存布局。
+`lib/` 放 Node 侧代码，`lib/client.js` 放浏览器侧代码（设置页和 Zotero tab）。你用 `dev-lib.cordis.yml` overlay 跑完整插件流程，见[开发指南](docs/development.md)。本仓库与 `../deepseek-harness` 并列，属本地暂存布局。
 
 ## 许可证
 

package/lib/ask.d.ts

@@ -17,11 +17,37 @@ import type { Context } from '@deepseek-ai/cordis';
 import { type ToolRunContext } from '@deepseek-ai/dsh-tools';
 /** The parts of a tool execution `withConnectivityAsk` needs. */
 type ConnectivityAskExec = Pick<ToolRunContext, 'signal' | 'agent'>;
+/**
+ * One recovery conversation per failure kind, owned by one plugin instance.
+ *
+ * Tool calls run concurrently, and a Zotero that is down fails all of them:
+ * five parallel reads used to put five identical cards in front of the user,
+ * each one demanding the same answer. The gate keeps one ask per failure
+ * kind in flight and lets every other caller wait on that same answer, so a
+ * single card decides for all of them — and each caller then retries, or
+ * does not, on its own request. The entry is dropped as soon as the question
+ * settles, so a later failure asks again instead of inheriting a stale
+ * answer.
+ */
+export declare class ConnectivityRecovery {
+    private readonly asking;
+    /**
+     * Answer one failure kind, asking only when no question for it is in
+     * flight.
+     * @param code - the failure kind the question belongs to.
+     * @param question - the ask to run when this caller is the first; resolves
+     *   true when the user chose to retry.
+     * @returns the answer this caller acts on.
+     */
+    ask(code: string, question: () => Promise<boolean>): Promise<boolean>;
+}
 /**
  * Run one Zotero request; on a connectivity failure, ask the user how to
  * proceed and retry at most once when they choose the recommended action.
  * @param ctx - the plugin context; the question service is looked up
  *   optionally, so headless compositions skip the ask.
+ * @param recovery - the instance's recovery gate: callers that failed the
+ *   same way share one question instead of stacking identical cards.
  * @param exec - the tool execution (signal and agent) the failure belongs to.
  * @param run - the request to attempt; must be re-runnable with identical
  *   arguments, because the retry path calls it a second time.
@@ -30,6 +56,6 @@ type ConnectivityAskExec = Pick<ToolRunContext, 'signal' | 'agent'>;
  *   does not choose to retry, the question itself fails, or the retry
  *   fails again.
  */
-export declare function withConnectivityAsk<T>(ctx: Context, exec: ConnectivityAskExec, run: () => Promise<T>): Promise<T>;
+export declare function withConnectivityAsk<T>(ctx: Context, recovery: ConnectivityRecovery, exec: ConnectivityAskExec, run: () => Promise<T>): Promise<T>;
 export {};
 //# sourceMappingURL=ask.d.ts.map

package/lib/ask.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ask.d.ts","sourceRoot":"","sources":["../src/ask.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAA;AAElD,OAAO,EAAgB,KAAK,cAAc,EAAE,MAAM,wBAAwB,CAAA;AAyB1E,iEAAiE;AACjE,KAAK,mBAAmB,GAAG,IAAI,CAAC,cAAc,EAAE,QAAQ,GAAG,OAAO,CAAC,CAAA;AA2EnE;;;;;;;;;;;;GAYG;AACH,wBAAsB,mBAAmB,CAAC,CAAC,EACzC,GAAG,EAAE,OAAO,EACZ,IAAI,EAAE,mBAAmB,EACzB,GAAG,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GACpB,OAAO,CAAC,CAAC,CAAC,CAgCZ"}
+{"version":3,"file":"ask.d.ts","sourceRoot":"","sources":["../src/ask.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAA;AAElD,OAAO,EAAgB,KAAK,cAAc,EAAE,MAAM,wBAAwB,CAAA;AA0B1E,iEAAiE;AACjE,KAAK,mBAAmB,GAAG,IAAI,CAAC,cAAc,EAAE,QAAQ,GAAG,OAAO,CAAC,CAAA;AA2EnE;;;;;;;;;;;GAWG;AACH,qBAAa,oBAAoB;IAC/B,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAsC;IAE7D;;;;;;;OAOG;IACG,GAAG,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAC,OAAO,CAAC,GAAG,OAAO,CAAC,OAAO,CAAC;CAW5E;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,mBAAmB,CAAC,CAAC,EACzC,GAAG,EAAE,OAAO,EACZ,QAAQ,EAAE,oBAAoB,EAC9B,IAAI,EAAE,mBAAmB,EACzB,GAAG,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GACpB,OAAO,CAAC,CAAC,CAAC,CAmCZ"}

package/lib/ask.js

@@ -15,6 +15,7 @@
  */
 import { HarnessError } from '@deepseek-ai/dsh-llm';
 import { TOOL_ABORTED } from '@deepseek-ai/dsh-tools';
+import { ZOTERO_LOCAL_API_VERSION } from './constants.js';
 import { ZOTERO_API_DISABLED, ZOTERO_API_VERSION, ZOTERO_NOT_RUNNING, ZOTERO_TIMEOUT, ZoteroError, } from './errors.js';
 /** The connectivity failure codes that warrant asking the user how to proceed. */
 const ASK_WORTHY_CODES = [
@@ -46,10 +47,10 @@ const FAILURE_SPECS = {
         abortDescription: ABORT_DESCRIPTION,
     },
     [ZOTERO_API_VERSION]: {
-        header: 'Zotero version too old',
-        question: 'The running Zotero does not speak local API version 3, which this plugin requires.',
-        detail: 'Upgrade Zotero to a version whose local API supports version 3.',
-        retryLabel: 'I upgraded Zotero, retry (Recommended)',
+        header: 'Zotero and this plugin share no API version',
+        question: `The running Zotero does not implement local API version ${ZOTERO_LOCAL_API_VERSION}, which this plugin requires.`,
+        detail: `Upgrade Zotero to a version whose local API supports version ${ZOTERO_LOCAL_API_VERSION} — or update dsh-zotero if the running Zotero is newer than this plugin line.`,
+        retryLabel: 'I fixed the version, retry (Recommended)',
         retryDescription: RETRY_DESCRIPTION,
         abortLabel: ABORT_LABEL,
         abortDescription: ABORT_DESCRIPTION,
@@ -79,11 +80,49 @@ function questionOf(spec) {
         ],
     };
 }
+/**
+ * One recovery conversation per failure kind, owned by one plugin instance.
+ *
+ * Tool calls run concurrently, and a Zotero that is down fails all of them:
+ * five parallel reads used to put five identical cards in front of the user,
+ * each one demanding the same answer. The gate keeps one ask per failure
+ * kind in flight and lets every other caller wait on that same answer, so a
+ * single card decides for all of them — and each caller then retries, or
+ * does not, on its own request. The entry is dropped as soon as the question
+ * settles, so a later failure asks again instead of inheriting a stale
+ * answer.
+ */
+export class ConnectivityRecovery {
+    asking = new Map();
+    /**
+     * Answer one failure kind, asking only when no question for it is in
+     * flight.
+     * @param code - the failure kind the question belongs to.
+     * @param question - the ask to run when this caller is the first; resolves
+     *   true when the user chose to retry.
+     * @returns the answer this caller acts on.
+     */
+    async ask(code, question) {
+        const inFlight = this.asking.get(code);
+        if (inFlight !== undefined)
+            return await inFlight;
+        const answer = question();
+        this.asking.set(code, answer);
+        try {
+            return await answer;
+        }
+        finally {
+            this.asking.delete(code);
+        }
+    }
+}
 /**
  * Run one Zotero request; on a connectivity failure, ask the user how to
  * proceed and retry at most once when they choose the recommended action.
  * @param ctx - the plugin context; the question service is looked up
  *   optionally, so headless compositions skip the ask.
+ * @param recovery - the instance's recovery gate: callers that failed the
+ *   same way share one question instead of stacking identical cards.
  * @param exec - the tool execution (signal and agent) the failure belongs to.
  * @param run - the request to attempt; must be re-runnable with identical
  *   arguments, because the retry path calls it a second time.
@@ -92,7 +131,7 @@ function questionOf(spec) {
  *   does not choose to retry, the question itself fails, or the retry
  *   fails again.
  */
-export async function withConnectivityAsk(ctx, exec, run) {
+export async function withConnectivityAsk(ctx, recovery, exec, run) {
     try {
         return await run();
     }
@@ -103,14 +142,23 @@ export async function withConnectivityAsk(ctx, exec, run) {
         if (questions === undefined)
             throw error;
         const spec = FAILURE_SPECS[error.code];
-        let answer;
+        let retry;
         try {
-            const request = {
-                questions: [questionOf(spec)],
-                ...(exec.agent !== undefined ? { agent: exec.agent } : {}),
-                signal: exec.signal,
-            };
-            answer = await questions.ask(request);
+            retry = await recovery.ask(error.code, async () => {
+                const request = {
+                    questions: [questionOf(spec)],
+                    ...(exec.agent !== undefined ? { agent: exec.agent } : {}),
+                    signal: exec.signal,
+                };
+                const answer = await questions.ask(request);
+                const answerItem = answer.answers.find((item) => item.id === 'zotero-failure');
+                // Matched by label string because the answer protocol carries only
+                // selected labels (no stable option ids); the labels are code
+                // constants (FAILURE_SPECS), never i18n copy, so a rename breaks the
+                // build's contract visibly in one place. Revisit if the protocol
+                // gains option ids.
+                return (answerItem?.selected ?? []).includes(spec.retryLabel);
+            });
         }
         catch {
             // A failed question (no provider, aborted ask, delegated caller) must
@@ -119,13 +167,7 @@ export async function withConnectivityAsk(ctx, exec, run) {
                 throw new HarnessError('tool call aborted', TOOL_ABORTED);
             throw error;
         }
-        const answerItem = answer.answers.find((item) => item.id === 'zotero-failure');
-        const selected = answerItem?.selected ?? [];
-        // Matched by label string because the answer protocol carries only
-        // selected labels (no stable option ids); the labels are code constants
-        // (FAILURE_SPECS), never i18n copy, so a rename breaks the build's
-        // contract visibly in one place. Revisit if the protocol gains option ids.
-        if (!selected.includes(spec.retryLabel))
+        if (!retry)
             throw error;
         // Outside the catch: a second failure propagates as-is, never re-asking.
         return await run();

package/lib/ask.js.map

@@ -1 +1 @@
-{"version":3,"file":"ask.js","sourceRoot":"","sources":["../src/ask.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAGH,OAAO,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,EAAE,YAAY,EAAuB,MAAM,wBAAwB,CAAA;AAO1E,OAAO,EACL,mBAAmB,EACnB,kBAAkB,EAClB,kBAAkB,EAClB,cAAc,EACd,WAAW,GACZ,MAAM,aAAa,CAAA;AAEpB,kFAAkF;AAClF,MAAM,gBAAgB,GAAG;IACvB,kBAAkB;IAClB,mBAAmB;IACnB,kBAAkB;IAClB,cAAc;CACN,CAAA;AAkBV,MAAM,WAAW,GAAG,kBAAkB,CAAA;AACtC,MAAM,iBAAiB,GAAG,kEAAkE,CAAA;AAC5F,MAAM,iBAAiB,GAAG,gDAAgD,CAAA;AAE1E,MAAM,aAAa,GAAuC;IACxD,CAAC,kBAAkB,CAAC,EAAE;QACpB,MAAM,EAAE,uBAAuB;QAC/B,QAAQ,EAAE,yEAAyE;QACnF,MAAM,EACJ,yHAAyH;QAC3H,UAAU,EAAE,uCAAuC;QACnD,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,mBAAmB,CAAC,EAAE;QACrB,MAAM,EAAE,8BAA8B;QACtC,QAAQ,EAAE,6DAA6D;QACvE,MAAM,EACJ,8GAA8G;QAChH,UAAU,EAAE,8CAA8C;QAC1D,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,kBAAkB,CAAC,EAAE;QACpB,MAAM,EAAE,wBAAwB;QAChC,QAAQ,EAAE,oFAAoF;QAC9F,MAAM,EAAE,iEAAiE;QACzE,UAAU,EAAE,wCAAwC;QACpD,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,cAAc,CAAC,EAAE;QAChB,MAAM,EAAE,kBAAkB;QAC1B,QAAQ,EAAE,iFAAiF;QAC3F,MAAM,EAAE,kDAAkD;QAC1D,UAAU,EAAE,qBAAqB;QACjC,gBAAgB,EAAE,6BAA6B;QAC/C,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;CACF,CAAA;AAED,SAAS,eAAe,CAAC,IAAY;IACnC,OAAQ,gBAAsC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAA;AAC/D,CAAC;AAED,SAAS,UAAU,CAAC,IAAiB;IACnC,OAAO;QACL,EAAE,EAAE,gBAAgB;QACpB,QAAQ,EAAE,IAAI,CAAC,QAAQ;QACvB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,OAAO,EAAE;YACP,EAAE,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,WAAW,EAAE,IAAI,CAAC,gBAAgB,EAAE;YAC9D,EAAE,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,WAAW,EAAE,IAAI,CAAC,gBAAgB,EAAE;SAC/D;KACF,CAAA;AACH,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CACvC,GAAY,EACZ,IAAyB,EACzB,GAAqB;IAErB,IAAI,CAAC;QACH,OAAO,MAAM,GAAG,EAAE,CAAA;IACpB,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAI,CAAC,CAAC,KAAK,YAAY,WAAW,CAAC,IAAI,CAAC,eAAe,CAAC,KAAK,CAAC,IAAI,CAAC;YAAE,MAAM,KAAK,CAAA;QAChF,MAAM,SAAS,GAAG,GAAG,CAAC,GAAG,CAAC,eAAe,CAAoC,CAAA;QAC7E,IAAI,SAAS,KAAK,SAAS;YAAE,MAAM,KAAK,CAAA;QACxC,MAAM,IAAI,GAAG,aAAa,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;QACtC,IAAI,MAA6B,CAAA;QACjC,IAAI,CAAC;YACH,MAAM,OAAO,GAA2B;gBACtC,SAAS,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC;gBAC7B,GAAG,CAAC,IAAI,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBAC1D,MAAM,EAAE,IAAI,CAAC,MAAM;aACpB,CAAA;YACD,MAAM,GAAG,MAAM,SAAS,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;QACvC,CAAC;QAAC,MAAM,CAAC;YACP,sEAAsE;YACtE,kDAAkD;YAClD,IAAI,IAAI,CAAC,MAAM,EAAE,OAAO;gBAAE,MAAM,IAAI,YAAY,CAAC,mBAAmB,EAAE,YAAY,CAAC,CAAA;YACnF,MAAM,KAAK,CAAA;QACb,CAAC;QACD,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,EAAE,KAAK,gBAAgB,CAAC,CAAA;QAC9E,MAAM,QAAQ,GAAG,UAAU,EAAE,QAAQ,IAAI,EAAE,CAAA;QAC3C,mEAAmE;QACnE,wEAAwE;QACxE,mEAAmE;QACnE,2EAA2E;QAC3E,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,UAAU,CAAC;YAAE,MAAM,KAAK,CAAA;QACpD,yEAAyE;QACzE,OAAO,MAAM,GAAG,EAAE,CAAA;IACpB,CAAC;AACH,CAAC"}
+{"version":3,"file":"ask.js","sourceRoot":"","sources":["../src/ask.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAGH,OAAO,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,EAAE,YAAY,EAAuB,MAAM,wBAAwB,CAAA;AAO1E,OAAO,EAAE,wBAAwB,EAAE,MAAM,gBAAgB,CAAA;AACzD,OAAO,EACL,mBAAmB,EACnB,kBAAkB,EAClB,kBAAkB,EAClB,cAAc,EACd,WAAW,GACZ,MAAM,aAAa,CAAA;AAEpB,kFAAkF;AAClF,MAAM,gBAAgB,GAAG;IACvB,kBAAkB;IAClB,mBAAmB;IACnB,kBAAkB;IAClB,cAAc;CACN,CAAA;AAkBV,MAAM,WAAW,GAAG,kBAAkB,CAAA;AACtC,MAAM,iBAAiB,GAAG,kEAAkE,CAAA;AAC5F,MAAM,iBAAiB,GAAG,gDAAgD,CAAA;AAE1E,MAAM,aAAa,GAAuC;IACxD,CAAC,kBAAkB,CAAC,EAAE;QACpB,MAAM,EAAE,uBAAuB;QAC/B,QAAQ,EAAE,yEAAyE;QACnF,MAAM,EACJ,yHAAyH;QAC3H,UAAU,EAAE,uCAAuC;QACnD,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,mBAAmB,CAAC,EAAE;QACrB,MAAM,EAAE,8BAA8B;QACtC,QAAQ,EAAE,6DAA6D;QACvE,MAAM,EACJ,8GAA8G;QAChH,UAAU,EAAE,8CAA8C;QAC1D,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,kBAAkB,CAAC,EAAE;QACpB,MAAM,EAAE,6CAA6C;QACrD,QAAQ,EAAE,2DAA2D,wBAAwB,+BAA+B;QAC5H,MAAM,EAAE,gEAAgE,wBAAwB,+EAA+E;QAC/K,UAAU,EAAE,0CAA0C;QACtD,gBAAgB,EAAE,iBAAiB;QACnC,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;IACD,CAAC,cAAc,CAAC,EAAE;QAChB,MAAM,EAAE,kBAAkB;QAC1B,QAAQ,EAAE,iFAAiF;QAC3F,MAAM,EAAE,kDAAkD;QAC1D,UAAU,EAAE,qBAAqB;QACjC,gBAAgB,EAAE,6BAA6B;QAC/C,UAAU,EAAE,WAAW;QACvB,gBAAgB,EAAE,iBAAiB;KACpC;CACF,CAAA;AAED,SAAS,eAAe,CAAC,IAAY;IACnC,OAAQ,gBAAsC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAA;AAC/D,CAAC;AAED,SAAS,UAAU,CAAC,IAAiB;IACnC,OAAO;QACL,EAAE,EAAE,gBAAgB;QACpB,QAAQ,EAAE,IAAI,CAAC,QAAQ;QACvB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,OAAO,EAAE;YACP,EAAE,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,WAAW,EAAE,IAAI,CAAC,gBAAgB,EAAE;YAC9D,EAAE,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,WAAW,EAAE,IAAI,CAAC,gBAAgB,EAAE;SAC/D;KACF,CAAA;AACH,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,OAAO,oBAAoB;IACd,MAAM,GAAG,IAAI,GAAG,EAA4B,CAAA;IAE7D;;;;;;;OAOG;IACH,KAAK,CAAC,GAAG,CAAC,IAAY,EAAE,QAAgC;QACtD,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QACtC,IAAI,QAAQ,KAAK,SAAS;YAAE,OAAO,MAAM,QAAQ,CAAA;QACjD,MAAM,MAAM,GAAG,QAAQ,EAAE,CAAA;QACzB,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QAC7B,IAAI,CAAC;YACH,OAAO,MAAM,MAAM,CAAA;QACrB,CAAC;gBAAS,CAAC;YACT,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAA;QAC1B,CAAC;IACH,CAAC;CACF;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CACvC,GAAY,EACZ,QAA8B,EAC9B,IAAyB,EACzB,GAAqB;IAErB,IAAI,CAAC;QACH,OAAO,MAAM,GAAG,EAAE,CAAA;IACpB,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAI,CAAC,CAAC,KAAK,YAAY,WAAW,CAAC,IAAI,CAAC,eAAe,CAAC,KAAK,CAAC,IAAI,CAAC;YAAE,MAAM,KAAK,CAAA;QAChF,MAAM,SAAS,GAAG,GAAG,CAAC,GAAG,CAAC,eAAe,CAAoC,CAAA;QAC7E,IAAI,SAAS,KAAK,SAAS;YAAE,MAAM,KAAK,CAAA;QACxC,MAAM,IAAI,GAAG,aAAa,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;QACtC,IAAI,KAAc,CAAA;QAClB,IAAI,CAAC;YACH,KAAK,GAAG,MAAM,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,IAAI,EAAE;gBAChD,MAAM,OAAO,GAA2B;oBACtC,SAAS,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC;oBAC7B,GAAG,CAAC,IAAI,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;oBAC1D,MAAM,EAAE,IAAI,CAAC,MAAM;iBACpB,CAAA;gBACD,MAAM,MAAM,GAA0B,MAAM,SAAS,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;gBAClE,MAAM,UAAU,GAAG,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,EAAE,KAAK,gBAAgB,CAAC,CAAA;gBAC9E,mEAAmE;gBACnE,8DAA8D;gBAC9D,qEAAqE;gBACrE,iEAAiE;gBACjE,oBAAoB;gBACpB,OAAO,CAAC,UAAU,EAAE,QAAQ,IAAI,EAAE,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;YAC/D,CAAC,CAAC,CAAA;QACJ,CAAC;QAAC,MAAM,CAAC;YACP,sEAAsE;YACtE,kDAAkD;YAClD,IAAI,IAAI,CAAC,MAAM,EAAE,OAAO;gBAAE,MAAM,IAAI,YAAY,CAAC,mBAAmB,EAAE,YAAY,CAAC,CAAA;YACnF,MAAM,KAAK,CAAA;QACb,CAAC;QACD,IAAI,CAAC,KAAK;YAAE,MAAM,KAAK,CAAA;QACvB,yEAAyE;QACzE,OAAO,MAAM,GAAG,EAAE,CAAA;IACpB,CAAC;AACH,CAAC"}