Skip to content

feat(unity): add telemetry to Unity SDK - #39

Draft
nicklasl wants to merge 6 commits into
mainfrom
nicklasl/add-unity-telemetry
Draft

feat(unity): add telemetry to Unity SDK#39
nicklasl wants to merge 6 commits into
mainfrom
nicklasl/add-unity-telemetry

Conversation

@nicklasl

@nicklasl nicklasl commented Mar 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Summary

  • Add opt-out telemetry for Unity resolve latency and evaluation outcomes
  • Upload JSON directly to the telemetry endpoint, matching confidence-sdk-js
  • Bound the shared buffer at 100 traces and flush on capacity, a jittered 30-second cadence, and application lifecycle events
  • Keep uploads single-flight and best-effort, with bounded exponential retry
  • Use canonical telemetry enum values and structured evaluation results

Test plan

  • Build with warnings as errors
  • Focused Unity telemetry and endpoint tests (17 passed)
  • Unity Editor build (Unity is unavailable locally)

@nicklasl @claude
feat(unity): add telemetry to Unity SDK
32ea846 
Track resolve latency and evaluation outcomes, sent via
X-CONFIDENCE-TELEMETRY header on resolve requests. Manual
protobuf encoding, all call sites guarded with try/catch.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
vahidlazio
vahidlazio reviewed Apr 2, 2026
View reviewed changes
Comment thread unity-sdk/Runtime/Providers/ConfidenceApiClient.cs Outdated
// Add the client as a component
ConfidenceApiClient client = clientGO.AddComponent<ConfidenceApiClient>();
client.clientSecret = clientSecret;
client.Telemetry = new Telemetry.Telemetry(Platform.DotNet, client.sdkVersion);

@vahidlazio vahidlazio Apr 2, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be unity?

@nicklasl nicklasl Apr 2, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vahidlazio vahidlazio vahidlazio left review comments
@fabriziodemaria fabriziodemaria Awaiting requested review from fabriziodemaria fabriziodemaria will be requested when the pull request is marked ready for review fabriziodemaria is a code owner
@mfranberg mfranberg Awaiting requested review from mfranberg mfranberg will be requested when the pull request is marked ready for review mfranberg is a code owner
@DennisPersson DennisPersson Awaiting requested review from DennisPersson DennisPersson will be requested when the pull request is marked ready for review DennisPersson is a code owner
@andreas-karlsson andreas-karlsson Awaiting requested review from andreas-karlsson andreas-karlsson will be requested when the pull request is marked ready for review andreas-karlsson is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nicklasl @vahidlazio