Skip to content

Switch pubnet node data from Stellarbeat to Radar API property names#365

Merged
marta-lokhova merged 2 commits intomainfrom
copilot/update-stellarbeat-to-radar
Feb 20, 2026
Merged

Switch pubnet node data from Stellarbeat to Radar API property names#365
marta-lokhova merged 2 commits intomainfrom
copilot/update-stellarbeat-to-radar

Conversation

Copy link
Contributor

Copilot AI commented Feb 20, 2026

Stellarbeat is reaching EOL; the replacement is the Radar API (radar.withobsrvr.com), which uses radar_* property names instead of sb_*.

Changes

  • sample-network-data.json: Rename all sb_* fields to radar_* (sb_homeDomainradar_homeDomain, sb_geoDataradar_geoData, sb_isValidatingradar_isValidating, etc.). This file drives the F# JSON type provider, so its keys determine the generated property names.

  • StellarNetworkData.fs: Update all generated property accesses to match:

    • SbHomeDomainRadarHomeDomain
    • SbGeoDataRadarGeoData
    • SbIsValidatingRadarIsValidating
    • Inline JSON literal and comment updated accordingly
  • doc/measuring-transaction-throughput.md: Update JSON example snippet and description text: sb_homeDomainradar_homeDomain.

  • topologies/theoretical-max-tps.json: Rename all sb_homeDomain fields to radar_homeDomain.

// Before
PubnetNode.Parse(""" [{ "publicKey": "%s", "sb_homeDomain": "home.domain.%d" }] """)
n.SbHomeDomain.IsSome
n.SbGeoData.Value.Latitude

// After
PubnetNode.Parse(""" [{ "publicKey": "%s", "radar_homeDomain": "home.domain.%d" }] """)
n.RadarHomeDomain.IsSome
n.RadarGeoData.Value.Latitude

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update Stellarbeat integration to support Radar API Switch pubnet node data from Stellarbeat to Radar API property names Feb 20, 2026
Copilot AI requested a review from marta-lokhova February 20, 2026 20:10
@marta-lokhova
Copy link
Contributor

Copy link
Contributor Author

Copilot AI commented Feb 20, 2026

@copilot you did not update [doc/measuring-transaction-throughput.md](https://github.com/stellar/supercluster/blob/327a928b54a94cb91735ee762fca04fe198979c8/doc/measuring-transaction-throughput.md?plai...

Fixed in 047d010 — updated sb_homeDomainradar_homeDomain in both doc/measuring-transaction-throughput.md (JSON example and description text) and topologies/theoretical-max-tps.json.

@marta-lokhova marta-lokhova marked this pull request as ready for review February 20, 2026 22:29
Copilot AI review requested due to automatic review settings February 20, 2026 22:29
Copilot AI and others added 2 commits February 20, 2026 14:30
Co-authored-by: marta-lokhova <9428003+marta-lokhova@users.noreply.github.com>
Co-authored-by: marta-lokhova <9428003+marta-lokhova@users.noreply.github.com>
@marta-lokhova marta-lokhova force-pushed the copilot/update-stellarbeat-to-radar branch from 047d010 to 67dc4c4 Compare February 20, 2026 22:30
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR renames all Stellarbeat API property names (sb_*) to Radar API property names (radar_*) across the codebase, as Stellarbeat is reaching end-of-life and being replaced by the Radar API at radar.withobsrvr.com.

Changes:

  • Updated JSON sample file to use radar_* property names, which drives F# type provider code generation
  • Updated all F# code to use generated properties with new names (RadarHomeDomain, RadarGeoData, RadarIsValidating)
  • Updated documentation and topology files to reflect the new property naming convention

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/FSLibrary/json-type-samples/sample-network-data.json Renamed all sb_* fields to radar_* in JSON type provider sample file
src/FSLibrary/StellarNetworkData.fs Updated all generated property accesses and inline JSON to use Radar* naming
doc/measuring-transaction-throughput.md Updated JSON example and description text to use radar_homeDomain
topologies/theoretical-max-tps.json Renamed sb_homeDomain to radar_homeDomain in topology configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@drebelsky drebelsky left a comment

Choose a reason for hiding this comment

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

Looks like it updates all the references to stellarbeat in this repo.

@marta-lokhova marta-lokhova merged commit 7e3428c into main Feb 20, 2026
3 checks passed
@marta-lokhova marta-lokhova deleted the copilot/update-stellarbeat-to-radar branch February 20, 2026 23:04
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.

4 participants