refactor(types): remove unused api.ts file #133

Merged

jonas.hanisch merged 3 commits from refactor/131-cleanup-api-types into main

2026-01-24 12:10:53 +00:00

jack commented

2026-01-24 09:43:45 +00:00

Owner

Summary

Removed entire api.ts file (373 lines)
22+ unused type definitions removed

Removed Types

Category	Types
Common	`ApiError`, `PaginationParams`, `PaginatedResponse`
Health	`HealthResponse`, `ReadinessResponse`
Sessions	`SessionListResponse`, `SessionDetailResponse`
Observations	`ObservationSearchParams`, `ObservationListResponse`
Search	`SearchRequest`, `SearchResponse`, `TimelineRequest`, `TimelineResponse`
Hooks	`HookObservationRequest/Response`, `HookSummarizeRequest/Response`, `HookContextRequest/Response`
Workers	`WorkersListResponse`
Queue	`QueueStatusResponse`, `TaskStatusResponse`
Settings	`SettingsResponse`, `SettingsUpdateRequest`
SSE	`SSEEventType`, `BaseSSEEvent`, `SSEEvent`, and event types

Note

SSE types are currently duplicated locally in:

packages/ui/src/hooks/useSSE.ts
packages/backend/src/services/sse-broadcaster.ts

Consider consolidating these in a future refactor to ensure type consistency.

Test plan

TypeScript compilation passes
All existing functionality works (types were never imported)

Closes #131

🤖 Generated with Claude Code

## Summary - Removed entire `api.ts` file (373 lines) - 22+ unused type definitions removed ## Removed Types | Category | Types | |----------|-------| | Common | `ApiError`, `PaginationParams`, `PaginatedResponse` | | Health | `HealthResponse`, `ReadinessResponse` | | Sessions | `SessionListResponse`, `SessionDetailResponse` | | Observations | `ObservationSearchParams`, `ObservationListResponse` | | Search | `SearchRequest`, `SearchResponse`, `TimelineRequest`, `TimelineResponse` | | Hooks | `HookObservationRequest/Response`, `HookSummarizeRequest/Response`, `HookContextRequest/Response` | | Workers | `WorkersListResponse` | | Queue | `QueueStatusResponse`, `TaskStatusResponse` | | Settings | `SettingsResponse`, `SettingsUpdateRequest` | | SSE | `SSEEventType`, `BaseSSEEvent`, `SSEEvent`, and event types | ## Note SSE types are currently duplicated locally in: - `packages/ui/src/hooks/useSSE.ts` - `packages/backend/src/services/sse-broadcaster.ts` Consider consolidating these in a future refactor to ensure type consistency. ## Test plan - [x] TypeScript compilation passes - [ ] All existing functionality works (types were never imported) Closes #131 🤖 Generated with [Claude Code](https://claude.com/claude-code)

jack added 1 commit

2026-01-24 09:43:45 +00:00

refactor(types): remove unused api.ts file

CI / check (pull_request) Successful in 1s

Details

CI / build (pull_request) Successful in 42s

Details

CI / validate-plugin (pull_request) Successful in 35s

Details

1129e08a5f

Removed entire api.ts file (373 lines) containing 22+ unused type
definitions that were never imported anywhere in the codebase:

- ApiError, HealthResponse, ReadinessResponse
- SessionListResponse, SessionDetailResponse
- ObservationSearchParams, ObservationListResponse
- SearchRequest, SearchResponse, TimelineRequest, TimelineResponse
- HookObservationRequest/Response, HookSummarizeRequest/Response
- HookContextRequest/Response
- WorkersListResponse, QueueStatusResponse, TaskStatusResponse
- SettingsResponse, SettingsUpdateRequest
- SSEEventType, BaseSSEEvent, SSEEvent, and related event types

Note: SSE types are duplicated locally in:
- packages/ui/src/hooks/useSSE.ts
- packages/backend/src/services/sse-broadcaster.ts

These local definitions should be consolidated in a future refactor.

Fixes #131

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>