Merge pull request #458 from splitio/fme-11995

Handle nullish prerequisites

Author: ZamoraEmmanuel

.nvmrc

@@ -1 +1 @@
-lts/*
+22

CHANGES.txt

@@ -1,3 +1,6 @@
+2.10.1 (December 18, 2025)
+ - Bugfix - Handle `null` prerequisites properly
+
 2.10.0 (December 16, 2025)
  - Added property `impressionsDisabled` in getTreatment(s) `evaluationOptions` parameter, to disable impressions per evaluations.
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-commons",
-  "version": "2.10.0",
+  "version": "2.10.1",
   "description": "Split JavaScript SDK common components",
   "main": "cjs/index.js",
   "module": "esm/index.js",

package.json

@@ -220,7 +220,7 @@ export interface ISplit {
   changeNumber: number,
   status: 'ACTIVE' | 'ARCHIVED',
   conditions: ISplitCondition[],
-  prerequisites?: {
+  prerequisites?: null | {
     n: string,
     ts: string[]
   }[]

src/dtos/types.ts

@@ -73,6 +73,9 @@ test('MATCHER PREREQUISITES / Edge cases', () => {
   const matcherTrueNoPrerequisites = prerequisitesMatcherContext(undefined, mockStorage, loggerMock);
   expect(matcherTrueNoPrerequisites({ key: 'a-key' }, evaluateFeature)).toBe(true);
 
+  const matcherTruePrerequisitesNull = prerequisitesMatcherContext(null, mockStorage, loggerMock);
+  expect(matcherTruePrerequisitesNull({ key: 'a-key' }, evaluateFeature)).toBe(true);
+
   const matcherTrueEmptyPrerequisites = prerequisitesMatcherContext([], mockStorage, loggerMock);
   expect(matcherTrueEmptyPrerequisites({ key: 'a-key' }, evaluateFeature)).toBe(true);
 

src/evaluator/matchers/__tests__/prerequisites.spec.ts

@@ -4,10 +4,12 @@ import { ILogger } from '../../logger/types';
 import { thenable } from '../../utils/promise/thenable';
 import { IDependencyMatcherValue, ISplitEvaluator } from '../types';
 
-export function prerequisitesMatcherContext(prerequisites: ISplit['prerequisites'] = [], storage: IStorageSync | IStorageAsync, log: ILogger) {
+export function prerequisitesMatcherContext(prerequisites: ISplit['prerequisites'], storage: IStorageSync | IStorageAsync, log: ILogger) {
 
   return function prerequisitesMatcher({ key, attributes }: IDependencyMatcherValue, splitEvaluator: ISplitEvaluator): MaybeThenable<boolean> {
 
+    prerequisites = prerequisites == null ? [] : prerequisites;
+
     function evaluatePrerequisite(prerequisite: { n: string; ts: string[] }): MaybeThenable<boolean> {
       const evaluation = splitEvaluator(log, key, prerequisite.n, attributes, storage);
       return thenable(evaluation) ?