Fix archive cmd using Flags (archive-only) instead of the merged flags variable that includes browser flags — browser flags were never registered
Move defer DeferClose() after error checks in 6 locations across the codebase, preventing potential nil dereference when Close is called on a nil value
- Fix archive cmd passing only archive-specific Flags instead of the
merged flags variable that includes browser flags (#8)
- Move defer DeferClose() after error checks in 6 locations to prevent
calling Close on nil values (#19):
- sites/duckduckgo/cmd/duckduckgo/main.go
- sites/duckduckgo/duckduckgo.go
- sites/google/cmd/google/main.go
- sites/wegmans/cmd/wegmans/main.go
- sites/wegmans/wegmans.go
- sites/aislegopher/aislegopher.go
Closes#8, #19
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
Flags(archive-only) instead of the mergedflagsvariable that includes browser flags — browser flags were never registereddefer DeferClose()after error checks in 6 locations across the codebase, preventing potential nil dereference when Close is called on a nil valueAffected files:
sites/archive/cmd/archive/main.go(flags fix)sites/duckduckgo/cmd/duckduckgo/main.go(defer fix)sites/duckduckgo/duckduckgo.go(defer fix)sites/google/cmd/google/main.go(defer fix)sites/wegmans/cmd/wegmans/main.go(defer fix)sites/wegmans/wegmans.go(defer fix)sites/aislegopher/aislegopher.go(defer fix)Test plan
go build ./...passesgo test ./...passes--show-browser)Closes #8, #19
Parent: #31 (Phase 2, PR 5)
🤖 Generated with Claude Code