Ungenutzte API-Types in types/api.ts prüfen und aufräumen #131

New issue

Closed

opened 2026-01-24 09:41:22 +00:00 by jack · 1 comment

jack commented

2026-01-24 09:41:22 +00:00

Owner

Problem

In packages/types/src/api.ts sind viele TypeScript-Interfaces definiert, die nirgendwo im Code verwendet werden.

Ungenutzte Types

Type	Kategorie	Verwendet
`ApiError`	Error	❌
`HealthResponse`	Health	❌
`ReadinessResponse`	Health	❌
`SessionListResponse`	Sessions	❌
`SessionDetailResponse`	Sessions	❌
`ObservationSearchParams`	Observations	❌
`ObservationListResponse`	Observations	❌
`SearchRequest`	Search	❌
`SearchResponse`	Search	❌
`TimelineRequest`	Search	❌
`TimelineResponse`	Search	❌
`HookObservationRequest`	Hooks	❌
`HookObservationResponse`	Hooks	❌
`HookSummarizeRequest`	Hooks	❌
`HookSummarizeResponse`	Hooks	❌
`HookContextRequest`	Hooks	❌
`HookContextResponse`	Hooks	❌
`WorkersListResponse`	Workers	❌
`QueueStatusResponse`	Queue	❌
`TaskStatusResponse`	Tasks	❌
`SettingsResponse`	Settings	❌
`SettingsUpdateRequest`	Settings	❌

Analyse

Diese Types wurden vermutlich als API-Contracts definiert, werden aber nicht konsequent verwendet:

Backend-Routes verwenden sie nicht für Typisierung
UI API-Client hat eigene lokale Types
Hooks-Package hat eigene Response-Types

Optionen

Option A: Entfernen

Ungenutzte Types entfernen - sie bieten keinen Mehrwert wenn sie nicht verwendet werden.

Option B: Konsequent verwenden

Types an allen Stellen einsetzen:

Backend-Routes für Request/Response Typisierung
UI API-Client für typsichere Responses
Hooks-Client für typsichere Kommunikation

Option B würde die Codequalität verbessern und Typsicherheit zwischen Frontend/Backend gewährleisten.

Betroffene Datei

packages/types/src/api.ts

## Problem In `packages/types/src/api.ts` sind viele TypeScript-Interfaces definiert, die nirgendwo im Code verwendet werden. ### Ungenutzte Types | Type | Kategorie | Verwendet | |------|-----------|-----------| | `ApiError` | Error | ❌ | | `HealthResponse` | Health | ❌ | | `ReadinessResponse` | Health | ❌ | | `SessionListResponse` | Sessions | ❌ | | `SessionDetailResponse` | Sessions | ❌ | | `ObservationSearchParams` | Observations | ❌ | | `ObservationListResponse` | Observations | ❌ | | `SearchRequest` | Search | ❌ | | `SearchResponse` | Search | ❌ | | `TimelineRequest` | Search | ❌ | | `TimelineResponse` | Search | ❌ | | `HookObservationRequest` | Hooks | ❌ | | `HookObservationResponse` | Hooks | ❌ | | `HookSummarizeRequest` | Hooks | ❌ | | `HookSummarizeResponse` | Hooks | ❌ | | `HookContextRequest` | Hooks | ❌ | | `HookContextResponse` | Hooks | ❌ | | `WorkersListResponse` | Workers | ❌ | | `QueueStatusResponse` | Queue | ❌ | | `TaskStatusResponse` | Tasks | ❌ | | `SettingsResponse` | Settings | ❌ | | `SettingsUpdateRequest` | Settings | ❌ | ## Analyse Diese Types wurden vermutlich als API-Contracts definiert, werden aber nicht konsequent verwendet: - Backend-Routes verwenden sie nicht für Typisierung - UI API-Client hat eigene lokale Types - Hooks-Package hat eigene Response-Types ## Optionen ### Option A: Entfernen Ungenutzte Types entfernen - sie bieten keinen Mehrwert wenn sie nicht verwendet werden. ### Option B: Konsequent verwenden Types an allen Stellen einsetzen: - Backend-Routes für Request/Response Typisierung - UI API-Client für typsichere Responses - Hooks-Client für typsichere Kommunikation Option B würde die Codequalität verbessern und Typsicherheit zwischen Frontend/Backend gewährleisten. ## Betroffene Datei `packages/types/src/api.ts`

jack added the

labels

2026-01-24 09:41:25 +00:00

jack commented

2026-01-24 09:42:50 +00:00

Author

Owner

⚠️ Wichtig: Nicht einfach löschen! Vor dem Entfernen sollte die komplette Implementierung geprüft werden. Diese API-Types wurden als Contracts definiert und sollten idealerweise konsequent verwendet werden für Typsicherheit zwischen Frontend/Backend.

⚠️ **Wichtig:** Nicht einfach löschen! Vor dem Entfernen sollte die komplette Implementierung geprüft werden. Diese API-Types wurden als Contracts definiert und sollten idealerweise konsequent verwendet werden für Typsicherheit zwischen Frontend/Backend.