Skip to content

PAB-66: fix piped interactive installs - #36

Merged
pablopunk merged 1 commit into
mainfrom
agent/coder/1eb9c2b4
Jul 18, 2026
Merged

pablopunk merged 1 commit into
mainfrom
agent/coder/1eb9c2b4

Conversation

@pablopunk

@pablopunk pablopunk commented Jul 18, 2026 •

Copy link
Copy Markdown
Owner

Title: Fix piped interactive installs

This pull request fixes a bug where running dot in interactive mode with piped input (e.g., echo "..." | dot) would fail because the standard input was not a terminal. The changes ensure interactive prompts are still displayed and read from the real terminal even when stdin is redirected or piped. Additionally, non-interactive install commands are now executed with their standard input explicitly ignored, preventing them from accidentally consuming the parent’s piped data.

Major changes include:

  • Added a openTerminalInput utility that opens /dev/tty to provide a terminal-backed input stream when process.stdin is not a TTY.
  • Updated the interactive mode to fall back to the terminal stream if stdin is not a TTY, and to use that stream for the prompts checklist.
  • Refactored command execution in installComponent and uninstallComponent to use Bun.spawn with stdin: "ignore" instead of appending < /dev/null to shell commands, avoiding unintended side effects with piped input.
  • Added new tests covering both interactive and non-interactive scenarios to validate the fix.

This ensures dot behaves correctly in complex shell pipelines, whether running an interactive setup or executing automated installs.

Co-authored-by: multica-agent <github@multica.ai>
@kody-ai

kody-ai Bot commented Jul 18, 2026 •

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug ✅
Performance ❌
Security ✅
Business Logic ❌

Access your configuration settings here.

​

@pablopunk
pablopunk merged commit 06e70e8 into main Jul 18, 2026
9 checks passed
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