Category: refactor Severity: major
Location: crates/arcp-runtime/src/runtime/server.rs:1-3
What
At 1534 lines, server.rs owns builder configuration, handshake, transport loops, job dispatch, cancellation, subscriptions, artifacts, and utility helpers. This size makes spec-critical behavior hard to audit and has already hidden several missing conformance paths.
Evidence
//! ARCP runtime — the server side that drives the ARCP v1.1 §6.2
//! hello/welcome handshake (serialized as `session.open` / `session.accepted`)
//! and dispatches subsequent envelopes.
Proposed fix
Split server.rs along existing cohesion boundaries: handshake/session lifecycle, job invocation/cancel, subscription handling, artifact dispatch, and builder/configuration.
Acceptance criteria
Category: refactor Severity: major
Location:
crates/arcp-runtime/src/runtime/server.rs:1-3What
At 1534 lines,
server.rsowns builder configuration, handshake, transport loops, job dispatch, cancellation, subscriptions, artifacts, and utility helpers. This size makes spec-critical behavior hard to audit and has already hidden several missing conformance paths.Evidence
Proposed fix
Split
server.rsalong existing cohesion boundaries: handshake/session lifecycle, job invocation/cancel, subscription handling, artifact dispatch, and builder/configuration.Acceptance criteria