This commit migrates the networking layer from OkHttp to the Ktor Android engine across the project. This simplifies the dependency graph by removing OkHttp-specific interceptors and logging in favor of Ktor's native plugins.

Specific changes include:
- **Dependency Updates**: Removed `okhttp`, `okhttp3-logging-interceptor`, and `dd-sdk-android-okhttp` from the version catalog and build files.
- **Engine Migration**: Replaced `ktor-client-okhttp` with `ktor-client-android` and updated `HttpClient` providers to use the `Android` engine instead of `OkHttp`.
- **Logging**: Switched from OkHttp's `HttpLoggingInterceptor` to the Ktor `Logging` plugin for debug builds.
- **Module Cleanup**:
    - Simplified `FDroidNetworkModule` and `GoogleNetworkModule` by removing manual `OkHttpClient` bean definitions and cache configurations.
    - Removed `DatadogInterceptor` and `DatadogEventListener` configurations.
- **Testing**: Updated `KoinVerificationTest` to remove `OkHttpClient` from the expected dependency graph.
- **Documentation**: Updated `AGENTS.md` to reflect the architectural decision to use pure Ktor for networking.

Signed-off-by: James Rich <[email protected]>

…ndencies

This commit refactors the network module to streamline dependency management and remove redundant platform-specific configurations. The `HttpClient` provision was removed from the Android-specific Koin module in favor of a more unified approach.

Specific changes include:
- **Dependency Cleanup**: Removed `libs.ktor.client.android` from `core:network` build configuration.
- **DI Refactoring**: Removed `provideHttpClient` from `CoreNetworkAndroidModule`, shifting away from the platform-specific Ktor Android engine configuration within that module.
- **ProGuard**: Cleaned up networking-related ProGuard rules, removing redundant `okhttp3` warnings.
- **Maintenance**: Deleted `googleReleaseRuntimeClasspath.txt` dependency snapshot.

Signed-off-by: James Rich <[email protected]>

This commit removes redundant and unused dependencies across several modules to streamline the build configuration and reduce potential classpath conflicts.

Specific changes include:
- **desktop**: Removed explicit `ktor-client-core` dependency as it is transitively included by `ktor-client-java`.
- **feature:node**: Removed redundant Material 2 Vico dependency and cleaned up duplicate markdown renderer declarations in `androidMain`.
- **feature:intro**: Removed `navigation3-ui` from `androidMain`.
- **feature:settings**: Removed duplicate markdown renderer dependencies in `androidMain`.
- **core:common**: Removed `aboutlibraries` dependencies.

Signed-off-by: James Rich <[email protected]>

This commit cleans up Navigation 3 dependency declarations by standardizing on `navigation3-ui` and removing the redundant `navigation3-runtime` alias from the version catalog and module build files.

Specific changes include:
- **Version Catalog**: Removed `jetbrains-navigation3-runtime` from `libs.versions.toml` as it was a duplicate entry pointing to the same Maven coordinates as `jetbrains-navigation3-ui`.
- **Dependency Migration**: Updated `core:navigation`, `app`, and several feature modules (`node`, `intro`, `messaging`, `connections`, `map`, `settings`, `firmware`) to use `libs.jetbrains.navigation3.ui` exclusively.
- **Dependency Cleanup**:
    - Removed unused `core.common` dependency from `feature:map`.
    - Added `kotlinx.collections.immutable` to `feature:map`.
    - Cleaned up redundant navigation imports in `feature:settings`, `feature:map`, `feature:firmware`, and the `app` module.

Signed-off-by: James Rich <[email protected]>

…roid and JVM

This commit refactors the build configuration for the `core:data` module to reduce duplication by grouping shared Room and SQLite dependencies into a common source set for Android and JVM targets.

Specific changes include:
- Applied the `meshtastic.kmp.jvm.android` plugin.
- Introduced a `jvmAndroidMain` source set to manage dependencies shared between Android and Desktop JVM targets.
- Moved `androidx.room.runtime`, `androidx.room.paging`, and `androidx.sqlite.bundled` from separate `androidMain` and `jvmMain` blocks into the unified `jvmAndroidMain` configuration.

