Skip to content

docs(node): retire the random peer list in favour of the Sei Labs seeds - #96

Merged
alexander-sei merged 1 commit into
mainfrom
monty/plt-816-retire-random-peer-list
Oct 12, 2026
Merged

alexander-sei merged 1 commit into
mainfrom
monty/plt-816-retire-random-peer-list

Conversation

@monty-sei

Copy link
Copy Markdown
Collaborator

Problem

snippets/random-peers.jsx serves 17 hardcoded community peer addresses, and most of them are gone. Dialling every entry today:

responding:  6/17
refused:     2
timed out:   9

That snippet is rendered on the node setup guide and the troubleshooting guide, so an operator following either one is copying a list that is roughly two thirds dead. It reads as a broken network rather than a stale page, and the troubleshooting guide in particular offers it as the fix for peer handshake failed, which is the error a dead peer produces.

There is also a contradiction on the setup page. It tells you peer discovery needs no manual configuration, then immediately offers the dead list above it, plus a sed snippet writing them to persistent-peers, which is the wrong field for seeds.

Change

Operators do not need a peer list any more. Since v6.6.2, seid init writes the Sei Labs seeds into bootstrap-peers for well-known public networks, so a fresh node bootstraps with no further configuration.

  • Replaces random-peers.jsx with seeds.jsx, serving the three Sei Labs seeds per network in the same shape, so both pages keep their copy button
  • Folds the setup page's peer content into one Peer discovery section, and drops the persistent-peers snippet
  • Points the troubleshooting step at the seeds, noting that init will not overwrite a bootstrap-peers value you have already set
  • Corrects max-packet-msg-payload-size in the config.toml example, which read 10240 against an actual default of 1000000

Verification

The six addresses match app/seeds/seeds.go in sei-chain v6.7.1 exactly, and all six complete a P2P handshake:

pacific-1/prod   SPEAKS      atlantic-2/prod   SPEAKS
pacific-1/euw1   SPEAKS      atlantic-2/euw1   SPEAKS
pacific-1/use2   SPEAKS      atlantic-2/use2   SPEAKS

Other [p2p] values in the example were checked against DefaultP2PConfig() and the config.toml template at v6.7.1 and are correct as written, max-outbound-connections included.

Notes

llms-full.txt still contains the old snippet. Left alone deliberately: AGENTS.md says not to hand-edit the generated .txt files, and the weekly regenerate-llms.yml workflow rebuilds them from the deployed site.

Supersedes #57, which predates the seed documentation already on main and had fallen into conflict.

The random peer snippet shipped 17 hardcoded community addresses, and
only 6 of them still answer: 2 refuse the connection outright and 9 time
out. Operators following the node guide were copying a list that is
roughly two thirds dead, which looks like a broken network rather than a
stale document.

They no longer need it. Since v6.6.2, `seid init` writes the Sei Labs
seeds into bootstrap-peers for well-known public networks, so peer
discovery works with no manual configuration. index.mdx already said so
in the init block while still offering the dead list above it, alongside
a persistent-peers snippet that is the wrong field for seeds. Those are
now one coherent section.

Replaces the snippet with one that serves the seeds, in the same shape,
so pages keep their copy button. The addresses match app/seeds in
sei-chain v6.7.1 exactly, and all six were verified to complete a P2P
handshake.

Also corrects max-packet-msg-payload-size in the config.toml example,
which read 10240 against an actual default of 1000000.

llms-full.txt still carries the old snippet, deliberately left alone:
AGENTS.md says not to hand-edit it, and the weekly regeneration workflow
picks it up from the deployed site.
@mintlify

mintlify Bot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
sei-docs 🟢 Ready View Preview Oct 11, 2026, 11:14 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@cursor

cursor Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation and a docs-only React snippet swap; no runtime or application code changes.

Overview
Replaces the random community peer list docs snippet with Sei Labs seed nodes and aligns setup/troubleshooting guidance with v6.6.2+ behavior where seid init fills bootstrap-peers automatically.

snippets/random-peers.jsx is removed and snippets/seeds.jsx is added, exposing three regional seeds per mainnet/testnet with copy-to-clipboard output as bootstrap-peers (not persistent-peers). The node setup guide gains a dedicated Peer discovery section: no more sed into persistent-peers or shuffled dead peers; manual seed values are optional for older inits.

Troubleshooting now tells operators to reset to those seeds for handshake/ID mismatches, with a note that re-init will not overwrite existing bootstrap-peers.

Technical reference fixes the config.toml example: max-packet-msg-payload-size 10240 → 1000000 to match the real default.

Reviewed by Cursor Bugbot for commit 8ecac82. Bugbot is set up for automated code reviews on this repo. Configure here.

@seidroid seidroid Bot left a comment

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.

This PR swaps the mostly-dead community peer list for the Sei Labs seeds, fixes the persistent-peers vs bootstrap-peers mix-up, and corrects the max-packet-msg-payload-size default. It is clean overall, with only small non-blocking notes.

Findings: 0 blocking | 7 non-blocking | 3 posted inline

Blockers

  • None at the file/PR level.

Non-blocking

  • The Cursor second-opinion review produced no output. The Codex review reported no material issues.
  • REVIEW_GUIDELINES.md is empty, so this review applies only AGENTS.md conventions.
  • I found no remaining references to random-peers.jsx / RandomPeers outside the generated llms-full.txt. Leaving that file for the weekly regeneration matches AGENTS.md.
  • The format='bash' branch, which outputs SEEDS="...", is no longer used by any caller. Consider dropping it, or making toml the default, so the snippet doesn't suggest a SEEDS shell variable that nothing reads.
  • 3 suggestion(s)/nit(s) flagged inline on specific lines.

Comment thread node/troubleshooting.mdx
addresses, so a stale or mismatched node ID is the usual cause of this error:

<RandomPeers network="mainnet" format="toml" />
<Seeds network="mainnet" format="toml" />

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.

[suggestion] This step only shows mainnet seeds. A Sei Testnet operator who hits peer handshake failed would copy pacific-1 seeds into an atlantic-2 node. Consider adding the testnet block with labels, as node/index.mdx does: - Sei Mainnet (pacific-1): / - Sei Testnet (atlantic-2):.

Comment thread snippets/seeds.jsx
className="inline-flex items-center gap-1 px-2 py-1 rounded-md text-[11px] font-semibold border border-neutral-300 dark:border-neutral-600 bg-white/80 dark:bg-neutral-800/80 text-neutral-700 dark:text-neutral-200 hover:bg-white dark:hover:bg-neutral-800 transition-colors">
{copied ? (
<>
<CheckIcon className="text-green-600" />

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.

[nit] The old snippet used text-green-600 dark:text-green-400 for the Copied check icon. The dark: variant was dropped here, so the icon has lower contrast in dark mode. Consider restoring it.

Comment thread snippets/seeds.jsx
<pre
className="m-0 p-3 rounded-md bg-neutral-100 dark:bg-neutral-800 text-sm opacity-70"
style={{ fontFamily: 'var(--sei-font-mono)' }}>
No seeds configured for network “{network}”.

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.

[nit] This line uses curly quotes (“ ”), while the rest of the codebase uses straight quotes. Purely cosmetic.

@alexander-sei
alexander-sei merged commit 0d05dda into main Oct 12, 2026
18 of 19 checks passed
@alexander-sei
alexander-sei deleted the monty/plt-816-retire-random-peer-list branch October 12, 2026 09:10

This branch was successfully deployed

1 active deployment
staging — 8ecac821 Deployed Oct 11, 2026 by mintlify[bot]
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