Skip to content

Bounty #2278: Ergo Anchor Chain Proof Verifier (100 RTC)#1735

Closed
kuanglaodi2-sudo wants to merge 2 commits intoScottcjn:mainfrom
kuanglaodi2-sudo:feature/ergo-verifier-v3
Closed

Bounty #2278: Ergo Anchor Chain Proof Verifier (100 RTC)#1735
kuanglaodi2-sudo wants to merge 2 commits intoScottcjn:mainfrom
kuanglaodi2-sudo:feature/ergo-verifier-v3

Conversation

@kuanglaodi2-sudo
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown

Welcome to RustChain! Thanks for your first pull request.

Before we review, please make sure:

  • Your PR has a BCOS-L1 or BCOS-L2 label
  • New code files include an SPDX license header
  • You've tested your changes against the live node

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150)

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions bot added documentation Improvements or additions to documentation BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) size/XL PR: 500+ lines labels Mar 21, 2026
@Scottcjn
Copy link
Copy Markdown
Owner

NEEDS WORK — Good architecture but several correctness issues:

  1. Wrong GENESIS_TIMESTAMP: You use 1728000000 (old Oct 2024 value). Production is 1764706927.
  2. Wrong register: Current anchor writer stores commitment in R5 with 0e40... prefix, not R4 with 0e20.... Your extract_r4_commitment() returns None against current-format transactions.
  3. Schema fallback broken: Query orders by rustchain_height which doesn't exist in ergo_miner_anchor schema. Crash confirmed.
  4. Wrong fallback field: Maps commitment_hash from tx_id instead of commitment column.
  5. Recomputation doesn't match: You hash miners from a 10-min window, but ergo_miner_anchor.py hashes LIMIT ? latest miners.

Fix these and resubmit. The dataclass architecture and offline mode are solid — just needs alignment with actual codebase.

100 RTC on merge if fixed.

@Scottcjn
Copy link
Copy Markdown
Owner

Thanks for your interest! These PRs have issues: PR #1748 destructively overwrites the project README, multiple PRs contain placeholder data, and 7 high-value bounty claims in one day from a 22-day account suggests bulk generation. Please review our contribution guidelines — start with one small, complete PR and build from there. Quality over quantity.

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

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) documentation Improvements or additions to documentation size/XL PR: 500+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants