create_blast_campaign should not schedule (#35)

Author: gmethvin

TOOLS.md

@@ -11,7 +11,7 @@
 - **activate_triggered_campaign** ✏️✉️: Activate a triggered campaign (requires API triggered campaign activation enabled)
 - **archive_campaigns** ✏️: Archive one or more campaigns. Scheduled/recurring campaigns will be cancelled, running campaigns will be aborted.
 - **cancel_campaign** ✏️: Cancel a scheduled or recurring campaign
-- **create_blast_campaign** ✏️✉️: Create a new blast campaign from an existing template. By default, the campaign is created without being scheduled. Set scheduleSend to true to immediately schedule it for delivery at the given sendAt time.
+- **create_blast_campaign** ✏️: Create a new blast campaign from an existing template. The campaign is created without being scheduled. Use schedule_campaign to schedule it for delivery.
 - **create_triggered_campaign** ✏️: Create a new triggered campaign from an existing template. The campaign is created in Ready state and must be activated before it can send.
 - **deactivate_triggered_campaign** ✏️: Deactivate a triggered campaign (requires API triggered campaign deactivation enabled)
 - **get_campaign**: Get detailed information about a specific campaign

package.json

@@ -76,7 +76,7 @@
   },
   "dependencies": {
     "@alcyone-labs/zod-to-json-schema": "4.0.10",
-    "@iterable/api": "0.8.1",
+    "@iterable/api": "0.8.2",
     "@modelcontextprotocol/sdk": "1.18.1",
     "@primno/dpapi": "2.0.1",
     "@types/json-schema": "7.0.15",

pnpm-lock.yaml

@@ -124,8 +124,6 @@ export const SEND_TOOLS: Set<string> = new Set([
   "send_campaign",
   "trigger_campaign",
   "schedule_campaign",
-  // Creating a blast campaign schedules a send
-  "create_blast_campaign",
   // Triggered campaigns can cause sends upon activation; block unless explicitly allowed
   "activate_triggered_campaign",
   // Journey triggers enqueue users which may send

src/tool-filter.ts

@@ -48,7 +48,7 @@ export function createCampaignTools(client: IterableClient): Tool[] {
     createTool({
       name: "create_blast_campaign",
       description:
-        "Create a new blast campaign from an existing template. By default, the campaign is created without being scheduled. Set scheduleSend to true to immediately schedule it for delivery at the given sendAt time.",
+        "Create a new blast campaign from an existing template. The campaign is created without being scheduled. Use schedule_campaign to schedule it for delivery.",
       schema: CreateBlastCampaignParamsSchema,
       execute: (params) => client.createBlastCampaign(params),
     }),

src/tools/campaigns.ts

@@ -28,7 +28,6 @@ describe("SEND_TOOLS registry", () => {
       "send_campaign",
       "trigger_campaign",
       "schedule_campaign",
-      "create_blast_campaign",
       "activate_triggered_campaign",
       "trigger_journey",
       "track_event",

tests/unit/send-tools-registry.test.ts

@@ -16,7 +16,6 @@ describe("filterTools with allowSends", () => {
       mkTool("send_campaign"),
       mkTool("trigger_campaign"),
       mkTool("schedule_campaign"),
-      mkTool("create_blast_campaign"),
       mkTool("track_event"),
       mkTool("track_bulk_events"),
       mkTool("trigger_journey"),
@@ -47,7 +46,6 @@ describe("filterTools with allowSends", () => {
     expect(names.has("send_campaign")).toBe(false);
     expect(names.has("trigger_campaign")).toBe(false);
     expect(names.has("schedule_campaign")).toBe(false);
-    expect(names.has("create_blast_campaign")).toBe(false);
     expect(names.has("track_event")).toBe(false);
     expect(names.has("track_bulk_events")).toBe(false);
     expect(names.has("trigger_journey")).toBe(false);