Migrate Relay-broker into relay main by willwashburn · Pull Request #412 · AgentWorkforce/relay

willwashburn · 2026-02-13T06:32:11Z

Major repository restructuring: migrate core to Rust and introduce a new TypeScript SDK package. Added Cargo.toml and Cargo.lock plus several Rust sources (src/auth.rs, src/config.rs, src/control.rs, src/conversation_log.rs) and a .github CI workflow. Removed many legacy JS/TS packages, tooling, examples, and workflow files; cleaned up docs and references. Updated top-level README, .gitignore, package.json, package-lock.json, install.sh, and refreshed relay-snippets. This change consolidates the project around a Rust core while providing an SDK for TypeScript consumers.

Major repository restructuring: migrate core to Rust and introduce a new TypeScript SDK package. Added Cargo.toml and Cargo.lock plus several Rust sources (src/auth.rs, src/config.rs, src/control.rs, src/conversation_log.rs) and a .github CI workflow. Removed many legacy JS/TS packages, tooling, examples, and workflow files; cleaned up docs and references. Updated top-level README, .gitignore, package.json, package-lock.json, install.sh, and refreshed relay-snippets. This change consolidates the project around a Rust core while providing an SDK for TypeScript consumers.

Copilot

Pull request overview

Migrates the project to a Rust-based broker with a new TypeScript SDK, along with a large cleanup/restructure of legacy Node tooling and documentation to match the new architecture.

Changes:

Replaces legacy daemon/dashboard-centric docs with SDK-first + Relaycast transport documentation.
Introduces a new Rust crate (Cargo.toml) and a simplified GitHub Actions CI focused on Rust + SDK typechecking.
Removes large amounts of legacy docs, workflows, and auxiliary tooling that no longer apply.

Reviewed changes

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

Show a summary per file

File	Description
docs/reference/api.mdx	Removes legacy JS/TS API reference page.
docs/quickstart.mdx	Rewrites quickstart around `@agent-relay/sdk` orchestration flow.
docs/mint.json	Updates docs navigation to remove deprecated pages and rename groups.
docs/introduction.mdx	Updates product positioning and architecture overview to SDK + broker + Relaycast.
docs/guides/session-continuity.mdx	Removes continuity guide (legacy feature set).
docs/guides/multi-project.mdx	Removes multi-project coordination guide (bridge-era docs).
docs/guides/editor-integration.mdx	Removes ACP editor integration guide (legacy integration).
docs/guide/agent-setup.md	Removes legacy setup guide focused on daemon/dashboard/MCP.
docs/features/spawning.mdx	Reworks spawning docs for SDK and broker-managed PTY/headless agents.
docs/features/messaging.mdx	Reworks messaging docs around Relaycast channels/DMs/threads and SDK APIs.
docs/features/hooks.mdx	Removes hooks documentation (legacy package/feature).
docs/features/dashboard.mdx	Removes dashboard documentation (legacy UI).
docs/features/shadows.mdx	Removes shadow agents documentation (legacy mechanism).
docs/features/cloud.mdx	Replaces “Cloud Features” with Relaycast transport documentation.
docs/features/bridging.mdx	Removes multi-project bridging docs (legacy topology).
docs/architecture/storage.md	Removes storage architecture documentation (legacy adapters).
docker-compose.test.yml	Removes full cloud QA docker-compose stack.
docker-compose.browser.yml	Removes browser testing docker-compose setup.
TRAIL_GIT_AUTH_FIX.md	Removes trail doc artifact from repo root.
TESTING.md	Removes manual testing guide for legacy daemon/bridge/dashboard flows.
README.md	Updates README to Rust broker + TS SDK (new install, usage, CLI modes).
Cargo.toml	Adds Rust crate definition and dependencies for the broker.
CLAUDE.md	Replaces placeholder with full project + workflow guidance for agents.
AGENTS.md	Removes legacy agent instruction/manifest content.
.openskills/frontend-design/SKILL.md	Removes openskills skill doc (repo cleanup).
.npmignore	Removes legacy npmignore (old packaging model).
.mcp.json	Removes MCP server config (legacy integration).
.github/workflows/test.yml	Removes old Node-centric CI workflow.
.github/workflows/test-install.yml	Removes install-script CI workflow.
.github/workflows/test-build.yml	Removes build-script CI workflow.
.github/workflows/stress-tests.yml	Removes legacy stress-test workflow (Node-based).
.github/workflows/storage-testing.yml	Removes storage-focused workflow (Node adapters).
.github/workflows/security.yml	Removes Node security scanning workflow (npm/CodeQL JS).
.github/workflows/package-validation.yml	Removes Node package validation workflow.
.github/workflows/node-compat.yml	Removes Node compatibility workflow.
.github/workflows/e2e-tests.yml	Removes Node E2E workflow.
.github/workflows/cancel-on-merge.yml	Removes post-merge workflow cancellation job.
.github/workflows/ci.yml	Adds new consolidated CI for Rust tests/clippy/fmt + SDK typecheck.
.eslintrc.cjs	Removes legacy ESLint config.
.env.example	Removes legacy env example (telemetry/storage/cloud stack).
.dockerignore	Removes legacy dockerignore (old dockerized stack).
.claude/rules/wrapper.md	Removes wrapper conventions (legacy tmux wrapper).
.claude/rules/wrapper-inheritance.md	Removes wrapper inheritance guidance (legacy TS wrappers).
.claude/rules/testing.md	Removes Vitest testing conventions (legacy TS tests).
.claude/rules/sdk-daemon-parity.md	Removes SDK/daemon parity guidelines (legacy architecture).
.claude/rules/protocol.md	Removes old protocol conventions doc (legacy TS protocol package).
.claude/rules/protocol-schema-sync.md	Removes protocol schema sync guidance (legacy relay-pty + TS protocol).
.claude/rules/hooks.md	Removes hooks conventions (legacy TS hooks).
.claude/rules/daemon.md	Removes daemon conventions (legacy TS daemon).
.claude/rules/bridge.md	Removes bridge conventions (legacy TS bridge).
.claude/rules/rust.md	Adds Rust conventions for `src/` and `tests/`.

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

