Repository navigation
fix(agent-relay): our own agent guide lost its identity and setup line - #163
Conversation
agentChatAgentGuide passed the normalized display name to the snippet, so the official page read "Chat with the registered agent 'Agent Relay'" and dropped the check-setup-with-our-agent line. Pass no name for our own agent. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 37 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (2)
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. |
|
Preview deployed!
This is a Cloudflare Workers preview version of this PR's build. |
Bug (live)
curl https://arelay.to/agent-relayshows "Chat with the registered agent "Agent Relay" for me" and no longer includes the "Before you set anything up for Agent Relay … confirm each step with it" line (added in #150 after a visitor's agent created a stray workspace).Cause:
agentChatAgentGuidepassednormalizeAgentName(agentName)('Agent Relay'when none was given) toagentChatSnippet. The snippet'sisAgentRelaycheck treats a named agent without theagent-relayhandle as a registered third-party agent.Fix
For our own agent, pass no name to the snippet. Company handles are unchanged.
Test
A new test covers both the no-args and the
agent-relayhandle guides (our identity plus the setup line) and a company handle (registered label, no Agent Relay setup line). It fails on main, passes with the fix. Snippet tests pass 18/18.🤖 Generated with Claude Code
Note
Low Risk
Copy-generation fix in agent chat snippets with a focused regression test; no auth or API behavior changes.
Overview
Fixes a regression where
agentChatAgentGuidepassed a normalized default name ("Agent Relay") intoagentChatSnippet, so the snippet treated our own agent like a third-party registered agent. Visitors saw "Chat with the registered agent …" and lost the "Before you set anything up for Agent Relay …" confirmation block oncurl/ agent-fetch of the guide.For Agent Relay (
isAgentRelay), the guide now passesagentName: undefinedinto the snippet so identity and setup wording stay correct; company handles are unchanged.Adds a test that locks in our-agent guides (no-args and
agent-relayhandle) vs a company handle.Reviewed by Cursor Bugbot for commit 76be395. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Fixes the agent guide at
arelay.to/agent-relayso our own agent keeps its identity and the "Before you set anything up" line instead of appearing as a registered third-party agent.agent-relayhandle guides plus a company handle case, and fails without the fix.Written for commit 76be395. Summary will update on new commits.
Agent Relay sessions
claudesession1e2adfbe· opened viagh pr create· last active 2026-10-08