Costco Android · Code Review Dashboard
Granular review across 20 categories with 626 concrete findings, each with file path, line number, severity, and a recommended fix. Use the charts below to slice the report and export to Excel.
67
OVERALL
Total findings
626
39 critical · 163 high
Critical
39
Likely crashes / serious bugs
High
163
Crash-prone or deprecated
Medium / Low
413
Quality & maintainability
Modules with findings
24
Out of 56 total modules
Source files
13.4k
4,347 Kotlin · 9,039 Java
Visual breakdown
Severity distribution
All 626 findings by severity
Findings by category
Concentration across the 20 evaluation categories
Findings by finding-type
Crash, lifecycle, security, lint, performance, etc.
Top 12 modules by finding count
Where the work is concentrated
Category scores radar
Health profile across all 20 categories (0–100)
Severity by category (stacked)
How critical/high/medium/low spread across categories
Granular reports
▲ Crash Risks
NPE / unsafe casts / index access / WebView injection — every finding with file:line.
↻ Lifecycle Issues
Fragment getActivity(), observe(this), listener leaks, state-loss commits, NetworkCallback.
≣ Lint Report
println, Log.d, runBlocking, hardcoded URLs, deprecated APIs, magic numbers.
▦ Class-by-Class
Sortable, filterable, full-table view with Excel export.
Category scorecard
Healthy (≥75)
Needs attention (60–74)
At risk (<60)
Architecture & Module Structure
01
Strong modular layout, clean MVVM, repository pattern adopted at scale.
Score78/100 · 0 findings
Code Quality & Maintainability
02
197 TODO/FIXME hot-spots and Java/Kotlin split create maintenance drag.
Score65/100 · 37 findings
Kotlin / Java Interop & Modernization
03
Kotlin-first new code, but ~9k Java files dominate; migration is incomplete.
Score55/100 · 27 findings
Jetpack Compose & UI Layer
04
Compose adopted in feature modules; legacy Views still present in core flows.
Score70/100 · 0 findings
Concurrency & Coroutines
05
Clean coroutine usage, no GlobalScope abuse, dispatcher discipline is good.
Score80/100 · 11 findings
Dependency Injection (Hilt)
06
Comprehensive Hilt graph with 60+ modules and constructor injection.
Score82/100 · 0 findings
Networking & Data Layer
07
Retrofit + OkHttp with interceptors, repository pattern, BFF abstraction.
Score75/100 · 0 findings
Persistence & Storage
08
Room with migrations + DataStore + EncryptDecrypt utilities.
Score72/100 · 0 findings
Security
09
Solid network policy, but exported components and deep links need audit.
Score70/100 · 59 findings
Performance
10
Heavy app startup, dual image libs (Coil + Glide), no baseline profile.
Score60/100 · 76 findings
Testing
11
~280 test files covering ViewModels, mappers, repos; UI test coverage moderate.
Score73/100 · 66 findings
Error Handling & Logging
12
Timber-based logging and Crashlytics; sealed Result types inconsistent.
Score68/100 · 252 findings
Accessibility (a11y)
13
Almost zero contentDescription / semantics usage in Compose layer.
Score35/100 · 0 findings
Internationalization & Localization
14
EN + FR (incl. CA variants); plural/format handling needs verification.
Score62/100 · 15 findings
Build, Gradle & Dependencies
15
Modern AGP/Kotlin/Gradle with version catalog; some bleeding-edge versions.
Score80/100 · 0 findings
Resource Management
16
No values-night for dark mode; resource hygiene needs an audit pass.
Score60/100 · 0 findings
Lifecycle & State Management
17
ViewModels + Flow widely used; SavedStateHandle adoption mixed.
Score70/100 · 83 findings
Documentation & Comments
18
README is stale; KDoc coverage is low; Dokka configured but underused.
Score50/100 · 0 findings
Static Analysis & Lint
19
Lint baseline likely; Detekt/Ktlint integration needs confirmation.
Score65/100 · 0 findings
Privacy & Compliance
20
Background location + camera + mic permissions need consent flow review.
Score68/100 · 0 findings
Costco Android · Code Review Report · Generated 2026-05-07 · 626 machine-curated findings