Fix CHOICES_SEARCH_KMP not being set was breaking e2e tests

public/types/src/scripts/interfaces/build-flags.d.ts

@@ -1,11 +1,11 @@
 export declare const canUseDom: boolean;
 export declare const searchFuse: string | undefined;
-export declare const searchKMP: string | undefined;
+export declare const searchKMP: boolean;
 /**
  * These are not directly used, as an exported object (even as const) will prevent tree-shake away code paths
  */
 export declare const BuildFlags: {
     readonly searchFuse: string | undefined;
-    readonly searchKMP: string | undefined;
+    readonly searchKMP: boolean;
     readonly canUseDom: boolean;
 };

public/types/src/scripts/search/kmp.d.ts

@@ -0,0 +1,11 @@
+import { Options } from '../interfaces';
+import { Searcher, SearchResult } from '../interfaces/search';
+export declare class SearchByKMP<T extends object> implements Searcher<T> {
+    _fields: string[];
+    _haystack: T[];
+    constructor(config: Options);
+    index(data: T[]): void;
+    reset(): void;
+    isEmptyIndex(): boolean;
+    search(_needle: string): SearchResult<T>[];
+}

scripts/rollup.config.mjs

@@ -11,6 +11,7 @@ const pckg = require('../package.json');
 
 const buildFeatures = {
   CHOICES_SEARCH_FUSE: "full", // "basic" / "null"
+  CHOICES_SEARCH_KMP: "0", // "1"
   CHOICES_CAN_USE_DOM: "1", // "0"
 }
 

src/scripts/interfaces/build-flags.ts

@@ -4,7 +4,7 @@ export const canUseDom: boolean =
     : !!(typeof document !== 'undefined' && document.createElement);
 
 export const searchFuse: string | undefined = process.env.CHOICES_SEARCH_FUSE;
-export const searchKMP: string | undefined = process.env.CHOICES_SEARCH_KMP;
+export const searchKMP: boolean = process.env.CHOICES_SEARCH_KMP === '1';
 
 /**
  * These are not directly used, as an exported object (even as const) will prevent tree-shake away code paths

src/scripts/search/index.ts

@@ -6,7 +6,7 @@ import { SearchByKMP } from './kmp';
 import { searchFuse, searchKMP } from '../interfaces/build-flags';
 
 export function getSearcher<T extends object>(config: Options): Searcher<T> {
-  if (searchFuse) {
+  if (searchFuse && !searchKMP) {
     return new SearchByFuse<T>(config);
   }
   if (searchKMP) {