[FME-9594] fix build, add github workflow, add evaluationWithDetails

Author: ZamoraEmmanuel

.github/workflows/ci-cd.yml

@@ -0,0 +1,49 @@
+name: ci-cd
+on:
+  pull_request:
+    branches:
+      - master
+      - development
+  push:
+    branches:
+      - '*'
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event_name == 'push' && github.run_number || github.event.pull_request.number }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+  id-token: write
+
+jobs:
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 'lts/*'
+          cache: 'npm'
+
+      - name: npm ci
+        run: npm ci
+
+      - name: npm ts tests
+        run: npm run test-ts-decls
+
+      - name: npm check
+        run: npm run check
+
+      - name: npm test-browser
+        run: npm run test-browser
+
+      - name: npm test-node
+        run: npm run test-node
+
+      - name: npm build
+        run: BUILD_BRANCH=$(echo "${GITHUB_REF#refs/heads/}") npm run build

scripts/build_esm_replace_imports.sh

@@ -3,6 +3,9 @@
 # replace splitio-commons imports to use ES modules
 replace '@splitsoftware/splitio-commons/src' '@splitsoftware/splitio-commons/esm' ./es -r
 
+# Fix import extension in es/index.js
+sed -i '' -e "s|from './lib/js-split-provider'|from './lib/js-split-provider.js'|" es/index.js
+
 if [ $? -eq 0 ]
 then
   exit 0

src/__tests__/nodeSuites/client.spec.js

@@ -47,11 +47,13 @@ export default async function(assert) {
   };
 
   const getBooleanSplitWithKeyTest = async (client) => {
-    let result = await client.getBooleanValue('my_feature', false);
-    assert.equals(result, true);
+    let result = await client.getBooleanDetails('my_feature', false);
+    assert.equals(result.value, true);
+    assert.looseEquals(result.flagMetadata, { desc: 'this applies only to ON treatment' });
 
-    result = await client.getBooleanValue('my_feature', true, { targetingKey: 'randomKey' });
-    assert.equals(result, false);
+    result = await client.getBooleanDetails('my_feature', true, { targetingKey: 'randomKey' });
+    assert.equals(result.value, false);
+    assert.looseEquals(result.flagMetadata, {});
   };
 
   const getStringSplitTest = async (client) => {

src/index.ts

@@ -1 +1 @@
-export * from './lib/js-split-provider.js';
+export * from './lib/js-split-provider';

src/lib/js-split-provider.ts

@@ -20,6 +20,7 @@ type Consumer = {
 };
 
 const CONTROL_VALUE_ERROR_MESSAGE = "Received the 'control' value from Split.";
+const CONTROL_TREATMENT = "control";
 
 export class OpenFeatureSplitProvider implements Provider {
   metadata = {
@@ -53,32 +54,15 @@ export class OpenFeatureSplitProvider implements Provider {
       this.transformContext(context)
     );
 
-    let value: boolean;
-    switch (details.value as unknown) {
-      case "on":
-        value = true;
-        break;
-      case "off":
-        value = false;
-        break;
-      case "true":
-        value = true;
-        break;
-      case "false":
-        value = false;
-        break;
-      case true:
-        value = true;
-        break;
-      case false:
-        value = false;
-        break;
-      case "control":
-        throw new FlagNotFoundError(CONTROL_VALUE_ERROR_MESSAGE);
-      default:
-        throw new ParseError(`Invalid boolean value for ${details.value}`);
+    if ( details.value === "on" || details.value === "true" ) {
+      return { ...details, value: true };
+    }
+
+    if ( details.value === "off" || details.value === "false" ) {
+      return { ...details, value: false };
     }
-    return { ...details, value };
+
+    throw new ParseError(`Invalid boolean value for ${details.value}`);
   }
 
   async resolveStringEvaluation(
@@ -90,9 +74,6 @@ export class OpenFeatureSplitProvider implements Provider {
       flagKey,
       this.transformContext(context)
     );
-    if (details.value === "control") {
-      throw new FlagNotFoundError(CONTROL_VALUE_ERROR_MESSAGE);
-    }
     return details;
   }
 
@@ -130,14 +111,19 @@ export class OpenFeatureSplitProvider implements Provider {
       );
     } else {
       await this.initialized;
-      const value = this.client.getTreatment(
+      const {treatment: value, config}: SplitIO.TreatmentWithConfig = this.client.getTreatmentWithConfig(
         consumer.key,
         flagKey,
         consumer.attributes
       );
+      if (value === CONTROL_TREATMENT) {
+        throw new FlagNotFoundError(CONTROL_VALUE_ERROR_MESSAGE);
+      }
+      const flagMetadata = config ? JSON.parse(config) : undefined;
       const details: ResolutionDetails<string> = {
         value: value,
         variant: value,
+        flagMetadata: flagMetadata,
         reason: StandardResolutionReasons.TARGETING_MATCH,
       };
       return details;