feat: dynamic actor loading is enabled by default

Author: MichalKalita

.actor/input_schema.json

@@ -25,7 +25,7 @@
             "title": "Enable adding Actors based on context and use-case (experimental, check if it supported by your client)",
             "type": "boolean",
             "description": "When enabled, the server can dynamically add Actors as tools based on user requests and context. \n\nNote: MCP client must support notification on tool updates. To try it, you can use the [Tester MCP Client](https://apify.com/jiri.spilka/tester-mcp-client). This is an experimental feature and may require client-specific support.",
-            "default": false
+            "default": true
         },
         "maxActorMemoryBytes": {
             "title": "Limit the maximum memory used by an Actor",
@@ -48,7 +48,7 @@
             "description": "Specify the input for the Actor that will be used for debugging in normal mode",
             "editor": "json",
             "prefill": {
-            "query": "hello world"
+                "query": "hello world"
             }
         }
     }

src/input.ts

@@ -24,7 +24,7 @@ export function processInput(originalInput: Partial<Input>): Input {
             log.warning('enableActorAutoLoading is deprecated, use enableAddingActors instead');
             input.enableAddingActors = input.enableActorAutoLoading === true || input.enableActorAutoLoading === 'true';
         } else {
-            input.enableAddingActors = false;
+            input.enableAddingActors = true;
         }
     } else {
         input.enableAddingActors = input.enableAddingActors === true || input.enableAddingActors === 'true';

src/mcp/server.ts

@@ -50,7 +50,7 @@ export class ActorsMcpServer {
 
     constructor(options: ActorsMcpServerOptions = {}, setupSigintHandler = true) {
         this.options = {
-            enableAddingActors: options.enableAddingActors ?? false,
+            enableAddingActors: options.enableAddingActors ?? true,
             enableDefaultActors: options.enableDefaultActors ?? true, // Default to true for backward compatibility
         };
         this.server = new Server(

src/stdio.ts

@@ -33,8 +33,9 @@ import { getActorsAsTools } from './tools/index.js';
  */
 interface CliArgs {
     actors?: string;
-    'enable-adding-actors'?: boolean;
-    enableActorAutoLoading?: boolean;
+    enableAddingActors: boolean;
+    /** @deprecated */
+    enableActorAutoLoading: boolean;
 }
 
 // Configure logging, set to ERROR
@@ -50,12 +51,12 @@ const argv = yargs(hideBin(process.argv))
     })
     .option('enable-adding-actors', {
         type: 'boolean',
-        default: false,
+        default: true,
         describe: 'Enable dynamically adding Actors as tools based on user requests',
     })
     .option('enableActorAutoLoading', {
         type: 'boolean',
-        default: false,
+        default: true,
         hidden: true,
         describe: 'Deprecated: use enable-adding-actors instead',
     })
@@ -69,7 +70,7 @@ const argv = yargs(hideBin(process.argv))
     .epilogue('For more information, visit https://github.com/apify/actors-mcp-server')
     .parseSync() as CliArgs;
 
-const enableAddingActors = argv['enable-adding-actors'] || argv.enableActorAutoLoading || false;
+const enableAddingActors = argv.enableAddingActors && argv.enableActorAutoLoading;
 const actors = argv.actors as string || '';
 const actorList = actors ? actors.split(',').map((a: string) => a.trim()) : [];
 

tests/unit/input.test.ts

@@ -39,11 +39,11 @@ describe('processInput', () => {
         expect(processed.enableAddingActors).toBe(false);
     });
 
-    it('should default enableAddingActors to false when not provided', async () => {
+    it('should default enableAddingActors to true when not provided', async () => {
         const input: Partial<Input> = {
             actors: ['actor1'],
         };
         const processed = processInput(input);
-        expect(processed.enableAddingActors).toBe(false);
+        expect(processed.enableAddingActors).toBe(true);
     });
 });