@xevos117/mcp-zotero 1.0.6 → 1.0.7

package/README.md

@@ -17,12 +17,18 @@ For advanced use cases (PDF upload policy, citation style guidance, source trans
 
 | Scenario | MCP server | Skill needed? |
 |---|---|---|
-| Local LLM (Claude Code, LM Studio, etc.) | All 15 tools | No |
-| Remote/sandboxed LLM (Claude.ai Projects) | API tools (search, add, metadata) | Yes, for citation injection |
+| LLM with filesystem access (Claude Code, LM Studio, etc.) | All 15 tools | No |
+| LLM without filesystem access (Claude.ai Projects, Claude Desktop) | API tools (search, add, metadata) | Yes, for citation injection |
 
-Local LLMs with filesystem access can use all tools directly, including `inject_citations` which reads and writes `.docx` files on disk.
+LLMs with filesystem access can use all tools directly, including `inject_citations` which reads and writes `.docx` files on disk.
 
-Remote LLMs without filesystem access can use the included **Claude skill** (`skills/zotero-skill-mcp-integrations/`), which runs citation injection entirely inside the sandbox. MCP tools handle all Zotero API operations; the skill handles document assembly.
+LLMs without filesystem access — including Claude Desktop, which connects to MCP but cannot generate files locally — can use the included **Claude skill** (`skills/zotero-skill-mcp-integrations/`), which runs citation injection entirely inside a sandbox. MCP tools handle all Zotero API operations; the skill handles document assembly.
+
+### Claude Skill Setup (for Claude.ai Projects and Claude Desktop)
+
+1. Download the skill `.zip` from the latest [GitHub Release](https://github.com/Xevos117/mcp-zotero/releases)
+2. Extract it and upload the folder to your Claude.ai Project as a skill
+3. The skill enables citation injection directly inside the sandbox, without requiring local filesystem access
 
 ## Setup
 
@@ -159,7 +165,7 @@ claude mcp add-json "zotero" '{"command":"npx","args":["tsx","src/server.ts"],"e
 ```bash
 npm install
 npm run build          # Compile TypeScript
-npm test               # Run tests (vitest, 377 tests)
+npm test               # Run tests (vitest, 382 tests)
 npx tsx src/server.ts  # Run directly without building
 ```
 

package/build/tools/add-items-by-doi.js

@@ -105,12 +105,31 @@ export async function handleAddItemsByDoi(zoteroApi, userId, args) {
             .library("user", userId)
             .items()
             .post(zoteroItems);
+        if (!response.isSuccess()) {
+            const errors = response.getErrors();
+            const errorMessages = Object.entries(errors)
+                .map(([idx, msg]) => `Item ${idx}: ${typeof msg === "object" ? JSON.stringify(msg) : msg}`)
+                .join(", ");
+            return formatErrorResponse(`Zotero API write failed: ${errorMessages}`);
+        }
         const createdItems = response.getData();
-        const success = resolved.success.map((r, i) => ({
-            doi: r.doi,
-            item_key: createdItems[i]?.key ?? "unknown",
-            title: createdItems[i]?.title ?? r.data.title ?? "Untitled",
-        }));
+        if (!createdItems || !Array.isArray(createdItems) || createdItems.length === 0) {
+            return formatErrorResponse("Zotero API returned empty response - items may not have been created");
+        }
+        const success = [];
+        for (let i = 0; i < resolved.success.length; i++) {
+            const r = resolved.success[i];
+            const entity = response.getEntityByIndex(i);
+            const itemKey = entity?.key;
+            if (!itemKey) {
+                return formatErrorResponse(`Failed to get item key for DOI ${r.doi} at index ${i}. API response may be invalid.`);
+            }
+            success.push({
+                doi: r.doi,
+                item_key: itemKey,
+                title: entity?.title ?? r.data.title ?? "Untitled",
+            });
+        }
         let pdf_results;
         if (auto_attach_pdf) {
             const apiKey = process.env.ZOTERO_API_KEY;

package/build/utils/csl-to-zotero.js

@@ -101,8 +101,8 @@ export function cslToZoteroItem(csl, options) {
         place: csl["publisher-place"] ?? "",
         url: csl.URL ?? "",
         abstractNote: csl.abstract ?? "",
-        ISBN: csl.ISBN ?? "",
-        ISSN: csl.ISSN ?? "",
+        ISBN: Array.isArray(csl.ISBN) ? csl.ISBN[0] : csl.ISBN ?? "",
+        ISSN: Array.isArray(csl.ISSN) ? csl.ISSN[0] : csl.ISSN ?? "",
         edition: csl.edition ?? "",
         numPages: csl["number-of-pages"] ?? "",
         series: csl["collection-title"] ?? "",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@xevos117/mcp-zotero",
   "type": "module",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "Unofficial MCP server for Zotero integration with citation management",
   "repository": {
     "type": "git",