Repository navigation
Fix ForkOP server lists, auto policies and confirmed VPN selection (2.5.20) - #3
Merged
Merged
Conversation
added 5 commits
October 18, 2026 22:20
RichkovGit
marked this pull request as ready for review
October 19, 2026 10:00
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The ForkOP screen currently invents four auto groups and sample servers/latencies, and treats whichever selector happens to be parsed last as the active server. This displays the wrong policy and can report a successful switch after failed commands.
Read actual groups, display names, raw tags and section selectors from Sing-box, Clash runtime state and all ForkOP cache documents, matching upstream
ushan0v/forkopcache metadata. Hide infrastructure outbounds, distinguish the selected section policy from its nested active path, use real timestamped measurements and expose only supported manual selections. Resolve the real parent selector and confirm its newnowvalue after switching; remove optimistic UI state, hard-coded router addresses and fake UCI writes. Provider/protocol filters follow the current router data.Adds shared Android/desktop topology fixtures and ten portable regression tests for metadata, routing roots, cycles, large lists, timestamps, errors and read-back confirmation. Windows tests include these same cases. Candidate version: 2.5.20; existing workflows produce the installer and portable ZIP.
The companion mobile/Linux changes are in
RichkovGit/OpenWrtStudio-Mobile, branchcodex/fix-vpn-selection. Linux uses that Flutter GUI, with AppImage/tar packaging and distro smoke tests. Live router and Windows UI verification completed on 2026-10-19; original VPN selectors and LuCI theme restored.Validation at
13fb5a0: Windows CI passed 128 tests (5 opt-in live-router tests skipped), and the portable core suite passed all 10 tests. CI run · Windows installer and portable artifacts. Companion mobile/Linux PR #1.Hardware validation — 2026-10-19
Проверка VPN на реальном оборудовании — 19.10.2026
Проверены Windows 2.5.20 и Android 2.5.24 на Cudy WR3000S v1 / OpenWrt 25.12.5 с ForkOP и sing-box. Получено 27 групп и 223 сервера из реального роутера.
Проверки: Windows 128 passed / 6 skipped; portable core 10 passed / 1 skipped; opt-in live VPN 1 passed; read-only router probes 2 passed; Flutter analyze без замечаний, 163 tests passed. Пропуски offline-прогона относятся к интеграциям, требующим явного включения.
Исправлена блокировка тестов Dashboard при наличии WPF Application без работающего Dispatcher: тесты передают синхронный диспетчер, приложение сохраняет штатный UI Dispatcher.
Пределы проверки: проверялся выбор маршрута через состояние роутера, а не полный захват пользовательского трафика; все 223 узла по отдельности не испытывались. WireGuard/AmneziaWG и новые Linux-пакеты в этом аппаратном прогоне не проверялись. Ранее выполненные Linux-проверки PR не заменяются этим отчётом.
Снимки конфигурации и данные доступа в репозиторий не включены. Для повторения live-теста нужны OPENWRT_LIVE_VPN_SWITCH=1, OPENWRT_TEST_HOST и OPENWRT_TEST_PASSWORD; запуск временно меняет реальный VPN и восстанавливает исходный выбор.