fix: implement code review feedback

Author: MichalKalita

src/stdio.ts

@@ -83,7 +83,9 @@ Note: Tools that enable you to search Actors from the Apify Store and get their
     .option('full-actor-schema', {
         type: 'boolean',
         default: false,
-        describe: 'Enable full Actor schema for all Actors.',
+        describe: 'Default behavior keeps tools simple: advanced inputs are grouped under "advancedInput" and their schema is not expanded in the tools list. '
+            + 'This reduces context size and makes tools easier to use. Use the "get-actor-details" tool to view the complete input schema and documentation. '
+            + 'Enable this option to disable the simplification and expose the full Actor input schema with all parameters at the top level.',
     })
     .help('help')
     .alias('h', 'help')

src/tools/actor.ts

@@ -10,7 +10,6 @@ import { ApifyClient } from '../apify-client.js';
 import {
     ACTOR_ADDITIONAL_INSTRUCTIONS,
     ACTOR_MAX_MEMORY_MBYTES,
-    ACTOR_README_MAX_LENGTH,
     ADVANCED_INPUT_KEY,
     HelperTools,
 } from '../const.js';
@@ -62,6 +61,7 @@ export async function callActorGetDataset(
     progressTracker?: ProgressTracker | null,
 ): Promise<CallActorGetDatasetResult> {
     let input = simplifiedInput;
+    // Moves the advances input properties into the root object
     if (input[ADVANCED_INPUT_KEY]) {
         const { [ADVANCED_INPUT_KEY]: advancedInput, ...rest } = input;
         input = { ...advancedInput, ...rest };
@@ -220,28 +220,24 @@ export async function getActorsAsTools(
 
     const actorsInfo: (ActorInfo | null)[] = await Promise.all(
         actorIdsOrNames.map(async (actorIdOrName) => {
-            // Always cache the full schema version under a stable key
-            const cacheKey = actorIdOrName;
-            let actorDefinition = actorDefinitionPrunedCache.get(cacheKey);
-            if (!actorDefinition) {
-                actorDefinition = await getActorDefinition(
-                    actorIdOrName,
-                    apifyToken,
-                    ACTOR_README_MAX_LENGTH,
-                );
+            const actorDefinitionPrunedCached = actorDefinitionPrunedCache.get(actorIdOrName);
+            if (actorDefinitionPrunedCached) {
+                return {
+                    actorDefinitionPruned: actorDefinitionPrunedCached,
+                    webServerMcpPath: getActorMCPServerPath(actorDefinitionPrunedCached),
+
+                } as ActorInfo;
             }
 
             const actorDefinitionPruned = await getActorDefinition(actorIdOrName, apifyToken);
             if (!actorDefinitionPruned) {
                 log.error('Actor not found or definition is not available', { actorName: actorIdOrName });
                 return null;
             }
-
-            // Cache canonical full schema without mutation
-            actorDefinitionPrunedCache.set(cacheKey, actorDefinitionPruned);
-
+            // Cache the pruned Actor definition
+            actorDefinitionPrunedCache.set(actorIdOrName, actorDefinitionPruned);
             return {
-                actorDefinitionPruned: actorDefinition,
+                actorDefinitionPruned,
                 webServerMcpPath: getActorMCPServerPath(actorDefinitionPruned),
             } as ActorInfo;
         }),

src/types.ts

@@ -164,11 +164,6 @@ export type ToolEntry =
         tool: ActorMcpTool;
     };
 
-/**
- * Type discriminator for tools - indicates whether a tool is internal or Actor-based.
- */
-export type ToolType = ToolEntry['type'];
-
 /**
  * Price for a single event in a specific tier.
  */

src/utils/tools.ts

@@ -9,18 +9,19 @@ export function getToolPublicFieldOnly(tool: ToolBase) {
     return {
         name: tool.name,
         description: tool.description,
-        inputSchema: simplifiedSchema(tool.inputSchema as IActorInputSchema),
+        inputSchema: clearAdvancedInputProperties(tool.inputSchema as IActorInputSchema),
     };
 }
 
-function simplifiedSchema(schema: IActorInputSchema): IActorInputSchema {
-    if (schema.properties && 'advancedInputs' in schema.properties) {
+/** Removes properties under ADVANCED_INPUT_KEY from the schema */
+function clearAdvancedInputProperties(schema: IActorInputSchema): IActorInputSchema {
+    if (schema.properties && ADVANCED_INPUT_KEY in schema.properties) {
         return {
             ...schema,
             properties: {
                 ...schema.properties,
                 [ADVANCED_INPUT_KEY]: {
-                    ...schema.properties.advancedInputs,
+                    ...schema.properties[ADVANCED_INPUT_KEY],
                     properties: {},
                     additionalProperties: true,
                 },