feat: implement TFT-to-SPORE bridge with batch_all memo extraction #1

Merged

mahmoud merged 16 commits from development into main 2026-02-26 13:13:29 +00:00

Conversation 0 Commits 16 Files changed 38 +4084 -331

mahmoud commented 2026-02-25 12:40:22 +00:00

Owner

Copy link

Changes

• Extract Hero Ledger address from Utility::batch_all extrinsics by
  scanning raw SCALE-encoded bytes for System::remark containing the
  64-char hex destination address
• Fix amount truncation: pass raw planck amounts (u64) through the
  event pipeline; handlers convert to SPORE raw units (u128, 18 decimals)
• Auto-register recipients on SPORE contract via storage_deposit before
  transfer (works around unreliable is_registered SDK method)
• Use HeroLedgerClient::connect_with_root with network-derived root
  account (devnet→dev.hero, testnet→test.hero, mainnet→hero)
• Switch hero_ledger dependency from local path to git for CI/deployment
• Document block cursor setup in README to avoid scanning from genesis

### Changes - Extract Hero Ledger address from Utility::batch_all extrinsics by scanning raw SCALE-encoded bytes for System::remark containing the 64-char hex destination address - Fix amount truncation: pass raw planck amounts (u64) through the event pipeline; handlers convert to SPORE raw units (u128, 18 decimals) - Auto-register recipients on SPORE contract via storage_deposit before transfer (works around unreliable is_registered SDK method) - Use HeroLedgerClient::connect_with_root with network-derived root account (devnet→dev.hero, testnet→test.hero, mainnet→hero) - Switch hero_ledger dependency from local path to git for CI/deployment - Document block cursor setup in README to avoid scanning from genesis

mahmoud added 1 commit 2026-02-25 12:40:22 +00:00

feat: implement TFT-to-SPORE bridge with batch_all memo extraction ... cd739c4de4

- Extract Hero Ledger address from Utility::batch_all extrinsics by
  scanning raw SCALE-encoded bytes for System::remark containing the
  64-char hex destination address
- Fix amount truncation: pass raw planck amounts (u64) through the
  event pipeline; handlers convert to SPORE raw units (u128, 18 decimals)
- Auto-register recipients on SPORE contract via storage_deposit before
  transfer (works around unreliable is_registered SDK method)
- Use HeroLedgerClient::connect_with_root with network-derived root
  account (devnet→dev.hero, testnet→test.hero, mainnet→hero)
- Switch hero_ledger dependency from local path to git for CI/deployment
- Document block cursor setup in README to avoid scanning from genesis

mahmoud added 4 commits 2026-02-25 21:11:50 +00:00

fix: prevent double-spend, u128 truncation, and broken historical scanning ... 8425c740f9

- Fix u128→u64 silent truncation on transfer amounts by widening
  ChainEvent.amount, PendingTx.amount, and calculate_spore_amount to u128
- Fix historical block scanning by using subxt LegacyRpcMethods for
  direct chain_getBlockHash RPC, bypassing broken upstream client method
- Prevent double-spend on crash recovery by replacing is_processed() with
  get_status() in process_event — PROCESSING records are now skipped
- Upgrade stuck transaction warnings to error-level with manual SQL
  resolution instructions and CRITICAL logging on mark_confirmed failure

feat: production hardening — graceful shutdown, reconnection, retry limits, min amount ... 6e987e9148

- Add graceful shutdown via SIGINT/SIGTERM with tokio::signal, saving
  cursor before clean exit
- Add RPC reconnection logic that recreates TFChain client after 3
  consecutive poll failures
- Add configurable max retry limit (MAX_RETRIES, default 5) to stop
  retrying permanently-failed transactions
- Add configurable minimum transfer amount (MIN_TFT_AMOUNT, default
  1 TFT) to reject dust transfers
- Improve memo extraction to handle multiple System::remark calls in
  batch_all, rejecting all-zeros addresses

feat: add HTTP API, CLI admin commands, and post-audit bug fixes ... 07019ae81d

- HTTP API server (/health, /status, /txs) with query filters and pagination
- CLI commands: status, retry, resolve for operator administration
- Dual hash: store both synthetic tx_hash and extrinsic_hash, lookup by either
- Remove unused retention feature (delete_older_than_days, tx_retention_days)
- Fix silent memo loss when block.extrinsics() fails (now warns)
- Fix negative limit bypass in /txs (clamp 1..500)
- Fix cmd_retry not resetting retry counter (new reset_for_retry method)
- Fix non-deterministic find_tx on hash collision (prefer tx_hash match)
- Consolidate duplicate extrinsic lookup in decode_transfer_event
- Replace serde_json unwrap with proper error handling in handlers
- Add DB indices on sender and created_at columns
- Validate API_PORT <= 65535 before u16 cast

