Repository navigation
feat: add manual server groups and Raw/provider VPN defaults - #236
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 18bc620317
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
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.
Summary
26.10.5+1, retain JNI-referenced Kotlin symbols in Android release builds, and surface/document Google Play review-policy errors.Evidence
After: organization survives restart/backup, and the configuration checks plus Android VPN probes below pass with missing DNS supplemented.
16b7b464: Flutter suite passed 1,715 tests, 5 skipped; Flutter analyze, layer checks and 3 Swift runtime-environment tests passed. Android and macOS debug builds succeeded.c13bcb85, including ordering, Save/Cancel and restart persistence. Backup roundtrip storage probes passed.16b7b464, Android emulator VPN flows passed for Raw profiles with missing DNS, hosts-only DNS and explicit DNS, plus provider profiles with missing DNS. HTTP and DNS A/TXT probes passed, including reconnect in the same native process; source configurations remained unchanged.479a90a4: 20 bytecode/manifest checks and two cold starts with settings persistence passed.git diff --check origin/main...HEADpassed.Merge Danger
Door: one-way
Opening an existing database migrates it to schema 9; reverting the application alone does not downgrade it. Backup protocol remains version 1, but older clients cannot preserve organization fields when rewriting backups.
Blast Radius: application
Changes affect persistent assets/restore, server catalogs, Raw/provider VPN configuration and Android release shrinking/upload behavior. Apple device VPN, Windows/Linux device networking, cloud backup authorization and IPv6 HTTP acceptance were not exercised; no macOS VPN was started.