fix: actor to tool name

Author: jirispilka

src/actors.ts

@@ -3,12 +3,7 @@ import { ApifyClient } from 'apify-client';
 
 import { ACTOR_ADDITIONAL_INSTRUCTIONS, defaults, MAX_DESCRIPTION_LENGTH } from './const.js';
 import { log } from './logger.js';
-import type {
-    ActorDefinitionPruned,
-    ActorDefinitionWithDesc,
-    SchemaProperties,
-    Tool,
-} from './types.js';
+import type { ActorDefinitionPruned, ActorDefinitionWithDesc, SchemaProperties, Tool } from './types.js';
 
 export function actorNameToToolName(actorName: string): string {
     return actorName.replace('/', '--');
@@ -59,7 +54,7 @@ export async function getActorDefinition(actorFullName: string): Promise<ActorDe
         return null;
     } catch (error) {
         log.error(`Failed to fetch input schema for actor: ${actorFullName} with error ${error}.`);
-        return null;
+        throw new Error(`Failed to fetch input schema for actor: ${actorFullName} with error ${error}.`);
     }
 }
 

src/server.ts

@@ -175,12 +175,12 @@ export class ApifyMcpServer {
                     case InternalTools.ADD_ACTOR_TO_TOOLS: {
                         const parsed = AddActorToToolsArgsSchema.parse(args);
                         await this.addToolsFromActors([parsed.actorFullName]);
-                        return { content: [{ type: 'text', text: `Actor ${args.name} was added to tools` }] };
+                        return { content: [{ type: 'text', text: `Actor ${parsed.actorFullName} was added to tools` }] };
                     }
                     case InternalTools.REMOVE_ACTOR_FROM_TOOLS: {
                         const parsed = RemoveActorToolArgsSchema.parse(args);
                         this.tools.delete(parsed.toolName);
-                        return { content: [{ type: 'text', text: `Actor ${args.name} was removed from tools` }] };
+                        return { content: [{ type: 'text', text: `Tool ${parsed.toolName} was removed` }] };
                     }
                     case InternalTools.DISCOVER_ACTORS: {
                         const parsed = DiscoverActorsArgsSchema.parse(args);

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 { toolNameToActorName } from './actors.js';
 import { InternalTools } from './const.js';
 import type { ActorStorePruned, PricingInfo, Tool } from './types.js';
 
@@ -35,14 +35,14 @@ export const RemoveActorToolArgsSchema = z.object({
     toolName: z.string()
         .describe('Full name of the Actor to remove. Actor full name is always composed from `username--name`'
             + 'Never use name or username only')
-        .transform((val) => actorNameToToolName(val)),
+        .transform((val) => toolNameToActorName(val)),
 });
 
 export const AddActorToToolsArgsSchema = z.object({
     actorFullName: z.string()
         .describe('Full name of the Actor to add as tool. Tool name is always composed from `username--name`'
             + 'Never use name or username only')
-        .transform((val) => actorNameToToolName(val)),
+        .transform((val) => toolNameToActorName(val)),
 });
 
 export const GetActorDefinition = z.object({