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

Epic: Error Handling & Correctness #1

New Issue
Closed
opened 2026-02-14 16:05:14 +00:00 by Claude · 3 comments
Claude commented 2026-02-14 16:05:14 +00:00
Collaborator
Copy Link

Parent epic for bugs related to error handling, nil pointer risks, and correctness issues across the codebase.

Scope

  • Deferred close before error checks
  • Silently discarded errors from ForEach and other operations
  • Nil pointer dereference risks (SelectFirst returning nil)
  • Incorrect return values (returning nil error when err is set)

Sub-tasks

Will be linked as they are created.

Parent epic for bugs related to error handling, nil pointer risks, and correctness issues across the codebase. ## Scope - Deferred close before error checks - Silently discarded errors from ForEach and other operations - Nil pointer dereference risks (SelectFirst returning nil) - Incorrect return values (returning nil error when err is set) ## Sub-tasks Will be linked as they are created.
Claude added the priority/hightype/epic labels 2026-02-14 16:05:31 +00:00
Claude commented 2026-02-14 16:09:00 +00:00
Author
Collaborator
Copy Link

Sub-tasks

  • #5 - DuckDuckGo Search() discards ForEach error (high)
  • #6 - DuckDuckGo searchPage.GetResults() cannot report errors (high)
  • #7 - updateCookies ignores error from Cookies() (medium)
  • #8 - archive cmd uses wrong flags variable (high)
  • #9 - archive cmd panics on short content (low)
  • #10 - archive.Archive() nil-pointer panic on SelectFirst (high)
  • #11 - document.Refresh() panics on nil response (medium)
  • #18 - archive.Archive() hardcoded time.Sleep ignores context (medium)
  • #19 - defer DeferClose before error check in multiple cmd tools (high)
## Sub-tasks - #5 - DuckDuckGo Search() discards ForEach error (high) - #6 - DuckDuckGo searchPage.GetResults() cannot report errors (high) - #7 - updateCookies ignores error from Cookies() (medium) - #8 - archive cmd uses wrong flags variable (high) - #9 - archive cmd panics on short content (low) - #10 - archive.Archive() nil-pointer panic on SelectFirst (high) - #11 - document.Refresh() panics on nil response (medium) - #18 - archive.Archive() hardcoded time.Sleep ignores context (medium) - #19 - defer DeferClose before error check in multiple cmd tools (high)
Claude commented 2026-02-14 16:09:43 +00:00
Author
Collaborator
Copy Link

Additional sub-task:

  • #24 - Site extractors silently ignore parsing errors with _ = (20+ locations) (high)
Additional sub-task: - #24 - Site extractors silently ignore parsing errors with `_ =` (20+ locations) (high)
Claude commented 2026-02-15 20:57:38 +00:00
Author
Collaborator
Copy Link

All sub-tasks resolved:

  • PR #33 — nil-pointer panics (#10, #11)
  • PR #34 — archive cmd short content panic (#9)
  • PR #35 — DuckDuckGo error handling (#5, #6)
  • PR #36 — defer ordering and cmd flags (#8, #19)
  • PR #37 — cookies error and context-aware sleep (#7, #18)
  • PR #41 — silently ignored parsing errors (#24)

Closing this epic.

All sub-tasks resolved: - PR #33 — nil-pointer panics (#10, #11) - PR #34 — archive cmd short content panic (#9) - PR #35 — DuckDuckGo error handling (#5, #6) - PR #36 — defer ordering and cmd flags (#8, #19) - PR #37 — cookies error and context-aware sleep (#7, #18) - PR #41 — silently ignored parsing errors (#24) Closing this epic.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something isn't working correctly
 claude
Trigger — Steve has marked this issue for Claude to work on.
 claude:blocked
Blocked — Waiting on a dependency (another issue/repo).
 claude:review
In Review — PR created, awaiting human review/merge.
 claude:running
Active — A Claude instance is currently working on this.
 documentation
Documentation improvements or additions
 enhancement
New feature or improvement to existing functionality
 performance
Performance optimization or resource usage improvement
 priority/critical
Showstopper — security vulnerability, data loss, or crash in production
 priority/high
Important — significant bug or high-value improvement
 priority/low
Nice to have — minor improvement or cleanup
 priority/medium
Normal — standard improvement or non-critical bug
 security
Security-related issue or vulnerability
 testing
Test coverage, test infrastructure, or test improvements
 type/epic
Parent issue grouping related stories/tasks
 type/refactor
Code restructuring without behavior change
 type/task
Concrete implementation work item
No Label priority/high type/epic
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Claude fizi steve
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: steve/go-extractor#1
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?