Skip to content

fix(sdk): update default serverUrl to official mainnet relayer endpoint - #666

Merged
ducnmm merged 1 commit into
devfrom
fix/gh-657-sdk-default-server-url
Aug 18, 2026
Merged

fix(sdk): update default serverUrl to official mainnet relayer endpoint#666
ducnmm merged 1 commit into
devfrom
fix/gh-657-sdk-default-server-url

Conversation

@ducnmm

@ducnmm ducnmm commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Resolves #657

Summary

Updates the default serverUrl in the TypeScript SDK (MemWal and MemWalManual) from legacy https://relayer.memwal.ai/ to the official production endpoint https://relayer.memory.walrus.xyz.

Changes

  • Updated fallback serverUrl in packages/sdk/src/memwal.ts and packages/sdk/src/manual.ts.
  • Updated TSDoc comments and type definitions in packages/sdk/src/types.ts.
  • Updated environment variable documentation in docs/reference/environment-variables.md.
  • Updated mismatch detection in apps/app/src/pages/Dashboard.tsx.
  • Added unit test packages/sdk/test/default-server-url.test.mjs to assert default serverUrl behavior for MemWal and MemWalManual.

Verification

  • pnpm --filter @mysten-incubation/memwal test (33/33 tests pass)
  • node scripts/check-docs-freshness.mjs (passes)

@jessiemongeon1

Copy link
Copy Markdown
Collaborator

Style Guide Audit

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

@HoangDucBach HoangDucBach 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.

LGTM ✅

NIT: I'm a bit worried the default host is hardcoded in ~5 places across sdk, mcp and status. Every domain change means touching all of them again, which is roughly how this issue came up. If there's an intentional reason to keep them separate that's fine, otherwise I think we should extract a shared constant.

@ducnmm
ducnmm merged commit 2ca2d95 into dev Aug 18, 2026
11 checks passed
@ducnmm ducnmm mentioned this pull request Aug 18, 2026
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.

Production default serverUrl mismatches documentation

3 participants