Signed-off-by: James Rich <[email protected]>

This commit relocates several dependencies in the firmware feature module from the Android-specific source set to `commonMain` to support better code sharing and multiplatform compatibility. It also includes a minor cleanup in the BLE core module.

Specific changes include:
- Moved `coil`, `markdown-renderer`, and `markdown-renderer-m3` from `androidMain` to `commonMain` in `feature/firmware/build.gradle.kts`.
- Removed an empty, unused `jvmMain.dependencies` block from `core/ble/build.gradle.kts`.

Signed-off-by: James Rich <[email protected]>

This commit refactors build-logic conventions and cleans up unused dependencies and properties to streamline the build process. Key changes include consolidating test configurations into `ProjectExtensions.kt` and removing deprecated plugins and experimental flags.

Specific changes include:
- **Build Logic**:
    - Moved `failOnNoMatchingTests` configuration from the root `build.gradle.kts` to `ProjectExtensions.kt`.
    - Integrated `configureTestOptions()` into `KmpLibraryConventionPlugin`.
    - Corrected `androidx-compose-ui-tooling` to use `debugImplementation` only.
- **Dependency Cleanup**:
    - Removed `vico-compose-m2`, `firebase-perf`, and `dependency-guard` from `libs.versions.toml`.
    - Removed the `meshtastic-android-test` plugin definition.
- **Properties & Config**:
    - Removed several Android and KSP properties from `gradle.properties`, including `android.enableJetifier`, `android.useAndroidX`, and `ksp.useKSP2`.
    - Cleaned up redundant whitespace and plugin aliases in the root `build.gradle.kts`.

Signed-off-by: James Rich <[email protected]>

This commit updates the copyright headers to include 2026 and applies project-wide code formatting and linting rules to the build-logic convention plugins.

Specific changes include:
- Updated copyright headers to `2025-2026 Meshtastic LLC` across convention plugin files.
- Applied `ktfmt` with a 120-character maximum width across `build-logic`.
- Refactored `KotlinAndroid.kt` to support a `warningsAsErrors` Gradle property, defaulting to `false`.
- Updated `Kover.kt` filters to exclude iOS compile-only stubs (`*NoopStubs*`).
- Removed `java.time.*` from `compose_compiler_config.conf`.
- Removed an unnecessary `afterEvaluate` block in `FlavorResolution.kt` to improve configuration performance.
- Consolidated various one-line lambda expressions and cleaned up redundant imports and whitespace.
- Updated `Graph.kt` to improve compatibility with Gradle Isolated Projects.

Signed-off-by: James Rich <[email protected]>

…lution

This commit replaces the legacy `afterEvaluate` approach of stamping attributes on every resolvable Android configuration with Gradle `AttributeDisambiguationRule`s. This ensures robust flavor selection when a consumer (e.g., a KMP module's host test) does not specify a marketplace attribute but the producer publishes multiple variants.

This change improves compatibility with Configuration Cache and Isolated Projects by moving resolution logic from configuration time to dependency resolution time.

Specific changes include:
- Introduced `ProductFlavorDisambiguationRule` and `StringDisambiguationRule` to handle flavor selection during variant-aware resolution.
- Updated `configureAndroidMarketplaceFallback` to register these rules in the `attributesSchema` for both `ProductFlavorAttr` and the legacy "marketplace" string attribute.
- Removed the manual configuration-looping logic (`configurations.configureEach`) that previously attempted to infer flavors from configuration names.
- Maintained support for the `meshtastic.defaultMarketplace` Gradle property to control the default fallback flavor.

Signed-off-by: James Rich <[email protected]>

…cies

This commit updates the build logic to prevent build failures when no tests are discovered and adds necessary UI dependencies to the widget feature module.

Specific changes include:
- **Build Logic**: Configured `AbstractTestTask` to set `failOnNoDiscoveredTests` to `false`. This prevents Gradle 9+ from failing builds in modules where tests might be present but are currently commented out or not yet implemented.
- **Widget Feature**: Added `androidx.compose.ui` dependency to `feature:widget` to support the use of `LocalConfiguration` and `LocalDensity`.

Signed-off-by: James Rich <[email protected]>