Closed scosman closed 2 months ago
[!CAUTION]
Review failed
The pull request is closed.
The update transitions the project from using the Fuse.js library for search functionality to the Lunr.js library. This change introduces new TypeScript dependencies and significantly modifies how search indexes are built and searched. The updated structure enhances efficiency, type safety, and clarity in handling search results.
Files | Change Summary |
---|---|
package.json |
Added dependencies: @types/lunr for TypeScript support and lunr for full-text search functionality. |
src/lib/build_index.ts , src/routes/(marketing)/search/+page.svelte |
Refactored search indexing logic from Fuse.js to Lunr.js. Updated data structures, query handling, and result rendering. |
sequenceDiagram
participant User
participant SearchPage
participant LunrIndex
User->>SearchPage: Enter search query
SearchPage->>LunrIndex: Query index
LunrIndex-->>SearchPage: Return search results
SearchPage-->>User: Display results
π° In fields of code, a change we see,
With Lunr hopping, wild and free!
A search so swift, a type so bright,
Our data dances, pure delight!
Hooray for updates, letβs cheer and play,
For joyful coding on this fine day! πΌ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Latest commit: |
374cf32
|
Status: | β Deploy successful! |
Preview URL: | https://eb4bcf2e.saasstarter.pages.dev |
Branch Preview URL: | https://lunr.saasstarter.pages.dev |
Lunr has real inverted search index, and better word matching (price -> pricing).
Summary by CodeRabbit
New Features
Bug Fixes
Chores