[confidence-agent]

Event publish requests from the JS SDK did not include an sdk field, so epx-events tagged them as SDK_ID_UNSPECIFIED. This change makes EventSenderEngine send the SDK's identity with every publish request.

• Confidence.create passes the existing sdk object (SDK_ID_JS_CONFIDENCE plus the version maintained by release-please) to EventSenderEngine. The flag resolver already receives the same object, so no second version string is added.
• EventSenderEngine now requires an internal sdk: Sdk option, using the generated resolver type, and stores it on each EventBatch.
• Both send paths serialize it with the generated Sdk.toJSON, which produces {"id":"SDK_ID_JS_CONFIDENCE","version":"…"}. This is the same proto-JSON enum-name form the resolver requests use.
  • Normal fetch flush, including keepalive and the fallback-to-fetch path, through upload.
  • sendBeacon on page unload, through beaconFlush.
• Batching, the sendTime refresh on retry, custom event URLs, rate limiting, keepalive, the beacon size cap and queue behavior are unchanged.

Decisions:

• sdk is required on the internal engine options rather than optional. This way the compiler flags any construction that leaves it out. EventSenderEngine is internal, so the public API does not change.
• This only fixes the JS instrumentation. The backend still treats sdk as optional, and unspecified traffic from other clients is outside this change.

Tests:

• EventSenderEngine.test.ts: the upload batch includes sdk. The serialized fetch body has sdk: {id:'SDK_ID_JS_CONFIDENCE', version}.
• EventSenderEngine.visibilitychange.test.ts: the beacon payload includes the serialized sdk. When the beacon is rejected, the fallback upload batch includes sdk.
• Confidence.int.test.ts: an event published through Confidence.create reaches the publish endpoint with sdk.id = 'SDK_ID_JS_CONFIDENCE' and a semver version.

Validation:

• yarn jest --selectProjects sdk --testPathIgnorePatterns e2e: 208/208 passed.
• yarn build, yarn lint and yarn prettier -c packages/sdk/src: all passed.
• *.e2e.test.ts files were not run. They need live backend credentials and fail in this sandbox both with and without this change.
• Running jest from the repo root, the openfeature provider suites fail to type-check in this environment both with and without this change.

Source: https://app.confidence.spotify.com/chats/ibdi0t34ewzwpbjwpcz7

---

Opened by the Confidence coding agent.