@sellable/mcp 0.1.522 → 0.1.524

package/dist/tools/refill-sends.js

@@ -330,9 +330,6 @@ function paidRefreshApprovalPacket(params) {
     const action = params.action.action ?? {};
     const ids = actionIds(action);
     const threshold = numberValue(action.oldThreshold) ?? numberValue(action.threshold);
-    const maxStalenessSeconds = Math.max(numberValue(action.maxStalenessSeconds) ??
-        numberValue(action.paidInmailCreditsMaxStalenessSeconds) ??
-        MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS, MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS);
     return {
         approvalSource: "explicit_yolo_flag",
         workspaceId: params.workspaceId ?? null,
@@ -343,7 +340,7 @@ function paidRefreshApprovalPacket(params) {
         tableId: stringValue(ids.tableId) ?? stringValue(action.tableId) ?? null,
         columnId: stringValue(ids.columnId) ?? stringValue(action.columnId) ?? null,
         threshold: threshold ?? null,
-        maxStalenessSeconds,
+        maxStalenessSeconds: MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS,
         targetPlanFingerprint: targetPlanFingerprint(params.targetPlan),
     };
 }

package/dist/tools/refill-target-plan.js

@@ -95,11 +95,8 @@ function numericRecord(value) {
 function campaignClassificationValue(value) {
     return value === "sales_nav_cascade" ? "sales_nav_cascade" : undefined;
 }
-function normalizePaidInmailMaxStalenessSeconds(value) {
-    const numeric = optionalNumberValue(value);
-    return numeric !== null && numeric > 0
-        ? Math.max(Math.trunc(numeric), MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS)
-        : MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS;
+function normalizePaidInmailMaxStalenessSeconds(_value) {
+    return MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS;
 }
 function isFreshWithinMcpPaidInmailWindow(checkedAt, maxStalenessSeconds) {
     if (!checkedAt)
@@ -404,6 +401,7 @@ function sanitizeToolInput(value) {
             : undefined,
         columnRole: stringValue(value.columnRole),
         rowSelector: sanitizeRowSelector(value.rowSelector),
+        excludeRowIds: stringArray(value.excludeRowIds),
         oldThreshold: typeof value.oldThreshold === "number" ? value.oldThreshold : undefined,
         suggestedThreshold: typeof value.suggestedThreshold === "number"
             ? value.suggestedThreshold
@@ -564,29 +562,47 @@ function sanitizePaidInmail(value) {
     const originalMaxStalenessSeconds = optionalNumberValue(value.maxStalenessSeconds);
     const maxStalenessSeconds = normalizePaidInmailMaxStalenessSeconds(value.maxStalenessSeconds);
     const checkedAt = stringValue(value.checkedAt) ?? null;
+    const balance = typeof value.balance === "number" ? value.balance : null;
     const availableCredits = numberValue(value.availableCredits);
     const threshold = numberValue(value.threshold);
-    let status = stringValue(value.status) ?? "missing_credit_facts";
+    const originalStatus = stringValue(value.status) ?? "missing_credit_facts";
+    let status = originalStatus;
     let reason = stringValue(value.reason) ?? "";
     let freshnessPolicy;
-    if (status === "stale_credit_facts" &&
-        isFreshWithinMcpPaidInmailWindow(checkedAt, maxStalenessSeconds)) {
+    if (originalStatus === "below_threshold") {
+        status = "below_threshold";
+    }
+    else if (!checkedAt || balance === null) {
+        status = "missing_credit_facts";
+        reason =
+            "MCP uses a fixed 4h paid InMail sender-credit freshness window; cached credits are missing.";
+    }
+    else if (!isFreshWithinMcpPaidInmailWindow(checkedAt, maxStalenessSeconds)) {
+        status = "stale_credit_facts";
+        reason =
+            "MCP uses a fixed 4h paid InMail sender-credit freshness window; cached credits are stale.";
+    }
+    else {
         status = availableCredits < threshold ? "below_threshold" : "ok";
         reason =
             status === "below_threshold"
-                ? "MCP normalized paid InMail freshness to 4h; cached credits are fresh enough but below threshold."
-                : "MCP normalized paid InMail freshness to 4h; cached credits are fresh enough.";
+                ? "MCP uses a fixed 4h paid InMail sender-credit freshness window; cached credits are fresh enough but below threshold."
+                : "MCP uses a fixed 4h paid InMail sender-credit freshness window; cached credits are fresh enough.";
+    }
+    if (status !== originalStatus ||
+        originalMaxStalenessSeconds !== maxStalenessSeconds) {
         freshnessPolicy = {
             normalizedBy: "mcp",
-            minMaxStalenessSeconds: MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS,
+            maxStalenessSeconds: MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS,
             originalMaxStalenessSeconds,
+            originalStatus,
         };
     }
     return {
         status,
         reason,
         checkedAt,
-        balance: typeof value.balance === "number" ? value.balance : null,
+        balance,
         sentSince: numberValue(value.sentSince),
         availableCredits,
         threshold,
@@ -616,6 +632,40 @@ function filterFreshPaidInmailRefreshAction(action, paidInmail) {
     return (action.type !== "refresh_paid_inmail_credits" ||
         paidInmailNeedsRefresh(paidInmail));
 }
+function paidInmailRefreshActionForSenderPlan(plan, paidInmail) {
+    if (!paidInmailNeedsRefresh(paidInmail))
+        return null;
+    const senderId = stringValue(plan.senderId);
+    if (!senderId)
+        return null;
+    const campaignId = stringValue(paidInmail?.campaignId) ?? "unknown";
+    const columnId = stringValue(paidInmail?.columnId) ?? "unknown";
+    return {
+        type: "refresh_paid_inmail_credits",
+        senderId,
+        senderName: stringValue(plan.senderName) ?? "",
+        actionType: "send_inmail_closed",
+        campaignId: stringValue(paidInmail?.campaignId),
+        campaignName: stringValue(paidInmail?.campaignName),
+        tableId: stringValue(paidInmail?.tableId),
+        tableName: stringValue(paidInmail?.tableName),
+        columnId: stringValue(paidInmail?.columnId),
+        availableCredits: numberValue(paidInmail?.availableCredits),
+        oldThreshold: numberValue(paidInmail?.threshold),
+        maxStalenessSeconds: MCP_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS,
+        actionKey: ["refresh_paid_inmail_credits", senderId, campaignId, columnId].join(":"),
+        rereadAfter: "get_refill_target_plan",
+        reason: stringValue(paidInmail?.reason) ??
+            "MCP paid InMail sender-credit facts require refresh.",
+        yoloEligible: true,
+        ids: {
+            senderId,
+            campaignId: stringValue(paidInmail?.campaignId),
+            tableId: stringValue(paidInmail?.tableId),
+            columnId: stringValue(paidInmail?.columnId),
+        },
+    };
+}
 function sanitizeStructuredSenderPlans(value, selectedBySender) {
     if (!Array.isArray(value))
         return [];
@@ -634,6 +684,11 @@ function sanitizeStructuredSenderPlans(value, selectedBySender) {
                 .filter((action) => Boolean(action))
                 .filter((action) => filterFreshPaidInmailRefreshAction(action, paidInmail))
             : [];
+        const paidRefreshAction = paidInmailRefreshActionForSenderPlan(plan, paidInmail);
+        const effectiveNextActions = paidRefreshAction &&
+            !nextActions.some((action) => action.type === "refresh_paid_inmail_credits")
+            ? [paidRefreshAction, ...nextActions]
+            : nextActions;
         const manualAlternates = Array.isArray(plan.manualAlternates)
             ? plan.manualAlternates
                 .map((action, index) => sanitizeStructuredAction(action, selectedBySender, "manual", index + 1))
@@ -680,7 +735,7 @@ function sanitizeStructuredSenderPlans(value, selectedBySender) {
             paidInmail,
             sourcePlan: sanitizeSourcePlan(plan.sourcePlan),
             refillReceipt: sanitizeRefillReceipt(plan.refillReceipt),
-            nextActions,
+            nextActions: effectiveNextActions,
             manualAlternates,
             ...(isRecord(plan.emptyState)
                 ? {

package/dist/tools/senders.js

@@ -15,13 +15,7 @@ const DEFAULT_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS = 4 * 60 * 60;
 function numberOrNull(value) {
     return typeof value === "number" && Number.isFinite(value) ? value : null;
 }
-function normalizePaidInmailCreditsMaxStalenessSeconds(...values) {
-    for (const value of values) {
-        const numeric = numberOrNull(value);
-        if (numeric === null || numeric <= 0)
-            continue;
-        return Math.max(Math.trunc(numeric), DEFAULT_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS);
-    }
+function normalizePaidInmailCreditsMaxStalenessSeconds(..._values) {
     return DEFAULT_PAID_INMAIL_CREDITS_MAX_STALENESS_SECONDS;
 }
 function computeDisplayName(sender) {
@@ -183,11 +177,11 @@ export const senderToolDefinitions = [
                 },
                 maxStalenessSeconds: {
                     type: "number",
-                    description: "Optional cache freshness window in seconds. Values below 4h are clamped to 4h.",
+                    description: "Optional legacy cache freshness window in seconds. MCP refresh requests use the fixed 4h sender-credit freshness policy.",
                 },
                 paidInmailCreditsMaxStalenessSeconds: {
                     type: "number",
-                    description: "Optional legacy alias for maxStalenessSeconds. Values below 4h are clamped to 4h.",
+                    description: "Optional legacy alias for maxStalenessSeconds. MCP refresh requests use the fixed 4h sender-credit freshness policy.",
                 },
             },
             required: ["senderId", "workspaceId"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sellable/mcp",
-  "version": "0.1.522",
+  "version": "0.1.524",
   "type": "module",
   "description": "Sellable MCP server for Claude Code and Codex campaign workflows",
   "main": "dist/index.js",