Repository navigation
desktop 1.1.10: the socket guard names an unset or missing relay_socket - #151
Conversation
The fail-fast check fires both when relay_socket is unset and when it names a missing or stale socket; the message now says which path it saw (agentrelay.com#172 review). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Session-Id: 26d00c25-77c9-401f-b9f0-51ce6c9deda6
|
@codex review |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 3 files
Reply with feedback, questions, or to request a fix.
View guided diff | Re-trigger cubic
…ays so test -S passes for a stale socket file a crashed app leaves behind, so the guard no longer claims the socket is live; the curl after it is what proves the app answers. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Session-Id: 26d00c25-77c9-401f-b9f0-51ce6c9deda6
khaliqgant
changed the title
|
@codex review |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
From a cubic P3 on AgentWorkforce/agentrelay.com#172. The fail-fast guard
test -S "${relay_socket:-/nonexistent}"fires both whenrelay_socketis unset and when it names a missing or stale socket (for example, a pointer to a previous app run). Its message said only "relay_socket is not set". It now reads "relay_socket is unset or not a live socket ()…", and notes that a stale path means the app isn't running.Desktop 1.1.9 → 1.1.10. Text only; no behaviour change.
🤖 Generated with Claude Code
Note
Low Risk
Text-only skill and package metadata updates with no runtime or setup behavior changes.
Overview
Bumps
setting-up-agent-relay-desktopto 1.1.10 inREADME.mdandprpm.json.The pre-install socket guard in the skill now uses a clearer fail-fast message when
relay_socketis unset or when it points at a path that is not a socket file (including stale paths), and it prints the path orunset. Short comments were added to state thattest -Sonly checks for a socket file on disk, while the followingcurlto/setup/statusis what confirms the Desktop app is actually responding.Documentation and messaging only; the guard logic is unchanged.
Reviewed by Cursor Bugbot for commit fd7d2b1. Bugbot is set up for automated code reviews on this repo. Configure here.
Agent Relay sessions
claudesession26d00c25· opened viagh pr create· last active 2026-10-08