@open-loyalty/mcp-server 1.3.3 → 1.3.4

package/dist/tools/points.js

@@ -80,9 +80,14 @@ export async function pointsSpend(input) {
         return { transferId: response.transferId };
     }
     catch (error) {
-        // Check for insufficient points error
+        // Check for insufficient points error - API returns various formats:
+        // "NotEnoughPoints", "insufficient", "not enough points" (lowercase)
+        // Must check both error.message AND axios response data
         const errorMessage = error instanceof Error ? error.message : String(error);
-        if (errorMessage.includes("NotEnoughPoints") || errorMessage.includes("insufficient")) {
+        const axiosError = error;
+        const responseMessage = axiosError.response?.data?.message || "";
+        const combinedMessage = `${errorMessage} ${responseMessage}`.toLowerCase();
+        if (combinedMessage.includes("notenoughpoints") || combinedMessage.includes("insufficient") || combinedMessage.includes("not enough points")) {
             throw new OpenLoyaltyError({
                 code: "INSUFFICIENT_BALANCE",
                 message: "Member does not have enough points to complete this operation",
@@ -110,8 +115,13 @@ export async function pointsTransfer(input) {
         return { transferId: response.transferId };
     }
     catch (error) {
+        // Check for insufficient points error - API returns various formats
+        // Must check both error.message AND axios response data
         const errorMessage = error instanceof Error ? error.message : String(error);
-        if (errorMessage.includes("NotEnoughPoints") || errorMessage.includes("insufficient")) {
+        const axiosError = error;
+        const responseMessage = axiosError.response?.data?.message || "";
+        const combinedMessage = `${errorMessage} ${responseMessage}`.toLowerCase();
+        if (combinedMessage.includes("notenoughpoints") || combinedMessage.includes("insufficient") || combinedMessage.includes("not enough points")) {
             throw new OpenLoyaltyError({
                 code: "INSUFFICIENT_BALANCE",
                 message: "Sender does not have enough points to transfer",

package/dist/tools/reward/handlers.js

@@ -132,10 +132,11 @@ export async function rewardUpdate(input) {
             visibilityPayload.to = existingVisibility.to;
         payload.visibility = visibilityPayload;
     }
-    // usageLimit: Required field, but only include perUser (API rejects extra fields)
+    // usageLimit: BOTH perUser AND general are REQUIRED by the API
     // The GET response may include derived fields that PUT doesn't accept
     payload.usageLimit = {
         perUser: existingUsageLimit?.perUser ?? 1,
+        general: existingUsageLimit?.general ?? 10000,
     };
     // Coupon-specific fields (required for static_coupon, dynamic_coupon, conversion_coupon)
     if (existing.couponValue !== undefined)
@@ -212,8 +213,14 @@ export async function rewardBuy(input) {
         };
     }
     catch (error) {
+        // Check for insufficient points error - API returns various formats:
+        // "NotEnoughPoints", "insufficient", "not enough points" (lowercase)
+        // Must check both error.message AND axios response data
         const errorMessage = error instanceof Error ? error.message : String(error);
-        if (errorMessage.includes("NotEnoughPoints") || errorMessage.includes("insufficient")) {
+        const axiosError = error;
+        const responseMessage = axiosError.response?.data?.message || "";
+        const combinedMessage = `${errorMessage} ${responseMessage}`.toLowerCase();
+        if (combinedMessage.includes("notenoughpoints") || combinedMessage.includes("insufficient") || combinedMessage.includes("not enough points")) {
             throw new OpenLoyaltyError({
                 code: "INSUFFICIENT_BALANCE",
                 message: "Member does not have enough points to purchase this reward",

package/dist/tools/tierset.d.ts

@@ -19,17 +19,6 @@ export declare const TierSetCreateInputSchema: {
         attribute: "activeUnits" | "totalEarnedUnits" | "totalSpending" | "monthsSinceJoiningProgram" | "cumulatedEarnedUnits";
         walletType?: string | undefined;
     }>, "many">;
-    downgrade: z.ZodOptional<z.ZodObject<{
-        mode: z.ZodEnum<["none", "automatic", "x_days"]>;
-        days: z.ZodOptional<z.ZodNumber>;
-    }, "strip", z.ZodTypeAny, {
-        mode: "none" | "automatic" | "x_days";
-        days?: number | undefined;
-    }, {
-        mode: "none" | "automatic" | "x_days";
-        days?: number | undefined;
-    }>>;
-    active: z.ZodOptional<z.ZodBoolean>;
 };
 export declare const TierSetGetInputSchema: {
     storeCode: z.ZodOptional<z.ZodString>;
@@ -107,11 +96,6 @@ type TierSetCreateInput = {
         attribute: string;
         walletType?: string;
     }[];
-    downgrade?: {
-        mode: string;
-        days?: number;
-    };
-    active?: boolean;
 };
 type TierSetGetInput = {
     storeCode?: string;
@@ -196,17 +180,6 @@ export declare const tiersetToolDefinitions: readonly [{
             attribute: "activeUnits" | "totalEarnedUnits" | "totalSpending" | "monthsSinceJoiningProgram" | "cumulatedEarnedUnits";
             walletType?: string | undefined;
         }>, "many">;
-        downgrade: z.ZodOptional<z.ZodObject<{
-            mode: z.ZodEnum<["none", "automatic", "x_days"]>;
-            days: z.ZodOptional<z.ZodNumber>;
-        }, "strip", z.ZodTypeAny, {
-            mode: "none" | "automatic" | "x_days";
-            days?: number | undefined;
-        }, {
-            mode: "none" | "automatic" | "x_days";
-            days?: number | undefined;
-        }>>;
-        active: z.ZodOptional<z.ZodBoolean>;
     };
     readonly handler: typeof tiersetCreate;
 }, {

package/dist/tools/tierset.js

@@ -20,11 +20,7 @@ export const TierSetCreateInputSchema = {
         walletType: z.string().optional().describe("Wallet type CODE (not UUID). Required for unit-based attributes (activeUnits, totalEarnedUnits, cumulatedEarnedUnits). " +
             "Use wallet_type_list to find walletType.code (e.g., 'default')."),
     })).describe("Array of conditions that define tier progression criteria. IMPORTANT: Use 'totalEarnedUnits' for lifetime points (NOT 'earnedUnits')."),
-    downgrade: z.object({
-        mode: DowngradeModeEnum.describe("Downgrade mode."),
-        days: z.number().optional().describe("Number of days for x_days mode (required if mode is x_days)."),
-    }).optional().describe("Downgrade configuration for the tier set."),
-    active: z.boolean().optional().describe("Whether the tier set is active. Defaults to true."),
+    // NOTE: downgrade and active are NOT supported at creation time - use tierset_update after creation
 };
 export const TierSetGetInputSchema = {
     storeCode: z.string().optional().describe("Store code for multi-tenant routing. DO NOT pass this parameter - the configured default will be used automatically. Only provide a value if the user explicitly asks to work with a different store."),
@@ -87,25 +83,34 @@ export async function tiersetList(input) {
 }
 export async function tiersetCreate(input) {
     const storeCode = getStoreCode(input.storeCode);
-    // API requires tierSet wrapper with translations (at least en is required)
-    // Also include empty tiers array - some API versions require it
-    const payload = {
-        tierSet: {
-            translations: {
-                en: {
-                    name: input.name,
-                    description: input.description || "",
-                },
+    // API QUIRK: Only accepts specific fields at creation time.
+    // Fields NOT supported at creation: tiers, active, downgrade
+    // These must be set via tierset_update or tierset_update_tiers after creation.
+    //
+    // REQUIRED fields:
+    // - translations.en.name (string)
+    // - conditions (array with attribute, optionally walletType)
+    const tierSetPayload = {
+        translations: {
+            en: {
+                name: input.name,
+                description: input.description || "",
             },
-            conditions: input.conditions,
-            tiers: [], // Required by API - tiers are added separately via tierset_update_tiers
-            downgrade: input.downgrade,
-            active: input.active ?? true,
         },
+        conditions: input.conditions,
     };
+    const payload = { tierSet: tierSetPayload };
     try {
         const response = await apiPost(`/${storeCode}/tierSet`, payload);
-        const validated = TierSetSchema.parse(response);
+        // API QUIRK: Create endpoint only returns { tierSetId: string }
+        // We need to fetch the full tier set to get the conditionId values
+        const createResponse = response;
+        if (!createResponse.tierSetId) {
+            throw new Error(`Unexpected response format: ${JSON.stringify(response)}`);
+        }
+        // Fetch the created tier set to get condition IDs
+        const tierSetResponse = await apiGet(`/${storeCode}/tierSet/${createResponse.tierSetId}`);
+        const validated = TierSetSchema.parse(tierSetResponse);
         return {
             tierSetId: validated.tierSetId,
             conditions: validated.conditions.map((c) => ({
@@ -228,7 +233,9 @@ export const tiersetToolDefinitions = [
         description: "Create a new tier set (loyalty program structure). " +
             "⚠️ LIMIT: Maximum 3 ACTIVE tier sets per store. ALWAYS call tierset_list FIRST to check existing tier sets - if one exists that matches your needs, REUSE IT instead of creating a new one. " +
             "WORKFLOW: 1) tierset_list (check existing) → 2) tierset_create (only if needed) → 3) tierset_get (to get conditionId) → 4) tierset_update_tiers (to define thresholds). " +
-            "Valid attributes: 'totalEarnedUnits' (lifetime points), 'activeUnits' (current balance), 'totalSpending', 'monthsSinceJoiningProgram'. " +
+            "⚠️ API LIMITATION: Only 'name', 'description', and 'conditions' are accepted at creation time. " +
+            "Fields like 'active', 'downgrade', 'tiers' are NOT supported at creation - use tierset_update after creation to set these. " +
+            "Valid condition attributes: 'totalEarnedUnits' (lifetime points), 'activeUnits' (current balance), 'totalSpending', 'monthsSinceJoiningProgram'. " +
             "COMMON MISTAKE: Use 'totalEarnedUnits' NOT 'earnedUnits' for lifetime points. " +
             "For unit-based attributes, set walletType to 'default'.",
         readOnly: false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-loyalty/mcp-server",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "type": "module",
   "description": "MCP server for Open Loyalty API - enables AI agents to manage loyalty programs, members, points, rewards, and transactions",
   "author": "Marcin Dyguda <md@openloyalty.io>",