Skip to content

Update Xray JSON completions and the managed FakeDNS IPv6 pool - #237

Merged
yiguodev merged 4 commits into
mainfrom
feat-xray-core-integration
Oct 22, 2026
Merged

yiguodev merged 4 commits into
mainfrom
feat-xray-core-integration

Conversation

@yiguodev

Copy link
Copy Markdown
Contributor

Summary

Update contextual JSON completion hints for Xray-core v26.10.21 and align the managed FakeDNS IPv6 pool with the Core default.

 JSON completions
+ XDrive/MASQUE XMUX, XDNS TLS/QUIC, ECH/REALITY, XDrive templates
 Managed FakeDNS IPv6 pool
- fc00:1::/64
+ 2001:2::/48

Hints remain advisory, with protocol validation handled by libXray. Unknown fields and explicit Raw, Remnawave and advanced-template pools are preserved. IPv4, pool sizes, IPv6-disabled query behavior, TUN addresses, MTU and platform network policies retain their existing behavior.

Evidence

  • Before: JSON hints omitted the newer nested transport fields and suggested the legacy XDrive service spelling; managed FakeDNS used fc00:1::/64.
    After: Context-specific completion and configuration round-trip tests cover the new fields, and managed pools use 2001:2::/48.
  • Full suite: 1,737 passed, 5 platform-dependent skips. flutter analyze --no-pub, layer dependency checks, formatting and git diff --check passed.
  • Android 17 arm64, full App: debug build/install passed; 23 actual UI completion cases, node/Raw save and reopen, advanced-editor boundaries, and 3 undo checks through the production controller via VM Service passed.
  • 14 controlled Android VPN cases passed: forced IPv6 HTTP/TLS, explicit UDP DNS, Smart/Custom routing, direct DNS and IPIfNonMatch fallback, IPv6 off/on, FakeDNS off and repeated reconnects in the same native process. Original settings were restored and QA data removed.
  • Live XDrive/MASQUE/XDNS servers, Chromium permission-prompt comparisons and other-platform VPN paths were not tested.

Merge Danger

Door: two-way

Dart and documentation changes can be reverted without a database migration or native interface change. Existing configurations are not rewritten. DNS caches are not flushed; FakeDNS mappings retain the existing Core lifecycle boundary.

Blast Radius: configuration

JSON hint selection and managed Smart/Custom FakeDNS IPv6 destinations are affected. Explicit user pools remain intact; native VPN coverage in this PR is limited to Android.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 10:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-22T10:11:42.463624Z e265a83 PR opened
🔒 Security Review ✅ Completed 2026-10-22T10:12:26.177864Z e265a83 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@yiguodev
yiguodev merged commit 73c1cb3 into main Oct 22, 2026
1 check passed
@yiguodev
yiguodev deleted the feat-xray-core-integration branch October 22, 2026 11:23
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