refactor: remove AIDL API and modernize radio architecture

Remove the deprecated AIDL/IPC API surface and perform deep architectural
modernization of the radio command pipeline, aligning with the meshtastic-sdk
AdminApiImpl pattern for future SDK migration.

Key changes:

1. AIDL Removal & Infrastructure Cleanup
   - Delete core:api module and all AIDL interfaces
   - Remove ServiceBroadcasts + CommonParcelable infrastructure
   - Remove core:api from CI workflow lint/publish steps

2. Model Modernization
   - Introduce NodeAddress sealed class with type-safe addressing
   - Remove deprecated DataPacket constants in favor of NodeAddress
   - Consolidate dual node maps into single source with getNodeById
   - Split large model files, deduplicate NodeEntity, flatten RadioController

3. Service Layer Refactoring (SDK-aligned)
   - Remove ServiceAction sealed class, use direct suspend calls
   - Convert CommandSender & MeshActionHandler to suspend APIs
   - Merge MeshActionHandler into DirectRadioControllerImpl
     (ViewModel → RadioController → CommandSender, no intermediate layer)
   - Build AdminMessage protos directly with typed protos end-to-end
   - Apply structured concurrency to NodeRequestActions/NodeManagementActions
   - Fix CancellationException handling throughout

Architecture (before → after):
  ViewModel → RadioController → Handler → CommandSender → PacketHandler
  ViewModel → RadioController → CommandSender → PacketHandler

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: jamesarich

README.md

@@ -87,7 +87,6 @@ Each module has its own README with details on its responsibilities, API surface
 
 | Module | Description |
 |---|---|
-| [core/api](core/api/README.md) | AIDL service API for third-party integrations |
 | [core/domain](core/domain/README.md) | Business-logic use cases (radio config, sessions, exports) |
 | [core/repository](core/repository/README.md) | Data & infrastructure contracts (RadioTransport, NodeRepository, ServiceRepository) |
 | [core/takserver](core/takserver/README.md) | Meshtastic ↔ TAK (ATAK/iTAK) bridge — CoT server & conversion |
@@ -123,13 +122,9 @@ Each module has its own README with details on its responsibilities, API surface
 
 You can help translate the app into your native language using [Crowdin](https://crowdin.meshtastic.org/android).
 
-## API & Integration
+## Integration
 
-Developers can integrate with the Meshtastic Android app using our published API library via **JitPack**. This allows third-party applications (like the ATAK plugin) to communicate with the mesh service via AIDL.
-
-For detailed integration instructions, see [core/api/README.md](core/api/README.md).
-
-Additionally, the app includes a built-in **Local TAK Server** feature that can be enabled in settings. This runs a local TCP server on port 8089 to allow ATAK clients to connect directly and route their traffic over the mesh.
+The app includes a built-in **Local TAK Server** feature that can be enabled in settings. This runs a local TCP server on port 8089 to allow ATAK clients to connect directly and route their traffic over the mesh.
 
 ## Building the Android App
 > [!WARNING]

androidApp/src/fdroid/kotlin/org/meshtastic/app/map/MapView.kt

@@ -87,6 +87,7 @@ import org.meshtastic.core.common.util.nowMillis
 import org.meshtastic.core.common.util.nowSeconds
 import org.meshtastic.core.model.DataPacket
 import org.meshtastic.core.model.Node
+import org.meshtastic.core.model.NodeAddress
 import org.meshtastic.core.resources.Res
 import org.meshtastic.core.resources.calculating
 import org.meshtastic.core.resources.cancel
@@ -433,7 +434,7 @@ fun MapView(
         }
     }
 
