Skip to content

[Refactor] Move LLM provider strings to type-safe enum in pkg/types #59

New issue
New issue
Closed
#73
Closed
[Refactor] Move LLM provider strings to type-safe enum in pkg/types#59
#73
Assignees
joshhn
Labels
enhancementNew feature or requestgoPull requests that update go codegood first issueGood for newcomershacktoberfestEligible for Hacktoberfest

Description

@DFanso
DFanso
opened on Oct 5, 2025

Currently, LLM provider names are hardcoded as strings throughout the codebase, which can lead to:

  • Typos and inconsistencies
  • No compile-time validation
  • Difficulty adding new providers
  • Magic strings scattered across files

Proposed Solution
Create a type-safe enum for LLM providers in pkg/types/types.go and update all references throughout the codebase.

Activity

  1. DFanso
    added
    enhancementNew feature or request
    good first issueGood for newcomers
    hacktoberfestEligible for Hacktoberfest
    goPull requests that update go code
    on Oct 5, 2025

  2. joshhn commented on Oct 7, 2025

    @joshhn
    joshhn
    on Oct 7, 2025
    Contributor

    Hey @DFanso, could you assign me this one?

  3. DFanso commented on Oct 7, 2025

    @DFanso
    DFanso
    on Oct 7, 2025
    OwnerAuthor

    Sure @joshhn

  4. DFanso
    assigned
    joshhn
    on Oct 7, 2025
  5. DFanso
    closed this as completedin #73on Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestgoPull requests that update go codegood first issueGood for newcomershacktoberfestEligible for Hacktoberfest

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions