Lint Report

Overview

	Lint
1	ObsoleteLintCustomCheck: Obsolete custom lint check
	Correctness
6	GradleDependency: Obsolete Gradle Dependency
5	NewerVersionAvailable: Newer Library Versions Available
4	ModifierParameter: Guidelines for Modifier parameters in a Composable function
	Performance
7	UnusedResources: Unused resources
	Productivity
7	UseKtx: Use KTX extension function
	Included Additional Checks (70)
	Disabled Checks (47)

Obsolete custom lint check

../../../../../.cache/bhdp/gradle-home/caches/9.9.1/transforms/26ace50a5136c6dadfc94dcb98a4525f/transformed/lifecycle-runtime-release/jars/lint.jar: Library lint checks reference invalid APIs; these checks will be skipped!

Lint found an issue registry (androidx.lifecycle.lint.LifecycleRuntimeIssueRegistry)
which contains some references to invalid API:
org.jetbrains.kotlin.asJava.elements.KtLightModifierList#getKotlinOrigin(): org.jetbrains.kotlin.psi.KtModifierList
(Referenced from androidx/lifecycle/lint/LifecycleWhenChecksKt.class)

Therefore, this lint check library is not included
in analysis. This affects the following lint checks:
UnsafeLifecycleWhenUsage,RepeatOnLifecycleWrongUsage

To use this lint check, upgrade to a more recent version
of the library.

ObsoleteLintCustomCheck Warning Priority 10/10

Obsolete Gradle Dependency

