chore: add precommit task to deno.json and fix formatting issues in compose.ts and logging-plugin.ts

Author: yaonyan

deno.json

@@ -1,7 +1,9 @@
 {
   "vendor": true,
   "nodeModulesDir": "auto",
-  "tasks": {},
+  "tasks": {
+    "precommit": "deno lint && deno check && deno fmt"
+  },
   "workspace": ["./packages/core"],
   "imports": {
     "@es-toolkit/es-toolkit": "jsr:@es-toolkit/es-toolkit@^1.37.2",

packages/core/src/compose.ts

@@ -69,7 +69,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
@@ -108,7 +108,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,
@@ -158,7 +158,7 @@ export class ComposableMCPServer extends Server {
         toolName,
         result,
         "output",
-        args
+        args,
       ) as CallToolResult;
     });
   }
@@ -207,7 +207,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);
@@ -241,7 +241,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),
     );
   }
 
@@ -258,7 +258,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);
@@ -371,10 +371,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) {
@@ -411,9 +411,9 @@ 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()) {
+    for (const [toolId, toolData] of this.toolRegistry.entries()) {
       const defaultSchema = {
         type: "object",
         properties: {},
@@ -429,7 +429,7 @@ export class ComposableMCPServer extends Server {
       const processedTool = await this.applyTransformToolHooks(
         tempTool,
         toolId,
-        mode
+        mode,
       );
 
       this.toolRegistry.set(toolId, {
@@ -449,7 +449,7 @@ export class ComposableMCPServer extends Server {
               toolId,
               processedTool,
               this,
-              externalTools
+              externalTools,
             );
           }
         } catch {
@@ -465,10 +465,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) {
@@ -482,7 +482,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, [
@@ -552,7 +552,7 @@ export class ComposableMCPServer extends Server {
             tool.attribs.name === toolId
           );
         });
-      }
+      },
     )) as {
       tools: Record<string, ComposedTool>;
       cleanupClients: () => Promise<void>;
@@ -567,8 +567,8 @@ export class ComposableMCPServer extends Server {
       });
     });
 
-  // Trigger transformation hooks for all tools
-  await this.processToolsWithPlugins(tools, options.mode ?? "agentic");
+    // Trigger transformation hooks for all tools
+    await this.processToolsWithPlugins(tools, options.mode ?? "agentic");
 
     // Cleanup clients when server is closed (pretty-printed to match logging plugin)
     this.onclose = async () => {
@@ -604,7 +604,7 @@ export class ComposableMCPServer extends Server {
         toolId,
         tool.description || "No description available",
         jsonSchema(tool.inputSchema as any),
-        tool.execute
+        tool.execute,
       );
     });
 
@@ -639,9 +639,11 @@ 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(
+              ", ",
+            )
+          }`,
         );
       }
 

packages/core/src/plugins/built-in/logging-plugin.ts

@@ -9,7 +9,7 @@ export const createLoggingPlugin = (
     enabled?: boolean;
     verbose?: boolean;
     compact?: boolean;
-  } = {}
+  } = {},
 ): ToolPlugin => {
   const { enabled = true, verbose = false, compact = true } = options;
 
@@ -33,7 +33,7 @@ export const createLoggingPlugin = (
         const globalCount = publicToolNames.length;
 
         console.log(
-          `🧩 [${context.toolName}] ${pluginCount} plugins • ${externalCount} external • ${internalCount} internal • ${hiddenCount} hidden • ${globalCount} global`
+          `🧩 [${context.toolName}] ${pluginCount} plugins • ${externalCount} external • ${internalCount} internal • ${hiddenCount} hidden • ${globalCount} global`,
         );
       } else if (verbose) {
         console.log(`🧩 [${context.toolName}]`);
@@ -42,18 +42,18 @@ export const createLoggingPlugin = (
         // Get all tool information from server
         const server = context.server;
         const globalToolNames = Array.from(
-          new Set(server.getPublicToolNames().map(String))
+          new Set(server.getPublicToolNames().map(String)),
         );
 
         // Ensure uniqueness across categories and coerce to string[]
         const external: string[] = Array.from(
-          new Set(server.getExternalToolNames().map(String))
+          new Set(server.getExternalToolNames().map(String)),
         );
         const internal: string[] = Array.from(
-          new Set(server.getInternalToolNames().map(String))
+          new Set(server.getInternalToolNames().map(String)),
         );
         const hidden: string[] = Array.from(
-          new Set(server.getHiddenToolNames().map(String))
+          new Set(server.getHiddenToolNames().map(String)),
         );
         const globalNames: string[] = globalToolNames.map(String);
         const totalSet = new Set<string>([