Skip to content

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Include full context in custom events and update to inline-context-all capability

Description

This PR updates the Flutter Client SDK to include the full context in custom events instead of just context keys. It also updates the contract test capabilities to use inline-context-all instead of inline-context.

Changes

  • Updated CustomEventSerialization class to include full context for custom events
  • Updated contract test capabilities to use inline-context-all

Testing

  • Changes will be verified by CI tests

Link to Devin run

https://app.devin.ai/sessions/6315fa1653c84dcc931a5b3ec6485a64

Requested by

[email protected]

@devin-ai-integration devin-ai-integration bot requested a review from a team as a code owner May 8, 2025 13:57
Copy link
Contributor Author

Original prompt from [email protected]:

Received message in Slack channel #sdk-tools:

@Devin I need you to make the same type of change across three different repositories -- erlang-server-sdk, flutter-client-sdk, and the cpp-sdks repository.

When we generate custom events (and migration op events if supported), we JSON encode the event to only include the context keys, not the full context. Please update those repositories so that custom and migration events include the full context.

For reference, you can look at the follow commits. This is prior work where we inlined the context for other events in those repositories.

erlang: e5c6cc4da4cd6dc59326bb5fa0b0a75864c72bc7
flutter: b2ebcbfd8c3c817595821f91ffb7ac02687bb41f
cpp: bc77e89d1bf5b2294e2b384363b32734fd1f75db

You can also look at <https://github.com/launchdarkly/ruby-server-sdk/commit/ab5b14fff2f04f1dfa7e6f61669bc348fffb5cac> which shows how we inlined contexts for custom and migration events in ruby.

When doing this work, please make sure to update all the unit tests. Also check for any code that is now unused as a result of the change.

In the contract tests for these repositories, replace the inline-context capability with inline-context-all.

When making the PRs, name as follows:

- erlang :: mk/sdk-1156/inline-contexts
- flutter :: mk/sdk-1165/inline-contexts
- cpp :: mk/sdk-1157/inline-contexts

Copy link
Contributor Author

🤖 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

@keelerm84 keelerm84 changed the title Include full context in custom events and update to inline-context-all capability feat: Inline context for custom events May 8, 2025
@keelerm84 keelerm84 merged commit 70a9ead into main May 9, 2025
3 checks passed
@keelerm84 keelerm84 deleted the mk/sdk-1165/inline-contexts branch May 9, 2025 16:36
keelerm84 pushed a commit that referenced this pull request May 9, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.6.0](launchdarkly_dart_common-v1.5.0...launchdarkly_dart_common-v1.6.0)
(2025-05-09)


### Features

* Inline context for custom events
([#199](#199))
([70a9ead](70a9ead))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants