@loopops/mcp-server 3.4.0 → 3.5.0

package/dist/tools/tal.js

@@ -15,25 +15,6 @@
 import { z } from "zod";
 import { trpcMutation, trpcQuery } from "../api-client.js";
 import { safeTool } from "./_helpers.js";
-// Local replicas of the criteria DSL types — kept narrow so the MCP
-// tool description doesn't have to ship the full zod schema. The
-// authoritative validator lives in mcp-schemas.ts; this is just for
-// tool-input typing on the client side.
-const conditionSchema = z.lazy(() => z.union([
-    z.object({
-        operator: z.enum(["AND", "OR", "NOT"]),
-        conditions: z.array(conditionSchema),
-    }),
-    z.object({
-        type: z.enum([
-            "attribute",
-            "score",
-            "lifecycle_state",
-            "member_of",
-            "exclusion",
-        ]),
-    }).passthrough(),
-]));
 const talIdentifierShape = {
     talName: z
         .string()
@@ -81,63 +62,36 @@ export function registerTalTools(server, allowed) {
                 .describe("If true, classify changes but skip writes. Useful for previewing the impact of a criteria edit. Default: false."),
         }, safeTool(async (input) => trpcMutation("mcp.evaluateTalNow", input)));
     }
-    if (allowed.has("create_tal")) {
-        server.tool("create_tal", [
-            "Create a new Target Account List with a name, purpose, criteria DSL, and refresh cadence.",
-            "The TAL starts active but empty — call evaluate_tal_now to populate it.",
+    if (allowed.has("sync_account_master_tals")) {
+        server.tool("sync_account_master_tals", [
+            "Apply config/account_master/tals.yaml to the DB. Reads the YAML from GitHub, validates,",
+            "upserts target_account_list rows by tal_name, and (by default) re-evaluates each affected",
+            "TAL so membership reflects criteria changes immediately.",
             "",
-            "Criteria is a tree of conditions evaluated against an account's resolved attributes,",
-            "score, lifecycle states, and existing TAL memberships. Operators: AND, OR, NOT. Condition",
-            "types: attribute (with operators =, !=, >, >=, <, <=, in, not_in, exists, matches), score",
-            "(model + band), lifecycle_state (lifecycle_type + state), member_of (tal_name), exclusion",
-            "(tal_name — sugar for NOT member_of).",
+            "This is the WRITE path for TAL definitions. To create or edit a TAL:",
+            "  1. update_account_master_config file=tals  (commits new YAML to main)",
+            "  2. sync_account_master_tals                (apply YAML → DB; this tool)",
+            "  3. evaluate_tal_now (optional)             (run sooner than the cron's next fire)",
             "",
-            "Example criteria for 'enterprise US accounts that are enriched and not in the do-not-contact list':",
-            "{",
-            '  "operator": "AND",',
-            '  "conditions": [',
-            '    {"type": "attribute", "attribute": "employee_count", "operator": ">=", "value": 500},',
-            '    {"type": "attribute", "attribute": "hq_country", "operator": "=", "value": "US"},',
-            '    {"type": "lifecycle_state", "lifecycle_type": "enrichment", "state": "enriched"},',
-            '    {"type": "exclusion", "tal_name": "do_not_contact"}',
-            "  ]",
-            "}",
+            "TALs in DB but absent from YAML are LEFT UNTOUCHED — to remove a TAL, set is_active: false",
+            "in YAML. Permanent deletion requires explicit DB action so an accidental YAML edit",
+            "doesn't orphan target_account_list_membership rows.",
+            "",
+            "Idempotent. Safe to call after every update_account_master_config. Pass dryRun: true to",
+            "preview before applying.",
         ].join("\n"), {
-            talName: z
-                .string()
-                .min(1)
-                .max(200)
-                .regex(/^[a-z0-9_]+$/, "tal_name must be snake_case")
-                .describe("Snake-case identifier (e.g. 'enterprise_expansion_targets'). Must be unique."),
-            description: z
-                .string()
-                .min(1)
-                .describe("One-sentence description of what this list is for and who consumes it."),
-            purpose: z
-                .enum([
-                "signal_scanner",
-                "abm_advertising",
-                "outbound_sequence",
-                "csm_book",
-                "event_invite",
-                "exclusion",
-            ])
-                .describe("What the TAL is for. Drives which consumers read it."),
-            consumerSystem: z
+            branch: z
                 .string()
                 .optional()
-                .describe("Free-text identifier for the system that reads this TAL (e.g. 'signal_scanner', 'gong_engage'). Defaults to the purpose string."),
-            criteria: conditionSchema.describe("DSL tree of membership conditions. See tool description for grammar + example."),
-            refreshCadence: z
-                .enum(["frozen", "weekly", "daily", "hourly"])
+                .describe("Branch to read YAML from. Default: main. Use a feature branch if you committed via update_account_master_config to a feature branch and want to test before merging."),
+            evaluate: z
+                .boolean()
                 .optional()
-                .describe("How often the evaluator should re-compute membership. 'frozen' = never recomputed. Default: weekly."),
-            maxSize: z
-                .number()
-                .int()
-                .positive()
+                .describe("After applying TAL changes, re-evaluate every affected TAL (criteria/is_active/max_size diffs) so membership picks up the new definition immediately. Default: true."),
+            dryRun: z
+                .boolean()
                 .optional()
-                .describe("Hard cap on TAL size. Matches above the cap are deterministically truncated. Omit for no cap."),
-        }, safeTool(async (input) => trpcMutation("mcp.createTal", input)));
+                .describe("Read + validate + diff against DB without writing. Useful for previewing a TAL edit before applying."),
+        }, safeTool(async (input) => trpcMutation("mcp.syncAccountMasterTals", input)));
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loopops/mcp-server",
-  "version": "3.4.0",
+  "version": "3.5.0",
   "description": "Loop Operations MCP Server — AI skills for RevOps",
   "license": "UNLICENSED",
   "type": "module",