Skip to content

feat(txtop): Track Materios L2 transactions#333

Merged
arepala-uml merged 1 commit into
mainfrom
feat/315/track_materios_l2_txs
Jun 4, 2026
Merged

feat(txtop): Track Materios L2 transactions#333
arepala-uml merged 1 commit into
mainfrom
feat/315/track_materios_l2_txs

Conversation

@arepala-uml

@arepala-uml arepala-uml commented Jun 3, 2026

Copy link
Copy Markdown
Contributor
  1. Using this transaction metadata (https://cexplorer.io/tx/b2b55a984af345b3ae758d7c92ee033e3066b24c17b7b1755f07a30e9bababf3?tab=metadata), I have made changes to detect materios transactions by Cardano metadata label 8746.
  2. For any of the pending transactions from mempool if it is detected as materios then i am showing it with icon 'M' and added the icon to the legend as well.
  3. Added a unit test for materios metadata detection using the above transaction metadata as sample.
  4. I have fixed the tcp retry behavior by creating a fresh ouroboros connection per retry attempt. (This is not related to this issue but i ran into some connection errors when trying to connect cardano node).

Closes #315


Summary by cubic

Detect Materios L2 transactions using Cardano metadata label 8746 and mark them with an Ⓜ️ icon in the mempool and legend. Also fixes retry stability by creating a new ouroboros connection on each attempt.

  • New Features

    • Detect Materios tx via label 8746 with marker k: "p", v: "materios".
    • Show Ⓜ️ for matching pending tx; add "Materios" to the legend.
    • Add unit tests for the detection logic.
  • Bug Fixes

    • Create a fresh ouroboros connection per retry to avoid closed error channels and half-open reconnects.

Written for commit d380a2e. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

Release Notes

  • New Features

    • Implemented detection of Materios transactions with dedicated Ⓜ️ icon display
    • Updated UI legend to include Materios transaction type
  • Bug Fixes

    • Improved connection retry handling to explicitly close failed socket connections before subsequent retry attempts

Signed-off-by: Akhil Repala <arepala@blinklabs.io>
@arepala-uml
arepala-uml marked this pull request as ready for review June 3, 2026 23:15
@arepala-uml
arepala-uml requested a review from a team as a code owner June 3, 2026 23:15
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This PR refactors the connection retry mechanism in GetConnection to create fresh connection objects and error channels per retry attempt, with explicit cleanup on dial failure. It adds recursive metadata detection to identify Materios L2 transactions via marker inspection (key/value pairs and integer marker 8746) within nested structures, applies the Materios icon (Ⓜ️) during transaction processing when metadata matches, updates the UI legend to display Materios, and includes unit tests validating the detection logic against in-memory fixtures.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main feature being implemented—tracking Materios L2 transactions with metadata detection and UI visibility.
Linked Issues check ✅ Passed The PR fulfills issue #315 requirements by detecting Materios transactions via metadata label 8746 and displaying them with an icon in the UI.
Out of Scope Changes check ✅ Passed The TCP retry connection fix is a related bug fix improving reliability when connecting to a Cardano node, not unrelated to the primary objective.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/315/track_materios_l2_txs

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.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 2 files

Re-trigger cubic

@arepala-uml
arepala-uml merged commit a2aefc1 into main Jun 4, 2026
13 checks passed
@arepala-uml
arepala-uml deleted the feat/315/track_materios_l2_txs branch June 4, 2026 14:20
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.

Track Materios L2 transactions

2 participants