steve/go-extractor
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix cmd flags and defer-before-error-check (#8, #19) #36

Merged
Claude merged 1 commits from fix/cmd-flags-and-defer-ordering into main 2026-02-15 16:18:54 +00:00
Conversation 0 Commits 1 Files Changed 7 +7 -13

1 Commits

Author SHA1 Message Date
Steve Dudenhoeffer
e5e0db85e8 fix: use merged flags in archive cmd and move defer after error checks
All checks were successful
CI / vet (pull_request) Successful in 29s
Details
CI / build (pull_request) Successful in 32s
Details
CI / test (pull_request) Successful in 57s
Details 
- 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 <noreply@anthropic.com>
 2026-02-15 16:17:38 +00:00