mcp-bitbucket-server 1.0.0 → 1.1.0

package/CLAUDE.md

@@ -7,7 +7,7 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 This is an MCP (Model Context Protocol) server for Bitbucket Server/Data Center integration. The server provides tools for:
 - **User management**: Get user profile, list all users
 - **Repository operations**: List repositories in a project
-- **Pull request operations**: Get changed files, get structured diffs, add comments (general, replies, and inline)
+- **Pull request operations**: Get changed files, get structured diffs, add comments (general, replies, and inline), update review status (approve/request changes)
 
 ## Architecture
 
@@ -65,6 +65,11 @@ src/
 ├── index.ts                     # Main entry point, MCP server setup
 ├── services/
 │   └── bitbucket.ts            # Axios client instance (export const bitbucketClient)
+├── types/
+│   ├── index.ts                # Barrel export for all types
+│   ├── common.ts               # Shared types (PaginatedResponse)
+│   ├── pull-request.ts         # Pull request types (RestComment, RestPullRequest, etc.)
+│   └── repository.ts           # Repository-specific types
 └── tools/
     ├── index.ts                # Tool registration
     ├── users/
@@ -83,9 +88,77 @@ src/
         ├── add_pr_comment.ts           # POST /projects/.../pull-requests/.../comments
         ├── get_pr_changes.ts           # GET /projects/.../pull-requests/.../changes
         ├── get_pr_file_diff.ts         # GET /projects/.../pull-requests/.../diff/{path}
-        └── get_pr_activities.ts        # GET /projects/.../pull-requests/.../activities
+        ├── get_pr_activities.ts        # GET /projects/.../pull-requests/.../activities
+        └── update_review_status.ts     # PUT /projects/.../pull-requests/.../participants/{userSlug}
 ```
 
+## TypeScript Types
+
+TypeScript type definitions for Bitbucket Server API responses are located in `src/types/`:
+
+```
+src/types/
+├── index.ts         # Barrel export (import from here)
+├── common.ts        # Shared types (PaginatedResponse)
+├── pull-request.ts  # Pull request types (RestComment, RestPullRequest, RestPullRequestParticipant, etc.)
+└── repository.ts    # Repository-specific types
+```
+
+### Type Naming Conventions
+
+- **Match Swagger schema names exactly**: `RestRepository`, `RestProject`, `RestPullRequest`
+- **Use interfaces for objects**: `interface RestRepository { ... }`
+- **Use union types for enums**: `type RepositoryState = "AVAILABLE" | "OFFLINE"`
+- **Generic wrapper types**: `PaginatedResponse<T>` for paginated endpoints
+- **Response type aliases**: `RepositoriesResponse = PaginatedResponse<RestRepository>`
+
+### Adding New Types
+
+1. **Find the schema in Swagger**:
+   ```bash
+   grep -n '"RestTypeName"' BitbucketServerSwagger.json
+   ```
+
+2. **Create/update type file** (e.g., `src/types/domain.ts`):
+   ```typescript
+   // Focus on readonly properties (what API returns)
+   export interface RestTypeName {
+     id: number;
+     name: string;
+     // ... other fields
+   }
+
+   export type DomainResponse = PaginatedResponse<RestTypeName>;
+   ```
+
+3. **Export from barrel file** (`src/types/index.ts`):
+   ```typescript
+   export type { RestTypeName, DomainResponse } from "./domain.js";
+   ```
+
+4. **Use in tool**:
+   ```typescript
+   import type { DomainResponse } from "../../types/index.js";
+
+   const response = await bitbucketClient.get<DomainResponse>("/endpoint");
+   ```
+
+### Type Design Principles
+
+Following the project's simplicity philosophy:
+
+**✅ DO:**
+- Map types directly from Swagger schemas
+- Focus on readonly properties (API responses)
+- Keep types simple and minimal
+- Use generic `PaginatedResponse<T>` for all paginated endpoints
+
+**❌ DON'T:**
+- Over-engineer with complex utility types
+- Include writeOnly properties (used for requests)
+- Add properties that aren't useful
+- Create unnecessary type abstractions
+
 ## Tool Development Workflow
 
 ### 1. Find the Endpoint in Swagger
@@ -115,6 +188,7 @@ Follow this simple pattern:
 import { z } from "zod";
 import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { bitbucketClient } from "../../services/bitbucket.js";
+import type { YourResponseType } from "../../types/index.js";
 
 const schema = z.object({
   requiredParam: z.string().describe("Description from Swagger"),
@@ -132,7 +206,7 @@ export const toolNameTool = (server: McpServer) => {
     async (params) => {
       const { requiredParam, optionalParam } = schema.parse(params);
 
-      const response = await bitbucketClient.get("/endpoint", {
+      const response = await bitbucketClient.get<YourResponseType>("/endpoint", {
         params: optionalParam ? { optionalParam } : {},
       });
 
@@ -164,6 +238,16 @@ export function registerTools(server: McpServer) {
 }
 ```
 
+### 5. Run the Linter
+
+Always run the linter after implementing a new tool:
+
+```bash
+npm run lint
+```
+
+The linter will auto-fix formatting, catch unused variables, and ensure code quality. Fix any warnings before committing.
+
 ## Code Style Guidelines
 
 ### Keep It Simple
