openthread-br: drop luci-app-openthread, proposed for luci repo

[LorbusChris]

📦 Package Details

Maintainer: @stintel

Description

Removes the luci-app-openthread subpackage from net/openthread-br. The LuCI
web interface is proposed for the openwrt/luci repository as
luci-app-openthread (openwrt/luci#8871), following the standard convention for
LuCI apps, as requested in #29791 and openthread/ot-br-posix#3431. Dropping it
here means the UI is no longer shipped from two places.

Split out of #30091 at @stintel's request so it can be reviewed and land
independently of the 2026.07.0 update. #30091 now contains the version update,
the switch to OpenThread's internal mDNS, and the REST listen-address option, and
will be rebased on this.

PKG_RELEASE is bumped since the package contents change without a version
change.

Note the app being removed is the old Lua/luci-lua-runtime implementation; the
openwrt/luci proposal is a rewrite using the modern client-side JS + rpcd ucode
conventions, so this is not a straight move of the same files.

openwrt/luci#8871 is still open, so this should ideally land together with (or
after) it, otherwise there is a window with no OpenThread web UI available at all.
The in-tree copy in ot-br-posix is proposed for removal separately in
openthread/ot-br-posix#3326 (draft).

🧪 Run Testing Details

Removal only — no code changes to the openthread-br package itself, and the
remaining Package/openthread-br recipe is untouched. Apart from
patches/102-openwrt-fix-LuCI-icons.patch, which only rewrites icon paths inside
those same src/openwrt/view/ files, the removed Package/luci-app-openthread
and its install recipe were the only users of the LuCI parts of the
src/openwrt/ tree. The rest of that tree stays in use: src/openwrt/ubus/
builds the otbr ubus object that the netifd proto handler waits on
(ubus -t30 wait_for otbr, then ubus call otbr threadstart), so
-DOTBR_OPENWRT=ON is untouched here and must stay.

The icon patch is left in place here; #30091 drops it, since the 2026.07.0 tag
already carries the fix upstream.

A test.sh is added in a second commit. The generic test requires an installed
executable to print PKG_VERSION, which none of this package's binaries do —
otbr-agent reports the upstream CMake project version instead. That already
fails on master, independently of this change:

openthread-br: No executables in the package provided version 2025.06.12~2f3c799c
openthread-br: Generic tests failed

The test exercises otbr-agent --version and ot-ctl -h instead, which also
covers their runtime library closure. It deliberately avoids ot-ctl --version,
which segfaults on an unterminated getopt_long() option array — fixed upstream
in openthread/openthread#13423.

✅ Formalities

• I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

[openwrt-ai]

Reviewed 2 new commits; both commit messages match their diffs. Three inline notes, none blocking.

---

Generated by Claude Code

[openwrt-ai]

Reviewed 2 new commits; no new issues found.

The only new content since dad9226 is the README.md "### LuCI" hunk, which resolves the earlier nit — test.sh and the Makefile are byte-identical to what was already reviewed. Both commit messages still match their diffs, and the three FormalityCheck runs on 300b53e are green.

---

Generated by Claude Code

[openwrt-ai]

Reviewed 1 new commit (300b53e → fabe4a6, the test.sh commit amended to also add test-version.sh). The amended commit message matches its diff.

The only new content since the last review is net/openthread-br/test-version.sh; test.sh, the Makefile and README.md are unchanged from what was already reviewed. One inline note on the new file. All three FormalityCheck runs on fabe4a6 are green.

---

Generated by Claude Code