Fix error message and error handling (#398)

NO-ISSUE Fix error message and error handling

Author: Yang-33

src/tools/cancelRichMenuDefault.ts

@@ -1,6 +1,9 @@
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { messagingApi } from "@line/bot-sdk";
-import { createSuccessResponse } from "../common/response.js";
+import {
+  createErrorResponse,
+  createSuccessResponse,
+} from "../common/response.js";
 import { AbstractTool } from "./AbstractTool.js";
 
 export default class CancelRichMenuDefault extends AbstractTool {
@@ -22,8 +25,14 @@ export default class CancelRichMenuDefault extends AbstractTool {
         },
       },
       async () => {
-        const response = await this.client.cancelDefaultRichMenu();
-        return createSuccessResponse(response);
+        try {
+          const response = await this.client.cancelDefaultRichMenu();
+          return createSuccessResponse(response);
+        } catch (error) {
+          return createErrorResponse(
+            `Failed to cancel default rich menu: ${error.message}`,
+          );
+        }
       },
     );
   }

src/tools/getMessageQuota.ts

@@ -1,6 +1,9 @@
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { messagingApi } from "@line/bot-sdk";
-import { createSuccessResponse } from "../common/response.js";
+import {
+  createErrorResponse,
+  createSuccessResponse,
+} from "../common/response.js";
 import { AbstractTool } from "./AbstractTool.js";
 
 export default class GetMessageQuota extends AbstractTool {
@@ -23,14 +26,20 @@ export default class GetMessageQuota extends AbstractTool {
         },
       },
       async () => {
-        const messageQuotaResponse = await this.client.getMessageQuota();
-        const messageQuotaConsumptionResponse =
-          await this.client.getMessageQuotaConsumption();
-        const response = {
-          limited: messageQuotaResponse.value,
-          totalUsage: messageQuotaConsumptionResponse.totalUsage,
-        };
-        return createSuccessResponse(response);
+        try {
+          const messageQuotaResponse = await this.client.getMessageQuota();
+          const messageQuotaConsumptionResponse =
+            await this.client.getMessageQuotaConsumption();
+          const response = {
+            limited: messageQuotaResponse.value,
+            totalUsage: messageQuotaConsumptionResponse.totalUsage,
+          };
+          return createSuccessResponse(response);
+        } catch (error) {
+          return createErrorResponse(
+            `Failed to get message quota: ${error.message}`,
+          );
+        }
       },
     );
   }

src/tools/getRichMenuList.ts

@@ -31,7 +31,7 @@ export default class GetRichMenuList extends AbstractTool {
           return createSuccessResponse(response);
         } catch (error) {
           return createErrorResponse(
-            `Failed to broadcast message: ${error.message}`,
+            `Failed to get rich menu list: ${error.message}`,
           );
         }
       },

src/tools/setRichMenuDefault.ts

@@ -1,6 +1,9 @@
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { messagingApi } from "@line/bot-sdk";
-import { createSuccessResponse } from "../common/response.js";
+import {
+  createErrorResponse,
+  createSuccessResponse,
+} from "../common/response.js";
 import { AbstractTool } from "./AbstractTool.js";
 import { z } from "zod";
 
@@ -32,8 +35,14 @@ export default class SetRichMenuDefault extends AbstractTool {
         },
       },
       async ({ richMenuId }) => {
-        const response = await this.client.setDefaultRichMenu(richMenuId);
-        return createSuccessResponse(response);
+        try {
+          const response = await this.client.setDefaultRichMenu(richMenuId);
+          return createSuccessResponse(response);
+        } catch (error) {
+          return createErrorResponse(
+            `Failed to set default rich menu: ${error.message}`,
+          );
+        }
       },
     );
   }