@@ -364,6 +448,29 @@ export const getAllUsersTool = (server: McpServer) => {
 
 **Purpose**: Get an overview of PR activity. Can filter to specific types (e.g., only comments) to reduce response size. Each comment activity includes full comment text, author, and creation date. For inline comments, use `commentAnchor` (path, line, lineType, fileType) to fetch the relevant diff separately.
 
+### bitbucket_update_review_status
+**File**: `src/tools/pull-requests/update_review_status.ts`
+**Endpoint**: `PUT /projects/{projectKey}/repos/{repositorySlug}/pull-requests/{pullRequestId}/participants/{userSlug}`
+**Parameters**:
+- `projectKey` (required): The Bitbucket Server project key
+- `repositorySlug` (required): The repository slug
+- `pullRequestId` (required): The pull request ID
+- `status` (required): Review status - `APPROVED`, `NEEDS_WORK`, or `UNAPPROVED`
+
+**Auto-detected**:
+- `userSlug`: Automatically detected from the authenticated user via `X-AUSERNAME` response header
+
+**Returns**: Updated participant object with review status, role, and lastReviewedCommit.
+
+**Purpose**: Change the authenticated user's review status for a PR. Automatically adds the user as a participant/reviewer if not already. The API automatically updates `lastReviewedCommit` to the latest commit when status is `APPROVED` or `NEEDS_WORK`. Only requires `REPO_READ` permission.
+
+**Status Values**:
+- `APPROVED` - Approve the PR
+- `NEEDS_WORK` - Request changes (shows as "Requested changes" in UI from 8.10+)
+- `UNAPPROVED` - Neutral/remove approval
+
+**Note**: The tool makes a lightweight request to `/application-properties` to extract the authenticated username from the `X-AUSERNAME` response header, eliminating the need for users to provide their own slug.
+
 ## Pull Request Review Workflow
 
 For agents reviewing PRs and leaving comments on specific lines:
@@ -411,11 +518,27 @@ For agents reviewing PRs and leaving comments on specific lines:
    )
    ```
 
+5. **Update review status**:
+   ```
+   # Approve the PR
+   bitbucket_update_review_status(
+     projectKey, repositorySlug, pullRequestId,
+     status="APPROVED"
+   )
+
+   # Or request changes
+   bitbucket_update_review_status(
+     projectKey, repositorySlug, pullRequestId,
+     status="NEEDS_WORK"
+   )
+   ```
+
 **Key Points**:
 - Use `destination` line numbers from the diff for the `TO` side (most common)
 - Use `source` line numbers for the `FROM` side
 - Match `lineType` to the segment type from the diff (ADDED/REMOVED/CONTEXT)
 - The structured diff ensures exact line number accuracy
+- `userSlug` is automatically detected - no need to provide it
 
 ## Development Commands
 
@@ -431,8 +554,13 @@ node dist/index.js
 
 # Type checking
 npx tsc --noEmit
+
+# Lint and auto-fix issues
+npm run lint
 ```
 
+**IMPORTANT**: Always run `npm run lint` after making code changes. The linter will auto-fix formatting issues and catch unused variables, type errors, and other code quality issues.
+
 ## Environment Setup
 
 Required environment variables in `.env`:

package/dist/tools/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAMzE,wBAAgB,aAAa,CAAC,MAAM,EAAE,SAAS,QAiB9C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAazE,wBAAgB,aAAa,CAAC,MAAM,EAAE,SAAS,QAkB9C"}

package/dist/tools/index.js

@@ -1,5 +1,5 @@
-import { addPrCommentTool, getPrChangesTool, getPrFileDiffTool, getPrActivitiesTool, getInboxPullRequestsTool } from './pull-requests/index.js';
 import { listProjectsTool } from './projects/index.js';
