@daghis/teamcity-mcp 1.10.6 → 1.10.8

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.10.6"
+  ".": "1.10.8"
 }

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [1.10.8](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.7...teamcity-mcp-v1.10.8) (2025-10-07)
+
+
+### Bug Fixes
+
+* **teamcity:** send snapshot options separately (238) ([fc9b313](https://github.com/Daghis/teamcity-mcp/commit/fc9b31379feccded78db5a35e1dc6f5ad13dccbd))
+* **teamcity:** send snapshot options separately (238) ([6af8ea6](https://github.com/Daghis/teamcity-mcp/commit/6af8ea63dcc7dfae915a55287f3e1bc31e0a5c03))
+
+## [1.10.7](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.6...teamcity-mcp-v1.10.7) (2025-10-06)
+
+
+### Bug Fixes
+
+* **teamcity:** serialize snapshot dependencies as xml ([#235](https://github.com/Daghis/teamcity-mcp/issues/235)) ([#236](https://github.com/Daghis/teamcity-mcp/issues/236)) ([9cec149](https://github.com/Daghis/teamcity-mcp/commit/9cec14934783f135b408211fefa41ba6a38391c8))
+
 ## [1.10.6](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.5...teamcity-mcp-v1.10.6) (2025-10-04)
 
 

package/dist/index.js

@@ -2321,6 +2321,12 @@ var JSON_HEADERS2 = {
     Accept: "application/json"
   }
 };
+var XML_HEADERS = {
+  headers: {
+    "Content-Type": "application/xml",
+    Accept: "application/json"
+  }
+};
 var JSON_GET_HEADERS2 = {
   headers: {
     Accept: "application/json"
@@ -2336,6 +2342,13 @@ 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 {};
@@ -2375,6 +2388,30 @@ var recordToProperties2 = (record) => {
     property: entries.map(([name, value]) => ({ name, value }))
   };
 };
+var optionsToRecord = (options) => {
+  if (!options) {
+    return {};
+  }
+  const optionEntries = options?.option;
+  const collection = Array.isArray(optionEntries) ? optionEntries : optionEntries != null ? [optionEntries] : [];
+  const map = {};
+  for (const item of collection) {
+    if (!item?.name) {
+      continue;
+    }
+    map[item.name] = item.value != null ? String(item.value) : "";
+  }
+  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)) {
@@ -2382,6 +2419,109 @@ var mergeRecords2 = (base, override) => {
   }
   return merged;
 };
+var escapeXml = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&apos;");
+var attributesToString = (attributes) => {
+  const parts = Object.entries(attributes).filter(([, value]) => value !== void 0).map(([key, value]) => `${key}="${escapeXml(value)}"`);
+  return parts.length > 0 ? ` ${parts.join(" ")}` : "";
+};
+var propertiesToXml = (properties) => {
+  if (!properties) {
+    return void 0;
+  }
+  const entries = properties.property;
+  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 `<property name="${escapeXml(name)}" value="${escapeXml(value)}"/>`;
+  });
+  if (nodes.length === 0) {
+    return void 0;
+  }
+  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="${escapeXml(name)}" value="${escapeXml(value)}"/>`;
+  });
+  if (nodes.length === 0) {
+    return void 0;
+  }
+  return `<options>${nodes.join("")}</options>`;
+};
+var sourceBuildTypeToXml = (source) => {
+  if (!source || typeof source !== "object") {
+    return void 0;
+  }
+  const { id, name } = source;
+  if (!id) {
+    return void 0;
+  }
+  const attributes = {
+    id,
+    name
+  };
+  return `<source-buildType${attributesToString(attributes)}/>`;
+};
+var dependencyToXml = (dependencyType, payload) => {
+  const root = dependencyType === "artifact" ? "artifact-dependency" : "snapshot-dependency";
+  const normalizeTypeAttribute = (value) => {
+    if (!value) {
+      return void 0;
+    }
+    if (dependencyType === "snapshot" && value === "snapshotDependency") {
+      return "snapshot_dependency";
+    }
+    if (dependencyType === "artifact" && value === "artifactDependency") {
+      return "artifact_dependency";
+    }
+    return value;
+  };
+  const attributes = {
+    id: typeof payload.id === "string" && payload.id.trim() !== "" ? payload.id : void 0,
+    name: typeof payload.name === "string" && payload.name.trim() !== "" ? payload.name : void 0,
+    type: typeof payload.type === "string" && payload.type.trim() !== "" ? normalizeTypeAttribute(payload.type) : void 0,
+    disabled: typeof payload.disabled === "boolean" ? payload.disabled ? "true" : "false" : void 0,
+    inherited: typeof payload.inherited === "boolean" ? payload.inherited ? "true" : "false" : void 0
+  };
+  const fragments = [];
+  const sourceBuildTypeXml = sourceBuildTypeToXml(
+    payload["source-buildType"]
+  );
+  if (sourceBuildTypeXml) {
+    fragments.push(sourceBuildTypeXml);
+  }
+  const propertiesXml = propertiesToXml(payload.properties);
+  if (propertiesXml) {
+    fragments.push(propertiesXml);
+  }
+  const optionsXml = optionsToXml(payload.options);
+  if (optionsXml) {
+    fragments.push(optionsXml);
+  }
+  return `<${root}${attributesToString(attributes)}>${fragments.join("")}</${root}>`;
+};
+var prepareArtifactRequest = (payload) => ({
+  body: payload,
+  headers: JSON_HEADERS2
+});
+var prepareSnapshotRequest = (payload) => ({
+  body: dependencyToXml("snapshot", payload),
+  headers: XML_HEADERS
+});
 var BuildDependencyManager = class {
   constructor(client) {
     this.client = client;
@@ -2436,36 +2576,40 @@ var BuildDependencyManager = class {
   }
   async createDependency(dependencyType, buildTypeId, payload) {
     if (dependencyType === "artifact") {
+      const { body: body2, headers: headers2 } = prepareArtifactRequest(payload);
       return this.client.modules.buildTypes.addArtifactDependencyToBuildType(
         buildTypeId,
         void 0,
-        payload,
-        JSON_HEADERS2
+        body2,
+        headers2
       );
     }
+    const { body, headers } = prepareSnapshotRequest(payload);
     return this.client.modules.buildTypes.addSnapshotDependencyToBuildType(
       buildTypeId,
       void 0,
-      payload,
-      JSON_HEADERS2
+      body,
+      headers
     );
   }
   async replaceDependency(dependencyType, buildTypeId, dependencyId, payload) {
     if (dependencyType === "artifact") {
+      const { body: body2, headers: headers2 } = prepareArtifactRequest(payload);
       return this.client.modules.buildTypes.replaceArtifactDependency(
         buildTypeId,
         dependencyId,
         void 0,
-        payload,
-        JSON_HEADERS2
+        body2,
+        headers2
       );
     }
+    const { body, headers } = prepareSnapshotRequest(payload);
     return this.client.modules.buildTypes.replaceSnapshotDependency(
       buildTypeId,
       dependencyId,
       void 0,
-      payload,
-      JSON_HEADERS2
+      body,
+      headers
     );
   }
   async fetchDependency(dependencyType, buildTypeId, dependencyId) {
@@ -2482,7 +2626,7 @@ var BuildDependencyManager = class {
       const response = await this.client.modules.buildTypes.getSnapshotDependency(
         buildTypeId,
         dependencyId,
-        "id,type,disabled,properties(property(name,value)),'source-buildType'(id)",
+        "id,type,disabled,properties(property(name,value)),options(option(name,value)),'source-buildType'(id)",
         JSON_GET_HEADERS2
       );
       return response.data;
@@ -2494,9 +2638,42 @@ var BuildDependencyManager = class {
     }
   }
   buildPayload(dependencyType, existing, input) {
-    const baseProps = propertiesToRecord2(existing?.properties);
-    const mergedProps = mergeRecords2(baseProps, toStringRecord2(input.properties));
+    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 mergedProps = mergeRecords2(baseProperties, propertyOverrides);
     const properties = recordToProperties2(mergedProps);
+    let mergedOptions = {};
+    if (dependencyType === "snapshot") {
+      mergedOptions = mergeRecords2(baseOptions, optionOverrides);
+    }
     const resolvedType = input.type ?? existing?.type ?? defaultTypeFor(dependencyType);
     const payload = {
       ...existing ?? {},
@@ -2507,6 +2684,16 @@ var BuildDependencyManager = class {
     }
     if (properties) {
       payload.properties = properties;
+    } 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) {
@@ -38706,13 +38893,13 @@ var DEV_TOOLS = [
             buildRequest.properties = propertiesPayload;
           }
           const sendXmlFallback = async (error2) => {
-            const escapeXml = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&apos;");
-            const branchPart = branchName ? `<branchName>${escapeXml(branchName)}</branchName>` : "";
-            const commentPart = commentText ? `<comment><text>${escapeXml(commentText)}</text></comment>` : "";
+            const escapeXml2 = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&apos;");
+            const branchPart = branchName ? `<branchName>${escapeXml2(branchName)}</branchName>` : "";
+            const commentPart = commentText ? `<comment><text>${escapeXml2(commentText)}</text></comment>` : "";
             const propertiesPart = propertiesPayload ? `<properties>${propertiesPayload.property.map(
-              (prop) => `<property name="${escapeXml(prop.name)}" value="${escapeXml(prop.value)}"/>`
+              (prop) => `<property name="${escapeXml2(prop.name)}" value="${escapeXml2(prop.value)}"/>`
             ).join("")}</properties>` : "";
-            const xml = `<?xml version="1.0" encoding="UTF-8"?><build><buildType id="${escapeXml(
+            const xml = `<?xml version="1.0" encoding="UTF-8"?><build><buildType id="${escapeXml2(
               typed.buildTypeId
             )}"/>${branchPart}${commentPart}${propertiesPart}</build>`;
             const response = await adapter.http.post("/app/rest/buildQueue", xml, {
@@ -41445,6 +41632,10 @@ var FULL_MODE_TOOLS = [
           type: "object",
           description: "Dependency properties (e.g. cleanDestinationDirectory, pathRules)"
         },
+        options: {
+          type: "object",
+          description: "Snapshot dependency options (e.g. run-build-on-the-same-agent)"
+        },
         type: {
           type: "string",
           description: "Override dependency type value sent to TeamCity"
@@ -41462,6 +41653,7 @@ var FULL_MODE_TOOLS = [
         dependencyId: import_zod4.z.string().min(1).optional(),
         dependsOn: import_zod4.z.string().min(1).optional(),
         properties: import_zod4.z.record(import_zod4.z.string(), propertyValue).optional(),
+        options: import_zod4.z.record(import_zod4.z.string(), propertyValue).optional(),
         type: import_zod4.z.string().min(1).optional(),
         disabled: import_zod4.z.boolean().optional()
       }).superRefine((value, ctx) => {
@@ -41493,6 +41685,7 @@ var FULL_MODE_TOOLS = [
                 dependencyType: typed.dependencyType,
                 dependsOn: typed.dependsOn,
                 properties: typed.properties,
+                options: typed.options,
                 type: typed.type,
                 disabled: typed.disabled
               });
@@ -41511,6 +41704,7 @@ var FULL_MODE_TOOLS = [
                 dependencyType: typed.dependencyType,
                 dependsOn: typed.dependsOn,
                 properties: typed.properties,
+                options: typed.options,
                 type: typed.type,
                 disabled: typed.disabled
               });