⚡ Bolt: Pre-calculate Search Index & Date for PDF Cards#27
⚡ Bolt: Pre-calculate Search Index & Date for PDF Cards#27MrAlokTech wants to merge 1 commit intomainfrom
Conversation
What: Pre-calculated searchable fields, formatted dates, and "is new" status directly onto the `pdfDatabase` objects. Why: The `renderPDFs` search loop and `createPDFCard` DOM rendering ran expensive string lowercase logic, regular expressions, and `new Date()` parses for every card, every time search input changed. Impact: Significant reduction in CPU time during high-frequency events like typing in the search box. Reduces GC pressure. Measurement: Search responsiveness during typing should be visually improved, especially as the number of PDFs in the database grows. Tested manually. Co-authored-by: MrAlokTech <107493955+MrAlokTech@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Deploying classnotes with
|
| Latest commit: |
fe279d4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://36eb7f2f.classnotes.pages.dev |
| Branch Preview URL: | https://bolt-precalc-search-index-88.classnotes.pages.dev |
1 participant
What: Pre-calculated searchable fields (
_searchStr), formatted dates (_formattedDate), and "is new" status (_isNew) directly onto thepdfDatabaseobjects in a newprepareSearchIndexfunction immediately following the database load. Removed duplicate date calculations and lowerecased string concatenations from therenderPDFsloop andcreatePDFCardgeneration.Why: The
renderPDFssearch loop andcreatePDFCardDOM rendering ran expensive string lowercase logic, regular expressions, andnew Date()parses for every card, every time the search input changed.Impact: Significant reduction in CPU time during high-frequency events like typing in the search box. Reduces GC pressure by avoiding the allocation of temporary
Dateobjects inside loops.Measurement: Search responsiveness during typing should be visually improved, especially as the number of PDFs in the database grows. Tested manually and verified via local Playwright frontend test (checking
renderPDFs).PR created automatically by Jules for task 8881242032742789864 started by @MrAlokTech