From 1f45cc152aa229cbf6846d8d55be3758f0487dde Mon Sep 17 00:00:00 2001 From: Keli Grubb Date: Wed, 19 Aug 2026 02:11:19 +0000 Subject: [PATCH] refactor(websocket): remove unused broadcastToSession export (#45) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Removed - `broadcastToSession` from `server/plugins/websocket.js` — an exported but never-imported, never-called dead-code function. Its dependencies (`getSessionConnections`, `addSessionConnection`, `removeSessionConnection`) are retained since they remain actively used for per-session connection tracking within the plugin. ## Notes - Full repository search confirms zero remaining references to `broadcastToSession`. - ESLint passes; full test suite passes (50 files, 406 tests). Closes #40 --------- Co-authored-by: Madison Grubb Reviewed-on: https://git.keligrubb.com/keligrubb/kestrelos/pulls/45 --- server/plugins/websocket.js | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/server/plugins/websocket.js b/server/plugins/websocket.js index f8cf7c6..9304054 100644 --- a/server/plugins/websocket.js +++ b/server/plugins/websocket.js @@ -43,16 +43,6 @@ export function removeSessionConnection(sessionId, ws) { } } -export function broadcastToSession(sessionId, message) { - const conns = getSessionConnections(sessionId) - const data = JSON.stringify(message) - for (const ws of conns) { - if (ws.readyState === 1) { // OPEN - ws.send(data) - } - } -} - export default defineNitroPlugin((nitroApp) => { nitroApp.hooks.hook('ready', async () => { const server = nitroApp.h3App.server || nitroApp.h3App.nodeServer