Unused methods in WebSocketClient: getState, getWorkerId, sendTaskProgress #173

New issue

Closed

opened 2026-01-24 10:24:37 +00:00 by jack · 0 comments

jack commented

2026-01-24 10:24:37 +00:00

Owner

Beschreibung

In packages/worker/src/connection/websocket-client.ts werden folgende Methoden nicht verwendet:

1. `getState()` (Zeile 94-96)

getState(): ConnectionState {
  return this.state;
}

2. `getWorkerId()` (Zeile 101-103)

getWorkerId(): string | null {
  return this.workerId;
}

3. `sendTaskProgress()` (Zeile 185-193)

sendTaskProgress(taskId: string, progress: number, message?: string): void {
  if (!this.workerId) return;

  this.send({
    type: 'task:progress',
    taskId,
    workerId: this.workerId,
    progress,
    message,
  });
}

Analyse

getState / getWorkerId: Nützliche Getter für externes State-Monitoring
sendTaskProgress: Ermöglicht Fortschritts-Updates während Task-Verarbeitung

sendTaskProgress sendet task:progress Messages - das Backend müsste diese verarbeiten und an die UI weiterleiten.

Zu prüfen

Wird task:progress vom Backend verarbeitet?
- Prüfen ob WorkerHub diesen Message-Type handelt
- SSE-Event für UI-Updates?
Sollten Handler sendTaskProgress nutzen?
- Z.B. bei langen Observation-Extraktionen
- Embedding-Generierung mit Fortschrittsanzeige
Sind die Getter für Debugging/Monitoring gedacht?
- Könnten für Health-Checks verwendet werden

Betroffene Datei

packages/worker/src/connection/websocket-client.ts

⚠️ Hinweis

Nicht einfach löschen! Zuerst prüfen:

sendTaskProgress ist ein geplantes Feature für UI-Feedback
Getter könnten für externe Integration vorgesehen sein
Backend-seitige Verarbeitung von task:progress prüfen

## Beschreibung In `packages/worker/src/connection/websocket-client.ts` werden folgende Methoden nicht verwendet: ### 1. `getState()` (Zeile 94-96) ```typescript getState(): ConnectionState { return this.state; } ``` ### 2. `getWorkerId()` (Zeile 101-103) ```typescript getWorkerId(): string | null { return this.workerId; } ``` ### 3. `sendTaskProgress()` (Zeile 185-193) ```typescript sendTaskProgress(taskId: string, progress: number, message?: string): void { if (!this.workerId) return; this.send({ type: 'task:progress', taskId, workerId: this.workerId, progress, message, }); } ``` ## Analyse - `getState` / `getWorkerId`: Nützliche Getter für externes State-Monitoring - `sendTaskProgress`: Ermöglicht Fortschritts-Updates während Task-Verarbeitung `sendTaskProgress` sendet `task:progress` Messages - das Backend müsste diese verarbeiten und an die UI weiterleiten. ## Zu prüfen 1. **Wird `task:progress` vom Backend verarbeitet?** - Prüfen ob `WorkerHub` diesen Message-Type handelt - SSE-Event für UI-Updates? 2. **Sollten Handler `sendTaskProgress` nutzen?** - Z.B. bei langen Observation-Extraktionen - Embedding-Generierung mit Fortschrittsanzeige 3. **Sind die Getter für Debugging/Monitoring gedacht?** - Könnten für Health-Checks verwendet werden ## Betroffene Datei - `packages/worker/src/connection/websocket-client.ts` ## ⚠️ Hinweis Nicht einfach löschen! Zuerst prüfen: - `sendTaskProgress` ist ein **geplantes Feature** für UI-Feedback - Getter könnten für externe Integration vorgesehen sein - Backend-seitige Verarbeitung von `task:progress` prüfen