@stubbedev/atlassian-mcp 0.1.9 → 0.1.11

package/dist/bitbucket.js

@@ -442,8 +442,8 @@ export class BitbucketClient {
         const existing = await this.request('GET', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`);
         if (!existing)
             throw new Error(`PR #${args.prId} not found.`);
-        const buildBody = (version) => {
-            const body = { version };
+        const buildBody = (pr) => {
+            const body = { version: pr.version };
             if (args.title !== undefined)
                 body.title = args.title;
             if (args.description !== undefined)
@@ -454,14 +454,16 @@ export class BitbucketClient {
                     repository: { slug: repoSlug, project: { key: projectKey } },
                 };
             }
-            if (args.reviewers !== undefined) {
-                body.reviewers = args.reviewers.map((name) => ({ user: { name } }));
-            }
+            // Always include reviewers to avoid Bitbucket clearing them on PUT.
+            // Only replace them when explicitly provided by the caller.
+            body.reviewers = args.reviewers !== undefined
+                ? args.reviewers.map((name) => ({ user: { name } }))
+                : pr.reviewers.map((r) => ({ user: { name: r.user.name } }));
             return body;
         };
         let updated;
         try {
-            updated = await this.request('PUT', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`, buildBody(existing.version));
+            updated = await this.request('PUT', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`, buildBody(existing));
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
@@ -470,7 +472,7 @@ export class BitbucketClient {
             const latest = await this.request('GET', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`);
             if (!latest)
                 throw error;
-            updated = await this.request('PUT', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`, buildBody(latest.version));
+            updated = await this.request('PUT', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`, buildBody(latest));
         }
         if (!updated)
             return text(`Updated PR #${args.prId}.`);
@@ -678,16 +680,40 @@ export class BitbucketClient {
     }
     async addPrComment(args) {
         const { projectKey, repoSlug } = this.resolveProjectAndRepo(args.projectKey, args.repoSlug);
-        const body = { text: validateCommentText(args.text) };
+        let commentText = args.text;
+        if (args.suggestion !== undefined) {
+            const suggestionBlock = `\`\`\`suggestion\n${args.suggestion}\n\`\`\``;
+            commentText = args.text ? `${args.text}\n\n${suggestionBlock}` : suggestionBlock;
+        }
+        const body = { text: validateCommentText(commentText) };
         if (args.parentCommentId)
             body.parent = { id: args.parentCommentId };
+        if (args.filePath !== undefined && args.line !== undefined) {
+            const pr = await this.request('GET', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}`);
+            const anchor = {
+                diffType: 'EFFECTIVE',
+                fileType: args.fileType ?? 'TO',
+                fromHash: pr?.fromRef.latestCommit ?? '',
+                toHash: pr?.toRef.latestCommit ?? '',
+                line: args.line,
+                lineType: args.lineType ?? 'ADDED',
+                path: args.filePath,
+                srcPath: args.srcPath ?? args.filePath,
+            };
+            if (args.multilineStartLine !== undefined) {
+                anchor.multilineStartLine = args.multilineStartLine;
+                anchor.multilineStartLineType = args.multilineStartLineType ?? args.lineType ?? 'ADDED';
+            }
+            body.anchor = anchor;
+        }
         const created = await this.request('POST', `/projects/${projectKey}/repos/${repoSlug}/pull-requests/${args.prId}/comments`, body);
         if (!created)
             return text(`Comment added to PR #${args.prId}.`);
         if (args.parentCommentId) {
             return text(`Reply #${created.id} added to comment #${args.parentCommentId} on PR #${args.prId}.`);
         }
-        return text(`Comment #${created.id} added to PR #${args.prId}.`);
+        const location = args.filePath && args.line ? ` on ${args.filePath}:${args.line}` : '';
+        return text(`Comment #${created.id} added to PR #${args.prId}${location}.`);
     }
     async updatePrComment(args) {
         const { projectKey, repoSlug } = this.resolveProjectAndRepo(args.projectKey, args.repoSlug);

package/dist/index.js

@@ -569,7 +569,13 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
         },
         {
             name: 'bitbucket_add_pr_comment',
-            description: 'Use when you want to add a PR review comment or reply to an existing thread. Keep comments concise, plain text, and free of filler. Never include emojis. You can pass projectKey/repoSlug or project/repo.',
+            description: `Add a PR review comment or reply to an existing thread.
+
+INLINE COMMENTS ARE STRONGLY PREFERRED: Whenever your comment refers to a specific line or block of code, you MUST provide filePath and line to anchor it as an inline comment on the diff. General top-level comments (no filePath/line) should only be used for overall PR feedback that does not relate to any particular line.
+
+SUGGESTIONS ARE STRONGLY PREFERRED OVER PLAIN COMMENTS: When you are pointing out something that should be changed, always provide a suggestion (the corrected code) rather than describing the change in words. A suggestion lets the author apply the fix with one click. Only omit suggestion if you are asking a question or raising a concern that has no clear single answer.
+
+Keep comments concise, plain text, and free of filler. Never include emojis. You can pass projectKey/repoSlug or project/repo.`,
             inputSchema: {
                 type: 'object',
                 properties: {
@@ -579,7 +585,15 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
                     repo: { type: 'string', description: 'Alias for repoSlug' },
                     prId: { type: 'number', description: 'Pull request number (PR ID)' },
                     parentCommentId: { type: 'number', description: 'Parent comment ID for reply mode (optional)' },
-                    text: { type: 'string', description: 'Concise comment text only. No filler. Do not include emojis.' },
+                    text: { type: 'string', description: 'Concise comment text. No filler. Do not include emojis. If suggestion is also provided, this text appears above the suggestion block.' },
+                    filePath: { type: 'string', description: 'Destination file path for inline comment, e.g. "src/index.ts". Must be provided together with line.' },
+                    srcPath: { type: 'string', description: 'Source file path. Only needed when the file was renamed; otherwise omit (defaults to filePath).' },
+                    line: { type: 'number', description: 'Line number in the file to anchor the comment to. Must be provided together with filePath.' },
+                    lineType: { type: 'string', enum: ['ADDED', 'REMOVED', 'CONTEXT'], description: 'The type of the anchored line in the diff. Defaults to ADDED. Use CONTEXT for unchanged lines, REMOVED for deleted lines.' },
+                    fileType: { type: 'string', enum: ['TO', 'FROM'], description: 'Which side of the diff the anchor refers to: TO (destination/new file, default) or FROM (source/old file).' },
+                    multilineStartLine: { type: 'number', description: 'First line of a multiline anchor. Set together with line (last line) to span multiple lines.' },
+                    multilineStartLineType: { type: 'string', enum: ['ADDED', 'REMOVED', 'CONTEXT'], description: 'Line type for the multilineStartLine. Defaults to lineType.' },
+                    suggestion: { type: 'string', description: 'Replacement code to suggest. Rendered as a suggestion block the author can apply with one click. STRONGLY PREFERRED whenever you are proposing a code change. Must be used with filePath and line.' },
                 },
                 required: ['prId', 'text'],
             },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stubbedev/atlassian-mcp",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "MCP server for self-hosted Jira and Bitbucket",
   "license": "MIT",
   "type": "module",