Ungenutzte Exporte in hooks/client.ts: logger und isReady #139

New issue

Closed

opened 2026-01-24 09:49:57 +00:00 by jack · 0 comments

jack commented

2026-01-24 09:49:57 +00:00

Owner

Problem

In packages/hooks/src/client.ts sind Exporte definiert, die nicht verwendet werden.

Ungenutzte Exporte

Export	Zeile	Verwendet
`logger`	10	❌ (definiert, aber keine `logger.*` Aufrufe)
`isReady`	63	❌ (nur `isCoreReady` wird aufgerufen)

Details

logger:

const logger = createLogger('hooks-client');
// Nirgends verwendet - kein logger.info(), logger.debug(), etc.

isReady:

async isReady(timeout = ...): Promise<boolean> {
  // Prüft /api/readiness Endpoint
}

Die Hooks verwenden stattdessen isCoreReady() welches /api/health prüft.

Analyse

logger - Wurde vermutlich für Debugging erstellt, aber nie eingesetzt
isReady - Prüft /api/readiness (volle Readiness inkl. DB), aber Hooks brauchen nur isCoreReady (schneller Check)

Optionen

Option A: Entfernen

logger entfernen wenn kein Logging benötigt wird
isReady entfernen wenn isCoreReady ausreicht

Option B: Verwenden

logger für Debugging/Tracing in Client-Methoden einsetzen
isReady für vollständige Readiness-Checks (z.B. vor kritischen Operationen)

⚠️ Wichtig: Nicht einfach löschen! Prüfen ob logger für besseres Debugging eingesetzt werden sollte und ob isReady für bestimmte Szenarien sinnvoll wäre.

Betroffene Datei

packages/hooks/src/client.ts

## Problem In `packages/hooks/src/client.ts` sind Exporte definiert, die nicht verwendet werden. ### Ungenutzte Exporte | Export | Zeile | Verwendet | |--------|-------|-----------| | `logger` | 10 | ❌ (definiert, aber keine `logger.*` Aufrufe) | | `isReady` | 63 | ❌ (nur `isCoreReady` wird aufgerufen) | ### Details **logger:** ```typescript const logger = createLogger('hooks-client'); // Nirgends verwendet - kein logger.info(), logger.debug(), etc. ``` **isReady:** ```typescript async isReady(timeout = ...): Promise<boolean> { // Prüft /api/readiness Endpoint } ``` Die Hooks verwenden stattdessen `isCoreReady()` welches `/api/health` prüft. ## Analyse - `logger` - Wurde vermutlich für Debugging erstellt, aber nie eingesetzt - `isReady` - Prüft `/api/readiness` (volle Readiness inkl. DB), aber Hooks brauchen nur `isCoreReady` (schneller Check) ## Optionen ### Option A: Entfernen - `logger` entfernen wenn kein Logging benötigt wird - `isReady` entfernen wenn `isCoreReady` ausreicht ### Option B: Verwenden - `logger` für Debugging/Tracing in Client-Methoden einsetzen - `isReady` für vollständige Readiness-Checks (z.B. vor kritischen Operationen) ⚠️ **Wichtig:** Nicht einfach löschen! Prüfen ob `logger` für besseres Debugging eingesetzt werden sollte und ob `isReady` für bestimmte Szenarien sinnvoll wäre. ## Betroffene Datei `packages/hooks/src/client.ts`