../../build.gradle.kts:53: A newer version of androidx.compose:compose-bom than 2025.08.01 is available: 2026.09.00

 50 }
 51 dependencies {
 52     // Mature, mutually compatible stable Compose/Kotlin line; no alpha dependencies.
 53     implementation(platform("androidx.compose:compose-bom:2025.08.01"))                             
 54     implementation("androidx.compose.ui:ui")
 55     implementation("androidx.compose.ui:ui-tooling-preview")
 56     implementation("androidx.compose.foundation:foundation")

../../build.gradle.kts:59: A newer version of androidx.lifecycle:lifecycle-runtime-compose than 2.9.4 is available: 2.11.0

 56     implementation("androidx.compose.foundation:foundation")
 57     implementation("androidx.compose.material3:material3")
 58     implementation("androidx.activity:activity-compose:1.13.0")
 59     implementation("androidx.lifecycle:lifecycle-runtime-compose:2.9.4")                            
 60     implementation("androidx.lifecycle:lifecycle-viewmodel-compose:2.9.4")
 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")
 62     implementation("androidx.navigation:navigation-compose:2.9.6")

../../build.gradle.kts:60: A newer version of androidx.lifecycle:lifecycle-viewmodel-compose than 2.9.4 is available: 2.11.0

 57     implementation("androidx.compose.material3:material3")
 58     implementation("androidx.activity:activity-compose:1.13.0")
 59     implementation("androidx.lifecycle:lifecycle-runtime-compose:2.9.4")
 60     implementation("androidx.lifecycle:lifecycle-viewmodel-compose:2.9.4")                          
 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")
 62     implementation("androidx.navigation:navigation-compose:2.9.6")
 63     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.10.2")

../../build.gradle.kts:61: A newer version of androidx.lifecycle:lifecycle-viewmodel-savedstate than 2.9.4 is available: 2.11.0

 58     implementation("androidx.activity:activity-compose:1.13.0")
 59     implementation("androidx.lifecycle:lifecycle-runtime-compose:2.9.4")
 60     implementation("androidx.lifecycle:lifecycle-viewmodel-compose:2.9.4")
 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")                       
 62     implementation("androidx.navigation:navigation-compose:2.9.6")
 63     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.10.2")
 64     implementation("com.squareup.okhttp3:okhttp:4.12.0")

../../build.gradle.kts:62: A newer version of androidx.navigation:navigation-compose than 2.9.6 is available: 2.10.2

 59     implementation("androidx.lifecycle:lifecycle-runtime-compose:2.9.4")
 60     implementation("androidx.lifecycle:lifecycle-viewmodel-compose:2.9.4")
 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")
 62     implementation("androidx.navigation:navigation-compose:2.9.6")                                  
 63     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.10.2")
 64     implementation("com.squareup.okhttp3:okhttp:4.12.0")
 65     implementation("org.jsoup:jsoup:1.24.1")

../../build.gradle.kts:70: A newer version of androidx.compose:compose-bom than 2025.08.01 is available: 2026.09.00

 67     testImplementation("org.json:json:20250517")
 68     testImplementation("com.squareup.okhttp3:mockwebserver:4.12.0")
 69     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.10.2")
 70     androidTestImplementation(platform("androidx.compose:compose-bom:2025.08.01"))                  
 71     androidTestImplementation("androidx.compose.ui:ui-test-junit4")
 72     androidTestImplementation("androidx.test:core:1.7.0")
 73     androidTestImplementation("androidx.test:runner:1.7.0")

GradleDependency Correctness Warning Priority 4/10

Newer Library Versions Available

../../build.gradle.kts:63: A newer version of org.jetbrains.kotlinx:kotlinx-coroutines-android than 1.10.2 is available: 1.11.0

 60     implementation("androidx.lifecycle:lifecycle-viewmodel-compose:2.9.4")
 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")
 62     implementation("androidx.navigation:navigation-compose:2.9.6")
 63     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.10.2")                       
 64     implementation("com.squareup.okhttp3:okhttp:4.12.0")
 65     implementation("org.jsoup:jsoup:1.24.1")
 66     testImplementation("junit:junit:4.13.2")

../../build.gradle.kts:64: A newer version of com.squareup.okhttp3:okhttp than 4.12.0 is available: 5.5.1

 61     implementation("androidx.lifecycle:lifecycle-viewmodel-savedstate:2.9.4")
 62     implementation("androidx.navigation:navigation-compose:2.9.6")
 63     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.10.2")
 64     implementation("com.squareup.okhttp3:okhttp:4.12.0")                                            
 65     implementation("org.jsoup:jsoup:1.24.1")
 66     testImplementation("junit:junit:4.13.2")
 67     testImplementation("org.json:json:20250517")

../../build.gradle.kts:67: A newer version of org.json:json than 20250517 is available: 20260814

 64     implementation("com.squareup.okhttp3:okhttp:4.12.0")
 65     implementation("org.jsoup:jsoup:1.24.1")
 66     testImplementation("junit:junit:4.13.2")
 67     testImplementation("org.json:json:20250517")                                                    
 68     testImplementation("com.squareup.okhttp3:mockwebserver:4.12.0")
 69     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.10.2")
 70     androidTestImplementation(platform("androidx.compose:compose-bom:2025.08.01"))

../../build.gradle.kts:68: A newer version of com.squareup.okhttp3:mockwebserver than 4.12.0 is available: 5.5.1

 65     implementation("org.jsoup:jsoup:1.24.1")
 66     testImplementation("junit:junit:4.13.2")
 67     testImplementation("org.json:json:20250517")
 68     testImplementation("com.squareup.okhttp3:mockwebserver:4.12.0")                                 
 69     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.10.2")
 70     androidTestImplementation(platform("androidx.compose:compose-bom:2025.08.01"))
 71     androidTestImplementation("androidx.compose.ui:ui-test-junit4")

../../build.gradle.kts:69: A newer version of org.jetbrains.kotlinx:kotlinx-coroutines-test than 1.10.2 is available: 1.11.0

 66     testImplementation("junit:junit:4.13.2")
 67     testImplementation("org.json:json:20250517")
 68     testImplementation("com.squareup.okhttp3:mockwebserver:4.12.0")
 69     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.10.2")                      
 70     androidTestImplementation(platform("androidx.compose:compose-bom:2025.08.01"))
 71     androidTestImplementation("androidx.compose.ui:ui-test-junit4")
 72     androidTestImplementation("androidx.test:core:1.7.0")

This detector checks with a central repository to see if there are newer versions available for the dependencies used by this project. This is similar to the GradleDependency check, which checks for newer versions available in the Android SDK tools and libraries, but this works with any MavenCentral dependency, and connects to the library every time, which makes it more flexible but also much slower.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.
To suppress this error, use the issue id "NewerVersionAvailable" as explained in the Suppressing Warnings and Errors section.

NewerVersionAvailable Correctness Warning Priority 4/10

Guidelines for Modifier parameters in a Composable function

../../src/main/java/com/bharatiessentials/deliverypartner/ui/DesignSystem.kt:110: Modifier parameter should be the first optional parameter

 107         Column(Modifier.padding(Space.Lg),verticalArrangement=Arrangement.spacedBy(Space.Md),content=content)
 108     }
 109 }
 110 @Composable fun PrimaryButton(text:String,onClick:()->Unit,enabled:Boolean=true,busy:Boolean=false,glyph:Glyph?=null,modifier:Modifier=Modifier){
 111     Button(onClick,modifier.fillMaxWidth().heightIn(min=UiSize.Button),enabled=enabled&&!busy,shape=ControlShape,
 112         contentPadding=PaddingValues(horizontal=Space.Lg,vertical=Space.Md),elevation=ButtonDefaults.buttonElevation(defaultElevation=Space.None,pressedElevation=Space.None)){
 113         if(busy){CircularProgressIndicator(Modifier.size(UiSize.Spinner),strokeWidth=UiSize.ProgressStroke,color=LocalContentColor.current);Spacer(Modifier.width(Space.Sm))}

../../src/main/java/com/bharatiessentials/deliverypartner/ui/DesignSystem.kt:118: Modifier parameter should be the first optional parameter

 115         Text(text,textAlign=TextAlign.Center,style=MaterialTheme.typography.labelLarge)
 116     }
 117 }
 118 @Composable fun SecondaryButton(text:String,onClick:()->Unit,glyph:Glyph?=null,enabled:Boolean=true,modifier:Modifier=Modifier){
 119     OutlinedButton(onClick,modifier.heightIn(min=UiSize.Touch),enabled=enabled,shape=ControlShape,border=BorderStroke(Space.Hairline,if(enabled)Line else Line.copy(alpha=.6f)),contentPadding=PaddingValues(horizontal=Space.Md,vertical=Space.Md)){
 120         glyph?.let{BIcon(it,Modifier.size(UiSize.IconSmall));Spacer(Modifier.width(Space.Sm))};Text(text,style=MaterialTheme.typography.labelLarge,textAlign=TextAlign.Center)
 121     }

../../src/main/java/com/bharatiessentials/deliverypartner/ui/DesignSystem.kt:140: Modifier parameter should be the first optional parameter

 137         Text(text,Modifier.padding(horizontal=Space.Sm,vertical=Space.Xs),style=MaterialTheme.typography.labelSmall,color=fg)
 138     }
 139 }
 140 @Composable fun InfoStrip(text:String,warning:Boolean=false,error:Boolean=false,modifier:Modifier=Modifier){
 141     val bg=if(error)DangerBg else if(warning)WarningBg else PaleGreen
 142     val fg=if(error)Danger else if(warning)WarningInk else DeepGreen
 143     Surface(modifier.fillMaxWidth(),color=bg,shape=ControlShape){

../../src/main/java/com/bharatiessentials/deliverypartner/ui/DesignSystem.kt:197: Optional Modifier parameter should have a default value of Modifier

 194 }
 195 
 196 enum class Glyph { Jobs, History, Cash, Alerts, Support, Arrow, Pin, Phone, More, Refresh, Back, Check, Close, Warning, Eye, EyeOff, Person, Logout, Chevron, Package, Shield, Chat, Location }
 197 @Composable fun BIcon(glyph: Glyph, modifier: Modifier = Modifier.size(UiSize.Icon), color: Color = LocalContentColor.current) {
 198     Canvas(modifier.clearAndSetSemantics { }) {
 199         scale(size.minDimension / 24f, pivot = Offset.Zero) {
 200             val stroke = Stroke(width = IconStroke, cap = StrokeCap.Round)

The first (or only) Modifier parameter in a Composable function should follow the following rules:
- Be named modifier
- Have a type of Modifier
- Either have no default value, or have a default value of Modifier
- If optional, be the first optional parameter in the parameter list
To suppress this error, use the issue id "ModifierParameter" as explained in the Suppressing Warnings and Errors section.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ModifierParameter Correctness Warning Priority 3/10

Unused resources

../../src/main/res/values/colors.xml:2: The resource R.color.brand_green_dark appears to be unused

 1 <resources>
 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>         
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>

../../src/main/res/values/colors.xml:3: The resource R.color.soft_bg appears to be unused

 1 <resources>
 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>                        
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>
 6 <color name="background_shell">#F5F7F2</color></resources>

../../src/main/res/values/colors.xml:4: The resource R.color.ink_muted appears to be unused

 1 <resources>
 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>                        
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>
 6 <color name="background_shell">#F5F7F2</color></resources>

../../src/main/res/values/colors.xml:4: The resource R.color.ink appears to be unused

 1 <resources>
 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>                        
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>
 6 <color name="background_shell">#F5F7F2</color></resources>

../../src/main/res/values/colors.xml:5: The resource R.color.warning_bg appears to be unused

 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>               
 6 <color name="background_shell">#F5F7F2</color></resources>

../../src/main/res/values/colors.xml:5: The resource R.color.warning_ink appears to be unused

 2     <color name="brand_green">#1A6B4F</color><color name="brand_green_dark">#156158</color>
 3     <color name="paper">#FFFFFF</color><color name="soft_bg">#F4F7F5</color>
 4     <color name="ink">#1A2623</color><color name="ink_muted">#55665F</color>
 5     <color name="warning_bg">#FFF3DC</color><color name="warning_ink">#62450C</color>               
 6 <color name="background_shell">#F5F7F2</color></resources>

../../src/main/res/values/strings.xml:1: The resource R.string.app_name appears to be unused

 1 <resources><string name="app_name">Bharati Delivery Partner</string></resources>

Unused resources make applications larger and slow down builds.

The unused resource check can ignore tests. If you want to include resources that are only referenced from tests, consider packaging them in a test source set instead.

You can include test sources in the unused resource check by setting the system property lint.unused-resources.include-tests=true, and to exclude them (usually for performance reasons), use lint.unused-resources.exclude-tests=true.

This check can be configured via the following options:

skip-libraries (default is true):
Whether the unused resource check should skip reporting unused resources in libraries.

Many libraries will declare resources that are part of the library surface; other modules depending on the library will also reference the resources. To avoid reporting all these resources as unused (in the context of a library), the unused resource check normally skips reporting unused resources in libraries. Instead, run the unused resource check on the consuming app module (along with checkDependencies=true).

However, there are cases where you want to check that all the resources declared in a library are used; in that case, you can disable the skip option.

To configure this option, use a `lint.xml` file in the project or source folder using an <option> block like the following:

 1 <lint>
 2     <issue id="UnusedResources">
 3         <option name="skip-libraries" value="true" />                                               
 4     </issue>
 5 </lint>

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.
To suppress this error, use the issue id "UnusedResources" as explained in the Suppressing Warnings and Errors section.

UnusedResources Performance Warning Priority 3/10

Use KTX extension function

../../src/main/java/com/bharatiessentials/deliverypartner/alerts/AssignmentAlertRuntime.kt:24: Use the KTX extension function SharedPreferences.edit instead?

 21   @Synchronized override fun save(state: AssignmentAlertLedgerState): Boolean {
 22       val riders = JSONArray().apply { state.baselinedRiders.filter { it > 0 }.sorted().forEach(::put) }
 23       val identities = JSONArray().apply { state.handledIdentityKeys.takeLast(MAX_KEYS).forEach(::put) }
 24       return prefs.edit().putString("baselined_riders", riders.toString())                        
 25           .putString("handled_identities", identities.toString()).commit()
 26   }

../../src/main/java/com/bharatiessentials/deliverypartner/ui/NativeApp.kt:200: Use the KTX extension function String.toUri instead?

 197 }
 198 fun openExternal(context:Context,uri:String?){
 199     if(uri==null||!ExternalLinks.safeExternal(uri)){Toast.makeText(context,"Verified destination unavailable. Contact dispatch.",Toast.LENGTH_LONG).show();return}
 200     try{context.startActivity(Intent(if(uri.startsWith("tel:"))Intent.ACTION_DIAL else Intent.ACTION_VIEW,Uri.parse(uri)))}
 201     catch(_:ActivityNotFoundException){Toast.makeText(context,"No compatible app is installed for this action.",Toast.LENGTH_LONG).show()}
 202     catch(_:SecurityException){Toast.makeText(context,"Android blocked this external action.",Toast.LENGTH_LONG).show()}
 203 }

../../src/main/java/com/bharatiessentials/deliverypartner/alerts/NewOrderAlerts.kt:113: Use the KTX extension function String.toUri instead?

 110       .setContentType(AudioAttributes.CONTENT_TYPE_SONIFICATION)
 111       .build()
 112 
 113   fun ownerSoundUri(context: Context): Uri = Uri.parse("android.resource://${context.packageName}/${R.raw.new_order_alert}")
 114 
 115   fun createChannels(context: Context) {
 116       // minSdk 29 guarantees notification-channel APIs are available.

../../src/main/java/com/bharatiessentials/deliverypartner/data/SecureStore.kt:44: Use the KTX extension function SharedPreferences.edit instead?

  41           cipher.init(Cipher.DECRYPT_MODE, secret(), GCMParameterSpec(128, bytes.copyOfRange(0, 12)))
  42           cipher.updateAAD(key.toByteArray(Charsets.UTF_8))
  43           cipher.doFinal(bytes.copyOfRange(12, bytes.size)).toString(Charsets.UTF_8)
  44       } catch (_: Exception) { prefs.edit().clear().commit(); null }                              
  45   }
  46   @Synchronized override fun put(key: String, value: String) {
  47       val cipher = Cipher.getInstance("AES/GCM/NoPadding")

../../src/main/java/com/bharatiessentials/deliverypartner/data/SecureStore.kt:50: Use the KTX extension function SharedPreferences.edit instead?

  47       val cipher = Cipher.getInstance("AES/GCM/NoPadding")
  48       cipher.init(Cipher.ENCRYPT_MODE, secret()); cipher.updateAAD(key.toByteArray(Charsets.UTF_8))
  49       val payload = cipher.iv + cipher.doFinal(value.toByteArray(Charsets.UTF_8))
  50       check(prefs.edit().putString(key, Base64.encodeToString(payload, Base64.NO_WRAP)).commit()) { "Secure session storage unavailable" }
  51   }
  52   @Synchronized override fun remove(key: String) { prefs.edit().remove(key).commit() }
  53   @Synchronized override fun clear() { prefs.edit().clear().commit() }

../../src/main/java/com/bharatiessentials/deliverypartner/data/SecureStore.kt:52: Use the KTX extension function SharedPreferences.edit instead?

  49         val payload = cipher.iv + cipher.doFinal(value.toByteArray(Charsets.UTF_8))
  50         check(prefs.edit().putString(key, Base64.encodeToString(payload, Base64.NO_WRAP)).commit()) { "Secure session storage unavailable" }
  51     }
  52     @Synchronized override fun remove(key: String) { prefs.edit().remove(key).commit() }            
  53     @Synchronized override fun clear() { prefs.edit().clear().commit() }
  54 }

