fix: improve Apify tools docstrings with Apify context#4
fix: improve Apify tools docstrings with Apify context#4jirispilka wants to merge 3 commits intoapify:feat/strands-core-apify-toolsfrom
Conversation
…ed details on input, usage, and examples.
jirispilka
force-pushed
the
feat/strands-core-apify-tools
branch
from
365fb16 to
1b9675f
Compare
daveomri
left a comment
daveomri
left a comment
There was a problem hiding this comment.
Looks much better now, just a few notes.
daveomri
left a comment
daveomri
left a comment
There was a problem hiding this comment.
Looks much better, pre-approving, but just a few notes:
| - 📁 **File Operations** - Read, write, and edit files with syntax highlighting and intelligent modifications | ||
| - 🖥️ **Shell Integration** - Execute and interact with shell commands securely | ||
| - 🧠 **Memory** - Store user and agent memories across agent runs to provide personalized experiences with both Mem0, Amazon Bedrock Knowledge Bases, Elasticsearch, and MongoDB Atlas | ||
| - 🧠 **Memory** - Store user and agent memories across agent runs to provide personalized experiences with Mem0, Amazon Bedrock Knowledge Bases, Elasticsearch, and MongoDB Atlas |
There was a problem hiding this comment.
Not sure if we should change other parts of the README that don't involve the Apify tools, so we don't run into issues with the Strands review process.
| ) | ||
|
|
||
| # Connect to a SSE-based server | ||
| # Connect to an SSE-based server |
There was a problem hiding this comment.
Also, I wouldn't change anything that's not necessary for the Apify tools.
| ) | ||
|
|
||
| # Convert HTML webpages to markdown for better readability | ||
| # Convert HTML webpages to Markdown for better readability |
There was a problem hiding this comment.
Also, I wouldn't change anything that's not necessary for the Apify tool.
| input schema - check the Actor README on Apify Store for required fields. | ||
| timeout_secs: Maximum time in seconds to wait for the Actor run to finish. Defaults to 300. | ||
| memory_mbytes: Memory allocation in MB for the Actor run. | ||
| memory_mbytes: Memory allocation in MB for the Actor run. Uses Actor default if not set. |
There was a problem hiding this comment.
just a nit: maybe adding "uses actor default memory if not set"
There was a problem hiding this comment.
maybe default value.. not sure what sounds better here
3 participants
Summary
actor_idandrun_inputparameter descriptions to mention Apify Storecrawler_typefrom a one-liner into three bullet points so the LLM can pick the right engineapify_run_actor_and_get_datasetvsapify_run_actor)🤖 Generated with Claude Code