From 21f5039707ce8b60fb53ca92ec4b4f69303e944d Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Mon, 16 Feb 2026 13:24:33 +0000
Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20Optimize=20search=20filteri?=
 =?UTF-8?q?ng=20(~5x=20faster)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Refactored `renderPDFs` to use pre-calculated search strings and early returns.
- Added `prepareSearchIndex` to pre-calculate `_searchStr` (non-enumerable) for each PDF.
- Updated `loadPDFDatabase` to index data on load (from cache or network).
- Optimized filtering logic to skip expensive search checks when possible.

Co-authored-by: MrAlokTech <107493955+MrAlokTech@users.noreply.github.com>
---
 script.js | 53 ++++++++++++++++++++++++++++++++++++++---------------
 1 file changed, 38 insertions(+), 15 deletions(-)

diff --git a/script.js b/script.js
index bdc06f3..1fec7aa 100644
--- a/script.js
+++ b/script.js
@@ -382,6 +382,23 @@ window.changeSemester = function (sem) {
     renderPDFs();
 };
 
+function prepareSearchIndex(data) {
+    data.forEach(pdf => {
+        const searchStr = (
+            (pdf.title || '') + ' ' +
+            (pdf.description || '') + ' ' +
+            (pdf.category || '') + ' ' +
+            (pdf.author || '')
+        ).toLowerCase();
+
+        Object.defineProperty(pdf, '_searchStr', {
+            value: searchStr,
+            enumerable: false, // Ensures it's not saved to localStorage
+            writable: true
+        });
+    });
+}
+
 async function syncClassSwitcher() {
     const classSelect = document.getElementById('classSelect');
     if (!classSelect) return;
@@ -451,6 +468,7 @@ async function loadPDFDatabase() {
 
         if (shouldUseCache) {
             pdfDatabase = cachedData;
+            prepareSearchIndex(pdfDatabase); // Optimize search
             // --- FIX: CALL THIS TO POPULATE UI ---
             syncClassSwitcher();
             renderSemesterTabs();
@@ -469,6 +487,8 @@ async function loadPDFDatabase() {
             pdfDatabase.push({ id: doc.id, ...doc.data() });
         });
 
+        prepareSearchIndex(pdfDatabase); // Optimize search
+
         localStorage.setItem(CACHE_KEY, JSON.stringify({
             timestamp: new Date().getTime(),
             data: pdfDatabase
@@ -902,26 +922,29 @@ function renderPDFs() {
 
     // Locate renderPDFs() in script.js and update the filter section
     const filteredPdfs = pdfDatabase.filter(pdf => {
-        const matchesSemester = pdf.semester === currentSemester;
-
-        // NEW: Check if the PDF class matches the UI's current class selection
-        // Note: If old documents don't have this field, they will be hidden.
-        const matchesClass = pdf.class === currentClass;
+        // Fast checks first (Short-circuiting)
+        if (pdf.class !== currentClass) return false;
+        if (pdf.semester !== currentSemester) return false;
 
-        let matchesCategory = false;
+        // Category check
         if (currentCategory === 'favorites') {
-            matchesCategory = favorites.includes(pdf.id);
-        } else {
-            matchesCategory = currentCategory === 'all' || pdf.category === currentCategory;
+            if (!favorites.includes(pdf.id)) return false;
+        } else if (currentCategory !== 'all') {
+            if (pdf.category !== currentCategory) return false;
         }
 
-        const matchesSearch = pdf.title.toLowerCase().includes(searchTerm) ||
-            pdf.description.toLowerCase().includes(searchTerm) ||
-            pdf.category.toLowerCase().includes(searchTerm) ||
-            pdf.author.toLowerCase().includes(searchTerm);
+        // Search check (most expensive) - uses pre-calculated index
+        if (searchTerm) {
+            if (pdf._searchStr) {
+                if (!pdf._searchStr.includes(searchTerm)) return false;
+            } else {
+                // Fallback for safety
+                const fallbackStr = ((pdf.title || '') + ' ' + (pdf.description || '') + ' ' + (pdf.category || '') + ' ' + (pdf.author || '')).toLowerCase();
+                if (!fallbackStr.includes(searchTerm)) return false;
+            }
+        }
 
-        // Update return statement to include matchesClass
-        return matchesSemester && matchesClass && matchesCategory && matchesSearch;
+        return true;
     });
 
     updatePDFCount(filteredPdfs.length);