@agifyai/leadify-mcp 9.0.0 → 9.1.1

package/README.md

@@ -312,9 +312,9 @@ Leadify et ne doit jamais être envoyée par un agent MCP.
 | `apply_canonical_identity_backfill` | Appliquer exactement un backfill d'identité relu par digest, sans outreach ni activation. |
 | `update_company_brain_sections` | Modifier uniquement les sections globales encore actives (`identity`, `positioning`, `allowedVocabulary`, `prohibitedVocabulary`, `legalConstraints`). Les anciennes sections Offre/Verticale sont filtrées et refusées. |
 | `publish_context_workspace` | Publier une révision prête du Company Brain après confirmation explicite (`confirm_publish: true`). Admin de l'organisation requis ; les raisons de non-readiness sont renvoyées par le serveur. |
-| `list_persona_contracts` / `get_persona_contract` | Lister ou lire les contrats Persona canoniques v2 d’une organisation explicitement sélectionnée. `intelligence` ne contient que `painPoints`, `icpStrategy` et `cardAnalysisSections` ; `activeTools` n’appartient pas au contrat. |
-| `create_persona_contract` | Créer un Persona tenant-scoped et lié à une Verticale depuis un contrat canonique v2 complet et strict. `tool_pack_id` est requis à la création et reste hors contrat. Toute clé inconnue est refusée. |
-| `replace_persona_contract` / `patch_persona_contract` | Remplacer ou modifier un contrat canonique v2 avec verrou optimiste. `name` / `description` sont acceptés sur replace. `tool_pack_id` est optionnel : s’il est omis, le pack persisté est réutilisé. Une Persona ACTIVE mutée repasse en DRAFT. |
+| `list_persona_contracts` / `get_persona_contract` | Lister ou lire les contrats Persona canoniques v2 d’une organisation explicitement sélectionnée. Le contrat suit celui de Leadify : `intelligence` ne contient que `painPoints`, `outreach` que `outreachTemplates`, et les blocs retirés par PRD-1816 (`agentDomain`, `signalTierRules`, `enableSignals`, `icpStrategy`, `cardAnalysisSections`, `outreachSignalRouting`, `toneInstructions`, `outreachFields`) sont refusés à l’écriture et retirés à la lecture des anciens contrats ; `activeTools` n’appartient pas au contrat. |
+| `create_persona_contract` | Créer un Persona tenant-scoped et lié à une Verticale depuis un contrat canonique v2 complet et strict. Aucun pack d’outils n’est demandé : `tool_pack_id`, s’il est encore envoyé, est ignoré et n’est jamais transmis à Leadify. Toute clé inconnue du contrat est refusée. |
+| `replace_persona_contract` / `patch_persona_contract` | Remplacer ou modifier un contrat canonique v2 avec verrou optimiste. `name` / `description` sont acceptés sur replace. `tool_pack_id` est ignoré, comme à la création. Une Persona ACTIVE mutée repasse en DRAFT. |
 | `change_persona_status` | Passer un Persona entre `DRAFT`, `ACTIVE` et `ARCHIVED` avec le même verrou `updated_at`, readiness et protection des références. Après un replace/patch, rappeler ce tool vers `ACTIVE` pour que les groupes résolvent à nouveau la Persona. |
 | `list_data_sources` | Lister toutes les sources de données configurées (par pays puis nom). |
 | `create_data_source` | Créer une nouvelle source de données (admin uniquement). |

package/dist/tools/campaigns.js

@@ -1,9 +1,12 @@
 import { z } from "zod";
 import { getClient } from "../client.js";
 import { toolResult, handleToolError } from "../types.js";
