@daeda/mcp-pro 0.1.0 → 0.1.1

package/dist/index.js

@@ -416,7 +416,6 @@ var saveErrorCsv = (portalId, objectType, csvContent) => pipe2(
 );
 
 // src/plugins/crm-data.ts
-var PROPERTY_SAMPLE_SIZE = 5;
 var ASSOCIATIONS_TABLE_STATEMENTS = [
   `CREATE TABLE IF NOT EXISTS associations (
     from_object_type TEXT NOT NULL,
@@ -649,23 +648,18 @@ var crmDataPlugin = {
         const count = Number(
           countReader.getRowObjects()[0].count
         );
-        let propertyKeys = [];
+        let propertyKeyCount = 0;
         if (count > 0) {
           const sampleReader = await conn.runAndReadAll(
-            `SELECT properties FROM "${tableName}" LIMIT ${PROPERTY_SAMPLE_SIZE}`
+            `SELECT properties FROM "${tableName}" LIMIT 1`
           );
           const sampleRows = sampleReader.getRowObjects();
-          const keySet = /* @__PURE__ */ new Set();
-          for (const row of sampleRows) {
+          if (sampleRows[0]) {
             try {
-              const props = JSON.parse(row.properties);
-              for (const key of Object.keys(props)) {
-                keySet.add(key);
-              }
+              propertyKeyCount = Object.keys(JSON.parse(sampleRows[0].properties)).length;
             } catch {
             }
           }
-          propertyKeys = Array.from(keySet).sort();
         }
         let lastSynced = null;
         try {
@@ -680,7 +674,7 @@ var crmDataPlugin = {
         return {
           objectType: tableName,
           recordCount: count,
-          propertyKeys,
+          propertyKeyCount,
           lastSynced
         };
       })
@@ -701,11 +695,6 @@ var crmDataPlugin = {
             to: group.to_object_type,
             count: Number(group.count)
           }))
-        },
-        queryHints: {
-          accessProperty: "json_extract_string(properties, '$.propertyName')",
-          exampleQuery: "SELECT id, json_extract_string(properties, '$.firstname') as first_name FROM contacts LIMIT 10",
-          allProperties: "SELECT id, properties FROM <table> LIMIT 1"
         }
       }
     };
@@ -3182,7 +3171,7 @@ Use "all" to get everything at once.`,
         result.schema = await buildSchemaSection(deps.getSelectedPortalId, deps.getEncryptionKey);
       }
       return {
-        content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
+        content: [{ type: "text", text: JSON.stringify(result) }]
       };
     } catch (e) {
       const message = e instanceof Error ? e.message : String(e);
@@ -5326,6 +5315,18 @@ var updatePropertyHandler = {
             message: `Property '${op.property_name}' groupName: expected '${op.updates.group_name}', got '${updated.groupName}'`
           });
         }
+        if (op.updates.options !== void 0) {
+          const expectedValues = new Set(op.updates.options.map((o) => o.value));
+          const actualValues = new Set(updated.options.map((o) => o.value));
+          if (expectedValues.size !== actualValues.size || [...expectedValues].some((v) => !actualValues.has(v))) {
+            issues.push({
+              severity: "warning",
+              operation_index: index,
+              code: "POSTCHECK_OPTIONS_MISMATCH",
+              message: `Property '${op.property_name}' options: expected ${expectedValues.size} options, got ${actualValues.size}`
+            });
+          }
+        }
         return issues;
       }),
       catchAllToWarning(
@@ -5626,6 +5627,14 @@ var updatePropertyGroupHandler = {
             message: `Property group '${op.group_name}' label: expected '${op.updates.label}', got '${updated.label}'`
           });
         }
+        if (op.updates.display_order !== void 0 && updated.displayOrder !== op.updates.display_order) {
+          issues.push({
+            severity: "warning",
+            operation_index: index,
+            code: "POSTCHECK_DISPLAY_ORDER_MISMATCH",
+            message: `Property group '${op.group_name}' displayOrder: expected ${op.updates.display_order}, got ${updated.displayOrder}`
+          });
+        }
         return issues;
       }),
       catchAllToWarning(
@@ -6080,7 +6089,8 @@ var updatePipelineHandler = {
               label: op.updates.label,
               displayOrder: op.updates.display_order
             }),
