keligrubb/kestrelos
1
0
Fork 0
Code Issues 2 Pull Requests 1 Actions Packages Projects Releases 27 Wiki Activity

refactor(websocket): remove unused broadcastToSession export #45

Merged
Keli Grubb merged 1 commits from fix/remove-unused-broadcastToSession into main 2026-08-19 02:11:19 +00:00
Conversation 0 Commits 1 Files Changed 1
-10
Keli Grubb commented 2026-08-15 02:04:12 +00:00
Owner
Copy Link
Copy Source

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

## 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
Keli Grubb added 1 commit 2026-08-15 02:04:12 +00:00
refactor(websocket): remove unused broadcastToSession export
PR / lint (pull_request) Successful in 24s
Details
PR / test (pull_request) Successful in 25s
Details
PR / e2e (pull_request) Successful in 38s
Details
PR / docker-build (pull_request) Successful in 41s
Details
ab0622fcaa 
Remove the dead-code broadcastToSession function from
server/plugins/websocket.js. It was exported but never imported or
called anywhere in the codebase. Its dependencies
(getSessionConnections, addSessionConnection, removeSessionConnection)
remain since they are actively used for per-session connection
tracking.

Closes #40
Keli Grubb merged commit 1f45cc152a into main 2026-08-19 02:11:19 +00:00
Keli Grubb deleted branch fix/remove-unused-broadcastToSession 2026-08-19 02:11:19 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
dependencies
Pull requests that update a dependency file
 security
Vulnerability-driven dependency update
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
jasper (Jasper) marshallgrubb (Marshall Grubb) renovate-bot (Renovate Bot) keligrubb (Keli Grubb)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: keligrubb/kestrelos#45