@sellable/mcp 0.1.747 → 0.1.749

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

@@ -688,6 +688,7 @@ type WorkspaceCoordinatorWaitAction = {
     senderId: string | null;
     reason: string | null;
 };
+declare function clearCompleteDefersForCampaign(runState: Record<string, unknown> | undefined, campaignId: string | null, preserveActionKey?: string | null): Record<string, unknown> | undefined;
 export declare function approvedPacketComparison(input: RefillSendsCommandInput, targetPlan: unknown): {
     changed: boolean;
     expectedTargetShapeRevision: string | null;
@@ -728,6 +729,10 @@ type RefillSenderSelectorMiss = {
         displayName: string;
     }>;
 };
+export declare const refillSendsDeferInternals: {
+    clearCompleteDefersForCampaign: typeof clearCompleteDefersForCampaign;
+    SUPPLY_INVALIDATED_DEFER_BLOCKERS: Set<string>;
+};
 export declare function executeRefillSendsCommand(input?: RefillSendsCommandInput): Promise<import("./workspace-context.js").WorkspaceRequiredResult | {
     ok: boolean;
     code: string;

package/dist/tools/refill-sends.js

@@ -918,7 +918,32 @@ const WORKSPACE_SUPPLY_ACTION_FAMILIES = new Set([
     "broaden_provider_search",
     "refresh_sender_post_engagers",
 ]);
-function clearCompleteDefersForCampaign(runState, campaignId) {
+// fix(112ar): the blocker vocabulary the ledger WRITES and the one this cleaner
+// MATCHED had drifted apart. `workspaceExactDeferredBlocker` records
+// "exact_action_complete" (see deferWorkspaceExactTarget), but the filter only
+// recognised "complete"/"capped_by_scheduler" -- so the clear never fired for
+// the blocker that actually gets written, and a satisfied-before-supply defer
+// became a STANDING claim for the rest of the session.
+//
+// Amplify 7-30: three exposed edges (run_scheduler_sweep, approve_messages,
+// prepare_messages) were all held at "exact_action_complete" while the same
+// payload reported 18 ready and 12 rows still to prepare. Every edge that could
+// have filled the day was suppressed, and the run terminaled
+// `actionable_supply_not_queued` -- an owner-invariant breach produced by a
+// stale proof, not by absent supply.
+//
+// Same shape as 112c: a bounded ATTEMPT must not harden into a standing claim.
+const SUPPLY_INVALIDATED_DEFER_BLOCKERS = new Set([
+    "complete",
+    "capped_by_scheduler",
+    "exact_action_complete",
+]);
+function clearCompleteDefersForCampaign(runState, campaignId, 
+// fix(112ar): the action that JUST landed keeps its own defer. Clearing it
+// would re-expose the edge we just completed and invite a duplicate copy /
+// approve. Only the OTHER campaign edges held on a now-stale completeness
+// proof are released.
+preserveActionKey) {
     if (!runState || !campaignId)
         return runState;
     const entries = Array.isArray(runState.coordinatorBlockedExactTargets)
@@ -929,7 +954,11 @@ function clearCompleteDefersForCampaign(runState, campaignId) {
     const kept = entries.filter((entry) => {
         const blocked = recordValue(entry);
         const blocker = stringValue(blocked?.blocker);
-        if (blocker !== "complete" && blocker !== "capped_by_scheduler") {
+        if (!SUPPLY_INVALIDATED_DEFER_BLOCKERS.has(blocker ?? "")) {
+            return true;
+        }
+        if (preserveActionKey &&
+            stringValue(blocked?.actionKey) === preserveActionKey) {
             return true;
         }
         return stringValue(blocked?.campaignId) !== campaignId;
@@ -1441,12 +1470,18 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
     // opening is barred for those targets (in the selection loop below).
     const completeDeferredTargets = blockedExactTargets.filter((blocked) => blocked.blocker === "complete" ||
         blocked.blocker === "capped_by_scheduler");
+    const waiveCompleteClaims = options.waiveCompleteClaims === true;
     const prefilterBlockedTargets = blockedExactTargets.filter((blocked) => blocked.blocker !== "complete" &&
-        blocked.blocker !== "capped_by_scheduler");
+        blocked.blocker !== "capped_by_scheduler" &&
+        // fix(112as): on the bounded release pass, a self-issued
+        // exact_action_complete claim no longer bars its own edge.
+        !(waiveCompleteClaims && blocked.blocker === "exact_action_complete"));
     const seenTargetKeys = new Set();
     // fix(112ak): every refusal is recorded so a non-empty-census terminal can
     // name the concrete per-lane blocker instead of reporting a bare dead end.
-    const coordinatorExcludedEdges = [];
+    const coordinatorExcludedEdges = [
+        ...(options.seedExcludedEdges ?? []),
+    ];
     const recordExcludedEdge = (candidate, blocked, stage) => {
         coordinatorExcludedEdges.push({
             campaignId: candidate.campaignId,
@@ -1458,6 +1493,46 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
             stage,
         });
     };
+    // fix(112as): a self-issued `exact_action_complete` claim is a BOUNDED
+    // ATTEMPT, not a standing proof — the same principle as 112c.
+    //
+    // The claim is dropped at BOTH refusal points (prefilter and post_rebind),
+    // and 112ar only clears it when a prepare/supply action LANDS on that
+    // campaign. When every edge the planner exposed for the campaign is already
+    // held by such a claim, nothing can land, so nothing ever clears it: a closed
+    // loop that terminals `actionable_supply_not_queued` against visible supply
+    // (Dotwork 2026-07-30/31: 100 enrichment candidates, approve_messages
+    // refused at post_rebind on every run). 112ak made that terminal NAME the
+    // refusal, which is honest but does not fill the day.
+    //
+    // So: if the ONLY reason we have no selection is our own completeness
+    // claims, and the census still shows actionable supply, waive them and
+    // re-select ONCE. Bounded by waiveCompleteClaims so it can never livelock,
+    // and it reuses the SAME workspacePlan so no extra planner call is made and
+    // the census stays consistent with the decision.
+    const completeClaimReleaseRetry = async () => {
+        if (waiveCompleteClaims)
+            return null;
+        if (coordinatorExcludedEdges.length === 0)
+            return null;
+        if (!coordinatorExcludedEdges.every((edge) => edge.blocker === "exact_action_complete")) {
+            return null;
+        }
+        const census = workspaceGapBearingActionableSupply(workspacePlan);
+        if (census.actionableRows <= 0 && !census.frontierHasMoreRows) {
+            return null;
+        }
+        const released = await selectWorkspaceExactTarget(input, workspacePlan, {
+            ...options,
+            waiveCompleteClaims: true,
+            seedExcludedEdges: [...coordinatorExcludedEdges],
+        });
+        // The release may only ADD a dispatch. If it fails to bind an edge, fall
+        // back to this pass's terminal, which already NAMES the refused edges —
+        // 112ak's honesty guarantee must not be traded away for an optimistic
+        // retry that lands on a vaguer blocker.
+        return released.status === "selected" ? released : null;
+    };
     const targetCandidates = actionTargets.filter((candidate) => {
         const key = workspaceExactTargetKey(candidate);
         if (seenTargetKeys.has(key))
@@ -1554,6 +1629,9 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
         const readyLoadedSelection = readySurplusLoadedSelection(workspacePlan);
         if (readyLoadedSelection)
             return readyLoadedSelection;
+        const releasedEmptyCandidates = await completeClaimReleaseRetry();
+        if (releasedEmptyCandidates)
+            return releasedEmptyCandidates;
         return {
             status: "blocked",
             ...classifyWorkspaceUnavailableTerminal(workspacePlan, (excludedTargets.length > 0 ||
@@ -1732,6 +1810,13 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
                 blocked.blocker === "loaded_awaiting_scheduler") {
                 return false;
             }
+            // fix(112as): the release pass must waive here too. post_rebind
+            // re-matches AFTER the exact preflight re-issues the semantic
+            // actionKey, so waiving only at prefilter would leave the edge blocked.
+            if (waiveCompleteClaims &&
+                blocked.blocker === "exact_action_complete") {
+                return false;
+            }
             recordExcludedEdge(exactSelectedTarget, blocked, "post_rebind");
             return true;
         })) {
@@ -1851,6 +1936,9 @@ async function selectWorkspaceExactTarget(input, workspacePlanOverride, options
     const readyLoadedSelection = readySurplusLoadedSelection(workspacePlan);
     if (readyLoadedSelection)
         return readyLoadedSelection;
+    const releasedFilteredCandidates = await completeClaimReleaseRetry();
+    if (releasedFilteredCandidates)
+        return releasedFilteredCandidates;
     return {
         status: "blocked",
         ...classifyWorkspaceUnavailableTerminal(workspacePlan, (workspaceCoverage.remainingReadyOrProjectedGap !== 0
@@ -3018,6 +3106,16 @@ async function resolveRefillSenderSelectors(params) {
         resolvedSelectors,
     };
 }
+// fix(112ar): exported for a DIRECT pin. Binding this rule through the full
+// command harness proved impossible without silently voiding the test --
+// deferWorkspaceExactTarget replaces same-identity entries, and
+// clearApprovalAttemptsForCampaign removes approve-family keys, so a seeded
+// defer disappears for reasons that have nothing to do with this fix. A unit
+// pin binds the writer/cleaner vocabulary contract directly.
+export const refillSendsDeferInternals = {
+    clearCompleteDefersForCampaign,
+    SUPPLY_INVALIDATED_DEFER_BLOCKERS,
+};
 export async function executeRefillSendsCommand(input = {}) {
     const normalizedInput = normalizeRefillSendsInput(input);
     const yolo = normalizedInput.yolo === true;
@@ -3871,10 +3969,10 @@ export async function executeRefillSendsCommand(input = {}) {
                     WORKSPACE_SUPPLY_ACTION_FAMILIES.has(supplyActionFamily)));
         const coordinatorWorkspaceRunState = prepFamilyLandedOnCampaign
             ? clearApprovalAttemptsForCampaign(supplyLandedOnCampaign
-                ? clearCompleteDefersForCampaign(deferredCoordinatorWorkspaceRunState, selectedCoordinatorTarget?.campaignId ?? null)
+                ? clearCompleteDefersForCampaign(deferredCoordinatorWorkspaceRunState, selectedCoordinatorTarget?.campaignId ?? null, selectedCoordinatorTarget?.actionKey ?? null)
                 : deferredCoordinatorWorkspaceRunState, selectedCoordinatorTarget?.campaignId ?? null)
             : supplyLandedOnCampaign
-                ? clearCompleteDefersForCampaign(deferredCoordinatorWorkspaceRunState, selectedCoordinatorTarget?.campaignId ?? null)
+                ? clearCompleteDefersForCampaign(deferredCoordinatorWorkspaceRunState, selectedCoordinatorTarget?.campaignId ?? null, selectedCoordinatorTarget?.actionKey ?? null)
                 : deferredCoordinatorWorkspaceRunState;
         // fix(112aj): continuation-level progress-or-settle. When this dispatch is an
         // active_exact_run resume (continuationHandle set, in_progress), fingerprint

package/package.json

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