722d85c2a5
All DIS-001 through DIS-110 set to status:done with merge dates. BOARD.md reorganized with Done sections per phase. Phase 1.5 Refinement epic added as planning placeholder. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
43 lines
1.3 KiB
Markdown
43 lines
1.3 KiB
Markdown
---
|
|
id: DIS-106
|
|
status: done
|
|
phase: 1
|
|
priority: p1
|
|
labels: [phase:1, type:refactor, priority:p1]
|
|
branch: phase-1/runner-api-runresult
|
|
assignee: claude-agent
|
|
started: null
|
|
pr: null
|
|
merged: 2026-04-10
|
|
---
|
|
|
|
# DIS-106: Runner-API-Refactoring → `RunResult`-Typ
|
|
|
|
## Ziel
|
|
`runner.ts` gibt einen typsicheren `RunResult`-Typ zurück statt rohem JSON/String.
|
|
Ermöglicht saubere Fehlerunterscheidung (Timeout, CLI-Error, Parse-Error, Success).
|
|
|
|
## Kontext
|
|
→ `docs/development-plan.md` §2 (Runner-Refactoring), Phase 1 Task 6.
|
|
|
|
## Scope
|
|
- **In**: `src/agent/types.ts` mit `RunResult` (discriminated union: `success | timeout | cli-error | parse-error`) · `runner.ts` gibt `RunResult` zurück · `router.ts` und alle Aufrufer nutzen den Typ
|
|
- **Out**: Streaming-Variante (Phase 5)
|
|
|
|
## Definition of Done
|
|
- [ ] `RunResult` ist ein discriminated union, kein `any`
|
|
- [ ] Alle Aufrufer handhaben alle vier Cases explizit (TypeScript enforcement)
|
|
- [ ] Unit-Test `tests/unit/runner-result.test.ts` mit Mock-Spawn für jeden Case
|
|
- [ ] `npm run build && npm test` grün
|
|
|
|
## Dateien (erwartet)
|
|
- `src/agent/types.ts`
|
|
- `src/agent/runner.ts`
|
|
- `src/router.ts`
|
|
- `tests/unit/runner-result.test.ts`
|
|
|
|
## Abhängigkeiten
|
|
Blockiert durch: **DIS-101** (Logger-Refactoring muss vorher stehen).
|
|
|
|
## Spec
|
|
→ `docs/backlog.md` #DIS-106
|