fix: getTicket for Jira uses specific API endpoint for it

Author: prokopsimek

packages/jira/src/api.ts

@@ -7,6 +7,7 @@ import {
 import type {
 	JiraAttachmentResponse,
 	JiraErrorResponse,
+	JiraIssue,
 	JiraProject,
 	JiraSearchResponse,
 	JiraStatus,
@@ -84,6 +85,36 @@ export async function queryAssignable(
 	}
 }
 
+/**
+ * Get a Jira ticket
+ */
+export async function getTicket(
+	issueIdOrKey: string,
+): Promise<JiraIssue | JiraErrorResponse> {
+	try {
+		const response = await fetch(
+			`${JIRA_URL}/rest/api/3/issue/${issueIdOrKey}`,
+			{
+				method: "GET",
+				headers: getAuthHeaders(),
+			},
+		);
+
+		if (!response.ok) {
+			const errorData = await response.json();
+			return {
+				error: errorData || `HTTP error: ${response.status}`,
+			};
+		}
+
+		return await response.json();
+	} catch (error) {
+		return {
+			error: error instanceof Error ? error.message : String(error),
+		};
+	}
+}
+
 /**
  * Create a new Jira ticket
  */

packages/jira/src/index.ts

@@ -22,6 +22,8 @@ async function main(): Promise<void> {
 		const server = new McpServer({
 			name: "jira-mcp-server",
 			version: "1.0.0",
+			description: "Talk to Jira - MCP Server",
+			author: "DX Heroes (https://dxheroes.io)",
 		});
 
 		// Register all tools

packages/jira/src/tools/getTicket.ts

@@ -17,12 +17,7 @@ interface TicketDescription {
 export const registerGetTicketTools = (server: McpServer): void => {
 	// Define schema for ticket retrieval
 	const getTicketParams = {
-		jql: z.string().describe("JQL query string"),
-		maxResults: z
-			.number()
-			.optional()
-			.default(10)
-			.describe("Maximum number of results to return"),
+		issueIdOrKey: z.string().describe("The issue ID or key of the ticket"),
 	};
 
 	// Register all ticket retrieval tools
@@ -35,9 +30,9 @@ export const registerGetTicketTools = (server: McpServer): void => {
 	];
 
 	for (const tool of getTicketTools) {
-		server.tool(tool, getTicketParams, async ({ jql, maxResults }) => {
+		server.tool(tool, getTicketParams, async ({ issueIdOrKey }) => {
 			try {
-				const response = await jiraApi.executeJQL(jql, maxResults);
+				const response = await jiraApi.getTicket(issueIdOrKey);
 
 				if ("error" in response) {
 					return {
@@ -51,26 +46,23 @@ export const registerGetTicketTools = (server: McpServer): void => {
 					};
 				}
 
-				// Return only the ticket name and description
-				const tickets = response.issues.map((issue) => {
-					const descObj = issue.fields.description as
-						| TicketDescription
-						| undefined;
-					const description =
-						descObj?.content?.[0]?.content?.[0]?.text || "No description";
-
-					return {
-						key: issue.key,
-						summary: issue.fields.summary,
-						description,
-					};
-				});
+				const ticket = {
+					key: response.fields.key,
+					summary: response.fields.summary,
+					description: response.fields.description,
+					attachment: response.fields.attachment?.map((attachment) => ({
+						id: attachment.id,
+						filename: attachment.filename,
+						size: attachment.size,
+						content: attachment.content,
+					})),
+				};
 
 				return {
 					content: [
 						{
 							type: "text",
-							text: `Found ${tickets.length} tickets: ${JSON.stringify(tickets, null, 2)}`,
+							text: `Found ticket: ${JSON.stringify(ticket, null, 2)}`,
 						},
 					],
 				};

packages/jira/src/types.ts

@@ -2,102 +2,112 @@
  * Error response from Jira API
  */
 export interface JiraErrorResponse {
-  error: string | object;
+	error: string | object;
 }
 
 /**
  * Jira attachment
  */
 export interface JiraAttachment {
-  id: string;
-  title: string;
-  filename: string;
-  mimeType: string;
-  size: number;
-  content: string;
-  [key: string]: unknown;
+	id: string;
+	title: string;
+	filename: string;
+	mimeType: string;
+	size: number;
+	content: string;
+	[key: string]: unknown;
 }
 
 /**
  * Jira attachment response
  */
 export interface JiraAttachmentResponse {
-  id: string;
-  self: string;
-  filename: string;
-  [key: string]: unknown;
+	id: string;
+	self: string;
+	filename: string;
+	[key: string]: unknown;
 }
 
 /**
  * Jira ticket response
  */
 export interface JiraTicketResponse {
-  id: string;
-  key: string;
-  self: string;
-  [key: string]: unknown;
+	id: string;
+	key: string;
+	self: string;
+	[key: string]: unknown;
 }
 
 /**
  * Jira user
  */
 export interface JiraUser {
-  accountId: string;
-  displayName: string;
-  emailAddress?: string;
-  active: boolean;
-  [key: string]: unknown;
+	accountId: string;
+	displayName: string;
+	emailAddress?: string;
+	active: boolean;
+	[key: string]: unknown;
 }
 
 /**
  * Jira project
  */
 export interface JiraProject {
-  id: string;
-  key: string;
-  name: string;
-  [key: string]: unknown;
+	id: string;
+	key: string;
+	name: string;
+	[key: string]: unknown;
 }
 
 /**
  * Jira status
  */
 export interface JiraStatus {
-  id: string;
-  name: string;
-  statusCategory: {
-    id: number;
-    key: string;
-    name: string;
-    [key: string]: unknown;
-  };
-  [key: string]: unknown;
+	id: string;
+	name: string;
+	statusCategory: {
+		id: number;
+		key: string;
+		name: string;
+		[key: string]: unknown;
+	};
+	[key: string]: unknown;
 }
 
 /**
  * Jira issue
  */
 export interface JiraIssue {
-  id: string;
-  key: string;
-  self: string;
-  fields: {
-    summary: string;
-    description?: unknown;
-    status?: {
-      name: string;
-      [key: string]: unknown;
-    };
-    [key: string]: unknown;
-  };
-  [key: string]: unknown;
+	id: string;
+	key: string;
+	self: string;
+	fields: {
+		summary: string;
+		description?: {
+			type: string;
+			content?: {
+				type: string;
+				content?: {
+					type: string;
+					text: string;
+				}[];
+			}[];
+		};
+		status?: {
+			name: string;
+			[key: string]: unknown;
+		};
+		attachment?: JiraAttachment[];
+		[key: string]: unknown;
+	};
+	[key: string]: unknown;
 }
 
 /**
  * Jira search response
  */
 export interface JiraSearchResponse {
-  issues: JiraIssue[];
-  total: number;
-  [key: string]: unknown;
-} 
+	issues: JiraIssue[];
+	total: number;
+	[key: string]: unknown;
+}