Skip to content

docs(relayer): document SUI_GRPC_URL gRPC write-path option#481

Merged
reemsabawi-mystenlabs merged 4 commits into
devfrom
claude/bulk-resolve-linear-issues-jbevl2
Jul 21, 2026
Merged

docs(relayer): document SUI_GRPC_URL gRPC write-path option#481
reemsabawi-mystenlabs merged 4 commits into
devfrom
claude/bulk-resolve-linear-issues-jbevl2

Conversation

@reemsabawi-mystenlabs

Copy link
Copy Markdown
Collaborator

Description

Documents the optional SUI_GRPC_URL environment variable added in #355, which opts the relayer write path (Walrus register and certify, SEAL, and Enoki build) into gRPC instead of JSON-RPC ahead of the Sui JSON-RPC sunset on 2026-07-31. The option is off by default, so leaving it empty keeps the existing JSON-RPC behavior. The blob query and restore path continues to use JSON-RPC index methods and is not affected.

Covers the four operator-facing pages so the behavior is consistent across the docs:

  • Environment Variables reference (reference/environment-variables.md) — new SUI_GRPC_URL row in the self-hosted relayer table plus a note (BEDU-816).
  • Configuration reference (reference/configuration.md) — a "Relayer configuration" pointer to the write-path transport option, since that page otherwise covers SDK config shapes (BEDU-813).
  • Relayer Overview (relayer/overview.md) — a "Sui RPC transport" subsection (BEDU-814).
  • Self-Hosting (relayer/self-hosting.md) — a "Sui RPC Transport (gRPC write path)" subsection with an example and the JSON-RPC caveat (BEDU-815).

Test plan

  • Sourced every claim from feat(relayer): gRPC write path for the JSON-RPC sunset (2026-07-31) #355 and the merged code (services/server/scripts/sidecar/config.ts, clients.ts, walrus-query.ts, and services/server/src/main.rs): the opt-in default, which write-path operations move to gRPC, the startup validation of the URL, and the fact that the blob query and restore path stays on JSON-RPC index methods.
  • Documentation-only change; no runtime code affected.

🤖 Generated with Claude Code

https://claude.ai/code/session_017oS4PdhkXAzbfhJx2ZhRvh


Generated by Claude Code

#355 adds an optional SUI_GRPC_URL environment variable
that opts the relayer write path (Walrus register/certify, SEAL, Enoki
build) into gRPC instead of JSON-RPC, ahead of the Sui JSON-RPC sunset.
It is off by default; the blob query/restore path stays on JSON-RPC.

Documents it across the operator-facing pages:

- Environment Variables reference: new SUI_GRPC_URL row + note [BEDU-816]
- Configuration reference: relayer transport pointer [BEDU-813]
- Relayer Overview: Sui RPC transport section [BEDU-814]
- Self-Hosting: Sui RPC Transport (gRPC write path) section [BEDU-815]

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017oS4PdhkXAzbfhJx2ZhRvh

@jessiemongeon1 jessiemongeon1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the docs repeatedly say the blob query / restore path stays on JSON-RPC when SUI_GRPC_URL is set. That does not match current code.

services/server/scripts/sidecar/routes/walrus-query.ts switches /walrus/query-blobs to gRPC via listOwnedObjects + getDynamicField when SUI_GRPC_URL is set, and /api/restore calls that sidecar route through query_blobs_by_owner.

…e path

Review feedback (jessiemongeon1): the docs claimed the blob query/restore
path stays on JSON-RPC when SUI_GRPC_URL is set. That is wrong. Per
services/server/scripts/sidecar/routes/walrus-query.ts and clients.ts, when
SUI_GRPC_URL is set the query/restore path also runs on gRPC
(listOwnedObjects + getDynamicField); the JSON-RPC getOwnedObjects/
getDynamicFieldObject/queryTransactionBlocks path is the fallback used only
when SUI_GRPC_URL is unset.

Corrected across the environment-variables, configuration, relayer overview,
and self-hosting pages to describe SUI_GRPC_URL as a single reversible switch
that moves both the write path and the query/restore path to gRPC.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017oS4PdhkXAzbfhJx2ZhRvh

Copy link
Copy Markdown
Collaborator Author

You're right, thanks for catching this. I checked routes/walrus-query.ts and clients.ts: when SUI_GRPC_URL is set, isGrpcClient is true and /walrus/query-blobs (and /api/restore through query_blobs_by_owner) runs on gRPC listOwnedObjects + getDynamicField. The JSON-RPC getOwnedObjects/getDynamicFieldObject/suix_queryTransactionBlocks path is the fallback used only when SUI_GRPC_URL is unset — clients.ts even documents suiJsonRpcClient as "only used by the blob query/restore path as the fallback when SUI_GRPC_URL is unset."

Fixed in 3c52a1a: all four pages now describe SUI_GRPC_URL as a single reversible switch that moves both the write path and the blob query/restore path to gRPC, rather than claiming query/restore stays on JSON-RPC. PTAL.


Generated by Claude Code

@jessiemongeon1

jessiemongeon1 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Style Guide Audit

All 4 file(s) pass the style guide audit.

Style-guide audit flagged title-cased 'Transport' in the self-hosting heading.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017oS4PdhkXAzbfhJx2ZhRvh

@jessiemongeon1 jessiemongeon1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A few style guide issues and inconsistencies - are you using the Sui Style Guide skill in your Claude configuration?

Comment thread docs/reference/configuration.md Outdated
Comment thread docs/reference/environment-variables.md Outdated
Comment thread docs/relayer/overview.md Outdated
Comment thread docs/relayer/self-hosting.md Outdated
Comment thread docs/relayer/self-hosting.md Outdated
…inline single bullet

- Normalize SEAL -> Seal (product name) in relayer/config docs
- Correct Sui JSON-RPC sunset to July 2026 (was 2026-07-31)
- Inline single-item SUI_GRPC_URL bullet into prose per review

Copy link
Copy Markdown
Collaborator Author

Thanks Jessie, all applied in 01d08b4:

  • Seal casing: standardized SEALSeal across the four touched pages (configuration.md, environment-variables.md, relayer/overview.md, relayer/self-hosting.md).
  • Sunset date: corrected 2026-07-31July 2026 everywhere.
  • Single bullet: inlined the lone SUI_GRPC_URL bullet into prose in both configuration.md and self-hosting.md.

One thing worth flagging: the rest of this repo overwhelmingly uses uppercase SEAL in prose (for example openclaw/overview.md, llms-full.txt). I've moved my touched pages to Seal to match the product name and the Sui style guide, but that leaves the repo internally inconsistent. Happy to open a separate PR to normalize the remaining SEAL prose references to Seal repo-wide if you'd like — kept it out of this PR to stay scoped.

Yes, I'm using the Sui Style Guide skill; these date/casing slips were mine, not the tooling. I'll keep them off the next batch.


Generated by Claude Code

@reemsabawi-mystenlabs
reemsabawi-mystenlabs merged commit bbc37e5 into dev Jul 21, 2026
9 checks passed
harrymove-ctrl pushed a commit that referenced this pull request Jul 22, 2026
…C_URL relayer docs, #482 agent-memory developer guides)
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