@sellable/mcp 0.1.642 → 0.1.643

package/dist/tools/refill-sends.d.ts

@@ -965,8 +965,8 @@ export declare function executeRefillSendsCommand(input?: RefillSendsCommandInpu
 } | {
     ok: boolean;
     status: "blocked";
-    code: "workspace_exact_target_unavailable" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
-    blocker: "workspace_exact_target_unavailable" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
+    code: "workspace_exact_target_unavailable" | "workspace_exact_target_ambiguous" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
+    blocker: "workspace_exact_target_unavailable" | "workspace_exact_target_ambiguous" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
     note: string;
     targetPlan: unknown;
     exactTargetPlan: {} | null;
@@ -1454,8 +1454,8 @@ export declare function executeRefillSendsCommand(input?: RefillSendsCommandInpu
 } | {
     ok: boolean;
     status: "blocked";
-    code: "workspace_exact_target_unavailable" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
-    blocker: "workspace_exact_target_unavailable" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
+    code: "workspace_exact_target_unavailable" | "workspace_exact_target_ambiguous" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
+    blocker: "workspace_exact_target_unavailable" | "workspace_exact_target_ambiguous" | "exact_target_config_unavailable" | "exact_target_config_mismatch" | "exact_target_projection_mismatch";
     note: string;
     targetPlan: unknown;
     exactTargetPlan: {} | null;

package/dist/tools/refill-sends.js

@@ -529,15 +529,60 @@ function recordsWithExactTarget(value) {
             stringValue(candidate.tableId)))
         : [];
 }
