dsh-plugin-library 0.1.2 → 0.1.3

package/dist/host.js

@@ -1415,6 +1415,50 @@ var LibraryManager = class {
     if (asset.status === "removed") throw new FeatureError2("library/not-found", "\u8D44\u6599\u5DF2\u79FB\u9664\u3002");
     return asset;
   }
+  /**
+   * Resolve whatever identifier a caller supplied into a live asset id.
+   *
+   * The model-facing tools address a document by `asset_id`, but the text a
+   * model actually reads (the search results) leads with the *name*. A model
+   * that passes the name used to get `library/not-found` — "资料不存在或已移除"
+   * — for a document that was perfectly readable, which reads as data loss.
+   * So a name is accepted too, with two deliberate refusals:
+   *
+   * - **Ambiguous** (two live documents share a name): refused with every
+   *   candidate's id listed. Silently picking one would read the wrong document.
+   * - **Unknown**: refused with a message that names both possibilities and
+   *   points at `library_search`, instead of the bare "not found" that hints
+   *   nothing about what was wrong with the identifier.
+   * @param {string} identifier - an asset id, or a document name.
+   * @returns {string} the live asset id.
+   * @throws {FeatureError} `library/invalid-request`, `library/ambiguous` or `library/not-found`.
+   */
+  resolveAssetId(identifier) {
+    const text = typeof identifier === "string" ? identifier.trim() : "";
+    if (text.length === 0) {
+      throw new FeatureError2("library/invalid-request", "\u9700\u8981 asset_id\uFF1A\u7528 library_search \u641C\u5230\u8D44\u6599\u540E\uFF0C\u628A\u5B83\u8FD4\u56DE\u7684 asset_id \u539F\u6837\u4F20\u8FDB\u6765\u3002");
+    }
+    const state = this.load();
+    const direct = state.assets[text];
+    const directNode = direct === void 0 ? void 0 : state.nodes[text];
+    if (direct !== void 0 && direct.status !== "removed" && directNode !== void 0 && directNode.removedAt === void 0) {
+      return text;
+    }
+    const live = Object.values(state.assets).filter((asset) => asset.status !== "removed").map((asset) => ({ asset, node: state.nodes[asset.id] })).filter((row) => row.node !== void 0 && row.node.removedAt === void 0 && row.node.kind === "asset");
+    const exact = live.filter((row) => row.node.name === text);
+    const matches = exact.length > 0 ? exact : live.filter((row) => row.node.name.toLocaleLowerCase() === text.toLocaleLowerCase());
+    if (matches.length === 1) return matches[0].asset.id;
+    if (matches.length > 1) {
+      throw new FeatureError2(
+        "library/ambiguous",
+        `\u6709 ${matches.length} \u4EFD\u8D44\u6599\u53EB\u300C${text}\u300D\uFF0C\u65E0\u6CD5\u786E\u5B9A\u8BFB\u54EA\u4E00\u4EFD\uFF1B\u8BF7\u628A asset_id \u6362\u6210\u5176\u4E2D\u4E00\u4E2A\uFF1A` + matches.map((row) => `${row.node.name}\uFF08${row.asset.id}\uFF09`).join("\u3001")
+      );
+    }
+    throw new FeatureError2(
+      "library/not-found",
+      `\u6CA1\u627E\u5230\u8D44\u6599\u300C${text}\u300D\uFF1A\u5B83\u65E2\u4E0D\u662F asset_id\uFF0C\u4E5F\u4E0D\u662F\u73B0\u6709\u8D44\u6599\u7684\u540D\u79F0\u3002\u8D44\u6599\u5728\u8D44\u6599\u5E93\u91CC\u3001\u4E0D\u5728\u5DE5\u4F5C\u533A\u6587\u4EF6\u7CFB\u7EDF\u91CC\uFF0C\u5148\u7528 library_search \u641C\u7D22\uFF0C\u518D\u628A\u8FD4\u56DE\u7684 asset_id \u539F\u6837\u4F20\u7ED9\u672C\u5DE5\u5177\u3002`
+    );
+  }
   /** An asset record a read is allowed to resolve: present and enabled. */
   requireReadableAsset(state, assetId) {
     const asset = this.requireAsset(state, assetId);
@@ -1586,7 +1630,13 @@ async function registerLibraryTools(ctx, manager) {
       },
       render: (_args, value) => [{
         type: "text",
+        // The id is printed here, not just carried in the structured payload:
+        // this rendered text is what the model actually reads, and the read /
+        // edit tools address a document by `asset_id`. Leaving it out (v0.1.x)
+        // made models pass the *name* instead and get "资料不存在或已移除" for a
+        // document that was perfectly readable.
         text: value.hits.length > 0 ? value.hits.map((hit) => `${hit.name} (${hit.folder_path} \xB7 ${hit.kind} \xB7 \u4FEE\u8BA2 ${hit.revision_number} \xB7 ${hit.conversion_status})
+asset_id: ${hit.asset_id}
 ${hit.excerpt}`).join("\n\n") : "\u8D44\u6599\u5E93\u91CC\u6CA1\u6709\u5339\u914D\u7684\u8D44\u6599\u3002"
       }]
     },
@@ -1626,15 +1676,16 @@ ${hit.excerpt}`).join("\n\n") : "\u8D44\u6599\u5E93\u91CC\u6CA1\u6709\u5339\u914
       if (!Number.isInteger(offset) || offset < 0 || !Number.isInteger(limit) || limit < 1 || limit > 2e4) {
         throw new Error("library/invalid-page");
       }
-      const detail = manager.assetDetail(args.asset_id);
+      const assetId = manager.resolveAssetId(args.asset_id);
+      const detail = manager.assetDetail(assetId);
       const revisionId = args.revision_id ?? detail.asset.currentRevisionId;
       const revision = detail.revisions.find((row) => row.id === revisionId);
-      const complete = await manager.readText(args.asset_id, revisionId);
+      const complete = await manager.readText(assetId, revisionId);
       const content = complete.slice(offset, offset + limit);
       const next = offset + content.length;
       const truncated = next < complete.length;
       return {
-        asset_id: args.asset_id,
+        asset_id: assetId,
         revision_id: revisionId,
         conversion_status: revision?.conversionStatus ?? "ready",
         content,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-library",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "本地文件资料库：内容寻址的不可变修订、派生的可检索正文、文件夹与模型工具。",
   "type": "module",
   "main": "dist/host.js",