@sellable/mcp 0.1.148 → 0.1.149

package/dist/tools/leads.d.ts

@@ -9,6 +9,9 @@ export declare function selectSignalPostsForImport<T extends SignalPostForImport
 }): {
     posts: T[];
     estimatedEngagers: number;
+    availableEngagers: number;
+    targetEngagerCount: number | null;
+    targetReached: boolean;
     limited: boolean;
 };
 export type GetProviderPromptInput = {

package/dist/tools/leads.js

@@ -343,13 +343,18 @@ export function selectSignalPostsForImport(posts, options) {
     const normalizedTargetEngagers = normalizePositiveInteger(options.targetEngagerCount);
     const normalizedMaxPosts = normalizePositiveInteger(options.maxPostsToScrape);
     if (!normalizedTargetEngagers && !normalizedMaxPosts) {
+        const availableEngagers = posts.reduce((sum, post) => sum + post.likes + post.comments, 0);
         return {
             posts,
-            estimatedEngagers: posts.reduce((sum, post) => sum + post.likes + post.comments, 0),
+            estimatedEngagers: availableEngagers,
+            availableEngagers,
+            targetEngagerCount: null,
+            targetReached: true,
             limited: false,
         };
     }
     const ranked = [...posts].sort((a, b) => b.likes + b.comments - (a.likes + a.comments));
+    const availableEngagers = ranked.reduce((sum, post) => sum + post.likes + post.comments, 0);
     const selected = [];
     let estimatedEngagers = 0;
     for (const post of ranked) {
@@ -366,6 +371,10 @@ export function selectSignalPostsForImport(posts, options) {
     return {
         posts: selected.length > 0 ? selected : ranked,
         estimatedEngagers,
+        availableEngagers,
+        targetEngagerCount: normalizedTargetEngagers ?? null,
+        targetReached: !normalizedTargetEngagers ||
+            estimatedEngagers >= normalizedTargetEngagers,
         limited: selected.length > 0 && selected.length < posts.length,
     };
 }
@@ -2312,12 +2321,19 @@ export async function importLeads(input) {
             maxPostsToScrape,
         });
         const postsToScrape = importSelection.posts;
+        if (!importSelection.targetReached && importSelection.targetEngagerCount) {
+            const capClause = importSelection.availableEngagers >= importSelection.targetEngagerCount
+                ? ` The selected posts can cover the target, but maxPostsToScrape=${normalizePositiveInteger(maxPostsToScrape)} prevents reaching it. Increase maxPostsToScrape or remove that cap.`
+                : " Select/promote more right-content posts, run another narrow Signal Discovery search, or switch to Sales Nav recent activity if the lane cannot produce enough source candidates.";
+            throw new Error(`Signal Discovery selected posts only cover about ${importSelection.estimatedEngagers.toLocaleString("en-US")} visible engagers, below the approved ${importSelection.targetEngagerCount.toLocaleString("en-US")} source-candidate target. Do not scrape this under-capacity post set.${capClause}`);
+        }
         const effectiveHeadlineICPCriteria = headlineICPCriteria && headlineICPCriteria.length > 0
             ? headlineICPCriteria
             : rubricGuidelines;
         // Start the scrape job
         const result = await api.post(`/api/v3/campaigns/${campaignOfferId}/signal-leads/create`, {
             posts: postsToScrape,
+            targetEngagerCount: importSelection.targetEngagerCount ?? undefined,
             ...(effectiveHeadlineICPCriteria &&
                 effectiveHeadlineICPCriteria.length > 0
                 ? {
@@ -2577,6 +2593,19 @@ export async function confirmLeadList(input) {
     const leadListConfig = leadListMeta.table?.config ?? null;
     const leadListRowCount = leadListMeta.rowCount ?? 0;
     const importProgress = leadListConfig?.importProgress ?? null;
+    const signalSourceTargetLeadCount = resolvedProvider === "signal-discovery" &&
+        typeof leadListConfig?.targetLeadCount === "number" &&
+        Number.isFinite(leadListConfig.targetLeadCount) &&
+        leadListConfig.targetLeadCount > 0
+        ? leadListConfig.targetLeadCount
+        : null;
+    if (resolvedProvider === "signal-discovery" &&
+        signalSourceTargetLeadCount !== null &&
+        leadListConfig?.importStatus === "complete" &&
+        leadListRowCount > 0 &&
+        leadListRowCount < signalSourceTargetLeadCount) {
+        throw new Error(`Signal Discovery source list is under capacity: it completed with ${leadListRowCount.toLocaleString("en-US")} source candidates, below the approved ${signalSourceTargetLeadCount.toLocaleString("en-US")} source-candidate target. Do not import the bounded review batch from this source. Select more posts, rerun Signal Discovery, or move to Sales Nav/Prospeo.`);
+    }
     const progressProcessed = typeof importProgress?.processed === "number"
         ? importProgress.processed
         : null;

package/dist/tools/readiness.js

@@ -101,7 +101,7 @@ export const readinessToolDefinitions = [
                 },
                 targetLeadCount: {
                     type: "number",
-                    description: "Target number of leads requested. Used as a fallback completion check when status is unavailable.",
+                    description: "Target number of leads requested. Used as a fallback completion check when status is unavailable. For Signal Discovery, pass the approved source-candidate target; if the completed source list lands below it, the tool returns source_under_capacity instead of ready.",
                 },
                 timeoutMs: {
                     type: "number",
@@ -472,6 +472,23 @@ export async function waitForLeadListReady(input) {
                 }
             }
             if ((!requireRows || rowCount > 0) && importComplete) {
+                if (provider === "signal-discovery" &&
+                    typeof targetLeadCount === "number" &&
+                    rowCount > 0 &&
+                    rowCount < targetLeadCount) {
+                    return {
+                        ready: false,
+                        reason: "source_under_capacity",
+                        leadListId,
+                        provider: provider ?? null,
+                        attempts,
+                        elapsedMs: Date.now() - start,
+                        rowCount,
+                        status: lastStatus,
+                        targetLeadCount,
+                        error: `Signal Discovery completed with ${rowCount.toLocaleString("en-US")} source candidates, below the approved ${targetLeadCount.toLocaleString("en-US")} source-candidate target. Do not confirm this lead list; select more posts, rerun source discovery, or move to Sales Nav/Prospeo.`,
+                    };
+                }
                 return {
                     ready: true,
                     leadListId,

package/dist/tools/registry.d.ts

@@ -3702,7 +3702,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3736,7 +3735,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3795,7 +3793,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3854,7 +3851,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3915,7 +3911,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3946,7 +3941,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -3977,7 +3971,6 @@ export declare const allTools: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;

package/dist/tools/rubrics.d.ts

@@ -33,7 +33,6 @@ type WaitForRubricResultsInput = {
     tableId?: string;
     targetCount?: number;
     minPassedCount?: number;
-    minMessagesCount?: number;
     timeoutMs?: number;
     intervalMs?: number;
     includeRows?: boolean;
@@ -115,7 +114,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -149,7 +147,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -208,7 +205,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -267,7 +263,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -328,7 +323,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -359,7 +353,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -390,7 +383,6 @@ export declare const rubricToolDefinitions: ({
             tableId?: undefined;
             targetCount?: undefined;
             minPassedCount?: undefined;
-            minMessagesCount?: undefined;
             timeoutMs?: undefined;
             intervalMs?: undefined;
             includeRows?: undefined;
@@ -420,10 +412,6 @@ export declare const rubricToolDefinitions: ({
                 type: string;
                 description: string;
             };
-            minMessagesCount: {
-                type: string;
-                description: string;
-            };
             timeoutMs: {
                 type: string;
                 description: string;
@@ -520,12 +508,6 @@ export declare function checkRubric(input: CheckRubricInput): Promise<{
 export declare function waitForRubricResults(input: WaitForRubricResultsInput): Promise<{
     stats: WorkflowTableStats;
     rows?: import("./rows.js").LightweightRow[] | undefined;
-    messageGeneration?: {
-        completed: number;
-        passingGeneratedMessages: number;
-        minMessagesCount: number;
-        floorMet: boolean;
-    } | undefined;
     ready: boolean;
     attempts: number;
     elapsedMs: number;
@@ -538,14 +520,28 @@ export declare function waitForRubricResults(input: WaitForRubricResultsInput):
         targetCount: number;
         pending?: undefined;
     };
+    reason?: undefined;
+    partialResult?: undefined;
+    diagnostic?: undefined;
+    guidance?: undefined;
 } | {
     stats: WorkflowTableStats;
     rows?: import("./rows.js").LightweightRow[] | undefined;
+    ready: boolean;
+    partial: boolean;
+    reason: string;
+    attempts: number;
+    elapsedMs: number;
+    tableId: string;
+    passRate: {
+        completed: number;
+        passed: number;
+        pending: number;
+        percent: number;
+        targetCount: number;
+        minPassedCount: number;
+    };
     partialResult: {
-        messagesCount?: number | undefined;
-        passingGeneratedMessages?: number | undefined;
-        minMessagesCount?: number | undefined;
-        messageFloorMet?: boolean | undefined;
         completed: number;
         passed: number;
         pending: number;
@@ -555,32 +551,23 @@ export declare function waitForRubricResults(input: WaitForRubricResultsInput):
         enoughToDiagnose: boolean;
         floorMet: boolean;
     };
-    messageGeneration?: {
-        completed: number;
-        passingGeneratedMessages: number;
-        minMessagesCount: number;
-        floorMet: boolean;
-    } | undefined;
+    diagnostic?: undefined;
+    guidance?: undefined;
+} | {
     ready: boolean;
-    partial: boolean;
     reason: string;
     attempts: number;
     elapsedMs: number;
     tableId: string;
     passRate: {
+        minPassedCount?: number | undefined;
         completed: number;
         passed: number;
         pending: number;
         percent: number;
         targetCount: number;
-        minPassedCount: number;
     };
-} | {
     partialResult: {
-        messagesCount?: number | undefined;
-        passingGeneratedMessages?: number | undefined;
-        minMessagesCount?: number | undefined;
-        messageFloorMet?: boolean | undefined;
         enoughToDiagnose: boolean;
         floorMet: boolean;
         minPassedCount?: number | undefined;
@@ -594,31 +581,12 @@ export declare function waitForRubricResults(input: WaitForRubricResultsInput):
         totalRows: number;
         enrichedCount: number | undefined;
         needsEnrichCount: number | undefined;
-        messagesCount: number;
+        messagesCount: number | undefined;
         needsApprovalCount: number | undefined;
         processingCount: number | undefined;
         failedCount: number | undefined;
     };
     guidance: string;
     stats: WorkflowTableStats | null;
-    messageGeneration?: {
-        completed: number;
-        passingGeneratedMessages: number;
-        minMessagesCount: number;
-        floorMet: boolean;
-    } | undefined;
-    ready: boolean;
-    reason: string;
-    attempts: number;
-    elapsedMs: number;
-    tableId: string;
-    passRate: {
-        minPassedCount?: number | undefined;
-        completed: number;
-        passed: number;
-        pending: number;
-        percent: number;
-        targetCount: number;
-    };
 }>;
 export {};

package/dist/tools/rubrics.js

@@ -9,11 +9,6 @@ const DEFAULT_INTERVAL_MS = 2000;
 function sleep(ms) {
     return new Promise((resolve) => setTimeout(resolve, ms));
 }
-function countPassingGeneratedMessages(rowSnapshot) {
-    if (!rowSnapshot?.rows?.length)
-        return 0;
-    return rowSnapshot.rows.filter((row) => row.icpPassed === true && Boolean(row.message?.trim())).length;
-}
 function normalizeRubricItemDefaults(item) {
     return {
         ...item,
@@ -398,10 +393,6 @@ export const rubricToolDefinitions = [
                     type: "number",
                     description: "Optional pass floor for bounded create-campaign samples. When this floor is met, the tool returns ready:true with partial:true instead of waiting for every target row to finish.",
                 },
-                minMessagesCount: {
-                    type: "number",
-                    description: "Optional generated-message floor. When provided with minPassedCount, the tool waits until both floors are met so create-campaign-v2 can review the first passing generated message without waiting for the full batch.",
-                },
                 timeoutMs: {
                     type: "number",
                     description: `Max time to wait in ms (default ${DEFAULT_TIMEOUT_MS}).`,
@@ -697,7 +688,6 @@ export async function waitForRubricResults(input) {
     const intervalMs = Math.max(500, input.intervalMs ?? DEFAULT_INTERVAL_MS);
     const targetCount = resolveMaxProspects(input.targetCount);
     const minPassedCount = resolveMinPassedCount(input.minPassedCount);
-    const minMessagesCount = resolveMinPassedCount(input.minMessagesCount);
     const includeRows = input.includeRows !== false;
     let tableId = input.tableId;
     if (!tableId && input.campaignOfferId) {
@@ -719,33 +709,19 @@ export async function waitForRubricResults(input) {
         const totalRows = stats.totalRows ?? 0;
         const effectiveTarget = totalRows > 0 ? Math.min(targetCount, totalRows) : targetCount;
         const pending = Math.max(effectiveTarget - completed, 0);
-        const messagesCount = stats.messagesCount ?? 0;
         const failedCount = stats.failedCount ?? 0;
         const processingCount = stats.processingCount ?? 0;
         const queuedCount = stats.queuedCount ?? 0;
         const cancellableCount = stats.cancellableCount ?? 0;
         const minPassFloorMet = minPassedCount !== null && passed >= minPassedCount;
-        const rawMessageFloorMet = minMessagesCount === null || messagesCount >= minMessagesCount;
-        let rowSnapshotForMessageCheck = null;
-        let passingGeneratedMessagesCount = null;
-        if (minMessagesCount !== null && minPassFloorMet && rawMessageFloorMet) {
-            rowSnapshotForMessageCheck = await getTableRowsMinimal(tableId, {
-                limit: effectiveTarget,
-            });
-            passingGeneratedMessagesCount = countPassingGeneratedMessages(rowSnapshotForMessageCheck);
-        }
-        const messageFloorMet = minMessagesCount === null ||
-            (passingGeneratedMessagesCount ?? 0) >= minMessagesCount;
-        const earlyFloorMet = minPassFloorMet && messageFloorMet;
         const unresolvedRowsResolvedAsFailures = pending > 0 && completed + failedCount >= effectiveTarget;
         const noActiveProcessing = processingCount === 0 && queuedCount === 0 && cancellableCount === 0;
-        if (completed >= effectiveTarget && messageFloorMet) {
+        if (completed >= effectiveTarget) {
             const percent = completed > 0 ? Math.round((passed / completed) * 100) : 0;
             const rowSnapshot = includeRows
-                ? (rowSnapshotForMessageCheck ??
-                    (await getTableRowsMinimal(tableId, {
-                        limit: effectiveTarget,
-                    })))
+                ? await getTableRowsMinimal(tableId, {
+                    limit: effectiveTarget,
+                })
                 : null;
             return {
                 ready: true,
@@ -759,21 +735,11 @@ export async function waitForRubricResults(input) {
                     targetCount: effectiveTarget,
                     ...(minPassedCount !== null ? { minPassedCount } : {}),
                 },
-                ...(minMessagesCount !== null
-                    ? {
-                        messageGeneration: {
-                            completed: messagesCount,
-                            passingGeneratedMessages: passingGeneratedMessagesCount ?? 0,
-                            minMessagesCount,
-                            floorMet: true,
-                        },
-                    }
-                    : {}),
                 ...(rowSnapshot ? { rows: rowSnapshot.rows } : {}),
                 stats,
             };
         }
-        if (earlyFloorMet) {
+        if (minPassFloorMet) {
             const percent = completed > 0 ? Math.round((passed / completed) * 100) : 0;
             const reason = !noActiveProcessing
                 ? "min_passed_count_met_with_active_processing"
@@ -781,10 +747,9 @@ export async function waitForRubricResults(input) {
                     ? "min_passed_count_met_with_resolved_failures"
                     : "min_passed_count_met_no_active_processing";
             const rowSnapshot = includeRows
-                ? (rowSnapshotForMessageCheck ??
-                    (await getTableRowsMinimal(tableId, {
-                        limit: effectiveTarget,
-                    })))
+                ? await getTableRowsMinimal(tableId, {
+                    limit: effectiveTarget,
+                })
                 : null;
             return {
                 ready: true,
@@ -801,16 +766,6 @@ export async function waitForRubricResults(input) {
                     targetCount: effectiveTarget,
                     minPassedCount,
                 },
-                ...(minMessagesCount !== null
-                    ? {
-                        messageGeneration: {
-                            completed: messagesCount,
-                            passingGeneratedMessages: passingGeneratedMessagesCount ?? 0,
-                            minMessagesCount,
-                            floorMet: true,
-                        },
-                    }
-                    : {}),
                 partialResult: {
                     completed,
                     passed,
@@ -820,14 +775,6 @@ export async function waitForRubricResults(input) {
                     minPassedCount,
                     enoughToDiagnose: true,
                     floorMet: true,
-                    ...(minMessagesCount !== null
-                        ? {
-                            messagesCount,
-                            passingGeneratedMessages: passingGeneratedMessagesCount ?? 0,
-                            minMessagesCount,
-                            messageFloorMet: true,
-                        }
-                        : {}),
                 },
                 ...(rowSnapshot ? { rows: rowSnapshot.rows } : {}),
                 stats,
@@ -838,21 +785,9 @@ export async function waitForRubricResults(input) {
     const completed = lastStats?.passRate?.completed ?? 0;
     const passed = lastStats?.passRate?.passed ?? 0;
     const percent = completed > 0 ? Math.round((passed / completed) * 100) : 0;
-    const messagesCount = lastStats?.messagesCount ?? 0;
     const totalRows = lastStats?.totalRows ?? 0;
     const effectiveTarget = totalRows > 0 ? Math.min(targetCount, totalRows) : targetCount;
     const pending = Math.max(effectiveTarget - completed, 0);
-    let timeoutPassingGeneratedMessagesCount = null;
-    if (minMessagesCount !== null) {
-        try {
-            timeoutPassingGeneratedMessagesCount = countPassingGeneratedMessages(await getTableRowsMinimal(tableId, {
-                limit: effectiveTarget,
-            }));
-        }
-        catch {
-            timeoutPassingGeneratedMessagesCount = null;
-        }
-    }
     return {
         ready: false,
         reason: "timeout",
@@ -867,16 +802,6 @@ export async function waitForRubricResults(input) {
             targetCount: effectiveTarget,
             ...(minPassedCount !== null ? { minPassedCount } : {}),
         },
-        ...(minMessagesCount !== null
-            ? {
-                messageGeneration: {
-                    completed: messagesCount,
-                    passingGeneratedMessages: timeoutPassingGeneratedMessagesCount ?? 0,
-                    minMessagesCount,
-                    floorMet: (timeoutPassingGeneratedMessagesCount ?? 0) >= minMessagesCount,
-                },
-            }
-            : {}),
         partialResult: {
             completed,
             passed,
@@ -886,25 +811,17 @@ export async function waitForRubricResults(input) {
             ...(minPassedCount !== null ? { minPassedCount } : {}),
             enoughToDiagnose: completed > 0,
             floorMet: minPassedCount !== null && passed >= minPassedCount,
-            ...(minMessagesCount !== null
-                ? {
-                    messagesCount,
-                    passingGeneratedMessages: timeoutPassingGeneratedMessagesCount ?? 0,
-                    minMessagesCount,
-                    messageFloorMet: (timeoutPassingGeneratedMessagesCount ?? 0) >= minMessagesCount,
-                }
-                : {}),
         },
         diagnostic: {
             totalRows,
             enrichedCount: lastStats?.enrichedCount,
             needsEnrichCount: lastStats?.needsEnrichCount,
-            messagesCount,
+            messagesCount: lastStats?.messagesCount,
             needsApprovalCount: lastStats?.needsApprovalCount,
             processingCount: lastStats?.processingCount,
             failedCount: lastStats?.failedCount,
         },
-        guidance: "If this is create-campaign-v2 validate-sample, do not repeat waits indefinitely. Use passRate/stats to diagnose and surface sample_revision_required before Settings when the sample is under the pass floor. If minMessagesCount is set, one passing generated message is enough to start review; do not wait for a stronger sample.",
+        guidance: "If this is create-campaign-v2 validate-sample, do not repeat waits indefinitely. Use passRate/stats to diagnose and surface sample_revision_required before Settings when the sample is under the pass floor or messages are incomplete.",
         stats: lastStats,
     };
 }

package/package.json

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

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

@@ -49,10 +49,9 @@ for debug output.
 12. Sync the approved template into the campaign brief.
 13. After message approval, keep the watched app on Filter Leads while the
     bounded enrichment/filter cascade starts.
-14. Move to Messages only after at least one review row passes and one generated
-    message is ready for review. Do not wait for a stronger sample once that
-    first passing message exists; review it, then hand off to Settings, sender,
-    sequence, and explicit launch greenlight.
+14. Move to Messages only after at least one review row passes, generate and
+    review bounded messages, then hand off to Settings, sender, sequence, and
+    explicit launch greenlight.
 
 There is no normal approval-packet, commit-gate, atomic-mint, or local
 artifact-validation step. Those belong only to legacy validation/rehearsal

package/skills/create-campaign-v2/core/flow.v2.json

@@ -1124,14 +1124,8 @@
           "cellSource": "pending_generate_message_cells_for_passing_rows using approved_campaign_brief_template"
         },
         {
-          "tool": "wait_for_rubric_results",
-          "purpose": "wait_for_first_passing_generated_message",
-          "requiredValues": {
-            "includeRows": false,
-            "minPassedCount": 1,
-            "minMessagesCount": 1
-          },
-          "readVia": "stats_only_tool_result"
+          "tool": "wait_for_campaign_table_ready",
+          "purpose": "wait_for_generate_message_cells_to_complete"
         },
         {
           "action": "observe_generate_message_results",
@@ -1175,24 +1169,25 @@
             "currentStep": "auto-execute-messaging",
             "watchNarration.stage": "review-ready"
           },
-          "watchNarrationRule": "Say the first passing generated message is ready in Messages; next is review before Settings."
+          "watchNarrationRule": "Say what just happened, that review-batch messages are ready in Messages, and that the next action is reviewing or approving generated messages before Settings."
         },
         {
           "action": "ask_generated_message_review_choice",
           "uses": "request_user_input",
           "choices": [
-            "Approve generated message and continue to Settings",
+            "Approve generated messages",
             "Revise filters",
             "Revise message template",
             "Pause here"
-          ]
+          ],
+          "rule": "Do not advance to Settings until the generated review-batch messages have been reviewed and approved."
         }
       ],
       "allowedTools": [
         "get_subskill_asset",
         "get_rows_minimal",
         "queue_cells",
-        "wait_for_rubric_results",
+        "wait_for_campaign_table_ready",
         "update_campaign",
         "AskUserQuestion",
         "request_user_input"
@@ -1206,7 +1201,6 @@
       "hardRules": [
         "critique_failure_never_escalates",
         "critique_sample_size_bounded_by_config",
-        "first_passing_generated_message_unblocks_review",
         "critics_fixed_at_targeting_copy_voice",
         "synthesis_enforces_phase_84_token_contract",
         "opus_reserved_for_highest_value_subset",

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

@@ -60,10 +60,9 @@ auto-revise leads.
      explicit batch count anyway so future larger expansion batches do not
      accidentally stop early
      (see §Known Tool Behaviors #3)
-   - minPassedCount=1 means one passing filtered row unblocks Step 15 Generate
-     Message observation. Step 15 then waits only for one generated message
-     (`minMessagesCount=1`) before review. Do not wait for all sample rows to
-     finish before messages start.
+   - minPassedCount=1 means one passing filtered row unblocks Step 15
+     Generate Message observation. Do not wait for all sample rows to finish
+     before messages start.
 
 7. call `wait_for_rubric_results` with `includeRows=false`; extract ONLY:
    - ready: boolean
@@ -102,9 +101,7 @@ auto-revise leads.
 10. branch:
    if passInSample >= 1:
        proceed to Step 15 (auto-execute-messaging) with currently passing rows
-       so Generate Message can start without waiting for the full sample. Once
-       one passing generated message is ready, stop for user review instead of
-       waiting for a stronger sample.
+       so Generate Message can start without waiting for the full sample
    else:
        diagnose (see Brief-vs-List Diagnosis below)
        revisionRound += 1
@@ -182,8 +179,7 @@ processing makes the experience feel frozen.
 
 Workaround: treat timeout stats as a partial sample. If at least one row has
 passed, move to Step 15 and observe or queue Generate Message for the passing
-rows; Step 15 stops when one generated message is ready. If zero rows have
-passed and no active processing is visible, stop at
+rows. If zero rows have passed and no active processing is visible, stop at
 `Status: sample-needs-revision` before Settings. Show the completed / passed /
 pending counts and ask whether to revise source, revise filter/rubric, or wait
 once only if active processing is still visible.

package/skills/create-campaign-v2/references/step-13-import-leads.md

@@ -127,9 +127,11 @@ rate with a conservative cleanup factor, cap the source candidates at the
 approved provider limit, and select only enough posts to reach that engager
 count. The planning floor is 10% projected fit after cleanup; if Signal
 Discovery falls below that floor, do not import the source list. Route back to
-find-leads and move to Sales Nav recent activity instead. The subsequent
-`confirm_lead_list` call still uses `targetLeadCount: <importLimit>` so only the
-bounded review batch enters the campaign table.
+find-leads and move to Sales Nav recent activity instead. After the scrape,
+`wait_for_lead_list_ready` must clear the approved source-candidate target; if
+it reports `source_under_capacity`, do not call `confirm_lead_list`. The
+subsequent `confirm_lead_list` call still uses `targetLeadCount: <importLimit>`
+so only the bounded review batch enters the campaign table.
 
 For supplied direct lists, `confirm_lead_list` must receive
 `targetLeadCount: <importLimit>` or explicit `sourceRowIds` so a 100-row source

package/skills/create-campaign-v2/references/step-15-re-cascade.md

@@ -4,16 +4,15 @@ This reference governs Step 15 (`auto-execute-messaging`) re-cascade
 behavior when Step 14's validate-sample loop graduates additional rows
 from pending → passed after the first Generate Message cells have already run.
 
-Load whenever Step 15 is about to ask for generated-message review or transition
-to `awaiting-user-greenlight`, and on every resume into Step 15.
+Load whenever Step 15 is about to transition to
+`awaiting-user-greenlight`, and on every resume into Step 15.
 
 ## Principle
 
-Generate Message cells are cascade-scoped. If Step 14's rubric flips rows from
-pending → passed AFTER Step 15 first observes messages for the review batch, the
-new rows can sit pending with no message. That must not block the first review
-handoff. Step 15 opens review as soon as one passing generated message exists;
-late-passed rows can be re-cascaded after explicit user continuation or resume.
+Generate Message cells are cascade-scoped. If Step 14's rubric flips rows
+from pending → passed AFTER Step 15 first observes messages for the review
+batch, the new rows can sit pending with no message. Step 15 must explicitly
+queue the newly-pending Generate Message cells for the review-batch subset.
 
 Observed on manual Phase-85 signal-discovery run (2026-04-20): 15 new
 rows graduated pending → passed mid-tail while `messagesCount` stayed
@@ -23,17 +22,13 @@ flat at 257. The new rows never got messages.
 
 Re-cascade runs whenever ALL of the following are true:
 
-1. The user has already approved the first generated message or explicitly asked
-   to continue processing more review-batch rows.
+1. Step 15 has already observed the initial review-batch message cascade.
 2. A subsequent check of rubric state shows rows that were pending at
    first message-generation pass are now passed.
 3. Those newly-passed rows do NOT yet have generated messages
    (`messagesCount` flat relative to pre-cascade).
 4. Step 15 has NOT yet transitioned to `awaiting-user-greenlight`.
 
-Before that first generated-message approval, do not run this loop just to wait
-for a bigger sample. One passing generated message is enough for review.
-
 If condition 4 already fired (i.e. we're in Step 16), the re-cascade
 runs on resume into Step 15 if validate-sample was re-entered and new
 rows graduated.

package/skills/create-campaign-v2/references/watch-guide-narration.md

@@ -252,17 +252,16 @@ Template approved, bounded filter test running:
   "stage": "fit-message",
   "headline": "Template saved",
   "visibleState": "The browser stays on Filter Leads while the bounded enrichment and filter test runs.",
-  "agentIntent": "Codex saved the approved message template, queued the review-batch Enrich Prospect cells, and is waiting for one passing row with one generated message before moving to review.",
-  "nextAction": "Review the first passing generated message"
+  "agentIntent": "Codex saved the approved message template, queued the review-batch Enrich Prospect cells, and is waiting for at least one row to pass before moving to Messages.",
+  "nextAction": "Move to Messages after a passing row"
 }
 ```
 
 Do not move to Messages immediately after `approve-message`. The visible route
 is already Filter Leads after `save_rubrics`; approving the message only unlocks
 the bounded cascade from that screen. Move to Messages only once at least one
-review-batch row passes and one generated message is ready for review. Do not
-wait for the rest of the review batch or a stronger sample before asking the
-user to approve the generated message.
+review-batch row passes and Generate Message cells are ready/running for the
+passing rows.
 
 Messages waiting for template:
 

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

@@ -58,12 +58,11 @@ Step 14 — kick bounded cascade + observe sample
    passing filtered row exists.)
 
 Step 15 — observe messaging
-  wait_for_rubric_results({ minPassedCount: 1, minMessagesCount: 1, includeRows: false })
-  get_rows_minimal                 # confirm the first passing generated message exists
+  get_rows_minimal                 # confirm passing rows have completed Generate Message cells
   (rare) queue_cells on any pending Generate Message cells
   token-contract spot check via get_rows
   update_campaign(currentStep=auto-execute-messaging) with review-ready narration
-  ask the user to approve the generated message before Settings
+  ask the user to approve generated review-batch messages before Settings
   only after approval: update_campaign(currentStep=awaiting-user-greenlight)
   (generate_messages is NOT an MCP tool; messages come from the cascade)
 
@@ -292,10 +291,8 @@ Do not route to a visible `validate-sample` step. Full decision tree lives in
 review batch only. After `save_rubrics` and the approved message template are
 persisted, Step 14 queues the review-batch Enrich Prospect cells, waits until
 filter results start landing, then moves to message observation as soon as one
-row passes. Step 15 opens review as soon as one passing generated message
-exists. Do not wait for a larger or stronger sample once that first passing
-message is ready. It does NOT call `check_rubric`,
-`bulk_enrich_with_prospeo`, or any other direct enrichment/scoring tool.
+row passes. It does NOT call `check_rubric`, `bulk_enrich_with_prospeo`, or any
+other direct enrichment/scoring tool.
 
 Shape:
 
@@ -310,7 +307,6 @@ projectedPass = round(passInSample / sampleSize * importLimit)
 if wait_for_rubric_results.ready === true and passRate.passed >= 1:
     advance to Step 15 to observe or queue Generate Message for currently passing rows
     do not wait for every sample row to finish before message generation starts
-    stop for review as soon as one passing generated message is ready
 else if wait_for_rubric_results.ready === false and reason === "timeout":
     use the partial passRate/stats as the sample diagnostic
     if passRate.passed >= 1:
@@ -386,20 +382,17 @@ Template`. If it does not, fail before the cascade runs. Do not repair
      already (cascade auto-fired it). If it is still `pending`, queue
      it explicitly: `queue_cells({ tableId, cellIds:
 <generateMessageCellIds> })`.
-3. `wait_for_rubric_results({ tableId, targetCount: reviewBatchSize,
-minPassedCount: 1, minMessagesCount: 1, includeRows: false })` until the
-   first passing generated message is ready. Do not wait for every passing row's
-   Generate Message cell, and do not add "one more wait" for a stronger sample.
-   Remaining review-batch rows can continue processing in the background.
-4. Read the first ready generated message back via `get_rows` (full) and
-   sanity-check that sample against the Phase 84 token contract: no unresolved
+3. `wait_for_campaign_table_ready` (or poll) until every passing
+   row's Generate Message cell is `completed`.
+4. Read the results back via `get_rows` (full) and sanity-check a
+   sample against the Phase 84 token contract: no unresolved
    `{{tokens}}`, no invented proof, one sentence per line, etc.
 5. If the sample fails the token contract, diagnose brief-vs-list
    (same revision loop as Step 14) and escalate if over
    `maxRevisionRounds`.
-6. On success, keep `currentStep: "auto-execute-messaging"` and ask the user to
-   approve the generated message before continuing to Settings. Only that
-   approval may move the campaign to Settings / `awaiting-user-greenlight`.
+6. On success, keep `currentStep: "auto-execute-messaging"` and ask the user
+   to approve the generated review-batch messages. Only that approval may move
+   the campaign to Settings / `awaiting-user-greenlight`.
 
 **Do NOT hand-write message bodies via `update_cell`.** `update_cell`
 is reachable for legitimate operator overrides AFTER the tail hands
@@ -439,16 +432,15 @@ strings, which is why Step 16 requires Step 15 to be complete.
 4. Check `messaging.tokenContract`. When `strict`, reject any sample
    message that contains unresolved or unsupported tokens (including
    any critique rewrite that tried to introduce one).
-5. If the first passing generated message passes the token contract (and
-   critique when enabled), stop at the review-batch handoff. Do NOT wait for the
-   remaining review-batch rows and do NOT scale to the full source list before
-   explicit user expansion approval.
+5. If the sample passes the token contract (and critique when enabled),
+   stop at the review-batch handoff. Do NOT scale to the full source list
+   before explicit user expansion approval.
 6. If the sample fails the token contract or critique, diagnose +
    loop the same way Step 14 does (brief-vs-list), subject to the same
    `maxRevisionRounds` cap.
 7. On success, keep `currentStep: "auto-execute-messaging"`, show that the
-   first passing generated message is ready in Messages, and ask for approval
-   before Settings. Only after the user approves the generated message should
+   review-batch messages are ready in Messages, and ask for approval before
+   Settings. Only after the user approves those generated messages should
    `update_campaign({ campaignId, currentStep: "awaiting-user-greenlight" })`
    run.
 

package/skills/providers/signal-discovery.md

@@ -459,6 +459,9 @@ targetLeadCount, targetEngagerCount, maxPostsToScrape })` for the approved
 source-capacity plan without asking for another yes/no gate. Then
 `confirm_lead_list` imports only the bounded review batch into the campaign
 table. Do not confuse the source-candidate target with the review-batch size.
+If the completed source scrape comes back below the approved source-candidate
+target, do not call `confirm_lead_list`; select more posts, rerun source
+discovery, or move to Sales Nav.
 
 The promotion/select step is required campaign state. Use post IDs from the
 current campaign-scoped search result, not stale IDs copied from a source review

package/skills/research/config.json

@@ -1,9 +0,0 @@
-{
-  "parallelMode": "wide",
-  "agentCount": 6,
-  "maxToolCallsPerAgent": 2,
-  "senderMaxAgents": 2,
-  "senderMaxToolCallsPerAgent": 3,
-  "progressMode": true,
-  "debugMode": true
-}