@daghis/teamcity-mcp 1.6.0 → 1.8.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.8.0](https://github.com/Daghis/teamcity-mcp/compare/v1.7.0...v1.8.0) (2025-09-20)
+
+
+### Features
+
+* **tools:** support streaming artifacts in get_build_results ([#173](https://github.com/Daghis/teamcity-mcp/issues/173)) ([f95c5e4](https://github.com/Daghis/teamcity-mcp/commit/f95c5e4da9ffa80adc8b8f322c127a440c4524b5)), closes [#169](https://github.com/Daghis/teamcity-mcp/issues/169)
+
+## [1.7.0](https://github.com/Daghis/teamcity-mcp/compare/v1.6.0...v1.7.0) (2025-09-20)
+
+
+### Features
+
+* **tools:** add streaming mode to fetch_build_log ([#171](https://github.com/Daghis/teamcity-mcp/issues/171)) ([1abce69](https://github.com/Daghis/teamcity-mcp/commit/1abce69b7fa3866ac289501e2369d5d06d57d57f))
+
 ## [1.6.0](https://github.com/Daghis/teamcity-mcp/compare/v1.5.0...v1.6.0) (2025-09-20)
 
 

package/dist/index.js

@@ -1771,6 +1771,7 @@ var BuildResultsManager = class _BuildResultsManager {
    */
   async fetchArtifacts(buildId, options) {
     try {
+      const encoding = options.artifactEncoding ?? "base64";
       const response = await this.client.modules.builds.getFilesListOfBuild(
         toBuildLocator(buildId)
       );
@@ -1783,6 +1784,7 @@ var BuildResultsManager = class _BuildResultsManager {
         artifacts.map(async (artifact) => {
           const artifactPath = artifact.fullName ?? artifact.name;
           const downloadHref = artifact.content?.href ?? `/app/rest/builds/id:${buildId}/artifacts/content/${artifactPath}`;
+          const shouldInlineContent = encoding === "base64" && (options.downloadArtifacts?.length ? options.downloadArtifacts.includes(artifact.name) || options.downloadArtifacts.includes(artifactPath) : true);
           const artifactData = {
             name: artifact.name,
             path: artifactPath,
@@ -1790,7 +1792,7 @@ var BuildResultsManager = class _BuildResultsManager {
             modificationTime: artifact.modificationTime ?? "",
             downloadUrl: this.buildAbsoluteUrl(downloadHref)
           };
-          if (options.downloadArtifacts?.includes(artifact.name)) {
+          if (shouldInlineContent) {
             const maxSize = options.maxArtifactSize ?? _BuildResultsManager.defaultMaxArtifactSize;
             if ((artifact.size ?? 0) <= maxSize) {
               try {
@@ -1799,6 +1801,16 @@ var BuildResultsManager = class _BuildResultsManager {
               } catch (err) {
               }
             }
+          } else if (encoding === "stream") {
+            artifactData.downloadHandle = {
+              tool: "download_build_artifact",
+              args: {
+                buildId,
+                artifactPath,
+                encoding: "stream",
+                ...options.maxArtifactSize ? { maxSize: options.maxArtifactSize } : {}
+              }
+            };
           }
           return artifactData;
         })
@@ -2117,6 +2129,7 @@ function createAdapterFromTeamCityAPI(api, options = {}) {
     builds: buildApi,
     listBuildArtifacts: (buildId, options2) => api.listBuildArtifacts(buildId, options2),
     downloadArtifactContent: (buildId, artifactPath, requestOptions) => api.downloadBuildArtifact(buildId, artifactPath, requestOptions),
+    downloadBuildLogContent: (buildId, requestOptions) => api.downloadBuildLog(buildId, requestOptions),
     getBuildStatistics: (buildId, fields) => api.getBuildStatistics(buildId, fields),
     listChangesForBuild: (buildId, fields) => api.listChangesForBuild(buildId, fields),
     listSnapshotDependencies: (buildId) => api.listSnapshotDependencies(buildId),
@@ -36696,6 +36709,26 @@ var TeamCityAPI = class _TeamCityAPI {
       requestOptions
     );
   }
+  async downloadBuildLog(buildId, options) {
+    const rawParams = options?.params ?? void 0;
+    const params = rawParams ? { ...rawParams } : {};
+    if (!Object.prototype.hasOwnProperty.call(params, "plain")) {
+      params["plain"] = true;
+    }
+    const rawHeaders = options?.headers ?? void 0;
+    const headers = rawHeaders ? { ...rawHeaders } : {};
+    const requestOptions = {
+      ...options,
+      params,
+      headers: {
+        Accept: "text/plain",
+        ...headers
+      },
+      responseType: options?.responseType ?? "text",
+      transformResponse: options?.transformResponse ?? [(data) => data]
+    };
+    return this.axiosInstance.get(`/app/rest/builds/id:${buildId}/log`, requestOptions);
+  }
   async getBuildStatistics(buildId, fields) {
     return this.builds.getBuildStatisticValues(toBuildLocator(buildId), fields);
   }
@@ -36764,6 +36797,7 @@ var TeamCityAPI = class _TeamCityAPI {
 };
 
 // src/tools.ts
+var isReadableStream = (value) => typeof value === "object" && value !== null && typeof value.pipe === "function";
 function getMCPMode2() {
   return getMCPMode();
 }
@@ -37160,7 +37194,17 @@ var DEV_TOOLS = [
         pageSize: { type: "number", description: "Lines per page (default 500)" },
         startLine: { type: "number", description: "0-based start line (overrides page)" },
         lineCount: { type: "number", description: "Max lines to return (overrides pageSize)" },
-        tail: { type: "boolean", description: "Tail mode: return last N lines" }
+        tail: { type: "boolean", description: "Tail mode: return last N lines" },
+        encoding: {
+          type: "string",
+          description: "Response encoding: 'text' (default) or 'stream'",
+          enum: ["text", "stream"],
+          default: "text"
+        },
+        outputPath: {
+          type: "string",
+          description: "Optional absolute path to write streamed logs; defaults to a temp file when streaming"
+        }
       },
       required: []
     },
@@ -37173,9 +37217,24 @@ var DEV_TOOLS = [
         pageSize: import_zod4.z.number().int().min(1).max(5e3).optional(),
         startLine: import_zod4.z.number().int().min(0).optional(),
         lineCount: import_zod4.z.number().int().min(1).max(5e3).optional(),
-        tail: import_zod4.z.boolean().optional()
-      }).refine((v) => Boolean(v.buildId) || Boolean(v.buildNumber), {
-        message: "Provide either buildId or buildNumber"
+        tail: import_zod4.z.boolean().optional(),
+        encoding: import_zod4.z.enum(["text", "stream"]).default("text"),
+        outputPath: import_zod4.z.string().min(1).optional()
+      }).superRefine((value, ctx) => {
+        if (!value.buildId && typeof value.buildNumber === "undefined") {
+          ctx.addIssue({
+            code: import_zod4.z.ZodIssueCode.custom,
+            message: "Provide either buildId or buildNumber",
+            path: ["buildId"]
+          });
+        }
+        if (value.encoding === "stream" && value.tail) {
+          ctx.addIssue({
+            code: import_zod4.z.ZodIssueCode.custom,
+            message: "Streaming mode does not support tail queries",
+            path: ["tail"]
+          });
+        }
       });
       return runTool(
         "fetch_build_log",
@@ -37252,10 +37311,22 @@ var DEV_TOOLS = [
             }
             return false;
           };
+          const normalizeError = (error2) => {
+            if ((0, import_axios35.isAxiosError)(error2)) {
+              const status = error2.response?.status;
+              const statusText = (error2.response?.statusText ?? "").trim();
+              const base = status ? `${status}${statusText ? ` ${statusText}` : ""}` : error2.message;
+              return new Error(base || "Request failed");
+            }
+            if (error2 instanceof Error) {
+              return error2;
+            }
+            return new Error(String(error2));
+          };
           const wait = (ms) => new Promise((resolve) => {
             setTimeout(resolve, ms);
           });
-          const attemptFetch = async () => {
+          const attemptBuffered = async () => {
             if (typed.tail) {
               const count = typed.lineCount ?? typed.pageSize ?? 500;
               const full = await adapter.getBuildLog(effectiveBuildId);
@@ -37303,16 +37374,52 @@ var DEV_TOOLS = [
               }
             });
           };
-          const maxAttempts = typed.tail ? 5 : 3;
+          const attemptStream = async () => {
+            const effectivePageSize = typed.lineCount ?? typed.pageSize ?? 500;
+            const startLine = typeof typed.startLine === "number" ? typed.startLine : ((typed.page ?? 1) - 1) * effectivePageSize;
+            const response = await adapter.downloadBuildLogContent(effectiveBuildId, {
+              params: {
+                start: startLine,
+                count: effectivePageSize
+              },
+              responseType: "stream"
+            });
+            const stream = response.data;
+            if (!isReadableStream(stream)) {
+              throw new Error("Streaming log download did not return a readable stream");
+            }
+            const safeBuildId = effectiveBuildId.replace(/[^a-zA-Z0-9._-]/g, "_") || "build";
+            const defaultFileName = `build-log-${safeBuildId}-${startLine}-${(0, import_node_crypto.randomUUID)()}.log`;
+            const targetPath = typed.outputPath ?? (0, import_node_path.join)((0, import_node_os.tmpdir)(), defaultFileName);
+            await import_node_fs.promises.mkdir((0, import_node_path.dirname)(targetPath), { recursive: true });
+            await (0, import_promises.pipeline)(stream, (0, import_node_fs.createWriteStream)(targetPath));
+            const stats = await import_node_fs.promises.stat(targetPath);
+            const page = Math.floor(startLine / effectivePageSize) + 1;
+            return json({
+              encoding: "stream",
+              outputPath: targetPath,
+              bytesWritten: stats.size,
+              meta: {
+                buildId: effectiveBuildId,
+                buildNumber: typeof typed.buildNumber !== "undefined" ? String(typed.buildNumber) : void 0,
+                buildTypeId: typed.buildTypeId,
+                page,
+                pageSize: effectivePageSize,
+                startLine
+              }
+            });
+          };
+          const isStream = typed.encoding === "stream";
+          const maxAttempts = isStream ? 3 : typed.tail ? 5 : 3;
           for (let attempt = 0; attempt < maxAttempts; attempt += 1) {
             try {
-              return await attemptFetch();
+              return await (isStream ? attemptStream() : attemptBuffered());
             } catch (error2) {
               if (shouldRetry2(error2) && attempt < maxAttempts - 1) {
                 await wait(500 * (attempt + 1));
                 continue;
               }
-              throw error2;
+              throw normalizeError(error2);
             }
           }
           throw new Error("Unable to fetch build log after retries");
@@ -38254,6 +38361,12 @@ var DEV_TOOLS = [
         maxArtifactSize: {
           type: "number",
           description: "Max artifact content size (bytes) when inlining"
+        },
+        artifactEncoding: {
+          type: "string",
+          description: "Encoding mode for artifacts when includeArtifacts is true",
+          enum: ["base64", "stream"],
+          default: "base64"
         }
       },
       required: ["buildId"]
@@ -38266,7 +38379,8 @@ var DEV_TOOLS = [
         includeChanges: import_zod4.z.boolean().optional(),
         includeDependencies: import_zod4.z.boolean().optional(),
         artifactFilter: import_zod4.z.string().min(1).optional(),
-        maxArtifactSize: import_zod4.z.number().int().min(1).optional()
+        maxArtifactSize: import_zod4.z.number().int().min(1).optional(),
+        artifactEncoding: import_zod4.z.enum(["base64", "stream"]).default("base64")
       });
       return runTool(
         "get_build_results",
@@ -38280,7 +38394,8 @@ var DEV_TOOLS = [
             includeChanges: typed.includeChanges,
             includeDependencies: typed.includeDependencies,
             artifactFilter: typed.artifactFilter,
-            maxArtifactSize: typed.maxArtifactSize
+            maxArtifactSize: typed.maxArtifactSize,
+            artifactEncoding: typed.artifactEncoding
           });
           return json(result);
         },
@@ -38321,7 +38436,6 @@ var DEV_TOOLS = [
         maxSize: import_zod4.z.number().int().positive().optional(),
         outputPath: import_zod4.z.string().min(1).optional()
       });
-      const isReadableStream = (value) => typeof value === "object" && value !== null && typeof value.pipe === "function";
       const toTempFilePath = (artifactName) => {
         const base = (0, import_node_path.basename)(artifactName || "artifact");
         const safeStem = base.replace(/[^a-zA-Z0-9._-]/g, "_") || "artifact";