chore(android): port widget triple to Java JNI shims, drop configure activity (#83)

[hyperpolymath]

Sub-PR #7 — port the widget triple

Part of the Android Kotlin→Rust/Gossamer migration (epic #83, RFC PR #97, sub-issue #113).

Replaces the home-screen widget Kotlin with thin hand-written Java shims that JNI into the Rust core (crates/neurophone-android) via NativeLib.getState() / NativeLib.query(...). The widget is now stateless: every render reads live salience / description straight from the Rust core instead of caching in SharedPreferences.

Files changed

Ported (Kotlin → Java):

• NeurophoneAppWidget.kt → NeurophoneAppWidget.java — AppWidgetProvider shim. Reads core state via NativeLib.getState() (JSON, parsed defensively) and isRunning(); toggle/refresh/ask actions preserved. Adds public static requestRefresh(Context) for non-widget callers.
• NeurophoneWidgetActions.kt → NeurophoneWidgetActions.java — BroadcastReceiver shim. The old PUBLISH_STATE (extras + SharedPreferences) path is gone; receiver now does FORCE_REFRESH (re-render from core) and ACTION_QUERY (one-shot NativeLib.query(...) then refresh).

Dropped (owner decision):

• NeurophoneWidgetConfigureActivity.kt — deleted, not ported. It had no manifest <activity> entry; its only registration was android:configure=... in res/xml/neurophone_widget_info.xml, now removed along with android:widgetFeatures="reconfigurable|configuration_optional" and the widget_configure_* strings. The widget works with no configure step.

Knock-on edits (Kotlin callers of the removed widget API, kept compiling):

• NeurophoneService.kt — publishState(...) call sites → requestRefresh(...) (core is now source of truth).
• BootReceiver.kt — dropped the removed NeurophoneAppWidget.prefs/KEY_RUNNING read (a core concern now); conservatively no-ops on boot pending a core-persisted autostart flag.
• AndroidManifest.xml, neurophone_widget_info.xml, strings.xml — configure-activity removal + explanatory comments.

Dropped configure-activity note

NeurophoneWidgetConfigureActivity is intentionally not ported per owner decision. No manifest registration existed to remove; the android:configure hook in the widget-info XML was the sole binding and has been removed. The single optional question it asked (local_only) is left for the core/UI to own post-migration.

What I verified

• All new Java files carry SPDX-License-Identifier: MPL-2.0; hand-written Java is permitted only under android/ via the existing .hypatia-baseline.json android/** carve-out.
• No live code references any removed symbol (publishState, prefs, KEY_RUNNING, NeurophoneWidgetConfigureActivity, KEY_LOCAL_ONLY) — remaining hits are comments/docstrings only.
• All referenced widget layout IDs (widget_state, widget_salience, widget_salience_value, widget_toggle, widget_refresh, widget_query) exist in widget_neurophone.xml.
• XML well-formedness checked for the manifest, widget-info, and strings.
• Rust workspace: cargo build --workspace / cargo test --workspace currently fail on a pre-existing breakage in esn and lsm (a rand / ndarray-rand API mismatch: no method named random/sample, impl Rng: ndarray_rand::rand::Rng unsatisfied). Reproduced on a clean origin/main with all PR changes stashed, so it is not introduced here. This PR touches no Rust (android-only diff) and does not regress the workspace. neurophone-android itself only fails transitively through esn/lsm.

TODOs / risks

• TODO(#83 rebase) markers flag the seams that depend on parallel sub-PRs chore(deps): update rand_distr requirement from 0.4 to 0.5 #3/chore(deps): update thiserror requirement from 1.0 to 2.0 #4/chore(deps): update ndarray requirement from 0.15 to 0.17 #5 (not yet merged):
  • NativeLib is still the pre-migration Kotlin object, so the Java shims use NativeLib.INSTANCE.…; drop .INSTANCE if chore(deps): update rand_distr requirement from 0.4 to 0.5 #3/chore(deps): update thiserror requirement from 1.0 to 2.0 #4/chore(deps): update ndarray requirement from 0.15 to 0.17 #5 republish it as a Java facade.
  • The getState() JSON schema (salience, description keys) is the agreed contract but should be confirmed/tightened on rebase. JNI calls are wrapped in try { } catch (Throwable) so the widget never crashes the launcher against the current stub crate (UnsatisfiedLinkError).
  • BootReceiver autostart and the service's per-tick salience push need the core-side persisted run/state APIs from chore(deps): update rand_distr requirement from 0.4 to 0.5 #3/chore(deps): update thiserror requirement from 1.0 to 2.0 #4/chore(deps): update ndarray requirement from 0.15 to 0.17 #5.

Open questions

• Should the dropped configure activity's local_only preference be re-surfaced somewhere (settings screen / core config), or is it gone for good?
• Confirm the final NativeLib shape (Kotlin object vs Java facade) so the .INSTANCE interop can be cleaned up on rebase.

Marked DRAFT: depends on parallel sub-PRs #3/#4/#5 (base off main, structure assumed, TODO(#83 rebase) markers).

https://claude.ai/code/session_01Gu1JFCZHuBtBhAWPr4sMQw

---

Generated by Claude Code