Merge main

Signed-off-by: marcozabel <[email protected]>

Author: marcozabel

.release-please-manifest.json

@@ -8,9 +8,9 @@
   "libs/providers/launchdarkly-client": "0.3.3",
   "libs/providers/go-feature-flag-web": "0.2.6",
   "libs/shared/flagd-core": "1.1.0",
-  "libs/shared/ofrep-core": "1.2.0",
+  "libs/shared/ofrep-core": "2.0.0",
   "libs/providers/ofrep": "0.2.1",
-  "libs/providers/ofrep-web": "0.3.4",
+  "libs/providers/ofrep-web": "0.3.5",
   "libs/providers/flipt": "0.1.3",
   "libs/providers/flagsmith-client": "0.1.3",
   "libs/providers/flipt-web": "0.1.5",

libs/providers/flagd/src/e2e/step-definitions/providerSteps.ts

@@ -71,6 +71,21 @@ export const providerSteps: Steps =
       state.providerType = providerType;
     });
 
+    function mapProviderState(state: string): ProviderStatus {
+      const mappedState = state.toUpperCase().replace('-', '_');
+      const status = Object.values(ProviderStatus).find((s) => s === mappedState);
+
+      if (!status) {
+        throw new Error(`Unknown provider status: ${state}`);
+      }
+
+      return status;
+    }
+
+    then(/^the client should be in (.*) state/, (providerState: string) => {
+      expect(state.client?.providerStatus).toBe(mapProviderState(providerState));
+    });
+
     when(/^the connection is lost for (\d+)s$/, async (time) => {
       console.log('stopping flagd');
       await fetch('http://' + container.getLaunchpadUrl() + '/restart?seconds=' + time);
@@ -79,26 +94,4 @@ export const providerSteps: Steps =
     when('the flag was modified', async () => {
       await fetch('http://' + container.getLaunchpadUrl() + '/change');
     });
-
-    function mapProviderState(state: string): ProviderStatus {
-      switch (state) {
-        case 'fatal':
-          return ProviderStatus.FATAL;
-        case 'error':
-          return ProviderStatus.ERROR;
-        case 'ready':
-          return ProviderStatus.READY;
-        case 'stale':
-          return ProviderStatus.STALE;
-        case 'not-ready':
-          return ProviderStatus.NOT_READY;
-
-        default:
-          throw new Error('Unknown provider status');
-      }
-    }
-
-    and(/^the client should be in (.*) state/, (providerState: string) => {
-      expect(state.client?.providerStatus).toBe(mapProviderState(providerState));
-    });
   };

libs/providers/flagd/src/e2e/tests/rpc.spec.ts

@@ -23,7 +23,7 @@ describe('rpc', () => {
       tagFilter:
         // remove filters as we add support for features
         // see: https://github.com/open-feature/js-sdk-contrib/issues/1096 and child issues
-        '@rpc and not @targetURI and not @customCert and not @events and not @stream and not @grace and not @metadata and not @contextEnrichment and not @caching',
+        '@rpc and not @targetURI and not @customCert and not @forbidden and not @events and not @stream and not @grace and not @metadata and not @contextEnrichment and not @caching',
       scenarioNameTemplate: (vars) => {
         return `${vars.scenarioTitle} (${vars.scenarioTags.join(',')} ${vars.featureTags.join(',')})`;
       },

libs/providers/flagd/src/lib/configuration.spec.ts

@@ -77,6 +77,20 @@ describe('Configuration', () => {
     expect(getConfig().contextEnricher({})).toStrictEqual({});
   });
 
+  it('should use flagd sync port over flagd port environment option', () => {
+    const port = 8080;
+    const syncPort = 9090;
+
+    process.env['FLAGD_PORT'] = `${port}`;
+    process.env['FLAGD_SYNC_PORT'] = `${syncPort}`;
+
+    expect(getConfig()).toStrictEqual(
+      expect.objectContaining({
+        port: syncPort,
+      }),
+    );
+  });
+
   it('should use incoming options over defaults and environment variable', () => {
     const contextEnricher = (syncContext: EvaluationContext | null): EvaluationContext => {
       return { ...syncContext, extraKey: 'extraValue' };
@@ -96,6 +110,7 @@ describe('Configuration', () => {
 
     process.env['FLAGD_HOST'] = 'override';
     process.env['FLAGD_PORT'] = '8080';
+    process.env['FLAGD_SYNC_PORT'] = '9090';
     process.env['FLAGD_TLS'] = 'false';
     process.env['FLAGD_DEFAULT_AUTHORITY'] = 'test-authority-override';
 
@@ -107,6 +122,11 @@ describe('Configuration', () => {
     expect(getConfig()).toStrictEqual(expect.objectContaining({ port: 8013 }));
   });
 
+  it('should ignore an invalid sync port set as an environment variable', () => {
+    process.env['FLAGD_SYNC_PORT'] = 'invalid number';
+    expect(getConfig()).toStrictEqual(expect.objectContaining({ port: 8013 }));
+  });
+
   describe('port handling', () => {
     describe('for "in-process" evaluation', () => {
       const resolverType = 'in-process';

libs/providers/flagd/src/lib/configuration.ts

@@ -116,6 +116,7 @@ const DEFAULT_IN_PROCESS_CONFIG: FlagdConfig = { ...DEFAULT_CONFIG, resolverType
 enum ENV_VAR {
   FLAGD_HOST = 'FLAGD_HOST',
   FLAGD_PORT = 'FLAGD_PORT',
+  FLAGD_SYNC_PORT = 'FLAGD_SYNC_PORT',
   FLAGD_DEADLINE_MS = 'FLAGD_DEADLINE_MS',
   FLAGD_TLS = 'FLAGD_TLS',
   FLAGD_SOCKET_PATH = 'FLAGD_SOCKET_PATH',
@@ -152,6 +153,9 @@ const getEnvVarConfig = (): Partial<Config> => {
     ...(Number(process.env[ENV_VAR.FLAGD_PORT]) && {
       port: Number(process.env[ENV_VAR.FLAGD_PORT]),
     }),
+    ...(Number(process.env[ENV_VAR.FLAGD_SYNC_PORT]) && {
+      port: Number(process.env[ENV_VAR.FLAGD_SYNC_PORT]),
+    }),
     ...(Number(process.env[ENV_VAR.FLAGD_DEADLINE_MS]) && {
       deadlineMs: Number(process.env[ENV_VAR.FLAGD_DEADLINE_MS]),
     }),

libs/providers/ofrep-web/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.3.5](https://github.com/open-feature/js-sdk-contrib/compare/ofrep-web-provider-v0.3.4...ofrep-web-provider-v0.3.5) (2025-11-24)
+
+
+### 🐛 Bug Fixes
+
+* use ofrep-core ^2.0.0 for ofrep providers ([#1411](https://github.com/open-feature/js-sdk-contrib/issues/1411)) ([c105aa3](https://github.com/open-feature/js-sdk-contrib/commit/c105aa3eb3b59b490c0fe7445f31e6e5e2a2f6ee))
+
 ## [0.3.4](https://github.com/open-feature/js-sdk-contrib/compare/ofrep-web-provider-v0.3.3...ofrep-web-provider-v0.3.4) (2025-11-19)
 
 

libs/providers/ofrep-web/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openfeature/ofrep-web-provider",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "license": "Apache-2.0",
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
@@ -12,6 +12,6 @@
     "@openfeature/web-sdk": "^1.4.0"
   },
   "dependencies": {
-    "@openfeature/ofrep-core": "^1.0.0"
+    "@openfeature/ofrep-core": "^2.0.0"
   }
 }

libs/providers/ofrep/package.json

@@ -12,6 +12,6 @@
     "@openfeature/server-sdk": "^1.6.0"
   },
   "dependencies": {
-    "@openfeature/ofrep-core": "^1.0.0"
+    "@openfeature/ofrep-core": "^2.0.0"
   }
 }

libs/shared/ofrep-core/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [2.0.0](https://github.com/open-feature/js-sdk-contrib/compare/ofrep-core-v1.2.0...ofrep-core-v2.0.0) (2025-11-24)
+
+
+### ⚠ BREAKING CHANGES
+
+* release 2.0.0 ([#1409](https://github.com/open-feature/js-sdk-contrib/issues/1409))
+
+### chore
+
+* release 2.0.0 ([#1409](https://github.com/open-feature/js-sdk-contrib/issues/1409)) ([0546d3f](https://github.com/open-feature/js-sdk-contrib/commit/0546d3f9c5d0ccf337635fd41a56d26cadad6929))
+
 ## [1.2.0](https://github.com/open-feature/js-sdk-contrib/compare/ofrep-core-v1.1.0...ofrep-core-v1.2.0) (2025-11-19)
 
 

libs/shared/ofrep-core/README.md

@@ -1,4 +1,4 @@
 # ofrep-core
 
 The core implementation of OFREP core providers.
-This package is intended to be used by the concrete OFREP provider implementations for API access, error handling, ... 
+This package is intended to be only used by the concrete OFREP provider implementations for API access, error handling, ...