fix actorNameToToolName

Author: jirispilka

src/actors.ts

@@ -8,13 +8,8 @@ import type { ActorDefinitionPruned, ActorDefinitionWithDesc, SchemaProperties,
 export function actorNameToToolName(actorName: string): string {
     return actorName
         .replace(/\//g, '-slash-')
-        .replace(/\./g, '-dot-');
-}
-
-export function toolNameToActorName(toolName: string): string {
-    return toolName
-        .replace(/-slash-/g, '/')
-        .replace(/-dot-/g, '.');
+        .replace(/\./g, '-dot-')
+        .slice(0, 64);
 }
 
 /**

src/input.ts

@@ -13,7 +13,7 @@ export async function processInput(originalInput: Partial<Input>): Promise<Input
         input.actors = input.actors.split(',').map((format: string) => format.trim()) as string[];
     }
     if (!input.enableActorAutoLoading) {
-        input.enableActorAutoLoading = true;
+        input.enableActorAutoLoading = false;
     }
     return input;
 }

src/server.ts

@@ -12,12 +12,10 @@ import { ApifyClient } from 'apify-client';
 import type { AxiosRequestConfig } from 'axios';
 
 import {
-    actorNameToToolName,
     filterSchemaProperties,
     getActorDefinition,
     getActorsAsTools,
     shortenProperties,
-    toolNameToActorName,
 } from './actors.js';
 import {
     ACTOR_OUTPUT_MAX_CHARS_PER_ITEM,
@@ -91,7 +89,7 @@ export class ApifyMcpServer {
         input: unknown,
         callOptions: ActorCallOptions | undefined = undefined,
     ): Promise<object[]> {
-        const name = toolNameToActorName(actorName);
+        const name = actorName;
         try {
             log.info(`Calling actor ${name} with input: ${JSON.stringify(input)}`);
 
@@ -162,8 +160,7 @@ export class ApifyMcpServer {
         this.server.setRequestHandler(CallToolRequestSchema, async (request) => {
             const { name, arguments: args } = request.params;
 
-            // Anthropic can't handle '/' in tool names. The replace is only necessary when calling the tool from stdio clients.
-            const tool = this.tools.get(name) || this.tools.get(actorNameToToolName(name));
+            const tool = Array.from(this.tools.values()).find((t) => t.name === name || t.actorFullName === name);
             if (!tool) {
                 throw new Error(`Unknown tool: ${name}`);
             }

src/tools.ts

@@ -4,7 +4,7 @@ import { ApifyClient } from 'apify-client';
 import { z } from 'zod';
 import { zodToJsonSchema } from 'zod-to-json-schema';
 
-import { actorNameToToolName, toolNameToActorName } from './actors.js';
+import { actorNameToToolName } from './actors.js';
 import { ACTOR_README_MAX_LENGTH, InternalTools } from './const.js';
 import type { ActorStorePruned, PricingInfo, Tool } from './types.js';
 
@@ -39,15 +39,14 @@ export const RemoveActorToolArgsSchema = z.object({
 
 export const AddActorToToolsArgsSchema = z.object({
     actorName: z.string()
-        .describe('Add an Actor to available tools by Actor ID or Actor name.')
-        .transform((val) => toolNameToActorName(val)),
+        .describe('Add an Actor to available tools by Actor ID or Actor full name.'
+            + 'Actor name is always composed from `username/name`'),
 });
 
 export const GetActorDefinition = z.object({
     actorName: z.string()
         .describe('Retrieve input, readme, and other details for Actor ID or Actor full name. '
-            + 'Actor name is always composed from `username/name`')
-        .transform((val) => toolNameToActorName(val)),
+            + 'Actor name is always composed from `username/name`'),
     limit: z.number()
         .int()
         .default(ACTOR_README_MAX_LENGTH)

tests/actors-test.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect } from 'vitest';
 
-import { actorNameToToolName, toolNameToActorName } from '../src/actors.js';
+import { actorNameToToolName } from '../src/actors.js';
 
 describe('actors', () => {
     describe('actorNameToToolName', () => {
@@ -20,24 +20,11 @@ describe('actors', () => {
         it('should handle strings with multiple slashes and dots', () => {
             expect(actorNameToToolName('actor/name.with/multiple.parts')).toBe('actor-slash-name-dot-with-slash-multiple-dot-parts');
         });
-    });
-
-    describe('toolNameToActorName', () => {
-        it('should convert dash notation back to slashes and dots', () => {
-            expect(toolNameToActorName('apify-slash-web-scraper')).toBe('apify/web-scraper');
-            expect(toolNameToActorName('my-dot-actor-dot-name')).toBe('my.actor.name');
-        });
-
-        it('should handle empty strings', () => {
-            expect(toolNameToActorName('')).toBe('');
-        });
-
-        it('should handle strings without dash notation', () => {
-            expect(toolNameToActorName('actorname')).toBe('actorname');
-        });
 
-        it('should handle strings with multiple dash notations', () => {
-            expect(toolNameToActorName('actor-slash-name-dot-with-slash-multiple-dot-parts')).toBe('actor/name.with/multiple.parts');
+        it('should handle tool names longer than 64 characters', () => {
+            const longName = 'a'.repeat(70);
+            const expected = 'a'.repeat(64);
+            expect(actorNameToToolName(longName)).toBe(expected);
         });
     });
 });