@daghis/teamcity-mcp 2.3.0 → 2.3.2

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [2.3.2](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v2.3.1...teamcity-mcp-v2.3.2) (2026-03-10)
+
+
+### Bug Fixes
+
+* write snapshot dependency options as properties, not XML options ([#406](https://github.com/Daghis/teamcity-mcp/issues/406)) ([#419](https://github.com/Daghis/teamcity-mcp/issues/419)) ([e13fa2d](https://github.com/Daghis/teamcity-mcp/commit/e13fa2dd1be5c387efd9b4fc0e38cb5e15634be4))
+
+## [2.3.1](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v2.3.0...teamcity-mcp-v2.3.1) (2026-03-10)
+
+
+### Bug Fixes
+
+* use toBuildLocator() for all build URL templates in api-client ([#416](https://github.com/Daghis/teamcity-mcp/issues/416)) ([2a10312](https://github.com/Daghis/teamcity-mcp/commit/2a10312e1469ed9d7c9b9e47caaddaac693df400))
+
 ## [2.3.0](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v2.2.3...teamcity-mcp-v2.3.0) (2026-03-10)
 
 

package/CLAUDE.md

@@ -64,7 +64,7 @@ Type support includes: `password`, `text`, `checkbox`, `select` with spec format
 
 When Marc says he wants to "check-in with GitHub", run these steps:
 
-1. Pull latest pulse stats (via project-pulse MCP, repo: `Daghis/teamcity-mcp`, npm: `@daghis/teamcity-mcp`)
+1. Snapshot pulse stats (via project-pulse MCP `snapshot_stats`, name: `teamcity-mcp`, repo: `Daghis/teamcity-mcp`, npm: `@daghis/teamcity-mcp`) — this records a data point for trend tracking
 2. Check new issues
 3. Check new PRs
 4. Check security alerts

package/dist/index.js

@@ -1205,7 +1205,7 @@ function debug2(message, meta) {
 // package.json
 var package_default = {
   name: "@daghis/teamcity-mcp",
-  version: "2.3.0",
+  version: "2.3.2",
   description: "Model Control Protocol server for TeamCity CI/CD integration with AI coding assistants",
   mcpName: "io.github.Daghis/teamcity",
   main: "dist/index.js",
@@ -1214,7 +1214,7 @@ var package_default = {
     "teamcity-mcp": "dist/index.js"
   },
   engines: {
-    node: ">=20.10.0"
+    node: ">=20.19.0"
   },
   scripts: {
     dev: "node ./node_modules/tsx/dist/cli.mjs watch src/index.ts",
@@ -1280,18 +1280,18 @@ var package_default = {
   devDependencies: {
     "@codecov/bundler-plugin-core": "^1.9.1",
     "@esbuild-plugins/tsconfig-paths": "^0.1.2",
+    "@eslint/js": "^10.0.1",
     "@trivago/prettier-plugin-sort-imports": "^6.0.1",
     "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^25.0.2",
-    "@typescript-eslint/eslint-plugin": "^8.51.0",
-    "@typescript-eslint/parser": "^8.51.0",
+    "@typescript-eslint/eslint-plugin": "^8.57.0",
+    "@typescript-eslint/parser": "^8.57.0",
     esbuild: "^0.27.0",
-    eslint: "^9.39.1",
+    eslint: "^10.0.3",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-import-resolver-typescript": "^4.4.4",
-    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-prettier": "^5.0.1",
+    globals: "^17.4.0",
     jest: "^30.1.3",
     "jest-junit": "^16.0.0",
     "js-yaml": "^4.1.1",
@@ -1562,13 +1562,13 @@ var ArtifactManager = class _ArtifactManager {
     } catch (error3) {
       const err = error3;
       if (err.response?.status === 401) {
-        throw new Error("Authentication failed: Invalid TeamCity token");
+        throw new Error("Authentication failed: Invalid TeamCity token", { cause: error3 });
       }
       if (err.response?.status === 404) {
-        throw new Error(`Build not found: ${buildId}`);
+        throw new Error(`Build not found: ${buildId}`, { cause: error3 });
       }
       const errMsg = err.message ?? String(error3);
-      throw new Error(`Failed to fetch artifacts: ${errMsg}`);
+      throw new Error(`Failed to fetch artifacts: ${errMsg}`, { cause: error3 });
     }
   }
   /**
@@ -1708,7 +1708,7 @@ var ArtifactManager = class _ArtifactManager {
       } else {
         errMsg = error3 instanceof Error ? error3.message : "Unknown error";
       }
-      throw new Error(`Failed to download artifact: ${errMsg}`);
+      throw new Error(`Failed to download artifact: ${errMsg}`, { cause: error3 });
     }
   }
   /**
@@ -2033,7 +2033,7 @@ var BuildConfigurationCloneManager = class {
         return null;
       }
       if (axiosError.response?.status === 403) {
-        throw new Error("Permission denied: No access to source configuration");
+        throw new Error("Permission denied: No access to source configuration", { cause: err });
       }
       throw err;
     }
@@ -2107,7 +2107,7 @@ var BuildConfigurationCloneManager = class {
       return { id: newId, name: newName };
     } catch (err) {
       error2("Failed to clone VCS root", err);
-      throw new Error(`Failed to clone VCS root: ${err.message}`);
+      throw new Error(`Failed to clone VCS root: ${err.message}`, { cause: err });
     }
   }
   /**
@@ -2237,14 +2237,14 @@ var BuildConfigurationCloneManager = class {
     } catch (err) {
       const error3 = err;
       if (error3.response?.status === 409) {
-        throw new Error(`Build configuration already exists with ID: ${configId}`);
+        throw new Error(`Build configuration already exists with ID: ${configId}`, { cause: err });
       }
       if (error3.response?.status === 403) {
-        throw new Error("Permission denied: You need project edit permissions");
+        throw new Error("Permission denied: You need project edit permissions", { cause: err });
       }
       if (error3.response?.status === 400) {
         const message = error3.response?.data?.message ?? "Invalid configuration";
-        throw new Error(`Invalid configuration: ${message}`);
+        throw new Error(`Invalid configuration: ${message}`, { cause: err });
       }
       error2(
         "Failed to clone build configuration",
@@ -2429,7 +2429,7 @@ var BuildConfigurationUpdateManager = class {
         return null;
       }
       if (err != null && typeof err === "object" && "response" in err && err.response?.status === 403) {
-        throw new Error("Permission denied: No access to build configuration");
+        throw new Error("Permission denied: No access to build configuration", { cause: err });
       }
       throw err;
     }
@@ -2622,17 +2622,17 @@ var BuildConfigurationUpdateManager = class {
     } catch (err) {
       const error3 = err;
       if (error3.response?.status === 409) {
-        throw new Error("Configuration was modified by another user");
+        throw new Error("Configuration was modified by another user", { cause: err });
       }
       if (error3.response?.status === 403) {
-        throw new Error("Permission denied: You need project edit permissions");
+        throw new Error("Permission denied: You need project edit permissions", { cause: err });
       }
       if (error3.response?.status === 400) {
         const message = error3.response?.data?.message ?? "Invalid configuration";
-        throw new Error(`Invalid update: ${message}`);
+        throw new Error(`Invalid update: ${message}`, { cause: err });
       }
       error2("Failed to apply updates", error3);
-      throw new Error("Partial update failure");
+      throw new Error("Partial update failure", { cause: err });
     }
   }
   /**
@@ -2735,7 +2735,7 @@ var BuildConfigurationUpdateManager = class {
       info2("Rollback completed successfully", { configId });
     } catch (err) {
       error2("Failed to rollback changes", err);
-      throw new Error("Rollback failed: Manual intervention may be required");
+      throw new Error("Rollback failed: Manual intervention may be required", { cause: err });
     }
   }
   /**
@@ -2780,13 +2780,6 @@ var defaultTypeFor = (dependencyType) => {
       return void 0;
   }
 };
-var SNAPSHOT_DEPENDENCY_OPTION_KEYS = /* @__PURE__ */ new Set([
-  "run-build-on-the-same-agent",
-  "sync-revisions",
-  "take-successful-builds-only",
-  "take-started-build-with-same-revisions",
-  "do-not-run-new-build-if-there-is-a-suitable-one"
-]);
 var toStringRecord2 = (input) => {
   if (!input) {
     return {};
@@ -2841,15 +2834,6 @@ var optionsToRecord = (options) => {
   }
   return map;
 };
-var recordToOptions = (record) => {
-  const entries = Object.entries(record);
-  if (entries.length === 0) {
-    return void 0;
-  }
-  return {
-    option: entries.map(([name, value]) => ({ name, value }))
-  };
-};
 var mergeRecords2 = (base, override) => {
   const merged = { ...base };
   for (const [key, value] of Object.entries(override)) {
@@ -2881,25 +2865,6 @@ var propertiesToXml2 = (properties) => {
   }
   return `<properties>${nodes.join("")}</properties>`;
 };
-var optionsToXml = (options) => {
-  if (!options) {
-    return void 0;
-  }
-  const entries = options.option;
-  const list = Array.isArray(entries) ? entries : entries != null ? [entries] : [];
-  if (list.length === 0) {
-    return void 0;
-  }
-  const nodes = list.filter((item) => item?.name).map((item) => {
-    const name = item?.name ?? "";
-    const value = item?.value != null ? String(item.value) : "";
-    return `<option name="${escapeXml2(name)}" value="${escapeXml2(value)}"/>`;
-  });
-  if (nodes.length === 0) {
-    return void 0;
-  }
-  return `<options>${nodes.join("")}</options>`;
-};
 var sourceBuildTypeToXml = (source) => {
   if (!source || typeof source !== "object") {
     return void 0;
@@ -2946,10 +2911,6 @@ var dependencyToXml = (dependencyType, payload) => {
   if (propertiesXml) {
     fragments.push(propertiesXml);
   }
-  const optionsXml = optionsToXml(payload.options);
-  if (optionsXml) {
-    fragments.push(optionsXml);
-  }
   return `<${root}${attributesToString2(attributes)}>${fragments.join("")}</${root}>`;
 };
 var prepareArtifactRequest = (payload) => ({
@@ -3076,42 +3037,19 @@ var BuildDependencyManager = class {
     }
   }
   buildPayload(dependencyType, existing, input) {
-    const existingSnapshot = existing;
     const baseProperties = propertiesToRecord2(existing?.properties);
-    const inputPropertyRecord = toStringRecord2(input.properties);
-    const inputExplicitOptions = toStringRecord2(input.options);
-    let optionOverrides = {};
-    let propertyOverrides = inputPropertyRecord;
-    let baseOptions = {};
     if (dependencyType === "snapshot") {
-      baseOptions = optionsToRecord(existingSnapshot?.options);
-      const knownOptionKeys = /* @__PURE__ */ new Set([
-        ...Object.keys(baseOptions),
-        ...Object.keys(inputExplicitOptions)
-      ]);
-      for (const key of SNAPSHOT_DEPENDENCY_OPTION_KEYS) {
-        knownOptionKeys.add(key);
-      }
-      const derivedOptionOverrides = { ...inputExplicitOptions };
-      const derivedPropertyOverrides = {};
-      for (const [key, value] of Object.entries(inputPropertyRecord)) {
-        if (knownOptionKeys.has(key)) {
-          derivedOptionOverrides[key] = value;
-        } else {
-          derivedPropertyOverrides[key] = value;
-        }
-      }
-      optionOverrides = derivedOptionOverrides;
-      propertyOverrides = derivedPropertyOverrides;
-    } else if (Object.keys(inputExplicitOptions).length > 0) {
-      optionOverrides = inputExplicitOptions;
+      const existingOptions = optionsToRecord(
+        existing?.options
+      );
+      Object.assign(baseProperties, existingOptions);
     }
-    const mergedProps = mergeRecords2(baseProperties, propertyOverrides);
+    const inputOverrides = mergeRecords2(
+      toStringRecord2(input.properties),
+      toStringRecord2(input.options)
+    );
+    const mergedProps = mergeRecords2(baseProperties, inputOverrides);
     const properties = recordToProperties2(mergedProps);
-    let mergedOptions = {};
-    if (dependencyType === "snapshot") {
-      mergedOptions = mergeRecords2(baseOptions, optionOverrides);
-    }
     const resolvedType = input.type ?? existing?.type ?? defaultTypeFor(dependencyType);
     const payload = {
       ...existing ?? {},
@@ -3125,14 +3063,6 @@ var BuildDependencyManager = class {
     } else {
       delete payload.properties;
     }
-    if (dependencyType === "snapshot") {
-      const options = recordToOptions(mergedOptions);
-      if (options) {
-        payload.options = options;
-      } else {
-        delete payload.options;
-      }
-    }
     const dependsOn = input.dependsOn ?? existing?.["source-buildType"]?.id;
     if (dependsOn) {
       payload["source-buildType"] = { id: dependsOn };
@@ -38652,12 +38582,15 @@ var TeamCityAPI = class _TeamCityAPI {
       });
       return response.data;
     } catch (primaryError) {
-      const response = await this.axiosInstance.get(`/app/rest/builds/id:${buildId}/log`, {
-        params: { plain: true },
-        headers: { Accept: "text/plain" },
-        responseType: "text",
-        transformResponse: [(data) => data]
-      });
+      const response = await this.axiosInstance.get(
+        `/app/rest/builds/${toBuildLocator(buildId)}/log`,
+        {
+          params: { plain: true },
+          headers: { Accept: "text/plain" },
+          responseType: "text",
+          transformResponse: [(data) => data]
+        }
+      );
       return response.data;
     }
   }
@@ -38669,16 +38602,19 @@ var TeamCityAPI = class _TeamCityAPI {
     const startLine = options?.startLine ?? 0;
     const lineCount = options?.lineCount ?? 500;
     try {
-      const response = await this.axiosInstance.get(`/app/rest/builds/id:${buildId}/log`, {
-        params: {
-          plain: true,
-          start: startLine,
-          count: lineCount
-        },
-        headers: { Accept: "text/plain" },
-        responseType: "text",
-        transformResponse: [(data) => data]
-      });
+      const response = await this.axiosInstance.get(
+        `/app/rest/builds/${toBuildLocator(buildId)}/log`,
+        {
+          params: {
+            plain: true,
+            start: startLine,
+            count: lineCount
+          },
+          headers: { Accept: "text/plain" },
+          responseType: "text",
+          transformResponse: [(data) => data]
+        }
+      );
       const text = response.data ?? "";
       const lines = text.replace(/\r\n/g, "\n").replace(/\r/g, "\n").split("\n");
       if (lines.length > 0 && lines[lines.length - 1] === "") lines.pop();
@@ -38712,7 +38648,9 @@ var TeamCityAPI = class _TeamCityAPI {
     return response.data;
   }
   async listTestFailures(buildId) {
-    const response = await this.tests.getAllTestOccurrences(`build:(id:${buildId}),status:FAILURE`);
+    const response = await this.tests.getAllTestOccurrences(
+      `build:(${toBuildLocator(buildId)}),status:FAILURE`
+    );
     return response.data;
   }
   async listBuildArtifacts(buildId, options) {
@@ -38732,7 +38670,7 @@ var TeamCityAPI = class _TeamCityAPI {
       responseType: options?.responseType ?? "arraybuffer"
     };
     return this.axiosInstance.get(
-      `/app/rest/builds/id:${buildId}/artifacts/content/${normalizedPath}`,
+      `/app/rest/builds/${toBuildLocator(buildId)}/artifacts/content/${normalizedPath}`,
       requestOptions
     );
   }
@@ -38754,13 +38692,16 @@ var TeamCityAPI = class _TeamCityAPI {
       responseType: options?.responseType ?? "text",
       transformResponse: options?.transformResponse ?? [(data) => data]
     };
-    return this.axiosInstance.get(`/app/rest/builds/id:${buildId}/log`, requestOptions);
+    return this.axiosInstance.get(
+      `/app/rest/builds/${toBuildLocator(buildId)}/log`,
+      requestOptions
+    );
   }
   async getBuildStatistics(buildId, fields) {
     return this.builds.getBuildStatisticValues(toBuildLocator(buildId), fields);
   }
   async listChangesForBuild(buildId, fields) {
-    return this.changes.getAllChanges(`build:(id:${buildId})`, fields);
+    return this.changes.getAllChanges(`build:(${toBuildLocator(buildId)})`, fields);
   }
   async listSnapshotDependencies(buildId) {
     const response = await this.builds.getBuild(toBuildLocator(buildId), "snapshot-dependencies");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@daghis/teamcity-mcp",
-  "version": "2.3.0",
+  "version": "2.3.2",
   "description": "Model Control Protocol server for TeamCity CI/CD integration with AI coding assistants",
   "mcpName": "io.github.Daghis/teamcity",
   "main": "dist/index.js",
@@ -9,7 +9,7 @@
     "teamcity-mcp": "dist/index.js"
   },
   "engines": {
-    "node": ">=20.10.0"
+    "node": ">=20.19.0"
   },
   "scripts": {
     "dev": "node ./node_modules/tsx/dist/cli.mjs watch src/index.ts",
@@ -75,18 +75,18 @@
   "devDependencies": {
     "@codecov/bundler-plugin-core": "^1.9.1",
     "@esbuild-plugins/tsconfig-paths": "^0.1.2",
+    "@eslint/js": "^10.0.1",
     "@trivago/prettier-plugin-sort-imports": "^6.0.1",
     "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^25.0.2",
-    "@typescript-eslint/eslint-plugin": "^8.51.0",
-    "@typescript-eslint/parser": "^8.51.0",
+    "@typescript-eslint/eslint-plugin": "^8.57.0",
+    "@typescript-eslint/parser": "^8.57.0",
     "esbuild": "^0.27.0",
-    "eslint": "^9.39.1",
+    "eslint": "^10.0.3",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-import-resolver-typescript": "^4.4.4",
-    "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-prettier": "^5.0.1",
+    "globals": "^17.4.0",
     "jest": "^30.1.3",
     "jest-junit": "^16.0.0",
     "js-yaml": "^4.1.1",

package/server.json

@@ -7,13 +7,13 @@
     "source": "github"
   },
   "websiteUrl": "https://github.com/Daghis/teamcity-mcp",
-  "version": "2.3.0",
+  "version": "2.3.2",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@daghis/teamcity-mcp",
-      "version": "2.3.0",
+      "version": "2.3.2",
       "runtimeHint": "npx",
       "runtimeArguments": [
         {