## Summary
- Change `SearchPage.GetResults()` interface to return `([]Result, error)` so `ForEach` errors are propagated instead of silently discarded
- Fix `Search()` to return `res, err` instead of `res, nil`, propagating the `ForEach` error
- Update cmd caller to check and handle `GetResults()` errors
**Note:** This is a breaking interface change to `SearchPage`. PR 9 (deduplication) depends on this PR.
## Test plan
- [x] `go build ./...` passes
- [x] `go test ./...` passes
- [ ] Verify errors from DOM extraction are now surfaced to callers
Closes #5, #6
**Parent:** #31 (Phase 2, PR 4)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
- Change SearchPage.GetResults() to return ([]Result, error) so ForEach
errors are no longer silently discarded
- Fix Search() to return the ForEach error instead of nil
- Update cmd caller to check GetResults() errors
Closes#5, #6
Co-Authored-By: Claude Opus 4.6 <[email protected]>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
SearchPage.GetResults()interface to return([]Result, error)soForEacherrors are propagated instead of silently discardedSearch()to returnres, errinstead ofres, nil, propagating theForEacherrorGetResults()errorsNote: This is a breaking interface change to
SearchPage. PR 9 (deduplication) depends on this PR.Test plan
go build ./...passesgo test ./...passesCloses #5, #6
Parent: #31 (Phase 2, PR 4)
🤖 Generated with Claude Code