Skip to content

fix: install broker binary to BIN_DIR so it's on PATH#689

Merged
khaliqgant merged 2 commits into
mainfrom
fix/install-broker-binary-path
Apr 7, 2026
Merged

fix: install broker binary to BIN_DIR so it's on PATH#689
khaliqgant merged 2 commits into
mainfrom
fix/install-broker-binary-path

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Apr 7, 2026

Summary

  • The broker binary was only installed to ~/.agent-relay/bin/ which isn't on PATH
  • getBrokerBinaryPath() falls back to which agent-relay-broker which fails
  • Now also copies the broker binary to ~/.local/bin/ alongside the CLI binary

Test plan

  • Run curl -fsSL ... | bash and verify which agent-relay-broker succeeds
  • Run agent-relay up from a project that doesn't have the SDK in node_modules

🤖 Generated with Claude Code


Open with Devin

khaliqgant and others added 2 commits April 7, 2026 10:04
The broker binary was only installed to ~/.agent-relay/bin/ which isn't
on PATH. When the standalone CLI runs getBrokerBinaryPath(), the which
lookup fails. Copy the binary to ~/.local/bin/ alongside the CLI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@khaliqgant khaliqgant requested a review from willwashburn as a code owner April 7, 2026 09:13
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@khaliqgant khaliqgant merged commit afec561 into main Apr 7, 2026
54 of 56 checks passed
@khaliqgant khaliqgant deleted the fix/install-broker-binary-path branch April 7, 2026 09:29
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