@mrrlin-dev/mcp 0.4.91 → 0.4.92

package/dist/bin.cjs

@@ -50980,7 +50980,7 @@ async function invokeCapability(server, toolName, payload) {
       throw classifyInvokeError(error51, target, toolName);
     }
     if (result.isError) {
-      throw new Error(`invokeCapability: tool "${toolName}" reported isError -- ${extractFirstText(result.content)}`);
+      throw new InvokeToolExecutionError(toolName, extractFirstText(result.content));
     }
     return { content: result.content, url: extractFirstUrl(result.content) };
   } finally {
@@ -50994,6 +50994,12 @@ var InvokeToolNotFoundError = class extends Error {
     this.name = "InvokeToolNotFoundError";
   }
 };
+var InvokeToolExecutionError = class extends Error {
+  constructor(toolName, detail, cause) {
+    super(`invokeCapability: tool "${toolName}" reported isError -- ${detail}`, { cause });
+    this.name = "InvokeToolExecutionError";
+  }
+};
 var InvokeUnreachableError = class extends Error {
   constructor(url2, cause) {
     super(`invokeCapability: could not reach the granted server at ${url2}`, { cause });
@@ -51027,9 +51033,12 @@ function classifyInvokeError(error51, url2, toolName) {
     return new InvokeUnreachableError(url2, error51);
   }
   if (error51 instanceof McpError) {
-    if (error51.code === ErrorCode.MethodNotFound || error51.code === ErrorCode.InvalidParams) {
+    if (error51.code === ErrorCode.MethodNotFound) {
       return new InvokeToolNotFoundError(toolName, error51);
     }
+    if (error51.code === ErrorCode.InvalidParams) {
+      return new InvokeMalformedResultError(`invalid parameters for tool "${toolName}": ${error51.message}`, error51);
+    }
     return new InvokeMalformedResultError(error51.message, error51);
   }
   if (error51 instanceof TypeError || error51 instanceof Error && (error51.name === "TimeoutError" || error51.name === "AbortError")) {
@@ -51041,8 +51050,15 @@ function isRecord(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 function extractFirstUrl(content) {
-  const match = findFirstMatch(content, /https?:\/\/\S+/);
-  return match ? match.replace(/[.,;:!?"')\]}>]+$/, "") : null;
+  const match = findFirstMatch(content, /https?:\/\/[^\s<>|]+/);
+  if (!match) return null;
+  const cleaned = match.replace(/[.,;:!?"')\]}]+$/, "");
+  try {
+    const parsed = new URL(cleaned);
+    return parsed.href;
+  } catch {
+    return null;
+  }
 }
 function extractFirstText(content) {
   if (Array.isArray(content)) {
@@ -51086,9 +51102,11 @@ function parsePullRequestUrl(url2) {
       const owner = parts[pullIdx - 2];
       const repo = parts[pullIdx - 1];
       const pullPart = parts[pullIdx + 1];
-      const pullNumber = pullPart ? parseInt(pullPart, 10) : NaN;
-      if (owner && repo && !Number.isNaN(pullNumber)) {
-        return { owner, repo, pullNumber };
+      if (pullPart && /^\d+$/.test(pullPart)) {
+        const pullNumber = parseInt(pullPart, 10);
+        if (owner && repo && !Number.isNaN(pullNumber)) {
+          return { owner, repo, pullNumber };
+        }
       }
     }
   } catch {
@@ -51128,7 +51146,7 @@ async function deliverToGit(server, toolName, payload) {
     }
     return invokeCapability(server, toolName, args);
   }
-  if (isDefaultGitHubRoute && toolName === defaultRoute.operations.pull_request_state) {
+  if (isDefaultGitHubRoute && (toolName === defaultRoute.operations.pull_request_state || toolName === "pull_request_read")) {
     const { operation: _operation, ...args } = payload;
     const urlParts = parsePullRequestUrl(payload.pullRequestUrl);
     let owner = args.owner ?? urlParts.owner;
@@ -51144,7 +51162,12 @@ async function deliverToGit(server, toolName, payload) {
     if (owner) adaptedArgs.owner = owner;
     if (repo) adaptedArgs.repo = repo;
     if (pullNumber !== void 0) {
-      adaptedArgs.pull_number = pullNumber;
+      if (toolName === "pull_request_read") {
+        adaptedArgs.method = "get";
+        adaptedArgs.pullNumber = pullNumber;
+      } else {
+        adaptedArgs.pull_number = pullNumber;
+      }
     }
     return invokeCapability(server, toolName, adaptedArgs);
   }
@@ -51232,8 +51255,10 @@ async function deliverToSend(server, toolName, payload) {
   const defaultRoute = server.id ? getDefaultCapabilityRouteForServer(server.id) : null;
   const isDefaultSlackRoute = defaultRoute?.capability === "send";
   if (payload.operation === "dispatch" && isDefaultSlackRoute && toolName === defaultRoute.operations.dispatch) {
-    const channelId = args.channel_id ?? args.destination ?? args.channel ?? args.to;
-    const text = args.text ?? args.body ?? args.message ?? args.content;
+    const rawChannel = args.channel_id ?? args.destination ?? args.channel ?? args.to;
+    const channelId = typeof rawChannel === "string" ? rawChannel.trim() : rawChannel !== void 0 ? String(rawChannel) : void 0;
+    const rawText = args.text ?? args.body ?? args.message ?? args.content;
+    const text = typeof rawText === "string" ? rawText.trim() : rawText !== void 0 ? String(rawText) : void 0;
     const adaptedArgs = {};
     if (channelId !== void 0) adaptedArgs.channel_id = channelId;
     if (text !== void 0) adaptedArgs.text = text;
@@ -57691,7 +57716,7 @@ Do NOT derail the current task - if the operator has an in-flight question, answ
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.4.91";
+var PKG_VERSION = "0.4.92";
 
 // src/api-base-url-migrate.ts
 var import_node_fs13 = __toESM(require("node:fs"), 1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrlin-dev/mcp",
-  "version": "0.4.91",
+  "version": "0.4.92",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"