feat(mcp): Add priority parameter to update_issue (#2155)

`update_issue` only took `status` and `assignedTo`. That meant an agent
couldn't change an issue's priority without also resolving, ignoring, or
reassigning it. This came up in user feedback (MCP-SERVER-GND).

Sentry's issue update endpoint already accepts `priority` with the
values `low`, `medium`, and `high`. It's validated in `GroupValidator`
and handled by `handle_priority` in `group_index/update.py`. A manual
priority also sets `priority_locked_at`, so Sentry stops changing it
automatically, and the param description says so. Issue types that don't
allow manual priority get a 400 from Sentry, which the tool returns to
the caller as is.

What changed:
- New optional `priority` input, sent in the PUT body. Passing only
`priority` is now a valid update.
- If the issue already has the requested priority, the PUT is skipped,
the same way it is for status and assignee.
- The structured output now includes `changes.priority` and
`current.priority`.

The tool description is also much shorter: about 730 characters, down
from about 2,020. It no longer repeats enum values and per-field rules,
because those are already in the input schema. It keeps a few examples
and the hints for rules that span more than one field.

<!-- junior-request-attribution:start -->
via @dcramer.
<!-- junior-request-attribution:end -->

<!-- junior-session-footer:start -->
<!-- junior-conversation-id:slack%3AC08J1NSPU6S%3A1794354108.041609 -->

--

[View
Session](https://junior-prod.sentry.dev/conversations/slack%3AC08J1NSPU6S%3A1794354108.041609)
[[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AC08J1NSPU6S%3A1794354108.041609/?project=4510944073809921)

<!-- junior-session-footer:end -->

---------

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: David Cramer <23610+dcramer@users.noreply.github.com>

Author: sentry-junior[bot]

packages/mcp-core/src/api-client/client.ts

@@ -4124,11 +4124,13 @@ export class SentryApiService {
       ignoreUserCount,
       ignoreUserWindow,
       statusDetails,
+      priority,
     }: {
       organizationSlug: string;
       issueId: string;
       status?: string;
       assignedTo?: string | null;
+      priority?: string;
       substatus?: string;
       ignoreDuration?: number;
       ignoreCount?: number;
@@ -4149,8 +4151,10 @@ export class SentryApiService {
       ignoreUserCount?: number;
       ignoreUserWindow?: number;
       statusDetails?: IssueResolutionDetails;
+      priority?: string;
     } = {};
     if (status !== undefined) updateData.status = status;
+    if (priority !== undefined) updateData.priority = priority;
     if (statusDetails !== undefined) updateData.statusDetails = statusDetails;
     if (assignedTo !== undefined) updateData.assignedTo = assignedTo ?? "";
     if (substatus !== undefined) updateData.substatus = substatus;

packages/mcp-core/src/api-client/schema.ts

@@ -860,6 +860,7 @@ export const IssueSchema = z
     platform: z.string().nullable().optional(),
     status: z.string(),
     substatus: z.string().nullable().optional(),
+    priority: z.string().nullable().optional(),
     culprit: z.string().nullable(),
     type: z
       .union([

packages/mcp-core/src/schema.ts

@@ -215,6 +215,12 @@ export const ParamIssueStatus = z
     "The new status for the issue. Valid values are 'resolved', 'resolvedInNextRelease', 'unresolved', and 'ignored'.",
   );
 
+export const ParamIssuePriority = z
+  .enum(["low", "medium", "high"])
+  .describe(
+    "The new priority for the issue. Setting it manually stops Sentry from changing it automatically.",
+  );
+
 export const ParamIssueIgnoreMode = z
   .enum([
     "untilEscalating",

packages/mcp-core/src/skillDefinitions.json

@@ -525,7 +525,7 @@
       },
       {
         "name": "update_issue",
-        "description": "Update a Sentry issue's status or assignment.\n\nUse this to resolve, reopen, assign, unassign, or ignore an issue.\n\n<examples>\n```\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='resolved')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo='user:123456')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo=null)\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='forever')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='untilOccurrenceCount', ignoreCount=100, ignoreWindowMinutes=60)\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', reason='Ignoring because this is expected noise from the staging deploy')\n```\n</examples>\n\n<hints>\n- Provide `issueUrl` or `organizationSlug` + `issueId`.\n- At least one of `status` or `assignedTo` is required.\n- Omit `assignedTo` to leave assignment unchanged. Pass `null` to unassign; otherwise use `user:ID` or `team:ID_OR_SLUG`.\n- Use the `whoami` tool to find your user ID for self-assignment.\n- Status values: `resolved`, `resolvedInNextRelease`, `unresolved`, `ignored`.\n- `status='resolved'` takes optional `inRelease`, or `inCommit` + `inCommitRepository`.\n- `status='ignored'` defaults to `ignoreMode='untilEscalating'`.\n- Ignore modes: `untilEscalating`, `forever`, `forDuration`, `untilOccurrenceCount`, `untilUserCount`.\n- Matching ignore inputs are `ignoreDurationMinutes`, `ignoreCount` + optional `ignoreWindowMinutes`, or `ignoreUserCount` + optional `ignoreUserWindowMinutes`.\n- To switch an already ignored issue between `untilEscalating`, `forever`, and condition-based ignore modes, first set `status='unresolved'`, then ignore it again with the new rule.\n- `reason` is optional. When provided, it will be posted as a comment on the issue's activity feed explaining why the action was taken.\n</hints>",
+        "description": "Update a Sentry issue's status, assignment, or priority.\n\n<examples>\n```\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='resolved')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo='user:123456')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='forever')\n```\n</examples>\n\n<hints>\n- Provide `issueUrl` or `organizationSlug` + `issueId`.\n- At least one of `status`, `assignedTo`, or `priority` is required.\n- `status='ignored'` defaults to `ignoreMode='untilEscalating'`.\n- To switch an already ignored issue between `untilEscalating`, `forever`, and condition-based ignore modes, first set `status='unresolved'`, then ignore it again.\n</hints>",
         "requiredScopes": ["event:write"]
       },
       {

packages/mcp-core/src/toolDefinitions.json

@@ -14481,7 +14481,7 @@
   {
     "name": "update_issue",
     "title": "Update Issue",
-    "description": "Update a Sentry issue's status or assignment.\n\nUse this to resolve, reopen, assign, unassign, or ignore an issue.\n\n<examples>\n```\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='resolved')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo='user:123456')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo=null)\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='forever')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='untilOccurrenceCount', ignoreCount=100, ignoreWindowMinutes=60)\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', reason='Ignoring because this is expected noise from the staging deploy')\n```\n</examples>\n\n<hints>\n- Provide `issueUrl` or `organizationSlug` + `issueId`.\n- At least one of `status` or `assignedTo` is required.\n- Omit `assignedTo` to leave assignment unchanged. Pass `null` to unassign; otherwise use `user:ID` or `team:ID_OR_SLUG`.\n- Use the `whoami` tool to find your user ID for self-assignment.\n- Status values: `resolved`, `resolvedInNextRelease`, `unresolved`, `ignored`.\n- `status='resolved'` takes optional `inRelease`, or `inCommit` + `inCommitRepository`.\n- `status='ignored'` defaults to `ignoreMode='untilEscalating'`.\n- Ignore modes: `untilEscalating`, `forever`, `forDuration`, `untilOccurrenceCount`, `untilUserCount`.\n- Matching ignore inputs are `ignoreDurationMinutes`, `ignoreCount` + optional `ignoreWindowMinutes`, or `ignoreUserCount` + optional `ignoreUserWindowMinutes`.\n- To switch an already ignored issue between `untilEscalating`, `forever`, and condition-based ignore modes, first set `status='unresolved'`, then ignore it again with the new rule.\n- `reason` is optional. When provided, it will be posted as a comment on the issue's activity feed explaining why the action was taken.\n</hints>",
+    "description": "Update a Sentry issue's status, assignment, or priority.\n\n<examples>\n```\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='resolved')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', assignedTo='user:123456')\nupdate_issue(organizationSlug='my-org', issueId='PROJECT-123', status='ignored', ignoreMode='forever')\n```\n</examples>\n\n<hints>\n- Provide `issueUrl` or `organizationSlug` + `issueId`.\n- At least one of `status`, `assignedTo`, or `priority` is required.\n- `status='ignored'` defaults to `ignoreMode='untilEscalating'`.\n- To switch an already ignored issue between `untilEscalating`, `forever`, and condition-based ignore modes, first set `status='unresolved'`, then ignore it again.\n</hints>",
     "inputSchema": {
       "type": "object",
       "properties": {
@@ -14532,6 +14532,11 @@
           ],
           "description": "The assignee in format 'user:ID' or 'team:ID_OR_SLUG' where ID is numeric. Pass null to unassign the issue. Example: 'user:123456', 'team:789', or 'team:my-team-slug'. Use the `whoami` tool to find your user ID."
         },
+        "priority": {
+          "type": "string",
+          "enum": ["low", "medium", "high"],
+          "description": "The new priority for the issue. Setting it manually stops Sentry from changing it automatically."
+        },
         "ignoreMode": {
           "type": "string",
           "enum": [
@@ -14676,9 +14681,29 @@
                   "type": "null"
                 }
               ]
+            },
+            "priority": {
+              "anyOf": [
+                {
+                  "type": "object",
+                  "properties": {
+                    "from": {
+                      "type": "string"
+                    },
+                    "to": {
+                      "type": "string"
+                    }
+                  },
+                  "required": ["from", "to"],
+                  "additionalProperties": false
+                },
+                {
+                  "type": "null"
+                }
+              ]
             }
           },
-          "required": ["status", "ignoreBehavior", "assignedTo"],
+          "required": ["status", "ignoreBehavior", "assignedTo", "priority"],
           "additionalProperties": false
         },
         "current": {
@@ -14692,9 +14717,12 @@
             },
             "assignedTo": {
               "type": "string"
+            },
+            "priority": {
+              "type": ["string", "null"]
             }
           },
-          "required": ["status", "ignoreBehavior", "assignedTo"],
+          "required": ["status", "ignoreBehavior", "assignedTo", "priority"],
           "additionalProperties": false
         },
         "effect": {

packages/mcp-core/src/tools/catalog/update-issue.test.ts

@@ -58,6 +58,7 @@ describe("update_issue", () => {
         "changes": {
           "assignedTo": null,
           "ignoreBehavior": null,
+          "priority": null,
           "status": {
             "from": "unresolved",
             "to": "resolved",
@@ -66,6 +67,7 @@ describe("update_issue", () => {
         "current": {
           "assignedTo": "Jane Developer",
           "ignoreBehavior": null,
+          "priority": "medium",
           "status": "resolved",
         },
         "effect": "The issue is now marked as resolved and will no longer generate alerts",
@@ -144,6 +146,7 @@ describe("update_issue", () => {
       status: null,
       ignoreBehavior: null,
       assignedTo: null,
+      priority: null,
     });
     expect(output.current.status).toBe("unresolved");
     expect(output.effect).toBeNull();
@@ -169,11 +172,13 @@ describe("update_issue", () => {
             "to": "john.doe",
           },
           "ignoreBehavior": null,
+          "priority": null,
           "status": null,
         },
         "current": {
           "assignedTo": "john.doe",
           "ignoreBehavior": null,
+          "priority": "medium",
           "status": "unresolved",
         },
         "effect": null,
@@ -338,9 +343,84 @@ describe("update_issue", () => {
       status: null,
       ignoreBehavior: null,
       assignedTo: null,
+      priority: null,
     });
   });
 
+  it("updates issue priority without changing status or assignment", async () => {
+    let putBody: unknown;
+    mswServer.use(
+      http.put(
+        "https://sentry.io/api/0/organizations/sentry-mcp-evals/issues/CLOUDFLARE-MCP-41/",
+        async ({ request }) => {
+          putBody = await request.json();
+          return HttpResponse.json(createIssue({ priority: "low" }));
+        },
+        { once: true },
+      ),
+    );
+
+    const result = await updateIssue.handler(
+      {
+        organizationSlug: "sentry-mcp-evals",
+        issueId: "CLOUDFLARE-MCP-41",
+        priority: "low",
+        issueUrl: undefined,
+        regionUrl: null,
+      },
+      serverContext,
+    );
+
+    expect(putBody).toEqual({ priority: "low" });
+    const output = structured(result);
+    expect(output.updated).toBe(true);
+    expect(output.changes).toEqual({
+      status: null,
+      ignoreBehavior: null,
+      assignedTo: null,
+      priority: { from: "medium", to: "low" },
+    });
+    expect(output.current.status).toBe("unresolved");
+    expect(output.current.priority).toBe("low");
+    expect(output.effect).toBeNull();
+  });
+
+  it("skips priority updates when the requested priority is already set", async () => {
+    let putCalled = false;
+    const currentIssue = createIssue({ priority: "high" });
+
+    mswServer.use(
+      http.get(
+        "https://sentry.io/api/0/organizations/sentry-mcp-evals/issues/CLOUDFLARE-MCP-41/",
+        () => HttpResponse.json(currentIssue),
+      ),
+      http.put(
+        "https://sentry.io/api/0/organizations/sentry-mcp-evals/issues/CLOUDFLARE-MCP-41/",
+        () => {
+          putCalled = true;
+          return HttpResponse.json(currentIssue);
+        },
+      ),
+    );
+
+    const result = await updateIssue.handler(
+      {
+        organizationSlug: "sentry-mcp-evals",
+        issueId: "CLOUDFLARE-MCP-41",
+        priority: "high",
+        issueUrl: undefined,
+        regionUrl: null,
+      },
+      serverContext,
+    );
+
+    expect(putCalled).toBe(false);
+    const output = structured(result);
+    expect(output.updated).toBe(false);
+    expect(output.changes.priority).toBeNull();
+    expect(output.current.priority).toBe("high");
+  });
+
   it("updates both status and assignment", async () => {
     const result = await updateIssue.handler(
       {
@@ -361,6 +441,7 @@ describe("update_issue", () => {
             "to": "You",
           },
           "ignoreBehavior": null,
+          "priority": null,
           "status": {
             "from": "unresolved",
             "to": "resolved",
@@ -369,6 +450,7 @@ describe("update_issue", () => {
         "current": {
           "assignedTo": "me",
           "ignoreBehavior": null,
+          "priority": "medium",
           "status": "resolved",
         },
         "effect": "The issue is now marked as resolved and will no longer generate alerts",
@@ -512,6 +594,7 @@ describe("update_issue", () => {
         "changes": {
           "assignedTo": null,
           "ignoreBehavior": null,
+          "priority": null,
           "status": {
             "from": "unresolved",
             "to": "resolvedInRelease",
@@ -520,6 +603,7 @@ describe("update_issue", () => {
         "current": {
           "assignedTo": "Jane Developer",
           "ignoreBehavior": null,
+          "priority": "medium",
           "status": "resolvedInRelease",
         },
         "effect": "The issue is now marked as resolved in release backend@1.4.2. Events from newer releases will reopen it as a regression",
@@ -872,6 +956,7 @@ describe("update_issue", () => {
             "from": null,
             "to": "Until escalating",
           },
+          "priority": null,
           "status": {
             "from": "unresolved",
             "to": "ignored",
@@ -880,6 +965,7 @@ describe("update_issue", () => {
         "current": {
           "assignedTo": "Jane Developer",
           "ignoreBehavior": "Until escalating",
+          "priority": "medium",
           "status": "ignored",
         },
         "effect": "The issue is now ignored until it escalates",
@@ -934,6 +1020,7 @@ describe("update_issue", () => {
             "from": null,
             "to": "Until it occurs 100 times in 60 minutes",
           },
+          "priority": null,
           "status": {
             "from": "unresolved",
             "to": "ignored",
@@ -942,6 +1029,7 @@ describe("update_issue", () => {
         "current": {
           "assignedTo": "Jane Developer",
           "ignoreBehavior": "Until it occurs 100 times in 60 minutes",
+          "priority": "medium",
           "status": "ignored",
         },
         "effect": "The issue is now ignored until it occurs 100 times in 60 minutes",
@@ -1313,7 +1401,7 @@ describe("update_issue", () => {
         serverContext,
       ),
     ).rejects.toThrow(
-      "At least one of `status` or `assignedTo` must be provided to update the issue",
+      "At least one of `status`, `assignedTo`, or `priority` must be provided to update the issue",
     );
   });