customable/claude-mem
1
0
Fork 0
Code Issues 1 Pull requests 5 Projects Releases 163 Packages 2 Wiki Activity Actions

refactor(types): remove unused exports from websocket.ts #170

Merged
jack merged 8 commits from refactor/143-cleanup-websocket-types into main 2026-01-24 17:06:10 +00:00
Conversation 0 Commits 8 Files changed 1 -1
jack commented 2026-01-24 10:22:35 +00:00
Owner
Copy link

Summary

Remove 3 unused exports from websocket.ts:

  • WorkerConnectionOptions interface
  • isWorkerToBackendMessage type guard function
  • isBackendToWorkerMessage type guard function

These were designed for type-safe WebSocket message validation but the actual implementation uses inline type assertions instead.

Reduced file from 235 to 178 lines (-57 lines, -24%).

Refs #143

Test Plan

  • pnpm run typecheck passes
  • Verified via grep that none of the removed exports are used anywhere
## Summary Remove 3 unused exports from websocket.ts: - `WorkerConnectionOptions` interface - `isWorkerToBackendMessage` type guard function - `isBackendToWorkerMessage` type guard function These were designed for type-safe WebSocket message validation but the actual implementation uses inline type assertions instead. Reduced file from 235 to 178 lines (-57 lines, -24%). Refs #143 ## Test Plan - [x] `pnpm run typecheck` passes - [x] Verified via grep that none of the removed exports are used anywhere
refactor(types): remove unused exports from websocket.ts
All checks were successful
CI / check (pull_request) Successful in 0s
Details
CI / build (pull_request) Successful in 42s
Details
CI / validate-plugin (pull_request) Successful in 34s
Details
b18e9ced34 
Remove 3 unused exports:
- WorkerConnectionOptions interface
- isWorkerToBackendMessage type guard function
- isBackendToWorkerMessage type guard function

These were designed for type-safe WebSocket message validation but
the actual implementation uses inline type assertions instead.

Reduced file from 235 to 178 lines (-57 lines, -24%).

Refs #143

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
jonas.hanisch force-pushed refactor/143-cleanup-websocket-types from b18e9ced34
All checks were successful
CI / check (pull_request) Successful in 0s
Details
CI / build (pull_request) Successful in 42s
Details
CI / validate-plugin (pull_request) Successful in 34s
Details
to 2c484c7c9c
All checks were successful
CI / build (pull_request) Successful in 46s
Details
2026-01-24 16:49:10 +00:00 Compare
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m2s
Details
535a441721
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m13s
Details
e36dfd8d1a
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m16s
Details
c611560573
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m16s
Details
de91385985
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m14s
Details
b54fdcb544
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 1m14s
Details
dfb81332e7
Merge branch 'main' into refactor/143-cleanup-websocket-types
All checks were successful
CI / build (pull_request) Successful in 47s
Details
3482cf4d6f
jack merged commit 04d5431591 into main 2026-01-24 17:06:10 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
good first issue

Good for newcomers

  
has-pr

Issue has an associated pull request

  
help wanted

Community contributions welcome

  
idea

Idea or concept - not yet ready for implementation

  
priority
critical
Urgent issue requiring immediate attention

  
priority
high
High priority issue

  
priority
low
Low priority issue

  
priority
medium
Medium priority issue

  
status
blocked
Blocked by external dependency

  
status
in-progress
Currently being worked on

  
status
needs-review
Awaiting code review

  
status
ready
Ready to be worked on

  
type
bug
Something isn't working

  
type
docs
Documentation improvements

  
type
enhancement
Improvement to existing functionality

  
type
feature
New feature or request

  
type
refactor
Code refactoring without functional changes

No labels
good first issue
has-pr
help wanted
idea
priority
critical
priority
high
priority
low
priority
medium
status
blocked
status
in-progress
status
needs-review
status
ready
type
bug
type
docs
type
enhancement
type
feature
type
refactor
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
customable/claude-mem!170
No description provided.