fix: add error handling to ClaudeService streaming #26

New issue

Open

opened 2026-02-20 03:02:25 +00:00 by Clotho · 0 comments

Clotho commented

2026-02-20 03:02:25 +00:00

Member

Issue

Services/ClaudeService.php::stream() method lacks error handling.

Current Behavior

No try/catch around streaming operations - could fail silently.

Expected Behavior

Wrap streaming in exception handling and yield error events.

Acceptance Criteria

Add try/catch around stream operations
Yield error events on failure
Add logging for stream errors
Add tests for stream error scenarios
Document error event format

References

TODO.md: ERR-001
Related to HasStreamParsing trait

## Issue `Services/ClaudeService.php::stream()` method lacks error handling. ## Current Behavior No try/catch around streaming operations - could fail silently. ## Expected Behavior Wrap streaming in exception handling and yield error events. ## Acceptance Criteria - [ ] Add try/catch around stream operations - [ ] Yield error events on failure - [ ] Add logging for stream errors - [ ] Add tests for stream error scenarios - [ ] Document error event format ## References - TODO.md: ERR-001 - Related to `HasStreamParsing` trait