## Summary
- Return errors for required fields (ID, price) in aislegopher and wegmans extractors
- Log warnings for optional fields (title, description, unit price) across all site extractors
- Fix 20+ locations that silently discarded errors with `_ =`
## Files changed
- `sites/aislegopher/aislegopher.go` — return errors for ID and price parsing, warn for name
- `sites/wegmans/wegmans.go` — return errors for ID and price, warn for name/unit price/network idle
- `sites/duckduckgo/page.go` — warn for title/description extraction
- `sites/google/google.go` — warn for title/description extraction
## Test plan
- [x] `go build ./...` passes
- [x] `go test ./...` passes
- [x] `go vet ./...` passes
Closes #24
**Parent:** #31 (Phase 3, PR 10)
Return errors for required fields (ID, price) and log warnings for
optional fields (title, description, unit price) across all site
extractors instead of silently discarding them with _ =.
Closes#24
Co-Authored-By: Claude Opus 4.6 <[email protected]>
Claude
merged commit 0ba9cc9b98 into main2026-02-15 16:32:14 +00:00
Claude
deleted branch fix/silent-parsing-errors2026-02-15 16:32:14 +00:00
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
_ =Files changed
sites/aislegopher/aislegopher.go— return errors for ID and price parsing, warn for namesites/wegmans/wegmans.go— return errors for ID and price, warn for name/unit price/network idlesites/duckduckgo/page.go— warn for title/description extractionsites/google/google.go— warn for title/description extractionTest plan
go build ./...passesgo test ./...passesgo vet ./...passesCloses #24
Parent: #31 (Phase 3, PR 10)