Skip to content

Conversation

@MicBun
Copy link
Member

@MicBun MicBun commented Jan 14, 2026

resolves: https://github.com/truflation/website/issues/3117

Summary by CodeRabbit

  • Chores

    • Updated Go module requirements to newer revisions; no behavioral or public API changes.
  • Migration

    • Adjusted ERC‑20 bridge configuration addresses; distribution period unchanged.
    • Added a second withdrawal-proof action and expanded the withdrawal-proof output to include chain, contract, timestamps, recipient, amount, block/hash/root, proofs and signatures; results remain confirmed-only and ordered newest-first.

✏️ Tip: You can customize this high-level summary in your review settings.

@MicBun MicBun self-assigned this Jan 14, 2026
@MicBun MicBun marked this pull request as ready for review January 14, 2026 06:27
@holdex
Copy link

holdex bot commented Jan 14, 2026

Time Submission Status

Member Status Time Action Last Update
MicBun ✅ Submitted 3h Update time Jan 14, 2026, 10:15 AM

@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updated Go module pseudo-version pins, changed escrow/proxy/explorer addresses in an ERC-20 bridge migration, and added a new public withdrawal-proof action plus expanded the existing withdrawal-proof action's return signature and implementation. No other exported declarations changed.

Changes

Cohort / File(s) Summary
Go Module Dependency Updates
go.mod
Bumped pseudo-version pins for github.com/trufnetwork/kwil-db and github.com/trufnetwork/kwil-db/core to newer pseudo-versions. No code logic changes.
ERC-20 Bridge Extension
internal/migrations/erc20-bridge/000-extension.sql
Updated hoodi_tt2 extension block addresses: escrow/proxy/explorer replaced with 0x80D9B3b6941367917816d36748C88B303f7F1415. distribution_period unchanged.
Withdrawal-proof Actions
internal/migrations/erc20-bridge/004-withdrawal-proof-action.sql
Added new public action hoodi_tt2_get_withdrawal_proof($wallet_address TEXT); expanded hoodi_tt_get_withdrawal_proof($wallet_address TEXT) RETURNS TABLE signature to include chain_id, contract, created_at, recipient, amount, block_hash, root, proofs, signatures and adjusted the function bodies to iterate hoodi_tt(.).list_wallet_rewards(... ) emitting the expanded columns. Header docs updated.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

type: feat

Suggested reviewers

  • pr-time-tracker

Poem

🐰 I hopped through rows of SQL and pins,
Swapped hashes, tweaked addresses, earned my wins.
A thump, a snuffle, a tidy little proof,
New actions bloom beneath the update roof —🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'allow different contract to listened in same chain' but the actual changes involve updating Go module dependencies and ERC-20 extension addresses, which are implementation details unrelated to the stated objective. Update the title to accurately reflect the changes made, such as 'chore: update kwil-db dependencies and ERC-20 bridge contract addresses' or provide changes that actually implement listening to different contracts in the same chain.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.



📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 268724f and 91df60f.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • go.mod
  • internal/migrations/erc20-bridge/004-withdrawal-proof-action.sql

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@MicBun MicBun merged commit 363e8ce into main Jan 14, 2026
6 of 7 checks passed
@MicBun MicBun deleted the alloCoExist branch January 14, 2026 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants