Remove cache

Author: sylhare

package.json

@@ -1,14 +1,15 @@
 {
   "name": "simple-jekyll-search",
-  "version": "1.15.1",
-  "description": "Fork of Simple Jekyll Search from https://github.com/christian-fei/Simple-Jekyll-Search",
+  "version": "2.0.0",
+  "description": "A simple JavaScript library to add search functionality to any Jekyll blog - Fast, lightweight, client-side search. Fork of Simple Jekyll Search from https://github.com/christian-fei/Simple-Jekyll-Search",
   "main": "dest/simple-jekyll-search.js",
   "type": "module",
   "scripts": {
     "cypress": "cypress",
     "cypress:ci": "node scripts/start-jekyll.js & sleep 5 && cypress run",
     "cypress:run": "node scripts/start-jekyll.js && sleep 8 && cypress run; EXIT_CODE=$?; node scripts/kill-jekyll.js; exit $EXIT_CODE",
     "lint": "eslint . --ext .ts",
+    "lint:fix": "eslint . --ext .ts --fix",
     "pretest": "yarn run lint",
     "build": "tsc && vite build && terser dest/simple-jekyll-search.js -o dest/simple-jekyll-search.min.js",
     "prebuild": "yarn run test",

src/SearchStrategies/SearchCache.ts

@@ -24,6 +24,8 @@ export function findLiteralMatches(text: string, criteria: string): MatchInfo[]
   const matches: MatchInfo[] = [];
 
   for (const word of pattern) {
+    if (!word || word.length === 0) continue;
+    
     let startIndex = 0;
     while ((startIndex = lowerText.indexOf(word, startIndex)) !== -1) {
       matches.push({

src/SearchStrategies/search/findLiteralMatches.ts

@@ -1,5 +1,3 @@
-import { SearchCache } from './SearchCache';
-
 export interface MatchInfo {
   start: number;
   end: number;
@@ -10,89 +8,29 @@ export interface MatchInfo {
 export interface Matcher {
   matches(text: string | null, criteria: string): boolean;
   findMatches?(text: string | null, criteria: string): MatchInfo[];
-  clearCache?(): void;
-  getCacheStats?(): { hitRate: number; size: number };
-}
-
-interface CachedResult {
-  matches: boolean;
-  matchInfo: MatchInfo[];
 }
 
 export class SearchStrategy implements Matcher {
   private readonly findMatchesFunction: (text: string, criteria: string) => MatchInfo[];
-  private readonly cache: SearchCache<CachedResult>;
 
-  constructor(
-    findMatchesFunction: (text: string, criteria: string) => MatchInfo[],
-    cacheOptions?: { maxSize?: number; ttl?: number }
-  ) {
+  constructor(findMatchesFunction: (text: string, criteria: string) => MatchInfo[]) {
     this.findMatchesFunction = findMatchesFunction;
-    this.cache = new SearchCache<CachedResult>({ 
-      maxSize: cacheOptions?.maxSize ?? 100, 
-      ttl: cacheOptions?.ttl ?? 60000 
-    });
   }
 
   matches(text: string | null, criteria: string): boolean {
     if (text === null || text.trim() === '' || !criteria) {
       return false;
     }
 
-    const cacheKey = this.getCacheKey(text, criteria);
-    const cached = this.cache.get(cacheKey);
-    if (cached !== undefined) {
-      return cached.matches;
-    }
-
-    const matchInfo = this.findMatchesInternal(text, criteria);
-    const result: CachedResult = {
-      matches: matchInfo.length > 0,
-      matchInfo
-    };
-    
-    this.cache.set(cacheKey, result);
-    return result.matches;
+    const matchInfo = this.findMatchesFunction(text, criteria);
+    return matchInfo.length > 0;
   }
 
   findMatches(text: string | null, criteria: string): MatchInfo[] {
     if (text === null || text.trim() === '' || !criteria) {
       return [];
     }
 
-    const cacheKey = this.getCacheKey(text, criteria);
-    const cached = this.cache.get(cacheKey);
-    if (cached !== undefined) {
-      return cached.matchInfo;
-    }
-
-    const matchInfo = this.findMatchesInternal(text, criteria);
-    const result: CachedResult = {
-      matches: matchInfo.length > 0,
-      matchInfo
-    };
-    
-    this.cache.set(cacheKey, result);
-    return result.matchInfo;
-  }
-
-  private findMatchesInternal(text: string, criteria: string): MatchInfo[] {
     return this.findMatchesFunction(text, criteria);
   }
-
-  private getCacheKey(text: string, criteria: string): string {
-    return `${text.length}:${criteria}:${text.substring(0, 20)}`;
-  }
-
-  clearCache(): void {
-    this.cache.clear();
-  }
-
-  getCacheStats(): { hitRate: number; size: number } {
-    const stats = this.cache.getStats();
-    return {
-      hitRate: stats.hitRate,
-      size: stats.size
-    };
-  }
 }

src/SearchStrategies/types.ts

@@ -142,23 +142,6 @@ describe('HybridSearchStrategy', () => {
     });
   });
 
-  describe('cache integration', () => {
-    const strategy = new HybridSearchStrategy();
-
-    it('should use cache for repeated searches', () => {
-      const matches1 = strategy.findMatches('hello world', 'hello');
-      const matches2 = strategy.findMatches('hello world', 'hello');
-      expect(matches1).toEqual(matches2);
-    });
-
-    it('should clear cache', () => {
-      strategy.findMatches('test', 'test');
-      strategy.clearCache();
-      const stats = strategy.getCacheStats();
-      expect(stats.size).toBe(0);
-    });
-  });
-
   describe('matches() method', () => {
     const strategy = new HybridSearchStrategy();