Skip to content

fix(ui): refresh latency settings summary on return - #268

Merged
yiguodev merged 3 commits into
YuanDevTeam:dev-26.11.3-1from
senzyo:dev-26.11.3-1
Nov 8, 2026
Merged

yiguodev merged 3 commits into
YuanDevTeam:dev-26.11.3-1from
senzyo:dev-26.11.3-1

Conversation

@senzyo

@senzyo senzyo commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Saving a new latency-probe URL or timeout now updates the speed-test summary on the Xray page. Cancelling retains the saved values.

Close latency settings
  await navigation result
  read persisted URL and timeout
  refresh the summary while the controller is active

The regression tests mock the macOS capability channel. Existing DNS assertions now match managed TCP resolvers and rewriteNetwork: tcp; explicit DNS and saved-configuration preservation checks remain intact.

Evidence

  • Before: The save regression fails on the base because the summary retains the previous values; cancel passes.
    After: Both save/cancel regressions pass with the production fix and an isolated native-channel mock.
  • macOS: all 81 Xray tests pass; the complete suite passes with 2,288 tests passed and 5 skipped.
  • Formatting, analysis, dependency-layer, native model contract and whitespace checks pass locally.
  • Validate CI passes, including source generation, analysis and the full Linux suite.
  • No device or release-build validation.

Merge Danger

Door: Two-way

No data migration or persistence changes.

Blast Radius: UI

@senzyo

senzyo commented Nov 7, 2026

Copy link
Copy Markdown
Contributor Author

All 23 CI failures also reproduce on base commit 315603a in an isolated Windows run without this PR. They are pre-existing DNS assertion mismatches, unrelated to this latency-summary fix. Both new save/cancel regression tests pass in Linux CI.

@yiguodev
yiguodev merged commit 4dda684 into YuanDevTeam:dev-26.11.3-1 Nov 8, 2026
1 check passed
@senzyo
senzyo deleted the dev-26.11.3-1 branch November 8, 2026 07:03
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.

2 participants