Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jirispilka
Copy link
Collaborator

@jirispilka jirispilka commented Sep 17, 2025

Update tool descriptions based on evaluations.

Also, keep only a few attributes for the RAG Web Browser. Having all attributes was confusing and just wasted LLM tokens.

So, the RAG Web Browser is a special case: it’s used as the default tool with limited attributes.

The obvious drawback is that it can’t be used with all attributes—but I think that’s fine.

https://app.phoenix.arize.com/s/michal-kalita/datasets/RGF0YXNldDo1NQ==/experiments

image

related to https://github.com/apify/ai-team/issues/67

@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Sep 17, 2025
if (!actorDefinitionPruned) continue;

const isRag = actorDefinitionPruned.actorFullName === RAG_WEB_BROWSER;
const { inputSchema } = buildActorInputSchema(actorDefinitionPruned.actorFullName, actorDefinitionPruned.input, isRag);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: why additional isRag param, we can you just check the actorFullName to match the RAG browser.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm also using it below .... but either fine is fine with me

@jirispilka jirispilka marked this pull request as ready for review September 19, 2025 12:26
@jirispilka
Copy link
Collaborator Author

I'll fix the conflict later

Copy link
Contributor

@MQ37 MQ37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice rework 👍 The structure makes more sense and hopefully it will work better for the LLMs. Just a few nits and one question otherwise LGTM (did not test though).


MANDATORY TWO-STEP WORKFLOW:
A successful run returns a \`datasetId\` (the Actor's output stored as an Apify dataset) and a short preview of items.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: the short preview of the items may or may not be included

@@ -344,7 +352,8 @@ Step 2: Call Actor (step="call")
- This calls and runs the Actor. It will create an output as an Apify dataset (with datasetId).
- This step returns a dataset preview, typically JSON-formatted tabular data.

The step parameter enforces this workflow - you cannot call an Actor without first getting its info.`,
EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: examples of what? I would include something like "EXAMPLE WHEN TO USE THIS TOOL"

USAGE:
- Use when you need dataset metadata to understand its structure before fetching items.

EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to read data from a dataset (all items or only selected fields).

EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to infer the structure of dataset items for downstream processing or validation.

EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to inspect run status or retrieve associated resource IDs (e.g., datasetId for output).
EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to inspect recent logs to debug or monitor a run.
EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to stop a run that is taking too long or misconfigured.
EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

USAGE:
- Use when you need to browse or filter recent Actor runs.

EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

- Use when a user asks about specific data and available Actors in the Apify Store
- Use when a user needs to discover MCP servers or automation tools
- Use when you need to discover Actors for a specific task or find MCP servers.
- Use to explore available tools in the Apify ecosystem based on keywords.

EXAMPLES:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-ai Issues owned by the AI team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants