Conversation
|
@launchdarkly/js-sdk-common size report |
|
@launchdarkly/browser size report |
|
@launchdarkly/js-client-sdk-common size report |
|
@launchdarkly/js-client-sdk size report |
| expect(mockLdClient.track).toHaveBeenNthCalledWith( | ||
| 1, | ||
| '$ld:ai:sdk-info', | ||
| { kind: 'user', key: 'ld-internal-tracking', anonymous: true }, |
There was a problem hiding this comment.
This is interesting. I feel like we probably shouldn't use 'user'? I feel like we should probably use something more esoteric. It is also interesting to be using a context, but anonymous should handle most concerns.
Note
Medium Risk
Changes the analytics/telemetry event names and payload schemas, which can break downstream dashboards or ingestion if consumers still expect the old keys. Functional AI behavior is otherwise unchanged.
Overview
Standardizes AI SDK usage reporting by renaming the
trackevent keys emitted fromLDAIClientImplfor config evaluation and object creation (completion/agent/judge, batch agent configs,createChat,createJudge) to the new$ld:ai:usage:*namespace.Adds constructor-time tracking of SDK identity via a new
$ld:ai:sdk-infoevent sent under an internal anonymous context, includingaiSdkName,aiSdkVersion, and newaiSdkLanguage.Simplifies per-operation tracking payloads by removing
aiSdkName/aiSdkVersionfromLDAIConfigTrackerImpl.getTrackData()and its interface, with tests updated accordingly.Written by Cursor Bugbot for commit ef82f52. This will update automatically on new commits. Configure here.