style(transport): combine toClientState fall-through, clarify CREATED exclusion

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: dimakis

packages/protocol/src/event-store.ts

@@ -32,9 +32,6 @@ function toClientState(state: SessionState): ClientSessionState {
     case 'CREATED':
     case 'CLOSING':
     case 'ENDED':
-      return 'idle';
-    // DETACHED/SUSPENDED: return 'idle' as safe default. The server layer
-    // can override if needed (e.g. to reflect last-known running state).
     case 'DETACHED':
     case 'SUSPENDED':
       return 'idle';
@@ -667,8 +664,8 @@ export class EventStore {
    * Returns the number of sessions recovered.
    */
   recoverStaleSessions(): number {
-    // CREATED is excluded: it's transient and moves to STARTING immediately (see isActive comment
-    // in setSessionState). All other non-terminal states are recovered.
+    // CREATED excluded: transient state, moves to STARTING synchronously in startChat().
+    // The crash window between CREATED and STARTING is negligible.
     const staleStates = ['ACTIVE', 'STARTING', 'DETACHED', 'SUSPENDED', 'CLOSING'];
     const placeholders = staleStates.map(() => '?').join(', ');
     // Inline prepare is intentional — this runs once at startup, not worth caching.