fix(websocket): add close reasons and suppress noisy warnings

[NathanFlurry]

Description

This PR fixes WebSocket close handling to provide complete close reason information and reduces noisy logging for normal disconnect scenarios.

When WebSocket connections close, the close reason was either missing entirely or would trigger warnings even on normal disconnects (network drops, browser closes). This fix ensures all closes include proper reason codes for debugging while only warning on actually malformed reasons.

Changes

• Add close reason codes to all bare .close() calls using the group.code format
• Suppress "failed to parse close reason" warnings when reason is empty (normal case for network disconnects)
• Standardize close reason formats across codebase (pegboard., ws., sandbox.*)

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

The changes suppress logging on normal WebSocket disconnects. Close reasons now follow the standard format for proper error tracking and debugging through the logging system.

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• My changes generate no new warnings

[railway-app]

🚅 Deployed to the rivet-pr-4476 environment in rivet-frontend

Service	Status	Web	Updated (UTC)
frontend-cloud	😴 Sleeping (View Logs)	Web	Mar 24, 2026 at 11:49 am
frontend-inspector	😴 Sleeping (View Logs)	Web	Mar 22, 2026 at 8:42 am
website	❌ Build Failed (View Logs)	Web	Mar 22, 2026 at 8:36 am
kitchen-sink	❌ Build Failed (View Logs)	Web	Mar 22, 2026 at 8:35 am
mcp-hub	✅ Success (View Logs)	Web	Mar 22, 2026 at 8:34 am
ladle	❌ Build Failed (View Logs)	Web	Mar 22, 2026 at 8:33 am

[claude]

PR Review: fix(websocket): add close reasons and suppress noisy warnings

Overall this is a clean, focused fix. A few observations:

Duplicate branch in ws-proxy.ts (minor)

After the change, both branches in onError are now identical:

if (state.targetWs.readyState === WebSocket.OPEN) {
    state.targetWs.close(1011, "ws.client_error");
} else if (state.targetWs.readyState === WebSocket.CONNECTING) {
    state.targetWs.close(1011, "ws.client_error");
}

These can be collapsed:

if (state.targetWs.readyState === WebSocket.OPEN || state.targetWs.readyState === WebSocket.CONNECTING) {
    state.targetWs.close(1011, "ws.client_error");
}

Incomplete standardization in ws-proxy.ts (follow-up)

The same file still has several non-group.code format close reasons not touched by this PR:

• Line 56: targetWs.close(1001, "Client disconnected")
• Line 102: closeWebSocketIfOpen(clientWs, 1011, "Target WebSocket error")
• Lines 135-136: closeWebSocketIfOpen(clientWs, 1011, "Failed to connect to target")
• Line 155: state.targetWs.close(1000, event.reason || "Client disconnected")

Not a blocker, but worth a follow-up to keep parseWebSocketCloseReason parsing consistent across the file.

Core fixes look correct

• Suppressing warnings on empty reason strings is correct. Empty reasons are the normal case for browser tab closes and network drops.
• The group.code format standardization (pegboard.shutdown_error, ws.client_error, sandbox.client_closed) aligns with the existing parsing convention in parseWebSocketCloseReason.
• Using 1011 (internal error) for the previously bare .close() on a CONNECTING socket in ws-proxy.ts is semantically correct for the error path.
• The Swift client change mirrors the TypeScript logic appropriately.

[pkg-pr-new]

More templates

• actor-actions
• actor-actions-vercel
• ai-agent
• ai-agent-vercel
• example-ai-and-user-generated-actors-freestyle
• chat-room
• chat-room-vercel
• example-cloudflare-workers
• example-cloudflare-workers-hono
• collaborative-document
• collaborative-document-vercel
• cross-actor-actions
• cross-actor-actions-vercel
• example-cursors
• example-cursors-raw-websocket
• example-cursors-raw-websocket-vercel
• example-cursors-vercel
• custom-serverless
• example-elysia
• example-experimental-durable-streams-ai-agent
• example-experimental-durable-streams-ai-agent-vercel
• geo-distributed-database
• geo-distributed-database-vercel
• hello-world
• hello-world-vercel
• example-hono
• hono-react
• hono-react-vercel
• example-kitchen-sink
• example-kitchen-sink-vercel
• multi-region-vercel
• multiplayer-game
• example-multiplayer-game-patterns
• example-multiplayer-game-patterns-vercel
• multiplayer-game-vercel
• native-websockets
• native-websockets-vercel
• next-js
• per-tenant-database
• per-tenant-database-vercel
• example-raw-fetch-handler
• example-raw-fetch-handler-vercel
• example-raw-websocket-handler
• example-raw-websocket-handler-proxy
• example-raw-websocket-handler-proxy-vercel
• example-raw-websocket-handler-vercel
• example-react
• example-react-vercel
• sandbox
• sandbox-coding-agent
• sandbox-coding-agent-vercel
• sandbox-vercel
• scheduling
• scheduling-vercel
• example-sqlite-drizzle
• example-sqlite-raw
• state
• state-vercel
• example-stream
• example-stream-vercel
• example-trpc

@rivetkit/cloudflare-workers

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/cloudflare-workers@4476

@rivetkit/framework-base

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/framework-base@4476

@rivetkit/next-js

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/next-js@4476

@rivetkit/react

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/react@4476

rivetkit

pnpm add https://pkg.pr.new/rivet-dev/rivet/rivetkit@4476

@rivetkit/sql-loader

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sql-loader@4476

@rivetkit/sqlite-vfs

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sqlite-vfs@4476

@rivetkit/traces

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/traces@4476

@rivetkit/workflow-engine

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/workflow-engine@4476

@rivetkit/virtual-websocket

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/virtual-websocket@4476

@rivetkit/engine-runner

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner@4476

@rivetkit/engine-runner-protocol

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner-protocol@4476

commit: 384d66b