Skip to content

Fix: Concurrent wallet scanning with progress bar#34

Closed
KHHH2312 wants to merge 1 commit into
codegraphtheory:mainfrom
KHHH2312:fix/bounty-issue-29-f3a1
Closed

Fix: Concurrent wallet scanning with progress bar#34
KHHH2312 wants to merge 1 commit into
codegraphtheory:mainfrom
KHHH2312:fix/bounty-issue-29-f3a1

Conversation

@KHHH2312

@KHHH2312 KHHH2312 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replaced sequential token scanning in rug_check_wallet with ThreadPoolExecutor.
  • Added a custom ANSI progress bar with ETA that dynamically updates the terminal.
  • Progress bar automatically hides when --json or --export is passed to cli_wallet.
  • Implemented robust rate-limit backoff handling for 429 errors using a global semaphore lock.
  • Documented the new WALLET_SCAN_WORKERS environment variable in README.

Bounty / Issue

Solution

  • Using a shared semaphore and concurrent futures to safely spawn parallel token queries while dynamically backing off upon encountering rate limits.

/claim #29

💳 Payment Details:
Method: USDC
Address: 0x43991A9dC8Ddf492eab6E55685644c2cb9B001D2
Network: Base

Tests

  • Ran python -m pytest tests/test_checks.py locally. All tests pass, including the sequential/concurrent equivalence tests.

Notes

  • A short video demo of the concurrency speedup and the progress bar will be attached shortly in the comments!

Copilot AI review requested due to automatic review settings June 4, 2026 16:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@KHHH2312 KHHH2312 force-pushed the fix/bounty-issue-29-f3a1 branch from 24fc6a6 to c21eb60 Compare June 4, 2026 16:30
- Replaces sequential token scanning in `rug_check_wallet` with ThreadPoolExecutor.
- Adds custom ANSI progress bar with ETA.
- Includes 429 rate limit backoff using global semaphore.
- Documents WALLET_SCAN_WORKERS env var in README.md.
@KHHH2312 KHHH2312 force-pushed the fix/bounty-issue-29-f3a1 branch from c21eb60 to c031b30 Compare June 4, 2026 16:35
@codegraphtheory

Copy link
Copy Markdown
Owner

Sorry about this. The concurrent wallet scanning was already merged in #43 from a different branch that had the same feature. Thank you for submitting and please try another bounty issue. We appreciate your contribution.

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.

Concurrent wallet scanning with progress bar

3 participants