@daghis/teamcity-mcp 1.10.3 → 1.10.5

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.10.3"
+  ".": "1.10.5"
 }

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.10.5](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.4...teamcity-mcp-v1.10.5) (2025-10-04)
+
+
+### Bug Fixes
+
+* **teamcity:** set dependency types when adding via tool ([#231](https://github.com/Daghis/teamcity-mcp/issues/231)) ([702764a](https://github.com/Daghis/teamcity-mcp/commit/702764a19de0a649e394d5b4679dc8ea6373dcf2))
+
+## [1.10.4](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.3...teamcity-mcp-v1.10.4) (2025-10-04)
+
+
+### Bug Fixes
+
+* **tools:** normalize branch locators for list_builds ([#228](https://github.com/Daghis/teamcity-mcp/issues/228)) ([16fe7f0](https://github.com/Daghis/teamcity-mcp/commit/16fe7f0e2b46d04e66253ddb3cd9a06c5febf0b6))
+
 ## [1.10.3](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.10.2...teamcity-mcp-v1.10.3) (2025-09-27)
 
 

package/dist/index.js

@@ -610,9 +610,7 @@ function loadConfig() {
 }
 var cachedConfig = null;
 function getConfig() {
-  if (!cachedConfig) {
-    cachedConfig = loadConfig();
-  }
+  cachedConfig ??= loadConfig();
   return cachedConfig;
 }
 function isTest() {
@@ -1756,18 +1754,14 @@ var BuildConfigurationCloneManager = class {
       };
     }
     if (options.copyBuildCounter && source.buildNumberCounter) {
-      if (configPayload.settings == null) {
-        configPayload.settings = { property: [] };
-      }
+      configPayload.settings ??= { property: [] };
       configPayload.settings.property?.push({
         name: "buildNumberCounter",
         value: source.buildNumberCounter.toString()
       });
     }
     if (source.buildNumberFormat) {
-      if (configPayload.settings == null) {
-        configPayload.settings = { property: [] };
-      }
+      configPayload.settings ??= { property: [] };
       configPayload.settings.property?.push({
         name: "buildNumberPattern",
         value: source.buildNumberFormat
@@ -2335,9 +2329,9 @@ var JSON_GET_HEADERS2 = {
 var defaultTypeFor = (dependencyType) => {
   switch (dependencyType) {
     case "artifact":
-      return void 0;
+      return "artifactDependency";
     case "snapshot":
-      return void 0;
+      return "snapshotDependency";
     default:
       return void 0;
   }
@@ -3586,6 +3580,100 @@ async function fetchAllPages(fetchFn, options = {}) {
   return allItems;
 }
 
+// src/utils/list-builds-locator.ts
+var SIMPLE_BRANCH_VALUES = /* @__PURE__ */ new Set([
+  "default:true",
+  "default:false",
+  "default:any",
+  "unspecified:true",
+  "unspecified:false",
+  "unspecified:any",
+  "branched:true",
+  "branched:false",
+  "branched:any"
+]);
+var BRANCH_PREFIXES_ALLOW_UNWRAPPED = ["default:", "unspecified:", "branched:", "policy:"];
+function splitLocatorParts(locator) {
+  const parts = [];
+  let current = "";
+  let depth = 0;
+  for (const char of locator) {
+    if (char === "," && depth === 0) {
+      const piece = current.trim();
+      if (piece.length > 0) {
+        parts.push(piece);
+      }
+      current = "";
+      continue;
+    }
+    if (char === "(") {
+      depth += 1;
+    } else if (char === ")" && depth > 0) {
+      depth -= 1;
+    }
+    current += char;
+  }
+  const finalPiece = current.trim();
+  if (finalPiece.length > 0) {
+    parts.push(finalPiece);
+  }
+  return parts;
+}
+function wrapBranchValue(value) {
+  const trimmed = value.trim();
+  if (trimmed.length === 0) {
+    return trimmed;
+  }
+  if (trimmed.startsWith("(")) {
+    return trimmed;
+  }
+  const lower = trimmed.toLowerCase();
+  if (SIMPLE_BRANCH_VALUES.has(lower)) {
+    return trimmed;
+  }
+  if (BRANCH_PREFIXES_ALLOW_UNWRAPPED.some((prefix) => lower.startsWith(prefix))) {
+    return trimmed;
+  }
+  if (trimmed.includes("*") && !trimmed.includes(":")) {
+    return trimmed;
+  }
+  if (trimmed.includes("/") || trimmed.includes(":") || /\s/.test(trimmed)) {
+    return `(${trimmed})`;
+  }
+  return trimmed;
+}
+function normalizeBranchSegment(segment) {
+  const trimmed = segment.trim();
+  if (trimmed.length === 0) {
+    return trimmed;
+  }
+  if (!trimmed.toLowerCase().startsWith("branch:")) {
+    return trimmed;
+  }
+  const rawValue = trimmed.slice("branch:".length).trim();
+  if (rawValue.length === 0) {
+    return trimmed;
+  }
+  if (rawValue.startsWith("(")) {
+    return `branch:${rawValue}`;
+  }
+  return `branch:${wrapBranchValue(rawValue)}`;
+}
+function normalizeLocatorSegments(locator) {
+  if (!locator) {
+    return [];
+  }
+  return splitLocatorParts(locator).map((segment) => normalizeBranchSegment(segment)).filter((segment) => segment.length > 0);
+}
+function hasBranchSegment(segments) {
+  return segments.some((segment) => segment.toLowerCase().startsWith("branch:"));
+}
+function buildBranchSegmentInput(branchInput) {
+  const normalized = branchInput.trim();
+  const withPrefix = normalized.toLowerCase().startsWith("branch:") ? normalized : `branch:${normalized}`;
+  return normalizeBranchSegment(withPrefix);
+}
+
 // src/utils/mcp.ts
 var import_zod3 = require("zod");
 
@@ -3686,9 +3774,7 @@ init_errors();
 var ErrorLogger = class _ErrorLogger {
   static instance;
   static getInstance() {
-    if (_ErrorLogger.instance == null) {
-      _ErrorLogger.instance = new _ErrorLogger();
-    }
+    _ErrorLogger.instance ??= new _ErrorLogger();
     return _ErrorLogger.instance;
   }
   /**
@@ -3773,9 +3859,7 @@ var GlobalErrorHandler = class _GlobalErrorHandler {
   }
   static instance;
   static getInstance(options) {
-    if (_GlobalErrorHandler.instance == null) {
-      _GlobalErrorHandler.instance = new _GlobalErrorHandler(options);
-    }
+    _GlobalErrorHandler.instance ??= new _GlobalErrorHandler(options);
     return _GlobalErrorHandler.instance;
   }
   /**
@@ -38447,6 +38531,7 @@ var DEV_TOOLS = [
         locator: { type: "string", description: "Optional build locator to filter builds" },
         projectId: { type: "string", description: "Filter by project ID" },
         buildTypeId: { type: "string", description: "Filter by build type ID" },
+        branch: { type: "string", description: "Filter by branch (logical or VCS name)" },
         status: {
           type: "string",
           enum: ["SUCCESS", "FAILURE", "ERROR"],
@@ -38467,6 +38552,7 @@ var DEV_TOOLS = [
         locator: import_zod4.z.string().min(1).optional(),
         projectId: import_zod4.z.string().min(1).optional(),
         buildTypeId: import_zod4.z.string().min(1).optional(),
+        branch: import_zod4.z.string().min(1).optional(),
         status: import_zod4.z.enum(["SUCCESS", "FAILURE", "ERROR"]).optional(),
         count: import_zod4.z.number().int().min(1).max(1e3).default(10).optional(),
         pageSize: import_zod4.z.number().int().min(1).max(1e3).optional(),
@@ -38479,10 +38565,17 @@ var DEV_TOOLS = [
         schema,
         async (typed) => {
           const adapter = createAdapterFromTeamCityAPI(TeamCityAPI.getInstance());
-          const baseParts = [];
-          if (typed.locator) baseParts.push(typed.locator);
+          const locatorSegments = normalizeLocatorSegments(typed.locator);
+          const hasBranchInLocator = hasBranchSegment(locatorSegments);
+          const baseParts = [...locatorSegments];
           if (typed.projectId) baseParts.push(`project:(id:${typed.projectId})`);
           if (typed.buildTypeId) baseParts.push(`buildType:(id:${typed.buildTypeId})`);
+          if (typed.branch) {
+            const branchSegment = buildBranchSegmentInput(typed.branch);
+            if (!hasBranchInLocator) {
+              baseParts.push(branchSegment);
+            }
+          }
           if (typed.status) baseParts.push(`status:${typed.status}`);
           const pageSize = typed.pageSize ?? typed.count ?? 100;
           const baseFetch = async ({ count, start }) => {