Skip to content

Fix incorrect command line example in Quick Start section #43

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

saiqulhaq
Copy link

@saiqulhaq saiqulhaq commented Jul 19, 2025

Problem:

The README documentation was incomplete regarding usage patterns for the strands command, particularly when combining piped input with additional command arguments.

Solution:

  • Updated Quick Start section to show the working combined usage pattern
  • Enhanced Usage Patterns section to document all three supported input methods:
    1. Direct command arguments only
    2. Piped input only
    3. Combined piped input with additional instructions (requires BYPASS_TOOL_CONSENT=true)
  • Clarified that combining piped input with command arguments requires setting BYPASS_TOOL_CONSENT=true to bypass tool confirmation prompts

Testing:

  • ✅ Verified strands "instruction" works (direct command only)
  • ✅ Verified cat agent-spec.txt | strands "Build a specialized agent based on these specifications" works when BYPASS_TOOL_CONSENT=true

Changes:

  • Added working example of combined usage in Quick Start section
  • Expanded Usage Patterns section with comprehensive documentation
  • Documented the BYPASS_TOOL_CONSENT requirement for combined usage

Type of Change

  • ✅ Documentation improvement
  • ✅ Clarification of existing functionality

@saiqulhaq saiqulhaq requested a review from a team as a code owner July 19, 2025 15:50
@saiqulhaq
Copy link
Author

I will update the README again
because using pipe | causes an error
This agent can not write any file because it can not accept confirmation (y/n)

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.

1 participant