@daghis/teamcity-mcp 1.10.7 → 1.10.9

package/.release-please-manifest.json

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

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## [1.10.9](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.8...teamcity-mcp-v1.10.9) (2025-11-04)
+
+
+### Bug Fixes
+
+* **server:** keep stdio transport alive ([#247](https://github.com/Daghis/teamcity-mcp/issues/247)) ([c325cf4](https://github.com/Daghis/teamcity-mcp/commit/c325cf4d436489a91cabf0bb8e0d84087c97a1e4))
+* **server:** keep stdio transport alive (247) ([3b77eea](https://github.com/Daghis/teamcity-mcp/commit/3b77eea2fc771964c78a3b011f67ffb8afdd02e9))
+
+## [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)
 
 

package/dist/index.js

@@ -642,6 +642,43 @@ function getTeamCityToken() {
   return config2.teamcity.token;
 }
 
+// src/server-runner.ts
+async function startServerLifecycle(server, transport) {
+  await server.connect(transport);
+  return new Promise((resolve2, reject) => {
+    const previousOnClose = server.onclose;
+    const previousOnError = server.onerror;
+    const cleanup = () => {
+      server.onclose = previousOnClose;
+      server.onerror = previousOnError;
+    };
+    server.onclose = () => {
+      previousOnClose?.();
+      cleanup();
+      resolve2();
+    };
+    server.onerror = (rawError) => {
+      previousOnError?.(rawError);
+      const error2 = rawError instanceof Error ? rawError : new Error(String(rawError));
+      const closeTransport = () => {
+        if (!transport.close) {
+          return Promise.resolve();
+        }
+        try {
+          return Promise.resolve(transport.close());
+        } catch (closeError) {
+          return Promise.reject(closeError);
+        }
+      };
+      void closeTransport().catch(() => {
+      }).finally(() => {
+        cleanup();
+        reject(error2);
+      });
+    };
+  });
+}
+
 // src/server.ts
 var import_server = require("@modelcontextprotocol/sdk/server/index.js");
 var import_types = require("@modelcontextprotocol/sdk/types.js");
@@ -2342,6 +2379,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 {};
@@ -2381,6 +2425,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)) {
@@ -2412,6 +2480,25 @@ var propertiesToXml = (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="${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;
@@ -2458,6 +2545,10 @@ var dependencyToXml = (dependencyType, payload) => {
   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) => ({
@@ -2572,7 +2663,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;
@@ -2584,9 +2675,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 ?? {},
@@ -2597,6 +2721,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) {
@@ -41535,6 +41669,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"
@@ -41552,6 +41690,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) => {
@@ -41583,6 +41722,7 @@ var FULL_MODE_TOOLS = [
                 dependencyType: typed.dependencyType,
                 dependsOn: typed.dependsOn,
                 properties: typed.properties,
+                options: typed.options,
                 type: typed.type,
                 disabled: typed.disabled
               });
@@ -41601,6 +41741,7 @@ var FULL_MODE_TOOLS = [
                 dependencyType: typed.dependencyType,
                 dependsOn: typed.dependsOn,
                 properties: typed.properties,
+                options: typed.options,
                 type: typed.type,
                 disabled: typed.disabled
               });
@@ -43017,6 +43158,34 @@ function createSimpleServer() {
 
 // src/index.ts
 dotenv2.config();
+var activeServer = null;
+var lifecyclePromise = null;
+var shuttingDown = false;
+var clearServerState = () => {
+  activeServer = null;
+  lifecyclePromise = null;
+};
+async function shutdown(exitCode) {
+  if (shuttingDown) {
+    process.exit(exitCode);
+  }
+  shuttingDown = true;
+  process.stderr.write("\nShutting down TeamCity MCP Server...\n");
+  try {
+    const serverToClose = activeServer;
+    const lifecycleToAwait = lifecyclePromise;
+    await serverToClose?.close();
+    await lifecycleToAwait;
+  } catch (error2) {
+    process.stderr.write(
+      `Error while closing server: ${error2 instanceof Error ? error2.message : String(error2)}
+`
+    );
+  } finally {
+    clearServerState();
+    process.exit(exitCode);
+  }
+}
 async function main() {
   try {
     let hasUrl;
@@ -43036,21 +43205,25 @@ async function main() {
 `);
     const server = createSimpleServer();
     const transport = new import_stdio.StdioServerTransport();
-    await server.connect(transport);
+    const lifecycle = startServerLifecycle(server, transport);
+    activeServer = server;
+    lifecyclePromise = lifecycle;
     process.stderr.write("TeamCity MCP Server is running and ready to accept connections\n");
+    await lifecycle;
+    clearServerState();
+    process.stderr.write("TeamCity MCP Server connection closed\n");
   } catch (error2) {
+    clearServerState();
     process.stderr.write(`Failed to start server: ${error2}
 `);
     process.exit(1);
   }
 }
 process.on("SIGINT", () => {
-  process.stderr.write("\nShutting down TeamCity MCP Server...\n");
-  process.exit(0);
+  void shutdown(0);
 });
 process.on("SIGTERM", () => {
-  process.stderr.write("\nShutting down TeamCity MCP Server...\n");
-  process.exit(0);
+  void shutdown(0);
 });
 main().catch((error2) => {
   process.stderr.write(`Unhandled error: ${error2}