@daghis/teamcity-mcp 1.3.4 → 1.3.5

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.3.5](https://github.com/Daghis/teamcity-mcp/compare/v1.3.4...v1.3.5) (2025-09-19)
+
+
+### Bug Fixes
+
+* **tools:** repair manage_build_steps updates ([#154](https://github.com/Daghis/teamcity-mcp/issues/154)) ([b557e44](https://github.com/Daghis/teamcity-mcp/commit/b557e4424d5129de9d3b6e3240e1a876488da040))
+
 ## [1.3.4](https://github.com/Daghis/teamcity-mcp/compare/v1.3.3...v1.3.4) (2025-09-19)
 
 

package/dist/index.js

@@ -1317,6 +1317,9 @@ var BuildConfigurationUpdateManager = class {
   }
 };
 
+// src/teamcity/utils/build-locator.ts
+var toBuildLocator = (buildId) => buildId.includes(":") ? buildId : `id:${buildId}`;
+
 // src/teamcity/build-results-manager.ts
 var BuildResultsManager = class _BuildResultsManager {
   client;
@@ -1399,7 +1402,7 @@ var BuildResultsManager = class _BuildResultsManager {
    */
   async fetchBuildSummary(buildId) {
     const response = await this.client.modules.builds.getBuild(
-      this.toBuildLocator(buildId),
+      toBuildLocator(buildId),
       _BuildResultsManager.fields
     );
     return response.data;
@@ -1455,7 +1458,7 @@ var BuildResultsManager = class _BuildResultsManager {
   async fetchArtifacts(buildId, options) {
     try {
       const response = await this.client.modules.builds.getFilesListOfBuild(
-        this.toBuildLocator(buildId)
+        toBuildLocator(buildId)
       );
       const artifactListing = response.data;
       let artifacts = artifactListing.file ?? [];
@@ -1507,7 +1510,7 @@ var BuildResultsManager = class _BuildResultsManager {
   async fetchStatistics(buildId) {
     try {
       const response = await this.client.modules.builds.getBuildStatisticValues(
-        this.toBuildLocator(buildId)
+        toBuildLocator(buildId)
       );
       const payload = response.data;
       const properties = payload.property ?? [];
@@ -1573,8 +1576,9 @@ var BuildResultsManager = class _BuildResultsManager {
    */
   async fetchDependencies(buildId) {
     try {
-      const response = await this.client.request(
-        (ctx) => ctx.axios.get(`${ctx.baseUrl}/app/rest/builds/id:${buildId}/snapshot-dependencies`)
+      const response = await this.client.modules.builds.getAllBuilds(
+        `snapshotDependency:(to:(id:${buildId}))`,
+        "build(id,number,buildTypeId,status)"
       );
       const depsData = response.data;
       const builds = depsData.build ?? [];
@@ -1606,20 +1610,18 @@ var BuildResultsManager = class _BuildResultsManager {
     const baseUrl = this.client.getApiConfig().baseUrl;
     return baseUrl.endsWith("/") ? baseUrl.slice(0, -1) : baseUrl;
   }
-  toBuildLocator(buildId) {
-    return buildId.includes(":") ? buildId : `id:${buildId}`;
-  }
   async downloadArtifactContent(buildId, artifactPath) {
     const normalizedPath = artifactPath.split("/").map((segment) => encodeURIComponent(segment)).join("/");
-    const response = await this.client.request(
-      (ctx) => ctx.axios.get(
-        `${ctx.baseUrl}/app/rest/builds/id:${buildId}/artifacts/content/${normalizedPath}`,
-        {
-          responseType: "arraybuffer"
-        }
-      )
+    const buildLocator = toBuildLocator(buildId);
+    const response = await this.client.modules.builds.downloadFileOfBuild(
+      `content/${normalizedPath}`,
+      buildLocator,
+      void 0,
+      void 0,
+      { responseType: "arraybuffer" }
     );
-    return response.data;
+    const axiosResponse = response;
+    return axiosResponse.data ?? new ArrayBuffer(0);
   }
   /**
    * Parse TeamCity date format
@@ -36242,7 +36244,7 @@ var TeamCityAPI = class _TeamCityAPI {
     return response.data;
   }
   async getBuild(buildId) {
-    const response = await this.builds.getBuild(this.toBuildLocator(buildId));
+    const response = await this.builds.getBuild(toBuildLocator(buildId));
     return response.data;
   }
   async triggerBuild(buildTypeId, branchName, comment) {
@@ -36333,7 +36335,7 @@ var TeamCityAPI = class _TeamCityAPI {
   }
   async listBuildArtifacts(buildId, options) {
     return this.builds.getFilesListOfBuild(
-      this.toBuildLocator(buildId),
+      toBuildLocator(buildId),
       options?.basePath,
       options?.locator,
       options?.fields,
@@ -36351,16 +36353,13 @@ var TeamCityAPI = class _TeamCityAPI {
     );
   }
   async getBuildStatistics(buildId, fields) {
-    return this.builds.getBuildStatisticValues(this.toBuildLocator(buildId), fields);
+    return this.builds.getBuildStatisticValues(toBuildLocator(buildId), fields);
   }
   async listChangesForBuild(buildId, fields) {
     return this.changes.getAllChanges(`build:(id:${buildId})`, fields);
   }
   async listSnapshotDependencies(buildId) {
-    const response = await this.builds.getBuild(
-      this.toBuildLocator(buildId),
-      "snapshot-dependencies"
-    );
+    const response = await this.builds.getBuild(toBuildLocator(buildId), "snapshot-dependencies");
     const dependencies = response.data["snapshot-dependencies"];
     if (dependencies == null) {
       return response;
@@ -36393,9 +36392,6 @@ var TeamCityAPI = class _TeamCityAPI {
     this.instance = void 0;
     this.instanceConfig = void 0;
   }
-  toBuildLocator(buildId) {
-    return buildId.includes(":") ? buildId : `id:${buildId}`;
-  }
   createApi(apiCtor) {
     return new apiCtor(this.config, this.baseUrl, this.axiosInstance);
   }
@@ -39215,16 +39211,39 @@ var FULL_MODE_TOOLS = [
               error: "Step ID is required for update action"
             });
           }
-          const props = Object.entries(typedArgs.properties ?? {});
-          for (const [k, v] of props) {
-            await adapter.modules.buildTypes.setBuildStepParameter(
-              typedArgs.buildTypeId,
-              typedArgs.stepId,
-              k,
-              String(v),
-              { headers: { "Content-Type": "text/plain", Accept: "application/json" } }
-            );
+          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
+          );
           return json({
             success: true,
             action: "update_build_step",