Commit 4525cdc
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?v=4&size=40){kind=avatar}
chore: add missing features to .sdk_metadata.json (#1190)
**Requirements**
- [x] I have added test coverage for new or changed functionality (N/A —
metadata-only change)
- [x] I have followed the repository's [pull request submission
guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests)
- [x] I have validated my changes against all supported platform
versions (N/A — metadata-only change)
**Related issues**
Cross-referenced against
[sdk-meta/feature_info.json](https://github.com/launchdarkly/sdk-meta/blob/main/products/feature_info.json).
**Describe the solution you've provided**
Adds 7 missing feature entries to `.sdk_metadata.json` that were
identified by comparing changelogs, source code, and dependency chains
against the canonical feature list in `sdk-meta/feature_info.json`.
**Feature Correlation Table**
| SDK | Feature Added | Introduced | Evidence |
|---|---|---|---|
| browser | `perContextSummaryEvents` | 0.7 |
[CHANGELOG](packages/sdk/browser/CHANGELOG.md) — js-client-sdk v0.7.0
added per-context summary events
([#859](#859)).
`createEventProcessor.ts` passes `summariesPerContext: true`. |
| combined-browser | `perContextSummaryEvents` | 0.1 | First release
(0.1.0) depended on js-client-sdk ≥ 0.12.0, which already includes the
feature from 0.7.0. |
| react-native | `perContextSummaryEvents` | 10.10.1 |
[CHANGELOG](packages/sdk/react-native/CHANGELOG.md) — v10.10.1 bumped
`js-client-sdk-common` from 1.13.0 → 1.14.0, which contains the
per-context summary feature. |
| node-server | `pollingResponseCompression` | 9.2 |
[CHANGELOG](packages/sdk/server-node/CHANGELOG.md) — v9.2.0: "Implement
handling for gzip compressed responses"
([#367](#367)).
`NodeRequests.ts` sets `accept-encoding: gzip` on GET requests. |
| node-server | `storingDataDynamodb` | 5.6 | Legacy
[node-server-sdk-dynamodb](https://github.com/launchdarkly/node-server-sdk-dynamodb)
v1.0.0 was released 2018-11-15, when the [legacy Node.js Server
SDK](https://github.com/launchdarkly/node-server-sdk) was at v5.6.x
(confirmed via git tag dates). |
| server-ai | `aiAgentConfig` | 0.11 |
[CHANGELOG](packages/sdk/server-ai/CHANGELOG.md) — v0.11.0: "Adding
agent support for AI Configs"
([#893](#893)).
`agentConfig()` method in `LDAIClientImpl.ts`. |
| server-ai | `aiJudgeConfig` | 0.14 |
[CHANGELOG](packages/sdk/server-ai/CHANGELOG.md) — v0.14.0: "Added
judgeConfig method", "Added createJudge method"
([#969](#969)).
`judgeConfig()` and `createJudge()` in `LDAIClientImpl.ts`. |
**Items for reviewer attention**
- **`storingDataDynamodb` version**: Listed as `5.6` based on
cross-referencing the legacy
[node-server-sdk](https://github.com/launchdarkly/node-server-sdk)
repo's git tag dates with the [DynamoDB store's
changelog](https://github.com/launchdarkly/node-server-sdk-dynamodb).
The DynamoDB store v1.0.0 was released 2018-11-15; the Node SDK tags
`5.6.0`–`5.6.2` land on 2018-11-14/15. Please verify this aligns with
the intended convention (consistent with `storingDataRedis: 2.0`).
- **Completeness**: Edge SDKs were evaluated but not updated. They
inherit `variationDetail()`, `track()`, etc. from `LDClientImpl` via
server-common, but their public options are restricted (e.g.,
edge-common's `validateOptions` only allows `logger`/`sendEvents`).
Features requiring config options that aren't exposed were not added.
**Describe alternatives you've considered**
Considered also adding `variationDetail` and other inherited
method-level features to edge SDKs, but decided against it since their
public configuration surfaces are intentionally restricted and the
metadata should reflect what's user-accessible, not just what code
exists in a dependency.
**Additional context**
All feature entries maintain alphabetical ordering within each SDK's
feature list, consistent with the existing file convention.
Link to Devin session:
https://app.devin.ai/sessions/42f7efc8482b4c2cbaa90fe04d8c40ff
Requested by: @kinyoklion
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> Low risk metadata-only change that updates `.sdk_metadata.json`
without altering runtime behavior. Main risk is incorrect
feature/version attribution affecting downstream reporting or docs.
>
> **Overview**
> Updates `.sdk_metadata.json` to fill in missing feature flags and
introduction versions across several SDKs.
>
> Adds `perContextSummaryEvents` for `browser`, `combined-browser`, and
`react-native`; adds `pollingResponseCompression` and
`storingDataDynamodb` for `node-server`; and adds `aiAgentConfig` and
`aiJudgeConfig` for `server-ai`.
>
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
54b7689. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>1 parent 5defaa7 commit 4525cdc
1 file changed
Lines changed: 7 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
| 59 | + | |
59 | 60 | | |
60 | 61 | | |
61 | 62 | | |
| |||
101 | 102 | | |
102 | 103 | | |
103 | 104 | | |
| 105 | + | |
104 | 106 | | |
105 | 107 | | |
106 | 108 | | |
| |||
148 | 150 | | |
149 | 151 | | |
150 | 152 | | |
| 153 | + | |
151 | 154 | | |
152 | 155 | | |
153 | 156 | | |
| |||
180 | 183 | | |
181 | 184 | | |
182 | 185 | | |
| 186 | + | |
183 | 187 | | |
184 | 188 | | |
185 | 189 | | |
186 | 190 | | |
| 191 | + | |
187 | 192 | | |
188 | 193 | | |
189 | 194 | | |
| |||
214 | 219 | | |
215 | 220 | | |
216 | 221 | | |
| 222 | + | |
217 | 223 | | |
| 224 | + | |
218 | 225 | | |
219 | 226 | | |
220 | 227 | | |
| |||
0 commit comments