../../src/main/java/com/bharatiessentials/deliverypartner/data/SecureStore.kt:53: Use the KTX extension function SharedPreferences.edit instead?

  50         check(prefs.edit().putString(key, Base64.encodeToString(payload, Base64.NO_WRAP)).commit()) { "Secure session storage unavailable" }
  51     }
  52     @Synchronized override fun remove(key: String) { prefs.edit().remove(key).commit() }
  53     @Synchronized override fun clear() { prefs.edit().clear().commit() }                            
  54 }
  55 
  56 /** Native persistence of WP session cookies never extends their server expiry; local upper bound is 48 hours. */

UseKtx Productivity Warning Priority 6/10

Included Additional Checks

This card lists all the extra checks run by lint, provided from libraries, build configuration and extra flags. This is included to help you verify whether a particular check is included in analysis when configuring builds. (Note that the list does not include the hundreds of built-in checks into lint, only additional ones.)

ArcAnimationSpecTypeIssue

ArcAnimationSpec is designed for 2D values. Particularly, for positional values such as Offset.
Trying to use it for values of different dimensions (Float, Size, Color, etc.) will result in unpredictable animation behavior.

Vendor: Jetpack Compose
Identifier: androidx.compose.animation.core
Feedback: https://issuetracker.google.com/issues/new?component=612128

AutoboxingStateCreation

Calling mutableStateOf<T>() when T is either backed by a primitive type on the JVM or is a value class results in a state implementation that requires all state values to be boxed. This usually causes an additional allocation for each state write, and adds some additional work to auto-unbox values when reading the value of the state. Instead, prefer to use a specialized primitive state implementation for Int, Long, Float, and Double when the state does not need to track null values and does not override the default SnapshotMutationPolicy. See mutableIntStateOf(), mutableLongStateOf(), mutableFloatStateOf(), and mutableDoubleStateOf() for more information.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

AutoboxingStateValueProperty

Avoid using the generic value property when using a specialized State type. Reading or writing to the state's generic value property will result in an unnecessary autoboxing operation. Prefer the specialized value property (e.g. intValue for MutableIntState), or use property delegation to avoid unnecessary allocations.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

ComposableDestinationInComposeScope

Composable destinations should only be constructed directly within a NavGraphBuilder scope. Composable destinations cannot be nested, and you should use the navigation function to create a nested graph instead.

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

ComposableNaming

@Composable functions without a return type should use similar naming to classes, starting with an uppercase letter and ending with a noun. @Composable functions with a return type should be treated as normal Kotlin functions, starting with a lowercase letter.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

ComposableNavGraphInComposeScope

Composable destinations should only be constructed directly within a NavGraphBuilder scope.

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

CompositionLocalNaming

CompositionLocal properties should be prefixed with Local. This helps make it clear at their use site that these values are local to the current composition. Typically the full name will be Local + the type of the CompositionLocal, for example val LocalFoo = compositionLocalOf { Foo() }.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

ConfigurationScreenWidthHeight

Configuration.screenWidthDp and Configuration.screenHeightDp have different insets behaviour depending on target SDK version, and are rounded to the nearest Dp. This means that using these values in composition to size a layout can result in issues, as these values do not accurately represent the actual available window size. Instead it is recommended to use WindowInfo.containerSize which accurately represents the window size.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ConstantContentStateKeyInItemsCall

When using shared elements (Modifier.sharedElement or Modifier.sharedBounds) in a LazyList, each Composable within an items block should have its own unique key. Otherwise, only one item of the visible layout will be rendered, regardless if there's an ongoing shared element transition.

Vendor: Jetpack Compose
Identifier: androidx.compose.animation
Feedback: https://issuetracker.google.com/issues/new?component=612128

ContextCastToActivity

Casting Context to Activity is an error as Contexts are not always Activities. Use LocalActivity instead

Vendor: Jetpack Activity Compose
Identifier: androidx.activity.compose
Feedback: https://issuetracker.google.com/issues/new?component=612128

CoroutineCreationDuringComposition

Creating a coroutine with async or launch during composition is often incorrect - this means that a coroutine will be created even if the composition fails / is rolled back, and it also means that multiple coroutines could end up mutating the same state, causing inconsistent results. Instead, use LaunchedEffect and create coroutines inside the suspending block. The block will only run after a successful composition, and will cancel existing coroutines when key changes, allowing correct cleanup.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

DeepLinkInActivityDestination

Attaching a <deeplink> to an <activity> destination will never give the right behavior when using an implicit deep link on another app's task (where the system back should immediately take the user back to the app that triggered the deep link). Instead, attach the deep link directly to the second activity (either by manually writing the appropriate <intent-filter> or by adding the <deeplink> to the start destination of a nav host in that second activity).

Vendor: Android Open Source Project
Identifier: androidx.navigation.runtime
Feedback: https://issuetracker.google.com/issues/new?component=409828

EmptyNavDeepLink

Attempting to create an empty NavDeepLink will result in an IllegalStateException at runtime. You may set these arguments within the lambda of the call to navDeepLink.

Vendor: Android Open Source Project
Identifier: androidx.navigation.common
Feedback: https://issuetracker.google.com/issues/new?component=409828

EnsureInitializerMetadata

When a library defines a Initializer, it needs to be accompanied by a corresponding <meta-data> entry in the AndroidManifest.xml file.

Vendor: Android Open Source Project
Identifier: androidx.startup
Feedback: https://issuetracker.google.com/issues/new?component=823348

EnsureInitializerNoArgConstr

Every Initializer must have a no argument constructor.

Vendor: Android Open Source Project
Identifier: androidx.startup
Feedback: https://issuetracker.google.com/issues/new?component=823348

ExperimentalAnnotationRetention

Experimental annotations defined in Java source should use default (CLASS) retention, while Kotlin-sourced annotations should use BINARY retention.

Vendor: Android Open Source Project
Identifier: androidx.annotation.experimental
Feedback: https://issuetracker.google.com/issues/new?component=459778

FlowOperatorInvokedInComposition

Calling a Flow operator function within composition will result in a new Flow being created every recomposition, which will reset collectAsState() and cause other related problems. Instead Flow operators should be called inside remember, or a side effect such as LaunchedEffect.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

FrequentlyChangingValue

Reading a value annotated with @FrequentlyChangingValue inside composition can cause performance issues due to frequent recompositions. To avoid frequent recompositions, instead consider:

- Using derivedStateOf to filter state changes based on a provided calculation. For example, rather than recomposing on every scroll position change, only recomposing if the scroll position changes from 0 (at the top of the list) to greater than 0 (not at the top of the list), and vice versa.
- Using snapshotFlow to create a flow of changes from a provided state. This can then be collected inside a LaunchedEffect, and used to make changes without needing to recompose.
- If using Compose UI, read this value inside measure / layout / draw, depending on where it is needed. This will cause invalidation of the corresponding phase, instead of a recomposition. See developer.android.com for more information on Jetpack Compose phases.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

InvalidColorHexValue

Vendor: Jetpack Compose
Identifier: androidx.compose.ui.graphics
Feedback: https://issuetracker.google.com/issues/new?component=612128

InvalidFragmentVersionForActivityResult

In order to use the ActivityResult APIs you must upgrade your Fragment version to 1.3.0. Previous versions of FragmentActivity failed to call super.onRequestPermissionsResult() and used invalid request codes

More info: https://developer.android.com/training/permissions/requesting#make-the-request

Vendor: Android Open Source Project
Identifier: androidx.activity
Feedback: https://issuetracker.google.com/issues/new?component=527362

InvalidLanguageTagDelimiter

A language tag must be compliant with IETF BCP47, specifically a sequence of subtags must be separated by hyphens (-) instead of underscores (_)

Vendor: Jetpack Compose
Identifier: androidx.compose.ui.text
Feedback: https://issuetracker.google.com/issues/new?component=779818

InvalidUseOfOnBackPressed

You should not used OnBackPressedCallback for non-UI cases. If you
|add a callback, you have to handle back completely in the callback.

More info: https://developer.android.com/guide/navigation/custom-back/predictive-back-gesture#ui-logic

Vendor: Android Open Source Project
Identifier: androidx.activity
Feedback: https://issuetracker.google.com/issues/new?component=527362

LaunchDuringComposition

Calling launch during composition is incorrect. Doing so will cause launch to be called multiple times resulting in a RuntimeException. Instead, use SideEffect and launch inside of the suspending block. The block will only run after a successful composition.

Vendor: Jetpack Activity Compose
Identifier: androidx.activity.compose
Feedback: https://issuetracker.google.com/issues/new?component=612128

LifecycleCurrentStateInComposition

Calling Lifecycle.currentState within composition will not observe changes to the Lifecycle, so changes might not be reflected within the composition. Instead you should use lifecycle.currentStateAsState() to observe changes to the Lifecycle, and recompose when it changes.

Vendor: Android Open Source Project
Identifier: androidx.lifecycle
Feedback: https://issuetracker.google.com/issues/new?component=413132

LocalContextConfigurationRead

Changes to the Configuration object will not cause LocalContext reads to be invalidated, so you may end up with stale values when the Configuration changes. Instead, use LocalConfiguration.current to retrieve the Configuration - this will recompose callers when the Configuration object changes.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

LocalContextResourcesRead

Changes to the Configuration object will not cause LocalContext.current.resources reads to be invalidated, so calls to APIs suchas Resources.getString() will not be updated when the Configuration changes. Instead, use LocalResources.current to retrieve the Resources - this will invalidate callers when the Configuration changes, to ensure that these calls reflect the latest values.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

MissingColorAlphaChannel

Creating a Color with a hex value requires a 32 bit value (such as 0xFF000000), with 8 bits being used per channel (ARGB). Not passing a full 32 bit value will result in channels being undefined. For example, passing 0xFF0000 will result in a missing alpha channel, so the color will not appear visible.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui.graphics
Feedback: https://issuetracker.google.com/issues/new?component=612128

MissingKeepAnnotation

Type-safe nav arguments such as Enum types can get incorrectly obfuscated in minified builds when not referenced directly

Vendor: Android Open Source Project
Identifier: androidx.navigation.runtime
Feedback: https://issuetracker.google.com/issues/new?component=409828

MissingKeepAnnotation

Type-safe nav arguments such as Enum types can get incorrectly obfuscated in minified builds when not referenced directly

Vendor: Android Open Source Project
Identifier: androidx.navigation.common
Feedback: https://issuetracker.google.com/issues/new?component=409828

MissingKeepAnnotation

Type-safe nav arguments such as Enum types can get incorrectly obfuscated in minified builds when not referenced directly

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

MissingSerializableAnnotation

The destination needs to be annotated with @Serializable in order for Navigation library to convert the class or object declaration into a NavDestination.

Vendor: Android Open Source Project
Identifier: androidx.navigation.runtime
Feedback: https://issuetracker.google.com/issues/new?component=409828

MissingSerializableAnnotation

The destination needs to be annotated with @Serializable in order for Navigation library to convert the class or object declaration into a NavDestination.

Vendor: Android Open Source Project
Identifier: androidx.navigation.common
Feedback: https://issuetracker.google.com/issues/new?component=409828

MissingSerializableAnnotation

The destination needs to be annotated with @Serializable in order for Navigation library to convert the class or object declaration into a NavDestination.

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

ModifierFactoryExtensionFunction

Modifier factory functions should be defined as extension functions on Modifier to allow modifiers to be fluently chained.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ModifierFactoryReturnType

Modifier factory functions should return Modifier as their type, and not a subtype of Modifier (such as Modifier.Element).

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ModifierFactoryUnreferencedReceiver

Modifier factory functions are fluently chained to construct a chain of Modifier objects that will be applied to a layout. As a result, each factory function must use the receiver Modifier parameter, to ensure that the function is returning a chain that includes previous items in the chain. Make sure the returned chain either explicitly includes this, such as return this.then(MyModifier) or implicitly by returning a chain that starts with an implicit call to another factory function, such as return myModifier(), where myModifier is defined as fun Modifier.myModifier(): Modifier.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ModifierNodeInspectableProperties

ModifierNodeElements may override inspectableProperties() to provide information about the modifier in the layout inspector. The default implementation attempts to read all of the properties on the class reflectively, which may not comprehensively or effectively describe the modifier.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

ModifierParameter

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

MultipleAwaitPointerEventScopes

Pointer Input events are queued inside awaitPointerEventScope. Multiple calls to awaitPointerEventScope may exit the scope. During this time there is no guarantee that the events will be queued and some events may be dropped. It is recommended to use a single top-level block and perform the pointer events processing within such block.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

MutableCollectionMutableState

Writes to mutable collections inside a MutableState will not cause a recomposition - only writes to the MutableState itself will. In most cases you should either use a read-only collection (such as List or Map) and assign a new instance to the MutableState when your data changes, or you can use an snapshot-backed collection such as SnapshotStateList or SnapshotStateMap which will correctly cause a recomposition when their contents are modified.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

NoCollectCallFound

You must call collect on the progress in the onBack function. The collect call is what properly splits the callback so it knows what to do when the back gestures is started vs when it is completed. Failing to call collect will cause all code in the block to run when the gesture is started.

Vendor: Jetpack Activity Compose
Identifier: androidx.activity.compose
Feedback: https://issuetracker.google.com/issues/new?component=612128

NullSafeMutableLiveData

This check ensures that LiveData values are not null when explicitly declared as non-nullable.

Kotlin interoperability does not support enforcing explicit null-safety when using generic Java type parameters. Since LiveData is a Java class its value can always be null even when its type is explicitly declared as non-nullable. This can lead to runtime exceptions from reading a null LiveData value that is assumed to be non-nullable.

Vendor: Android Open Source Project
Identifier: androidx.lifecycle
Feedback: https://issuetracker.google.com/issues/new?component=413132

OpaqueUnitKey

Certain Compose functions including remember, LaunchedEffect, and DisposableEffect declare (and sometimes require) one or more key parameters. When a key parameter changes, it is a signal that the previous invocation is now invalid. In certain cases, it may be required to pass Unit as a key to one of these functions, indicating that the invocation never becomes invalid. Using Unit as a key should be done infrequently, and should always be done explicitly by passing the Unit literal. This inspection checks for invocations where Unit is being passed as a key argument in any form other than the Unit literal. This is usually done by mistake, and can harm readability. If a Unit expression is being passed as a key, it is always equivalent to move the expression before the function invocation and pass the Unit literal instead.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

ProduceStateDoesNotAssignValue

produceState returns an observable State using values assigned inside the producer lambda. If the lambda never assigns (i.e value = foo), then the State will never change. Make sure to assign a value when the source you are producing values from changes / emits a new value. For sample usage see the produceState documentation.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

RememberInComposition

APIs annotated with @RememberInComposition must not be called inside composition. This can lead to correctness issues and / or performance issues. Instead, use remember to cache the value across compositions, or hoist this call outside of composition.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

RememberReturnType

A call to remember that returns Unit is always an error. This typically happens when using remember to mutate variables on an object. remember is executed during the composition, which means that if the composition fails or is happening on a separate thread, the mutated variables may not reflect the true state of the composition. Instead, use SideEffect to make deferred changes once the composition succeeds, or mutate MutableState backed variables directly, as these will handle composition failure for you.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

RememberSaveableSaverParameter

The first parameter to rememberSaveable is a vararg parameter for inputs that when changed will cause the state to reset. Passing a Saver object to this parameter is an error, as the intention is to pass the Saver object to the saver parameter. Since the saver parameter is not the first parameter, it must be explicitly named.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime.saveable
Feedback: https://issuetracker.google.com/issues/new?component=612128

ReturnFromAwaitPointerEventScope

Pointer Input events are queued inside awaitPointerEventScope. By using the return value of awaitPointerEventScope one might unexpectedly lose events. If another awaitPointerEventScope is restarted there is no guarantee that the events will persist between those calls. In this case you should keep all events inside the awaitPointerEventScope block

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

StateFlowValueCalledInComposition

Calling StateFlow.value within composition will not observe changes to the StateFlow, so changes might not be reflected within the composition. Instead you should use stateFlow.collectAsState() to observe changes to the StateFlow, and recompose when it changes.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

SuspiciousCompositionLocalModifierRead

Jetpack Compose is unable to send updated values of a CompositionLocal when it's read in a Modifier.Node's initializer and onAttach() or onDetach() callbacks. Modifier.Node's callbacks are not aware of snapshot reads, and their lifecycle callbacks are not invoked on every recomposition. If you read a CompositionLocal in onAttach() or onDetach(), you will only get the CompositionLocal's value once at the moment of the read, which may lead to unexpected behaviors. We recommend instead reading CompositionLocals at time-of-use in callbacks that apply your Modifier's behavior, like measure() for LayoutModifierNode, draw() for DrawModifierNode, and so on. To observe the value of the CompositionLocal manually, extend from the ObserverNode interface and place the read inside an observeReads {} block within the onObservedReadsChanged() callback.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

SuspiciousModifierThen

Calling a Modifier factory function with an implicit receiver inside Modifier.then will result in the receiver (this) being added twice to the chain. For example, fun Modifier.myModifier() = this.then(otherModifier()) - the implementation of factory functions such as Modifier.otherModifier() will internally call this.then(...) to chain the provided modifier with their implementation. When you expand this.then(otherModifier()), it becomes: this.then(this.then(OtherModifierImplementation)) - so you can see that this is included twice in the chain, which results in modifiers such as padding being applied twice, for example. Instead, you should either remove the then() and directly chain the factory function on the receiver, this.otherModifier(), or add the empty Modifier as the receiver for the factory, such as this.then(Modifier.otherModifier())

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

TestManifestGradleConfiguration

The androidx.compose.ui:ui-test-manifest dependency is needed for launching a Compose host, such as with createComposeRule. However, it only needs to be present in testing configurations therefore use this dependency with the debugImplementation configuration

More info: https://developer.android.com/jetpack/compose/testing#setup

Vendor: Android Open Source Project
Identifier: androidx.compose.ui.test.manifest
Feedback: https://issuetracker.google.com/issues/new?component=741505

UnnecessaryComposedModifier

Modifier.composed allows invoking @Composable functions when creating a Modifier instance - for example, using remember to have instance-specific state, allowing the same Modifier object to be safely used in multiple places. Using Modifier.composed without calling any @Composable functions inside is unnecessary, and since the Modifier is no longer skippable, this can cause a lot of extra work inside the composed body, leading to worse performance.

Vendor: Jetpack Compose
Identifier: androidx.compose.ui
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnrememberedGetBackStackEntry

Backstack entries retrieved during composition need to be `remember`ed, otherwise they will be retrieved from the navController again, and be changed. You also need to pass in a key of a NavBackStackEntry to the remember call or they will not be updated properly. If this is in a NavGraphBuilder.composable scope, you should pass in the lambda's given entry as the key. Either hoist the state to an object that is not created during composition, or wrap the state in a call to remember with a NavBackStackEntry as a key.

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

UnrememberedMutableState

State objects created during composition need to be `remember`ed, otherwise they will be recreated during recomposition, and lose their state. Either hoist the state to an object that is not created during composition, or wrap the state in a call to remember.

Vendor: Jetpack Compose
Identifier: androidx.compose.runtime
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnsafeOptInUsageError

This API has been flagged as opt-in with error-level severity.

Any declaration annotated with this marker is considered part of an unstable or
otherwise non-standard API surface and its call sites should accept the opt-in
aspect of it by using the @OptIn annotation, using the marker annotation --
effectively causing further propagation of the opt-in aspect -- or configuring
the UnsafeOptInUsageError check's options for project-wide opt-in.

To configure project-wide opt-in, specify the opt-in option value in lint.xml
as a comma-delimited list of opted-in annotations:

<lint>
    <issue id="UnsafeOptInUsageError">
        <option name="opt-in" value="com.foo.ExperimentalBarAnnotation" />
    </issue>
</lint>

Vendor: Android Open Source Project
Identifier: androidx.annotation.experimental
Feedback: https://issuetracker.google.com/issues/new?component=459778

UnsafeOptInUsageWarning

This API has been flagged as opt-in with warning-level severity.

Any declaration annotated with this marker is considered part of an unstable or
otherwise non-standard API surface and its call sites should accept the opt-in
aspect of it by using the @OptIn annotation, using the marker annotation --
effectively causing further propagation of the opt-in aspect -- or configuring
the UnsafeOptInUsageWarning check's options for project-wide opt-in.

To configure project-wide opt-in, specify the opt-in option value in lint.xml
as a comma-delimited list of opted-in annotations:

<lint>
    <issue id="UnsafeOptInUsageWarning">
        <option name="opt-in" value="com.foo.ExperimentalBarAnnotation" />
    </issue>
</lint>

Vendor: Android Open Source Project
Identifier: androidx.annotation.experimental
Feedback: https://issuetracker.google.com/issues/new?component=459778

UnusedBoxWithConstraintsScope

The content lambda in BoxWithConstraints has a scope which will include the incoming constraints. If this scope is ignored, then the cost of subcomposition is being wasted and this BoxWithConstraints should be replaced with a Box.

Vendor: Jetpack Compose
Identifier: androidx.compose.foundation
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnusedContentLambdaTargetStateParameter

content lambda in AnimatedContent works as a lookup function that returns the corresponding content based on the parameter (a state of type T). It is important for this lambda to return content specific to the input parameter, so that the different contents can be properly animated. Not using the input parameter to the content lambda will result in the same content for different input (i.e. target state) and therefore an erroneous transition between the exact same content.`

Vendor: Jetpack Compose
Identifier: androidx.compose.animation
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnusedCrossfadeTargetStateParameter

content lambda in Crossfade works as a lookup function that returns the corresponding content based on the parameter (a state of type T). It is important for this lambda to return content specific to the input parameter, so that the different contents can be properly crossfaded. Not using the input parameter to the content lambda will result in the same content for different input (i.e. target state) and therefore an erroneous crossfade between the exact same content.`

Vendor: Jetpack Compose
Identifier: androidx.compose.animation
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnusedSharedTransitionModifierParameter

When using SharedTransitionScope the provided Modifier should always be used on the top-most child, as the Modifier both obtains the root coordinates and creates an overlay. Otherwise, consider using SharedTransitionLayout.

Vendor: Jetpack Compose
Identifier: androidx.compose.animation
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnusedTargetStateInContentKeyLambda

In AnimatedContent, the contentKey lambda may be used when the targetState is expected to mutate frequently but not all mutations are desired to be considered a target state change. So contentKey is expected to always use the given targetState parameter to calculate its result.

Vendor: Jetpack Compose
Identifier: androidx.compose.animation
Feedback: https://issuetracker.google.com/issues/new?component=612128

UnusedTransitionTargetStateParameter

Transition.animate* functions provide a target state parameter in the lambda that will be used to calculate the value for a given state. This target state parameter in the lambda may or may not be the same as the actual state, as the animation system occasionally needs to look up target values for other states to do proper seeking/tooling preview. Relying on other state than the provided targetState could also result in unnecessary recompositions. Therefore, it is generally considered an error if this targetState parameter is not used.

Vendor: Jetpack Compose
Identifier: androidx.compose.animation.core
Feedback: https://issuetracker.google.com/issues/new?component=612128

UseOfNonLambdaOffsetOverload

Modifier.offset() is recommended to be used with static arguments only to avoid unnecessary recompositions. Modifier.offset{ } is preferred in the cases where the arguments are backed by a State.

Vendor: Jetpack Compose
Identifier: androidx.compose.foundation
Feedback: https://issuetracker.google.com/issues/new?component=612128

ViewModelConstructorInComposable

View models should not be constructed directly inside composable functions. Instead you should use the lifecycle viewmodel extensionfunctions e.g. viewModel<MyViewModel>()

Vendor: Android Open Source Project
Identifier: androidx.lifecycle
Feedback: https://issuetracker.google.com/issues/new?component=413132

WrongNavigateRouteType

If the destination class contains arguments, the route is expected to be class instance with the arguments filled in.

Vendor: Android Open Source Project
Identifier: androidx.navigation.runtime
Feedback: https://issuetracker.google.com/issues/new?component=409828

WrongRequiresOptIn

Experimental features defined in Kotlin source code must be annotated with the Kotlin
@RequiresOptIn annotation. Using androidx.annotation.RequiresOptIn will prevent the
Kotlin compiler from enforcing its opt-in policies.

Vendor: Android Open Source Project
Identifier: androidx.annotation.experimental
Feedback: https://issuetracker.google.com/issues/new?component=459778

WrongStartDestinationType

If the startDestination contains arguments, the arguments must be provided to navigation via a fully formed route (a class instance with argumentsfilled in), or else it will be treated as a case of missing arguments.

Vendor: Android Open Source Project
Identifier: androidx.navigation.runtime
Feedback: https://issuetracker.google.com/issues/new?component=409828

WrongStartDestinationType

Vendor: Android Open Source Project
Identifier: androidx.navigation.common
Feedback: https://issuetracker.google.com/issues/new?component=409828

WrongStartDestinationType

Vendor: Jetpack Navigation Compose
Identifier: androidx.navigation.compose

Disabled Checks

One or more issues were not run by lint, either because the check is not enabled by default, or because it was disabled with a command line flag or via one or more lint.xml configuration files in the project directories.

AppCompatMethod

Disabled By: Default

When using the appcompat library, there are some methods you should be calling instead of the normal ones; for example, getSupportActionBar() instead of getActionBar(). This lint check looks for calls to the wrong method.

More info: https://developer.android.com/topic/libraries/support-library/

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

AppLinksAutoVerify

Disabled By: Default

Ensures that app links are correctly set and associated with website.

More info: https://g.co/appindexing/applinks

BackButton

Disabled By: Default

According to the Android Design Guide,

"Other platforms use an explicit back button with label to allow the user to navigate up the application's hierarchy. Instead, Android uses the main action bar's app icon for hierarchical navigation and the navigation bar's back button for temporal navigation."

This check is not very sophisticated (it just looks for buttons with the label "Back"), so it is disabled by default to not trigger on common scenarios like pairs of Back/Next buttons to paginate through screens.

More info: https://d.android.com/r/studio-ui/designer/material/design

ComposableLambdaParameterNaming

Disabled By: Default

Composable functions with only one composable lambda parameter should use the name content for the parameter.

ComposableLambdaParameterPosition

Disabled By: Default

Composable functions with only one composable lambda parameter should place the parameter at the end of the parameter list, so it can be used as a trailing lambda.

ConvertToWebp

Disabled By: Default

The WebP format is typically more compact than PNG and JPEG. As of Android 4.2.1 it supports transparency and lossless conversion as well. Note that there is a quickfix in the IDE which lets you perform conversion.

Previously, launcher icons were required to be in the PNG format but that restriction is no longer there, so lint now flags these.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

DefaultEncoding

Disabled By: Default

Some APIs will implicitly use the default system character encoding instead of UTF-8 when converting to or from bytes, such as when creating a default FileReader.

This is usually not correct; you only want to do this if you need to read files created by other programs where they have deliberately written in the same encoding. The default encoding varies from platform to platform and can vary from locale to locale, so this makes it difficult to interpret files containing non-ASCII characters.

We recommend using UTF-8 everywhere.