-    fun getUsername(id: String?) = if (id == DataPacket.ID_LOCAL || (myId != null && id == myId)) {
+    fun getUsername(id: String?) = if (id == NodeAddress.ID_LOCAL || (myId != null && id == myId)) {
         getString(Res.string.you)
     } else {
         mapViewModel.getUser(id).long_name

androidApp/src/google/kotlin/org/meshtastic/app/map/MapViewModel.kt

@@ -50,6 +50,7 @@ import org.meshtastic.app.map.model.CustomTileProviderConfig
 import org.meshtastic.app.map.prefs.map.GoogleMapsPrefs
 import org.meshtastic.app.map.repository.CustomTileProviderRepository
 import org.meshtastic.core.di.CoroutineDispatchers
+import org.meshtastic.core.model.NodeAddress
 import org.meshtastic.core.model.RadioController
 import org.meshtastic.core.repository.MapPrefs
 import org.meshtastic.core.repository.NodeRepository
@@ -670,8 +671,7 @@ class MapViewModel(
         (currentTileProvider as? MBTilesProvider)?.close()
     }
 
-    override fun getUser(userId: String?) =
-        nodeRepository.getUser(userId ?: org.meshtastic.core.model.DataPacket.ID_BROADCAST)
+    override fun getUser(userId: String?) = nodeRepository.getUser(userId ?: NodeAddress.ID_BROADCAST)
 }
 
 enum class LayerType {

androidApp/src/main/AndroidManifest.xml

@@ -161,16 +161,12 @@
             android:name="google_analytics_default_allow_analytics_storage"
             android:value="false" />
 
-        <!-- This is the public API for doing mesh radio operations from android apps -->
+        <!-- In-app mesh foreground service -->
         <service
             android:name="org.meshtastic.core.service.MeshService"
             android:enabled="true"
             android:foregroundServiceType="connectedDevice|location"
-            android:exported="true" tools:ignore="ExportedActivity">
-            <intent-filter>
-                <action android:name="com.geeksville.mesh.Service" />
-            </intent-filter>
-        </service>
+            android:exported="false" />
 
         <service
             android:name="androidx.appcompat.app.AppLocalesMetadataHolderService"
@@ -272,7 +268,7 @@
                 <action android:name="com.htc.intent.action.QUICKBOOT_POWERON" />
 
                 <!-- for testing -->
-                <action android:name="com.geeksville.mesh.SIM_BOOT" />
+                <action android:name="org.meshtastic.app.SIM_BOOT" />
             </intent-filter>
 
             <!-- Also restart our service if the app gets upgraded -->

androidApp/src/main/kotlin/org/meshtastic/app/MainActivity.kt

@@ -63,7 +63,8 @@ import org.meshtastic.core.network.repository.UsbRepository
 import org.meshtastic.core.nfc.NfcScannerEffect
 import org.meshtastic.core.resources.Res
 import org.meshtastic.core.resources.channel_invalid
-import org.meshtastic.core.service.MeshServiceClient
+import org.meshtastic.core.service.MeshService
+import org.meshtastic.core.service.startService
 import org.meshtastic.core.ui.theme.AppTheme
 import org.meshtastic.core.ui.theme.MODE_DYNAMIC
 import org.meshtastic.core.ui.util.LocalAnalyticsIntroProvider
@@ -95,18 +96,9 @@ class MainActivity : AppCompatActivity() {
 
     private val usbRepository: UsbRepository by inject()
 
-    /**
-     * Activity-lifecycle-aware client that binds to the mesh service. Note: This is used implicitly as it registers
-     * itself as a LifecycleObserver in its init block.
-     */
-    internal val meshServiceClient: MeshServiceClient by inject { parametersOf(this) }
-
     override fun onCreate(savedInstanceState: Bundle?) {
         installSplashScreen()
 
-        // Eagerly evaluate lazy Koin dependency so it registers its LifecycleObserver
-        meshServiceClient.hashCode()
-
         super.onCreate(savedInstanceState)
 
         enableEdgeToEdge()
@@ -168,6 +160,11 @@ class MainActivity : AppCompatActivity() {
         handleIntent(intent)
     }
 
+    override fun onStart() {
+        super.onStart()
+        MeshService.startService(this)
+    }
+
     override fun onResume() {
         super.onResume()
         // Belt-and-suspenders for the Android 12+ attach-intent quirk: if the activity is

build-logic/convention/src/main/kotlin/RootConventionPlugin.kt

@@ -81,7 +81,6 @@ private val DEVICE_TEST_MODULES = listOf(":core:database", ":core:model")
 private val ALL_MODULES_FULL =
     listOf(
         ":androidApp",
-        ":core:api",
         ":core:barcode",
         ":core:ble",
         ":core:common",
@@ -115,7 +114,7 @@ private val ALL_MODULES_FULL =
     )
 
 /** Android-only modules that don't apply the KMP plugin. */
-private val ANDROID_ONLY_MODULES = setOf(":androidApp", ":core:api", ":core:barcode", ":feature:widget")
+private val ANDROID_ONLY_MODULES = setOf(":androidApp", ":core:barcode", ":feature:widget")
 
 /**
  * Modules excluded from Dokka aggregation. :core:proto contains only auto-generated Wire classes (no KDoc value) and
@@ -128,6 +127,6 @@ private fun allModules(): List<String> = ALL_MODULES_FULL
 
 /**
  * Modules that apply the KMP plugin and should be compiled for JVM + iOS targets. Excludes pure-Android modules
- * (:androidApp, :core:api, :core:barcode, :feature:widget) and the desktop JVM-only module.
+ * (:androidApp, :core:barcode, :feature:widget) and the desktop JVM-only module.
  */
 private fun kmpModules(): List<String> = allModules().filter { it !in ANDROID_ONLY_MODULES + ":desktopApp" }

codecov.yml

@@ -61,8 +61,6 @@ component_management:
 ignore:
   - "**/build/**"
   - "**/*.pb.kt"          # Generated Protobuf code
-  - "**/*.aidl"           # AIDL interface files
-  - "**/aidl/**"           # Generated AIDL code
   - "core/resources/**"   # Centralized resources
   - "**/test/**"          # Unit tests
   - "**/androidTest/**"   # Instrumented tests