Skip to content

Use Relaycast SDK workspace helpers in openclaw setup#658

Open
noodleonthecape wants to merge 1 commit intomainfrom
noodle/audit-relaycast-sdk-second-pass
Open

Use Relaycast SDK workspace helpers in openclaw setup#658
noodleonthecape wants to merge 1 commit intomainfrom
noodle/audit-relaycast-sdk-second-pass

Conversation

@noodleonthecape
Copy link
Copy Markdown
Contributor

@noodleonthecape noodleonthecape commented Mar 27, 2026

Summary

  • replace the hand-rolled OpenClaw setup workspace create/409/lookup flow with RelayCast.ensureWorkspace()
  • route the patched Relaycast MCP create_workspace helper through RelayCast.createWorkspace() instead of raw fetch logic
  • bump relay to @relaycast/sdk 1.1.0 so the PR Dm messages #98 workspace helpers are available everywhere relay consumes the SDK
  • update the MCP startup test to assert SDK usage

Testing

  • npx vitest run src/cli/relaycast-mcp.startup.test.ts packages/openclaw/src/__tests__/gateway-control.test.ts packages/openclaw/src/__tests__/gateway-poll-fallback.test.ts packages/openclaw/src/__tests__/gateway-threads.test.ts packages/openclaw/src/__tests__/ws-client.test.ts

Audit notes

  • I did a second-pass audit for missed Relaycast SDK integrations after PR Dm messages #98.
  • I did not find a justified follow-up change needed in AgentWorkforce/relaycast; remaining @relaycast/sdk/internal usage in relay's patched MCP path still appears intentional for internal origin tagging / WS client wiring rather than a missed public API migration.

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 4 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