Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@android/app/build.gradle`:
- Around line 116-120: The comment stating that Kotlin is "provided by Flutter’s
built-in Kotlin support" is misleading because gradle.properties sets
android.builtInKotlin=false; update the comment near the kotlin {
compilerOptions { jvmTarget = org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_11 }
} block to reflect that the project is opting out of AGP built-in Kotlin
(legacy/manual Kotlin plugin usage), or instead migrate to AGP built-in Kotlin
by removing/setting android.builtInKotlin to true and adjusting the Kotlin
plugin setup accordingly; ensure the comment accurately references
android.builtInKotlin and the kotlin { ... } configuration so future readers
understand which mode is in use.

In `@test/notifiers/take_order_notifier_test.dart`:
- Around line 50-56: The test constructs MockSessionNotifier with a throwaway
ProviderContainer (via createTestRef(refContainer)) and then injects it with
sessionNotifierProvider.overrideWith, causing ref-based reads in SessionNotifier
to resolve against the wrong provider graph; change each override (the
sessionNotifierProvider.overrideWith calls around the noted locations) to create
MockSessionNotifier using the override callback’s ref (i.e., overrideWith((ref)
=> MockSessionNotifier(ref, mockKeyManager, mockSessionStorage, testSettings)))
instead of prebuilding a ref from ProviderContainer, and remove the throwaway
createTestRef usage and its ProviderContainer/tearDown for those overrides so
provider wiring uses the same ref throughout.

In `@test/services/mostro_service_test.dart`:
- Around line 66-73: The file defines a file-scoped ProviderContainer named
dummyContainer (and dummyRef created via createTestRef) that's never disposed,
leaving Riverpod listeners from MockSubscriptionManager(dummyRef) alive for the
whole test suite; add a suite-level teardown to call dummyContainer.dispose()
(e.g., using tearDownAll(() => dummyContainer.dispose())) so
sessionNotifierProvider overrides and ref.onDispose handlers are cleaned up
after the suite.

---

Outside diff comments:
In `@lib/features/relays/relays_notifier.dart`:
- Line 692: Replace the Spanish inline comment "🔥 RESET COMPLETO: Limpiar todos
los relays y hacer sync fresco" with an English equivalent, e.g. "🔥 FULL RESET:
Clear all relays and perform a fresh sync", so comments in relays_notifier.dart
are in English.

---

Nitpick comments:
In `@lib/features/auth/providers/auth_notifier_provider.dart`:
- Around line 1-2: The file imports both flutter_riverpod.dart and legacy.dart
but only uses legacy APIs; remove the top-level import
"package:flutter_riverpod/flutter_riverpod.dart" from
auth_notifier_provider.dart and keep only
"package:flutter_riverpod/legacy.dart"; ensure the file still compiles by
verifying it only references Provider, StateNotifierProvider, StateProvider,
Ref, and StateNotifier (the symbols used by the provider/notifier code) and that
no ProviderSubscription or other flutter_riverpod-only types are referenced
here.

In `@lib/features/settings/settings_notifier.dart`:
- Around line 2-3: Remove the redundant import of
"package:flutter_riverpod/flutter_riverpod.dart" and keep only
"package:flutter_riverpod/legacy.dart" so StateNotifier and related legacy
exports resolve correctly; update settings_notifier.dart (look for class
SettingsNotifier and any StateNotifier/StateNotifierProvider usages) to import
only the legacy entrypoint and repeat the same change in trades_provider.dart,
first_run_provider.dart, and nwc_provider.dart to eliminate the unused-import
analyzer warnings and complete the Riverpod 3 migration.