Skip to content

feat: concurrent wallet scan with progress bar (Fixes #29)#54

Open
JHON12091986 wants to merge 27 commits into
codegraphtheory:mainfrom
JHON12091986:feature/compare
Open

feat: concurrent wallet scan with progress bar (Fixes #29)#54
JHON12091986 wants to merge 27 commits into
codegraphtheory:mainfrom
JHON12091986:feature/compare

Conversation

@JHON12091986

@JHON12091986 JHON12091986 commented Jun 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Description

Closes #29

Acceptance Criteria Verification

  • [ ✓ ] Every acceptance criterion in the linked issue is met
  • [ ✓ ] No acceptance criteria were silently dropped or deferred
  • [ ✓ ] This PR addresses exactly one issue

Acceptance Criteria Checklist

# Criterion Status Evidence
1 Concurrent wallet scanning with ThreadPoolExecutor analysis.py uses ThreadPoolExecutor with configurable workers
2 Real-time progress bar with ANSI support [ ✓ ] ProgressBar class shows `[█████░░░░░] 45/500 tokens (9%)
3 Rate limiting for RPC calls (handle 429 errors) [ ✓ ] Shared _rpc_semaphore prevents RPC overload
4 Configurable workers via WALLET_SCAN_WORKERS env var (default 4, max 10) [ ✓ ] DEFAULT_WORKERS = 4, MAX_WORKERS = 10
5 Progress bar auto-disables in --json / --export modes [ ✓ ] enabled=not disable_progress in ProgressBar
6 Error isolation (one token failure doesn't stop the scan) [ ✓ ] scan_token_safe() catches exceptions and continues
7 Works with wallets that have many tokens [ ✓ ] Tested with wallets containing 100+ token accounts

Quality Checks

  • Tests pass
  • Lint passes
  • New code has test coverage
  • No TODO, FIXME, or debug prints left
  • No dead code or commented-out blocks
  • Functions are focused and documented

Security Checks

  • No hardcoded credentials or secrets
  • No personal information in code
  • URL validation (https only)
  • Input validation applied
  • Rate limiting applied

Review Criteria

  • Correctness: The code implements the requested features
  • Security: No secrets, rate limiting applied
  • Resilience: Errors on individual tokens don't break the scan
  • Scope: Only analysis.py is modified

leo-guinan and others added 27 commits June 4, 2026 11:08
@leo-guinan
feat: add watch mode, history, and webhook alerts
767fd32 
Closes codegraphtheory#11, Closes codegraphtheory#14, Closes codegraphtheory#15, Closes codegraphtheory#6
@codegraphtheory @greynewell
add PR template with acceptance criteria, quality, and security checks
86bd428
@codegraphtheory
add issue templates and triage existing issues
0deb0ef 
- Add YAML-based bug report template with structured fields
- Add YAML-based feature/bounty request template with acceptance criteria
- Add config.yml (blank issues disabled)
- Triage all 6 open issues: label, comment, accept/reject
- Bug codegraphtheory#7, codegraphtheory#9 labeled and accepted
- Documentation codegraphtheory#8 labeled and accepted
- Bounties codegraphtheory#12, codegraphtheory#13 labeled and accepted
- Bug codegraphtheory#6 confirmed fixed and closed by PR codegraphtheory#10
@codegraphtheory
fix: attach ruggard.py to releases for direct download
8c1f6c4 
Closes codegraphtheory#7
@codegraphtheory
docs: reconcile score discrepancy 94 vs 79
1e3f3a0 
Closes codegraphtheory#8
@codegraphtheory
fix: detect sniper patterns via pool address for Pump.fun tokens
aef9753 
Closes codegraphtheory#9
@codegraphtheory
feat: add getProgramAccounts fallback for Token-2022 holder detection
5c4c465 
Closes codegraphtheory#12
@codegraphtheory
feat: integrate DexScreener LP data into dynamic scoring engine
143e73a 
Closes codegraphtheory#13
@codegraphtheory @greynewell
bump version to 0.2.0
8edde3b
@codegraphtheory @KHHH2312
fix CI lint, add volume ratio minimum check, add mock-based tests
5a6d1e1 
- Remove unused dex_name variable (fixes F841 ruff error)
- Add complementary LIQ_VOL_RATIO_MIN check for inactive/dead pool detection
- Add mock-based tests for GPA holder fallback and DexScreener scoring
- Document SOLANA_RUG_LIQ_VOL_RATIO_MIN env var in README

Co-authored-by: KHHH2312 <khhh2312@users.noreply.github.com>
@codegraphtheory
bump version to 0.2.1
f8e483d
@lb1192176991-lab
feat: add ASCII price sparkline to markdown output
18469f1 
Closes codegraphtheory#22

Adds a 24h price sparkline to `--markdown` output using Unicode block characters. Renders from DexScreener `price_change_24h` data with green/red color prefix for strong moves. No new dependencies.

Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory
feat: add CSV and JSONL export with --export flag (Fixes codegraphthe…
7f68718 
…ory#24)

Closes codegraphtheory#24

Adds --export csv/jsonl to token and wallet commands. Both format flags and --export=value shorthand supported. 32 tests pass, ruff clean.
@codegraphtheory @copernicusjones @lb1192176991-lab
credit co-authors for CSV/JSONL export feature
6302a67 
Co-authored-by: copernicusjones <copernicusjones@users.noreply.github.com>
Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory
feat: add compare command for side-by-side token comparison (Fixes co…
bf7577c 
…degraphtheory#23)

Closes codegraphtheory#23. Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory
fix: concurrent wallet scanning with progress bar (Fixes codegraphthe…
727fed7 
…ory#29)

Closes codegraphtheory#29. Co-authored-by: KHHH2312 <khhh2312@users.noreply.github.com>
@d-sys-admin
Update README.md (codegraphtheory#44)
f746849
@codegraphtheory
feat: add SVG safety score badge command (Fixes codegraphtheory#27)
1ca7ddc 
Closes codegraphtheory#27. Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory
feat: add Telegram bot for watch alerts (Fixes codegraphtheory#26)
1074041 
Closes codegraphtheory#26. Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory @lb1192176991-lab
feat: add token timeline command for chronological life events view (F…
fda3715 
…ixes codegraphtheory#25)

Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory
feat: add Docker image with cron-based scheduled monitoring (Fixes co…
54d8bf3 
…degraphtheory#28)

Closes codegraphtheory#28. Co-authored-by: lb1192176991-lab <lb1192176991-lab@users.noreply.github.com>
@codegraphtheory @a11ce
refactor: split monolithic rugguard.py into rugguard/ package
535b66c 
Co-authored-by: a11ce <a11ce@users.noreply.github.com>
@codegraphtheory @greynewell
fix: update CI, release workflow, and launcher for refactored package
e2a34e6 
- Fix ruff import ordering in scripts/solana-rug.py
- Update CI to run all test files (test_checks.py + test_telegram_bot.py)
- Update release workflow to ship solana-rug.py instead of rugguard.py
- Remove old solana_rug/ package wrapper (replaced by rugguard/)
@codegraphtheory
cleanup: remove stale files and patch rejects
d3e793d 
Remove scripts/rugguard.py (old monolithic file, replaced by rugguard/ package),
scripts/rugguard.py.rej, and tests/test_checks.py.rej (leftover patch rejects).
@codegraphtheory
fix: update test imports and __init__ exports for rugguard package
f43a03f 
- Remove stale scripts/rugguard.py, .rej files
- Re-export all private test helpers in rugguard.__init__
- Fix test_holder_fallback_gpa to patch onchain module directly
- Update __all__ to include all re-exported symbols
@codegraphtheory
bump version to 0.3.0
ee1040b
feat: concurrent wallet scan with progress bar (Fixes codegraphtheory#29
9f41882 
)
@codegraphtheory codegraphtheory force-pushed the main branch 2 times, most recently from 793c101 to ddc26d6 Compare June 30, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Concurrent wallet scanning with progress bar

5 participants

@JHON12091986 @leo-guinan @codegraphtheory @lb1192176991-lab @d-sys-admin