@daghis/teamcity-mcp 1.10.6 → 1.10.7

package/.release-please-manifest.json

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

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [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"
@@ -2382,6 +2388,86 @@ 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 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);
+  }
+  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 +2522,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) {
@@ -38706,13 +38796,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, {