fix: handle review comments

Author: jirispilka

src/tools/actor.ts

@@ -405,7 +405,7 @@ EXAMPLES:
                             }
                             const toolsResponse = await client.listTools();
 
-                            const toolsInfo = toolsResponse.tools.map((tool) => `**${tool.name}**\n${tool.description || 'No description'}\nInput Schema: ${JSON.stringify(tool.inputSchema, null, 2)}`,
+                            const toolsInfo = toolsResponse.tools.map((tool) => `**${tool.name}**\n${tool.description || 'No description'}\nInput schema: ${JSON.stringify(tool.inputSchema, null, 2)}`,
                             ).join('\n\n');
 
                             return buildMCPResponse([`This is an MCP Server Actor with the following tools:\n\n${toolsInfo}\n\nTo call a tool, use step="call" with actor name format: "${baseActorName}:{toolName}"`]);
@@ -420,7 +420,7 @@ EXAMPLES:
                         }
                         const content = [
                             // TODO: update result to say: this is result of info step, you must now call again with step=call and proper input
-                            { type: 'text', text: `# Input Schema: \n${JSON.stringify(details.inputSchema, null, 0)}` },
+                            { type: 'text', text: `Input schema: \n${JSON.stringify(details.inputSchema, null, 0)}` },
                         ];
                         /**
                          * Add Skyfire instructions also in the info step since clients are most likely truncating the long tool description of the call-actor.
@@ -499,7 +499,7 @@ EXAMPLES:
                     if (errors && errors.length > 0) {
                         return buildMCPResponse([
                             `Input validation failed for Actor '${actorName}': ${errors.map((e) => e.message).join(', ')}`,
-                            `Input Schema:\n${JSON.stringify(actor.tool.inputSchema)}`,
+                            `Input schema:\n${JSON.stringify(actor.tool.inputSchema)}`,
                         ]);
                     }
                 }

src/tools/store_collection.ts

@@ -5,7 +5,7 @@ import zodToJsonSchema from 'zod-to-json-schema';
 import { ApifyClient } from '../apify-client.js';
 import { ACTOR_SEARCH_ABOVE_LIMIT, HelperTools } from '../const.js';
 import type { ActorPricingModel, ExtendedActorStoreList, HelperTool, ToolEntry } from '../types.js';
-import { formatActorsListToActorCard } from '../utils/actor-card.js';
+import { formatActorToActorCard } from '../utils/actor-card.js';
 import { ajv } from '../utils/ajv.js';
 
 export async function searchActorsByKeywords(
@@ -99,7 +99,12 @@ USAGE EXAMPLES:
                 parsed.offset,
             );
             actors = filterRentalActors(actors || [], userRentedActorIds || []).slice(0, parsed.limit);
-            const actorCards = formatActorsListToActorCard(actors);
+            const actorCards = actors.length === 0 ? [] : actors.map(formatActorToActorCard);
+
+            const actorsText = actorCards.length
+                ? actorCards.join('\n\n')
+                : 'No Actors were found for the given search query. Please try different keywords or simplify your query.';
+
             return {
                 content: [
                     {
@@ -111,7 +116,7 @@ USAGE EXAMPLES:
 
 # Actors:
 
-${actorCards.join('\n\n')}`,
+${actorsText}`,
                     },
                 ],
             };

src/utils/actor-card.ts

@@ -102,17 +102,3 @@ export function formatActorToActorCard(
     }
     return markdownLines.join('\n');
 }
-
-/**
- * Formats a list of Actors into Actor cards
- * @param actors - Array of Actor information
- * @returns Formatted markdown string
- */
-export function formatActorsListToActorCard(actors: (Actor | ExtendedActorStoreList)[]): string[] {
-    if (actors.length === 0) {
-        return [];
-    }
-    return actors.map((actor) => {
-        return formatActorToActorCard(actor);
-    });
-}

src/utils/pricing-info.ts

@@ -55,7 +55,7 @@ function convertMinutesToGreatestUnit(minutes: number): { value: number; unit: s
  */
 
 function payPerEventPricingToString(pricingPerEvent: ExtendedPricingInfo['pricingPerEvent']): string {
-    if (!pricingPerEvent || !pricingPerEvent.actorChargeEvents) return 'No event pricing information available.';
+    if (!pricingPerEvent || !pricingPerEvent.actorChargeEvents) return 'Pricing information for events is not available.';
     const eventStrings: string[] = [];
     for (const event of Object.values(pricingPerEvent.actorChargeEvents)) {
         let eventStr = `\t- **${event.eventTitle}**: ${event.eventDescription} `;
@@ -78,7 +78,7 @@ export function pricingInfoToString(pricingInfo: ExtendedPricingInfo | null): st
     // If there is no pricing infos entries the Actor is free to use
     // based on https://github.com/apify/apify-core/blob/058044945f242387dde2422b8f1bef395110a1bf/src/packages/actor/src/paid_actors/paid_actors_common.ts#L691
     if (pricingInfo === null || pricingInfo.pricingModel === ACTOR_PRICING_MODEL.FREE) {
-        return 'User pays for the computing resources consumed by the Actor';
+        return 'This Actor is free to use. You are only charged for Apify platform usage.';
     }
     if (pricingInfo.pricingModel === ACTOR_PRICING_MODEL.PRICE_PER_DATASET_ITEM) {
         const customUnitName = pricingInfo.unitName !== 'result' ? pricingInfo.unitName : '';
@@ -98,12 +98,12 @@ export function pricingInfoToString(pricingInfo: ExtendedPricingInfo | null): st
             const tiers = Object.entries(pricingInfo.tieredPricing)
                 .map(([tier, obj]) => `${tier}: $${obj.tieredPricePerUnitUsd} per month`)
                 .join(', ');
-            return `This Actor is rental and thus has tiered pricing per month: ${tiers}, with a trial period of ${value} ${unit}.`;
+            return `This Actor is rental and has tiered pricing per month: ${tiers}, with a trial period of ${value} ${unit}.`;
         }
-        return `This Actor is rental and thus has a flat price of ${pricingInfo.pricePerUnitUsd} USD per month, with a trial period of ${value} ${unit}.`;
+        return `This Actor is rental and has a flat price of ${pricingInfo.pricePerUnitUsd} USD per month, with a trial period of ${value} ${unit}.`;
     }
     if (pricingInfo.pricingModel === ACTOR_PRICING_MODEL.PAY_PER_EVENT) {
         return payPerEventPricingToString(pricingInfo.pricingPerEvent);
     }
-    return 'Not available';
+    return 'Pricing information is not available.';
 }