@daghis/teamcity-mcp 2.3.1 → 2.3.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # 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)
 
 

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.1",
+  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",
@@ -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 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@daghis/teamcity-mcp",
-  "version": "2.3.1",
+  "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",

package/server.json

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