+import { addPrCommentTool, getInboxPullRequestsTool, getPrActivitiesTool, getPrChangesTool, getPrFileDiffTool, updateReviewStatusTool, } from './pull-requests/index.js';
 import { listRepositoriesTool } from './repositories/index.js';
 import { getAllUsersTool, getUserProfileTool } from './users/index.js';
 export function registerTools(server) {
@@ -16,5 +16,6 @@ export function registerTools(server) {
     getPrChangesTool(server);
     getPrFileDiffTool(server);
     getPrActivitiesTool(server);
+    updateReviewStatusTool(server);
 }
 //# sourceMappingURL=index.js.map

package/dist/tools/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AAChJ,OAAO,EAAE,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AACvD,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAC;AAC/D,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAEvE,MAAM,UAAU,aAAa,CAAC,MAAiB;IAC7C,aAAa;IACb,kBAAkB,CAAC,MAAM,CAAC,CAAC;IAC3B,eAAe,CAAC,MAAM,CAAC,CAAC;IAExB,gBAAgB;IAChB,gBAAgB,CAAC,MAAM,CAAC,CAAC;IAEzB,mBAAmB;IACnB,oBAAoB,CAAC,MAAM,CAAC,CAAC;IAE7B,qBAAqB;IACrB,wBAAwB,CAAC,MAAM,CAAC,CAAC;IACjC,gBAAgB,CAAC,MAAM,CAAC,CAAC;IACzB,gBAAgB,CAAC,MAAM,CAAC,CAAC;IACzB,iBAAiB,CAAC,MAAM,CAAC,CAAC;IAC1B,mBAAmB,CAAC,MAAM,CAAC,CAAC;AAC9B,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,gBAAgB,EAAE,MAAM,qBAAqB,CAAC;AACvD,OAAO,EACL,gBAAgB,EAChB,wBAAwB,EACxB,mBAAmB,EACnB,gBAAgB,EAChB,iBAAiB,EACjB,sBAAsB,GACvB,MAAM,0BAA0B,CAAC;AAClC,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAC;AAC/D,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAEvE,MAAM,UAAU,aAAa,CAAC,MAAiB;IAC7C,aAAa;IACb,kBAAkB,CAAC,MAAM,CAAC,CAAC;IAC3B,eAAe,CAAC,MAAM,CAAC,CAAC;IAExB,gBAAgB;IAChB,gBAAgB,CAAC,MAAM,CAAC,CAAC;IAEzB,mBAAmB;IACnB,oBAAoB,CAAC,MAAM,CAAC,CAAC;IAE7B,qBAAqB;IACrB,wBAAwB,CAAC,MAAM,CAAC,CAAC;IACjC,gBAAgB,CAAC,MAAM,CAAC,CAAC;IACzB,gBAAgB,CAAC,MAAM,CAAC,CAAC;IACzB,iBAAiB,CAAC,MAAM,CAAC,CAAC;IAC1B,mBAAmB,CAAC,MAAM,CAAC,CAAC;IAC5B,sBAAsB,CAAC,MAAM,CAAC,CAAC;AACjC,CAAC"}

package/dist/tools/projects/list_projects.d.ts

@@ -1,3 +1,3 @@
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 export declare const listProjectsTool: (server: McpServer) => void;
 //# sourceMappingURL=list_projects.d.ts.map

package/dist/tools/projects/list_projects.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"list_projects.d.ts","sourceRoot":"","sources":["../../../src/tools/projects/list_projects.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAsBzE,eAAO,MAAM,gBAAgB,GAAI,QAAQ,SAAS,SAwCjD,CAAC"}
+{"version":3,"file":"list_projects.d.ts","sourceRoot":"","sources":["../../../src/tools/projects/list_projects.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAWzE,eAAO,MAAM,gBAAgB,GAAI,QAAQ,SAAS,SAwBjD,CAAC"}

package/dist/tools/projects/list_projects.js

@@ -1,50 +1,24 @@
-import { z } from "zod";
-import { bitbucketClient } from "../../services/bitbucket.js";
+import { z } from 'zod';
+import { bitbucketClient } from '../../services/bitbucket.js';
 const schema = z.object({
-    name: z
-        .string()
-        .optional()
-        .describe("Filter projects by name (partial match)"),
-    permission: z
-        .string()
-        .optional()
-        .describe("Filter by permission (e.g., PROJECT_READ, PROJECT_WRITE, PROJECT_ADMIN)"),
-    start: z
-        .number()
-        .optional()
-        .describe("Starting index for pagination (default: 0)"),
-    limit: z
-        .number()
-        .optional()
-        .describe("Maximum number of projects to return (default: 25)"),
+    name: z.string().optional().describe('Filter projects by name (partial match)'),
+    permission: z.string().optional().describe('Filter by permission (e.g., PROJECT_READ, PROJECT_WRITE, PROJECT_ADMIN)'),
+    start: z.number().optional().describe('Starting index for pagination (default: 0)'),
+    limit: z.number().optional().describe('Maximum number of projects to return (default: 25)'),
 });
 export const listProjectsTool = (server) => {
-    server.registerTool("bitbucket_list_projects", {
-        title: "List Projects",
-        description: "Retrieve a page of projects. Only projects for which the authenticated user has PROJECT_VIEW permission will be returned. Can filter by name or permission level.",
+    server.registerTool('bitbucket_list_projects', {
+        title: 'List Projects',
+        description: 'Retrieve a page of projects. Only projects for which the authenticated user has PROJECT_VIEW permission will be returned. Can filter by name or permission level.',
         inputSchema: schema.shape,
-    }, async (params) => {
-        const { name, permission, start, limit } = schema.parse(params);
-        const queryParams = {};
-        if (name !== undefined) {
-            queryParams.name = name;
-        }
-        if (permission !== undefined) {
-            queryParams.permission = permission;
-        }
-        if (start !== undefined) {
-            queryParams.start = start.toString();
-        }
-        if (limit !== undefined) {
-            queryParams.limit = limit.toString();
-        }
-        const response = await bitbucketClient.get("/projects", {
-            params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+    }, async ({ name, permission, start, limit }) => {
+        const response = await bitbucketClient.get('/projects', {
+            params: { name, permission, start, limit },
         });
         return {
             content: [
                 {
-                    type: "text",
+                    type: 'text',
                     text: JSON.stringify(response.data, null, 2),
                 },
             ],

package/dist/tools/projects/list_projects.js.map

@@ -1 +1 @@
-{"version":3,"file":"list_projects.js","sourceRoot":"","sources":["../../../src/tools/projects/list_projects.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAE9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACvB,IAAI,EAAE,CAAC;SACL,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,yCAAyC,CAAC;IACrD,UAAU,EAAE,CAAC;SACX,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,yEAAyE,CAAC;IACrF,KAAK,EAAE,CAAC;SACN,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,4CAA4C,CAAC;IACxD,KAAK,EAAE,CAAC;SACN,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,oDAAoD,CAAC;CAChE,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,MAAiB,EAAE,EAAE;IACrD,MAAM,CAAC,YAAY,CAClB,yBAAyB,EACzB;QACC,KAAK,EAAE,eAAe;QACtB,WAAW,EACV,mKAAmK;QACpK,WAAW,EAAE,MAAM,CAAC,KAAK;KACzB,EACD,KAAK,EAAE,MAAM,EAAE,EAAE;QAChB,MAAM,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;QAEhE,MAAM,WAAW,GAA2B,EAAE,CAAC;QAC/C,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;YACxB,WAAW,CAAC,IAAI,GAAG,IAAI,CAAC;QACzB,CAAC;QACD,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;YAC9B,WAAW,CAAC,UAAU,GAAG,UAAU,CAAC;QACrC,CAAC;QACD,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACzB,WAAW,CAAC,KAAK,GAAG,KAAK,CAAC,QAAQ,EAAE,CAAC;QACtC,CAAC;QACD,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACzB,WAAW,CAAC,KAAK,GAAG,KAAK,CAAC,QAAQ,EAAE,CAAC;QACtC,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE;YACvD,MAAM,EAAE,MAAM,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,SAAS;SACrE,CAAC,CAAC;QAEH,OAAO;YACN,OAAO,EAAE;gBACR;oBACC,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;iBAC5C;aACD;SACD,CAAC;IACH,CAAC,CACD,CAAC;AACH,CAAC,CAAC"}
+{"version":3,"file":"list_projects.js","sourceRoot":"","sources":["../../../src/tools/projects/list_projects.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAE9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,yCAAyC,CAAC;IAC/E,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,yEAAyE,CAAC;IACrH,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,4CAA4C,CAAC;IACnF,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,oDAAoD,CAAC;CAC5F,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,MAAiB,EAAE,EAAE;IACpD,MAAM,CAAC,YAAY,CACjB,yBAAyB,EACzB;QACE,KAAK,EAAE,eAAe;QACtB,WAAW,EACT,mKAAmK;QACrK,WAAW,EAAE,MAAM,CAAC,KAAK;KAC1B,EACD,KAAK,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,EAAE;QAC3C,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE;YACtD,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE;SAC3C,CAAC,CAAC;QAEH,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;iBAC7C;aACF;SACF,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,CAAC"}

package/dist/tools/pull-requests/add_pr_comment.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"add_pr_comment.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/add_pr_comment.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAgBzE,eAAO,MAAM,gBAAgB,GAAI,QAAQ,SAAS,SA6CjD,CAAC"}
+{"version":3,"file":"add_pr_comment.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/add_pr_comment.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAiBzE,eAAO,MAAM,gBAAgB,GAAI,QAAQ,SAAS,SA0CjD,CAAC"}

package/dist/tools/pull-requests/add_pr_comment.js

@@ -16,8 +16,7 @@ export const addPrCommentTool = (server) => {
         title: 'Add Pull Request Comment',
         description: 'Add a comment to a pull request. Supports general comments, replies, and inline file/line comments',
         inputSchema: schema.shape,
-    }, async (params) => {
-        const { projectKey, repositorySlug, pullRequestId, text, parentId, path, line, lineType, fileType } = schema.parse(params);
+    }, async ({ projectKey, repositorySlug, pullRequestId, text, parentId, path, line, lineType, fileType }) => {
         const body = { text };
         if (parentId) {
             body.parent = { id: parentId };

package/dist/tools/pull-requests/add_pr_comment.js.map

@@ -1 +1 @@
-{"version":3,"file":"add_pr_comment.js","sourceRoot":"","sources":["../../../src/tools/pull-requests/add_pr_comment.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAE9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,2BAA2B,CAAC;IAC5D,cAAc,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;IAC1D,aAAa,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;IACzD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,kBAAkB,CAAC;IAC7C,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,+BAA+B,CAAC;IACzE,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,sCAAsC,CAAC;IAC5E,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACvE,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACxG,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,4BAA4B,CAAC;CACnF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,MAAiB,EAAE,EAAE;IACpD,MAAM,CAAC,YAAY,CACjB,0BAA0B,EAC1B;QACE,KAAK,EAAE,0BAA0B;QACjC,WAAW,EAAE,oGAAoG;QACjH,WAAW,EAAE,MAAM,CAAC,KAAK;KAC1B,EACD,KAAK,EAAE,MAAM,EAAE,EAAE;QACf,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,aAAa,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,QAAQ,EAAE,GACjG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;QAEvB,MAAM,IAAI,GAAQ,EAAE,IAAI,EAAE,CAAC;QAE3B,IAAI,QAAQ,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,GAAG,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;QACjC,CAAC;QAED,IAAI,IAAI,EAAE,CAAC;YACT,IAAI,CAAC,MAAM,GAAG;gBACZ,IAAI;gBACJ,QAAQ,EAAE,WAAW;gBACrB,GAAG,CAAC,IAAI,IAAI;oBACV,IAAI;oBACJ,QAAQ,EAAE,QAAQ,IAAI,SAAS;oBAC/B,QAAQ,EAAE,QAAQ,IAAI,IAAI;iBAC3B,CAAC;aACH,CAAC;QACJ,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,IAAI,CACzC,aAAa,UAAU,UAAU,cAAc,kBAAkB,aAAa,WAAW,EACzF,IAAI,CACL,CAAC;QAEF,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;iBAC7C;aACF;SACF,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,CAAC"}
+{"version":3,"file":"add_pr_comment.js","sourceRoot":"","sources":["../../../src/tools/pull-requests/add_pr_comment.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAG9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,2BAA2B,CAAC;IAC5D,cAAc,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;IAC1D,aAAa,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,qBAAqB,CAAC;IACzD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,kBAAkB,CAAC;IAC7C,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,+BAA+B,CAAC;IACzE,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,sCAAsC,CAAC;IAC5E,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACvE,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,iCAAiC,CAAC;IACxG,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,4BAA4B,CAAC;CACnF,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,MAAiB,EAAE,EAAE;IACpD,MAAM,CAAC,YAAY,CACjB,0BAA0B,EAC1B;QACE,KAAK,EAAE,0BAA0B;QACjC,WAAW,EAAE,oGAAoG;QACjH,WAAW,EAAE,MAAM,CAAC,KAAK;KAC1B,EACD,KAAK,EAAE,EAAE,UAAU,EAAE,cAAc,EAAE,aAAa,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE;QACtG,MAAM,IAAI,GAAmB,EAAE,IAAI,EAAE,CAAC;QAEtC,IAAI,QAAQ,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,GAAG,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;QACjC,CAAC;QAED,IAAI,IAAI,EAAE,CAAC;YACT,IAAI,CAAC,MAAM,GAAG;gBACZ,IAAI;gBACJ,QAAQ,EAAE,WAAW;gBACrB,GAAG,CAAC,IAAI,IAAI;oBACV,IAAI;oBACJ,QAAQ,EAAE,QAAQ,IAAI,SAAS;oBAC/B,QAAQ,EAAE,QAAQ,IAAI,IAAI;iBAC3B,CAAC;aACH,CAAC;QACJ,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,IAAI,CACzC,aAAa,UAAU,UAAU,cAAc,kBAAkB,aAAa,WAAW,EACzF,IAAI,CACL,CAAC;QAEF,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;iBAC7C;aACF;SACF,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,CAAC"}

package/dist/tools/pull-requests/get_inbox_pull_requests.d.ts

@@ -1,3 +1,3 @@
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 export declare const getInboxPullRequestsTool: (server: McpServer) => void;
 //# sourceMappingURL=get_inbox_pull_requests.d.ts.map

package/dist/tools/pull-requests/get_inbox_pull_requests.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get_inbox_pull_requests.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_inbox_pull_requests.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAczE,eAAO,MAAM,wBAAwB,GAAI,QAAQ,SAAS,SA0DzD,CAAC"}
+{"version":3,"file":"get_inbox_pull_requests.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_inbox_pull_requests.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAezE,eAAO,MAAM,wBAAwB,GAAI,QAAQ,SAAS,SAgDzD,CAAC"}

package/dist/tools/pull-requests/get_inbox_pull_requests.js

@@ -1,31 +1,17 @@
-import { z } from "zod";
-import { bitbucketClient } from "../../services/bitbucket.js";
+import { z } from 'zod';
+import { bitbucketClient } from '../../services/bitbucket.js';
 const schema = z.object({
-    start: z
-        .number()
-        .optional()
-        .describe("Starting index for pagination (default: 0)"),
-    limit: z
-        .number()
-        .optional()
-        .describe("Maximum number of pull requests to return (default: 25)"),
+    start: z.number().optional().describe('Starting index for pagination (default: 0)'),
+    limit: z.number().optional().describe('Maximum number of pull requests to return (default: 25)'),
 });
 export const getInboxPullRequestsTool = (server) => {
-    server.registerTool("bitbucket_get_inbox_pull_requests", {
-        title: "Get Inbox Pull Requests",
+    server.registerTool('bitbucket_get_inbox_pull_requests', {
+        title: 'Get Inbox Pull Requests',
         description: "Returns pull requests in the authenticated user's inbox where they are assigned as a reviewer. Returns PRs across all projects and repositories in one call. Each PR includes only essential review information: id, title, description, state, author name, projectKey, and repositorySlug - everything needed to review the PR using other tools. Response is heavily optimized to reduce token usage by ~92%.",
         inputSchema: schema.shape,
-    }, async (params) => {
-        const { start, limit } = schema.parse(params);
-        const queryParams = {};
-        if (start !== undefined) {
-            queryParams.start = start.toString();
-        }
-        if (limit !== undefined) {
-            queryParams.limit = limit.toString();
-        }
-        const response = await bitbucketClient.get("/inbox/pull-requests", {
-            params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+    }, async ({ start, limit }) => {
+        const response = await bitbucketClient.get('/inbox/pull-requests', {
+            params: { start, limit },
         });
         // Strip to minimal PR - only what agent needs to review
         const minimalPr = (pr) => {
@@ -38,7 +24,7 @@ export const getInboxPullRequestsTool = (server) => {
                 projectKey: pr.toRef.repository.project.key,
                 repositorySlug: pr.toRef.repository.slug,
                 createdDate: pr.createdDate,
-                updatedDate: pr.updatedDate
+                updatedDate: pr.updatedDate,
             };
         };
         const minimal = {
@@ -47,12 +33,12 @@ export const getInboxPullRequestsTool = (server) => {
             isLastPage: response.data.isLastPage,
             start: response.data.start,
             nextPageStart: response.data.nextPageStart,
-            values: response.data.values.map(minimalPr)
+            values: response.data.values.map(minimalPr),
         };
         return {
             content: [
                 {
-                    type: "text",
+                    type: 'text',
                     text: JSON.stringify(minimal, null, 2),
                 },
             ],

package/dist/tools/pull-requests/get_inbox_pull_requests.js.map

@@ -1 +1 @@
-{"version":3,"file":"get_inbox_pull_requests.js","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_inbox_pull_requests.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAE9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACvB,KAAK,EAAE,CAAC;SACN,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,4CAA4C,CAAC;IACxD,KAAK,EAAE,CAAC;SACN,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,yDAAyD,CAAC;CACrE,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,MAAiB,EAAE,EAAE;IAC7D,MAAM,CAAC,YAAY,CAClB,mCAAmC,EACnC;QACC,KAAK,EAAE,yBAAyB;QAChC,WAAW,EACV,kZAAkZ;QACnZ,WAAW,EAAE,MAAM,CAAC,KAAK;KACzB,EACD,KAAK,EAAE,MAAM,EAAE,EAAE;QAChB,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;QAE9C,MAAM,WAAW,GAA2B,EAAE,CAAC;QAC/C,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACzB,WAAW,CAAC,KAAK,GAAG,KAAK,CAAC,QAAQ,EAAE,CAAC;QACtC,CAAC;QACD,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACzB,WAAW,CAAC,KAAK,GAAG,KAAK,CAAC,QAAQ,EAAE,CAAC;QACtC,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,GAAG,CAAC,sBAAsB,EAAE;YAClE,MAAM,EAAE,MAAM,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,SAAS;SACrE,CAAC,CAAC;QAEH,wDAAwD;QACxD,MAAM,SAAS,GAAG,CAAC,EAAO,EAAO,EAAE;YAClC,OAAO;gBACN,EAAE,EAAE,EAAE,CAAC,EAAE;gBACT,KAAK,EAAE,EAAE,CAAC,KAAK;gBACf,WAAW,EAAE,EAAE,CAAC,WAAW;gBAC3B,KAAK,EAAE,EAAE,CAAC,KAAK;gBACf,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW;gBAClC,UAAU,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,OAAO,CAAC,GAAG;gBAC3C,cAAc,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI;gBACxC,WAAW,EAAE,EAAE,CAAC,WAAW;gBAC3B,WAAW,EAAE,EAAE,CAAC,WAAW;aAC3B,CAAC;QACH,CAAC,CAAC;QAEF,MAAM,OAAO,GAAG;YACf,IAAI,EAAE,QAAQ,CAAC,IAAI,CAAC,IAAI;YACxB,KAAK,EAAE,QAAQ,CAAC,IAAI,CAAC,KAAK;YAC1B,UAAU,EAAE,QAAQ,CAAC,IAAI,CAAC,UAAU;YACpC,KAAK,EAAE,QAAQ,CAAC,IAAI,CAAC,KAAK;YAC1B,aAAa,EAAE,QAAQ,CAAC,IAAI,CAAC,aAAa;YAC1C,MAAM,EAAE,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC;SAC3C,CAAC;QAEF,OAAO;YACN,OAAO,EAAE;gBACR;oBACC,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;iBACtC;aACD;SACD,CAAC;IACH,CAAC,CACD,CAAC;AACH,CAAC,CAAC"}
+{"version":3,"file":"get_inbox_pull_requests.js","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_inbox_pull_requests.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAQ9D,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,4CAA4C,CAAC;IACnF,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,yDAAyD,CAAC;CACjG,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,wBAAwB,GAAG,CAAC,MAAiB,EAAE,EAAE;IAC5D,MAAM,CAAC,YAAY,CACjB,mCAAmC,EACnC;QACE,KAAK,EAAE,yBAAyB;QAChC,WAAW,EACT,kZAAkZ;QACpZ,WAAW,EAAE,MAAM,CAAC,KAAK;KAC1B,EACD,KAAK,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,EAAE;QACzB,MAAM,QAAQ,GAAG,MAAM,eAAe,CAAC,GAAG,CAAsC,sBAAsB,EAAE;YACtG,MAAM,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE;SACzB,CAAC,CAAC;QAEH,wDAAwD;QACxD,MAAM,SAAS,GAAG,CAAC,EAAoB,EAAsB,EAAE;YAC7D,OAAO;gBACL,EAAE,EAAE,EAAE,CAAC,EAAE;gBACT,KAAK,EAAE,EAAE,CAAC,KAAK;gBACf,WAAW,EAAE,EAAE,CAAC,WAAW;gBAC3B,KAAK,EAAE,EAAE,CAAC,KAAK;gBACf,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW;gBAClC,UAAU,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,OAAO,CAAC,GAAG;gBAC3C,cAAc,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI;gBACxC,WAAW,EAAE,EAAE,CAAC,WAAW;gBAC3B,WAAW,EAAE,EAAE,CAAC,WAAW;aAC5B,CAAC;QACJ,CAAC,CAAC;QAEF,MAAM,OAAO,GAA8B;YACzC,IAAI,EAAE,QAAQ,CAAC,IAAI,CAAC,IAAI;YACxB,KAAK,EAAE,QAAQ,CAAC,IAAI,CAAC,KAAK;YAC1B,UAAU,EAAE,QAAQ,CAAC,IAAI,CAAC,UAAU;YACpC,KAAK,EAAE,QAAQ,CAAC,IAAI,CAAC,KAAK;YAC1B,aAAa,EAAE,QAAQ,CAAC,IAAI,CAAC,aAAa;YAC1C,MAAM,EAAE,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC;SAC5C,CAAC;QAEF,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;iBACvC;aACF;SACF,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,CAAC"}

package/dist/tools/pull-requests/get_pr_activities.d.ts

@@ -1,3 +1,3 @@
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 export declare const getPrActivitiesTool: (server: McpServer) => void;
 //# sourceMappingURL=get_pr_activities.d.ts.map

package/dist/tools/pull-requests/get_pr_activities.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get_pr_activities.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_pr_activities.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AA4CzE,eAAO,MAAM,mBAAmB,GAAI,QAAQ,SAAS,SA6HpD,CAAC"}
+{"version":3,"file":"get_pr_activities.d.ts","sourceRoot":"","sources":["../../../src/tools/pull-requests/get_pr_activities.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAiDzE,eAAO,MAAM,mBAAmB,GAAI,QAAQ,SAAS,SAuGpD,CAAC"}

package/dist/tools/pull-requests/get_pr_activities.js

@@ -1,63 +1,47 @@
-import { z } from "zod";
-import { bitbucketClient } from "../../services/bitbucket.js";
+import { z } from 'zod';
+import { bitbucketClient } from '../../services/bitbucket.js';
 // Activity action types from Bitbucket Server Swagger
 const ActivityType = z.enum([
-    "APPROVED",
-    "AUTO_MERGE_CANCELLED",
-    "AUTO_MERGE_REQUESTED",
-    "COMMENTED",
-    "DECLINED",
-    "DELETED",
-    "MERGED",
-    "OPENED",
-    "REOPENED",
-    "RESCOPED",
-    "REVIEW_COMMENTED",
-    "REVIEW_DISCARDED",
-    "REVIEW_FINISHED",
-    "REVIEWED",
-    "UNAPPROVED",
-    "UPDATED",
+    'APPROVED',
+    'AUTO_MERGE_CANCELLED',
+    'AUTO_MERGE_REQUESTED',
+    'COMMENTED',
+    'DECLINED',
+    'DELETED',
+    'MERGED',
+    'OPENED',
+    'REOPENED',
+    'RESCOPED',
+    'REVIEW_COMMENTED',
+    'REVIEW_DISCARDED',
+    'REVIEW_FINISHED',
+    'REVIEWED',
+    'UNAPPROVED',
+    'UPDATED',
 ]);
 const schema = z.object({
-    projectKey: z.string().describe("The Bitbucket project key"),
-    repositorySlug: z.string().describe("The repository slug"),
-    pullRequestId: z.number().describe("The pull request ID"),
+    projectKey: z.string().describe('The Bitbucket project key'),
+    repositorySlug: z.string().describe('The repository slug'),
+    pullRequestId: z.number().describe('The pull request ID'),
     activityTypes: z
         .array(ActivityType)
         .optional()
         .describe('Filter activities by type. Example: ["COMMENTED", "REVIEW_COMMENTED"] to get only comments, or ["APPROVED", "UNAPPROVED"] for approvals.'),
-    start: z
-        .number()
-        .optional()
-        .describe("Starting index for pagination (default: 0)"),
-    limit: z
-        .number()
-        .optional()
-        .describe("Maximum number of items to return (default: 25)"),
+    start: z.number().optional().describe('Starting index for pagination (default: 0)'),
+    limit: z.number().optional().describe('Maximum number of items to return (default: 25)'),
 });
 export const getPrActivitiesTool = (server) => {
-    server.registerTool("bitbucket_get_pull_request_activities", {
-        title: "Get Pull Request Activities",
+    server.registerTool('bitbucket_get_pull_request_activities', {
+        title: 'Get Pull Request Activities',
         description: "Gets activity on a pull request including comments, approvals, merges, reviews, and updates. Can optionally filter by activity types (e.g., only comments). Returns a paginated list with action types like COMMENTED, APPROVED, DECLINED, MERGED, REVIEWED, etc. Use activityTypes parameter to filter: e.g., ['COMMENTED', 'REVIEW_COMMENTED'] for only comments. Response is optimized for token usage - use commentAnchor.path with get_pr_file_diff to fetch code context when needed.",
         inputSchema: schema.shape,
-    }, async (params) => {
-        const { projectKey, repositorySlug, pullRequestId, activityTypes, start, limit } = schema.parse(params);
-        const queryParams = {};
-        if (start !== undefined) {
-            queryParams.start = start.toString();
-        }
-        if (limit !== undefined) {
-            queryParams.limit = limit.toString();
-        }
-        queryParams.fromType = "COMMENT";
+    }, async ({ projectKey, repositorySlug, pullRequestId, activityTypes, start, limit }) => {
         const response = await bitbucketClient.get(`/projects/${projectKey}/repos/${repositorySlug}/pull-requests/${pullRequestId}/activities`, {
-            params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+            params: { fromType: 'COMMENT', start, limit },
         });
-        // Helper function to strip user bloat
+        // Helper function to strip user links (API returns links but we exclude from type)
         const stripUserBloat = (user) => {
-            if (!user)
-                return user;
+            // biome-ignore lint/correctness/noUnusedVariables: Removing links field from API response
             const { links, ...userWithoutLinks } = user;
             return userWithoutLinks;
         };
@@ -72,46 +56,37 @@ export const getPrActivitiesTool = (server) => {
         };
         // Helper function to simplify likedBy to just count
         const simplifyLikedBy = (likedBy) => {
-            if (!likedBy)
-                return likedBy;
             return { total: likedBy.total || 0 };
         };
         // Helper function to strip bloat from comments
         const stripBloatFromComment = (comment) => {
-            if (!comment)
-                return comment;
-            const { anchor, permittedOperations, ...strippedComment } = comment;
-            // Strip user links from author
-            if (strippedComment.author) {
-                strippedComment.author = stripUserBloat(strippedComment.author);
-            }
-            // Simplify reactions and likedBy in properties
-            if (strippedComment.properties) {
-                if (strippedComment.properties.reactions) {
-                    strippedComment.properties.reactions = simplifyReactions(strippedComment.properties.reactions);
-                }
-                if (strippedComment.properties.likedBy) {
-                    strippedComment.properties.likedBy = simplifyLikedBy(strippedComment.properties.likedBy);
-                }
-            }
-            // Recursively strip from nested comments (replies)
-            if (strippedComment.comments && Array.isArray(strippedComment.comments)) {
-                strippedComment.comments = strippedComment.comments.map(stripBloatFromComment);
-            }
-            return strippedComment;
+            // biome-ignore lint/correctness/noUnusedVariables: Removing anchor, commentAnchor, and permittedOperations from API response
+            const { anchor, commentAnchor, permittedOperations, ...rest } = comment;
+            // Build clean comment with stripped user and simplified properties (no commentAnchor)
+            const cleanComment = {
+                ...rest,
+                author: stripUserBloat(comment.author),
+                properties: comment.properties
+                    ? {
+                        reactions: comment.properties.reactions ? simplifyReactions(comment.properties.reactions) : undefined,
+                        likedBy: comment.properties.likedBy ? simplifyLikedBy(comment.properties.likedBy) : undefined,
+                    }
+                    : undefined,
+                comments: comment.comments ? comment.comments.map(stripBloatFromComment) : undefined,
+            };
+            return cleanComment;
         };
-        // Strip bloat from all activities
+        // Strip bloat from all activities (remove diff field which is large)
         const strippedValues = response.data.values.map((activity) => {
-            const { diff, ...activityWithoutDiff } = activity;
-            // Strip user links
-            if (activityWithoutDiff.user) {
-                activityWithoutDiff.user = stripUserBloat(activityWithoutDiff.user);
-            }
-            // Strip comment bloat
-            if (activityWithoutDiff.comment) {
-                activityWithoutDiff.comment = stripBloatFromComment(activityWithoutDiff.comment);
-            }
-            return activityWithoutDiff;
+            // biome-ignore lint/correctness/noUnusedVariables: Removing diff field from API response
+            const { diff, ...rest } = activity;
+            // Build clean activity with stripped user and comment
+            const cleanActivity = {
+                ...rest,
+                user: stripUserBloat(activity.user),
+                comment: activity.comment ? stripBloatFromComment(activity.comment) : undefined,
+            };
+            return cleanActivity;
         });
         // Filter by activity types if specified
         let responseData = {
@@ -129,7 +104,7 @@ export const getPrActivitiesTool = (server) => {
         return {
             content: [
                 {
-                    type: "text",
+                    type: 'text',
                     text: JSON.stringify(responseData, null, 2),
                 },
             ],