feat(audience): MessageBuilder — wire-format message construction (SDK-129)#688
Merged
Conversation
nattb8
force-pushed
the
feat/sdk-129-message-builder
branch
from
ca28579 to
6deae7a
Compare
ImmutableJeffrey
previously approved these changes
Apr 16, 2026
ImmutableJeffrey
force-pushed
the
feat/sdk-129-message-builder
branch
from
6deae7a to
e88c9cb
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e88c9cb. Configure here.
ImmutableJeffrey
previously approved these changes
Apr 17, 2026
Addresses cursor[bot] review comment: every other user-supplied string field in MessageBuilder (eventName, anonymousId, userId, fromId, toId, libraryVersion) is truncated to 256 chars per the backend schema. The three identity-type fields were unintentionally exempt. While the plan's IdentityType values are fixed strings that fit within 256 chars, applying Truncate consistently matches the stated behavior and prevents future surprise if callers pass arbitrary type strings. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ImmutableJeffrey
approved these changes
Apr 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
MessageBuilder.cstoRuntime/Events/— builds wire-formatDictionary<string, object>payloads matching backend OAS schematrack,identify, andaliasmessage typesmessageId,eventTimestamp,context,surfaceTest plan
Linear: SDK-129
🤖 Generated with Claude Code
Note
Low Risk
Adds new isolated message-construction utility and tests without modifying existing runtime behavior; risk is mainly schema/field-mapping correctness.
Overview
Adds a new internal
MessageBuilderthat constructs wire-formatDictionary<string, object>payloads fortrack,identify, andaliasevents, including shared metadata (messageId,eventTimestamp,context.library/libraryVersion, andsurface).Applies 256-character truncation to key string fields and omits optional identifiers/traits/properties when null or empty. Adds NUnit coverage to verify required fields, truncation behavior, and consistent
context/surfaceacross message types.Reviewed by Cursor Bugbot for commit 415a800. Bugbot is set up for automated code reviews on this repo. Configure here.