refactor(mcp): remove redundant try-catch in tools, let wrapper handle errors

- Remove inner try-catch from storage, databaseSQL, databaseNoSQL,
  security-rule, cloudrun, miniprogram, download, invite-code, dataModel
- Errors now propagate to tool-wrapper for unified handling (telemetry,
  enhanced error message, GitHub Issue link)
- Keep business-return only (e.g. force confirmation, path validation)

Author: binggg

mcp/src/tools/cloudrun.ts

@@ -192,18 +192,17 @@ export function registerCloudRunTools(server: ExtendedMcpServer) {
       }
     },
     async (args: queryCloudRunInput) => {
-      try {
-        const input = args;
-        const manager = await getManager();
+      const input = args;
+      const manager = await getManager();
 
-        if (!manager) {
-          throw new Error("Failed to initialize CloudBase manager. Please check your credentials and environment configuration.");
-        }
+      if (!manager) {
+        throw new Error("Failed to initialize CloudBase manager. Please check your credentials and environment configuration.");
+      }
 
-        const cloudrunService = manager.cloudrun;
+      const cloudrunService = manager.cloudrun;
 
-        switch (input.action) {
-          case 'list': {
+      switch (input.action) {
+        case 'list': {
             const listParams: any = {
               pageSize: input.pageSize,
               pageNum: input.pageNum,
@@ -295,23 +294,8 @@ export function registerCloudRunTools(server: ExtendedMcpServer) {
             };
           }
 
-          default:
-            throw new Error(`Unsupported action: ${input.action}`);
-        }
-
-      } catch (error: any) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: JSON.stringify({
-                success: false,
-                error: error.message || 'Unknown error occurred',
-                message: "Failed to query CloudRun information. Please check your permissions and try again."
-              }, null, 2)
-            }
-          ]
-        };
+        default:
+          throw new Error(`Unsupported action: ${input.action}`);
       }
     }
   );
@@ -335,24 +319,22 @@ export function registerCloudRunTools(server: ExtendedMcpServer) {
       }
     },
     async (args: ManageCloudRunInput) => {
-      try {
-        const input = args;
-        const manager = await getManager();
+      const input = args;
+      const manager = await getManager();
 
-        if (!manager) {
-          throw new Error("Failed to initialize CloudBase manager. Please check your credentials and environment configuration.");
-        }
+      if (!manager) {
+        throw new Error("Failed to initialize CloudBase manager. Please check your credentials and environment configuration.");
+      }
 
-        const cloudrunService = manager.cloudrun;
-        let targetPath: string | undefined;
+      const cloudrunService = manager.cloudrun;
+      let targetPath: string | undefined;
 
-        // Validate and normalize path for operations that require it
-        if (input.targetPath) {
-          targetPath = validateAndNormalizePath(input.targetPath);
-        }
+      if (input.targetPath) {
+        targetPath = validateAndNormalizePath(input.targetPath);
+      }
 
-        switch (input.action) {
-          case 'createAgent': {
+      switch (input.action) {
+        case 'createAgent': {
             if (!targetPath) {
               throw new Error("targetPath is required for createAgent operation");
             }
@@ -939,23 +921,8 @@ for await (let x of res.textStream) {
             };
           }
 
-          default:
-            throw new Error(`Unsupported action: ${input.action}`);
-        }
-
-      } catch (error: any) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: JSON.stringify({
-                success: false,
-                error: error.message || 'Unknown error occurred',
-                message: `Failed to ${args.action} CloudRun service. Please check your permissions and parameters.`
-              }, null, 2)
-            }
-          ]
-        };
+        default:
+          throw new Error(`Unsupported action: ${input.action}`);
       }
     }
   );

mcp/src/tools/dataModel.ts

@@ -655,13 +655,12 @@ export function registerDataModelTools(server: ExtendedMcpServer) {
       name?: string;
       names?: string[];
     }) => {
-      try {
-        const cloudbase = await getManager();
-        let currentEnvId = await getEnvId(cloudBaseOptions);
+      const cloudbase = await getManager();
+      let currentEnvId = await getEnvId(cloudBaseOptions);
 
-        let result;
+      let result;
 
-        switch (action) {
+      switch (action) {
           case "get":
             if (!name) {
               throw new Error("获取数据模型需要提供模型名称");
@@ -982,28 +981,8 @@ export function registerDataModelTools(server: ExtendedMcpServer) {
               throw error;
             }
 
-          default:
-            throw new Error(`不支持的操作类型: ${action}`);
-        }
-      } catch (error: any) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: JSON.stringify(
-                {
-                  success: false,
-                  action,
-                  error: error.message || error.original?.Message || "未知错误",
-                  code: error.original?.Code,
-                  message: "数据模型操作失败",
-                },
-                null,
-                2
-              ),
-            },
-          ],
-        };
+        default:
+          throw new Error(`不支持的操作类型: ${action}`);
       }
     }
   );
@@ -1092,12 +1071,10 @@ classDiagram
       publish?: boolean;
       dbInstanceType?: string;
     }) => {
-      try {
-        const cloudbase = await getManager();
-        let currentEnvId = await getEnvId(cloudBaseOptions);
+      const cloudbase = await getManager();
+      let currentEnvId = await getEnvId(cloudBaseOptions);
 
-        // 使用mermaidToJsonSchema转换Mermaid图表
-        const schemas = mermaidToJsonSchema(mermaidDiagram);
+      const schemas = mermaidToJsonSchema(mermaidDiagram);
 
         if (!schemas || Object.keys(schemas).length === 0) {
           return {
@@ -1226,25 +1203,6 @@ classDiagram
             },
           ],
         };
-      } catch (error: any) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: JSON.stringify(
-                {
-                  success: false,
-                  error: error.message || error.original?.Message || "未知错误",
-                  code: error.original?.Code,
-                  message: "数据模型修改操作失败",
-                },
-                null,
-                2
-              ),
-            },
-          ],
-        };
-      }
     }
   );
 }