diff options
| author | Claudomator Agent <agent@claudomator> | 2026-04-11 18:10:32 +0000 |
|---|---|---|
| committer | Claudomator Agent <agent@claudomator> | 2026-04-11 18:10:32 +0000 |
| commit | e94573bb84874eda7d233cafc36f3a21688c0568 (patch) | |
| tree | 31a3bce72364b08829502b3d3ffa1abbc0962f3d /internal/api/validate_test.go | |
| parent | 40513ffddba01467193c3c3e19468c7090f06215 (diff) | |
cleanup: remove dead code (QuestionRegistry, changestats wrappers, scanner.Err)
Fix 1: Remove QuestionRegistry and related types (QuestionHandler, PendingQuestion)
from question.go -- nothing reads Pool.Questions or uses the registry. Remove
NewQuestionRegistry() call from NewPool and the Questions field from Pool.
Remove the now-superfluous registry tests; keep stream/parse helpers which are
still used by the claude runner.
Fix 2: Check scanner.Err() after the parseStream loop so I/O errors from the
scanner are not silently swallowed when streamErr is still nil.
Fix 3: Delete internal/api/changestats.go -- the parseChangestatFromFile and
parseChangestatFromOutput wrappers were only needed to support processResult(),
which no longer calls them; they are unreachable dead code.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/api/validate_test.go')
0 files changed, 0 insertions, 0 deletions