Skip to content

docs(drift): draft from internal source#341

Open
pedromcunha wants to merge 1 commit into
mainfrom
docs-drift/src-629f06161c-35a8d92
Open

docs(drift): draft from internal source#341
pedromcunha wants to merge 1 commit into
mainfrom
docs-drift/src-629f06161c-35a8d92

Conversation

@pedromcunha

Copy link
Copy Markdown
Member

Drafted from an internal source repo, requested by @matirelay.

This PR was auto-generated by the reusable-docs-draft-pr workflow from a Claude-drafted update in the source repo. It is opened as a draft — please review, edit as needed, and mark ready for review before merging.

Review checklist

  • Content matches the behavior shipped in the source PR
  • No internal-only details (service names, file paths, unreleased names, security internals) leaked into the draft
  • Conforms to CLAUDE.md style rules
  • Cross-references and internal links resolve
  • Any Needs human input items from the source comment are addressed

Generated by reusable-docs-draft-pr.yml in relayprotocol/agents-config.

Requested by @matirelay via /publish-docs-pr
@mintlify

mintlify Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
unevenlabs 🟢 Ready View Preview Jun 18, 2026, 2:33 PM

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

@pedromcunha pedromcunha marked this pull request as ready for review June 19, 2026 12:39
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

Claude finished @pedromcunha's task in 47s —— View job


Code Review — PR #341

  • Read PR diff
  • Check for orphaned references to removed chains
  • Verify style/consistency
  • Post review

This PR removes two chains from the protocol docs: perennial (chain id 1424) and soon (a solana-vm chain). The edits span three files.

Summary of changes

File Change
references/protocol/addresses.mdx Removes perennial and soon rows from the Allocator/Security Council address table
references/protocol/depository/addresses.mdx Removes perennial and soon rows from the Depository address table
references/protocol/components/allocator.mdx Drops Soon from the Solana Payload Builder chains list

Assessment ✅

The change is clean, consistent, and complete:

  • Both address tables stay in sync. addresses.mdx and depository/addresses.mdx both had identical perennial/soon rows, and both are updated. Good — these two tables should never diverge.
  • The allocator cross-reference is handled. soon was also listed under the Solana Payload Builder, and that mention is removed too, so the docs don't leave a dangling reference to a now-undocumented chain.
  • No orphaned references remain. I searched the full docs tree for perennial and the soon chain entry — the only remaining soon hits are the ordinary English word ("as soon as…"), not the chain. Nothing else points at either removed chain.
  • Table formatting is preserved — column alignment and the surrounding rows are untouched.

Notes (non-blocking)

  • These files live under references/protocol/, which AGENTS.md lists as out of scope for the v1 style guide ("Edit them by pattern-matching against existing pages"). The edits correctly pattern-match the existing table rows, so there's nothing to flag against the style rules.
  • This is a content removal driven by an upstream source change. The only thing worth confirming before merge is the review checklist item Add Just-In-Time gas guide #1 from the PR body: that perennial and soon are genuinely being retired from the Relay protocol (and not, say, renamed). That's a product-truth check the human reviewer should sign off on — the docs change itself is correct given that premise.

Verdict: Looks good to me. No code changes needed.
docs-drift/src-629f06161c-35a8d92

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