@daghis/teamcity-mcp 1.9.3 → 1.9.5

package/.release-please-config.json

@@ -0,0 +1,20 @@
+{
+  "packages": {
+    ".": {
+      "release-type": "node",
+      "release-as": "1.9.5",
+      "extra-files": [
+        {
+          "type": "json",
+          "path": "server.json",
+          "jsonpath": "$.version"
+        },
+        {
+          "type": "json",
+          "path": "server.json",
+          "jsonpath": "$.packages[0].version"
+        }
+      ]
+    }
+  }
+}

package/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "1.9.5"
+}

package/CHANGELOG.md

@@ -1,5 +1,59 @@
 # Changelog
 
+## [0.1.2](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.9.4...teamcity-mcp-v0.1.2) (2025-09-25)
+
+
+### Features
+
+* add Codecov bundle analysis ([#28](https://github.com/Daghis/teamcity-mcp/issues/28)) ([87c3de8](https://github.com/Daghis/teamcity-mcp/commit/87c3de85af34bec5b071d82612d67ba4d5a52702))
+* **publish:** automate mcp registry publishing ([#190](https://github.com/Daghis/teamcity-mcp/issues/190)) ([285b737](https://github.com/Daghis/teamcity-mcp/commit/285b737c5993332d8cf5e81e4db5be9864355701))
+* **teamcity:** add runtime guards for manager responses ([#179](https://github.com/Daghis/teamcity-mcp/issues/179)) ([9d7eaef](https://github.com/Daghis/teamcity-mcp/commit/9d7eaef161e675246d317603df47f7f39407b7d8))
+* **teamcity:** add streaming artifact downloads ([#161](https://github.com/Daghis/teamcity-mcp/issues/161)) ([b50b773](https://github.com/Daghis/teamcity-mcp/commit/b50b773ebee6f5cc85f4c53c98fef525e7098fe9)), closes [#151](https://github.com/Daghis/teamcity-mcp/issues/151)
+* **teamcity:** add streaming artifact downloads ([#166](https://github.com/Daghis/teamcity-mcp/issues/166)) ([18830ed](https://github.com/Daghis/teamcity-mcp/commit/18830ed4805ad215518cc5a51640094adc35e31b)), closes [#151](https://github.com/Daghis/teamcity-mcp/issues/151)
+* **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)
+* **tools:** add streaming mode to fetch_build_log ([#171](https://github.com/Daghis/teamcity-mcp/issues/171)) ([1abce69](https://github.com/Daghis/teamcity-mcp/commit/1abce69b7fa3866ac289501e2369d5d06d57d57f))
+* **tools:** add VCS root property management tools + integration test ([#95](https://github.com/Daghis/teamcity-mcp/issues/95)) ([13bbe17](https://github.com/Daghis/teamcity-mcp/commit/13bbe178564a53564aad4fcdb99bb3f2e5db4cb8))
+* **tools:** expose change and admin REST tools ([#112](https://github.com/Daghis/teamcity-mcp/issues/112)) ([e7f4e1e](https://github.com/Daghis/teamcity-mcp/commit/e7f4e1e74bb28572ffc3ee7a0fced6090c4f92ef))
+* **tools:** support streaming artifacts in get_build_results ([#173](https://github.com/Daghis/teamcity-mcp/issues/173)) ([f95c5e4](https://github.com/Daghis/teamcity-mcp/commit/f95c5e4da9ffa80adc8b8f322c127a440c4524b5)), closes [#169](https://github.com/Daghis/teamcity-mcp/issues/169)
+* **utils:** add TeamCity service message escaping ([#85](https://github.com/Daghis/teamcity-mcp/issues/85)) ([6ee3b72](https://github.com/Daghis/teamcity-mcp/commit/6ee3b72e4db065faa0e01c7046328e1d4e1375c8))
+
+
+### Bug Fixes
+
+* authorize_agent uses authorizedInfo JSON endpoint (TeamCity 2025.07 compatibility) ([#83](https://github.com/Daghis/teamcity-mcp/issues/83)) ([198fd02](https://github.com/Daghis/teamcity-mcp/commit/198fd02ca8b6e89d83f87a1a94aafeadda593504)), closes [#78](https://github.com/Daghis/teamcity-mcp/issues/78)
+* **cli:** improve npx execution; release 0.9.2 ([#61](https://github.com/Daghis/teamcity-mcp/issues/61)) ([42577c3](https://github.com/Daghis/teamcity-mcp/commit/42577c3ed78dd7e29f21f6c8138d96da49582892))
+* **cli:** remove duplicate shebang in entrypoint; bump to 0.9.1 ([#59](https://github.com/Daghis/teamcity-mcp/issues/59)) ([740530d](https://github.com/Daghis/teamcity-mcp/commit/740530d7bfba6bc81a42f8057fd017a2f8cae14e))
+* delete_parameter: correct endpoint + arg order ([#84](https://github.com/Daghis/teamcity-mcp/issues/84)) ([9832821](https://github.com/Daghis/teamcity-mcp/commit/9832821e65fd945ec5ff0da2b85f744f413e7782))
+* **health:** accept empty locator; normalize category; fallback on 400 ([#76](https://github.com/Daghis/teamcity-mcp/issues/76)) ([174c8e9](https://github.com/Daghis/teamcity-mcp/commit/174c8e932301b1b2262bfb704a73c4647520d3bf)), closes [#75](https://github.com/Daghis/teamcity-mcp/issues/75)
+* **mcp:** normalize TeamCity errors and wrap server info/metrics with runTool ([#53](https://github.com/Daghis/teamcity-mcp/issues/53)) ([2c8ab85](https://github.com/Daghis/teamcity-mcp/commit/2c8ab855a85e5faec4216a498c089e3a1a93ed7b))
+* **release:** restore 1.9.4 version metadata ([#205](https://github.com/Daghis/teamcity-mcp/issues/205)) ([c10c436](https://github.com/Daghis/teamcity-mcp/commit/c10c4362934f3af116e7d22537481aaceceeff40))
+* **teamcity:** expose unified adapter surface ([#129](https://github.com/Daghis/teamcity-mcp/issues/129)) ([b0b15e8](https://github.com/Daghis/teamcity-mcp/commit/b0b15e8597d5a79c064388be38c3ffe9c0e1fc09)), closes [#114](https://github.com/Daghis/teamcity-mcp/issues/114)
+* **teamcity:** resolve nested artifact downloads ([#188](https://github.com/Daghis/teamcity-mcp/issues/188)) ([e309b90](https://github.com/Daghis/teamcity-mcp/commit/e309b90de42fe121f072fe5b549ab25df3a91aaf))
+* **teamcity:** restore artifact rules update compatibility ([#185](https://github.com/Daghis/teamcity-mcp/issues/185)) ([1d67268](https://github.com/Daghis/teamcity-mcp/commit/1d67268e6ee837db38ab9be27a94a5f3e072ab83))
+* **teamcity:** retain adapter api credentials ([#138](https://github.com/Daghis/teamcity-mcp/issues/138)) ([3ad31ed](https://github.com/Daghis/teamcity-mcp/commit/3ad31edf227eac904eee84ec3a80d16455ae37fc)), closes [#135](https://github.com/Daghis/teamcity-mcp/issues/135)
+* **teamcity:** route results + problems via shared client ([#111](https://github.com/Daghis/teamcity-mcp/issues/111)) ([3797bb5](https://github.com/Daghis/teamcity-mcp/commit/3797bb57d8421e2cb243112a9b1cdfda92fff1f7))
+* **tools:** allow manage_build_steps updates ([#182](https://github.com/Daghis/teamcity-mcp/issues/182)) ([2c6bea0](https://github.com/Daghis/teamcity-mcp/commit/2c6bea0cf6770d22068b9e4b733542a056399148))
+* **tools:** ensure multi-ref branchSpec; remove alias logic ([#103](https://github.com/Daghis/teamcity-mcp/issues/103)) ([5012439](https://github.com/Daghis/teamcity-mcp/commit/5012439caa926422960d5ac569579a3a725732ad))
+* **tools:** harden batch artifact streaming outputs ([#177](https://github.com/Daghis/teamcity-mcp/issues/177)) ([0874325](https://github.com/Daghis/teamcity-mcp/commit/0874325c77aa2a79050cff57fced0aeec5a55ba0))
+* **tools:** merge build step defaults during update (199) ([#203](https://github.com/Daghis/teamcity-mcp/issues/203)) ([43a668f](https://github.com/Daghis/teamcity-mcp/commit/43a668f0bdc3b0f69c6a3bca0e2a47c49f1cd1a7))
+* **tools:** migrate MCP handlers to unified client ([#141](https://github.com/Daghis/teamcity-mcp/issues/141)) ([bc3a909](https://github.com/Daghis/teamcity-mcp/commit/bc3a909f6eb8c30798865c8b48604aca5405efc8))
+* **tools:** repair manage_build_steps updates ([#154](https://github.com/Daghis/teamcity-mcp/issues/154)) ([b557e44](https://github.com/Daghis/teamcity-mcp/commit/b557e4424d5129de9d3b6e3240e1a876488da040))
+* **tools:** stabilize batch artifact downloads ([#175](https://github.com/Daghis/teamcity-mcp/issues/175)) ([91a0ec3](https://github.com/Daghis/teamcity-mcp/commit/91a0ec33ac61e680dd34393fca46914e2ba3fe7f))
+* **tools:** update_build_config uses 'settings/artifactRules' path; add tests ([#66](https://github.com/Daghis/teamcity-mcp/issues/66)) ([8b8afc6](https://github.com/Daghis/teamcity-mcp/commit/8b8afc6f41038bcde21a50a8662f90fa4acb7e9a))
+* trigger 1.0.2 release (no functional change) ([392be27](https://github.com/Daghis/teamcity-mcp/commit/392be27fe705d55fb8a3057120ff5e2c0a41ca8a))
+* trigger 1.0.3 release (no functional change) ([069212a](https://github.com/Daghis/teamcity-mcp/commit/069212a181861b17e7a1d5cfda05b40c2d11e933))
+
+
+### Miscellaneous Chores
+
+* release 0.1.2 ([21e2595](https://github.com/Daghis/teamcity-mcp/commit/21e25950074ed49bd3e6c571f432f27fb8bd434e))
+
+## [1.9.4](https://github.com/Daghis/teamcity-mcp/compare/v1.9.3...v1.9.4) (2025-09-25)
+
+### Bug Fixes
+
+* **tools:** merge build step defaults during update (199) ([#203](https://github.com/Daghis/teamcity-mcp/issues/203)) ([43a668f](https://github.com/Daghis/teamcity-mcp/commit/43a668f0bdc3b0f69c6a3bca0e2a47c49f1cd1a7))
+
 ## [1.9.3](https://github.com/Daghis/teamcity-mcp/compare/v1.9.2...v1.9.3) (2025-09-21)
 
 

package/dist/index.js

@@ -655,6 +655,7 @@ var import_server = require("@modelcontextprotocol/sdk/server/index.js");
 var import_types = require("@modelcontextprotocol/sdk/types.js");
 
 // src/utils/logger/index.ts
+var import_node_fs = require("node:fs");
 var import_winston = __toESM(require("winston"));
 var TeamCityLogger = class _TeamCityLogger {
   winston;
@@ -758,9 +759,8 @@ var TeamCityLogger = class _TeamCityLogger {
    */
   ensureLogDirectory(directory) {
     try {
-      const fs2 = require("fs");
-      if (fs2.existsSync(directory) === false) {
-        fs2.mkdirSync(directory, { recursive: true });
+      if ((0, import_node_fs.existsSync)(directory) === false) {
+        (0, import_node_fs.mkdirSync)(directory, { recursive: true });
       }
     } catch (error2) {
       this.winston?.warn?.("Failed to create log directory, using current directory", { error: error2 });
@@ -922,7 +922,7 @@ function debug(message, meta) {
 
 // src/tools.ts
 var import_node_crypto = require("node:crypto");
-var import_node_fs = require("node:fs");
+var import_node_fs2 = require("node:fs");
 var import_node_os = require("node:os");
 var import_node_path = require("node:path");
 var import_promises = require("node:stream/promises");
@@ -2763,7 +2763,7 @@ function formatError(err, context) {
   }
   if (err instanceof import_zod2.z.ZodError) {
     error("Validation Error", err, {
-      errors: err.errors,
+      errors: err.issues,
       ...context
     });
     return {
@@ -2771,7 +2771,7 @@ function formatError(err, context) {
       error: {
         message: "Validation failed",
         code: "VALIDATION_ERROR",
-        data: err.errors
+        data: err.issues
       }
     };
   }
@@ -37237,7 +37237,7 @@ var buildRandomFileName = (artifactName) => {
 };
 var sanitizePathSegments = (artifactPath, fallbackName) => {
   const rawSegments = artifactPath?.split("/") ?? [];
-  const sanitizedSegments = rawSegments.map((segment) => segment.trim()).filter((segment) => segment && segment !== "." && segment !== "..").map((segment) => segment.replace(/[^a-zA-Z0-9._-]/g, "_"));
+  const sanitizedSegments = rawSegments.map((segment) => segment.trim()).filter((segment) => segment.length > 0 && segment !== "." && segment !== "..").map((segment) => segment.replace(/[^a-zA-Z0-9._-]/g, "_"));
   if (sanitizedSegments.length === 0) {
     const { sanitizedBase } = sanitizeFileName(fallbackName);
     sanitizedSegments.push(sanitizedBase);
@@ -37250,7 +37250,7 @@ var ensureUniquePath = async (candidate) => {
   const probe = async (attempt) => {
     const next = attempt === 0 ? candidate : `${stem}-${attempt}${ext}`;
     try {
-      const handle = await import_node_fs.promises.open(next, "wx");
+      const handle = await import_node_fs2.promises.open(next, "wx");
       await handle.close();
       return next;
     } catch (error2) {
@@ -37264,9 +37264,10 @@ var ensureUniquePath = async (candidate) => {
   return probe(0);
 };
 var resolveStreamOutputPath = async (artifact, options) => {
-  if (options.explicitOutputPath) {
-    const target = options.explicitOutputPath;
-    await import_node_fs.promises.mkdir((0, import_node_path.dirname)(target), { recursive: true });
+  const { explicitOutputPath } = options;
+  if (typeof explicitOutputPath === "string" && explicitOutputPath.length > 0) {
+    const target = explicitOutputPath;
+    await import_node_fs2.promises.mkdir((0, import_node_path.dirname)(target), { recursive: true });
     return target;
   }
   if (options.outputDir) {
@@ -37279,17 +37280,17 @@ var resolveStreamOutputPath = async (artifact, options) => {
     if (relativePath.startsWith("..") || (0, import_node_path.isAbsolute)(relativePath)) {
       throw new Error("Resolved artifact path escapes the configured output directory");
     }
-    await import_node_fs.promises.mkdir((0, import_node_path.dirname)(candidate), { recursive: true });
+    await import_node_fs2.promises.mkdir((0, import_node_path.dirname)(candidate), { recursive: true });
     return ensureUniquePath(candidate);
   }
   const tempFilePath = (0, import_node_path.join)((0, import_node_os.tmpdir)(), buildRandomFileName(artifact.name));
-  await import_node_fs.promises.mkdir((0, import_node_path.dirname)(tempFilePath), { recursive: true });
+  await import_node_fs2.promises.mkdir((0, import_node_path.dirname)(tempFilePath), { recursive: true });
   return tempFilePath;
 };
 var writeArtifactStreamToDisk = async (artifact, stream, options) => {
   const targetPath = await resolveStreamOutputPath(artifact, options);
-  await (0, import_promises.pipeline)(stream, (0, import_node_fs.createWriteStream)(targetPath));
-  const stats = await import_node_fs.promises.stat(targetPath);
+  await (0, import_promises.pipeline)(stream, (0, import_node_fs2.createWriteStream)(targetPath));
+  const stats = await import_node_fs2.promises.stat(targetPath);
   return { outputPath: targetPath, bytesWritten: stats.size };
 };
 var buildArtifactPayload = async (artifact, encoding, options) => {
@@ -38028,9 +38029,9 @@ var DEV_TOOLS = [
             const safeBuildId = effectiveBuildId.replace(/[^a-zA-Z0-9._-]/g, "_") || "build";
             const defaultFileName = `build-log-${safeBuildId}-${startLine}-${(0, import_node_crypto.randomUUID)()}.log`;
             const targetPath = typed.outputPath ?? (0, import_node_path.join)((0, import_node_os.tmpdir)(), defaultFileName);
-            await import_node_fs.promises.mkdir((0, import_node_path.dirname)(targetPath), { recursive: true });
-            await (0, import_promises.pipeline)(stream, (0, import_node_fs.createWriteStream)(targetPath));
-            const stats = await import_node_fs.promises.stat(targetPath);
+            await import_node_fs2.promises.mkdir((0, import_node_path.dirname)(targetPath), { recursive: true });
+            await (0, import_promises.pipeline)(stream, (0, import_node_fs2.createWriteStream)(targetPath));
+            const stats = await import_node_fs2.promises.stat(targetPath);
             const page = Math.floor(startLine / effectivePageSize) + 1;
             return json({
               encoding: "stream",
@@ -40492,7 +40493,7 @@ var FULL_MODE_TOOLS = [
         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()
+        properties: import_zod4.z.record(import_zod4.z.string(), import_zod4.z.unknown()).optional()
       }).superRefine((value, ctx) => {
         if (value.action === "update" || value.action === "delete") {
           if (!value.stepId || value.stepId.trim() === "") {
@@ -40504,6 +40505,16 @@ var FULL_MODE_TOOLS = [
           }
         }
       });
+      const decodeScriptContent = (key, value) => {
+        if (key !== "script.content" && key !== "kotlinScript.content") {
+          return value;
+        }
+        const normalised = value.replace(/\r\n/g, "\n");
+        if (!normalised.includes("\\")) {
+          return normalised;
+        }
+        return normalised.replace(/\\r\\n/g, "\n").replace(/\\r/g, "\n").replace(/\\n/g, "\n");
+      };
       return runTool(
         "manage_build_steps",
         schema,
@@ -40512,7 +40523,10 @@ var FULL_MODE_TOOLS = [
           switch (typedArgs.action) {
             case "add": {
               const stepProps = Object.fromEntries(
-                Object.entries(typedArgs.properties ?? {}).map(([k, v]) => [k, String(v)])
+                Object.entries(typedArgs.properties ?? {}).map(([k, v]) => {
+                  const value = String(v);
+                  return [k, decodeScriptContent(k, value)];
+                })
               );
               if (typedArgs.type === "simpleRunner" && stepProps["script.content"]) {
                 stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
@@ -40539,24 +40553,59 @@ var FULL_MODE_TOOLS = [
               });
             }
             case "update": {
+              const existingStepResponse = await adapter.modules.buildTypes.getBuildStep(
+                typedArgs.buildTypeId,
+                typedArgs.stepId,
+                "id,name,type,disabled,properties(property(name,value))",
+                {
+                  headers: {
+                    Accept: "application/json"
+                  }
+                }
+              );
+              const existingStep = existingStepResponse.data;
+              const toRecord2 = (collection) => {
+                if (!collection || !Array.isArray(collection.property)) {
+                  return {};
+                }
+                const entries = collection.property.filter((item) => {
+                  return Boolean(item?.name);
+                }).map((item) => {
+                  return [item.name, item.value != null ? String(item.value) : ""];
+                });
+                return Object.fromEntries(entries);
+              };
+              const existingProperties = toRecord2(existingStep?.properties);
               const updatePayload = {};
-              if (typedArgs.name != null) {
-                updatePayload["name"] = typedArgs.name;
+              const mergedName = typedArgs.name ?? existingStep?.name;
+              if (mergedName != null) {
+                updatePayload["name"] = mergedName;
+              }
+              const mergedType = typedArgs.type ?? existingStep?.type;
+              if (mergedType != null) {
+                updatePayload["type"] = mergedType;
               }
-              if (typedArgs.type != null) {
-                updatePayload["type"] = typedArgs.type;
+              if (existingStep?.disabled != null) {
+                updatePayload["disabled"] = existingStep.disabled;
               }
               const rawProps = typedArgs.properties ?? {};
-              const stepProps = Object.fromEntries(
-                Object.entries(rawProps).map(([k, v]) => [k, String(v)])
+              const providedProps = Object.fromEntries(
+                Object.entries(rawProps).map(([k, v]) => {
+                  const value = String(v);
+                  return [k, decodeScriptContent(k, value)];
+                })
               );
-              if (stepProps["script.content"]) {
-                stepProps["use.custom.script"] = stepProps["use.custom.script"] ?? "true";
-                stepProps["script.type"] = stepProps["script.type"] ?? "customScript";
+              const mergedProps = {
+                ...existingProperties,
+                ...providedProps
+              };
+              if (mergedProps["script.content"] && mergedType === "simpleRunner") {
+                mergedProps["use.custom.script"] = mergedProps["use.custom.script"] ?? "true";
+                mergedProps["script.type"] = mergedProps["script.type"] ?? "customScript";
               }
-              if (Object.keys(stepProps).length > 0) {
+              if (Object.keys(mergedProps).length > 0) {
                 updatePayload["properties"] = {
-                  property: Object.entries(stepProps).map(([name, value]) => ({ name, value }))
+                  property: Object.entries(mergedProps).map(([name, value]) => ({ name, value }))
                 };
               }
               if (Object.keys(updatePayload).length === 0) {
@@ -40710,7 +40759,9 @@ var FULL_MODE_TOOLS = [
             const queue = await adapter.modules.buildQueue.getAllQueuedBuilds();
             const builds = queue.data?.build ?? [];
             const ids = new Set(typed.buildTypeIds);
-            const toCancel = builds.filter((b) => b.buildTypeId && ids.has(b.buildTypeId));
+            const toCancel = builds.filter(
+              (build) => typeof build.buildTypeId === "string" && ids.has(build.buildTypeId)
+            );
             for (const b of toCancel) {
               if (b.id == null) continue;
               await adapter.modules.buildQueue.deleteQueuedBuild(String(b.id));