-            Effect44.catchAll(() => Effect44.succeed(null))
+            Effect44.map(() => ({ label: "update pipeline metadata", ok: true })),
+            Effect44.catchAll(() => Effect44.succeed({ label: "update pipeline metadata", ok: false }))
           )
         );
       }
@@ -6097,7 +6107,8 @@ var updatePipelineHandler = {
                 }
               }
             }),
-            Effect44.catchAll(() => Effect44.succeed(null))
+            Effect44.map(() => ({ label: `add stage '${stage.label}'`, ok: true })),
+            Effect44.catchAll(() => Effect44.succeed({ label: `add stage '${stage.label}'`, ok: false }))
           )
         );
       }
@@ -6114,7 +6125,8 @@ var updatePipelineHandler = {
                 }
               }
             }),
-            Effect44.catchAll(() => Effect44.succeed(null))
+            Effect44.map(() => ({ label: `update stage '${stageUpdate.stage_id}'`, ok: true })),
+            Effect44.catchAll(() => Effect44.succeed({ label: `update stage '${stageUpdate.stage_id}'`, ok: false }))
           )
         );
       }
@@ -6122,19 +6134,42 @@ var updatePipelineHandler = {
         steps.push(
           pipe33(
             hs.deletePipelineStage(op.object_type, op.pipeline_id, stageId),
-            Effect44.catchAll(() => Effect44.succeed(null))
+            Effect44.map(() => ({ label: `remove stage '${stageId}'`, ok: true })),
+            Effect44.catchAll(() => Effect44.succeed({ label: `remove stage '${stageId}'`, ok: false }))
           )
         );
       }
       return pipe33(
         Effect44.forEach(steps, (step) => step, { concurrency: 1 }),
-        Effect44.map(() => ({
-          index,
-          type: "update_pipeline",
-          status: "success",
-          records_affected: 1,
-          affected_ids: [op.pipeline_id]
-        }))
+        Effect44.map((results) => {
+          const failed = results.filter((r) => !r.ok);
+          if (failed.length === results.length) {
+            return {
+              index,
+              type: "update_pipeline",
+              status: "failed",
+              records_affected: 0,
+              error: `All ${failed.length} sub-steps failed: ${failed.map((f) => f.label).join(", ")}`
+            };
+          }
+          if (failed.length > 0) {
+            return {
+              index,
+              type: "update_pipeline",
+              status: "failed",
+              records_affected: 1,
+              affected_ids: [op.pipeline_id],
+              error: `Partial failure: ${failed.length}/${results.length} sub-steps failed (${failed.map((f) => f.label).join(", ")})`
+            };
+          }
+          return {
+            index,
+            type: "update_pipeline",
+            status: "success",
+            records_affected: 1,
+            affected_ids: [op.pipeline_id]
+          };
+        })
       );
     }),
     catchAllToFailed("update_pipeline", index)
@@ -6174,6 +6209,18 @@ var updatePipelineHandler = {
             });
           }
         }
