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

Skip to content

fix: support for STDIN via ZSH#1606

Merged
tusharmath merged 11 commits into
mainfrom
stdin-support-zsh
Sep 29, 2025
Merged

fix: support for STDIN via ZSH#1606
tusharmath merged 11 commits into
mainfrom
stdin-support-zsh

Conversation

@tusharmath

@tusharmath tusharmath commented Sep 28, 2025

Copy link
Copy Markdown
Collaborator
  • Pass conversation id via env variable
  • Pass agent id via env variable
  • Make transformation look better
  • Add support for stdin
  • Delete unused transformation function
  • forge info initialized conversation ALWAYS
  • Remove :reset usage

@github-actions github-actions Bot added the type: chore Routine tasks like conversions, reorganization, and maintenance work. label Sep 28, 2025
@tusharmath tusharmath changed the title chore: remove unused function fix: support for STDIN via ZSH Sep 28, 2025
@github-actions github-actions Bot added the type: fix Iterations on existing features or infrastructure. label Sep 29, 2025
user_action="${match[1]}"
input_text="${match[3]:-}" # Use empty string if no parameters
elif [[ "$BUFFER" =~ "^: (.*)$" ]]; then
elif [[ "$BUFFER" =~ "^: (.*)$" ]]; then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The indentation in this line is inconsistent with the previous if statement. The elif should be aligned at the same level:

- user_action="${match[1]}"
- input_text="${match[3]:-}"  # Use empty string if no parameters
-        elif [[ "$BUFFER" =~ "^: (.*)$" ]]; then
+ user_action="${match[1]}"
+ input_text="${match[3]:-}"  # Use empty string if no parameters
+ elif [[ "$BUFFER" =~ "^: (.*)$" ]]; then

This will maintain consistent indentation throughout the conditional block and improve code readability.

Spotted by Diamond

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@openhands-ai

openhands-ai Bot commented Sep 29, 2025

Copy link
Copy Markdown

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • ci

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #1606 at branch `stdin-support-zsh`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@tusharmath tusharmath merged commit 1da3b54 into main Sep 29, 2025
9 checks passed
@tusharmath tusharmath deleted the stdin-support-zsh branch September 29, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: chore Routine tasks like conversions, reorganization, and maintenance work. type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant