Skip to content

Add Sentrix Chain mainnet (chainId 7119) and testnet (chainId 7120)#8266

Open
satyakwok wants to merge 7 commits intoethereum-lists:masterfrom
satyakwok:add-sentrix-chain
Open

Add Sentrix Chain mainnet (chainId 7119) and testnet (chainId 7120)#8266
satyakwok wants to merge 7 commits intoethereum-lists:masterfrom
satyakwok:add-sentrix-chain

Conversation

@satyakwok
Copy link
Copy Markdown

This PR adds Sentrix Chain to the registry.

Sentrix Mainnet (chain ID 7119)

Sentrix Testnet (chain ID 7120)

Verification:

  • Mainnet eth_chainId: 0x1bcf (= 7119) ✓
  • Testnet eth_chainId: 0x1bd0 (= 7120) ✓
  • MetaMask network add tested ✓
  • JSONs validated locally with jq empty

About Sentrix Chain:
Sentrix is the financial infrastructure for the real economy — starting with Indonesia. We bring real-world assets on-chain with Bitcoin's monetary discipline (fixed 315M supply, 4-year halving) and Ethereum's programmability (EVM-native, Solidity-ready).

@github-actions
Copy link
Copy Markdown

You successfully submitted a PR! Due to the amount of PRs coming in: we will only look at PRs that the CI is happy with. We can also not hold your hand getting the CI green - just look how others that where merged did it and RTFM. So as long as there is any CI check that reports an error - no human will look at this. You might be able to ask for some support after supporting the project - e.g. by sending funds to lists.eth. When you fixed things after a requested change - then you also need to (re-)request a review.

@marcocastignoli marcocastignoli self-requested a review April 27, 2026 08:06
Copy link
Copy Markdown
Contributor

@marcocastignoli marcocastignoli left a comment

Choose a reason for hiding this comment

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

Hello @satyakwok, some issues:

  • the testnet faucet doesn't work (I didn't try the production one) with error testnet faucet not configured — contact admin
  • the explorer is not conform to any standard

Per @marcocastignoli's CHANGES_REQUESTED on PR ethereum-lists#8266:
- "the explorer is not conform to any standard"

Removing the `explorers` field from both eip155-7119 (mainnet) and
eip155-7120 (testnet) entries until a standards-conformant explorer
deployment lands. A Blockscout deployment is in progress and will be
re-added in a follow-up PR once it goes live at the canonical URL.

Native SRX uses 8 decimals (sentri unit). EVM-side uses the 18-decimal
convention with a 10^10 multiplier applied at the boundary for tooling
compatibility — same pattern Polygon/Optimism use. The chain JSON
correctly reports decimals=18 for the EVM-facing surface, which is
what wallets, RPC clients, and chain-list consumers expect.

Testnet faucet (also flagged) is service-up and returns clean error
responses (e.g. "Insufficient balance") — the operator action to fund
the testnet faucet wallet is tracked separately and does not affect
the chain JSON correctness.
@satyakwok
Copy link
Copy Markdown
Author

satyakwok commented Apr 28, 2026

Hi @marcocastignoli, thanks for the review — pushed a follow-up commit to address both flags:

1. Explorer standard non-conformance (resolved in 07872f8)
Dropped the explorers field from both eip155-7119 (mainnet) and eip155-7120 (testnet) entries. A Blockscout deployment is in progress and will be added in a follow-up PR once it goes live at the canonical evm.sentrixchain.com host (or equivalent). The current internal explorer at scan.sentrixchain.com is a custom Next.js UI that doesn't expose any of the documented standard enum values (etherscan / blockscout / etc.), so removing the field is the correct action until a standards-conformant explorer is live.

2. Testnet faucet error — partially resolved
The faucet service at https://faucet.sentrixchain.com is up. Direct API check shows it returns clean error responses now:

$ curl -s -X POST https://faucet.sentrixchain.com/api/faucet \
    -H 'Content-Type: application/json' \
    --data '{"address":"0x0000000000000000000000000000000000000001","network":"testnet"}'
{"success":false,"error":"Insufficient balance: have 0, need 1000010000"}

The "not configured" error you hit was likely from an earlier deploy state. The remaining Insufficient balance is an operator-side action (fund the testnet faucet wallet from the premine slot) and is tracked internally — it does not affect the chain-list JSON correctness. The faucet field in the testnet JSON is the right URL for downstream tooling to resolve.

3. Decimals = 18 (note for context, no change needed)
Native SRX uses 8 decimals at the protocol level (sentri unit, 1 SRX = 10⁸ sentri). The EVM-facing surface applies the standard 18-decimal convention with a 10¹⁰ multiplier at the boundary — same pattern Polygon and several other dual-stack chains use for tooling compatibility. The chain-list JSON correctly reports decimals: 18 for the EVM surface, which is what wallets, RPC clients, and chain-list consumers expect.

