fix: standardize formatting by removing trailing commas in function parameters

Author: yaonyan

packages/core/src/compose.ts

@@ -70,7 +70,7 @@ export class ComposableMCPServer extends Server {
     _toolName: string,
     args: unknown,
     _mode: "input" | "output",
-    _originalArgs?: unknown,
+    _originalArgs?: unknown
   ): unknown {
     // For now, just return args unchanged
     // TODO: Implement transformResult hooks for runtime transformation
@@ -109,7 +109,7 @@ export class ComposableMCPServer extends Server {
     description: string,
     paramsSchema: Schema<T>,
     cb: (args: T, extra?: unknown) => unknown,
-    options: { internal?: boolean; plugins?: ToolPlugin[] } = {},
+    options: { internal?: boolean; plugins?: ToolPlugin[] } = {}
   ) {
     this.toolRegistry.set(name, {
       callback: cb as ToolCallback,
@@ -159,7 +159,7 @@ export class ComposableMCPServer extends Server {
         toolName,
         result,
         "output",
-        args,
+        args
       ) as CallToolResult;
     });
   }
@@ -208,7 +208,7 @@ export class ComposableMCPServer extends Server {
     const processedArgs = this.applyPluginTransforms(
       resolvedName,
       args,
-      "input",
+      "input"
     );
     const result = await callback(processedArgs);
     return this.applyPluginTransforms(resolvedName, result, "output", args);
@@ -242,7 +242,7 @@ export class ComposableMCPServer extends Server {
     const hiddenSet = new Set(this.getHiddenToolNames());
 
     return allRegistered.filter(
-      (n) => !publicSet.has(n) && !internalSet.has(n) && !hiddenSet.has(n),
+      (n) => !publicSet.has(n) && !internalSet.has(n) && !hiddenSet.has(n)
     );
   }
 
@@ -259,7 +259,7 @@ export class ComposableMCPServer extends Server {
    * Get internal tool schema by name
    */
   getInternalToolSchema(
-    name: string,
+    name: string
   ): { description: string; schema: JSONSchema } | undefined {
     const tool = this.toolRegistry.get(name);
     const config = this.toolConfigs.get(name);
@@ -372,10 +372,10 @@ export class ComposableMCPServer extends Server {
   private async applyTransformToolHooks(
     tool: ComposedTool,
     toolName: string,
-    mode: "agentic" | "agentic_workflow",
+    mode: "agentic" | "agentic_workflow"
   ): Promise<ComposedTool> {
     const transformPlugins = this.globalPlugins.filter(
-      (p) => p.transformTool && shouldApplyPlugin(p, mode),
+      (p) => p.transformTool && shouldApplyPlugin(p, mode)
     );
 
     if (transformPlugins.length === 0) {
@@ -412,7 +412,7 @@ export class ComposableMCPServer extends Server {
    */
   private async processToolsWithPlugins(
     externalTools: Record<string, ComposedTool>,
-    mode: "agentic" | "agentic_workflow",
+    mode: "agentic" | "agentic_workflow"
   ): Promise<void> {
     for (const [toolId, toolData] of this.toolRegistry.entries()) {
       const defaultSchema = {
@@ -430,7 +430,7 @@ export class ComposableMCPServer extends Server {
       const processedTool = await this.applyTransformToolHooks(
         tempTool,
         toolId,
-        mode,
+        mode
       );
 
       this.toolRegistry.set(toolId, {
@@ -450,7 +450,7 @@ export class ComposableMCPServer extends Server {
               toolId,
               processedTool,
               this,
-              externalTools,
+              externalTools
             );
           }
         } catch {
@@ -466,10 +466,10 @@ export class ComposableMCPServer extends Server {
    * Trigger composeEnd hooks for all plugins
    */
   private async triggerComposeEndHooks(
-    context: ComposeEndContext,
+    context: ComposeEndContext
   ): Promise<void> {
     const endPlugins = this.globalPlugins.filter(
-      (p) => p.composeEnd && shouldApplyPlugin(p, context.mode),
+      (p) => p.composeEnd && shouldApplyPlugin(p, context.mode)
     );
 
     for (const plugin of endPlugins) {
@@ -483,7 +483,7 @@ export class ComposableMCPServer extends Server {
     name: string | null,
     description: string,
     depsConfig: z.infer<typeof McpSettingsSchema> = { mcpServers: {} },
-    options: ComposeDefinition["options"] = { mode: "agentic" },
+    options: ComposeDefinition["options"] = { mode: "agentic" }
   ) {
     const refDesc = options.refs?.join("") ?? "";
     const { tagToResults } = parseTags(description + refDesc, ["tool", "fn"]);
@@ -550,7 +550,7 @@ export class ComposableMCPServer extends Server {
             tool.attribs.name === toolId
           );
         });
-      },
+      }
     )) as {
       tools: Record<string, ComposedTool>;
       cleanupClients: () => Promise<void>;
@@ -595,7 +595,7 @@ export class ComposableMCPServer extends Server {
     const contextToolNames = toolNameToDetailList
       .map(([name]) => name)
       .filter(
-        (n) => !globalToolNames.includes(n) && !hideToolNames.includes(n),
+        (n) => !globalToolNames.includes(n) && !hideToolNames.includes(n)
       );
 
     // For agentic interface: external tools (non-hidden) + internal tools
@@ -604,11 +604,18 @@ export class ComposableMCPServer extends Server {
     // Add global tools to server
     globalToolNames.forEach((toolId) => {
       const tool = tools[toolId];
+      if (!tool) {
+        throw new Error(
+          `Global tool ${toolId} not found in registry, available: ${Object.keys(
+            tools
+          ).join(", ")}`
+        );
+      }
       this.tool(
         toolId,
         tool.description || "No description available",
         jsonSchema(tool.inputSchema as any),
-        tool.execute,
+        tool.execute
       );
     });
 
@@ -646,11 +653,9 @@ export class ComposableMCPServer extends Server {
       }
       if (!tool) {
         throw new Error(
-          `Action ${toolName} not found, available action list: ${
-            allToolNames.join(
-              ", ",
-            )
-          }`,
+          `Action ${toolName} not found, available action list: ${allToolNames.join(
+            ", "
+          )}`
         );
       }
 
@@ -659,10 +664,10 @@ export class ComposableMCPServer extends Server {
         (tool.inputSchema.jsonSchema as JSONSchema) ??
           // Standard definition
           tool.inputSchema ?? {
-          type: "object",
-          properties: {},
-          required: [],
-        };
+            type: "object",
+            properties: {},
+            required: [],
+          };
 
       const baseProperties =
         baseSchema.type === "object" && baseSchema.properties