Skip to content

feat: implement web worker for stellar scanning#35

Open
bbkenny wants to merge 1 commit into
wraith-protocol:developfrom
bbkenny:fix/web-worker-scanning-issue-3
Open

feat: implement web worker for stellar scanning#35
bbkenny wants to merge 1 commit into
wraith-protocol:developfrom
bbkenny:fix/web-worker-scanning-issue-3

Conversation

@bbkenny
Copy link
Copy Markdown

@bbkenny bbkenny commented May 29, 2026

This PR moves the Stellar scanning logic to a Web Worker to improve UI responsiveness.

Key changes:

  • Created src/workers/stellar-scanner.worker.ts to handle fetching and scanning announcements.
  • Updated StellarReceive.tsx to use the Web Worker for background scanning.
  • Improved error handling and worker lifecycle management (termination on unmount or re-scan).

Closes #3

@vercel
Copy link
Copy Markdown

vercel Bot commented May 29, 2026

@bbkenny is attempting to deploy a commit to the truthixify's projects Team on Vercel.

A member of the Team first needs to authorize it.

@bbkenny
Copy link
Copy Markdown
Author

bbkenny commented May 29, 2026

Hi @wraith-protocol, I've implemented the Web Worker for background scanning as requested in issue #3. This ensures the UI remains responsive even during heavy scanning tasks. Ready for review!

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 29, 2026

@bbkenny Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@truthixify truthixify changed the base branch from main to develop June 1, 2026 15:44
@truthixify
Copy link
Copy Markdown
Contributor

The Web Worker implementation is correct in shape (offloads scanAnnouncements from main thread, uses Soroban RPC ledger range probing, paginates via cursor, posts progress + result messages).

One blocker: the PR also adds a 16,456-line package-lock.json. This repo uses pnpm with pnpm-lock.yaml, so the npm lockfile would conflict and shouldn't be committed. Could you remove it?

rm package-lock.json
echo "package-lock.json" >> .gitignore  # if not already there
pnpm install  # regenerates pnpm-lock.yaml with any deps you added
git add -u .gitignore pnpm-lock.yaml
git commit --amend  # or new commit
git push --force-with-lease

Once the lockfile is sorted I will merge. Thanks @bbkenny.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web Worker scanning for Stellar Receive

2 participants