Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add telemetry abstraction to remove Sentry dep (round 2) #47

Merged
merged 2 commits into from
Aug 9, 2024

Commits on Aug 8, 2024

  1. Add telemetry abstraction to remove Sentry dep

    This is a simplified version of #46. It was easier to copy the important
    parts and start clean than try and delete all of the generics code.
    
    Replace the direct dependency on Sentry with the abstract `Telemetry`
    interface from @transitive-bullshit. This should have no impact on
    users who weren't using the Sentry integration, and requires passing
    Sentry as an argument when creating a model instance.
    
    ```ts
    import { ChatModel } from '@dexaai/dexter'
    import * as Sentry from '@sentry/node'
    
    const model = new ChatModel({ telemetry: Sentry })
    ```
    rileytomasek committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    ccfa7ac View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2024

  1. Configuration menu
    Copy the full SHA
    b147543 View commit details
    Browse the repository at this point in the history