+        for (const stageUpdate of op.stages_to_update ?? []) {
+          const stage = pipeline.stages.find((s) => s.id === stageUpdate.stage_id);
+          if (!stage) continue;
+          if (stageUpdate.label !== void 0 && stage.label !== stageUpdate.label) {
+            issues.push({
+              severity: "warning",
+              operation_index: index,
+              code: "POSTCHECK_STAGE_LABEL_MISMATCH",
+              message: `Stage '${stageUpdate.stage_id}' label: expected '${stageUpdate.label}', got '${stage.label}'`
+            });
+          }
+        }
         const stageIds = new Set(pipeline.stages.map((s) => s.id));
         for (const removedId of op.stage_ids_to_remove ?? []) {
           if (stageIds.has(removedId)) {
@@ -6685,13 +6732,51 @@ var batchUpdateAssociationsHandler = {
     }
     return [];
   },
-  validateEffectful: () => Effect49.succeed([]),
+  validateEffectful: (op, index) => pipe38(
+    HubSpotService,
+    Effect49.flatMap((hs) => hs.getAssociationLabels(op.from_object_type, op.to_object_type)),
+    Effect49.map((labels) => {
+      const issues = [];
+      const match = labels.find((l) => l.typeId === op.new_association_type_id);
+      if (!match) {
+        issues.push({
+          severity: "error",
+          operation_index: index,
+          code: "ASSOCIATION_TYPE_NOT_FOUND",
+          message: `Association type ID ${op.new_association_type_id} does not exist between ${op.from_object_type} and ${op.to_object_type}`
+        });
+      }
+      return issues;
+    }),
+    catchAllToWarning(
+      "VALIDATION_API_UNAVAILABLE",
+      index,
+      `Unable to verify association labels between ${op.from_object_type} and ${op.to_object_type} (API error)`
+    )
+  ),
   preCheck: () => Effect49.succeed([]),
   execute: (op, index) => pipe38(
     HubSpotService,
-    Effect49.flatMap(
-      (hs) => pipe38(
-        hs.batchDeleteAssociations({
+    Effect49.flatMap((hs) => {
+      const deleteInputs = {
+        fromObjectType: op.from_object_type,
+        toObjectType: op.to_object_type,
+        inputs: op.associations.map((a) => ({
+          fromId: a.from_id,
+          toId: a.to_id
+        }))
+      };
+      const createWithType = (typeId) => hs.batchCreateAssociations({
+        fromObjectType: op.from_object_type,
+        toObjectType: op.to_object_type,
+        inputs: op.associations.map((a) => ({
+          fromId: a.from_id,
+          toId: a.to_id,
+          associationTypeId: typeId
+        }))
+      });
+      const recreateUnlabeled = pipe38(
+        hs.batchCreateAssociations({
           fromObjectType: op.from_object_type,
           toObjectType: op.to_object_type,
           inputs: op.associations.map((a) => ({
@@ -6699,19 +6784,23 @@ var batchUpdateAssociationsHandler = {
             toId: a.to_id
           }))
         }),
+        Effect49.catchAll(() => Effect49.succeed(void 0))
+      );
+      return pipe38(
+        hs.batchDeleteAssociations(deleteInputs),
         Effect49.flatMap(
-          () => hs.batchCreateAssociations({
-            fromObjectType: op.from_object_type,
-            toObjectType: op.to_object_type,
-            inputs: op.associations.map((a) => ({
-              fromId: a.from_id,
-              toId: a.to_id,
-              associationTypeId: op.new_association_type_id
-            }))
-          })
+          () => pipe38(
+            createWithType(op.new_association_type_id),
+            Effect49.catchAll(
+              (createError) => pipe38(
+                recreateUnlabeled,
+                Effect49.flatMap(() => Effect49.fail(createError))
+              )
+            )
+          )
         )
-      )
-    ),
+      );
+    }),
     Effect49.map(() => ({
       index,
       type: "batch_update_associations",
@@ -7283,20 +7372,33 @@ var updateCustomObjectHandler = {
   execute: (op, index) => pipe45(
     HubSpotService,
     Effect56.flatMap((hs) => {
-      const updates = {};
-      if (op.new_singular_label !== void 0 || op.new_plural_label !== void 0) {
-        updates.labels = {
-          singular: op.new_singular_label ?? "",
-          plural: op.new_plural_label ?? ""
-        };
-      }
-      if (op.new_primary_display_property !== void 0) {
-        updates.primaryDisplayProperty = op.new_primary_display_property;
-      }
-      if (op.new_required_properties !== void 0) {
-        updates.requiredProperties = [...op.new_required_properties];
-      }
-      return hs.updateCustomObject(op.object_type, updates);
+      const needsLabelMerge = (op.new_singular_label !== void 0 || op.new_plural_label !== void 0) && !(op.new_singular_label !== void 0 && op.new_plural_label !== void 0);
+      const getExistingLabels = needsLabelMerge ? pipe45(
+        hs.getCustomObjectSchemas,
+        Effect56.map((schemas) => {
+          const schema = schemas.find((s) => s.name === op.object_type || s.objectTypeId === op.object_type);
+          return schema?.labels ?? { singular: "", plural: "" };
+        })
+      ) : Effect56.succeed({ singular: "", plural: "" });
+      return pipe45(
+        getExistingLabels,
+        Effect56.flatMap((existingLabels) => {
+          const updates = {};
+          if (op.new_singular_label !== void 0 || op.new_plural_label !== void 0) {
+            updates.labels = {
+              singular: op.new_singular_label ?? existingLabels.singular,
+              plural: op.new_plural_label ?? existingLabels.plural
+            };
+          }
+          if (op.new_primary_display_property !== void 0) {
+            updates.primaryDisplayProperty = op.new_primary_display_property;
+          }
+          if (op.new_required_properties !== void 0) {
+            updates.requiredProperties = [...op.new_required_properties];
+          }
+          return hs.updateCustomObject(op.object_type, updates);
+        })
+      );
     }),
     Effect56.map((result) => ({
       index,
@@ -7808,7 +7910,27 @@ var batchAssignOwnerHandler = {
     }
     return issues;
   },
-  validateEffectful: () => Effect64.succeed([]),
+  validateEffectful: (op, index) => pipe53(
+    HubSpotService,
+    Effect64.flatMap((hs) => hs.getOwners),
+    Effect64.map((owners) => {
+      const issues = [];
+      if (!owners.find((o) => o.id === op.owner_id)) {
+        issues.push({
+          severity: "error",
+          operation_index: index,
+          code: "OWNER_NOT_FOUND",
+          message: `Owner ID '${op.owner_id}' does not exist in this portal \u2014 use the query tool to find valid owner IDs`
+        });
+      }
+      return issues;
+    }),
+    catchAllToWarning(
+      "VALIDATION_API_UNAVAILABLE",
+      index,
+      "Unable to verify owner ID during validation (API error)"
+    )
+  ),
   preCheck: () => Effect64.succeed([]),
   execute: (op, index) => pipe53(
     HubSpotService,
@@ -8100,7 +8222,15 @@ var formatPlanView = () => {
 };
 function registerBuildPlanTool(server2) {
   server2.registerTool("build_plan", {
-    description: `Create, view, and edit a write plan for modifying HubSpot CRM data. This single tool handles the entire plan-building workflow.
+    description: `Use this tool whenever the user wants to make changes to their HubSpot CRM configuration or data. This is the ONLY way to modify HubSpot \u2014 there is no direct API access. All CRM write operations must go through a plan.
+
+Supports 28 operation types across these categories:
+- Properties & groups: create, update, or delete custom properties and property groups
+- Pipelines & stages: create, update, or delete deal/ticket pipelines with stages
+- Records: batch create, update, delete, merge, or reassign owners for CRM records
+- Associations: create, update, or delete associations and labels between object types
+- Lists: create, update, delete lists, or add/remove members
+- Custom objects: create, update, or delete custom object schemas
 
 CREATING A PLAN:
   Provide title + description (+ optional operations) to start a new draft.
@@ -8122,7 +8252,7 @@ VIEWING THE PLAN:
 IMPORTANT: After building the plan, present it to the user and ask for confirmation before calling submit_plan.
 
 WORKFLOW:
-1. describe_operations - Discover available operation types
+1. describe_operations - Discover available operation types and their required fields
 2. build_plan - Create draft with title, description, and operations
 3. Present the plan to the user for review
 4. submit_plan (dry_run: true) - Validate without saving (optional)
@@ -8258,7 +8388,11 @@ ${formatPlanView()}`
 import { z as z43 } from "zod";
 function registerDescribeOperationsTool(server2) {
   server2.registerTool("describe_operations", {
-    description: `Discover available write operation types and their field schemas. Use this before build_plan to understand what fields each operation type requires.
+    description: `Discover what CRM modifications are available and what fields they require. Call this tool when the user asks about making changes to HubSpot, or before using build_plan to check operation schemas.
+
+Use cases:
+- User asks "can I delete custom properties?" or "what can you do with pipelines?" \u2014 call with no arguments to list all 28 operation types.
+- User wants to build a plan \u2014 call with specific types to see their full field schemas and examples (e.g. ['delete_property', 'create_pipeline']).
 
 - Call with no arguments to list all available operation types.
 - Call with specific types to see their full field schemas and examples.`,

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@daeda/mcp-pro",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "MCP server for HubSpot CRM — sync, query, and manage your portal data",
   "type": "module",
   "bin": {
-    "mcp-pro": "./dist/index.js"
+    "mcp-pro": "dist/index.js"
   },
   "main": "./dist/index.js",
   "exports": {
@@ -21,6 +21,9 @@
     "dev": "bun run --watch src/index.ts",
     "start": "bun run src/index.ts",
     "prepublishOnly": "bun run build",
+    "release:patch": "npm version patch && npm publish --access public",
+    "release:minor": "npm version minor && npm publish --access public",
+    "release:major": "npm version major && npm publish --access public",
     "db:stats": "bun run scripts/db-stats.ts",
     "db:query": "bun run scripts/db-query.ts",
     "db:schema": "bun run scripts/db-schema.ts",