Skip to content

Comments

fix: Specify ld-ai context kind for SDK info tracking#93

Draft
jsonbailey wants to merge 1 commit intomainfrom
devin/AIC-1636-1771611449-specify-context-kind
Draft

fix: Specify ld-ai context kind for SDK info tracking#93
jsonbailey wants to merge 1 commit intomainfrom
devin/AIC-1636-1771611449-specify-context-kind

Conversation

@jsonbailey
Copy link
Contributor

@jsonbailey jsonbailey commented Feb 20, 2026

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Describe the solution you've provided

Sets the context kind to ld-ai on the anonymous context used for the $ld:ai:sdk-info tracking event, per the updated spec in sdk-specs#134. Previously the context used the default user kind.

Human review checklist

  • Verify the kind string ld-ai matches the spec exactly
  • The existing test_sdk_info_tracked_on_init test imports _INIT_TRACK_CONTEXT by reference rather than constructing an expected context independently — it will pass but does not explicitly assert the kind is ld-ai. Consider whether a more explicit assertion is warranted.

Additional context

Co-Authored-By: jbailey@launchdarkly.com <accounts@sidewaysgravity.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant