Unused methods in SSEBroadcaster: getClientCount, hasClients, broadcastProcessingStatus #161

New issue

Closed

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

jack commented

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

Owner

Beschreibung

In packages/backend/src/services/sse-broadcaster.ts werden folgende Methoden nicht verwendet:

1. `getClientCount()` (Zeile 159-161)

getClientCount(): number {
  return this.clients.size;
}

2. `hasClients()` (Zeile 166-168)

hasClients(): boolean {
  return this.clients.size > 0;
}

3. `broadcastProcessingStatus()` (Zeile 173-182)

broadcastProcessingStatus(status: {
  pendingTasks: number;
  activeSessions: number;
  connectedWorkers: number;
}): void {
  this.broadcast({
    type: 'processing:status',
    data: status,
  });
}

Analyse

getClientCount / hasClients: Nützliche Utility-Methoden für Monitoring/Debugging
broadcastProcessingStatus: Definiert Event-Type processing:status - könnte für Dashboard-Status gedacht sein

Der Event-Type processing:status ist auch in der SSEEventType Union definiert (Zeile 18), wird aber nirgends gesendet.

Zu prüfen

Ist broadcastProcessingStatus für ein Status-Dashboard geplant?
- Die Signatur deutet auf Live-Status-Updates hin
- Könnte periodisch oder bei Änderungen aufgerufen werden
Werden getClientCount/hasClients für Optimierungen benötigt?
- Z.B. Broadcasting nur wenn Clients verbunden sind
- Monitoring/Health-Checks
UI-Komponenten prüfen:
- Gibt es Frontend-Code der processing:status Events erwartet?

Betroffene Datei

packages/backend/src/services/sse-broadcaster.ts

⚠️ Hinweis

Nicht einfach löschen! Zuerst prüfen:

Ob diese Methoden für geplante Features benötigt werden (Dashboard-Status)
Ob UI-Komponenten diese Events erwarten
processing:status Event-Type müsste ebenfalls aus der Union entfernt werden

## Beschreibung In `packages/backend/src/services/sse-broadcaster.ts` werden folgende Methoden nicht verwendet: ### 1. `getClientCount()` (Zeile 159-161) ```typescript getClientCount(): number { return this.clients.size; } ``` ### 2. `hasClients()` (Zeile 166-168) ```typescript hasClients(): boolean { return this.clients.size > 0; } ``` ### 3. `broadcastProcessingStatus()` (Zeile 173-182) ```typescript broadcastProcessingStatus(status: { pendingTasks: number; activeSessions: number; connectedWorkers: number; }): void { this.broadcast({ type: 'processing:status', data: status, }); } ``` ## Analyse - `getClientCount` / `hasClients`: Nützliche Utility-Methoden für Monitoring/Debugging - `broadcastProcessingStatus`: Definiert Event-Type `processing:status` - könnte für Dashboard-Status gedacht sein Der Event-Type `processing:status` ist auch in der `SSEEventType` Union definiert (Zeile 18), wird aber nirgends gesendet. ## Zu prüfen 1. **Ist `broadcastProcessingStatus` für ein Status-Dashboard geplant?** - Die Signatur deutet auf Live-Status-Updates hin - Könnte periodisch oder bei Änderungen aufgerufen werden 2. **Werden `getClientCount`/`hasClients` für Optimierungen benötigt?** - Z.B. Broadcasting nur wenn Clients verbunden sind - Monitoring/Health-Checks 3. **UI-Komponenten prüfen:** - Gibt es Frontend-Code der `processing:status` Events erwartet? ## Betroffene Datei - `packages/backend/src/services/sse-broadcaster.ts` ## ⚠️ Hinweis Nicht einfach löschen! Zuerst prüfen: - Ob diese Methoden für geplante Features benötigt werden (Dashboard-Status) - Ob UI-Komponenten diese Events erwarten - `processing:status` Event-Type müsste ebenfalls aus der Union entfernt werden