Skip to content

refactor(relaycast): consume sdk bootstrap helpers#657

Open
noodleonthecape wants to merge 2 commits intomainfrom
noodle/relay-use-relaycast-sdk-bootstrap
Open

refactor(relaycast): consume sdk bootstrap helpers#657
noodleonthecape wants to merge 2 commits intomainfrom
noodle/relay-use-relaycast-sdk-bootstrap

Conversation

@noodleonthecape
Copy link
Copy Markdown
Contributor

@noodleonthecape noodleonthecape commented Mar 27, 2026

Summary

  • switch relay's broker-side Relaycast bootstrap logic to the new SDK helpers
  • use the Rust SDK's idempotent workspace-stream and channel ensure/join APIs instead of hand-rolled 409 handling in relay
  • pin the Rust dependency to the relaycast SDK branch commit so this PR remains buildable before a crates.io release

Why

Relay had SDK-shaped logic locally in src/relaycast_ws.rs for:

  • enabling the workspace stream before websocket connect
  • creating broker channels if missing
  • joining those channels while treating create/join conflicts as no-ops

That logic now lives in the relaycast SDK, so relay should consume it rather than re-implement it.

Validation

  • Cross-repo validation completed for the new SDK API surface in AgentWorkforce/relaycast:
    • npm run build --workspace=@relaycast/types
    • npx vitest run packages/sdk-typescript/src/__tests__/agent.test.ts packages/sdk-typescript/src/__tests__/relay.test.ts
  • Relay-side Rust tests could not be executed in this environment because cargo is not installed on the host.
  • The relay Cargo.lock entry was updated to the exact relaycast git revision used by this PR so CI can resolve the dependency consistently.

Depends on


Open with Devin

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 2 additional findings.

Open in Devin Review

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