docs: add CLI reference and fix tx_hash query parameter ... 62aad83eb2

- Add docs/cli.md with full command reference, build instructions, and examples
- Accept both ?tx= and ?tx_hash= as query parameter names for /txs endpoint
- Document that hash values must not be wrapped in quotes
- Link CLI and API docs from README

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mahmoud added 1 commit 2026-02-25 22:25:27 +00:00

feat: add automatic TFT refund system for invalid transfers ... 367bd539ce

- Add Refunder trait and TfChainRefunder for sending TFT back on invalid memo
- Add RefundHandler as catch-all for unmatched treasury transfers
- Add REFUNDED/REFUND_FAILED statuses with V3 DB migration
- Extend config with REFUND_ENABLED, TFCHAIN_TREASURY_MNEMONIC, REFUND_FEE_PLANCK
- Add is_refundable() to BridgeError for inline refund on handler failure
- Add comprehensive unit tests for refund flows in bridge, store, config, error

mahmoud added 1 commit 2026-02-25 22:30:10 +00:00

fix: use GITHUB_HEAD_REF for PR checkout branch in CI ... 01a4a9600d

- GITHUB_REF_NAME resolves to the PR number on pull_request events,
  causing "Remote branch 1 not found" clone failures

mahmoud added 1 commit 2026-02-25 22:33:53 +00:00

style: reformat code for improved readability ... 5d5d964439

- Break long lines into multiple for clarity
- Adjust indentation and newline usage
- Reformat function calls and macro arguments

mahmoud added 1 commit 2026-02-25 22:34:50 +00:00

ci: add fmt check and make clippy a hard failure ... 438a8e1444

- Add cargo fmt --check step to enforce formatting
- Remove continue-on-error from clippy so lint warnings block CI

mahmoud added 1 commit 2026-02-25 22:48:55 +00:00

docs: add API integration guide and update config docs ... 171580febd

- Add API integration guide for UI clients
- Detail API endpoints, transaction lifecycle, and polling
- Expand optional configuration variables documentation
- Add refund-specific configuration variables
- Update NEAR RPC endpoint example

mahmoud added 1 commit 2026-02-25 23:03:17 +00:00

feat: add Docker support, configurable operational tuning, and server tests ... 9a7549b2c6

Add multi-stage Dockerfile with dependency caching, docker-compose with
healthcheck, and .dockerignore. Make STUCK_CHECK_INTERVAL and
RECONNECT_THRESHOLD configurable via env vars instead of hardcoded
constants. Add comprehensive API server tests using tower::oneshot.
Update Makefile with Docker and lint targets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mahmoud added 1 commit 2026-02-26 07:18:10 +00:00

style: reformat code for improved readability ... a85645e3af

- Apply consistent line wrapping and indentation
- Adjust function signature formatting
- Reorder some import statements

mahmoud added 1 commit 2026-02-26 07:48:58 +00:00

fix: bump Dockerfile Rust version to 1.92 (required by dependencies) 1171cd4742

mahmoud added 1 commit 2026-02-26 12:32:44 +00:00

perf: add concurrent block scanning with configurable batch size: ... 916e2ab0c4

- Fetch blocks concurrently via buffer_unordered (SCAN_CONCURRENCY)
- Cap scan range per poll cycle (SCAN_BATCH_SIZE) for incremental cursor advance- Add progress logging every N blocks (SCAN_PROGRESS_INTERVAL)
- Cancel in-flight polls immediately on shutdown signal
- Add futures crate dependency for stream concurrency

mahmoud added 1 commit 2026-02-26 12:50:17 +00:00

docs: update documentation with refund flow ... adba99dff3

- Convert state, overview, and sequence diagrams to Mermaid
- Document TFT refund mechanism and new transaction statuses
- Add `refund_tx` to transaction examples and schema
- Introduce new block scanning and refund configuration
- Update quick start with Docker commands and make targets

mahmoud added 1 commit 2026-02-26 13:03:18 +00:00

fix: resolve mermaid syntax and clippy lint errors ... d90639cf7a

- Fix stateDiagram-v2 note syntax in api-integration.md
- Replace modulo check with is_multiple_of() for Rust 1.92 clippy

mahmoud merged commit 7c754c73c6 into main 2026-02-26 13:13:29 +00:00

mahmoud referenced this pull request from a commit 2026-02-26 13:13:30 +00:00

feat: implement TFT-to-SPORE bridge with batch_all memo extraction (#1)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

prio_critical

  

prio_low

  

type_bug

  

type_contact

  

type_issue

  

type_lead

  

type_question

  

type_story

  

type_task

No labels

prio_critical

prio_low

type_bug

type_contact

type_issue

type_lead

type_question

type_story

type_task

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

lhumina_code/hero_tfspores!1

No description provided.