feat: Add support for TrackedChats in the AI SDK #939
Conversation
|
@launchdarkly/browser size report |
|
@launchdarkly/js-sdk-common size report |
|
@launchdarkly/js-client-sdk size report |
|
@launchdarkly/js-client-sdk-common size report |
| // Try specific implementations for the provider | ||
| switch (providerName) { | ||
| case 'openai': | ||
| trackedChat = undefined; | ||
| break; | ||
| case 'bedrock': | ||
| trackedChat = undefined; | ||
| break; | ||
| default: | ||
| trackedChat = undefined; | ||
| } |
There was a problem hiding this comment.
I assume the plan is to fill this out later?
There was a problem hiding this comment.
Yes, we will add support for additional providers
| // Track chat initialization | ||
| this._ldClient.track('$ld:ai:config:function:initChat', context, key, 1); | ||
|
|
||
| const config = await this.config(key, context, defaultValue, variables); |
There was a problem hiding this comment.
I do not care for this line. Can we rename one of these configs to disambiguate them?
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Note
Adds a provider-based TrackedChat with initChat entrypoint, generic metrics tracking via trackMetricsOf, and logger-enabled client/provider wiring.
TrackedChat,TrackedChatFactory, and abstractAIProviderwithinvokeModeland staticcreate.ChatResponseand export new chat modules (src/api/chat/*).initChat(key, context, defaultValue, variables?)toLDAIClient/LDAIClientImpl; emits'$ld:ai:config:function:initChat'and returnsTrackedChat | undefined.LDLoggerintoLDAIClientImpland exposelogger?onLDClientMin.LDAIMetricsand export viaapi/metrics.LDAIConfigTracker.trackMetricsOfto record duration, success/error, and optional token usage; use inTrackedChat.invoke.TrackedChattests for message history and invoke integration.LDAIConfigTrackerImpltests to covertrackMetricsOfsuccess/failure and token usage.api/index.ts.tsconfig.eslint.jsonincludes/excludes.Written by Cursor Bugbot for commit cd534e3. This will update automatically on new commits. Configure here.