Note that on Android, the default file encoding is always UTF-8 (see https://developer.android.com/reference/java/nio/charset/Charset#defaultCharset() for more), so this lint check deliberately does not flag any problems in Android code, since it is always safe to rely on the default character encoding there.

DuplicateStrings

Disabled By: Default

Duplicate strings can make applications larger unnecessarily.

This lint check looks for duplicate strings, including differences for strings where the only difference is in capitalization. Title casing and all uppercase can all be adjusted in the layout or in code.

More info: https://developer.android.com/reference/android/widget/TextView.html#attr_android:inputType

EasterEgg

Disabled By: Default

An "easter egg" is code deliberately hidden in the code, both from potential users and even from other developers. This lint check looks for code which looks like it may be hidden from sight.

ExpensiveAssertion

Disabled By: Default

In Kotlin, assertions are not handled the same way as from the Java programming language. In particular, they're just implemented as a library call, and inside the library call the error is only thrown if assertions are enabled.

This means that the arguments to the assert call will always be evaluated. If you're doing any computation in the expression being asserted, that computation will unconditionally be performed whether or not assertions are turned on. This typically turns into wasted work in release builds.

This check looks for cases where the assertion condition is nontrivial, e.g. it is performing method calls or doing more work than simple comparisons on local variables or fields.

You can work around this by writing your own inline assert method instead:

@Suppress("INVISIBLE_REFERENCE", "INVISIBLE_MEMBER")
inline fun assert(condition: () -> Boolean) {
    if (_Assertions.ENABLED && !condition()) {
        throw AssertionError()
    }
}

In Android, because assertions are not enforced at runtime, instead use this:

inline fun assert(condition: () -> Boolean) {
    if (BuildConfig.DEBUG && !condition()) {
        throw AssertionError()
    }
}

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

GuardedBy

Disabled By: Default

The GuardedBy analysis checks that fields or methods annotated with @GuardedBy(lock) are only accessed when the specified lock is held.

More info: https://errorprone.info/bugpattern/GuardedBy

IconExpectedSize

Disabled By: Default

There are predefined sizes (for each density) for launcher icons. You should follow these conventions to make sure your icons fit in with the overall look of the platform.

More info: https://d.android.com/r/studio-ui/designer/material/iconography

Immutable

Disabled By: Default

This check validates that all classes annotated with Error Prone's @Immutable annotation are deeply immutable. It also checks that any class extending an @Immutable-annotated class or implementing an @Immutable-annotated interface are also immutable.

More info: https://errorprone.info/bugpattern/Immutable

InvalidPackage

Disabled By: Default

This check scans through libraries looking for calls to APIs that are not included in Android.

When you create Android projects, the classpath is set up such that you can only access classes in the API packages that are included in Android. However, if you add other projects to your libs/ folder, there is no guarantee that those .jar files were built with an Android specific classpath, and in particular, they could be accessing unsupported APIs such as java.applet.

This check scans through library jars and looks for references to API packages that are not included in Android and flags these. This is only an error if your code calls one of the library classes which wind up referencing the unsupported package.

KotlinPropertyAccess

Disabled By: Default

For a method to be represented as a property in Kotlin, strict “bean”-style prefixing must be used.

Accessor methods require a get prefix or for boolean-returning methods an is prefix can be used.

More info: https://android.github.io/kotlin-guides/interop.html#property-prefixes

KotlincFE10

Disabled By: Default

K2, the new version of Kotlin compiler, which encompasses the new frontend, is coming. Try to avoid using internal APIs from the old frontend if possible.

LambdaLast

Disabled By: Default

To improve calling this code from Kotlin, parameter types eligible for SAM conversion should be last.

More info: https://android.github.io/kotlin-guides/interop.html#lambda-parameters-last

LintDocExample

Disabled By: Default

Lint's tool for generating documentation for each issue has special support for including a code example which shows how to trigger the report. It will pick the first unit test it can find and pick out the source file referenced from the error message, but you can instead designate a unit test to be the documentation example, and in that case, all the files are included.

To designate a unit test as the documentation example for an issue, name the test testDocumentationExample, or if your detector reports multiple issues, testDocumentationExample<Id>, such as testDocumentationExampleMyId.

LintImplPsiEquals

Disabled By: Default

You should never compare two PSI elements for equality with equals; use PsiEquivalenceUtil.areElementsEquivalent(PsiElement, PsiElement) instead.

LintImplUnexpectedDomain

Disabled By: Default

This checks flags URLs to domains that have not been explicitly allowed for use as a documentation source.

LogConditional

Disabled By: Default

The BuildConfig class provides a constant, DEBUG, which indicates whether the code is being built in release mode or in debug mode. In release mode, you typically want to strip out all the logging calls. Since the compiler will automatically remove all code which is inside a if (false) check, surrounding your logging calls with a check for BuildConfig.DEBUG is a good idea.

If you really intend for the logging to be present in release mode, you can suppress this warning with a @SuppressLint annotation for the intentional logging calls.

MangledCRLF

Disabled By: Default

On Windows, line endings are typically recorded as carriage return plus newline: \r\n.

This detector looks for invalid line endings with repeated carriage return characters (without newlines). Previous versions of the ADT plugin could accidentally introduce these into the file, and when editing the file, the editor could produce confusing visual artifacts.

More info: https://bugs.eclipse.org/bugs/show_bug.cgi?id=375421

MemberExtensionConflict

Disabled By: Default

When both member and extension declarations are applicable, the resolution takes the member. This also implies that, if an extension existed first, but then a member is added later, the same call-site may end up with different call resolutions depending on target environment. This results in a potential runtime exception if the generated binary (library or app) targets earlier environment (i.e., without the new member, but only extension). More concrete example is found at: https://issuetracker.google.com/issues/350432371

MinSdkTooLow

Disabled By: Default

The value of the minSdkVersion property is too low and can be incremented without noticeably reducing the number of supported devices.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

NegativeMargin

Disabled By: Default

Margin values should be positive. Negative values are generally a sign that you are making assumptions about views surrounding the current one, or may be tempted to turn off child clipping to allow a view to escape its parent. Turning off child clipping to do this not only leads to poor graphical performance, it also results in wrong touch event handling since touch events are based strictly on a chain of parent-rect hit tests. Finally, making assumptions about the size of strings can lead to localization problems.

NoHardKeywords

Disabled By: Default

Do not use Kotlin’s hard keywords as the name of methods or fields. These require the use of backticks to escape when calling from Kotlin. Soft keywords, modifier keywords, and special identifiers are allowed.

For example, ActionEvent's getWhen() method requires backticks when used from Kotlin:

val timestamp = event.`when`

More info: https://android.github.io/kotlin-guides/interop.html#no-hard-keywords

NoOp

Disabled By: Default

This check looks for code which looks like it's a no-op -- usually leftover expressions from interactive debugging, but in some cases bugs where you had intended to do something with the expression such as assign it to a field.

This check can be configured via the following options:

pure-getters (default is false):
Whether to assume methods with getter-names have no side effects.

Getter methods (where names start with get or is, and have non-void return types, and no arguments) should not have side effects. With this option turned on, lint will assume that is the case and will list any getter calls whose results are ignored as suspicious code.

To configure this option, use a `lint.xml` file in the project or source folder using an <option> block like the following:

 1 <lint>
 2     <issue id="NoOp">
 3         <option name="pure-getters" value="false" />                                                
 4     </issue>
 5 </lint>

PermissionNamingConvention

Disabled By: Default

Permissions should be prefixed with an app's package name, using reverse-domain-style naming. This prefix should be followed by .permission., and then a description of the capability that the permission represents, in upper SNAKE_CASE. For example, com.example.myapp.permission.ENGAGE_HYPERSPACE.

Following this recommendation avoids naming collisions, and helps clearly identify the owner and intention of a custom permission.

ReflectionAnnotation

Disabled By: Default

When methods are using reflection to access APIs, they should declare what APIs they are accessing using the new @UsesReflectionToAccessMethod and @UsesReflectionToAccessField annotations. This makes it possible for the shrinker (R8) to correctly remove unused code and resources without accidentally also removing reflectively accessed code, which can lead to program crashes.

Registered

Disabled By: Default

Activities, services and content providers should be registered in the AndroidManifest.xml file using <activity>, <service> and <provider> tags.

If your activity is simply a parent class intended to be subclassed by other "real" activities, make it an abstract class.

More info: https://developer.android.com/guide/topics/manifest/manifest-intro.html

RequiredSize

Disabled By: Default

All views must specify an explicit layout_width and layout_height attribute. There is a runtime check for this, so if you fail to specify a size, an exception is thrown at runtime.

It's possible to specify these widths via styles as well. GridLayout, as a special case, does not require you to specify a size.

SelectableText

Disabled By: Default

If a <TextView> is used to display data, the user might want to copy that data and paste it elsewhere. To allow this, the <TextView> should specify android:textIsSelectable="true".

This lint check looks for TextViews which are likely to be displaying data: views whose text is set dynamically.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

StopShip

Disabled By: Default

Using the comment // STOPSHIP can be used to flag code that is incomplete but checked in. This comment marker can be used to indicate that the code should not be shipped until the issue is addressed, and lint will look for these. In Gradle projects, this is only checked for non-debug (release) builds.

In Kotlin, the TODO() method is also treated as a stop ship marker; you can use it to make incomplete code compile, but it will throw an exception at runtime and therefore should be fixed before shipping releases.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

StringFormatTrivial

Disabled By: Default

Every call to String.format creates a new Formatter instance, which will decrease the performance of your app. String.format should only be used when necessary--if the formatted string contains only trivial conversions (e.g. b, s, c) and there are no translation concerns, it will be more efficient to replace them and concatenate with +.

SyntheticAccessor

Disabled By: Default

A private inner class which is accessed from the outer class will force the compiler to insert a synthetic accessor; this means that you are causing extra overhead. This is not important in small projects, but is important for large apps running up against the 64K method handle limit, and especially for libraries where you want to make sure your library is as small as possible for the cases where your library is used in an app running up against the 64K limit.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

TypographyQuotes

Disabled By: Default

Straight single quotes and double quotes, when used as a pair, can be replaced by "curvy quotes" (or directional quotes). Use the right single quotation mark for apostrophes. Never use generic quotes ", ' or free-standing accents `, ´ for quotation marks, apostrophes, or primes. This can make the text more readable.

More info: https://en.wikipedia.org/wiki/Quotation_mark

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

UnknownNullness

Disabled By: Default

To improve referencing this code from Kotlin, consider adding explicit nullness information here with either @NonNull or @Nullable.

This check can be configured via the following options:

ignore-deprecated (default is false):
Whether to ignore classes and members that have been annotated with @Deprecated.

Normally this lint check will flag all unannotated elements, but by setting this option to true it will skip any deprecated elements.

To configure this option, use a `lint.xml` file in the project or source folder using an <option> block like the following:

 1 <lint>
 2     <issue id="UnknownNullness">
 3         <option name="ignore-deprecated" value="false" />                                           
 4     </issue>
 5 </lint>

More info: https://developer.android.com/kotlin/interop#nullability_annotations

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

UnsupportedChromeOsHardware

Disabled By: Default

The <uses-feature> element should not require this unsupported large screen hardware feature. Any <uses-feature> not explicitly marked with required="false" is necessary on the device to be installed on. Ensure that any features that might prevent it from being installed on a ChromeOS, large screen, or foldable device are reviewed and marked as not required in the manifest.

More info: https://developer.android.com/topic/arc/manifest.html#incompat-entries

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

UnusedIds

Disabled By: Default

This resource id definition appears not to be needed since it is not referenced from anywhere. Having id definitions, even if unused, is not necessarily a bad idea since they make working on layouts and menus easier, so there is not a strong reason to delete these.

The unused resource check can ignore tests. If you want to include resources that are only referenced from tests, consider packaging them in a test source set instead.

You can include test sources in the unused resource check by setting the system property lint.unused-resources.include-tests=true, and to exclude them (usually for performance reasons), use lint.unused-resources.exclude-tests=true.

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

ValidActionsXml

Disabled By: Default

Ensures that an actions XML file is properly formed
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

VulnerableCordovaVersion

Disabled By: Default

The version of Cordova used in the app is vulnerable to security issues. Please update to the latest Apache Cordova version.

More info: https://cordova.apache.org/announcements/2015/11/20/security.html

WatchFaceEditor

Disabled By: Default

Watch face editor activities must be able to launch in the Wear OS companion app activity task in order to work correctly. Thus only launchMode="standard" is allowed. The watch face will not be shown on the watch if it does not satisfy this requirement.
Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

WatchFaceForAndroidX

Disabled By: Default

If the package depends on androidx.wear:wear-watchface, and an AndroidX watch face declares the wearableConfigurationAction metadata, its value should be androidx.wear.watchface.editor.action.WATCH_FACE_EDITOR.

More info: https://developer.android.com/training/wearables/watch-faces/configuration

Note: This issue has an associated quickfix operation in Android Studio and IntelliJ IDEA.

WearableActionDuplicate

Disabled By: Default

If and only if a watch face service defines wearableConfigurationAction metadata, with the value WATCH_FACE_EDITOR, there should be an activity in the same package, which has an intent filter for WATCH_FACE_EDITOR (with com.google.android.wearable.watchface.category.WEARABLE_CONFIGURATION if minSdkVersion is less than 30).

More info: https://developer.android.com/training/wearables/watch-faces/configuration

WearableConfigurationAction

Disabled By: Default

Only when a watch face service defines wearableConfigurationAction metadata, with the value WATCH_FACE_EDITOR, there should be an activity in the same package, which has an intent filter for WATCH_FACE_EDITOR (with com.google.android.wearable.watchface.category.WEARABLE_CONFIGURATION if minSdkVersion is less than 30).

More info: https://developer.android.com/training/wearables/watch-faces/configuration

WrongThread

Disabled By: Default

Ensures that a method which expects to be called on a specific thread, is actually called from that thread. For example, calls on methods in widgets should always be made on the UI thread.

More info: https://developer.android.com/guide/components/processes-and-threads.html#Threads

WrongThreadInterprocedural

Disabled By: Default

Searches for interprocedural call paths that violate thread annotations in the program. Tracks the flow of instantiated types and lambda expressions to increase accuracy across method boundaries.

More info: https://developer.android.com/guide/components/processes-and-threads.html#Threads

Suppressing Warnings and Errors

Lint errors can be suppressed in a variety of ways:

1. With a @SuppressLint annotation in the Java code
2. With a tools:ignore attribute in the XML file
3. With a //noinspection comment in the source code
4. With ignore flags specified in the build.gradle file, as explained below
5. With a lint.xml configuration file in the project
6. With a lint.xml configuration file passed to lint via the --config flag
7. With the --ignore flag passed to lint.

To suppress a lint warning with an annotation, add a @SuppressLint("id") annotation on the class, method or variable declaration closest to the warning instance you want to disable. The id can be one or more issue id's, such as "UnusedResources" or {"UnusedResources","UnusedIds"}, or it can be "all" to suppress all lint warnings in the given scope.

To suppress a lint warning with a comment, add a //noinspection id comment on the line before the statement with the error.

To suppress a lint warning in an XML file, add a tools:ignore="id" attribute on the element containing the error, or one of its surrounding elements. You also need to define the namespace for the tools prefix on the root element in your document, next to the xmlns:android declaration:
xmlns:tools="http://schemas.android.com/tools"

To suppress a lint warning in a build.gradle file, add a section like this:

android {
    lintOptions {
        disable 'TypographyFractions','TypographyQuotes'
    }
}

Here we specify a comma separated list of issue id's after the disable command. You can also use warning or error instead of disable to change the severity of issues.

To suppress lint warnings with a configuration XML file, create a file named lint.xml and place it at the root directory of the module in which it applies.

The format of the lint.xml file is something like the following:

<?xml version="1.0" encoding="UTF-8"?>
<lint>
    <!-- Ignore everything in the test source set -->
    <issue id="all">
        <ignore path="\*/test/\*" />
    </issue>

    <!-- Disable this given check in this project -->
    <issue id="IconMissingDensityFolder" severity="ignore" />

    <!-- Ignore the ObsoleteLayoutParam issue in the given files -->
    <issue id="ObsoleteLayoutParam">
        <ignore path="res/layout/activation.xml" />
        <ignore path="res/layout-xlarge/activation.xml" />
        <ignore regexp="(foo|bar)\.java" />
    </issue>

    <!-- Ignore the UselessLeaf issue in the given file -->
    <issue id="UselessLeaf">
        <ignore path="res/layout/main.xml" />
    </issue>

    <!-- Change the severity of hardcoded strings to "error" -->
    <issue id="HardcodedText" severity="error" />
</lint>

To suppress lint checks from the command line, pass the --ignore flag with a comma separated list of ids to be suppressed, such as:
$ lint --ignore UnusedResources,UselessLeaf /my/project/path

For more information, see https://developer.android.com/studio/write/lint.html#config