@stubbedev/atlassian-mcp 0.2.9 → 0.2.11

package/dist/bitbucket.js

@@ -340,6 +340,37 @@ export class BitbucketClient {
         const lines = data.values.map((r, i) => `${start + i + 1}. ${r.project.key}/${r.slug} — ${r.name}`);
         return text(`${data.values.length} repo(s)${pageHint(data)}:\n${lines.join('\n')}`);
     }
+    async searchUsers(args) {
+        const params = new URLSearchParams();
+        if (args.query)
+            params.set('filter', args.query);
+        params.set('limit', String(args.limit ?? 25));
+        if (args.start)
+            params.set('start', String(args.start));
+        let path;
+        if (args.projectKey && args.repoSlug) {
+            path = `${this.rp(args.projectKey, args.repoSlug)}/permissions/users?${params}`;
+        }
+        else if (args.projectKey) {
+            path = `/projects/${encodeURIComponent(args.projectKey)}/permissions/users?${params}`;
+        }
+        else {
+            path = `/users?${params}`;
+        }
+        const data = await this.request('GET', path);
+        if (!data || data.values.length === 0)
+            return text('No users found.');
+        const lines = data.values.map((entry, i) => {
+            const user = entry.user ?? entry;
+            const parts = [`${i + 1}. ${user.displayName} (${user.name})`];
+            if (user.emailAddress)
+                parts.push(`— ${user.emailAddress}`);
+            if (user.active === false)
+                parts.push('[inactive]');
+            return parts.join(' ');
+        });
+        return text(`${data.values.length} user(s)${pageHint(data)}:\n${lines.join('\n')}`);
+    }
     async listPullRequests(args) {
         const { projectKey, repoSlug } = this.resolveProjectAndRepo(args.projectKey, args.repoSlug);
         const { state = 'OPEN', fromBranch, text: searchText, limit = 25, start = 0 } = args;
@@ -972,7 +1003,7 @@ export class BitbucketClient {
         return text(`${data.values.length} task(s) on PR #${args.prId} (${open} open)${pageHint(data)}:\n${lines.join('\n')}`);
     }
     async mutatePrTask(args) {
-        const { projectKey, repoSlug } = this.resolveProjectAndRepo(args.projectKey, args.repoSlug);
+        this.resolveProjectAndRepo(args.projectKey, args.repoSlug);
         if (args.action === 'create') {
             if (!args.text)
                 throw new Error('text is required to create a task.');

package/dist/index.js

@@ -263,17 +263,18 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
         ] : []),
         ...(bitbucket ? [{
                 name: 'bitbucket_search',
-                description: 'Discover Bitbucket resources. Use when asked "what PRs are open?", "show me the repos", "find the PR for this branch", or "list branches". Set resource:\n• "pull_requests" (default) — list PRs by state/branch/text; mine=true for your inbox\n• "repos" — list repositories in a project\n• "branches" — list or filter branches in a repo',
+                description: 'Discover Bitbucket resources. Use when asked "what PRs are open?", "show me the repos", "find the PR for this branch", or "list branches". Set resource:\n• "pull_requests" (default) — list PRs by state/branch/text; mine=true for your inbox\n• "repos" — list repositories in a project\n• "branches" — list or filter branches in a repo\n• "users" — find users by name/email (pass query); add projectKey+repoSlug to restrict to users with repo access. ALWAYS use this to look up valid usernames before adding reviewers to a PR.',
                 inputSchema: {
                     type: 'object',
                     properties: {
-                        resource: { type: 'string', enum: ['pull_requests', 'repos', 'branches'], description: 'What to search (default: pull_requests)' },
+                        resource: { type: 'string', enum: ['pull_requests', 'repos', 'branches', 'users'], description: 'What to search (default: pull_requests)' },
                         mine: { type: 'boolean', description: 'Return your own PRs by role (resource=pull_requests only)' },
                         role: { type: 'string', enum: ['author', 'reviewer', 'participant'], description: 'Your role filter when mine=true' },
                         projectKey: { type: 'string', description: 'Bitbucket project code, e.g. "ENG"' },
                         project: { type: 'string', description: 'Alias for projectKey' },
                         repoSlug: { type: 'string', description: 'Repository slug' },
                         repo: { type: 'string', description: 'Alias for repoSlug' },
+                        query: { type: 'string', description: 'Name or email filter (resource=users only)' },
                         state: { type: 'string', enum: ['OPEN', 'MERGED', 'DECLINED'], description: 'PR state filter (default OPEN)' },
                         fromBranch: { type: 'string', description: 'Filter PRs from this source branch' },
                         text: { type: 'string', description: 'Filter PRs by title/description text' },
@@ -285,7 +286,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
             },
             {
                 name: 'bitbucket_get_pr',
-                description: 'Full details for one PR: metadata, commits, open comments, blockers, and optional diff. Use when asked to "review this PR", "show me the review comments", "what\'s blocking the merge", or after get_dev_context surfaces a prId. For full file context during review, follow up with bitbucket_get_file. The response includes a "Viewing as" line — if it says "you are the author", do NOT add review comments or a summary unless explicitly asked; just answer questions about the PR. If it says "you are a reviewer", default to posting inline comments for suggested changes and a final summary comment.',
+                description: 'Full details for one PR: metadata, commits, open comments, blockers, and optional diff. Use when asked to "review this PR", "show me the review comments", "what\'s blocking the merge", or after get_dev_context surfaces a prId. IMPORTANT: The PR branch is often not the locally checked-out branch. Do NOT read files with local tools (Read, git_get_diff, etc.) for PR context — use bitbucket_get_file with the PR\'s source branch instead. The response includes a "Viewing as" line — if it says "you are the author", do NOT add review comments or a summary unless explicitly asked; just answer questions about the PR. If it says "you are a reviewer", default to posting inline comments for suggested changes and a final summary comment.',
                 inputSchema: {
                     type: 'object',
                     properties: {
@@ -330,7 +331,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
                                 description: { type: 'string', description: 'PR description (optional)' },
                                 fromBranch: { type: 'string', description: 'Source branch (defaults to current branch)' },
                                 toBranch: { type: 'string', description: 'Target branch (default: master)' },
-                                reviewers: { type: 'array', items: { type: 'string' }, description: 'Reviewer usernames (optional)' },
+                                reviewers: { type: 'array', items: { type: 'string' }, description: 'Reviewer usernames. Use bitbucket_search resource=users to look up valid usernames before setting this.' },
                             },
                             required: ['title'],
                         },
@@ -340,7 +341,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
                                 title: { type: 'string', description: 'Updated PR title (optional)' },
                                 description: { type: 'string', description: 'Updated description, or empty string to clear (optional)' },
                                 toBranch: { type: 'string', description: 'Updated target branch (optional)' },
-                                reviewers: { type: 'array', items: { type: 'string' }, description: 'Updated reviewer usernames. Empty array clears reviewers.' },
+                                reviewers: { type: 'array', items: { type: 'string' }, description: 'Updated reviewer usernames. Empty array clears reviewers. Use bitbucket_search resource=users to look up valid usernames before setting this.' },
                             },
                         },
                     },
@@ -377,7 +378,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
             },
             {
                 name: 'bitbucket_get_file',
-                description: 'Raw file content from Bitbucket at a branch, tag, or commit. Use during PR reviews to get full-file context without relying on local checkout.',
+                description: 'Raw file content from Bitbucket at a branch, tag, or commit. CRITICAL: if the PR branch being reviewed is NOT the currently checked-out local branch, ALL additional file context for that review MUST come from this tool — never from local Read, git_get_diff, or any tool that reads local disk. Pass the PR source branch as ref.',
                 inputSchema: {
                     type: 'object',
                     properties: {
@@ -611,6 +612,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     return await bitbucket.listRepos(a);
                 if (resource === 'branches')
                     return await bitbucket.getBranches(a);
+                if (resource === 'users')
+                    return await bitbucket.searchUsers({ projectKey: a.projectKey, repoSlug: a.repoSlug, query: a.query, limit: a.limit, start: a.start });
                 // pull_requests (default)
                 if (a.mine)
                     return await bitbucket.myPrs({ limit: a.limit, start: a.start, role: a.role });

package/dist/jira.js

@@ -95,6 +95,7 @@ export class JiraClient {
     headers;
     currentUserCache;
     projectsCache;
+    issueLinkingEnabled;
     constructor(baseUrl, token) {
         this.baseUrl = baseUrl.replace(/\/$/, '');
         this.headers = {
@@ -147,6 +148,13 @@ export class JiraClient {
         this.currentUserCache = me;
         return me;
     }
+    async getIssueLinkingEnabled() {
+        if (this.issueLinkingEnabled !== undefined)
+            return this.issueLinkingEnabled;
+        const config = await this.request('GET', '/configuration');
+        this.issueLinkingEnabled = config?.issueLinkingEnabled ?? false;
+        return this.issueLinkingEnabled;
+    }
     async assertOwnComment(comment) {
         const me = await this.getCurrentUser();
         const commentAuthorName = this.normalizeIdentity(comment.author.name);
@@ -602,14 +610,20 @@ export class JiraClient {
             await this.request('POST', `/issue/${encodeURIComponent(issueKey)}/comment`, { body: validateCommentBody(args.comment) });
             actions.push('added comment');
         }
+        const warnings = [];
         if (args.link) {
-            const dir = args.link.direction ?? 'outward';
-            await this.request('POST', '/issueLink', {
-                type: { name: args.link.linkType },
-                outwardIssue: { key: dir === 'outward' ? issueKey : args.link.targetIssueKey },
-                inwardIssue: { key: dir === 'outward' ? args.link.targetIssueKey : issueKey },
-            });
-            actions.push(`linked ${args.link.linkType} → ${args.link.targetIssueKey}`);
+            if (!(await this.getIssueLinkingEnabled())) {
+                warnings.push(`issue linking is disabled in this Jira instance — add the link manually`);
+            }
+            else {
+                const dir = args.link.direction ?? 'outward';
+                await this.request('POST', '/issueLink', {
+                    type: { name: args.link.linkType },
+                    outwardIssue: { key: dir === 'outward' ? issueKey : args.link.targetIssueKey },
+                    inwardIssue: { key: dir === 'outward' ? args.link.targetIssueKey : issueKey },
+                });
+                actions.push(`linked ${args.link.linkType} → ${args.link.targetIssueKey}`);
+            }
         }
         if (args.worklog) {
             const wBody = { timeSpent: args.worklog.timeSpent };
@@ -623,7 +637,10 @@ export class JiraClient {
         if (actions.length === 0) {
             return text('Nothing to mutate.');
         }
-        return text(`Mutated ${issueKey}: ${actions.join(', ')}.\n${this.issueUrl(issueKey)}`);
+        const parts = [`Mutated ${issueKey}: ${actions.join(', ')}.`, this.issueUrl(issueKey)];
+        if (warnings.length)
+            parts.push(`Warnings: ${warnings.join('; ')}`);
+        return text(parts.join('\n'));
     }
     async getComments(args) {
         const { issueKey, maxResults = 50, startAt = 0 } = args;

package/package.json

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