@bojanrajkovic/mcp-paprika 1.2.0-beta.1 → 1.2.0-beta.2

package/dist/paprika/client.js

@@ -22,6 +22,20 @@ class TransientHTTPError extends Error {
         this.name = "TransientHTTPError";
     }
 }
+// Private marker class used to route network-level fetch failures (DNS,
+// TCP reset, TLS handshake, etc.) through cockatiel's handleType-based
+// retry policy. undici throws a bare TypeError for these; the runtime
+// has no dedicated subclass we can match on directly. The cause is the
+// original TypeError so callers can unwrap and surface the real error
+// once retries are exhausted.
+class NetworkRetryableError extends Error {
+    cause;
+    constructor(cause) {
+        super(`Network error: ${cause.message}`, { cause });
+        this.cause = cause;
+        this.name = "NetworkRetryableError";
+    }
+}
 class TokenExpiredError extends Error {
     constructor() {
         super("Token expired");
@@ -29,14 +43,14 @@ class TokenExpiredError extends Error {
     }
 }
 const RETRYABLE_STATUSES = new Set([429, 500, 502, 503]);
-const retryPolicy = retry(handleType(TransientHTTPError), {
+const retryPolicy = retry(handleType(TransientHTTPError).orType(NetworkRetryableError), {
     maxAttempts: 3,
     backoff: new ExponentialBackoff({
         initialDelay: 500,
         maxDelay: 10_000,
     }),
 });
-const breakerPolicy = circuitBreaker(handleType(TransientHTTPError), {
+const breakerPolicy = circuitBreaker(handleType(TransientHTTPError).orType(NetworkRetryableError), {
     halfOpenAfter: 30_000,
     breaker: new ConsecutiveBreaker(5),
 });
@@ -178,7 +192,20 @@ export class PaprikaClient {
             if (body !== undefined) {
                 fetchInit.body = body;
             }
-            const response = await fetch(url, fetchInit);
+            let response;
+            try {
+                response = await fetch(url, fetchInit);
+            }
+            catch (error) {
+                // undici throws a bare TypeError for network-level failures (DNS,
+                // TCP reset, TLS handshake, abort). Re-throw as a retryable marker
+                // so cockatiel's handleType policy applies the same backoff +
+                // circuit-breaker treatment as 5xx HTTP responses.
+                if (error instanceof TypeError) {
+                    throw new NetworkRetryableError(error);
+                }
+                throw error;
+            }
             if (!response.ok) {
                 if (RETRYABLE_STATUSES.has(response.status)) {
                     throw new TransientHTTPError(response.status);
@@ -199,6 +226,11 @@ export class PaprikaClient {
             if (error instanceof BrokenCircuitError) {
                 throw new PaprikaAPIError("Service unavailable (circuit open)", 503, url);
             }
+            // Unwrap the retry marker so callers see the original undici TypeError —
+            // tools that surface .message stay consistent with the pre-retry shape.
+            if (error instanceof NetworkRetryableError) {
+                throw error.cause;
+            }
             if (error instanceof TokenExpiredError) {
                 if (!this.token) {
                     throw new PaprikaAuthError("Authentication required (HTTP 401)");
@@ -214,6 +246,9 @@ export class PaprikaClient {
                     if (retryError instanceof BrokenCircuitError) {
                         throw new PaprikaAPIError("Service unavailable (circuit open)", 503, url);
                     }
+                    if (retryError instanceof NetworkRetryableError) {
+                        throw retryError.cause;
+                    }
                     throw retryError;
                 }
             }

package/dist/tools/create.js

@@ -1,7 +1,8 @@
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { RecipeUidSchema } from "../paprika/types.js";
 import { coldStartGuard, commitRecipe, recipeToMarkdown, resolveCategoryNames, textResult } from "./helpers.js";
+const log = createLogger("mcp-paprika:create_recipe");
 export function registerCreateTool(server, ctx) {
     server.registerTool("create_recipe", {
         description: "Create a new recipe in the Paprika account.",
@@ -69,7 +70,9 @@ export function registerCreateTool(server, ctx) {
             }
             catch (error) {
                 // AC2.8: store/cache not updated — commitRecipe not reached
-                return textResult(`Failed to create recipe: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`saveRecipe failed for name=${args.name}: ${message}`);
+                return textResult(`Failed to create recipe: ${message}`);
             }
             const categoryNames = ctx.store.resolveCategories(saved.categories);
             const markdown = recipeToMarkdown(saved, categoryNames);

package/dist/tools/delete.js

@@ -1,7 +1,8 @@
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { RecipeUidSchema } from "../paprika/types.js";
 import { coldStartGuard, commitRecipe, textResult } from "./helpers.js";
+const log = createLogger("mcp-paprika:delete_recipe");
 export function registerDeleteTool(server, ctx) {
     server.registerTool("delete_recipe", {
         description: "Soft-delete a recipe by UID, moving it to the Paprika trash. " +
@@ -26,7 +27,9 @@ export function registerDeleteTool(server, ctx) {
                 await commitRecipe(ctx, saved);
             }
             catch (error) {
-                return textResult(`Failed to delete recipe: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`saveRecipe (soft-delete) failed for uid=${trashed.uid}: ${message}`);
+                return textResult(`Failed to delete recipe: ${message}`);
             }
             return textResult(`Recipe "${recipe.name}" has been moved to the trash.`);
         }, (guard) => guard);

package/dist/tools/pantry-add.js

@@ -1,10 +1,11 @@
 // pattern: Imperative Shell
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { PantryItemUidSchema } from "../paprika/types.js";
 import { normalizePaprikaDate, paprikaDateToday } from "../paprika/dates.js";
 import { textResult } from "./helpers.js";
 import { commitPantryItem, pantryItemToMarkdown, pantryStartGuard } from "./pantry-helpers.js";
+const log = createLogger("mcp-paprika:add_pantry_item");
 export function registerAddPantryItemTool(server, ctx) {
     server.registerTool("add_pantry_item", {
         description: "Add a new item to the pantry. Rejects duplicates by case-insensitive ingredient name; " +
@@ -64,7 +65,9 @@ export function registerAddPantryItemTool(server, ctx) {
             }
             catch (error) {
                 // AC4.7: store/cache not updated — commitPantryItem not reached
-                return textResult(`Failed to add pantry item: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`savePantryItem failed for ${args.ingredient}: ${message}`);
+                return textResult(`Failed to add pantry item: ${message}`);
             }
             return textResult(pantryItemToMarkdown(saved));
         }, (guard) => guard);

package/dist/tools/pantry-delete.js

@@ -1,8 +1,9 @@
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { PantryItemUidSchema } from "../paprika/types.js";
 import { textResult } from "./helpers.js";
 import { commitPantryItem, pantryStartGuard } from "./pantry-helpers.js";
+const log = createLogger("mcp-paprika:delete_pantry_item");
 export function registerDeletePantryItemTool(server, ctx) {
     server.registerTool("delete_pantry_item", {
         description: "Soft-delete a pantry item by UID. Idempotent: a second delete on the same UID " +
@@ -37,7 +38,9 @@ export function registerDeletePantryItemTool(server, ctx) {
                 await commitPantryItem(ctx, saved);
             }
             catch (error) {
-                return textResult(`Failed to delete pantry item: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`savePantryItem (soft-delete) failed for uid=${trashed.uid}: ${message}`);
+                return textResult(`Failed to delete pantry item: ${message}`);
             }
             return textResult(`Pantry item "${existing.ingredient}" has been deleted.`);
         }, (guard) => guard);

package/dist/tools/pantry-update.js

@@ -1,9 +1,10 @@
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { PantryItemUidSchema } from "../paprika/types.js";
 import { normalizePaprikaDate } from "../paprika/dates.js";
 import { textResult } from "./helpers.js";
 import { commitPantryItem, pantryItemToMarkdown, pantryStartGuard } from "./pantry-helpers.js";
+const log = createLogger("mcp-paprika:update_pantry_item");
 export function registerUpdatePantryItemTool(server, ctx) {
     server.registerTool("update_pantry_item", {
         description: "Update an existing pantry item by UID. Only provided fields are changed; " +
@@ -64,7 +65,9 @@ export function registerUpdatePantryItemTool(server, ctx) {
                 await commitPantryItem(ctx, saved);
             }
             catch (error) {
-                return textResult(`Failed to update pantry item: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`savePantryItem failed for uid=${updated.uid}: ${message}`);
+                return textResult(`Failed to update pantry item: ${message}`);
             }
             return textResult(pantryItemToMarkdown(saved));
         }, (guard) => guard);

package/dist/tools/update.js

@@ -1,7 +1,8 @@
-import { toMessage } from "../utils/log.js";
+import { createLogger, toMessage } from "../utils/log.js";
 import { z } from "zod";
 import { RecipeUidSchema } from "../paprika/types.js";
 import { coldStartGuard, commitRecipe, recipeToMarkdown, resolveCategoryNames, textResult } from "./helpers.js";
+const log = createLogger("mcp-paprika:update_recipe");
 export function registerUpdateTool(server, ctx) {
     server.registerTool("update_recipe", {
         description: "Update an existing recipe by UID. Only provided fields are changed; " +
@@ -67,7 +68,9 @@ export function registerUpdateTool(server, ctx) {
                 await commitRecipe(ctx, saved); // AC3.4
             }
             catch (error) {
-                return textResult(`Failed to update recipe: ${toMessage(error)}`);
+                const message = toMessage(error);
+                log(`saveRecipe failed for uid=${updated.uid}: ${message}`);
+                return textResult(`Failed to update recipe: ${message}`);
             }
             const categoryNames = ctx.store.resolveCategories(saved.categories);
             const markdown = recipeToMarkdown(saved, categoryNames);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bojanrajkovic/mcp-paprika",
-  "version": "1.2.0-beta.1",
+  "version": "1.2.0-beta.2",
   "description": "MCP server for Paprika recipe manager",
   "license": "MIT",
   "repository": {