fix: improve error handling for 404s to use UserInputError

[dcramer]

• Add reusable error handling utilities in api-utils.ts
• Convert 404 API errors to UserInputError with clear messages
• Handle HTML error pages from server errors separately
• Update get-issue-details and update-issue tools to use new pattern
• Add comprehensive tests for error handling utilities

This ensures users get helpful error messages when they provide invalid issue IDs instead of seeing raw API errors.

🤖 Generated with Claude Code

[cursor]

Bug: Regex Bug Affects API Error Handling

The regex /^get|update/ in handleApiError is incorrect. It matches "get" at the beginning OR "update" anywhere in the string, but should match "get" OR "update" only at the beginning. The correct regex is /^(get|update)/. This flaw could lead to incorrect resourceType derivation for operations where "update" is not at the start (e.g., "analyzeUpdateData" would incorrectly become "analyzeData").

packages/mcp-server/src/tools/utils/api-utils.ts#L77-L78

sentry-mcp/packages/mcp-server/src/tools/utils/api-utils.ts

Lines 77 to 78 in 47a3241

	context.resourceType ||
	context.operation.replace(/^get|update/, "").toLowerCase();

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎

[graphite-app]

The regex pattern /^get|update/ will match either "get" at the start of a string OR "update" anywhere in the string, which produces incorrect results. For example, with "getEvent", it removes only "get" leaving "Event", then lowercases to "event" (which happens to work), but "updateEvent" would become "event" (correct) while "getUpdateInfo" would become "updateInfo" (incorrect).

Consider using /^(get|update)/ to properly group the alternation, ensuring only prefixes are matched. Alternatively, a more explicit mapping approach might be more maintainable for future operation types:

const operationToResourceType = {
  getIssue: 'issue',
  updateIssue: 'issue',
  getEvent: 'event',
  // other mappings...
};

const resourceType = context.resourceType || 
  operationToResourceType[context.operation] || 
  context.operation.toLowerCase();

Suggested change

	const resourceType =
	context.resourceType ||
	context.operation.replace(/^get|update/, "").toLowerCase();
	const resourceType =
	context.resourceType ||
	context.operation.replace(/^(get|update)/, "").toLowerCase();

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.