Skip to content

revamp mainnet tooling and infra#31

Merged
Troublor merged 10 commits into
mainfrom
sami/mainnet-tooling-revamp
Apr 8, 2026
Merged

revamp mainnet tooling and infra#31
Troublor merged 10 commits into
mainfrom
sami/mainnet-tooling-revamp

Conversation

@0xSami-m

@0xSami-m 0xSami-m commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

full revamp of mainnet infra and tooling

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
@claude

claude Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Review

Good restructuring for mainnet — the content is much more comprehensive. A few issues to fix before merging:

Section headings should be H2, not H1.
Every section (RPC Providers, Explorers, Oracles, etc.) uses # (H1), but the page already has an H1 title on line 4. These should all be ##. Multiple H1s on one page breaks the GitBook navigation sidebar and page structure.

Three rendering bugs and one question flagged inline.

Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md
@claude

claude Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Good overall — the revamp is a solid step up from the testnet-era draft. All four prior review threads are resolved. Five issues remaining:

  1. H1 section headings (inline, line 9): Every section uses # but the page title is already #. Change all section headings to ## — affects GitBook sidebar anchors.
  2. Frontmatter description (inline, line 2): Doesn't reflect the new sections (bridges, wallets, security, custody, on-ramps). Suggestion attached.
  3. # Explorers double space (inline, line 34): Typo in heading.
  4. Bare URLs in Explorers and Data & Analytics tables (inline, lines 38–40): GitBook won't auto-link bare URLs in table cells; wrap them with [text](url).
  5. Canonical Bridge blockquote (inline, line 102): Use {% hint style="info" %} instead of > for proper GitBook rendering.

Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
Comment thread docs/dev/tooling.md Outdated
made it more gitbook compatible 

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
Comment thread docs/dev/tooling.md Outdated
Comment on lines +74 to +84
| Tool | URL | Mainnet |
|---|---|---|
| Uptime Dashboard | https://uptime.megaeth.com | Supported |
| Dune Analytics | https://dune.com/blockchains/megaeth | Supported |
| Birdeye | https://birdeye.so/megaeth/find-gems | Supported |
| DexScreener | https://dexscreener.com/megaeth | Supported |
| Defined.fi | https://www.defined.fi/tokens/discover?network=mega | Supported |
| Miniblocks | https://miniblocks.io | Supported |
| DefiLlama | https://defillama.com/chain/megaeth | Supported |
| CoinGecko | https://www.coingecko.com/en/categories/megaeth-ecosystem | Supported |
| Grow The Pie | https://www.growthepie.com/chains/megaeth | Supported |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The previous review thread (marked resolved) asked to link the tool names and remove bare URLs — but neither was applied. Tool names are still unlinked plain text and all URLs are bare, which won't render as links in GitBook. Drop the URL column and link the names:

