refactor: split main.ts god module into focused sub-modules #1137

New issue

Closed

opened 2026-05-13 21:28:00 +00:00 by claude-desktop · 0 comments

claude-desktop commented

2026-05-13 21:28:00 +00:00

Collaborator

As a developer maintaining the server, I want main.ts to be a thin bootstrap orchestrator (≤300 lines), so that each concern — HTTP routing, worker registry, task history, SSE broadcast, startup wiring — has a dedicated module that can be read, tested, and changed independently.

Decomposition

Extract WorkerRegistry (FIFO queue + concurrency limiter) → infrastructure/worker/worker-registry.ts
Extract SSEBroadcaster (per-client streams + fanout) → infrastructure/sse/sse-broadcaster.ts
Extract task history append / query → already partially in infrastructure/database/; wire consistently
Extract HTTP server construction + route mounting → http/server.ts
main.ts becomes: parse env, open DB, construct dependencies, call startServer()

Constraints

No change to public HTTP API
No change to systemd unit or just dev / just start entrypoints

Out of scope

Splitting webhook-config.ts (tracked separately)
Splitting board.ts (tracked separately)
Drizzle ORM migration

References

apps/server/src/main.ts (4924 lines)
Codebase audit 2026-05-13

As a developer maintaining the server, I want `main.ts` to be a thin bootstrap orchestrator (≤300 lines), so that each concern — HTTP routing, worker registry, task history, SSE broadcast, startup wiring — has a dedicated module that can be read, tested, and changed independently. ### Decomposition - [ ] Extract `WorkerRegistry` (FIFO queue + concurrency limiter) → `infrastructure/worker/worker-registry.ts` - [ ] Extract `SSEBroadcaster` (per-client streams + fanout) → `infrastructure/sse/sse-broadcaster.ts` - [ ] Extract task history append / query → already partially in `infrastructure/database/`; wire consistently - [ ] Extract HTTP server construction + route mounting → `http/server.ts` - [ ] `main.ts` becomes: parse env, open DB, construct dependencies, call `startServer()` ### Constraints - [ ] No change to public HTTP API - [ ] No change to systemd unit or `just dev` / `just start` entrypoints ### Out of scope - Splitting `webhook-config.ts` (tracked separately) - Splitting `board.ts` (tracked separately) - Drizzle ORM migration ### References - `apps/server/src/main.ts` (4924 lines) - Codebase audit 2026-05-13

claude-desktop added the

area:infra

type:chore

labels

2026-05-13 21:28:20 +00:00