feat: Always inline contexts for feature events (#351)

launchdarkly · Feb 6, 2024 · 381f008 · 381f008
1 parent 2c6add5
commit 381f008
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 18 deletions.
diff --git a/contract-tests/index.js b/contract-tests/index.js
@@ -31,6 +31,7 @@ app.get('/', (req, res) => {
       'migrations',
       'event-sampling',
       'strongly-typed',
+      'inline-context',
     ],
   });
 });

diff --git a/packages/shared/common/__tests__/internal/events/EventProcessor.test.ts b/packages/shared/common/__tests__/internal/events/EventProcessor.test.ts
@@ -73,19 +73,11 @@ function makeFeatureEvent(
     variation,
     value,
     default: def,
-    ...(debug
-      ? {
-          context: {
-            key: 'userKey',
-            name: 'Red',
-            kind: 'user',
-          },
-        }
-      : {
-          contextKeys: {
-            user: 'userKey',
-          },
-        }),
+    context: {
+      key: 'userKey',
+      name: 'Red',
+      kind: 'user',
+    },
   };
 }
 

diff --git a/packages/shared/common/src/internal/events/EventProcessor.ts b/packages/shared/common/src/internal/events/EventProcessor.ts
@@ -261,6 +261,7 @@ export default class EventProcessor implements LDEventProcessor {
         const out: FeatureOutputEvent = {
           kind: debug ? 'debug' : 'feature',
           creationDate: event.creationDate,
+          context: this.contextFilter.filter(event.context),
           key: event.key,
           value: event.value,
           default: event.default,
@@ -280,11 +281,6 @@ export default class EventProcessor implements LDEventProcessor {
         if (event.reason) {
           out.reason = event.reason;
         }
-        if (debug) {
-          out.context = this.contextFilter.filter(event.context);
-        } else {
-          out.contextKeys = event.context.kindsAndKeys;
-        }
         return out;
       }
       case 'index': // Intentional fallthrough.