@sellable/mcp 0.1.957 → 0.1.959

package/dist/tools/refill-v3-continue.js

@@ -18,7 +18,7 @@ export const REFILL_V3_CONTINUE_TOOL_NAME = "refill_v3_continue";
 export const refillV3ContinueToolDefinitions = [
     {
         name: REFILL_V3_CONTINUE_TOOL_NAME,
-        description: "Continue exactly one server-issued Refill V3 campaign-attention packet. continuation_pending means the same operation is still running: wait retryAfterMs and call this tool with identical scope and no decision until it returns. Reconcile transport timeouts the same way; do not call advance or dispatch another search while pending. Call first without decision: after a fresh revalidation, the tool deterministically owns live-job waiting, selected-source row copying, exact actionable enrichment, and same-template message generation. It returns model_decision_required only when the accurate packet needs a genuine repair, criteria, rubric, waterfall, or source-expansion judgment; then call again with that one closed decision. Only an exact fresh packet can dispatch one existing campaign-work authority, which re-observes the exact sender/date and settles before a sibling campaign may advance. The caller cannot supply batch sizes, readiness digests, retry counters, fill inputs, scheduler inputs, or idempotency keys.",
+        description: "Continue exactly one server-issued Refill V3 campaign-attention packet. continuation_pending means the same operation is still running: wait retryAfterMs and call this tool with identical scope and no decision until it returns. Reconcile transport timeouts the same way; do not call advance or dispatch another search while pending. Call first without decision: after a fresh revalidation, the tool deterministically owns live-job waiting, selected-source row copying, exact actionable enrichment, and same-template message generation. It returns model_decision_required only when the accurate packet needs a genuine repair, criteria, rubric, waterfall, or source-expansion judgment; then call again with that one closed decision. Only an exact fresh packet can dispatch one existing campaign-work authority, which re-observes the exact sender/date and settles before a sibling campaign may advance. The caller cannot supply batch sizes, readiness digests, retry counters, fill inputs, scheduler inputs, or idempotency keys. A result with recoveredFromError already re-observed after a failure: handle it by its kind. A thrown error is not a terminal: call refill_v3_advance once more with the identical scope and report a blocker only if it fails the same way.",
         inputSchema: REFILL_V3_CONTINUE_INPUT_SCHEMA,
         outputSchema: REFILL_V3_CONTINUE_OUTPUT_SCHEMA,
         annotations: {
@@ -183,6 +183,33 @@ const DEFAULT_PRODUCTION_OWNERS = {
 export async function runRefillV3CampaignWorkWithProductionOwners(input, owners = DEFAULT_PRODUCTION_OWNERS) {
     const decoded = decodeRefillV3CampaignWorkOutput(input.output);
     const chosen = decoded.ok ? decoded.value.actions[0] : null;
+    // A source-cycle row is the run's BOOKKEEPING for a source effect that has
+    // already happened or been refused; it is never the effect itself. Throwing
+    // here on 2026-10-05 discarded Justin's applied 396-row engager import and
+    // stopped the Slack run with nothing scheduled. A transient failure is
+    // retried in place (retrying the whole stage would re-dispatch the search);
+    // any other failure rides on the receipt, and the persisted campaign-work
+    // receipt plus the fresh re-observation still guard against a repeat.
+    const recordSourceCycle = async (record) => {
+        if (!owners.recordSourceCycle)
+            return undefined;
+        const sleep = owners.sleep ??
+            ((ms) => new Promise((resolve) => setTimeout(resolve, ms)));
+        for (let attempt = 0;; attempt += 1) {
+            try {
+                await owners.recordSourceCycle(record);
+                return undefined;
+            }
+            catch (error) {
+                const retryDelay = POST_ADVANCE_TRANSPORT_RETRY_DELAYS_MS[attempt];
+                if (retryDelay === undefined ||
+                    !isRetryablePostAdvanceTransportError(error)) {
+                    return error instanceof Error ? error.message : String(error);
+                }
+                await sleep(retryDelay);
+            }
+        }
+    };
     let diagnosticRows = [];
     const record = (value) => value && typeof value === "object" && !Array.isArray(value)
         ? value
@@ -195,6 +222,9 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
     const scopeEcho = `${input.packet.scope.workspaceId}:${input.packet.scope.campaignId}:${input.packet.scope.tableId}`;
     const receipt = (params) => ({
         ...(params.batchPlan === undefined ? {} : { batchPlan: params.batchPlan }),
+        ...(params.sourceCycleRecordError === undefined
+            ? {}
+            : { sourceCycleRecordError: params.sourceCycleRecordError }),
         outcome: params.outcome,
         result: params.result,
         scopeEcho,
@@ -813,7 +843,10 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     senderId: request.scope.senderId,
                     date: request.scope.date,
                     lane: input.packet.scope.lane,
-                    family: input.packet.evidence.leadSourceFamily,
+                    // The family being SEARCHED, which a family switch makes differ from
+                    // the packet's observed family. Its negative set is the one that
+                    // guards this dispatch, and the receipt below names the same family.
+                    family: request.sourceCommand.family,
                     protectedFitDigest,
                 };
                 // The run fact scopes both sides of reconciliation: writes are stamped
@@ -884,28 +917,27 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                         : null,
                 });
                 if (sourceOutcome.kind === "dispatched") {
-                    if (owners.recordSourceCycle) {
-                        await owners.recordSourceCycle({
-                            ...sourceCycleScope,
-                            ...sourceCycleRunFact,
-                            receipt: {
-                                schemaVersion: 1,
-                                kind: "search",
-                                campaignId: request.scope.campaignId,
-                                family: request.sourceCommand.family,
-                                protectedFitDigest,
-                                requestFingerprint: request.sourceCommand.requestFingerprint,
-                                sourceFingerprint: sourceOutcome.receipt.sourceAfterFingerprint,
-                                keywordBatch: request.sourceCommand.keywordBatch,
-                                postIds: request.sourceCommand.selectedPostIds,
-                                importedCount: sourceOutcome.receipt.importedCount,
-                                yieldEvidence: sourceOutcome.receipt.yieldEvidence,
-                                observedAt: request.scope.date,
-                            },
-                        });
-                    }
+                    const sourceCycleRecordError = await recordSourceCycle({
+                        ...sourceCycleScope,
+                        ...sourceCycleRunFact,
+                        receipt: {
+                            schemaVersion: 1,
+                            kind: "search",
+                            campaignId: request.scope.campaignId,
+                            family: request.sourceCommand.family,
+                            protectedFitDigest,
+                            requestFingerprint: request.sourceCommand.requestFingerprint,
+                            sourceFingerprint: sourceOutcome.receipt.sourceAfterFingerprint,
+                            keywordBatch: request.sourceCommand.keywordBatch,
+                            postIds: request.sourceCommand.selectedPostIds,
+                            importedCount: sourceOutcome.receipt.importedCount,
+                            yieldEvidence: sourceOutcome.receipt.yieldEvidence,
+                            observedAt: request.scope.date,
+                        },
+                    });
                     const changed = sourceOutcome.receipt.importedCount > 0;
                     return receipt({
+                        sourceCycleRecordError,
                         outcome: changed ? "applied" : "no_effect",
                         result,
                         idempotencyKey: request.idempotencyKey,
@@ -914,27 +946,26 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     });
                 }
                 if (sourceOutcome.kind === "recorded_exhaustion") {
-                    if (owners.recordSourceCycle) {
-                        await owners.recordSourceCycle({
-                            ...sourceCycleScope,
-                            ...sourceCycleRunFact,
-                            receipt: {
-                                schemaVersion: 1,
-                                kind: "exhaustion",
-                                campaignId: request.scope.campaignId,
-                                family: request.sourceCommand.family,
-                                protectedFitDigest,
-                                requestFingerprint: request.idempotencyKey,
-                                sourceFingerprint: priorSourceFingerprint ?? request.scope.evidenceFingerprint,
-                                keywordBatch: null,
-                                postIds: null,
-                                importedCount: 0,
-                                yieldEvidence: sourceOutcome.receipt.yieldEvidence,
-                                observedAt: request.scope.date,
-                            },
-                        });
-                    }
+                    const sourceCycleRecordError = await recordSourceCycle({
+                        ...sourceCycleScope,
+                        ...sourceCycleRunFact,
+                        receipt: {
+                            schemaVersion: 1,
+                            kind: "exhaustion",
+                            campaignId: request.scope.campaignId,
+                            family: request.sourceCommand.family,
+                            protectedFitDigest,
+                            requestFingerprint: request.idempotencyKey,
+                            sourceFingerprint: priorSourceFingerprint ?? request.scope.evidenceFingerprint,
+                            keywordBatch: null,
+                            postIds: null,
+                            importedCount: 0,
+                            yieldEvidence: sourceOutcome.receipt.yieldEvidence,
+                            observedAt: request.scope.date,
+                        },
+                    });
                     return receipt({
+                        sourceCycleRecordError,
                         outcome: "no_effect",
                         result,
                         idempotencyKey: request.idempotencyKey,
@@ -949,7 +980,7 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     // returning the retained job wait. Previously the pending branch
                     // wrote nothing, so the job could settle and disappear while the
                     // next advance still advertised the identical round as untried.
-                    await owners.recordSourceCycle({
+                    const sourceCycleRecordError = await recordSourceCycle({
                         ...sourceCycleScope,
                         ...sourceCycleRunFact,
                         receipt: {
@@ -975,6 +1006,7 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     // evidence-free timeout: reobserve so the next coordinator step can
                     // wait on (or consume) the exact job without replaying the search.
                     return receipt({
+                        sourceCycleRecordError,
                         outcome: "no_effect",
                         result,
                         idempotencyKey: request.idempotencyKey,
@@ -995,27 +1027,26 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     // authority, and let the same outer run re-observe/continue. Returning
                     // `receipt_incomplete` here left settlement null and repeated the
                     // same packet while every sibling sender/lane stayed unreachable.
-                    if (owners.recordSourceCycle) {
-                        await owners.recordSourceCycle({
-                            ...sourceCycleScope,
-                            ...sourceCycleRunFact,
-                            receipt: {
-                                schemaVersion: 1,
-                                kind: "exhaustion",
-                                campaignId: request.scope.campaignId,
-                                family: request.sourceCommand.family,
-                                protectedFitDigest,
-                                requestFingerprint: request.idempotencyKey,
-                                sourceFingerprint: priorSourceFingerprint ?? request.scope.evidenceFingerprint,
-                                keywordBatch: null,
-                                postIds: null,
-                                importedCount: 0,
-                                yieldEvidence: input.packet.evidence.priorYieldReceipt,
-                                observedAt: request.scope.date,
-                            },
-                        });
-                    }
+                    const sourceCycleRecordError = await recordSourceCycle({
+                        ...sourceCycleScope,
+                        ...sourceCycleRunFact,
+                        receipt: {
+                            schemaVersion: 1,
+                            kind: "exhaustion",
+                            campaignId: request.scope.campaignId,
+                            family: request.sourceCommand.family,
+                            protectedFitDigest,
+                            requestFingerprint: request.idempotencyKey,
+                            sourceFingerprint: priorSourceFingerprint ?? request.scope.evidenceFingerprint,
+                            keywordBatch: null,
+                            postIds: null,
+                            importedCount: 0,
+                            yieldEvidence: input.packet.evidence.priorYieldReceipt,
+                            observedAt: request.scope.date,
+                        },
+                    });
                     return receipt({
+                        sourceCycleRecordError,
                         outcome: "no_effect",
                         result: "source_frontier_exhausted",
                         idempotencyKey: request.idempotencyKey,
@@ -1059,28 +1090,26 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                             yieldEvidence: input.packet.evidence.priorYieldReceipt,
                         });
                         if (exhaustionVerdict.exhausted) {
-                            if (owners.recordSourceCycle) {
-                                await owners.recordSourceCycle({
-                                    ...sourceCycleScope,
-                                    ...sourceCycleRunFact,
-                                    receipt: {
-                                        schemaVersion: 1,
-                                        kind: "exhaustion",
-                                        campaignId: request.scope.campaignId,
-                                        family: request.sourceCommand.family,
-                                        protectedFitDigest,
-                                        requestFingerprint: request.idempotencyKey,
-                                        sourceFingerprint: priorSourceFingerprint ??
-                                            request.scope.evidenceFingerprint,
-                                        keywordBatch: null,
-                                        postIds: null,
-                                        importedCount: 0,
-                                        yieldEvidence: input.packet.evidence.priorYieldReceipt,
-                                        observedAt: request.scope.date,
-                                    },
-                                });
-                            }
+                            const sourceCycleRecordError = await recordSourceCycle({
+                                ...sourceCycleScope,
+                                ...sourceCycleRunFact,
+                                receipt: {
+                                    schemaVersion: 1,
+                                    kind: "exhaustion",
+                                    campaignId: request.scope.campaignId,
+                                    family: request.sourceCommand.family,
+                                    protectedFitDigest,
+                                    requestFingerprint: request.idempotencyKey,
+                                    sourceFingerprint: priorSourceFingerprint ?? request.scope.evidenceFingerprint,
+                                    keywordBatch: null,
+                                    postIds: null,
+                                    importedCount: 0,
+                                    yieldEvidence: input.packet.evidence.priorYieldReceipt,
+                                    observedAt: request.scope.date,
+                                },
+                            });
                             return receipt({
+                                sourceCycleRecordError,
                                 outcome: "no_effect",
                                 result: "source_frontier_exhausted",
                                 idempotencyKey: request.idempotencyKey,
@@ -1143,12 +1172,13 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     });
             }
             if (request.action === "record_source_frontier_exhausted") {
+                let sourceCycleRecordError;
                 if (owners.recordSourceCycle) {
                     const protectedFitDigest = input.packet.evidence.protectedFitDigest ||
                         `sha256:${createHash("sha256")
                             .update(input.packet.evidence.campaignObjective)
                             .digest("hex")}`;
-                    await owners.recordSourceCycle({
+                    sourceCycleRecordError = await recordSourceCycle({
                         workspaceId: request.scope.workspaceId,
                         runToken: input.runToken,
                         scopeFingerprint: input.scopeFingerprint,
@@ -1176,6 +1206,7 @@ export async function runRefillV3CampaignWorkWithProductionOwners(input, owners
                     });
                 }
                 return receipt({
+                    sourceCycleRecordError,
                     outcome: "no_effect",
                     result,
                     idempotencyKey: request.idempotencyKey,
@@ -1368,7 +1399,63 @@ function deterministicCampaignDecision(packet) {
         consumedDiagnosticRowIds: null,
     };
 }
+/**
+ * V33-JUSTIN-SELF-HEAL · an unattended refill never ends on a raw error that
+ * a fresh observation can answer.
+ *
+ * On 2026-10-05 Justin's own-post refresh imported 396 engagers and a
+ * bookkeeping 409 then escaped as a raw error, so the Slack host reported a
+ * blocker with nothing scheduled although the run credential was still
+ * current. Once this continuation holds a refreshed credential, any failure
+ * gets ONE re-observation with it. When the world moved, that observation is
+ * returned like any other and names the error it recovered from. When nothing
+ * moved, the same work would fail the same way, so the original error stands:
+ * a deterministic failure surfaces instead of looping. A failure before the
+ * credential was refreshed is never masked.
+ */
 async function executeRefillV3Continuation(input, dependencies) {
+    const progress = {
+        runToken: null,
+        packet: null,
+    };
+    try {
+        return await executeRefillV3ContinuationStep(input, dependencies, progress);
+    }
+    catch (error) {
+        const runToken = progress.runToken;
+        const failedPacket = progress.packet;
+        if (!runToken || !failedPacket)
+            throw error;
+        let observed;
+        try {
+            observed = await reconnectPostAdvanceStage({
+                sleep: dependencies.sleep,
+                operation: () => dependencies.advance({
+                    workspaceId: failedPacket.scope.workspaceId,
+                    scope: input.scope,
+                    ...(input.campaignId === undefined
+                        ? {}
+                        : { campaignId: input.campaignId }),
+                    ...(input.yolo === undefined ? {} : { yolo: input.yolo }),
+                    runToken,
+                }),
+            });
+        }
+        catch {
+            throw error;
+        }
+        if (observed.kind === "campaign_attention_required" &&
+            samePacket(failedPacket, observed.packet)) {
+            throw error;
+        }
+        return {
+            ...continuationResultFromAdvance(observed),
+            recoveredFromError: (error instanceof Error ? error.message : String(error)) ||
+                "unknown_error",
+        };
+    }
+}
+async function executeRefillV3ContinuationStep(input, dependencies, progress) {
     const workspaceId = normalizeExplicitWorkspaceId(input.workspaceId);
     if (!workspaceId || workspaceId !== input.packet.scope.workspaceId) {
         throw new Error("refill_v3_continue requires one exact workspace matching the server-issued packet.");
@@ -1422,6 +1509,8 @@ async function executeRefillV3Continuation(input, dependencies) {
     if (!runToken || !scopeFingerprint) {
         throw new Error("refill_v3_continue got no refreshed run credential from its own internal advance, which already consumed this run's sequence. Refusing to return a spent token.");
     }
+    progress.runToken = runToken;
+    progress.packet = advanceResult.packet;
     const deterministicDecision = deterministicCampaignDecision(advanceResult.packet);
     const decision = deterministicDecision ?? input.decision;
     if (decision === undefined) {
@@ -1642,6 +1731,9 @@ async function executeRefillV3Continuation(input, dependencies) {
     if (input.yolo === true &&
         (preparationOwnsSettlement || directlyQueuedRowsOwnSettlement) &&
         campaignSettlementIsReady(settlement)) {
+        // This advance spends the refreshed credential, so a failure past here has
+        // no unspent credential left to re-observe with.
+        progress.runToken = null;
         const postSettlementAdvance = await reconnectPostAdvanceStage({
             sleep: dependencies.sleep,
             operation: () => dependencies.advance({

package/dist/tools/registry.d.ts

@@ -1313,7 +1313,7 @@ export declare const allTools: ({
         properties: {
             subskillName: {
                 type: string;
-                enum: readonly ["avoid-ai-writing", "building-gtm-tables", "campaign-daily-results", "campaign-daily-review", "campaign-weekly-review", "content", "conversation-chess", "create-ab-test", "create-campaign", "create-campaign-brief", "create-campaign-v2", "create-campaign-v2-tail", "create-campaign-v2-validation", "create-evergreen-campaigns", "create-post", "create-rubric", "engage", "enrich-prospects", "fill-send-horizon", "find-leads", "find-leads-v2", "first-campaign-onboarding", "foundation", "generate-messages", "interview", "load-voice", "refresh-sender-engagement", "refill-sends", "refill-sends-waterfall-order", "refill-sends-work-campaign", "refill-sends-workflow", "research", "research-prospect", "research-sender", "weekly-campaign-summary", "workflow-sequences"];
+                enum: readonly ["avoid-ai-writing", "building-gtm-tables", "campaign-daily-results", "campaign-daily-review", "campaign-weekly-review", "content", "conversation-chess", "create-ab-test", "create-campaign", "create-campaign-brief", "create-campaign-v2", "create-campaign-v2-tail", "create-campaign-v2-validation", "create-evergreen-campaigns", "create-post", "create-rubric", "draft-message", "engage", "enrich-prospects", "fill-send-horizon", "find-leads", "find-leads-v2", "first-campaign-onboarding", "foundation", "generate-messages", "interview", "load-voice", "refresh-sender-engagement", "refill-sends", "refill-sends-waterfall-order", "refill-sends-work-campaign", "refill-sends-workflow", "research", "research-prospect", "research-sender", "weekly-campaign-summary", "workflow-sequences"];
                 description: string;
             };
             offset: {
@@ -1348,7 +1348,7 @@ export declare const allTools: ({
         properties: {
             subskillName: {
                 type: string;
-                enum: readonly ["avoid-ai-writing", "building-gtm-tables", "campaign-daily-results", "campaign-daily-review", "campaign-weekly-review", "content", "conversation-chess", "create-ab-test", "create-campaign", "create-campaign-brief", "create-campaign-v2", "create-campaign-v2-tail", "create-campaign-v2-validation", "create-evergreen-campaigns", "create-post", "create-rubric", "engage", "enrich-prospects", "fill-send-horizon", "find-leads", "find-leads-v2", "first-campaign-onboarding", "foundation", "generate-messages", "interview", "load-voice", "refresh-sender-engagement", "refill-sends", "refill-sends-waterfall-order", "refill-sends-work-campaign", "refill-sends-workflow", "research", "research-prospect", "research-sender", "weekly-campaign-summary", "workflow-sequences"];
+                enum: readonly ["avoid-ai-writing", "building-gtm-tables", "campaign-daily-results", "campaign-daily-review", "campaign-weekly-review", "content", "conversation-chess", "create-ab-test", "create-campaign", "create-campaign-brief", "create-campaign-v2", "create-campaign-v2-tail", "create-campaign-v2-validation", "create-evergreen-campaigns", "create-post", "create-rubric", "draft-message", "engage", "enrich-prospects", "fill-send-horizon", "find-leads", "find-leads-v2", "first-campaign-onboarding", "foundation", "generate-messages", "interview", "load-voice", "refresh-sender-engagement", "refill-sends", "refill-sends-waterfall-order", "refill-sends-work-campaign", "refill-sends-workflow", "research", "research-prospect", "research-sender", "weekly-campaign-summary", "workflow-sequences"];
                 description: string;
             };
             assetPath: {
@@ -11516,6 +11516,10 @@ export declare const allTools: ({
             readonly receiptPersistence: {
                 readonly type: "object";
             };
+            readonly recoveredFromError: {
+                readonly type: "string";
+                readonly minLength: 1;
+            };
         };
         readonly additionalProperties: false;
         readonly allOf: readonly [{

package/package.json

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

package/skills/create-campaign/SKILL.md

@@ -216,8 +216,8 @@ preparation, the target is wrong, or status shows the wrong campaign/table, call
 `cancel_campaign_message_preparation`; otherwise do not cancel a healthy prepare
 run. Never call `start_campaign` from message preparation; final launch remains
 a separate explicit user greenlight.
-Use Template is the default message path; AI Generated is only an explicit
-opt-out.
+First messages use Conversations or Message Testing; name the one you pick
+as create-campaign-v2 `references/message-approach.md` says.
 
 ## DNC And Blocklist Handling
 
@@ -598,7 +598,7 @@ Drafting finishes or the recommendation is reviewed. If the user skips filters,
 first, then move to Messages/message review after it has started or returned a
 ready recommendation. `update_campaign({ currentStep: "messages" })` is not
 proof of kickoff. Enrichment/filtering and Generate Message cells wait for
-message approval. AI Generated is an explicit opt-out from the template path.
+message approval (Message Testing only; Conversations has no template).
 
 The Message Drafting handoff must stay lean. Include only `campaignId`,
 `workflowTableId`, a concise brief summary, concise source summary/source-use

package/skills/create-campaign-v2/SKILL.md

@@ -86,7 +86,7 @@ and carry proof gaps into the brief.
 
 Before `create_campaign`, load `references/message-approach.md` and choose how
 first messages are written: Conversations (each written for the person, no
-template) or Message Lab (one approved template). Pass it as `messageApproach`.
+template) or Message Testing (one approved template). Pass it as `messageApproach`.
 
 ## Brief Provenance
 

package/skills/create-campaign-v2/SOUL.md

@@ -339,7 +339,8 @@ Leads with durable `currentStep: "apply-icp-rubric"` while Message Drafting
 finishes or the template is approved. The join gate is saved-filter approval or
 a resolved skip-filter choice, plus a message recommendation grounded in the
 same campaign/table basis. Enrichment, filtering, and Generate Message cells
-wait for template approval on the Use Template path.
+wait for template approval in a Message Testing campaign; a Conversations
+campaign has no template (see `references/message-approach.md`).
 Message Drafting does not wait for `save_rubrics`: if filters were chosen but
 saved `leadScoringRubrics` are not visible yet, it should still return the
 initial reusable template from the campaign-table execution slice with

package/skills/create-campaign-v2/core/policy.md

@@ -39,8 +39,8 @@ mutation gates:
 - Copy the confirmed source list into the campaign, but process only the first
   review/process sample before message approval.
 - Do not call `queue_campaign_cells` for enrichment/filtering until filters are
-  approved when enabled and the default Use Template path has an approved
-  message template/token rule set.
+  approved when enabled and, in a Message Testing campaign, there is an approved
+  message template/token rule set (a Conversations campaign has none).
 - Do not attach a sequence or call `start_campaign` until the user explicitly chooses to launch, unless `launchConsent.mode` is `autopilot`: that one-time answer pre-approves every gate in this list, while the quality floors and sample caps still apply.
 - If the user requested an exact scheduled send count, or asked to fill up/load
   sends for attached senders, prepare that bounded count first. Capacity-fill

package/skills/create-campaign-v2/references/approval-gate-framing.md

@@ -213,7 +213,8 @@ If `message-validation.md` has an extractable `Selected Winner`, the packet's
 supported prospect-row `{{tokens}}`. Do not write a fresh approval-template
 message from the brief. If no winner/template can be extracted, this exact-match
 gate is skipped, but the packet still needs a usable approved template unless
-the user explicitly chose AI-generated messages.
+the campaign uses Conversations (`messageApproach: "conversation"`), which has
+no template.
 
 Do NOT render the packet as a bundle of opaque validation anchors or a dump
 of raw JSON. The user is approving a live campaign — they need to see what

package/skills/create-campaign-v2/references/message-approach.md

@@ -2,25 +2,34 @@
 
 Every campaign writes its first message one of two ways. Decide which before
 `create_campaign`, pass it as `messageApproach`, and say which you picked in
-one plain line the customer can correct.
+one plain line the customer can correct. Use these two names, and no other
+names, for them.
 
-- **Conversations** (`messageApproach: "conversation"`). Each first message is
-  written for that one person by Sellable's writer and opens a conversation.
-  It carries no pitch and no ask; the offer comes up once they reply. Choose
-  it when replies have an owner: reply drafting is on for the campaign (the
-  default), so each reply arrives as a draft to approve, or the workspace's
-  Reply Agent answers.
-- **Message Lab** (`messageApproach: "message_test"`). One approved message
-  goes to everyone, so the customer can test which copy wins. This is the
-  template flow described in the create-campaign-v2 prompt and flow file. Choose it when the
-  customer wants to approve exact wording, test copy against copy, or has
-  switched reply drafting off.
+- **Conversations** (`messageApproach: "conversation"`). Each first message is written for that one person and starts a real conversation. The pitch waits until they reply.
+  Sellable writes each message from the person's profile; it carries no pitch
+  and no ask, and the offer comes up once they reply. Choose it when replies
+  have an owner: reply drafting is on for the campaign (the default), so each
+  reply arrives as a draft to approve, or the workspace's Reply Agent answers.
+  Best for fewer, higher-value prospects, where you want replies you can talk with.
+- **Message Testing** (`messageApproach: "message_test"`). You approve one message, it goes to everyone, and you see which wording gets replies.
+  This is the template flow described in the create-campaign-v2 prompt and
+  flow file. Choose it when the customer wants to approve exact wording, test
+  copy against copy, or has switched reply drafting off. Best for a clear
+  offer and a large market, where you want to know which copy wins.
+
+When you explain the choice, use the two one-line descriptions above as they
+are. Short form: "Message Testing finds the right pitch. Conversations earn
+the right to pitch."
 
 When the customer has not said, use Conversations if replies have an owner and
-Message Lab otherwise. "Same message to everyone", "test my copy" or "I want
-to approve the wording" means Message Lab. "Write each one", "personal",
-"conversational" or "use the reply agent" means Conversations. Change an
-existing campaign with `update_campaign({ campaignId, messageApproach })`.
+Message Testing otherwise. "Same message to everyone", "test my copy" or "I
+want to approve the wording" means Message Testing. "Write each one",
+"personal", "conversational" or "use the reply agent" means Conversations.
+Change an existing campaign with `update_campaign({ campaignId,
+messageApproach })`; the customer can also change it in the app. Both
+approaches draft new copy with the same writing skill, `draft-message`; the
+approach decides whether one approved message goes to everyone or each person
+gets their own.
 
 **A Conversations campaign has no template.** The backend writes each message
 only when the campaign carries no approved template, so:

package/skills/create-campaign-v2/references/parallel-critique-protocol.md

@@ -108,8 +108,8 @@ return results
 `is_highest_value(...)` and Opus eligibility live in
 `references/thomas-variant-selection.md`. The synthesis step's
 finalizer pass belongs to this optional legacy/template critique protocol.
-It does not apply a second writing-strategy gate to new Conversation Chess
-recommendations; their selected Chess assets and one language check remain
+It does not apply a second writing-strategy gate to new draft-message
+recommendations; their selected draft-message assets and one language check remain
 authoritative.
 
 ## Critic Groups

package/skills/create-campaign-v2/references/sample-validation-loop.md

@@ -54,7 +54,7 @@ rerun `import_leads`.
   only after saved-filter approval; `validate-sample` is the logical loop name,
   not a visible route)
 - Approved message template/token rules exist in the campaign brief before any
-  enrichment or scoring cells are queued (Message Lab only; a Conversations
+  enrichment or scoring cells are queued (Message Testing only; a Conversations
   campaign has no template)
 - Confirmed campaign rows from Step 13, with the internal execution slice stored
   in `WorkflowTable.config.mcp.reviewBatch` (default 15)

package/skills/create-campaign-v2-tail/SKILL.md

@@ -12,7 +12,7 @@ This is the tail detail extracted from the main create-campaign-v2 SKILL.md to k
 `create-campaign-v2` `references/message-approach.md`) have no message
 template. Every rule below that requires an approved template, `{{...}}`
 tokens in the brief, Message Drafting, or template approval before queueing
-applies to Message Lab campaigns only. In a Conversations campaign the same
+applies to Message Testing campaigns only. In a Conversations campaign the same
 cascade writes each message once a row passes the fit check.
 
 CampaignOffer state and the watch link are canonical. Disk artifacts are
@@ -409,7 +409,7 @@ the output — not to generate messages manually.
 
 1. Before queueing or waiting on Generate Message cells, confirm the minted
    campaign brief still contains `{{...}}` in `## Approved Message Template`
-   (Message Lab only; a Conversations campaign must have no template).
+   (Message Testing only; a Conversations campaign must have no template).
    If it does not, fail before the cascade runs. Do not repair after mint; the
    template must be present during mint so Step 15 never starts in freeform
    generation mode.

package/skills/create-campaign-v2-validation/SKILL.md

@@ -216,12 +216,12 @@ This gate has two existing callers:
   call generate-messages in DRY MODE and write message-validation.md atomically.
   No campaignId, live context reads, fresh research or DB writes in DRY MODE.
 
-For NEW, UNAPPROVED recommendations, require the actual Conversation Chess
+For NEW, UNAPPROVED recommendations, require the actual draft-message
 prompt, core input contract and selected mode/examples loaded through
 packaged MCP asset reads, with complete pagination. Preserve approved template
 bytes and explicit operator instructions; there is no writing-strategy flag.
 Do not layer gold-standard product-clarity, CTA/PS, required candidates,
-critic/finalizer or source→product→next-step recipes over Chess.
+critic/finalizer or source→product→next-step recipes over draft-message.
 
 Validate these observable properties:
 

package/skills/{conversation-chess → draft-message}/SKILL.md

@@ -1,9 +1,9 @@
 ---
-name: conversation-chess
+name: draft-message
 description: Draft or improve a prospect opener, reply or caller-eligible follow-up from the actual sender, offer and conversation, with relevant message examples.
 ---
 
-# Conversation Chess
+# Draft Message
 
 Write a message a person can naturally answer. One skill, three sections; select a useful conversational move rather than assemble a fixed sequence.
 

package/skills/{conversation-chess → draft-message}/compiled/follow-up/endpoint.md

@@ -1,4 +1,4 @@
-# Conversation Chess
+# Draft Message
 
 One draft for the actual sender/situation. Rules/modes/examples loaded: select relevant ones; never fetch assets. Synthetic examples and histories: their facts, links and outcomes are not customer evidence. Creative rapport needs scoped permission.