@ecubelabs/atlassian-mcp 1.9.0 → 1.10.0

package/dist/jira-tools-register.js

@@ -1,5 +1,6 @@
 import { z } from 'zod';
 import { JiraService } from './libs/jira-client.js';
+import { formatToolResponse } from './libs/response-formatter.js';
 export const registerJiraTools = (server) => {
     // Initialize Jira service
     const jiraService = new JiraService();
@@ -22,14 +23,10 @@ export const registerJiraTools = (server) => {
     }, async ({ issueKey, expand }) => {
         try {
             const issue = await jiraService.getIssue(issueKey, expand);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(issue),
-                    },
-                ],
-            };
+            return formatToolResponse(issue, {
+                toolName: 'get-issue',
+                summary: `Issue ${issue.key}: ${issue.fields?.summary ?? ''} (Status: ${issue.fields?.status?.name ?? 'unknown'})`,
+            });
         }
         catch (error) {
             return {
@@ -102,14 +99,12 @@ export const registerJiraTools = (server) => {
                 failFast,
                 reconcileIssues,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            const issues = result.issues ?? [];
+            const keys = issues.map((i) => i.key).join(', ');
+            return formatToolResponse(result, {
+                toolName: 'search-issues',
+                summary: `Found ${issues.length} issues (total: ${result.total ?? 'unknown'}): ${keys}`,
+            });
         }
         catch (error) {
             return {
@@ -151,14 +146,10 @@ export const registerJiraTools = (server) => {
                 orderBy,
                 expand,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-comments',
+                summary: `${result.comments?.length ?? 0} comments for ${issueKey} (total: ${result.total ?? 'unknown'})`,
+            });
         }
         catch (error) {
             return {
@@ -247,14 +238,9 @@ export const registerJiraTools = (server) => {
                 expand,
                 status,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-projects',
+            });
         }
         catch (error) {
             return {
@@ -272,14 +258,9 @@ export const registerJiraTools = (server) => {
     }, async ({ projectIdOrKey }) => {
         try {
             const result = await jiraService.getCreateMetadataIssueTypes(projectIdOrKey);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-create-metadata-issue-types',
+            });
         }
         catch (error) {
             return {
@@ -298,14 +279,9 @@ export const registerJiraTools = (server) => {
     }, async ({ projectIdOrKey, issueTypeId }) => {
         try {
             const result = await jiraService.getCreateFieldMetadata(projectIdOrKey, issueTypeId);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-create-field-metadata',
+            });
         }
         catch (error) {
             return {
@@ -333,14 +309,9 @@ export const registerJiraTools = (server) => {
                 startAt,
                 maxResults,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-fields',
+            });
         }
         catch (error) {
             return {
@@ -362,14 +333,9 @@ export const registerJiraTools = (server) => {
     }, async ({ query, validation }) => {
         try {
             const result = await jiraService.parseJql(query, validation);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'parse-jql',
+            });
         }
         catch (error) {
             return {
@@ -413,14 +379,9 @@ export const registerJiraTools = (server) => {
                 labels,
                 additionalFields,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(issue),
-                    },
-                ],
-            };
+            return formatToolResponse(issue, {
+                toolName: 'create-issue',
+            });
         }
         catch (error) {
             return {
@@ -523,14 +484,9 @@ export const registerJiraTools = (server) => {
                 maxResults,
                 property,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(users),
-                    },
-                ],
-            };
+            return formatToolResponse(users, {
+                toolName: 'search-users',
+            });
         }
         catch (error) {
             return {
@@ -562,14 +518,9 @@ export const registerJiraTools = (server) => {
                 avatarSize,
                 excludeConnectUsers,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-user-picker',
+            });
         }
         catch (error) {
             return {
@@ -599,14 +550,9 @@ export const registerJiraTools = (server) => {
                 startAt,
                 maxResults,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(users),
-                    },
-                ],
-            };
+            return formatToolResponse(users, {
+                toolName: 'search-assignable-users',
+            });
         }
         catch (error) {
             return {
@@ -640,14 +586,9 @@ export const registerJiraTools = (server) => {
                 startAt,
                 maxResults,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(users),
-                    },
-                ],
-            };
+            return formatToolResponse(users, {
+                toolName: 'search-users-by-permission',
+            });
         }
         catch (error) {
             return {
@@ -668,14 +609,9 @@ export const registerJiraTools = (server) => {
     }, async ({ expand }) => {
         try {
             const user = await jiraService.getMyself(expand);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(user),
-                    },
-                ],
-            };
+            return formatToolResponse(user, {
+                toolName: 'get-myself',
+            });
         }
         catch (error) {
             return {
@@ -709,14 +645,9 @@ export const registerJiraTools = (server) => {
                 projectConfigurationUuid,
                 commentId,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-my-permissions',
+            });
         }
         catch (error) {
             return {
@@ -742,14 +673,9 @@ export const registerJiraTools = (server) => {
                 timeZone,
                 locale,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(user),
-                    },
-                ],
-            };
+            return formatToolResponse(user, {
+                toolName: 'update-myself',
+            });
         }
         catch (error) {
             return {
@@ -767,14 +693,9 @@ export const registerJiraTools = (server) => {
     }, async ({ key }) => {
         try {
             const preferences = await jiraService.getMyPreferences(key);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(preferences),
-                    },
-                ],
-            };
+            return formatToolResponse(preferences, {
+                toolName: 'get-my-preferences',
+            });
         }
         catch (error) {
             return {
@@ -793,14 +714,9 @@ export const registerJiraTools = (server) => {
     }, async ({ key, value }) => {
         try {
             const result = await jiraService.updateMyPreferences(key, value);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'update-my-preferences',
+            });
         }
         catch (error) {
             return {
@@ -816,14 +732,9 @@ export const registerJiraTools = (server) => {
     server.tool('get-my-locale', "Get current user's locale setting", {}, async () => {
         try {
             const locale = await jiraService.getMyLocale();
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(locale),
-                    },
-                ],
-            };
+            return formatToolResponse(locale, {
+                toolName: 'get-my-locale',
+            });
         }
         catch (error) {
             return {
@@ -841,14 +752,9 @@ export const registerJiraTools = (server) => {
     }, async ({ locale }) => {
         try {
             const result = await jiraService.updateMyLocale(locale);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'update-my-locale',
+            });
         }
         catch (error) {
             return {
@@ -864,14 +770,9 @@ export const registerJiraTools = (server) => {
     server.tool('get-priorities', 'Get all priorities', {}, async () => {
         try {
             const priorities = await jiraService.getPriorities();
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(priorities),
-                    },
-                ],
-            };
+            return formatToolResponse(priorities, {
+                toolName: 'get-priorities',
+            });
         }
         catch (error) {
             return {
@@ -893,14 +794,9 @@ export const registerJiraTools = (server) => {
     }, async ({ issueKey, expand }) => {
         try {
             const result = await jiraService.getIssueTransitions(issueKey, expand);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-issue-transitions',
+            });
         }
         catch (error) {
             return {
@@ -972,14 +868,9 @@ export const registerJiraTools = (server) => {
     server.tool('get-statuses', 'Get all workflow statuses', {}, async () => {
         try {
             const statuses = await jiraService.getStatuses();
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(statuses),
-                    },
-                ],
-            };
+            return formatToolResponse(statuses, {
+                toolName: 'get-statuses',
+            });
         }
         catch (error) {
             return {
@@ -997,14 +888,9 @@ export const registerJiraTools = (server) => {
     }, async ({ idOrName }) => {
         try {
             const status = await jiraService.getStatus(idOrName);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(status),
-                    },
-                ],
-            };
+            return formatToolResponse(status, {
+                toolName: 'get-status',
+            });
         }
         catch (error) {
             return {
@@ -1032,14 +918,9 @@ export const registerJiraTools = (server) => {
                 projectId,
                 expand,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'search-statuses',
+            });
         }
         catch (error) {
             return {
@@ -1057,14 +938,9 @@ export const registerJiraTools = (server) => {
     }, async ({ id }) => {
         try {
             const priority = await jiraService.getPriority(id);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(priority),
-                    },
-                ],
-            };
+            return formatToolResponse(priority, {
+                toolName: 'get-priority',
+            });
         }
         catch (error) {
             return {
@@ -1094,14 +970,9 @@ export const registerJiraTools = (server) => {
                 priorityName,
                 onlyDefault,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'search-priorities',
+            });
         }
         catch (error) {
             return {
@@ -1138,14 +1009,9 @@ export const registerJiraTools = (server) => {
                 visibility,
                 properties: properties,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(comment),
-                    },
-                ],
-            };
+            return formatToolResponse(comment, {
+                toolName: 'create-comment',
+            });
         }
         catch (error) {
             return {
@@ -1183,14 +1049,9 @@ export const registerJiraTools = (server) => {
                 visibility,
                 properties: properties,
             });
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(comment),
-                    },
-                ],
-            };
+            return formatToolResponse(comment, {
+                toolName: 'update-comment',
+            });
         }
         catch (error) {
             return {
@@ -1258,14 +1119,9 @@ export const registerJiraTools = (server) => {
     server.tool('get-issue-link-types', 'Get available issue link types', {}, async () => {
         try {
             const result = await jiraService.getIssueLinkTypes();
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-issue-link-types',
+            });
         }
         catch (error) {
             return {
@@ -1283,14 +1139,9 @@ export const registerJiraTools = (server) => {
     }, async ({ issueLinkTypeId }) => {
         try {
             const result = await jiraService.getIssueLinkType(issueLinkTypeId);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-issue-link-type',
+            });
         }
         catch (error) {
             return {
@@ -1393,14 +1244,9 @@ export const registerJiraTools = (server) => {
     }, async ({ linkId }) => {
         try {
             const result = await jiraService.getIssueLink(linkId);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-issue-link',
+            });
         }
         catch (error) {
             return {
@@ -1443,14 +1289,11 @@ export const registerJiraTools = (server) => {
     }, async ({ issueKey }) => {
         try {
             const result = await jiraService.getIssueWithLinks(issueKey);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            const links = result.fields?.issuelinks ?? [];
+            return formatToolResponse(result, {
+                toolName: 'get-issue-with-links',
+                summary: `Issue ${result.key}: ${result.fields?.summary ?? ''} with ${links.length} links`,
+            });
         }
         catch (error) {
             return {
@@ -1468,14 +1311,9 @@ export const registerJiraTools = (server) => {
     }, async ({ issueKey }) => {
         try {
             const result = await jiraService.getIssueWatchers(issueKey);
-            return {
-                content: [
-                    {
-                        type: 'text',
-                        text: JSON.stringify(result),
-                    },
-                ],
-            };
+            return formatToolResponse(result, {
+                toolName: 'get-issue-watchers',
+            });
         }
         catch (error) {
             return {

package/dist/libs/confluence-client.js

@@ -395,7 +395,7 @@ export class ConfluenceService extends BaseApiService {
         const { targetId, position } = options;
         return this.makeRequest(() => this.v1Client.put(`/content/${pageId}/move/${position}/${targetId}`, null, {
             headers: {
-                'Accept': 'application/json',
+                Accept: 'application/json',
             },
         }));
     }

package/dist/libs/response-formatter.js

@@ -0,0 +1,49 @@
+import { writeFileSync, readdirSync, statSync, unlinkSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { randomUUID } from 'node:crypto';
+const DEFAULT_THRESHOLD = 20_000;
+const MAX_AGE_MS = 60 * 60 * 1000; // 1 hour
+export function formatToolResponse(data, options) {
+    const text = typeof data === 'string' ? data : JSON.stringify(data);
+    const threshold = options.threshold ?? DEFAULT_THRESHOLD;
+    if (text.length <= threshold) {
+        return { content: [{ type: 'text', text }] };
+    }
+    const fileName = `atlassian-mcp-${options.toolName}-${Date.now()}-${randomUUID().slice(0, 8)}.json`;
+    const filePath = join(tmpdir(), fileName);
+    writeFileSync(filePath, text, 'utf-8');
+    const message = [
+        `Response too large (${text.length.toLocaleString()} chars). Full data saved to temporary file.`,
+        ``,
+        `File: ${filePath}`,
+        ``,
+        `Use the Read tool to read this file for full content.`,
+        options.summary ? `\nSummary: ${options.summary}` : '',
+    ]
+        .filter(Boolean)
+        .join('\n');
+    return { content: [{ type: 'text', text: message }] };
+}
+export function cleanupOldTempFiles() {
+    try {
+        const dir = tmpdir();
+        const files = readdirSync(dir).filter((f) => f.startsWith('atlassian-mcp-') && f.endsWith('.json'));
+        const now = Date.now();
+        for (const file of files) {
+            try {
+                const filePath = join(dir, file);
+                const stat = statSync(filePath);
+                if (now - stat.mtimeMs > MAX_AGE_MS) {
+                    unlinkSync(filePath);
+                }
+            }
+            catch {
+                /* skip individual file errors */
+            }
+        }
+    }
+    catch {
+        /* non-critical, skip silently */
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ecubelabs/atlassian-mcp",
-  "version": "1.9.0",
+  "version": "1.10.0",
   "bin": "./dist/index.js",
   "repository": {
     "url": "https://github.com/Ecube-Labs/skynet.git"