@daghis/teamcity-mcp 1.3.4 → 1.4.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.4.0](https://github.com/Daghis/teamcity-mcp/compare/v1.3.5...v1.4.0) (2025-09-20)
+
+
+### Features
+
+* **tests:** add batched mcp tool execution ([#163](https://github.com/Daghis/teamcity-mcp/issues/163)) ([5f48060](https://github.com/Daghis/teamcity-mcp/commit/5f4806043b95686d5dac41a9d67515740b82a3f8)), closes [#162](https://github.com/Daghis/teamcity-mcp/issues/162)
+
+## [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,25 @@ 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;
+    const { data } = axiosResponse;
+    if (data instanceof ArrayBuffer) {
+      return data.slice(0);
+    }
+    if (Buffer.isBuffer(data)) {
+      return data.buffer.slice(data.byteOffset, data.byteOffset + data.byteLength);
+    }
+    throw new Error("Artifact download returned unexpected binary payload type");
   }
   /**
    * Parse TeamCity date format
@@ -1693,12 +1702,24 @@ var BuildResultsManager = class _BuildResultsManager {
 // src/teamcity/client-adapter.ts
 var import_axios = __toESM(require("axios"));
 var FALLBACK_BASE_URL = "http://not-configured";
+var toRecord = (value) => {
+  if (typeof value === "object" && value !== null) {
+    return value;
+  }
+  return {};
+};
+var createBuildApiBridge = (api) => ({
+  getAllBuilds: (locator, fields, options) => api.getAllBuilds(locator, fields, options),
+  getBuild: (buildLocator, fields, options) => api.getBuild(buildLocator, fields, options),
+  getMultipleBuilds: (locator, fields, options) => api.getMultipleBuilds(locator, fields, options),
+  getBuildProblems: (buildLocator, fields, options) => api.getBuildProblems(buildLocator, fields, options)
+});
 var resolveModules = (api) => {
   const candidate = api.modules;
   if (candidate != null) {
     return candidate;
   }
-  const legacy = api;
+  const legacy = toRecord(api);
   const pick = (key) => legacy[key] ?? {};
   const fallback = {
     agents: pick("agents"),
@@ -1760,7 +1781,7 @@ function createAdapterFromTeamCityAPI(api, options = {}) {
     });
   }
   const request = async (fn) => fn({ axios: httpInstance, baseUrl: resolvedApiConfig.baseUrl, requestId: void 0 });
-  const buildApi = modules.builds;
+  const buildApi = createBuildApiBridge(modules.builds);
   return {
     modules,
     http: httpInstance,
@@ -2481,6 +2502,12 @@ var esm_default = axiosRetry;
 // src/teamcity/auth.ts
 var import_crypto = require("crypto");
 init_errors();
+var asTimingMetaContainer = (value) => {
+  if (typeof value === "object" && value !== null) {
+    return value;
+  }
+  return null;
+};
 function generateRequestId() {
   return (0, import_crypto.randomUUID)();
 }
@@ -2489,7 +2516,10 @@ function addRequestId(config2) {
   config2.headers["X-Request-ID"] = requestId;
   const configWithId = config2;
   configWithId.requestId = requestId;
-  config2._tcMeta = { start: Date.now() };
+  const metaContainer = asTimingMetaContainer(config2);
+  if (metaContainer) {
+    metaContainer._tcMeta = { start: Date.now() };
+  }
   info("Starting TeamCity API request", {
     requestId,
     method: config2.method?.toUpperCase(),
@@ -2503,7 +2533,7 @@ function addRequestId(config2) {
 }
 function logResponse(response) {
   const requestId = response.config?.requestId;
-  const meta = response.config._tcMeta;
+  const meta = asTimingMetaContainer(response.config)?._tcMeta;
   const headers = response.headers;
   const headerDuration = headers?.["x-response-time"] ?? headers?.["x-response-duration"];
   const duration = headerDuration ?? (meta?.start ? Date.now() - meta.start : void 0);
@@ -2519,7 +2549,7 @@ function logResponse(response) {
 function logAndTransformError(error2) {
   const requestId = error2.config?.requestId;
   const tcError = TeamCityAPIError.fromAxiosError(error2, requestId);
-  const meta = error2.config?._tcMeta;
+  const meta = asTimingMetaContainer(error2.config)?._tcMeta;
   const duration = meta?.start ? Date.now() - meta.start : void 0;
   const sanitize = (val) => {
     const redact = (s) => s.replace(/(token[=:\s]*)[^\s&]+/gi, "$1***").replace(/(password[=:\s]*)[^\s&]+/gi, "$1***").replace(/(apikey[=:\s]*)[^\s&]+/gi, "$1***").replace(/(authorization[=:\s:]*)[^\s&]+/gi, "$1***");
@@ -36242,7 +36272,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 +36363,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 +36381,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 +36420,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 +39239,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",