Skip to content

fix: use DNS for registry/beacon defaults in SDK (PILOT-330)#7

Open
matthew-pilot wants to merge 1 commit into
mainfrom
openclaw/pilot-330-20260530-035123
Open

fix: use DNS for registry/beacon defaults in SDK (PILOT-330)#7
matthew-pilot wants to merge 1 commit into
mainfrom
openclaw/pilot-330-20260530-035123

Conversation

@matthew-pilot
Copy link
Copy Markdown
Collaborator

What

Replace hardcoded IP 34.71.57.205 with DNS registry.pilotprotocol.network for DEFAULT_REGISTRY and DEFAULT_BEACON constants.

Why

If the production server moves, every SDK-using application breaks until the SDK is updated. Using DNS makes the address mutable.

Changes

  • src/runtime.ts: 2 lines (+2/-2)

Ticket

🔗 https://vulturelabs.atlassian.net/browse/PILOT-330

Replace hardcoded IP 34.71.57.205 with registry.pilotprotocol.network
for DEFAULT_REGISTRY and DEFAULT_BEACON constants.
@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

🦾 Matthew PR Status — #7

Title: fix: use DNS for registry/beacon defaults in SDK (PILOT-330)
Status: OPEN | Mergeable: MERGEABLE
Author: @matthew-pilot (matthew-pilot bot)
Created: 2026-05-30T03:51:41Z
Branch: openclaw/pilot-330-20260530-035123main
Changes: +2/-2 across 1 file

Tickets

🔗 PILOT-330

Labels

None

Files Changed

  • src/runtime.ts (+2)

Next Actions

  • Explain: /pr explain #7 — detailed analysis
  • Canary retry: /pr retry-canary #7 (if CI failed)
  • Fix & update: /pr fix #7 <instructions>
  • Rebase: /pr rebase #7
  • Close: /pr close #7 <reason>

🦾 Auto-generated status check by matthew-pr-worker

@matthew-pilot
Copy link
Copy Markdown
Collaborator Author

🦾 Matthew PR Explain — #7

What this PR does

fix: use DNS for registry/beacon defaults in SDK (PILOT-330)

Scope

  • Files: 1 file
  • Delta: +2/-2 lines
  • Labels: none
  • Mergeable: MERGEABLE

Tickets

🔗 PILOT-330

Files

  • src/runtime.ts (+2/-2)

Review Notes

  • This is an automated code-maintenance PR from matthew-pilot
  • Operator review required before merge
  • Check CI status and canary results above

🦾 Auto-generated explain by matthew-pr-worker

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.

1 participant