Add nil check for resp in document.go:Refresh() — Playwright's Reload() can return a nil response
Add nil guards for SelectFirst() results in archive.go:Archive() before calling .Type() and .Click(), returning descriptive errors instead of panicking
Test plan
go build ./... passes
go test ./... passes
Verify archive extractor returns error (not panic) when DOM elements are missing
- document.go: check if resp is nil before calling resp.Status() in
Refresh(), since Playwright's Reload() can return a nil response
- archive.go: check SelectFirst() results for nil before calling
Type() and Click(), preventing panics when DOM elements are missing
Closes#10, #11
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
respindocument.go:Refresh()— Playwright'sReload()can return a nil responseSelectFirst()results inarchive.go:Archive()before calling.Type()and.Click(), returning descriptive errors instead of panickingTest plan
go build ./...passesgo test ./...passesRefresh()handles nil response gracefullyCloses #10, #11
Parent: #31 (Phase 1, PR 2)
🤖 Generated with Claude Code