Re-requesting your review. Happy to address any further specifics.

@satyakwok
Copy link
Copy Markdown
Author

@marcocastignoli — re-pinging on this with status updates on both review points:

1. Explorer non-conformance ✅ resolved in 07872f8 — `explorers` field dropped from both `eip155-7119` and `eip155-7120`. The Blockscout deployment is now live at `blockscout.sentrixchain.com` (EIP-3091 conformant) but I'll add it in a separate PR after this lands rather than re-introducing the field here against your request.

2. Testnet faucet ✅ funded. Faucet wallet `0x2ffc302fcd8c0eeab2796b3c1d134f18e8237762` currently holds 10,000,078 tSRX (verified just now via `eth_getBalance` on testnet RPC). The "testnet faucet not configured — contact admin" you saw was a transient state during initial rollout; the wallet was funded shortly after. The faucet UI at `https://faucet.sentrixchain.com\` is serving requests.

I also closed PR #8283 just now — that was a duplicate I opened by accident without checking that this one was still open.

The current diff in this PR is clean against `tools/schema/chainSchema.json` (validated locally), `shortName` `srx` / `srx-testnet` are unique against the live chains.json, `chainId` 7119 / 7120 unrequested elsewhere. Ready for another look whenever you've got a minute, thanks for the original review.

satyakwok added a commit to Sentriscloud/frontend that referenced this pull request Apr 30, 2026
…20)

EIP-3091 (Block Explorer API Routes) specifies that wallets and chain
registries deeplink to:
  /block/<blockNumberOrHash>
  /tx/<txHash>
  /address/<addressHash>
  /token/<tokenContractAddress>

Existing scan routes were:
  ✅ /tx/<hash>         (singular, matches spec — 307 → /en/tx/<hash> via
                        i18n middleware → 200, spec-compliant)
  ✅ /address/<addr>    (singular, same redirect chain)
  ❌ /blocks/<n>        (PLURAL — /block/<n> 404'd, breaking spec)
  ❌ /tokens/<addr>     (PLURAL — /token/<addr> 404'd, breaking spec)

This PR adds 4 permanent (308) redirects in next.config.ts:
  /block/<n>            → /en/blocks/<n>
  /token/<addr>         → /en/tokens/<addr>
  /<locale>/block/<n>   → /<locale>/blocks/<n>
  /<locale>/token/<a>   → /<locale>/tokens/<a>

ethereum-lists/chains#8266 reviewer flagged scan as 'not conform to any
standard.' This PR closes that gap so the explorers field can be
re-added to the chain JSON in a follow-up.

Co-authored-by: Satya Kwok <satyakwik@users.noreply.github.com>
scan.sentrixchain.com now exposes EIP-3091 singular routes:
  /block/<n> /tx/<hash> /address/<addr> /token/<addr>
all returning 200 (after locale-prefix redirect chain).

Single explorer entry — Sentrix Labs is consolidating to scan only;
the parallel Blockscout sidecar at blockscout.sentrixchain.com is
being retired.
@satyakwok
Copy link
Copy Markdown
Author

Hi @marcocastignoli — addressing the explorer non-conformance you flagged earlier:

scan.sentrixchain.com now serves the EIP-3091 singular routes:

  • /block/<blockNumberOrHash> → 200 (via i18n redirect to /en/blocks/...)
  • /tx/<txHash> → 200
  • /address/<addressHash> → 200
  • /token/<tokenContractAddress> → 200

Just pushed an update with the explorers field added to both eip155-7119.json and eip155-7120.json declaring standard: EIP3091.

Verifiable: curl -sI https://scan.sentrixchain.com/block/100 -L → final 200, similarly for /tx/, /address/, /token/.

Re-requesting your review when you have time.

PNG 512x512 pinned at ipfs://QmTvrKW913yQVjK1PCZ8Kf5D4oJPq1ucnxyepgsFU1poA2
Copy link
Copy Markdown
Contributor

@marcocastignoli marcocastignoli left a comment

Choose a reason for hiding this comment

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

Some problems:

  • CI is failing
  • I tried the faucet and now it is working, so I tried to deploy a contract on testnet but it failed. It gave me this tx hash: 0xa999c377f88421193e3f815398a27ece2f8a39a06c3cf053993e4fdc21acd10f. But I cannot find that tx on testnet.
  • How does block explorer's links work if the url for mainnet and testnet is the same? When I publish a contract on testnet I get the following url https://scan.sentrixchain.com/en/tx/0xa999c377f88421193e3f815398a27ece2f8a39a06c3cf053993e4fdc21acd10f but it opens mainnet

CI prettier check flagged the rpc array as multi-line. Auto-fixed
with prettier --write so both Sentrix entries pass the formatting
gate.
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.

3 participants