Skip to content

fix: improve Apify tools docstrings with Apify context#4

Merged
daveomri merged 5 commits intoapify:feat/strands-core-apify-toolsfrom
jirispilka:feat/strands-core-apify-tools
Apr 15, 2026
Merged

fix: improve Apify tools docstrings with Apify context#4
daveomri merged 5 commits intoapify:feat/strands-core-apify-toolsfrom
jirispilka:feat/strands-core-apify-tools

Conversation

@jirispilka
Copy link
Copy Markdown

Summary

  • Inject Apify platform context into module and tool docstrings (what an Actor is, what a dataset is, storage types)
  • Improve actor_id and run_input parameter descriptions to mention Apify Store
  • Expand crawler_type from a one-liner into three bullet points so the LLM can pick the right engine
  • Add cross-tool pointers (e.g. when to prefer apify_run_actor_and_get_dataset vs apify_run_actor)

🤖 Generated with Claude Code

@jirispilka jirispilka force-pushed the feat/strands-core-apify-tools branch from 365fb16 to 1b9675f Compare April 2, 2026 09:56
@jirispilka jirispilka mentioned this pull request Apr 2, 2026
7 tasks
Copy link
Copy Markdown
Collaborator

@daveomri daveomri left a comment

Choose a reason for hiding this comment

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

Looks much better now, just a few notes.

Comment thread src/strands_tools/apify.py Outdated
Comment thread src/strands_tools/apify.py Outdated
Comment thread src/strands_tools/apify.py Outdated
@jirispilka jirispilka requested a review from daveomri April 8, 2026 07:05
Copy link
Copy Markdown
Collaborator

@daveomri daveomri left a comment

Choose a reason for hiding this comment

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

Looks much better, pre-approving, but just a few notes:

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread src/strands_tools/apify.py Outdated
@jirispilka jirispilka requested a review from daveomri April 14, 2026 12:10
Copy link
Copy Markdown
Collaborator

@daveomri daveomri left a comment

Choose a reason for hiding this comment

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

LGTM!

@jirispilka
Copy link
Copy Markdown
Author

@daveomri I can't merge, please merge it

@daveomri daveomri merged commit 38ccda7 into apify:feat/strands-core-apify-tools Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants