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

Epic: Code Quality & Duplication #3

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

Parent epic for code quality improvements, deduplication, and consistency.

Scope

  • Duplicate numericOnly helper in powerball and megamillions
  • Duplicate Result struct across duckduckgo and google
  • Duplicate result extraction logic in duckduckgo Search() vs searchPage.GetResults()
  • Inconsistent browser defaults (Chromium vs Firefox)
  • Inconsistent/outdated user-agent strings
  • mergeOptions boolean field handling

Sub-tasks

Will be linked as they are created.

Parent epic for code quality improvements, deduplication, and consistency. ## Scope - Duplicate `numericOnly` helper in powerball and megamillions - Duplicate `Result` struct across duckduckgo and google - Duplicate result extraction logic in duckduckgo Search() vs searchPage.GetResults() - Inconsistent browser defaults (Chromium vs Firefox) - Inconsistent/outdated user-agent strings - mergeOptions boolean field handling ## Sub-tasks Will be linked as they are created.
Claude added the priority/mediumtype/epic labels 2026-02-14 16:05:32 +00:00
Claude commented 2026-02-14 16:09:05 +00:00
Author
Collaborator
Copy Link
Copy Source

Sub-tasks

  • #13 - Deduplicate numericOnly helper in powerball/megamillions (low)
  • #14 - Deduplicate DuckDuckGo result extraction logic (low)
  • #15 - Inconsistent default browser types Chromium vs Firefox (low)
  • #16 - mergeOptions always overwrites ShowBrowser (medium)
  • #17 - Hardcoded user-agent strings inconsistent/outdated (medium)
  • #20 - staticCookieJar is not thread-safe (low)
  • #22 - Cookie type missing SameSite attribute (low)
  • #23 - Google search only supports 4 countries (low)
## Sub-tasks - #13 - Deduplicate numericOnly helper in powerball/megamillions (low) - #14 - Deduplicate DuckDuckGo result extraction logic (low) - #15 - Inconsistent default browser types Chromium vs Firefox (low) - #16 - mergeOptions always overwrites ShowBrowser (medium) - #17 - Hardcoded user-agent strings inconsistent/outdated (medium) - #20 - staticCookieJar is not thread-safe (low) - #22 - Cookie type missing SameSite attribute (low) - #23 - Google search only supports 4 countries (low)
Claude commented 2026-02-15 20:57:40 +00:00
Author
Collaborator
Copy Link
Copy Source

All sub-tasks resolved:

  • PR #38 — ShowBrowser merge behavior and browser defaults (#15, #16)
  • PR #39 — user-agent consolidation (#17)
  • PR #40 — deduplicate helpers (#13, #14)
  • PR #42 — Google countries and SameSite (#22, #23)

Closing this epic.

All sub-tasks resolved: - PR #38 — ShowBrowser merge behavior and browser defaults (#15, #16) - PR #39 — user-agent consolidation (#17) - PR #40 — deduplicate helpers (#13, #14) - PR #42 — Google countries and SameSite (#22, #23) 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 labels priority/medium type/epic
Milestone
No items
No Milestone
Projects
Clear projects
No projects
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#3