feat(worker): offload browser search to web worker by Vexxo-Dev · Pull Request #88 · adelpro/quran-search-engine

Vexxo-Dev · 2026-03-15T10:14:09Z

Closes #12

Summary

Add a dedicated search Web Worker that loads Quran data and indices off the main thread.
Introduce a typed worker client with requestId correlation and a main-thread fallback path.
Update React, Angular, and vanilla TS examples to use the worker when available, keeping synchronous search as a fallback.

Test plan

pnpm run lint
pnpm run test
pnpm run build

docs: Create Structured Documentation (adelpro#70)

feat(search): add optional regex query support (adelpro#71)

… architecture documentation

- Replace pnpm with yarn for package management and workspace setup - Update CI workflows, documentation, and example scripts accordingly - Add new GitHub workflows for security scanning and package manager detection - Reorganize project files and update workspace configuration

…an search logic

…f rootna api

…tionality

- Set splitting to false in tsup config to reduce bundle complexity - Remove size-limit check for CommonJS bundle while keeping ES module check

into develop

…nal Quran database

….json

…arabic-mapping-dataset-nlp English-Arabic Dataset and NLP Processing

- Add dedicated search worker and client wrapper. - Update React, Angular, and vanilla TS examples to use the worker with a main-thread fallback.

coderabbitai · 2026-03-15T10:14:17Z

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 17c6aae3-b7c3-4da5-9c30-7831854a21e9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📝 Coding Plan

Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can enforce grammar and style rules using `languagetool`.

Configure the reviews.tools.languagetool setting to enable/disable rules and categories. Refer to the LanguageTool Community to learn more.

- Update worker support check to use globalThis for better compatibility. - Refactor worker instantiation to handle unsupported environments gracefully.

- Declare worker globals (postMessage, self) in searchWorker.ts - Use globalThis.Worker in searchWorkerClient.ts to fix no-undef - Remove unused imports and fix types in english-to-arabic-builder.ts - Remove unused parse import and type quranRows in word-level-transliteration.ts - Add normalized_text to ayahData type in word-level-transliteration.ts

- validateLengths: use Record<string, unknown> for parsed - main: use Record<string, unknown> for sourceData from JSON

- Import LRUCache from utils/lru-cache in searchWorker and searchWorkerClient - Fix TS2322 by typing pending resolve as PendingResolve and asserting in initData/runSearch

adelpro · 2026-03-15T21:14:43Z

Salam @Vexxo-Dev, this is the most complete solution to the issue. Baraka Allahu fik. I need a bit more time to explore and analyze your implementation.

Vexxo-Dev · 2026-03-15T21:16:12Z

و عليكم السلام ي هندسة @adelpro

مفيش اي مشكله خد وقتك و بارك الله ف وقتك ❤️

- Fix ESLint no-undef in worker files and switch LRUCache imports to utils/lru-cache - Tighten types and remove remaining any in english-to-arabic-builder and word-level-transliteration - Configure ESLint tsconfigRootDir in eslint.config.mjs to avoid multi-TSConfig parsing errors

adelpro and others added 30 commits March 4, 2026 08:05

Merge pull request adelpro#76 from Vexxo-Dev/develop

ada505d

docs: Create Structured Documentation (adelpro#70)

chore: add docs:start script to launch documentation site

4ae4308

Merge pull request adelpro#78 from Vexxo-Dev/develop

e1fa7b2

feat(search): add optional regex query support (adelpro#71)

feat: add English-Arabic dictionary builder

6774fcd

feat: add boolean search implementation and test suite

5f5d596

docs: documented the boolean search function

1ca558a

feat: implement a layered search pipeline, core search utilities, and…

f1509b3

… architecture documentation

docs: add acknowledgments section with ITQAN community

003b231

docs: link moon icon to community post in README

540849f

style(assets): update moon icon fill color to match them

70706d6

style(assets): reduce moon.svg icon size from 64px to 16px

0046bd1

chore: remove yarn folder

f03ff11

docs: replace custom moon.svg badge with shields.io badge

cbfc65d

chore: bump version to 0.2.0-athar for pre-release build

e0ffb7f

Merge branch 'adelpro:develop' into develop

8578baf

Merge branch 'adelpro:develop' into develop

c574658

refactor: moved the files to layers directory and reimplemented boole…

ab77e9f

…an search logic

feat: initial workflow of boolean filtering logic integrated into search

6f3a903

docs: document boolean refactor in search function

f998e3b

fix: fix till line 5000 in word-map.json

563bc4b

fix: fix till line 5000 in word-map.json

858b335

refactor: rewrite tests for boolean search to use normal search

38735f8

refactor: normalize words before addition to boolean lists

0bcc0c0

feat: add simple search with OR logic

de939a3

docs: add comments and explanation to the OR logic

7bfcc0d

Merge branch 'boolean' into develop

0466789

chore: pin yarn version to 4.13.0 in package.json

ff25002

chore: update yarn install

7d03d80

chore: update dependencies and add .yarn to gitignore

999ac98

MostafaOsmanFathi and others added 13 commits March 12, 2026 18:54

fix: word-map root words using rootna api

bc8fe0a

refactor: english-to-arabic-builder.ts to use word-map.json instead o…

71c02ef

…f rootna api

refactor: restructure english-to-arabic-builder to group related func…

3a71f77

…tionality

refactor: update CI workflow to use Yarn v4 and simplify steps

04eacec

refactor: replace size.yml with simplified size-limit.yml workflow

bb6dba8

ci: replace master branch with staging in size-limit workflow

5709038

build: disable code splitting and update size-limit config

213bc74

- Set splitting to false in tsup config to reduce bundle complexity - Remove size-limit check for CommonJS bundle while keeping ES module check

Merge branch 'develop' of https://github.com/adelpro/quran-search-engine

3edc9e0

into develop

refactor(word-level-transliteration): use quran.json instead of exter…

b3dc569

…nal Quran database

refactor: use standard instead of uthmani and add record for phonetic…

3067df9

….json

docs: add phonetic-search-indexing documentation

0299117

Merge pull request adelpro#85 from MostafaOsmanFathi/feat/english-to-…

f401e82

…arabic-mapping-dataset-nlp English-Arabic Dataset and NLP Processing

feat(worker): offload browser search to web worker

4103cb1

- Add dedicated search worker and client wrapper. - Update React, Angular, and vanilla TS examples to use the worker with a main-thread fallback.

Vexxo-Dev force-pushed the develop branch from cbc9c42 to 4103cb1 Compare March 15, 2026 10:15

Merge upstream develop into fork develop

3e77897

Vexxo-Dev force-pushed the develop branch from 0c89017 to 3e77897 Compare March 15, 2026 10:23

Vexxo-Dev and others added 7 commits March 15, 2026 12:27

Merge branch 'develop' into develop

d030b14

fix: restore Worker utilities in index.ts

727f811

chore: update yarn.lock to Solve action Problem

497450b

fix(worker): improve worker support detection and initialization

466200f

- Update worker support check to use globalThis for better compatibility. - Refactor worker instantiation to handle unsupported environments gracefully.

fix: replace remaining any types in english-to-arabic-builder

6b6dcd9

- validateLengths: use Record<string, unknown> for parsed - main: use Record<string, unknown> for sourceData from JSON

fix: worker imports and promise resolve types

f08e3eb

- Import LRUCache from utils/lru-cache in searchWorker and searchWorkerClient - Fix TS2322 by typing pending resolve as PendingResolve and asserting in initData/runSearch

Vexxo-Dev mentioned this pull request Mar 15, 2026

[HARD] Web Worker Offloading for Search #12

Closed

Vexxo-Dev added 2 commits March 16, 2026 16:10

Merge remote-tracking branch 'upstream/develop' into develop

c3d8b5b

adelpro merged commit 0477000 into adelpro:develop Mar 16, 2026
7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(worker): offload browser search to web worker#88

feat(worker): offload browser search to web worker#88
adelpro merged 81 commits intoadelpro:developfrom
Vexxo-Dev:develop

Vexxo-Dev commented Mar 15, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Mar 15, 2026 •

edited

Loading

Review skipped

Uh oh!

adelpro commented Mar 15, 2026

Uh oh!

Vexxo-Dev commented Mar 15, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

Vexxo-Dev commented Mar 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

coderabbitai Bot commented Mar 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

adelpro commented Mar 15, 2026

Uh oh!

Vexxo-Dev commented Mar 15, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Vexxo-Dev commented Mar 15, 2026 •

edited

Loading

coderabbitai Bot commented Mar 15, 2026 •

edited

Loading