@daghis/teamcity-mcp 0.2.0 → 0.2.1

package/.github/workflows/publish.yml

@@ -39,7 +39,27 @@ jobs:
       - name: Verify package contents
         run: npm pack --dry-run
 
+      - name: Read package version
+        id: pkg
+        run: |
+          node -e "console.log('version=' + require('./package.json').version)" >> "$GITHUB_OUTPUT"
+
+      - name: Check if version already exists on npm
+        id: exists
+        run: |
+          set -e
+          PKG_NAME=$(node -p -e "require('./package.json').name")
+          VERSION=${{ steps.pkg.outputs.version }}
+          if npm view "${PKG_NAME}@${VERSION}" version > /dev/null 2>&1; then
+            echo "exists=true" >> "$GITHUB_OUTPUT"
+            echo "Version ${VERSION} already published for ${PKG_NAME}; skipping publish."
+          else
+            echo "exists=false" >> "$GITHUB_OUTPUT"
+            echo "Version ${VERSION} not found on registry; will publish."
+          fi
+
       - name: Publish to npm
+        if: steps.exists.outputs.exists == 'false'
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
           NPM_CONFIG_PROVENANCE: 'true'

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.2.1](https://github.com/Daghis/teamcity-mcp/compare/v0.2.0...v0.2.1) (2025-09-12)
+
+
+### Bug Fixes
+
+* **mcp:** normalize TeamCity errors and wrap server info/metrics with runTool ([#53](https://github.com/Daghis/teamcity-mcp/issues/53)) ([2c8ab85](https://github.com/Daghis/teamcity-mcp/commit/2c8ab855a85e5faec4216a498c089e3a1a93ed7b))
+
 ## [0.2.0](https://github.com/Daghis/teamcity-mcp/compare/v0.1.2...v0.2.0) (2025-09-11)
 
 

package/dist/index.js

@@ -11652,49 +11652,8 @@ var require_follow_redirects = __commonJS({
   }
 });
 
-// node_modules/is-retry-allowed/index.js
-var require_is_retry_allowed = __commonJS({
-  "node_modules/is-retry-allowed/index.js"(exports2, module2) {
-    "use strict";
-    var denyList = /* @__PURE__ */ new Set([
-      "ENOTFOUND",
-      "ENETUNREACH",
-      // SSL errors from https://github.com/nodejs/node/blob/fc8e3e2cdc521978351de257030db0076d79e0ab/src/crypto/crypto_common.cc#L301-L328
-      "UNABLE_TO_GET_ISSUER_CERT",
-      "UNABLE_TO_GET_CRL",
-      "UNABLE_TO_DECRYPT_CERT_SIGNATURE",
-      "UNABLE_TO_DECRYPT_CRL_SIGNATURE",
-      "UNABLE_TO_DECODE_ISSUER_PUBLIC_KEY",
-      "CERT_SIGNATURE_FAILURE",
-      "CRL_SIGNATURE_FAILURE",
-      "CERT_NOT_YET_VALID",
-      "CERT_HAS_EXPIRED",
-      "CRL_NOT_YET_VALID",
-      "CRL_HAS_EXPIRED",
-      "ERROR_IN_CERT_NOT_BEFORE_FIELD",
-      "ERROR_IN_CERT_NOT_AFTER_FIELD",
-      "ERROR_IN_CRL_LAST_UPDATE_FIELD",
-      "ERROR_IN_CRL_NEXT_UPDATE_FIELD",
-      "OUT_OF_MEM",
-      "DEPTH_ZERO_SELF_SIGNED_CERT",
-      "SELF_SIGNED_CERT_IN_CHAIN",
-      "UNABLE_TO_GET_ISSUER_CERT_LOCALLY",
-      "UNABLE_TO_VERIFY_LEAF_SIGNATURE",
-      "CERT_CHAIN_TOO_LONG",
-      "CERT_REVOKED",
-      "INVALID_CA",
-      "PATH_LENGTH_EXCEEDED",
-      "INVALID_PURPOSE",
-      "CERT_UNTRUSTED",
-      "CERT_REJECTED",
-      "HOSTNAME_MISMATCH"
-    ]);
-    module2.exports = (error2) => !denyList.has(error2 && error2.code);
-  }
-});
-
 // src/teamcity/errors.ts
-function isRetryableError2(error2) {
+function isRetryableError(error2) {
   if (error2 instanceof TeamCityAPIError) {
     if (error2 instanceof TeamCityNetworkError) {
       return true;
@@ -11836,6 +11795,47 @@ var init_errors = __esm({
   }
 });
 
+// node_modules/is-retry-allowed/index.js
+var require_is_retry_allowed = __commonJS({
+  "node_modules/is-retry-allowed/index.js"(exports2, module2) {
+    "use strict";
+    var denyList = /* @__PURE__ */ new Set([
+      "ENOTFOUND",
+      "ENETUNREACH",
+      // SSL errors from https://github.com/nodejs/node/blob/fc8e3e2cdc521978351de257030db0076d79e0ab/src/crypto/crypto_common.cc#L301-L328
+      "UNABLE_TO_GET_ISSUER_CERT",
+      "UNABLE_TO_GET_CRL",
+      "UNABLE_TO_DECRYPT_CERT_SIGNATURE",
+      "UNABLE_TO_DECRYPT_CRL_SIGNATURE",
+      "UNABLE_TO_DECODE_ISSUER_PUBLIC_KEY",
+      "CERT_SIGNATURE_FAILURE",
+      "CRL_SIGNATURE_FAILURE",
+      "CERT_NOT_YET_VALID",
+      "CERT_HAS_EXPIRED",
+      "CRL_NOT_YET_VALID",
+      "CRL_HAS_EXPIRED",
+      "ERROR_IN_CERT_NOT_BEFORE_FIELD",
+      "ERROR_IN_CERT_NOT_AFTER_FIELD",
+      "ERROR_IN_CRL_LAST_UPDATE_FIELD",
+      "ERROR_IN_CRL_NEXT_UPDATE_FIELD",
+      "OUT_OF_MEM",
+      "DEPTH_ZERO_SELF_SIGNED_CERT",
+      "SELF_SIGNED_CERT_IN_CHAIN",
+      "UNABLE_TO_GET_ISSUER_CERT_LOCALLY",
+      "UNABLE_TO_VERIFY_LEAF_SIGNATURE",
+      "CERT_CHAIN_TOO_LONG",
+      "CERT_REVOKED",
+      "INVALID_CA",
+      "PATH_LENGTH_EXCEEDED",
+      "INVALID_PURPOSE",
+      "CERT_UNTRUSTED",
+      "CERT_REJECTED",
+      "HOSTNAME_MISMATCH"
+    ]);
+    module2.exports = (error2) => !denyList.has(error2 && error2.code);
+  }
+});
+
 // src/teamcity/build-status-manager.ts
 var build_status_manager_exports = {};
 __export(build_status_manager_exports, {
@@ -16510,6 +16510,9 @@ function formatError(err, context) {
   };
 }
 
+// src/middleware/global-error-handler.ts
+init_errors();
+
 // src/utils/error-logger.ts
 var ErrorLogger = class _ErrorLogger {
   static instance;
@@ -16648,6 +16651,14 @@ var GlobalErrorHandler = class _GlobalErrorHandler {
    * Transform raw errors into structured MCP errors
    */
   transformError(error2, context) {
+    if (error2 instanceof TeamCityAPIError) {
+      return new MCPTeamCityError(
+        this.sanitizeErrorMessage(error2.message),
+        error2.statusCode ?? 500,
+        error2.code,
+        context.requestId
+      );
+    }
     if (error2 instanceof MCPToolError) {
       if (this.options.sanitizeErrors) {
         const sanitizedMessage = this.sanitizeErrorMessage(error2.message);
@@ -16787,20 +16798,20 @@ function isNetworkError(error2) {
 }
 var SAFE_HTTP_METHODS = ["get", "head", "options"];
 var IDEMPOTENT_HTTP_METHODS = SAFE_HTTP_METHODS.concat(["put", "delete"]);
-function isRetryableError(error2) {
+function isRetryableError2(error2) {
   return error2.code !== "ECONNABORTED" && (!error2.response || error2.response.status === 429 || error2.response.status >= 500 && error2.response.status <= 599);
 }
 function isSafeRequestError(error2) {
   if (!error2.config?.method) {
     return false;
   }
-  return isRetryableError(error2) && SAFE_HTTP_METHODS.indexOf(error2.config.method) !== -1;
+  return isRetryableError2(error2) && SAFE_HTTP_METHODS.indexOf(error2.config.method) !== -1;
 }
 function isIdempotentRequestError(error2) {
   if (!error2.config?.method) {
     return false;
   }
-  return isRetryableError(error2) && IDEMPOTENT_HTTP_METHODS.indexOf(error2.config.method) !== -1;
+  return isRetryableError2(error2) && IDEMPOTENT_HTTP_METHODS.indexOf(error2.config.method) !== -1;
 }
 function isNetworkOrIdempotentRequestError(error2) {
   return isNetworkError(error2) || isIdempotentRequestError(error2);
@@ -16947,11 +16958,12 @@ axiosRetry.isIdempotentRequestError = isIdempotentRequestError;
 axiosRetry.isNetworkOrIdempotentRequestError = isNetworkOrIdempotentRequestError;
 axiosRetry.exponentialDelay = exponentialDelay;
 axiosRetry.linearDelay = linearDelay;
-axiosRetry.isRetryableError = isRetryableError;
+axiosRetry.isRetryableError = isRetryableError2;
 var esm_default = axiosRetry;
 
 // src/teamcity/auth.ts
 var import_crypto2 = require("crypto");
+init_errors();
 function generateRequestId() {
   return (0, import_crypto2.randomUUID)();
 }
@@ -16972,36 +16984,6 @@ function addRequestId(config2) {
   });
   return config2;
 }
-function extractErrorDetails(error2) {
-  const requestId = error2.config?.requestId;
-  if (error2.response != null) {
-    const data = error2.response.data;
-    return {
-      code: data?.code ?? `HTTP_${error2.response.status}`,
-      message: data?.message ?? error2.message,
-      details: data?.details ?? JSON.stringify(data),
-      requestId,
-      statusCode: error2.response.status,
-      originalError: error2
-    };
-  } else if (error2.request != null) {
-    return {
-      code: "NO_RESPONSE",
-      message: "No response received from TeamCity server",
-      details: error2.message,
-      requestId,
-      originalError: error2
-    };
-  } else {
-    return {
-      code: "REQUEST_SETUP_ERROR",
-      message: "Error setting up the request",
-      details: error2.message,
-      requestId,
-      originalError: error2
-    };
-  }
-}
 function logResponse(response) {
   const requestId = response.config?.requestId;
   const meta = response.config._tcMeta;
@@ -17018,7 +17000,8 @@ function logResponse(response) {
   return response;
 }
 function logAndTransformError(error2) {
-  const teamcityError = extractErrorDetails(error2);
+  const requestId = error2.config?.requestId;
+  const tcError = TeamCityAPIError.fromAxiosError(error2, requestId);
   const meta = error2.config?._tcMeta;
   const duration = meta?.start ? Date.now() - meta.start : void 0;
   const sanitize = (val) => {
@@ -17032,14 +17015,14 @@ function logAndTransformError(error2) {
     }
   };
   error("TeamCity API request failed", void 0, {
-    requestId: teamcityError.requestId,
-    code: teamcityError.code,
-    message: sanitize(teamcityError.message),
-    statusCode: teamcityError.statusCode,
-    details: sanitize(teamcityError.details),
+    requestId: tcError.requestId,
+    code: tcError.code,
+    message: sanitize(tcError.message),
+    statusCode: tcError.statusCode,
+    details: sanitize(tcError.details),
     duration
   });
-  return Promise.reject(teamcityError);
+  return Promise.reject(tcError);
 }
 function validateToken(token) {
   if (!token || token.length === 0) {
@@ -39438,7 +39421,7 @@ var TeamCityAPI = class _TeamCityAPI {
       retryCondition: (error2) => {
         const reqId = error2?.config?.requestId;
         const tcError = TeamCityAPIError.fromAxiosError(error2, reqId);
-        return isRetryableError2(tcError);
+        return isRetryableError(tcError);
       }
     });
     this.axiosInstance.interceptors.request.use((config2) => addRequestId(config2));
@@ -40442,9 +40425,16 @@ var DEV_TOOLS = [
     description: "Fetch server metrics (CPU/memory/disk/load) if available",
     inputSchema: { type: "object", properties: {} },
     handler: async (_args) => {
-      const api = TeamCityAPI.getInstance();
-      const metrics = await api.server.getAllMetrics();
-      return json(metrics.data);
+      return runTool(
+        "get_server_metrics",
+        null,
+        async () => {
+          const api = TeamCityAPI.getInstance();
+          const metrics = await api.server.getAllMetrics();
+          return json(metrics.data);
+        },
+        {}
+      );
     },
     mode: "full"
   },
@@ -40453,9 +40443,16 @@ var DEV_TOOLS = [
     description: "Get TeamCity server info (version, build number, state)",
     inputSchema: { type: "object", properties: {} },
     handler: async (_args) => {
-      const api = TeamCityAPI.getInstance();
-      const info2 = await api.server.getServerInfo();
-      return json(info2.data);
+      return runTool(
+        "get_server_info",
+        null,
+        async () => {
+          const api = TeamCityAPI.getInstance();
+          const info2 = await api.server.getServerInfo();
+          return json(info2.data);
+        },
+        {}
+      );
     }
   },
   {