Suggested change
| Tool | URL | Mainnet |
|---|---|---|
| Uptime Dashboard | https://uptime.megaeth.com | Supported |
| Dune Analytics | https://dune.com/blockchains/megaeth | Supported |
| Birdeye | https://birdeye.so/megaeth/find-gems | Supported |
| DexScreener | https://dexscreener.com/megaeth | Supported |
| Defined.fi | https://www.defined.fi/tokens/discover?network=mega | Supported |
| Miniblocks | https://miniblocks.io | Supported |
| DefiLlama | https://defillama.com/chain/megaeth | Supported |
| CoinGecko | https://www.coingecko.com/en/categories/megaeth-ecosystem | Supported |
| Grow The Pie | https://www.growthepie.com/chains/megaeth | Supported |
| Tool | Mainnet |
|---|---|
| [Uptime Dashboard](https://uptime.megaeth.com) | Supported |
| [Dune Analytics](https://dune.com/blockchains/megaeth) | Supported |
| [Birdeye](https://birdeye.so/megaeth/find-gems) | Supported |
| [DexScreener](https://dexscreener.com/megaeth) | Supported |
| [Defined.fi](https://www.defined.fi/tokens/discover?network=mega) | Supported |
| [Miniblocks](https://miniblocks.io) | Supported |
| [DefiLlama](https://defillama.com/chain/megaeth) | Supported |
| [CoinGecko](https://www.coingecko.com/en/categories/megaeth-ecosystem) | Supported |
| [Grow The Pie](https://www.growthepie.com/chains/megaeth) | Supported |

Comment thread docs/dev/tooling.md Outdated
Comment on lines +36 to +40
| Explorer | URL | Mainnet |
|---|---|---|
| [Etherscan](https://mega.etherscan.io) | https://mega.etherscan.io | Supported |
| [Blockscout](https://megaeth.blockscout.com) | https://megaeth.blockscout.com | Supported |
| [Explore Mega](https://exploremega.xyz) | https://exploremega.xyz | Supported |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The previous review thread (marked resolved) asked to drop the redundant URL column and link the names — but the fix was not applied. The URL column duplicates the link already in the Explorer name, and bare URLs won't render as links in GitBook:

Suggested change
| Explorer | URL | Mainnet |
|---|---|---|
| [Etherscan](https://mega.etherscan.io) | https://mega.etherscan.io | Supported |
| [Blockscout](https://megaeth.blockscout.com) | https://megaeth.blockscout.com | Supported |
| [Explore Mega](https://exploremega.xyz) | https://exploremega.xyz | Supported |
| Explorer | Mainnet |
|---|---|
| [Etherscan](https://mega.etherscan.io) | Supported |
| [Blockscout](https://megaeth.blockscout.com) | Supported |
| [Explore Mega](https://exploremega.xyz) | Supported |

fixed some formatting issues 

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Review

The revamp is a substantial improvement — good coverage of mainnet partners across many new sections. Two previously-raised issues remain unaddressed (flagged in the open inline threads), plus a couple of new findings:

Still open from previous review rounds:

  1. Data & Analytics table (lines 68–79): Tool names are unlinked plain text and all URLs are bare <https://...> autolinks. GitBook will not render these as clickable links. Drop the URL column and hyperlink the tool names instead (see the open inline thread for the exact suggestion).

  2. Explorers table (lines 32–36): The URL column is redundant — the Explorer name in column 1 already links to the same URL. Bare autolinks also won't render as links in GitBook. Drop the URL column (see the open inline thread).

New issues:

  1. mega-evme link text (line 131): The display text mega-evme appears to be a typo for mega-evm — the linked GitHub repo is megaeth-labs/mega-evm.

  2. Mobula row extra spaces (line 59): Streams (WebSockets) has 5 spaces between "Streams" and "(WebSockets)" — should be one space.

  3. SUMMARY.md title mismatch: docs/SUMMARY.md still lists the page as Tooling & Infrastructure but the page H1 is now Tooling and Infrastructure. They should match.

Everything else looks good — heading hierarchy, frontmatter description, hint blocks, and table structure are all correct.

Comment thread docs/dev/tooling.md Outdated
| Tool | Type | Get Started | Mainnet |
|---|---|---|---|
| [Tenderly](https://tenderly.co) | Debugging, simulation, alerts | [Docs](https://docs.tenderly.co) | Planned |
| [mega-evme](https://github.com/megaeth-labs/mega-evm) | EVM debugger / tx replay | [Docs](https://github.com/megaeth-labs/mega-evm/blob/main/bin/mega-evme/README.md) | Supported |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mega-evme looks like a typo — the repository is megaeth-labs/mega-evm (no trailing e).

Suggested change
| [mega-evme](https://github.com/megaeth-labs/mega-evm) | EVM debugger / tx replay | [Docs](https://github.com/megaeth-labs/mega-evm/blob/main/bin/mega-evme/README.md) | Supported |
| [mega-evm](https://github.com/megaeth-labs/mega-evm) | EVM debugger / tx replay | [Docs](https://github.com/megaeth-labs/mega-evm/blob/main/bin/mega-evme/README.md) | Supported |

Comment thread docs/dev/tooling.md Outdated
|---|---|---|---|
| [Goldsky](https://goldsky.com) | [Docs](https://docs.goldsky.com/chains/megaeth) | [Subgraphs](https://docs.goldsky.com/chains/megaeth#subgraphs), [Mirror (Data Pipelines)](https://docs.goldsky.com/chains/megaeth#mirror) | Supported |
| [Envio](https://envio.dev/) | [Docs](https://docs.envio.dev/docs/HyperIndex/contract-import) | [HyperIndex (Subgraphs)](https://docs.envio.dev/blog/how-to-index-megaeth-data-using-envio#hyperindex), [HyperSync (Data Access)](https://docs.envio.dev/blog/how-to-index-megaeth-data-using-envio#hypersync) | Supported |
| [Mobula](https://mobula.io) | [Docs](https://mobula.io/chain-mega) | [Real-time Market & Wallet Data API](https://mobula.io/chain-mega), [Streams (WebSockets)](https://docs.mobula.io/indexing-stream/introduction) | Supported |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra spaces between "Streams" and "(WebSockets)".

Suggested change
| [Mobula](https://mobula.io) | [Docs](https://mobula.io/chain-mega) | [Real-time Market & Wallet Data API](https://mobula.io/chain-mega), [Streams (WebSockets)](https://docs.mobula.io/indexing-stream/introduction) | Supported |
| [Mobula](https://mobula.io) | [Docs](https://mobula.io/chain-mega) | [Real-time Market & Wallet Data API](https://mobula.io/chain-mega), [Streams (WebSockets)](https://docs.mobula.io/indexing-stream/introduction) | Supported |

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

SUMMARY.md out of sync: The page H1 was changed from Tooling & Infrastructure to Tooling and Infrastructure, but docs/SUMMARY.md line 37 still reads [Tooling & Infrastructure](dev/tooling.md). Update to match the new H1.

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Most previous feedback has been addressed. Four issues remain open in existing threads:

  1. Data & Analytics — tool names are unlinked plain text and URLs are bare (won't render as links in GitBook). Drop the URL column and link the names. (thread)

  2. Explorers — redundant URL column with bare <https://...> syntax. Drop the URL column; the linked Explorer name is sufficient. (thread)

  3. Developer Tooling, line 124 — tool name is mega-evme but the repo is mega-evm (no trailing e). (thread)

  4. Indexers, line 56 — extra spaces between "Streams" and "(WebSockets)" in the Mobula row. (thread)

Comment thread docs/dev/tooling.md
Comment on lines 73 to +74

| Provider | Get Started | Contract Addresses | Supported Feeds | Testnet (Carrot) | Mainnet |
| ----------------------------------------- | ------------------------------------------------------- | -------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------- | ---------------- | ------- |
| [Redstone](https://www.redstone.finance/) | [Docs](https://docs.redstone.finance/docs/introduction) | `0xbEb886F1A51803d7e9FD4514a91E89b81F4fdE58` | [Push Model (on-chain Feeds)](https://app.redstone.finance/app/feeds/?networks=6342&page=1&sortBy=popularity&sortDesc=false&perPage=32) | Supported | Planned |
| [Chainlink](https://chain.link) | [Docs](https://docs.chain.link/) | `0xfBFff08fE4169853F7B1b5Ac67eC10dc8806801d` | [Data Streams](https://docs.chain.link/data-streams/crypto-streams?page=1) | Supported | Planned |
| [Gelato](https://www.gelato.network) | [Docs](https://docs.gelato.network/web3-services/vrf) | — | [Verifiable Random Function](https://www.notion.so/gelato-network/Gelato-VRF-1cb69e877ab681ad8867ca621802022b) | Supported | — |
| [Pyth](https://www.pyth.network/) | [Docs](https://docs.pyth.network/price-feeds) | `0x2880aB155794e7179c9eE2e38200202908C17B43` | [Price Feeds](https://docs.pyth.network/price-feeds/contract-addresses/evm) | Supported | — |
## Bridges & Interoperability

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing --- separator before this section — every other section is preceded by --- but Bridges is not, making the layout inconsistent.

Suggested change
| Provider | Get Started | Contract Addresses | Supported Feeds | Testnet (Carrot) | Mainnet |
| ----------------------------------------- | ------------------------------------------------------- | -------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------- | ---------------- | ------- |
| [Redstone](https://www.redstone.finance/) | [Docs](https://docs.redstone.finance/docs/introduction) | `0xbEb886F1A51803d7e9FD4514a91E89b81F4fdE58` | [Push Model (on-chain Feeds)](https://app.redstone.finance/app/feeds/?networks=6342&page=1&sortBy=popularity&sortDesc=false&perPage=32) | Supported | Planned |
| [Chainlink](https://chain.link) | [Docs](https://docs.chain.link/) | `0xfBFff08fE4169853F7B1b5Ac67eC10dc8806801d` | [Data Streams](https://docs.chain.link/data-streams/crypto-streams?page=1) | Supported | Planned |
| [Gelato](https://www.gelato.network) | [Docs](https://docs.gelato.network/web3-services/vrf) || [Verifiable Random Function](https://www.notion.so/gelato-network/Gelato-VRF-1cb69e877ab681ad8867ca621802022b) | Supported ||
| [Pyth](https://www.pyth.network/) | [Docs](https://docs.pyth.network/price-feeds) | `0x2880aB155794e7179c9eE2e38200202908C17B43` | [Price Feeds](https://docs.pyth.network/price-feeds/contract-addresses/evm) | Supported ||
## Bridges & Interoperability
---
## Bridges & Interoperability

@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

4 threads from the previous review round are still open and unaddressed:

  • Data & Analytics — tool names are unlinked plain text and URLs are bare <https://...> (won't render as links in GitBook). Drop the URL column and link the names instead.
  • Explorers — redundant URL column with bare URLs. Drop the column and rely on the linked names.
  • Developer Toolingmega-evme display name looks like a typo; the suggested fix was mega-evm.
  • Indexers / Mobula — extra spaces between "Streams" and "(WebSockets)".

One new issue: the Bridges & Interoperability section is missing the --- separator that every other section has before it (inline comment added).

Signed-off-by: 0xSami-m <samimuduroglu1@gmail.com>
@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

The five open review threads are still unaddressed in this revision. Here's a quick summary of what remains:

  1. Data & Analytics table — tool names are unlinked plain text; drop the URL column and link the names instead (thread on line 68).
  2. Explorers table — redundant URL column; drop it and keep only the linked Explorer names (thread on line 33).
  3. Bridges section — missing --- separator before ## Bridges & Interoperability (thread on line 78).
  4. Developer Toolingmega-evme display name (thread on line 123); either confirm the tool is intentionally named mega-evme (the binary) or correct to mega-evm.
  5. Indexers / Mobula — extra spaces inside the link text [Streams (WebSockets)] (thread on line 56).

Please address these before merging.

@Troublor

Troublor commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

@claude can you help udpate this PR by fixing the CI failures?

@claude

claude Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Claude encountered an error —— View job


I'll analyze this and get back to you.

Troublor added 2 commits April 8, 2026 15:02
- Drop redundant URL column from Explorers table
- Link tool names and drop URL column in Data & Analytics table
- Add missing --- separator before Bridges section
- Fix mega-evme display name to mega-evm
@Troublor Troublor merged commit 6205a01 into main Apr 8, 2026
4 of 5 checks passed
@Troublor Troublor deleted the sami/mainnet-tooling-revamp branch April 8, 2026 08:49
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.

2 participants