-function workspaceExactTargetsFromAction(action) {
+function workspaceExactActionSignature(action) {
+    const campaignId = actionCampaignId(action);
+    const tableId = actionTableId(action);
+    const actionKey = stringValue(action.actionKey);
+    const actionType = stringValue(action.actionType);
+    if (!campaignId || !tableId || !actionKey || !actionType)
+        return null;
+    return JSON.stringify([campaignId, tableId, actionKey, actionType]);
+}
+function workspaceAggregateActionSenderIds(workspacePlan) {
+    const target = yoloTarget(workspacePlan);
+    const senderPlans = Array.isArray(target?.senderRefillPlans)
+        ? target.senderRefillPlans
+            .map(recordValue)
+            .filter((plan) => Boolean(plan))
+        : [];
+    const senderIdsByAction = new Map();
+    for (const senderPlan of senderPlans) {
+        const senderId = stringValue(senderPlan.senderId);
+        const nextActions = Array.isArray(senderPlan.nextActions)
+            ? senderPlan.nextActions
+                .map(recordValue)
+                .filter((action) => Boolean(action))
+            : [];
+        if (!senderId)
+            continue;
+        for (const action of nextActions) {
+            const signature = workspaceExactActionSignature(action);
+            if (!signature)
+                continue;
+            const senderIds = senderIdsByAction.get(signature) ?? new Set();
+            senderIds.add(senderId);
+            senderIdsByAction.set(signature, senderIds);
+        }
+    }
+    return new Map([...senderIdsByAction.entries()]
+        .filter(([, senderIds]) => senderIds.size > 1)
+        .map(([signature, senderIds]) => [
+        signature,
+        [...senderIds],
+    ]));
+}
+function workspaceExactTargetsFromAction(action, aggregateSenderIdsByAction = new Map()) {
     const toolInput = recordValue(action.toolInput) ?? {};
+    const exactTargets = recordsWithExactTarget(toolInput.exactTargets);
     const expectedTargets = recordsWithExactTarget(toolInput.expectedTargets);
     const sourceScopes = recordsWithExactTarget(toolInput.sourceScopes);
-    const exactEdges = expectedTargets.length > 0
-        ? expectedTargets
-        : sourceScopes.length > 0
-            ? sourceScopes
-            : [null];
+    const exactEdges = exactTargets.length > 0
+        ? exactTargets
+        : expectedTargets.length > 0
+            ? expectedTargets
+            : sourceScopes.length > 0
+                ? sourceScopes
+                : [null];
     const campaignIds = normalizeStrings(toolInput.campaignIds);
     const tableIds = normalizeStrings(toolInput.tableIds);
     return exactEdges.flatMap((exactEdge) => {
@@ -548,12 +593,26 @@ function workspaceExactTargetsFromAction(action) {
         if (!campaignId || !tableId)
             return [];
         const exactEdgeSenderIds = normalizeStrings(exactEdge?.senderIds);
-        const senderId = stringValue(exactEdge?.senderId) ?? actionSenderId(action) ?? null;
-        const senderIds = senderId
-            ? [senderId]
-            : exactEdgeSenderIds.length > 0
-                ? exactEdgeSenderIds
-                : normalizeStrings(toolInput.senderIds);
+        const exactEdgeSenderId = stringValue(exactEdge?.senderId);
+        const toolInputSenderIds = normalizeStrings(toolInput.senderIds);
+        const aggregateSenderIds = aggregateSenderIdsByAction.get(workspaceExactActionSignature(action) ?? "") ?? [];
+        const actionSenderIdValue = actionSenderId(action);
+        // Explicit edge/list scope is authoritative. Legacy shared campaign
+        // actions predate expectedTargets and can repeat the same representative
+        // sender action across several senderRefillPlans. Preserve that recovered
+        // member set instead of narrowing the exact coordinator to the complete
+        // representative sender and hiding another member's actionable gap.
+        const senderIds = exactEdgeSenderIds.length > 0
+            ? exactEdgeSenderIds
+            : exactEdgeSenderId
+                ? [exactEdgeSenderId]
+                : toolInputSenderIds.length > 0
+                    ? toolInputSenderIds
+                    : aggregateSenderIds.length > 0
+                        ? aggregateSenderIds
+                        : actionSenderIdValue
+                            ? [actionSenderIdValue]
+                            : [];
         const directActionType = stringValue(exactEdge?.actionType) ??
             stringValue(exactEdge?.refillLaneKey) ??
             stringValue(action.actionType);
@@ -665,7 +724,34 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
             .filter((action) => Boolean(action))
         : [];
     const workspaceActions = [...globalActions, ...rankedCandidates];
-    const actionTargets = workspaceActions.flatMap(workspaceExactTargetsFromAction);
+    const aggregateSenderIdsByAction = workspaceAggregateActionSenderIds(workspacePlan);
+    const legacyGlobalActionCounts = new Map();
+    for (const action of globalActions) {
+        const signature = workspaceExactActionSignature(action);
+        if (!signature)
+            continue;
+        legacyGlobalActionCounts.set(signature, (legacyGlobalActionCounts.get(signature) ?? 0) + 1);
+    }
+    const ambiguousLegacyAction = globalActions.find((action) => {
+        const signature = workspaceExactActionSignature(action);
+        if (!signature || (legacyGlobalActionCounts.get(signature) ?? 0) < 2) {
+            return false;
+        }
+        const toolInput = recordValue(action.toolInput) ?? {};
+        const hasExplicitTargets = recordsWithExactTarget(toolInput.exactTargets).length > 0 ||
+            recordsWithExactTarget(toolInput.expectedTargets).length > 0 ||
+            recordsWithExactTarget(toolInput.sourceScopes).length > 0;
+        return !hasExplicitTargets && !aggregateSenderIdsByAction.has(signature);
+    });
+    if (ambiguousLegacyAction) {
+        return {
+            status: "blocked",
+            code: "workspace_exact_target_ambiguous",
+            note: "The workspace planner repeated one legacy aggregate action without explicit exact targets or recoverable sender membership. No refill run or mutation was started.",
+            workspacePlan,
+        };
+    }
+    const actionTargets = workspaceActions.flatMap((action) => workspaceExactTargetsFromAction(action, aggregateSenderIdsByAction));
     const evergreenTargets = evergreenExactTargetCandidates(workspacePlan, {
         preferredSenderId: options.preferredSenderId,
         actionTypes: input.actionTypes,

package/package.json

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