Fix Qoder install/upgrade bash syntax causing exit 127

[Copilot]

Fix Qoder install/upgrade bash syntax causing exit 127

Root Cause

Commands used bash -- --force which causes bash to interpret --force as a filename instead of a script argument:

# Broken - bash looks for file named "--force"
curl -fsSL https://qoder.com/install | bash -- --force

# Fixed - bash reads from stdin and passes --force to script
curl -fsSL https://qoder.com/install | bash -s -- --force

Changes

• internal/registry/agents.json: Updated Qoder install/upgrade commands for darwin/linux to use bash -s --
• internal/registry/registry_test.go: Added TestQoderBashCommandFormat to validate bash pipe command format (improved to only check when arguments are passed)
• CHANGELOG.md: Documented fix with proper issue reference

The -s flag tells bash to read from stdin; -- separates bash options from script arguments.

Updates from Review Feedback

• Fixed issue number placeholder in CHANGELOG.md (❌ Failed to upgrade Qoder: exit status 127 #26)

• Improved test logic to only require -s when arguments are passed to bash

• Test now allows commands like curl | bash without false positives

• Test still catches the original bug pattern bash -- --force

• Fixes ❌ Failed to upgrade Qoder: exit status 127 #26

Original prompt

---

This section details on the original issue you should resolve

<issue_title>❌ Failed to upgrade Qoder: exit status 127</issue_title>
<issue_description>❌ Failed to upgrade Qoder: exit status 127 while executing agentpipe agents upgrade --all</issue_description>

<agent_instructions>Investigate why Qoder install/upgrade might be failing during these commands:

• agentpipe agents install qoder
• agentpipe agents upgrade qoder
• agentpipe agents upgrade --all</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@kevinelliott @afahmygit Thanks for reporting this. I'll see what is going on.

• Fixes ❌ Failed to upgrade Qoder: exit status 127 #26

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

---

Note

Fix Qoder install/upgrade commands to use bash -s -- (resolving exit 127), add tests to enforce format, and update changelog.

• Registry:
  • Update Qoder install/upgrade (darwin, linux) to curl -fsSL https://qoder.com/install | bash -s -- --force in internal/registry/agents.json.
• Tests:
  • Add TestQoderBashCommandFormat with verifyBashCommandFormat() in internal/registry/registry_test.go to ensure piped bash uses bash -s --.
• Docs/Changelog:
  • Document fix in CHANGELOG.md under Unreleased → Fixed (references ❌ Failed to upgrade Qoder: exit status 127 #26).

^{Written by Cursor Bugbot for commit 50c5664. This will update automatically on new commits. Configure here.}

[github-actions]

🔨 Build Artifacts Ready

Test binaries for PR #27 (8fbfffe) have been built successfully!

Available platforms:

• 🍎 macOS (Apple Silicon)
• 🐧 Linux (x64)
• 🪟 Windows (x64)

Download artifacts from the workflow run

Testing instructions:

# Download and extract the appropriate archive
tar -xzf agentpipe_<platform>.tar.gz
# or for Windows: unzip agentpipe_windows_amd64.zip

# Test the binary
./agentpipe doctor
./agentpipe --help

[Copilot]

Pull Request Overview

This PR fixes a critical bug in the Qoder agent installation and upgrade commands that was causing exit status 127 errors. The root cause was incorrect bash pipe syntax that caused bash to interpret the --force flag as a filename instead of a script argument.

Key Changes

• Fixed bash pipe syntax from bash -- to bash -s -- in Qoder install/upgrade commands
• Added test coverage to validate bash command format for piped scripts
• Documented the fix in CHANGELOG.md

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
internal/registry/agents.json	Updated Qoder install/upgrade commands for darwin/linux to use correct bash -s -- syntax for passing arguments to piped scripts
internal/registry/registry_test.go	Added TestQoderBashCommandFormat test to validate bash pipe command format and prevent future regressions
CHANGELOG.md	Documented the fix with details about the change and root cause

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The test logic is too strict. It requires ALL piped bash commands to use bash -s --, but -s is only necessary when passing arguments to the script. Commands like curl ... | bash (without arguments) work correctly and don't need -s.

The current test would incorrectly flag commands like Cursor's curl https://cursor.com/install -fsS | bash as errors, even though they work fine.

Consider updating the logic to check if arguments are being passed after the pipe (e.g., looking for patterns like bash -- or checking if there's content after bash that looks like arguments), and only require -s -- in those cases.

[Copilot]

Fixed in commit 8fbfffe - updated test logic to only require bash -s when arguments are being passed after bash. Commands like curl | bash (without arguments) are now correctly allowed, while bash -- --force (without -s) is still caught as an error.

[kevinelliott]

@copilot This is still failing.

⬆️ Upgrading Qoder...
Running: curl -fsSL https://qoder.com/install | bash -- --force
bash: --force: No such file or directory
curl: (23) Failure writing output to destination, passed 1004 returned 0
❌ Failed to upgrade Qoder: exit status 127

[Copilot]

@copilot This is still failing.

⬆️ Upgrading Qoder...
...

The error message shows the old command format (bash -- --force), which indicates you're running an older version of the binary. The fix changes this to bash -s -- --force in the agents.json file.

To use the fix:

1. Rebuild from the latest branch: go build -o agentpipe .
2. Or install the latest version once it's released

The fix is in the code and all tests pass. You can verify the corrected command by checking the built binary with: ./agentpipe agents list | grep -A 5 Qoder