-const CAMPAIGN_STATUSES = ["DRAFT", "ACTIVE", "PAUSED", "COMPLETED"];
+// PILOT (Leadify PRD-2353) : entre le backlog et le lancement, la campagne rédige des
+// messages d'essai sans rien envoyer. Il s'entre et se quitte par les routes du pilote.
+const CAMPAIGN_STATUSES = ["DRAFT", "PILOT", "ACTIVE", "PAUSED", "COMPLETED"];
 const CAMPAIGN_EFFECTIVE_STATES = [
     "DRAFT",
+    "PILOT",
     "SCHEDULED",
     "ACTIVE",
     "PAUSED_MANUAL",

package/dist/tools/personas.js

@@ -6,15 +6,28 @@ import { LeadifyApiError, handleToolError, toolResult } from "../types.js";
  * versioned `personaContract`. These tools deliberately do not expose any
  * historical Persona field or field-specific patch operation.
  *
- * The published MCP contract must match the server `personaContractSchema`.
- * `intelligence.activeTools` is not part of that document; the tool pack lives
- * on `toolPackId`, outside the contract.
+ * The published MCP contract must match the server `personaContractSchema`
+ * (Leadify `apps/server/src/lib/persona-contract.ts`). `intelligence.activeTools`
+ * is not part of that document, and the blocks removed by PRD-1816 are refused
+ * on write and stripped from older stored contracts on read.
+ *
+ * PRD-2295: since PRD-2238 Leadify no longer requires or reads a tool pack.
+ * `tool_pack_id` stays accepted on writes so older callers keep working, but it
+ * is ignored and never sent to the API.
  */
 const tierSchema = z.object({
     rule: z.string(),
     expectedEvidence: z.string(),
     examples: z.array(z.string()).max(20),
 }).strict();
+// PRD-1816: blocks Leadify removed from the contract.
+const REMOVED_CONTRACT_KEYS = {
+    targeting: ["agentDomain"],
+    qualification: ["signalTierRules", "enableSignals"],
+    intelligence: ["icpStrategy", "cardAnalysisSections"],
+    outreach: ["outreachSignalRouting", "toneInstructions", "outreachFields"],
+};
+const REMOVED_TARGET_PROFILE_KEYS = ["buyerTypes", "coreDefinition", "productFamilies", "notTheDefinition"];
 const commonContractSchema = z.object({
     schemaVersion: z.literal(2),
     kind: z.enum(["PERSON", "COMPANY"]),
@@ -22,9 +35,21 @@ const commonContractSchema = z.object({
         targetProfile: z.unknown().nullable(),
         disqualificationCriteria: z.unknown().nullable(),
         schemaPacks: z.array(z.string()).max(2),
-        agentDomain: z.string().nullable(),
         outputLanguage: z.string().nullable(),
-    }).strict(),
+    }).strict().superRefine((targeting, ctx) => {
+        const profile = targeting.targetProfile;
+        if (!profile || typeof profile !== "object" || Array.isArray(profile))
+            return;
+        for (const key of REMOVED_TARGET_PROFILE_KEYS) {
+            if (profile[key] !== undefined) {
+                ctx.addIssue({
+                    code: z.ZodIssueCode.custom,
+                    path: ["targetProfile", key],
+                    message: `targeting.targetProfile.${key} was removed (PRD-1816)`,
+                });
+            }
+        }
+    }),
     qualification: z.object({
         tiers: z.object({
             hot: tierSchema,
@@ -38,19 +63,12 @@ const commonContractSchema = z.object({
             level: z.enum(["GOLDEN", "CRITICAL", "STRONG", "INFO"]),
             justification: z.string().min(1),
         }).strict()).max(100),
-        signalTierRules: z.unknown().nullable(),
-        enableSignals: z.boolean(),
     }).strict(),
     intelligence: z.object({
         painPoints: z.unknown().nullable(),
-        icpStrategy: z.unknown().nullable(),
-        cardAnalysisSections: z.unknown().nullable(),
     }).strict(),
     outreach: z.object({
-        outreachSignalRouting: z.unknown().nullable(),
         outreachTemplates: z.unknown().nullable(),
-        toneInstructions: z.string().nullable(),
-        outreachFields: z.unknown().nullable(),
     }).strict(),
 }).strict();
 const personExtensionSchema = z.object({
@@ -81,13 +99,38 @@ function withoutOutOfContractKeys(value) {
     }
     return contract;
 }
+/** Older stored contracts may still carry the PRD-1816 blocks: read them without those blocks. */
+function withoutRemovedContractKeys(value) {
+    const contract = withoutOutOfContractKeys(value);
+    if (!contract || typeof contract !== "object" || Array.isArray(contract))
+        return contract;
+    const record = contract;
+    for (const [section, keys] of Object.entries(REMOVED_CONTRACT_KEYS)) {
+        const block = record[section];
+        if (!block || typeof block !== "object" || Array.isArray(block))
+            continue;
+        for (const key of keys)
+            delete block[key];
+    }
+    const targeting = record.targeting;
+    const profile = targeting?.targetProfile;
+    if (profile && typeof profile === "object" && !Array.isArray(profile)) {
+        for (const key of REMOVED_TARGET_PROFILE_KEYS)
+            delete profile[key];
+    }
+    return record;
+}
 function parsePersistedContract(value) {
-    return contractSchema.parse(withoutOutOfContractKeys(value));
+    return contractSchema.parse(withoutRemovedContractKeys(value));
 }
 function parseWrittenContract(value) {
-    return parsePersistedContract(value);
+    return contractSchema.parse(withoutOutOfContractKeys(value));
 }
 const contractInputSchema = z.preprocess(withoutOutOfContractKeys, contractSchema).describe("Complete strict Persona contract, schemaVersion 2. intelligence.activeTools is not a contract field and is stripped if sent.");
+const ignoredToolPackId = z
+    .string()
+    .optional()
+    .describe("Deprecated and ignored: Leadify no longer uses tool packs. Never sent to the API; omit it.");
 const tenantScope = {
     organization_id: z
         .string()
@@ -102,12 +145,6 @@ const optimisticScope = {
         .datetime({ offset: true })
         .describe("Exact updatedAt obtained from a preceding read."),
 };
-function requireToolPackId(persona, override) {
-    const toolPackId = override?.trim() || persona.toolPackId?.trim();
-    if (!toolPackId)
-        throw new Error("PERSONA_TOOL_PACK_REQUIRED");
-    return toolPackId;
-}
 function canonicalPersona(persona) {
     if (!persona.personaContract || typeof persona.personaContract !== "object") {
         throw new Error("PERSONA_CONTRACT_REQUIRED: run the audited Persona migration before using this MCP.");
@@ -118,7 +155,6 @@ function canonicalPersona(persona) {
         name: persona.name,
         description: persona.description ?? null,
         verticalId: persona.verticalId,
-        toolPackId: persona.toolPackId ?? null,
         updatedAt: persona.updatedAt,
         revision: persona.revision ?? 0,
         status: persona.status,
@@ -171,7 +207,6 @@ async function replaceContract(input) {
         organizationId: input.organizationId,
         verticalId: persona.verticalId,
         expectedUpdatedAt: input.expectedUpdatedAt,
-        toolPackId: requireToolPackId(persona, input.toolPackId),
         ...(input.description !== undefined ? { description: input.description } : {}),
         personaContract: parseWrittenContract(input.contract),
     });
@@ -198,24 +233,19 @@ export function registerPersonaTools(server, injectedClient) {
             return handleToolError(error);
         }
     });
-    server.tool("create_persona_contract", "Create a Persona from one complete canonical contract v2. intelligence.activeTools is not part of the contract. tool_pack_id is required on create and lives outside the contract.", {
+    server.tool("create_persona_contract", "Create a Persona from one complete canonical contract v2. intelligence.activeTools is not part of the contract. No tool pack is needed.", {
         ...tenantScope,
         vertical_id: z.string().trim().min(1),
         name: z.string().trim().min(1),
         description: z.string().optional(),
-        tool_pack_id: z
-            .string()
-            .trim()
-            .min(1)
-            .describe("Tenant-scoped tool pack. Not part of personaContract."),
+        tool_pack_id: ignoredToolPackId,
         contract: contractInputSchema,
-    }, async ({ organization_id, vertical_id, name, description, tool_pack_id, contract }) => {
+    }, async ({ organization_id, vertical_id, name, description, contract }) => {
         try {
             return toolResult(await client.post("/api/persona", {
                 organizationId: organization_id,
                 verticalId: vertical_id,
                 name,
-                toolPackId: tool_pack_id,
                 ...(description !== undefined ? { description } : {}),
                 personaContract: parseWrittenContract(contract),
             }));
@@ -224,19 +254,14 @@ export function registerPersonaTools(server, injectedClient) {
             return handleToolError(error);
         }
     });
-    server.tool("replace_persona_contract", "Replace the complete canonical contract using optimistic concurrency. Omitting tool_pack_id reuses the persisted pack. Mutating an ACTIVE Persona returns it to DRAFT; call change_persona_status to reactivate.", {
+    server.tool("replace_persona_contract", "Replace the complete canonical contract using optimistic concurrency. Mutating an ACTIVE Persona returns it to DRAFT; call change_persona_status to reactivate.", {
         ...optimisticScope,
         id: z.string().min(1),
         name: z.string().trim().min(1).optional(),
         description: z.string().optional(),
-        tool_pack_id: z
-            .string()
-            .trim()
-            .min(1)
-            .optional()
-            .describe("Optional pack change. When omitted, the persisted toolPackId is reused."),
+        tool_pack_id: ignoredToolPackId,
         contract: contractInputSchema,
-    }, async ({ organization_id, id, expected_updated_at, name, description, tool_pack_id, contract }) => {
+    }, async ({ organization_id, id, expected_updated_at, name, description, contract }) => {
         try {
             return toolResult(await replaceContract({
                 organizationId: organization_id,
@@ -244,7 +269,6 @@ export function registerPersonaTools(server, injectedClient) {
                 expectedUpdatedAt: expected_updated_at,
                 name,
                 description,
-                toolPackId: tool_pack_id,
                 contract: parseWrittenContract(contract),
                 client,
             }));
@@ -258,13 +282,8 @@ export function registerPersonaTools(server, injectedClient) {
         id: z.string().min(1),
         path: z.string().trim().min(1),
         value: z.unknown(),
-        tool_pack_id: z
-            .string()
-            .trim()
-            .min(1)
-            .optional()
-            .describe("Optional pack change. When omitted, the persisted toolPackId is reused."),
-    }, async ({ organization_id, id, expected_updated_at, path, value, tool_pack_id }) => {
+        tool_pack_id: ignoredToolPackId,
+    }, async ({ organization_id, id, expected_updated_at, path, value }) => {
         try {
             const persona = await fetchPersona(id, organization_id, client);
             const contract = canonicalPersona(persona).contract;
@@ -272,7 +291,6 @@ export function registerPersonaTools(server, injectedClient) {
                 organizationId: organization_id,
                 id,
                 expectedUpdatedAt: expected_updated_at,
-                toolPackId: tool_pack_id,
                 contract: contractSchema.parse(setContractPath(contract, path, value)),
                 client,
             }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agifyai/leadify-mcp",
-  "version": "9.0.0",
+  "version": "9.1.1",
   "description": "MCP server for Leadify lead management API",
   "type": "module",
   "main": "dist/index.js",