@daghis/teamcity-mcp 1.9.0 → 1.9.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.9.1](https://github.com/Daghis/teamcity-mcp/compare/v1.9.0...v1.9.1) (2025-09-20)
+
+
+### Bug Fixes
+
+* **tools:** allow manage_build_steps updates ([#182](https://github.com/Daghis/teamcity-mcp/issues/182)) ([2c6bea0](https://github.com/Daghis/teamcity-mcp/commit/2c6bea0cf6770d22068b9e4b733542a056399148))
+
 ## [1.9.0](https://github.com/Daghis/teamcity-mcp/compare/v1.8.2...v1.9.0) (2025-09-20)
 
 

package/dist/index.js

@@ -40400,103 +40400,122 @@ var FULL_MODE_TOOLS = [
       required: ["buildTypeId", "action"]
     },
     handler: async (args) => {
-      const typedArgs = args;
-      const adapter = createAdapterFromTeamCityAPI(TeamCityAPI.getInstance());
-      switch (typedArgs.action) {
-        case "add": {
-          const stepProps = Object.fromEntries(
-            Object.entries(typedArgs.properties ?? {}).map(([k, v]) => [k, String(v)])
-          );
-          if (typedArgs.type === "simpleRunner" && stepProps["script.content"]) {
-            stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
-          }
-          const step = {
-            name: typedArgs.name,
-            type: typedArgs.type,
-            properties: {
-              property: Object.entries(stepProps).map(([k, v]) => ({ name: k, value: v }))
-            }
-          };
-          await adapter.modules.buildTypes.addBuildStepToBuildType(
-            typedArgs.buildTypeId,
-            void 0,
-            step,
-            {
-              headers: { "Content-Type": "application/json", Accept: "application/json" }
-            }
-          );
-          return json({
-            success: true,
-            action: "add_build_step",
-            buildTypeId: typedArgs.buildTypeId
-          });
-        }
-        case "update": {
-          if (typedArgs.stepId == null || typedArgs.stepId === "") {
-            return json({
-              success: false,
-              action: "update_build_step",
-              error: "Step ID is required for update action"
-            });
-          }
-          const updatePayload = {};
-          if (typedArgs.name != null) {
-            updatePayload["name"] = typedArgs.name;
-          }
-          if (typedArgs.type != null) {
-            updatePayload["type"] = typedArgs.type;
-          }
-          const rawProps = typedArgs.properties ?? {};
-          const stepProps = Object.fromEntries(
-            Object.entries(rawProps).map(([k, v]) => [k, String(v)])
-          );
-          if (stepProps["script.content"]) {
-            stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
-            stepProps["script.type"] = stepProps["script.type"] ?? "customScript";
-          }
-          if (Object.keys(stepProps).length > 0) {
-            updatePayload["properties"] = {
-              property: Object.entries(stepProps).map(([name, value]) => ({ name, value }))
-            };
-          }
-          if (Object.keys(updatePayload).length === 0) {
-            return json({
-              success: false,
-              action: "update_build_step",
-              error: "No update fields provided"
+      const schema = import_zod4.z.object({
+        buildTypeId: import_zod4.z.string().min(1, "buildTypeId is required"),
+        action: import_zod4.z.enum(["add", "update", "delete"]),
+        stepId: import_zod4.z.string().min(1).optional(),
+        name: import_zod4.z.string().optional(),
+        type: import_zod4.z.string().optional(),
+        properties: import_zod4.z.record(import_zod4.z.unknown()).optional()
+      }).superRefine((value, ctx) => {
+        if (value.action === "update" || value.action === "delete") {
+          if (!value.stepId || value.stepId.trim() === "") {
+            ctx.addIssue({
+              code: import_zod4.z.ZodIssueCode.custom,
+              message: "stepId is required for update or delete actions",
+              path: ["stepId"]
             });
           }
-          await adapter.modules.buildTypes.replaceBuildStep(
-            typedArgs.buildTypeId,
-            typedArgs.stepId,
-            void 0,
-            updatePayload
-          );
-          return json({
-            success: true,
-            action: "update_build_step",
-            buildTypeId: typedArgs.buildTypeId,
-            stepId: typedArgs.stepId
-          });
         }
-        case "delete":
-          if (typedArgs.stepId == null || typedArgs.stepId === "") {
-            return json({
-              success: false,
-              action: "delete_build_step",
-              error: "Step ID is required for delete action"
-            });
+      });
+      return runTool(
+        "manage_build_steps",
+        schema,
+        async (typedArgs) => {
+          const adapter = createAdapterFromTeamCityAPI(TeamCityAPI.getInstance());
+          switch (typedArgs.action) {
+            case "add": {
+              const stepProps = Object.fromEntries(
+                Object.entries(typedArgs.properties ?? {}).map(([k, v]) => [k, String(v)])
+              );
+              if (typedArgs.type === "simpleRunner" && stepProps["script.content"]) {
+                stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
+              }
+              const step = {
+                name: typedArgs.name,
+                type: typedArgs.type,
+                properties: {
+                  property: Object.entries(stepProps).map(([k, v]) => ({ name: k, value: v }))
+                }
+              };
+              await adapter.modules.buildTypes.addBuildStepToBuildType(
+                typedArgs.buildTypeId,
+                void 0,
+                step,
+                {
+                  headers: { "Content-Type": "application/json", Accept: "application/json" }
+                }
+              );
+              return json({
+                success: true,
+                action: "add_build_step",
+                buildTypeId: typedArgs.buildTypeId
+              });
+            }
+            case "update": {
+              const updatePayload = {};
+              if (typedArgs.name != null) {
+                updatePayload["name"] = typedArgs.name;
+              }
+              if (typedArgs.type != null) {
+                updatePayload["type"] = typedArgs.type;
+              }
+              const rawProps = typedArgs.properties ?? {};
+              const stepProps = Object.fromEntries(
+                Object.entries(rawProps).map(([k, v]) => [k, String(v)])
+              );
+              if (stepProps["script.content"]) {
+                stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
+                stepProps["script.type"] = stepProps["script.type"] ?? "customScript";
+              }
+              if (Object.keys(stepProps).length > 0) {
+                updatePayload["properties"] = {
+                  property: Object.entries(stepProps).map(([name, value]) => ({ name, value }))
+                };
+              }
+              if (Object.keys(updatePayload).length === 0) {
+                return json({
+                  success: false,
+                  action: "update_build_step",
+                  error: "No update fields provided"
+                });
+              }
+              await adapter.modules.buildTypes.replaceBuildStep(
+                typedArgs.buildTypeId,
+                typedArgs.stepId,
+                void 0,
+                updatePayload,
+                {
+                  headers: {
+                    "Content-Type": "application/json",
+                    Accept: "application/json"
+                  }
+                }
+              );
+              return json({
+                success: true,
+                action: "update_build_step",
+                buildTypeId: typedArgs.buildTypeId,
+                stepId: typedArgs.stepId
+              });
+            }
+            case "delete":
+              await adapter.modules.buildTypes.deleteBuildStep(
+                typedArgs.buildTypeId,
+                typedArgs.stepId
+              );
+              return json({
+                success: true,
+                action: "delete_build_step",
+                buildTypeId: typedArgs.buildTypeId,
+                stepId: typedArgs.stepId
+              });
+            default:
+              return json({ success: false, error: "Invalid action" });
           }
-          await adapter.modules.buildTypes.deleteBuildStep(typedArgs.buildTypeId, typedArgs.stepId);
-          return json({
-            success: true,
-            action: "delete_build_step",
-            buildTypeId: typedArgs.buildTypeId,
-            stepId: typedArgs.stepId
-          });
-        default:
-          return json({ success: false, error: "Invalid action" });
-      }
+        },
+        args
+      );
     },
     mode: "full"
   },