Skip to content

chore: remove dead broker mode code#70

Merged
khaliqgant merged 1 commit intomainfrom
remove-dead-broker-code
Mar 17, 2026
Merged

chore: remove dead broker mode code#70
khaliqgant merged 1 commit intomainfrom
remove-dead-broker-code

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Mar 17, 2026

Summary

  • Removes the dead integrated broker server (server.ts) and all 23 files exclusively used by it (~7,700 lines deleted)
  • Removes unused dependencies: @agent-relay/storage, @agent-relay/protocol, @agent-relay/utils
  • All entry points already route through proxy-server.ts — broker mode was completely dead code

What was removed

  • Core: server.ts (the integrated broker server entry point)
  • Routes (11 files): auth, messaging, history, channels-integrated, decisions, tasks, fleet, spawn, system, settings, ui
  • Libraries (7 files): server-state, channel-state, broadcast, websocket-runtime, attachment-storage, cli-auth, data-assembly
  • Services (2 files): broker-spawn-reader, user-bridge
  • WebSocket (3 files): main, bridge, presence
  • Cleaned up: types/index.ts, services/index.ts, websocket/logs.ts (removed setupLogsWebSocket)

Test plan

  • TypeScript typecheck passes (tsc --noEmit)
  • All 210 tests pass across 15 test files
  • Verify dashboard runs correctly in standalone/proxy mode

🤖 Generated with Claude Code


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

The integrated broker server (server.ts) is completely unused — all entry
points route through proxy-server.ts. This removes server.ts and all 23
files exclusively used by it (~7,700 lines), plus the @agent-relay/storage,
@agent-relay/protocol, and @agent-relay/utils dependencies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@khaliqgant khaliqgant force-pushed the remove-dead-broker-code branch from f735ae5 to cd12a45 Compare March 17, 2026 10:38
@khaliqgant khaliqgant merged commit e201082 into main Mar 17, 2026
1 check failed
@khaliqgant khaliqgant deleted the remove-dead-broker-code branch March 17, 2026 10:38
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