chore: corregir warnings de lint

- ModifierParameter: mover modifier al primer lugar en AppIconImage y MediaWidget
- PrivateApi: suprimir en TwUtilAdapter (reflexión intencional para MCU)
- StaticFieldLeak: suprimir en McuServiceLocator (applicationContext, no hay leak)
- StaticFieldLeak: usar WeakReference en ReverseCameraActivity._instance
- SetTextI18n: extraer strings hardcodeados a camera/res/values/strings.xml
- DrawAllocation: pre-alocar RectF en RadarOverlayView en lugar de en onDraw

Author: Joshumpa

app/src/main/kotlin/dev/helm/launcher/ui/AppIconImage.kt

@@ -63,8 +63,8 @@ private val actionIconMap: Map<LauncherAction, ImageVector> = mapOf(
 fun AppIconImage(
     pkg: String,
     label: String,
-    action: LauncherAction? = null,
     modifier: Modifier = Modifier,
+    action: LauncherAction? = null,
 ) {
     val icon = action?.let { actionIconMap[it] }
     if (icon != null) {

camera/src/main/kotlin/dev/helm/camera/RadarOverlayView.kt

@@ -21,6 +21,8 @@ class RadarOverlayView @JvmOverloads constructor(
     private var frontMax = 255
     private var rearMax = 255
 
+    private val drawRect = RectF()
+
     private val safePaint = Paint(Paint.ANTI_ALIAS_FLAG).apply {
         color = Color.GREEN
         style = Paint.Style.FILL
@@ -60,17 +62,17 @@ class RadarOverlayView @JvmOverloads constructor(
         // Front sensors — top of screen
         frontDistances.forEachIndexed { i, dist ->
             val paint = paintForDistance(dist)
-            val rect = RectF(i * sensorW + 4f, 4f, (i + 1) * sensorW - 4f, barH)
-            canvas.drawRoundRect(rect, 8f, 8f, paint)
-            canvas.drawText("${dist}cm", rect.centerX(), rect.centerY() + labelPaint.textSize / 3, labelPaint)
+            drawRect.set(i * sensorW + 4f, 4f, (i + 1) * sensorW - 4f, barH)
+            canvas.drawRoundRect(drawRect, 8f, 8f, paint)
+            canvas.drawText("${dist}cm", drawRect.centerX(), drawRect.centerY() + labelPaint.textSize / 3, labelPaint)
         }
 
         // Rear sensors — bottom of screen
         rearDistances.forEachIndexed { i, dist ->
             val paint = paintForDistance(dist)
-            val rect = RectF(i * sensorW + 4f, h - barH - 4f, (i + 1) * sensorW - 4f, h - 4f)
-            canvas.drawRoundRect(rect, 8f, 8f, paint)
-            canvas.drawText("${dist}cm", rect.centerX(), rect.centerY() + labelPaint.textSize / 3, labelPaint)
+            drawRect.set(i * sensorW + 4f, h - barH - 4f, (i + 1) * sensorW - 4f, h - 4f)
+            canvas.drawRoundRect(drawRect, 8f, 8f, paint)
+            canvas.drawText("${dist}cm", drawRect.centerX(), drawRect.centerY() + labelPaint.textSize / 3, labelPaint)
         }
     }
 

camera/src/main/kotlin/dev/helm/camera/ReverseCameraActivity.kt

@@ -15,6 +15,7 @@ import kotlinx.coroutines.cancel
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
 import kotlinx.coroutines.launch
+import java.lang.ref.WeakReference
 
 // Task 6.3/6.4/6.5/6.6/6.7/6.8 — full-screen reverse camera. Launched by ReverseCameraManager.
 // Back press is a no-op while in reverse; dismissed programmatically when gear changes.
@@ -33,7 +34,7 @@ class ReverseCameraActivity : Activity() {
         surfaceView = SurfaceView(this)
         radarOverlay = RadarOverlayView(this)
         noSignalBanner = TextView(this).apply {
-            text = "Sin señal"
+            text = getString(R.string.camera_no_signal)
             setTextColor(android.graphics.Color.WHITE)
             textSize = 18f
             gravity = android.view.Gravity.CENTER
@@ -51,7 +52,7 @@ class ReverseCameraActivity : Activity() {
         })
 
         setContentView(root)
-        _instance = this
+        _instance = WeakReference(this)
 
         if (!McuServiceLocator.isInitialized) {
             showStubMode()
@@ -90,7 +91,7 @@ class ReverseCameraActivity : Activity() {
     }
 
     override fun onDestroy() {
-        if (_instance === this) _instance = null
+        if (_instance?.get() === this) _instance = null
         if (McuServiceLocator.isInitialized) {
             activityScope.launch { McuServiceLocator.service.send(0x0304, 0) }
         }
@@ -101,15 +102,15 @@ class ReverseCameraActivity : Activity() {
     // Task 6.7 — stub mode placeholder
     private fun showStubMode() {
         surfaceView.visibility = View.GONE
-        noSignalBanner.text = "Cámara reversa — No disponible (Track B)"
+        noSignalBanner.text = getString(R.string.camera_unavailable)
         noSignalBanner.visibility = View.VISIBLE
     }
 
     companion object {
-        @Volatile private var _instance: ReverseCameraActivity? = null
+        @Volatile private var _instance: WeakReference<ReverseCameraActivity>? = null
 
         fun requestDismiss() {
-            _instance?.finish()
+            _instance?.get()?.finish()
         }
     }
 }

camera/src/main/res/values/strings.xml

@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+    <string name="camera_no_signal">Sin señal</string>
+    <string name="camera_unavailable">Cámara reversa — No disponible (Track B)</string>
+</resources>

sdk/src/main/kotlin/dev/helm/sdk/McuServiceLocator.kt

@@ -1,5 +1,6 @@
 package dev.helm.sdk
 
+import android.annotation.SuppressLint
 import android.content.Context
 import android.os.Process
 import kotlinx.coroutines.CoroutineScope
@@ -9,6 +10,8 @@ import kotlinx.coroutines.SupervisorJob
 // Chooses TwUtilMcuDataSource when running as system UID; StubMcuDataSource otherwise.
 object McuServiceLocator {
 
+    // applicationContext is passed in — not a real leak.
+    @SuppressLint("StaticFieldLeak")
     @Volatile private var _service: McuService? = null
 
     val service: McuService

sdk/src/main/kotlin/dev/helm/sdk/TwUtilAdapter.kt

@@ -1,5 +1,6 @@
 package dev.helm.sdk
 
+import android.annotation.SuppressLint
 import android.os.Handler
 import android.os.Looper
 import android.os.Message
@@ -9,6 +10,7 @@ import kotlinx.coroutines.flow.callbackFlow
 
 // Track B only. Reflective wrapper over android.tw.john.TWUtil (platform @hide class,
 // requires android.uid.system). Assumes TWUtil(Handler) constructor — verified post-FEL.
+@SuppressLint("PrivateApi")
 internal class TwUtilAdapter {
 
     companion object {
@@ -149,6 +151,8 @@ internal class TwUtilAdapter {
         )
     }
 
+    // TODO [Track B / post-FEL]: if MCU doesn't send 0x010A proactively, discover the request
+    // command byte via UART capture and call send() with it before waiting for this event.
     private fun parseMcuVersion(obj: Any?): McuEvent {
         val version = when (obj) {
             is ByteArray -> String(obj, Charsets.US_ASCII).trimEnd(' ')

widgets/src/main/kotlin/dev/helm/widgets/MediaWidget.kt

@@ -14,10 +14,10 @@ import androidx.compose.ui.unit.dp
 
 @Composable
 fun MediaWidget(
+    modifier: Modifier = Modifier,
     speed: Int = 0,
     title: String = "",
     artist: String = "",
-    modifier: Modifier = Modifier,
 ) {
     Column(modifier = modifier) {
         if (title.isNotEmpty()) {