Copilot · 2026-02-13T06:33:29Z

+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo test
+      - run: cargo test --test 'stress_*' -- --ignored


cargo test --test does not support glob patterns (it expects an exact integration test target name), so --test 'stress_*' will fail the workflow. If the goal is to run all ignored “stress” tests, use the standard name filter instead (e.g., run ignored tests with a substring filter) or explicitly enumerate each stress test target.

Suggested change

- run: cargo test --test 'stress_*' -- --ignored

- run: cargo test stress_ -- --ignored

Copilot · 2026-02-13T06:33:29Z

+relay.onMessageReceived = (msg) => {
+  console.log(`${msg.from}: ${msg.body}`);
+};

-# for Claude
-npx prpm install @agent-relay/agent-relay-snippet --location CLAUDE.md
-```
+// Send messages
+await codex.sendMessage({ body: "Ready for review" });


The README example is internally inconsistent with the updated docs elsewhere in this PR (quickstart.mdx, messaging.mdx) where messages use text (not body). This will confuse SDK users and may be incorrect if the SDK’s public types are text-based. Align the README snippet to the SDK’s actual message shape (field names and required routing fields like to, if applicable) and keep it consistent across docs.

devin-ai-integration

✅ Devin Review: No Issues Found

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

Pull relay-broker directory from relay-broker-integration branch as the foundation for the hybrid approach. Will apply bug fixes from PR #412 on top. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Prevent agents from spoofing human identity by naming themselves "human:..." to bypass release ACL checks. When sender_kind is explicitly Agent, immediately return false regardless of name string. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

When base URL already ends with /stream (e.g. wss://rt.relaycast.dev/stream), avoid appending /v1/stream which produced invalid double-path URLs like wss://host/stream/v1/stream. Now correctly preserves the existing /stream path. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Rename ensure_reaycast_mcp_config to ensure_relaycast_mcp_config (missing 'l' in relaycast). Updated all call sites in main.rs. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Without a body-size bound, a burst of large messages from the same sender within the coalesce window would concatenate without limit, potentially causing large memory allocations. Add MAX_COALESCED_BODY_SIZE (32 KiB) check that flushes the current group when exceeded. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Make terminate_child public so it can be reused from main.rs instead of being duplicated inline, reducing code duplication. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* feat: add relay-broker Rust binary from PR #404 Pull relay-broker directory from relay-broker-integration branch as the foundation for the hybrid approach. Will apply bug fixes from PR #412 on top. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(security): add sender_kind guard in control.rs Prevent agents from spoofing human identity by naming themselves "human:..." to bypass release ACL checks. When sender_kind is explicitly Agent, immediately return false regardless of name string. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(relaycast): prevent URL double-path construction When base URL already ends with /stream (e.g. wss://rt.relaycast.dev/stream), avoid appending /v1/stream which produced invalid double-path URLs like wss://host/stream/v1/stream. Now correctly preserves the existing /stream path. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(snippets): correct relaycast typo in function name Rename ensure_reaycast_mcp_config to ensure_relaycast_mcp_config (missing 'l' in relaycast). Updated all call sites in main.rs. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(scheduler): add 32KiB coalescing body size limit Without a body-size bound, a burst of large messages from the same sender within the coalesce window would concatenate without limit, potentially causing large memory allocations. Add MAX_COALESCED_BODY_SIZE (32 KiB) check that flushes the current group when exceeded. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor(spawner): export terminate_child as pub Make terminate_child public so it can be reused from main.rs instead of being duplicated inline, reducing code duplication. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(security): remove leaked live API key from .mcp.json Replace committed rk_live_... key with environment variable placeholder. The exposed key (rk_live_b00fd0ddeca96468d72140c9c4b3a910) must be rotated immediately. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(compat): add agent-relay-path aliases and env fallback in utils * fix(compat): export RelayBrokerOrchestrator alias from wrapper * fix(utf8): use char-boundary-safe string truncation Fix potential panics when truncating multi-byte UTF-8 strings (emoji, CJK, accented characters) in conversation_log.rs and main.rs listen mode. Adds floor_char_boundary helper to conversation_log.rs and applies it to truncate(), pad_or_truncate(), and short_id(). Also fixes the same pattern in main.rs listen-mode message display. Addresses Devin review finding on PR #415. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

This reverts commit 0a53aec.

Pull relay-broker directory from relay-broker-integration branch as the foundation for the hybrid approach. Will apply bug fixes from PR #412 on top. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Prevent agents from spoofing human identity by naming themselves "human:..." to bypass release ACL checks. When sender_kind is explicitly Agent, immediately return false regardless of name string. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

When base URL already ends with /stream (e.g. wss://rt.relaycast.dev/stream), avoid appending /v1/stream which produced invalid double-path URLs like wss://host/stream/v1/stream. Now correctly preserves the existing /stream path. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Rename ensure_reaycast_mcp_config to ensure_relaycast_mcp_config (missing 'l' in relaycast). Updated all call sites in main.rs. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Without a body-size bound, a burst of large messages from the same sender within the coalesce window would concatenate without limit, potentially causing large memory allocations. Add MAX_COALESCED_BODY_SIZE (32 KiB) check that flushes the current group when exceeded. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Make terminate_child public so it can be reused from main.rs instead of being duplicated inline, reducing code duplication. Cherry-picked from PR #412. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

khaliqgant · 2026-02-14T21:29:56Z

🤖 Agent Analysis: PR #412 Impact Assessment

Analyzed by: DeepThinker-1 + DeepThinker-2

🔴 Critical Breaking Changes

Impact	Description
relay-dashboard	Will break - imports `@agent-relay/bridge`, `utils`, `wrapper` (all deleted)
npm users	Package becomes `private: true`, all `@agent-relay/*` imports fail
Protocol	Wire format changes from framed binary → newline-delimited JSON
CLI	Commands completely changed (`up/down` → `init/pty/headless/listen`)
API Port	Dashboard :3888 → API :3889

📊 Change Summary

664 files changed (+18,124 / -178,397 lines)
23 packages deleted → 1 package retained (sdk-ts)
12 CI workflows deleted → 1 new Rust CI
Version: 2.1.6 → 3.0.0

⚠️ Operational Gotchas

Build trap: npm ci fails tests without --include=dev flag
Test coverage gap: 3/4 SDK tests skip without RELAY_API_KEY
Platform mismatch: Windows built in CI but not in install.sh
Rust verification: Cannot verify without cargo toolchain

✅ Must-Run Before Merge

Create relay-dashboard migration PR
Verify relay-cloud dependencies
Test binary install (macOS arm64, Linux x64)
Test PTY injection with Claude Code
Verify Relaycast WebSocket connection
Add RELAY_API_KEY to CI for real test coverage

💡 Recommendations

Do not merge until relay-dashboard PR is ready
Consider beta release (3.0.0-beta.1) first
Create migration guide for existing users
Update docs.agent-relay.com with new API

This analysis was performed by AI agents coordinating via agent-relay protocol.

willwashburn requested review from Copilot and khaliqgant February 13, 2026 06:32

Copilot AI reviewed Feb 13, 2026

View reviewed changes

devin-ai-integration Bot reviewed Feb 13, 2026

View reviewed changes

khaliqgant mentioned this pull request Feb 13, 2026

Hybrid relay-broker integration: best of #412 + #404 #415

Merged

7 tasks

khaliqgant added a commit that referenced this pull request Feb 13, 2026

Revert "Hybrid relay-broker integration: best of #412 + #404 (#415)"

6a3f754

This reverts commit 0a53aec.

khaliqgant mentioned this pull request Feb 13, 2026

Hybrid relay-broker integration: best of #412 + #404 #416

Closed

khaliqgant mentioned this pull request Feb 15, 2026

feat: add Rust broker and sdk-ts (additive, no removals) #424

Merged

3 tasks

willwashburn closed this Feb 17, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate Relay-broker into relay main#412

Migrate Relay-broker into relay main#412
willwashburn wants to merge 1 commit intomainfrom
migration

willwashburn commented Feb 13, 2026 •

edited by devin-ai-integration Bot

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 13, 2026

Uh oh!

Copilot AI Feb 13, 2026

Uh oh!

devin-ai-integration Bot left a comment

Uh oh!

khaliqgant commented Feb 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	- run: cargo test --test 'stress_*' -- --ignored
	- run: cargo test stress_ -- --ignored

Conversation

willwashburn commented Feb 13, 2026 • edited by devin-ai-integration Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 13, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 13, 2026

Choose a reason for hiding this comment

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

✅ Devin Review: No Issues Found

Uh oh!

khaliqgant commented Feb 14, 2026

🤖 Agent Analysis: PR #412 Impact Assessment

🔴 Critical Breaking Changes

📊 Change Summary

⚠️ Operational Gotchas

✅ Must-Run Before Merge

💡 Recommendations

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

willwashburn commented Feb 13, 2026 •

edited by devin-ai-integration Bot

Loading