Skip to content

Revert "OPEN-7: Route/latency tools — region wiring, ICE policy, richer region pings"#175

Merged
zortos293 merged 1 commit intomainfrom
revert-174-cursor/OPEN-7-network-route-optimization-a6c4
Mar 31, 2026
Merged

Revert "OPEN-7: Route/latency tools — region wiring, ICE policy, richer region pings"#175
zortos293 merged 1 commit intomainfrom
revert-174-cursor/OPEN-7-network-route-optimization-a6c4

Conversation

@zortos293
Copy link
Copy Markdown
Collaborator

@zortos293 zortos293 commented Mar 31, 2026

Reverts #174


Note

Medium Risk
Touches WebRTC connection configuration and region selection/ping behavior, which can affect stream connectivity and latency. Changes are mostly removal/simplification, but regressions are possible for users relying on relay-only ICE or detailed ping diagnostics.

Overview
Reverts the prior route/latency tools work by removing iceTransportPolicy from shared settings/types, the renderer settings UI, and the GfnWebRtcClient RTCPeerConnection configuration.

Simplifies region latency probing: main process PING_REGIONS now runs 3 TCP connect samples and returns only an averaged pingMs (dropping min/max/jitter/sample metadata), and the settings page ping cache/UI is updated accordingly (including removing jitter display).

Also simplifies streaming base URL derivation to always use the selected provider’s streamingServiceUrl, and updates package-lock.json metadata/version fields.

Written by Cursor Bugbot for commit 9c30b5c. This will update automatically on new commits. Configure here.

@linear
Copy link
Copy Markdown

linear bot commented Mar 31, 2026

Issue reopened: OPEN-7 try to add a thing like exitlag lagofast

@zortos293 zortos293 merged commit b9d446d into main Mar 31, 2026
6 checks passed
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

return base;
}, [selectedProvider, settings.region]);
return selectedProvider?.streamingServiceUrl ?? "";
}, [selectedProvider]);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Region selection no longer affects routing

Medium Severity

effectiveStreamingBaseUrl now always uses selectedProvider.streamingServiceUrl and ignores settings.region. The Region picker in SettingsPage.tsx still lets users choose specific regions, but that value is no longer applied when calling streaming APIs, so manual region selection silently has no effect.

Fix in Cursor Fix in Web

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