@daghis/teamcity-mcp 1.12.0 → 1.12.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.12.1](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.12.0...teamcity-mcp-v1.12.1) (2025-12-22)
+
+
+### Bug Fixes
+
+* handle queued builds in get_build and get_build_status ([#324](https://github.com/Daghis/teamcity-mcp/issues/324)) ([4cb2dab](https://github.com/Daghis/teamcity-mcp/commit/4cb2dabfc9161e07708a899622d78b886742459d))
+
 ## [1.12.0](https://github.com/Daghis/teamcity-mcp/compare/teamcity-mcp-v1.11.20...teamcity-mcp-v1.12.0) (2025-12-06)
 
 

package/dist/index.js

@@ -221,6 +221,12 @@ var build_status_manager_exports = {};
 __export(build_status_manager_exports, {
   BuildStatusManager: () => BuildStatusManager
 });
+function isAxios404(error3) {
+  return error3 != null && typeof error3 === "object" && "response" in error3 && error3.response?.status === 404;
+}
+function isAxios403(error3) {
+  return error3 != null && typeof error3 === "object" && "response" in error3 && error3.response?.status === 403;
+}
 var BuildStatusManager;
 var init_build_status_manager = __esm({
   "src/teamcity/build-status-manager.ts"() {
@@ -235,7 +241,11 @@ var init_build_status_manager = __esm({
         this.cache = /* @__PURE__ */ new Map();
       }
       /**
-       * Get build status by ID or number
+       * Get build status by ID or number.
+       * Uses 3-step fallback to handle builds in queue and race conditions:
+       * 1. Try builds endpoint
+       * 2. On 404, try build queue (build may be queued)
+       * 3. On 404 again, retry builds endpoint (build may have left queue between checks)
        */
       async getBuildStatus(options) {
         if (!options.buildId && !options.buildNumber) {
@@ -252,40 +262,92 @@ var init_build_status_manager = __esm({
           }
         }
         try {
-          let buildData;
-          if (options.buildId) {
-            const response = await this.client.builds.getBuild(
-              `id:${options.buildId}`,
-              this.getFieldSelection(options)
-            );
-            buildData = response.data;
-          } else {
-            const locator = this.buildLocator(options);
-            const response = await this.client.builds.getBuild(
-              locator,
-              this.getFieldSelection(options)
-            );
-            buildData = response.data;
-          }
-          if (buildData == null) {
-            throw new BuildNotFoundError("Build data is undefined");
-          }
-          const result = this.transformBuildResponse(buildData, options);
-          if (result.state === "finished" || result.state === "canceled") {
-            this.setCachedResult(cacheKey, result);
-          }
-          return result;
+          return await this.getBuildStatusFromBuildsEndpoint(options, cacheKey);
         } catch (error3) {
-          if (error3 != null && typeof error3 === "object" && "response" in error3 && error3.response?.status === 404) {
-            throw new BuildNotFoundError(`Build not found: ${options.buildId ?? options.buildNumber}`);
+          if (!isAxios404(error3) || !options.buildId) {
+            this.handleBuildStatusError(error3, options);
           }
-          if (error3 != null && typeof error3 === "object" && "response" in error3 && error3.response?.status === 403) {
-            throw new BuildAccessDeniedError(
-              `Access denied to build: ${options.buildId ?? options.buildNumber}`
-            );
+        }
+        const buildId = options.buildId;
+        try {
+          return await this.getQueuedBuildStatus(buildId);
+        } catch (queueError) {
+          if (!isAxios404(queueError)) {
+            this.handleBuildStatusError(queueError, options);
           }
-          throw error3;
         }
+        try {
+          return await this.getBuildStatusFromBuildsEndpoint(options, cacheKey);
+        } catch (error3) {
+          this.handleBuildStatusError(error3, options);
+        }
+      }
+      /**
+       * Get build status from the builds endpoint
+       */
+      async getBuildStatusFromBuildsEndpoint(options, cacheKey) {
+        let buildData;
+        if (options.buildId) {
+          const response = await this.client.builds.getBuild(
+            `id:${options.buildId}`,
+            this.getFieldSelection(options)
+          );
+          buildData = response.data;
+        } else {
+          const locator = this.buildLocator(options);
+          const response = await this.client.builds.getBuild(locator, this.getFieldSelection(options));
+          buildData = response.data;
+        }
+        if (buildData == null) {
+          throw new BuildNotFoundError("Build data is undefined");
+        }
+        const result = this.transformBuildResponse(buildData, options);
+        if (result.state === "finished" || result.state === "canceled") {
+          this.setCachedResult(cacheKey, result);
+        }
+        return result;
+      }
+      /**
+       * Get build status from the build queue
+       */
+      async getQueuedBuildStatus(buildId) {
+        const response = await this.client.modules.buildQueue.getQueuedBuild(
+          `id:${buildId}`,
+          "id,number,state,status,buildTypeId,branchName,webUrl,queuedDate,waitReason"
+        );
+        const queuedBuild = response.data;
+        if (queuedBuild == null) {
+          throw new BuildNotFoundError("Queued build data is undefined");
+        }
+        const result = {
+          buildId: String(queuedBuild.id),
+          buildNumber: queuedBuild.number,
+          buildTypeId: queuedBuild.buildTypeId,
+          state: "queued",
+          status: void 0,
+          percentageComplete: 0,
+          branchName: queuedBuild.branchName,
+          webUrl: queuedBuild.webUrl,
+          waitReason: queuedBuild.waitReason
+        };
+        if (queuedBuild.queuedDate) {
+          result.queuedDate = this.parseDate(queuedBuild.queuedDate);
+        }
+        return result;
+      }
+      /**
+       * Handle and re-throw build status errors with appropriate error types
+       */
+      handleBuildStatusError(error3, options) {
+        if (isAxios404(error3)) {
+          throw new BuildNotFoundError(`Build not found: ${options.buildId ?? options.buildNumber}`);
+        }
+        if (isAxios403(error3)) {
+          throw new BuildAccessDeniedError(
+            `Access denied to build: ${options.buildId ?? options.buildNumber}`
+          );
+        }
+        throw error3;
       }
       /**
        * Get build status using custom locator
@@ -954,7 +1016,7 @@ function debug2(message, meta) {
 // package.json
 var package_default = {
   name: "@daghis/teamcity-mcp",
-  version: "1.12.0",
+  version: "1.12.1",
   description: "Model Control Protocol server for TeamCity CI/CD integration with AI coding assistants",
   mcpName: "io.github.Daghis/teamcity",
   main: "dist/index.js",
@@ -1040,7 +1102,7 @@ var package_default = {
     "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
     "@types/morgan": "^1.9.9",
-    "@types/node": "^24.10.0",
+    "@types/node": "^25.0.2",
     "@typescript-eslint/eslint-plugin": "^8.46.3",
     "@typescript-eslint/parser": "^8.46.3",
     "axios-retry": "^4.5.0",
@@ -38578,6 +38640,7 @@ var TeamCityAPI = class _TeamCityAPI {
 
 // src/tools.ts
 var isReadableStream = (value) => typeof value === "object" && value !== null && typeof value.pipe === "function";
+var isAxios4042 = (error3) => (0, import_axios36.isAxiosError)(error3) && error3.response?.status === 404;
 var sanitizeFileName = (artifactName) => {
   const base = (0, import_node_path.basename)(artifactName || "artifact");
   const safeBase = base.replace(/[^a-zA-Z0-9._-]/g, "_") || "artifact";
@@ -39001,7 +39064,7 @@ var DEV_TOOLS = [
   },
   {
     name: "get_build",
-    description: "Get details of a specific build",
+    description: "Get details of a specific build (works for both queued and running/finished builds)",
     inputSchema: {
       type: "object",
       properties: {
@@ -39016,6 +39079,18 @@ var DEV_TOOLS = [
         schema,
         async (typed) => {
           const adapter = createAdapterFromTeamCityAPI(TeamCityAPI.getInstance());
+          try {
+            const build2 = await adapter.getBuild(typed.buildId);
+            return json(build2);
+          } catch (error3) {
+            if (!isAxios4042(error3)) throw error3;
+          }
+          try {
+            const qb = await adapter.modules.buildQueue.getQueuedBuild(`id:${typed.buildId}`);
+            return json({ ...qb.data, state: "queued" });
+          } catch (queueError) {
+            if (!isAxios4042(queueError)) throw queueError;
+          }
           const build = await adapter.getBuild(typed.buildId);
           return json(build);
         },
@@ -39233,21 +39308,19 @@ var DEV_TOOLS = [
             if (typeof result.queuePosition === "number") {
               enrich.canMoveToTop = result.queuePosition > 1;
             }
-            if (typed.includeQueueTotals) {
-              try {
-                const countResp = await adapter.modules.buildQueue.getAllQueuedBuilds(
-                  void 0,
-                  "count"
-                );
-                enrich.totalQueued = countResp.data.count;
-              } catch {
-              }
+            try {
+              const countResp = await adapter.modules.buildQueue.getAllQueuedBuilds(
+                void 0,
+                "count"
+              );
+              enrich.totalQueued = countResp.data.count;
+            } catch {
             }
-            if (typed.includeQueueReason) {
+            if (!result.waitReason) {
               try {
                 const targetBuildId = typed.buildId ?? result.buildId;
                 if (targetBuildId) {
-                  const qb = await adapter.modules.buildQueue.getQueuedBuild(targetBuildId);
+                  const qb = await adapter.modules.buildQueue.getQueuedBuild(`id:${targetBuildId}`);
                   enrich.waitReason = qb.data.waitReason;
                 }
               } catch {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@daghis/teamcity-mcp",
-  "version": "1.12.0",
+  "version": "1.12.1",
   "description": "Model Control Protocol server for TeamCity CI/CD integration with AI coding assistants",
   "mcpName": "io.github.Daghis/teamcity",
   "main": "dist/index.js",
@@ -86,7 +86,7 @@
     "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
     "@types/morgan": "^1.9.9",
-    "@types/node": "^24.10.0",
+    "@types/node": "^25.0.2",
     "@typescript-eslint/eslint-plugin": "^8.46.3",
     "@typescript-eslint/parser": "^8.46.3",
     "axios-retry": "^4.5.0",

package/server.json

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