@rui.branco/jira-mcp 1.7.1 → 1.7.3

package/index.js

@@ -459,7 +459,73 @@ async function parseInlineFormatting(text, instance = defaultInstance) {
     nodes.push({ type: "text", text: text.substring(lastIndex) });
   }
 
-  return nodes.length > 0 ? nodes : [{ type: "text", text: text }];
+  const finalNodes = nodes.length > 0 ? nodes : [{ type: "text", text }];
+  return autoLinkTextNodes(finalNodes, instance);
+}
+
+// Post-process plain text nodes: convert bare URLs and bare Jira ticket keys
+// (e.g. PROJ-123) into ADF link nodes. Jira does NOT auto-link plain text in
+// ADF — links must be explicit `link` marks.
+function autoLinkTextNodes(nodes, instance = defaultInstance) {
+  const baseUrl = instance && instance.baseUrl;
+  const result = [];
+  // Bare URL OR Jira wiki-style [text|url] OR bare ticket key (PROJ-123)
+  const re = /(\[([^\]|\n]+)\|(https?:\/\/[^\]\s]+)\])|(https?:\/\/[^\s<>()\[\]]+)|(\b[A-Z][A-Z0-9]+-\d+\b)/g;
+  for (const node of nodes) {
+    if (node.type !== "text" || (node.marks && node.marks.length > 0)) {
+      result.push(node);
+      continue;
+    }
+    const text = node.text;
+    let last = 0;
+    let m;
+    re.lastIndex = 0;
+    while ((m = re.exec(text)) !== null) {
+      if (m.index > last) {
+        result.push({ type: "text", text: text.substring(last, m.index) });
+      }
+      if (m[1]) {
+        // [label|url] wiki markup
+        result.push({
+          type: "text",
+          text: m[2],
+          marks: [{ type: "link", attrs: { href: m[3] } }],
+        });
+      } else if (m[4]) {
+        // Bare URL — strip trailing punctuation that's unlikely to be part of the URL
+        let url = m[4];
+        const trailingMatch = url.match(/[.,;:!?]+$/);
+        let consumed = m[0].length;
+        if (trailingMatch) {
+          url = url.substring(0, url.length - trailingMatch[0].length);
+          consumed -= trailingMatch[0].length;
+        }
+        result.push({
+          type: "text",
+          text: url,
+          marks: [{ type: "link", attrs: { href: url } }],
+        });
+        last = m.index + consumed;
+        re.lastIndex = last;
+        continue;
+      } else if (m[5] && baseUrl) {
+        // Bare ticket key
+        const key = m[5];
+        result.push({
+          type: "text",
+          text: key,
+          marks: [{ type: "link", attrs: { href: `${baseUrl}/browse/${key}` } }],
+        });
+      } else if (m[5]) {
+        result.push({ type: "text", text: m[5] });
+      }
+      last = m.index + m[0].length;
+    }
+    if (last < text.length) {
+      result.push({ type: "text", text: text.substring(last) });
+    }
+  }
+  return result;
 }
 
 // Parse text with markdown formatting and @mentions, build ADF content
@@ -2505,24 +2571,26 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 
       const content = [{ type: "text", text: result.text }];
 
-      // Add Jira images. Anthropic vision only accepts png/jpeg/gif/webp;
-      // SVG and other formats must be skipped or the API rejects the whole
-      // response with "Improperly formed request". Also skip files over 5MB.
-      const mimeByExt = {
-        ".png": "image/png",
-        ".jpg": "image/jpeg",
-        ".jpeg": "image/jpeg",
-        ".gif": "image/gif",
-        ".webp": "image/webp",
-      };
+      // Anthropic vision only accepts png/jpeg/gif/webp under 5MB. We sniff
+      // the magic bytes instead of trusting the filename, since a misnamed
+      // file (e.g. SVG saved as .png) gets the whole response rejected with
+      // "Could not process image".
       const MAX_IMAGE_BYTES = 5 * 1024 * 1024;
+      const detectImageMime = (buf) => {
+        if (!buf || buf.length < 12) return null;
+        if (buf[0] === 0x89 && buf[1] === 0x50 && buf[2] === 0x4e && buf[3] === 0x47) return "image/png";
+        if (buf[0] === 0xff && buf[1] === 0xd8 && buf[2] === 0xff) return "image/jpeg";
+        if (buf[0] === 0x47 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x38) return "image/gif";
+        if (buf[0] === 0x52 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x46 &&
+            buf[8] === 0x57 && buf[9] === 0x45 && buf[10] === 0x42 && buf[11] === 0x50) return "image/webp";
+        return null;
+      };
       for (const imagePath of result.jiraImages) {
         try {
-          const ext = path.extname(imagePath).toLowerCase();
-          const mimeType = mimeByExt[ext];
-          if (!mimeType) continue;
           const imageData = fs.readFileSync(imagePath);
           if (imageData.length > MAX_IMAGE_BYTES) continue;
+          const mimeType = detectImageMime(imageData);
+          if (!mimeType) continue;
           content.push({
             type: "image",
             data: imageData.toString("base64"),
@@ -2533,17 +2601,18 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
       }
 
-      // Add Figma images (now supports multiple images per design)
+      // Figma images (multiple per design). Validate magic bytes here too.
       for (const design of result.figmaDesigns) {
         if (design.images && design.images.length > 0) {
           for (const img of design.images) {
-            if (img.buffer && img.buffer.length <= MAX_IMAGE_BYTES) {
-              content.push({
-                type: "image",
-                data: img.buffer.toString("base64"),
-                mimeType: "image/png",
-              });
-            }
+            if (!img.buffer || img.buffer.length > MAX_IMAGE_BYTES) continue;
+            const mimeType = detectImageMime(img.buffer);
+            if (!mimeType) continue;
+            content.push({
+              type: "image",
+              data: img.buffer.toString("base64"),
+              mimeType: mimeType,
+            });
           }
         }
       }
@@ -3828,5 +3897,5 @@ if (require.main === module) {
 
 // Export for testing
 if (typeof module !== "undefined") {
-  module.exports = { buildCommentADF, parseInlineFormatting, findJiraTicketKeys, resolveTeamId, fetchJiraTeams, listTeams, searchTeamsViaJql };
+  module.exports = { buildCommentADF, parseInlineFormatting, autoLinkTextNodes, findJiraTicketKeys, resolveTeamId, fetchJiraTeams, listTeams, searchTeamsViaJql };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rui.branco/jira-mcp",
-  "version": "1.7.1",
+  "version": "1.7.3",
   "description": "Jira MCP server for Claude Code - fetch tickets, search with JQL, update tickets, manage comments, change status, and get Figma designs",
   "main": "index.js",
   "bin": {