fix: properly handle allow-tools all

Author: kombucha

src/app.ts

@@ -56,7 +56,7 @@ const ALLOW_TOOLS_OPTIONS_TUPLE = [
   "Comma separated list of tool ids (or all)",
   (val: string) => {
     if (val.trim().toLowerCase() === "all") {
-      return undefined;
+      return [];
     }
     return val.split(",").map((tool) => tool.trim());
   },

src/toolFilters.ts

@@ -15,8 +15,8 @@ const INTERNAL_DENIED_TOOLS = new Set(["customGet", "customPost", "customPut", "
 
 export const getToolFilter = (opts: CliFilteringOptions): ToolFilter => {
   return {
-    allowedTools: opts.allowTools ? new Set(opts.allowTools) : undefined,
-    deniedTools: opts.denyTools ? new Set(opts.denyTools) : undefined,
+    allowedTools: opts.allowTools?.length ? new Set(opts.allowTools) : undefined,
+    deniedTools: opts.denyTools?.length ? new Set(opts.denyTools) : undefined,
   };
 };