Merge pull request #162 from Flagsmith/feat/v4

feat!: Custom fetch support, remove node-fetch, ESM+CJS dual build, migrate to vitest, TS fixes, test improvements

Author: rolodato

.github/workflows/publish.yml

@@ -14,9 +14,9 @@ jobs:
       - name: Cloning repo
         uses: actions/checkout@v3
 
-      - uses: actions/setup-node@v3
+      - uses: actions/setup-node@v4
         with:
-          node-version: '16.x'
+          node-version: '18.x'
           registry-url: 'https://registry.npmjs.org'
 
       - run: npm ci

.github/workflows/pull_request.yaml

@@ -23,9 +23,9 @@ jobs:
           auth_header="$(git config --local --get http.https://github.com/.extraheader)"
           git submodule sync --recursive
           git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1
-      - uses: actions/setup-node@v1
+      - uses: actions/setup-node@v4
         with:
-          node-version: "15.x"
+          node-version: "18.x"
       - name: cache node modules
         uses: actions/cache@v1
         with:
@@ -34,8 +34,7 @@ jobs:
           restore-keys: |
             npm-${{ hashFiles('package-lock.json') }}
             npm-
-      - run: npm i -g [email protected]
-      - run: npm install
+      - run: npm ci
       - run: npm test
         env:
           CI: true

.gitignore

@@ -8,5 +8,6 @@
 
 node_modules/
 build/
+coverage/
 
 .tool-versions

flagsmith-engine/environments/models.ts

@@ -1,6 +1,6 @@
-import { FeatureStateModel } from '../features/models';
-import { IdentityModel } from '../identities/models';
-import { ProjectModel } from '../projects/models';
+import { FeatureStateModel } from '../features/models.js';
+import { IdentityModel } from '../identities/models.js';
+import { ProjectModel } from '../projects/models.js';
 
 export class EnvironmentAPIKeyModel {
     id: number;

flagsmith-engine/environments/util.ts

@@ -1,7 +1,7 @@
-import { buildFeatureStateModel } from '../features/util';
-import { buildIdentityModel } from '../identities/util';
-import { buildProjectModel } from '../projects/util';
-import { EnvironmentAPIKeyModel, EnvironmentModel } from './models';
+import { buildFeatureStateModel } from '../features/util.js';
+import { buildIdentityModel } from '../identities/util.js';
+import { buildProjectModel } from '../projects/util.js';
+import { EnvironmentAPIKeyModel, EnvironmentModel } from './models.js';
 
 export function buildEnvironmentModel(environmentJSON: any) {
     const project = buildProjectModel(environmentJSON.project);

flagsmith-engine/features/models.ts

@@ -1,5 +1,5 @@
 import { v4 as uuidv4 } from 'uuid';
-import { getHashedPercentateForObjIds } from '../utils/hashing';
+import { getHashedPercentateForObjIds } from '../utils/hashing/index.js';
 
 export class FeatureModel {
     id: number;

flagsmith-engine/features/util.ts

@@ -4,7 +4,7 @@ import {
     FeatureStateModel,
     MultivariateFeatureOptionModel,
     MultivariateFeatureStateValueModel
-} from './models';
+} from './models.js';
 
 export function buildFeatureModel(featuresModelJSON: any): FeatureModel {
     return new FeatureModel(featuresModelJSON.id, featuresModelJSON.name, featuresModelJSON.type);

flagsmith-engine/identities/models.ts

@@ -1,6 +1,5 @@
-import { FeatureStateModel } from '../features/models';
-import { IdentityFeaturesList } from '../utils/collections';
-import { TraitModel } from './traits/models';
+import { IdentityFeaturesList } from '../utils/collections.js';
+import { TraitModel } from './traits/models.js';
 
 const { v4: uuidv4 } = require('uuid');
 

flagsmith-engine/identities/util.ts

@@ -1,7 +1,7 @@
-import { buildFeatureStateModel } from '../features/util';
-import { IdentityFeaturesList } from '../utils/collections';
-import { IdentityModel } from './models';
-import { TraitModel } from './traits/models';
+import { buildFeatureStateModel } from '../features/util.js';
+import { IdentityFeaturesList } from '../utils/collections.js';
+import { IdentityModel } from './models.js';
+import { TraitModel } from './traits/models.js';
 
 export function buildTraitModel(traitJSON: any): TraitModel {
     return new TraitModel(traitJSON.trait_key, traitJSON.trait_value);

flagsmith-engine/index.ts

@@ -1,17 +1,17 @@
-import { EnvironmentModel } from './environments/models';
-import { FeatureStateModel } from './features/models';
-import { IdentityModel } from './identities/models';
-import { TraitModel } from './identities/traits/models';
-import { getIdentitySegments } from './segments/evaluators';
-import { SegmentModel } from './segments/models';
-import { FeatureStateNotFound } from './utils/errors';
+import { EnvironmentModel } from './environments/models.js';
+import { FeatureStateModel } from './features/models.js';
+import { IdentityModel } from './identities/models.js';
+import { TraitModel } from './identities/traits/models.js';
+import { getIdentitySegments } from './segments/evaluators.js';
+import { SegmentModel } from './segments/models.js';
+import { FeatureStateNotFound } from './utils/errors.js';
 
-export { EnvironmentModel } from './environments/models';
-export { FeatureStateModel } from './features/models';
-export { IdentityModel } from './identities/models';
-export { TraitModel } from './identities/traits/models';
-export { SegmentModel } from './segments/models';
-export { OrganisationModel } from './organisations/models';
+export { EnvironmentModel } from './environments/models.js';
+export { FeatureStateModel } from './features/models.js';
+export { IdentityModel } from './identities/models.js';
+export { TraitModel } from './identities/traits/models.js';
+export { SegmentModel } from './segments/models.js';
+export { OrganisationModel } from './organisations/models.js';
 
 function getIdentityFeatureStatesDict(
     environment: EnvironmentModel,