1.3.0 - Graphing Fixes

.idea/deploymentTargetSelector.xml

@@ -4,6 +4,14 @@
     <selectionStates>
       <SelectionState runConfigName="app">
         <option name="selectionMode" value="DROPDOWN" />
+        <DropdownSelection timestamp="2025-08-27T22:55:36.064836Z">
+          <Target type="DEFAULT_BOOT">
+            <handle>
+              <DeviceId pluginId="LocalEmulator" identifier="path=/Users/atridad/.android/avd/Pixel_9a.avd" />
+            </handle>
+          </Target>
+        </DropdownSelection>
+        <DialogSelection />
       </SelectionState>
     </selectionStates>
   </component>

app/build.gradle.kts

@@ -1,3 +1,5 @@
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget
+
 plugins {
     alias(libs.plugins.android.application)
     alias(libs.plugins.kotlin.android)
@@ -12,10 +14,10 @@ android {
 
     defaultConfig {
         applicationId = "com.atridad.openclimb"
-        minSdk = 33
+        minSdk = 34
         targetSdk = 36
-        versionCode = 15
+        versionCode = 19
-        versionName = "1.0.1"
+        versionName = "1.3.0"
 
         testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
     }
@@ -33,9 +35,6 @@ android {
         sourceCompatibility = JavaVersion.VERSION_17
         targetCompatibility = JavaVersion.VERSION_17
     }
-    kotlinOptions {
-        jvmTarget = "17"
-    }
     
     java {
         toolchain {
@@ -48,6 +47,12 @@ android {
     }
 }
 
+kotlin {
+    compilerOptions {
+        jvmTarget.set(JvmTarget.JVM_17)
+    }
+}
+
 dependencies {
     // Core Android libraries
     implementation(libs.androidx.core.ktx)
@@ -64,6 +69,7 @@ dependencies {
     // Room Database
     implementation(libs.androidx.room.runtime)
     implementation(libs.androidx.room.ktx)
+
     ksp(libs.androidx.room.compiler)
     
     // Navigation
@@ -81,6 +87,8 @@ dependencies {
     // Image Loading
     implementation(libs.coil.compose)
     
+
+    
     // Testing
     testImplementation(libs.junit)
     testImplementation(libs.mockk)

app/src/main/AndroidManifest.xml

@@ -8,11 +8,13 @@
         android:maxSdkVersion="28" />
     <uses-permission android:name="android.permission.CAMERA" />
     
+    <!-- Hardware features -->
+    <uses-feature android:name="android.hardware.camera" android:required="false" />
+    
     <!-- Permissions for notifications and foreground service -->
     <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE_SPECIAL_USE" />
-    <uses-permission android:name="android.permission.WAKE_LOCK" />
     
     <application
         android:allowBackup="true"
@@ -52,7 +54,8 @@
             android:name=".service.SessionTrackingService"
             android:enabled="true"
             android:exported="false"
-            android:foregroundServiceType="specialUse">
+            android:foregroundServiceType="specialUse"
+            android:description="@string/session_tracking_service_description">
             <meta-data
                 android:name="android.app.foreground_service_type"
                 android:value="specialUse" />

app/src/main/java/com/atridad/openclimb/data/database/dao/AttemptDao.kt

@@ -53,6 +53,9 @@ interface AttemptDao {
     @Query("SELECT COUNT(*) FROM attempts")
     suspend fun getAttemptsCount(): Int
     
+    @Query("DELETE FROM attempts")
+    suspend fun deleteAllAttempts()
+    
     @Query("SELECT COUNT(*) FROM attempts WHERE sessionId = :sessionId")
     suspend fun getAttemptsCountBySession(sessionId: String): Int
     

app/src/main/java/com/atridad/openclimb/data/database/dao/ClimbSessionDao.kt

@@ -59,6 +59,9 @@ interface ClimbSessionDao {
     @Query("SELECT * FROM climb_sessions WHERE status = 'ACTIVE' ORDER BY date DESC LIMIT 1")
     suspend fun getActiveSession(): ClimbSession?
     
+    @Query("DELETE FROM climb_sessions")
+    suspend fun deleteAllSessions()
+    
     @Query("SELECT * FROM climb_sessions WHERE status = 'ACTIVE' ORDER BY date DESC LIMIT 1")
     fun getActiveSessionFlow(): Flow<ClimbSession?>
 }

app/src/main/java/com/atridad/openclimb/data/database/dao/GymDao.kt

@@ -37,4 +37,7 @@ interface GymDao {
     
     @Query("SELECT * FROM gyms WHERE name LIKE '%' || :searchQuery || '%' OR location LIKE '%' || :searchQuery || '%'")
     fun searchGyms(searchQuery: String): Flow<List<Gym>>
+    
+    @Query("DELETE FROM gyms")
+    suspend fun deleteAllGyms()
 }

app/src/main/java/com/atridad/openclimb/data/database/dao/ProblemDao.kt

@@ -59,4 +59,10 @@ interface ProblemDao {
         ORDER BY updatedAt DESC
     """)
     fun searchProblems(searchQuery: String): Flow<List<Problem>>
+    
+    @Query("SELECT COUNT(*) FROM problems")
+    suspend fun getProblemsCount(): Int
+    
+    @Query("DELETE FROM problems")
+    suspend fun deleteAllProblems()
 }

app/src/main/java/com/atridad/openclimb/data/repository/ClimbRepository.kt

@@ -64,181 +64,148 @@ class ClimbRepository(
     
 
     
-    // JSON Export
+    // ZIP Export with images - Single format for reliability
-    suspend fun exportAllDataToJson(directory: File? = null): File {
-        val exportDir = directory ?: File(context.getExternalFilesDir(Environment.DIRECTORY_DOCUMENTS), "OpenClimb")
-        if (!exportDir.exists()) {
-            exportDir.mkdirs()
-        }
-        
-        val timestamp = LocalDateTime.now().toString().replace(":", "-").replace(".", "-")
-        val exportFile = File(exportDir, "openclimb_export_$timestamp.json")
-        
-        val allGyms = gymDao.getAllGyms().first()
-        val allProblems = problemDao.getAllProblems().first()
-        val allSessions = sessionDao.getAllSessions().first()
-        val allAttempts = attemptDao.getAllAttempts().first()
-        
-        val exportData = ClimbDataExport(
-            exportedAt = LocalDateTime.now().toString(),
-            gyms = allGyms,
-            problems = allProblems,
-            sessions = allSessions,
-            attempts = allAttempts
-        )
-        
-        val jsonString = json.encodeToString(exportData)
-        exportFile.writeText(jsonString)
-        
-        return exportFile
-    }
-    
-    suspend fun exportAllDataToUri(context: Context, uri: android.net.Uri) {
-        val gyms = gymDao.getAllGyms().first()
-        val problems = problemDao.getAllProblems().first()
-        val sessions = sessionDao.getAllSessions().first()
-        val attempts = attemptDao.getAllAttempts().first()
-        
-        val exportData = ClimbDataExport(
-            exportedAt = LocalDateTime.now().toString(),
-            gyms = gyms,
-            problems = problems,
-            sessions = sessions,
-            attempts = attempts
-        )
-        
-        val jsonString = json.encodeToString(exportData)
-        
-        context.contentResolver.openOutputStream(uri)?.use { outputStream ->
-            outputStream.write(jsonString.toByteArray())
-        } ?: throw Exception("Could not open output stream")
-    }
-    
-    suspend fun importDataFromJson(file: File) {
-        try {
-            val jsonContent = file.readText()
-            val importData = json.decodeFromString<ClimbDataExport>(jsonContent)
-            
-            // Import gyms
-            importData.gyms.forEach { gym ->
-                try {
-                    gymDao.insertGym(gym)
-                } catch (_: Exception) {
-                    // If insertion fails, update instead
-                    gymDao.updateGym(gym)
-                }
-            }
-            
-            // Import problems
-            importData.problems.forEach { problem ->
-                try {
-                    problemDao.insertProblem(problem)
-                } catch (_: Exception) {
-                    problemDao.updateProblem(problem)
-                }
-            }
-            
-            // Import sessions
-            importData.sessions.forEach { session ->
-                try {
-                    sessionDao.insertSession(session)
-                } catch (_: Exception) {
-                    sessionDao.updateSession(session)
-                }
-            }
-            
-            // Import attempts
-            importData.attempts.forEach { attempt ->
-                try {
-                    attemptDao.insertAttempt(attempt)
-                } catch (_: Exception) {
-                    attemptDao.updateAttempt(attempt)
-                }
-            }
-            
-        } catch (e: Exception) {
-            throw Exception("Failed to import data: ${e.message}")
-        }
-    }
-    
-    // ZIP Export with images
     suspend fun exportAllDataToZip(directory: File? = null): File {
-        val allGyms = gymDao.getAllGyms().first()
+        try {
-        val allProblems = problemDao.getAllProblems().first()
+            // Collect all data with proper error handling
-        val allSessions = sessionDao.getAllSessions().first()
+            val allGyms = gymDao.getAllGyms().first()
-        val allAttempts = attemptDao.getAllAttempts().first()
+            val allProblems = problemDao.getAllProblems().first()
-        
+            val allSessions = sessionDao.getAllSessions().first()
-        val exportData = ClimbDataExport(
+            val allAttempts = attemptDao.getAllAttempts().first()
-            exportedAt = LocalDateTime.now().toString(),
+            
-            gyms = allGyms,
+            // Validate data integrity before export
-            problems = allProblems,
+            validateDataIntegrity(allGyms, allProblems, allSessions, allAttempts)
-            sessions = allSessions,
+            
-            attempts = allAttempts
+            val exportData = ClimbDataExport(
-        )
+                exportedAt = LocalDateTime.now().toString(),
-        
+                version = "1.0",
-        // Collect all referenced image paths
+                gyms = allGyms,
-        val referencedImagePaths = allProblems.flatMap { it.imagePaths }.toSet()
+                problems = allProblems,
-        
+                sessions = allSessions,
-        return ZipExportImportUtils.createExportZip(
+                attempts = allAttempts
-            context = context,
+            )
-            exportData = exportData,
+            
-            referencedImagePaths = referencedImagePaths,
+            // Collect all referenced image paths and validate they exist
-            directory = directory
+            val referencedImagePaths = allProblems.flatMap { it.imagePaths }.toSet()
-        )
+            val validImagePaths = referencedImagePaths.filter { imagePath ->
+                try {
+                    val imageFile = com.atridad.openclimb.utils.ImageUtils.getImageFile(context, imagePath)
+                    imageFile.exists() && imageFile.length() > 0
+                } catch (e: Exception) {
+                    false
+                }
+            }.toSet()
+            
+            // Log any missing images for debugging
+            val missingImages = referencedImagePaths - validImagePaths
+            if (missingImages.isNotEmpty()) {
+                android.util.Log.w("ClimbRepository", "Some referenced images are missing: $missingImages")
+            }
+            
+            return ZipExportImportUtils.createExportZip(
+                context = context,
+                exportData = exportData,
+                referencedImagePaths = validImagePaths,
+                directory = directory
+            )
+        } catch (e: Exception) {
+            throw Exception("Export failed: ${e.message}")
+        }
     }
     
     suspend fun exportAllDataToZipUri(context: Context, uri: android.net.Uri) {
-        val gyms = gymDao.getAllGyms().first()
+        try {
-        val problems = problemDao.getAllProblems().first()
+            // Collect all data with proper error handling
-        val sessions = sessionDao.getAllSessions().first()
+            val allGyms = gymDao.getAllGyms().first()
-        val attempts = attemptDao.getAllAttempts().first()
+            val allProblems = problemDao.getAllProblems().first()
-        
+            val allSessions = sessionDao.getAllSessions().first()
-        val exportData = ClimbDataExport(
+            val allAttempts = attemptDao.getAllAttempts().first()
-            exportedAt = LocalDateTime.now().toString(),
+            
-            gyms = gyms,
+            // Validate data integrity before export
-            problems = problems,
+            validateDataIntegrity(allGyms, allProblems, allSessions, allAttempts)
-            sessions = sessions,
+            
-            attempts = attempts
+            val exportData = ClimbDataExport(
-        )
+                exportedAt = LocalDateTime.now().toString(),
-        
+                version = "1.0",
-        // Collect all image paths
+                gyms = allGyms,
-        val referencedImagePaths = problems.flatMap { it.imagePaths }.toSet()
+                problems = allProblems,
-        
+                sessions = allSessions,
-        ZipExportImportUtils.createExportZipToUri(
+                attempts = allAttempts
-            context = context,
+            )
-            uri = uri,
+            
-            exportData = exportData,
+            // Collect all referenced image paths and validate they exist
-            referencedImagePaths = referencedImagePaths
+            val referencedImagePaths = allProblems.flatMap { it.imagePaths }.toSet()
-        )
+            val validImagePaths = referencedImagePaths.filter { imagePath ->
+                try {
+                    val imageFile = com.atridad.openclimb.utils.ImageUtils.getImageFile(context, imagePath)
+                    imageFile.exists() && imageFile.length() > 0
+                } catch (e: Exception) {
+                    false
+                }
+            }.toSet()
+            
+            ZipExportImportUtils.createExportZipToUri(
+                context = context,
+                uri = uri,
+                exportData = exportData,
+                referencedImagePaths = validImagePaths
+            )
+        } catch (e: Exception) {
+            throw Exception("Export failed: ${e.message}")
+        }
     }
     
     suspend fun importDataFromZip(file: File) {
         try {
-            val importResult = ZipExportImportUtils.extractImportZip(context, file)
+            // Validate the ZIP file
-            val importData = json.decodeFromString<ClimbDataExport>(importResult.jsonContent)
+            if (!file.exists() || file.length() == 0L) {
+                throw Exception("Invalid ZIP file: file is empty or doesn't exist")
+            }
             
-            // Update problem image paths with the new imported paths
+            // Extract and validate the ZIP contents
+            val importResult = ZipExportImportUtils.extractImportZip(context, file)
+            
+            // Validate JSON content
+            if (importResult.jsonContent.isBlank()) {
+                throw Exception("Invalid ZIP file: no data.json found or empty content")
+            }
+            
+            // Parse and validate the data structure
+            val importData = try {
+                json.decodeFromString<ClimbDataExport>(importResult.jsonContent)
+            } catch (e: Exception) {
+                throw Exception("Invalid data format: ${e.message}")
+            }
+            
+            // Validate data integrity
+            validateImportData(importData)
+            
+            // Clear existing data to avoid conflicts
+            attemptDao.deleteAllAttempts()
+            sessionDao.deleteAllSessions()
+            problemDao.deleteAllProblems()
+            gymDao.deleteAllGyms()
+            
+            // Import gyms first (problems depend on gyms)
+            importData.gyms.forEach { gym ->
+                try {
+                    gymDao.insertGym(gym)
+                } catch (e: Exception) {
+                    throw Exception("Failed to import gym ${gym.name}: ${e.message}")
+                }
+            }
+            
+            // Import problems with updated image paths
             val updatedProblems = ZipExportImportUtils.updateProblemImagePaths(
                 importData.problems,
                 importResult.importedImagePaths
             )
             
-            // Import gyms
-            importData.gyms.forEach { gym ->
-                try {
-                    gymDao.insertGym(gym)
-                } catch (e: Exception) {
-                    // If insertion fails update instead
-                    gymDao.updateGym(gym)
-                }
-            }
-            
-            // Import problems with updated image paths
             updatedProblems.forEach { problem ->
                 try {
                     problemDao.insertProblem(problem)
                 } catch (e: Exception) {
-                    problemDao.updateProblem(problem)
+                    throw Exception("Failed to import problem ${problem.name}: ${e.message}")
                 }
             }
             
@@ -247,21 +214,100 @@ class ClimbRepository(
                 try {
                     sessionDao.insertSession(session)
                 } catch (e: Exception) {
-                    sessionDao.updateSession(session)
+                    throw Exception("Failed to import session: ${e.message}")
                 }
             }
             
-            // Import attempts
+            // Import attempts last (depends on problems and sessions)
             importData.attempts.forEach { attempt ->
                 try {
                     attemptDao.insertAttempt(attempt)
                 } catch (e: Exception) {
-                    attemptDao.updateAttempt(attempt)
+                    throw Exception("Failed to import attempt: ${e.message}")
                 }
             }
             
         } catch (e: Exception) {
-            throw Exception("Failed to import data: ${e.message}")
+            throw Exception("Import failed: ${e.message}")
+        }
+    }
+    
+    private fun validateDataIntegrity(
+        gyms: List<Gym>,
+        problems: List<Problem>,
+        sessions: List<ClimbSession>,
+        attempts: List<Attempt>
+    ) {
+        // Validate that all problems reference valid gyms
+        val gymIds = gyms.map { it.id }.toSet()
+        val invalidProblems = problems.filter { it.gymId !in gymIds }
+        if (invalidProblems.isNotEmpty()) {
+            throw Exception("Data integrity error: ${invalidProblems.size} problems reference non-existent gyms")
+        }
+        
+        // Validate that all sessions reference valid gyms
+        val invalidSessions = sessions.filter { it.gymId !in gymIds }
+        if (invalidSessions.isNotEmpty()) {
+            throw Exception("Data integrity error: ${invalidSessions.size} sessions reference non-existent gyms")
+        }
+        
+        // Validate that all attempts reference valid problems and sessions
+        val problemIds = problems.map { it.id }.toSet()
+        val sessionIds = sessions.map { it.id }.toSet()
+        
+        val invalidAttempts = attempts.filter { 
+            it.problemId !in problemIds || it.sessionId !in sessionIds 
+        }
+        if (invalidAttempts.isNotEmpty()) {
+            throw Exception("Data integrity error: ${invalidAttempts.size} attempts reference non-existent problems or sessions")
+        }
+    }
+    
+    private fun validateImportData(importData: ClimbDataExport) {
+        if (importData.gyms.isEmpty()) {
+            throw Exception("Import data is invalid: no gyms found")
+        }
+        
+        if (importData.version.isBlank()) {
+            throw Exception("Import data is invalid: no version information")
+        }
+        
+        // Check for reasonable data sizes to prevent malicious imports
+        if (importData.gyms.size > 1000 || 
+            importData.problems.size > 10000 || 
+            importData.sessions.size > 10000 || 
+            importData.attempts.size > 100000) {
+            throw Exception("Import data is too large: possible corruption or malicious file")
+        }
+    }
+    
+    suspend fun resetAllData() {
+        try {
+            // Clear all data from database
+            attemptDao.deleteAllAttempts()
+            sessionDao.deleteAllSessions()
+            problemDao.deleteAllProblems()
+            gymDao.deleteAllGyms()
+            
+            // Clear all images from storage
+            clearAllImages()
+            
+        } catch (e: Exception) {
+            throw Exception("Reset failed: ${e.message}")
+        }
+    }
+    
+    private fun clearAllImages() {
+        try {
+            // Get the images directory
+            val imagesDir = File(context.filesDir, "images")
+            if (imagesDir.exists() && imagesDir.isDirectory) {
+                val deletedCount = imagesDir.listFiles()?.size ?: 0
+                imagesDir.deleteRecursively()
+                android.util.Log.i("ClimbRepository", "Cleared $deletedCount image files")
+            }
+        } catch (e: Exception) {
+            android.util.Log.w("ClimbRepository", "Failed to clear some images: ${e.message}")
         }
     }
 }
@@ -269,6 +315,7 @@ class ClimbRepository(
 @kotlinx.serialization.Serializable
 data class ClimbDataExport(
     val exportedAt: String,
+    val version: String = "1.0",
     val gyms: List<Gym>,
     val problems: List<Problem>,
     val sessions: List<ClimbSession>,

app/src/main/java/com/atridad/openclimb/service/SessionTrackingService.kt

@@ -7,7 +7,6 @@ import android.app.Service
 import android.content.Context
 import android.content.Intent
 import android.os.IBinder
-import android.os.PowerManager
 import androidx.core.app.NotificationCompat
 import com.atridad.openclimb.MainActivity
 import com.atridad.openclimb.R
@@ -17,13 +16,13 @@ import kotlinx.coroutines.*
 import kotlinx.coroutines.flow.firstOrNull
 import java.time.LocalDateTime
 import java.time.temporal.ChronoUnit
+import kotlinx.coroutines.runBlocking
 
 class SessionTrackingService : Service() {
     
     private val serviceScope = CoroutineScope(Dispatchers.IO + SupervisorJob())
     private var notificationJob: Job? = null
     private var monitoringJob: Job? = null
-    private var wakeLock: PowerManager.WakeLock? = null
     
     private lateinit var repository: ClimbRepository
     private lateinit var notificationManager: NotificationManager
@@ -58,7 +57,6 @@ class SessionTrackingService : Service() {
         notificationManager = getSystemService(NOTIFICATION_SERVICE) as NotificationManager
         
         createNotificationChannel()
-        acquireWakeLock()
     }
     
     override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {
@@ -87,59 +85,51 @@ class SessionTrackingService : Service() {
                 }
             }
         }
-        // Return START_STICKY to restart service if it gets killed
+
-        return START_STICKY
+        return START_REDELIVER_INTENT
     }
     
     override fun onTaskRemoved(rootIntent: Intent?) {
         super.onTaskRemoved(rootIntent)
-        // If the app is removed from recent tasks, ensure the service keeps running
-        // This helps maintain the notification even if the user swipes away the app
-    }
-    
-    override fun onLowMemory() {
-        super.onLowMemory()
-        // Don't stop the service on low memory, just log it
-        // The notification is important for user experience
     }
     
     override fun onBind(intent: Intent?): IBinder? = null
     
     private fun startSessionTracking(sessionId: String) {
-        // Cancel any existing jobs
         notificationJob?.cancel()
         monitoringJob?.cancel()
         
-        // Start the main notification update job
+        try {
+            createAndShowNotification(sessionId)
+        } catch (e: Exception) {
+            e.printStackTrace()
+        }
+        
         notificationJob = serviceScope.launch {
             try {
-                // Initial notification update
+                if (!isNotificationActive()) {
-                updateNotification(sessionId)
+                    delay(1000L)
+                    createAndShowNotification(sessionId)
+                }
                 
-                // Update every 2 seconds for better performance
                 while (isActive) {
-                    delay(2000L)
+                    delay(5000L)
                     updateNotification(sessionId)
                 }
             } catch (e: Exception) {
-                // Log error and continue
                 e.printStackTrace()
             }
         }
         
-        // Start the monitoring job that ensures notification stays active
         monitoringJob = serviceScope.launch {
             try {
                 while (isActive) {
-                    delay(5000L) // Check every 5 seconds
+                    delay(10000L)
                     
-                    // Verify the notification is still active
                     if (!isNotificationActive()) {
-                        // Notification was dismissed, recreate it
                         updateNotification(sessionId)
                     }
                     
-                    // Verify the session is still active
                     val session = repository.getSessionById(sessionId)
                     if (session == null || session.status != com.atridad.openclimb.data.model.SessionStatus.ACTIVE) {
                         stopSessionTracking()
@@ -155,7 +145,6 @@ class SessionTrackingService : Service() {
     private fun stopSessionTracking() {
         notificationJob?.cancel()
         monitoringJob?.cancel()
-        releaseWakeLock()
         stopForeground(STOP_FOREGROUND_REMOVE)
         stopSelf()
     }
@@ -171,14 +160,37 @@ class SessionTrackingService : Service() {
     
     private suspend fun updateNotification(sessionId: String) {
         try {
-            val session = repository.getSessionById(sessionId)
+            createAndShowNotification(sessionId)
+        } catch (e: Exception) {
+            e.printStackTrace()
+
+            try {
+                delay(10000L)
+                createAndShowNotification(sessionId)
+            } catch (retryException: Exception) {
+                retryException.printStackTrace()
+                stopSessionTracking()
+            }
+        }
+    }
+    
+    private fun createAndShowNotification(sessionId: String) {
+        try {
+            val session = runBlocking {
+                repository.getSessionById(sessionId)
+            }
             if (session == null || session.status != com.atridad.openclimb.data.model.SessionStatus.ACTIVE) {
                 stopSessionTracking()
                 return
             }
             
-            val gym = repository.getGymById(session.gymId)
+            val gym = runBlocking {
-            val attempts = repository.getAttemptsBySession(sessionId).firstOrNull() ?: emptyList()
+                repository.getGymById(session.gymId)
+            }
+            
+            val attempts = runBlocking {
+                repository.getAttemptsBySession(sessionId).firstOrNull() ?: emptyList()
+            }
             
             val duration = session.startTime?.let { startTime ->
                 try {
@@ -205,7 +217,7 @@ class SessionTrackingService : Service() {
                 .setSmallIcon(R.drawable.ic_mountains)
                 .setOngoing(true)
                 .setAutoCancel(false)
-                .setPriority(NotificationCompat.PRIORITY_LOW)
+                .setPriority(NotificationCompat.PRIORITY_DEFAULT)
                 .setCategory(NotificationCompat.CATEGORY_SERVICE)
                 .setVisibility(NotificationCompat.VISIBILITY_PUBLIC)
                 .setContentIntent(createOpenAppIntent())
@@ -221,24 +233,13 @@ class SessionTrackingService : Service() {
                 )
                 .build()
             
-            // Always start foreground to ensure service stays alive
             startForeground(NOTIFICATION_ID, notification)
             
-            // Also notify separately to ensure it's visible
             notificationManager.notify(NOTIFICATION_ID, notification)
             
         } catch (e: Exception) {
             e.printStackTrace()
-            // Don't stop the service on notification errors, just log them
+            throw e
-            // Try to restart the notification after a delay
-            try {
-                delay(5000L)
-                updateNotification(sessionId)
-            } catch (retryException: Exception) {
-                retryException.printStackTrace()
-                // If retry fails, stop the service to prevent infinite loops
-                stopSessionTracking()
-            }
         }
     }
     
@@ -269,50 +270,23 @@ class SessionTrackingService : Service() {
         val channel = NotificationChannel(
             CHANNEL_ID,
             "Session Tracking",
-            NotificationManager.IMPORTANCE_LOW
+            NotificationManager.IMPORTANCE_DEFAULT
         ).apply {
             description = "Shows active climbing session information"
             setShowBadge(false)
             lockscreenVisibility = NotificationCompat.VISIBILITY_PUBLIC
             enableLights(false)
             enableVibration(false)
+            setSound(null, null)
         }
 
         notificationManager.createNotificationChannel(channel)
     }
     
-    private fun acquireWakeLock() {
-        try {
-            val powerManager = getSystemService(Context.POWER_SERVICE) as PowerManager
-            wakeLock = powerManager.newWakeLock(
-                PowerManager.PARTIAL_WAKE_LOCK,
-                "OpenClimb:SessionTrackingWakeLock"
-            ).apply {
-                acquire(10*60*1000L) // 10 minutes timeout
-            }
-        } catch (e: Exception) {
-            e.printStackTrace()
-        }
-    }
-    
-    private fun releaseWakeLock() {
-        try {
-            wakeLock?.let {
-                if (it.isHeld) {
-                    it.release()
-                }
-            }
-            wakeLock = null
-        } catch (e: Exception) {
-            e.printStackTrace()
-        }
-    }
-    
     override fun onDestroy() {
         super.onDestroy()
         notificationJob?.cancel()
         monitoringJob?.cancel()
-        releaseWakeLock()
         serviceScope.cancel()
     }
 }

app/src/main/java/com/atridad/openclimb/ui/components/LineChart.kt

@@ -0,0 +1,302 @@
+package com.atridad.openclimb.ui.components
+
+import androidx.compose.foundation.Canvas
+import androidx.compose.foundation.layout.Box
+import androidx.compose.foundation.layout.fillMaxSize
+import androidx.compose.foundation.layout.padding
+import androidx.compose.material3.MaterialTheme
+import androidx.compose.runtime.*
+import androidx.compose.ui.Modifier
+import androidx.compose.ui.geometry.Offset
+import androidx.compose.ui.graphics.*
+import androidx.compose.ui.graphics.drawscope.DrawScope
+import androidx.compose.ui.graphics.drawscope.Stroke
+import androidx.compose.ui.platform.LocalDensity
+import androidx.compose.ui.text.TextMeasurer
+import androidx.compose.ui.text.TextStyle
+import androidx.compose.ui.text.drawText
+import androidx.compose.ui.text.rememberTextMeasurer
+import androidx.compose.ui.unit.dp
+import androidx.compose.ui.unit.sp
+
+/**
+ * Data point for the line chart
+ */
+data class ChartDataPoint(
+    val x: Float,
+    val y: Float,
+    val label: String? = null
+)
+
+/**
+ * Configuration for chart styling
+ */
+data class ChartStyle(
+    val lineColor: Color,
+    val fillColor: Color,
+    val lineWidth: Float = 3f,
+    val gridColor: Color,
+    val textColor: Color,
+    val backgroundColor: Color
+)
+
+/**
+ * Custom Line Chart with area fill below the line
+ */
+@Composable
+fun LineChart(
+    data: List<ChartDataPoint>,
+    modifier: Modifier = Modifier,
+    style: ChartStyle = ChartStyle(
+        lineColor = MaterialTheme.colorScheme.primary,
+        fillColor = MaterialTheme.colorScheme.primary.copy(alpha = 0.2f),
+        gridColor = MaterialTheme.colorScheme.outline.copy(alpha = 0.3f),
+        textColor = MaterialTheme.colorScheme.onSurfaceVariant,
+        backgroundColor = MaterialTheme.colorScheme.surface
+    ),
+    showGrid: Boolean = true,
+    xAxisFormatter: (Float) -> String = { it.toString() },
+    yAxisFormatter: (Float) -> String = { it.toString() }
+) {
+    val textMeasurer = rememberTextMeasurer()
+    val density = LocalDensity.current
+    
+    Box(modifier = modifier) {
+        Canvas(
+            modifier = Modifier
+                .fillMaxSize()
+                .padding(16.dp)
+        ) {
+            if (data.isEmpty()) return@Canvas
+            
+            val padding = with(density) { 32.dp.toPx() }
+            val chartWidth = size.width - padding * 2
+            val chartHeight = size.height - padding * 2
+            
+            // Calculate data bounds
+            val dataMinY = data.minOf { it.y }
+            val dataMaxY = data.maxOf { it.y }
+            
+            // Add some padding to Y-axis (10% above and below the data range)
+            val yPadding = if (dataMaxY == dataMinY) 1f else (dataMaxY - dataMinY) * 0.1f
+            val minY = dataMinY - yPadding
+            val maxY = dataMaxY + yPadding
+            
+            val minX = data.minOf { it.x }
+            val maxX = data.maxOf { it.x }
+            
+            val xRange = if (maxX - minX == 0f) 1f else maxX - minX // Minimum range of 1 for single points
+            val yRange = maxY - minY
+            
+            // Ensure we have valid ranges
+            if (yRange == 0f) return@Canvas
+            
+            // Convert data points to screen coordinates
+            val screenPoints = data.map { point ->
+                val x = padding + (point.x - minX) / xRange * chartWidth
+                val y = padding + chartHeight - (point.y - minY) / yRange * chartHeight
+                Offset(x, y)
+            }
+            
+            // Draw background
+            drawRect(
+                color = style.backgroundColor,
+                topLeft = Offset(padding, padding),
+                size = androidx.compose.ui.geometry.Size(chartWidth, chartHeight)
+            )
+            
+            // Draw grid
+            if (showGrid) {
+                drawGrid(
+                    padding = padding,
+                    chartWidth = chartWidth,
+                    chartHeight = chartHeight,
+                    gridColor = style.gridColor,
+                    minX = minX,
+                    maxX = maxX,
+                    minY = minY,
+                    maxY = maxY,
+                    textMeasurer = textMeasurer,
+                    textColor = style.textColor,
+                    xAxisFormatter = xAxisFormatter,
+                    yAxisFormatter = yAxisFormatter,
+                    actualDataPoints = data
+                )
+            }
+            
+            // Draw area fill
+            if (screenPoints.size > 1) {
+                drawAreaFill(
+                    points = screenPoints,
+                    padding = padding,
+                    chartHeight = chartHeight,
+                    fillColor = style.fillColor
+                )
+            }
+            
+            // Draw line
+            if (screenPoints.size > 1) {
+                drawLine(
+                    points = screenPoints,
+                    lineColor = style.lineColor,
+                    lineWidth = style.lineWidth
+                )
+            }
+            
+            // Draw data points - more pronounced
+            screenPoints.forEach { point ->
+                // Draw outer circle (larger)
+                drawCircle(
+                    color = style.lineColor,
+                    radius = 8f,
+                    center = point
+                )
+                // Draw inner circle (white center)
+                drawCircle(
+                    color = style.backgroundColor,
+                    radius = 5f,
+                    center = point
+                )
+                // Draw border for better visibility
+                drawCircle(
+                    color = style.lineColor,
+                    radius = 8f,
+                    center = point,
+                    style = Stroke(width = 2f)
+                )
+            }
+        }
+    }
+}
+
+private fun DrawScope.drawGrid(
+    padding: Float,
+    chartWidth: Float,
+    chartHeight: Float,
+    gridColor: Color,
+    minX: Float,
+    maxX: Float,
+    minY: Float,
+    maxY: Float,
+    textMeasurer: TextMeasurer,
+    textColor: Color,
+    xAxisFormatter: (Float) -> String,
+    yAxisFormatter: (Float) -> String,
+    actualDataPoints: List<ChartDataPoint>
+) {
+    val textStyle = TextStyle(
+        color = textColor,
+        fontSize = 10.sp
+    )
+    
+    // Draw vertical grid lines (X-axis) - only at integer values for sessions
+    val xRange = maxX - minX
+    if (xRange > 0) {
+        val startX = kotlin.math.ceil(minX).toInt()
+        val endX = kotlin.math.floor(maxX).toInt()
+        
+        for (sessionNum in startX..endX) {
+            val x = padding + (sessionNum.toFloat() - minX) / xRange * chartWidth
+            
+            // Draw grid line
+            drawLine(
+                color = gridColor,
+                start = Offset(x, padding),
+                end = Offset(x, padding + chartHeight),
+                strokeWidth = 1.dp.toPx()
+            )
+            
+            // X-axis labels removed per user request
+        }
+    }
+    
+    // Draw horizontal grid lines (Y-axis) - only at actual data point values
+    val yRange = maxY - minY
+    if (yRange > 0) {
+        // Get unique Y values from actual data points
+        val actualYValues = actualDataPoints.map { kotlin.math.round(it.y).toInt() }.toSet()
+        
+        actualYValues.forEach { gradeValue ->
+            val y = padding + chartHeight - (gradeValue.toFloat() - minY) / yRange * chartHeight
+            
+            // Only draw if within chart bounds
+            if (y >= padding && y <= padding + chartHeight) {
+                // Draw grid line
+                drawLine(
+                    color = gridColor,
+                    start = Offset(padding, y),
+                    end = Offset(padding + chartWidth, y),
+                    strokeWidth = 1.dp.toPx()
+                )
+                
+                // Draw label
+                val text = yAxisFormatter(gradeValue.toFloat())
+                val textSize = textMeasurer.measure(text, textStyle)
+                drawText(
+                    textMeasurer = textMeasurer,
+                    text = text,
+                    style = textStyle,
+                    topLeft = Offset(
+                        padding - textSize.size.width - 8.dp.toPx(),
+                        y - textSize.size.height / 2f
+                    )
+                )
+            }
+        }
+    }
+}
+
+private fun DrawScope.drawAreaFill(
+    points: List<Offset>,
+    padding: Float,
+    chartHeight: Float,
+    fillColor: Color
+) {
+    val bottomY = padding + chartHeight // This represents the bottom of the chart area
+    
+    val path = Path().apply {
+        // Start from bottom-left (at chart bottom level)
+        moveTo(points.first().x, bottomY)
+        
+        // Draw to first point
+        lineTo(points.first().x, points.first().y)
+        
+        // Draw line through all points
+        for (i in 1 until points.size) {
+            lineTo(points[i].x, points[i].y)
+        }
+        
+        // Close the path by going to bottom-right (at chart bottom level) and back to start
+        lineTo(points.last().x, bottomY)
+        lineTo(points.first().x, bottomY)
+        close()
+    }
+    
+    drawPath(
+        path = path,
+        color = fillColor
+    )
+}
+
+private fun DrawScope.drawLine(
+    points: List<Offset>,
+    lineColor: Color,
+    lineWidth: Float
+) {
+    val path = Path().apply {
+        moveTo(points.first().x, points.first().y)
+        for (i in 1 until points.size) {
+            lineTo(points[i].x, points[i].y)
+        }
+    }
+    
+    drawPath(
+        path = path,
+        color = lineColor,
+        style = Stroke(
+            width = lineWidth,
+            cap = StrokeCap.Round,
+            join = StrokeJoin.Round
+        )
+    )
+}

app/src/main/java/com/atridad/openclimb/ui/screens/AddEditScreens.kt

@@ -16,6 +16,7 @@ import androidx.compose.ui.semantics.Role
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.text.input.KeyboardType
 import androidx.compose.ui.unit.dp
+import androidx.compose.ui.platform.LocalContext
 import com.atridad.openclimb.data.model.*
 import com.atridad.openclimb.ui.components.ImagePicker
 import com.atridad.openclimb.ui.viewmodel.ClimbViewModel
@@ -80,7 +81,7 @@ fun AddEditGymScreen(
                             val gym = Gym.create(name, location, selectedClimbTypes.toList(), selectedDifficultySystems.toList(), notes = notes)
                             
                             if (isEditing) {
-                                viewModel.updateGym(gym.copy(id = gymId))
+                                viewModel.updateGym(gym.copy(id = gymId!!))
                             } else {
                                 viewModel.addGym(gym)
                             }
@@ -348,7 +349,7 @@ fun AddEditProblemScreen(
                                 )
                                 
                                 if (isEditing) {
-                                    viewModel.updateProblem(problem.copy(id = problemId))
+                                    viewModel.updateProblem(problem.copy(id = problemId!!))
                                 } else {
                                     viewModel.addProblem(problem)
                                 }
@@ -542,11 +543,18 @@ fun AddEditProblemScreen(
                             if (selectedDifficultySystem == DifficultySystem.CUSTOM) {
                                 OutlinedTextField(
                                     value = difficultyGrade,
-                                    onValueChange = { difficultyGrade = it },
+                                    onValueChange = { newValue ->
+                                        // Only allow integers for custom scales
+                                        if (newValue.isEmpty() || newValue.all { it.isDigit() }) {
+                                            difficultyGrade = newValue
+                                        }
+                                    },
                                     label = { Text("Grade *") },
                                     modifier = Modifier.fillMaxWidth(),
                                     singleLine = true,
-                                    placeholder = { Text("Enter custom grade") }
+                                    placeholder = { Text("Enter numeric grade (e.g. 5, 10, 15)") },
+                                    supportingText = { Text("Custom grades must be whole numbers") },
+                                    keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Number)
                                 )
                             } else {
                                 var expanded by remember { mutableStateOf(false) }
@@ -565,7 +573,7 @@ fun AddEditProblemScreen(
                                         trailingIcon = { ExposedDropdownMenuDefaults.TrailingIcon(expanded = expanded) },
                                         colors = ExposedDropdownMenuDefaults.outlinedTextFieldColors(),
                                         modifier = Modifier
-                                            .menuAnchor()
+                                            .menuAnchor(androidx.compose.material3.MenuAnchorType.PrimaryNotEditable, enabled = true)
                                             .fillMaxWidth()
                                     )
                                     ExposedDropdownMenu(
@@ -688,6 +696,7 @@ fun AddEditSessionScreen(
 ) {
     val isEditing = sessionId != null
     val gyms by viewModel.gyms.collectAsState()
+    val context = LocalContext.current
 
     // Session form state
     var selectedGym by remember { mutableStateOf<Gym?>(gymId?.let { id -> gyms.find { it.id == id } }) }
@@ -727,15 +736,14 @@ fun AddEditSessionScreen(
                     TextButton(
                         onClick = {
                             selectedGym?.let { gym ->
-                                val session = ClimbSession.create(
-                                    gymId = gym.id,
-                                    notes = sessionNotes.ifBlank { null }
-                                )
-                                
                                 if (isEditing) {
-                                    viewModel.updateSession(session.copy(id = sessionId))
+                                    val session = ClimbSession.create(
+                                        gymId = gym.id,
+                                        notes = sessionNotes.ifBlank { null }
+                                    )
+                                    viewModel.updateSession(session.copy(id = sessionId!!))
                                 } else {
-                                    viewModel.addSession(session)
+                                    viewModel.startSession(context, gym.id, sessionNotes.ifBlank { null })
                                 }
                                 onNavigateBack()
                             }

app/src/main/java/com/atridad/openclimb/ui/screens/AnalyticsScreen.kt

@@ -11,6 +11,10 @@ import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.dp
 import com.atridad.openclimb.R
 import com.atridad.openclimb.ui.viewmodel.ClimbViewModel
+import com.atridad.openclimb.data.model.ClimbType
+import com.atridad.openclimb.data.model.DifficultySystem
+import com.atridad.openclimb.ui.components.ChartDataPoint
+import com.atridad.openclimb.ui.components.LineChart
 
 @Composable
 fun AnalyticsScreen(
@@ -57,20 +61,10 @@ fun AnalyticsScreen(
             )
         }
         
-        // Success Rate
+        // Progress Chart
         item {
-            val successfulAttempts = attempts.count { 
+            val progressData = calculateProgressOverTime(sessions, problems, attempts)
-                it.result.name in listOf("SUCCESS", "FLASH", "REDPOINT", "ONSIGHT") 
+            ProgressChartCard(progressData = progressData, problems = problems)
-            }
-            val successRate = if (attempts.isNotEmpty()) {
-                (successfulAttempts.toDouble() / attempts.size * 100).toInt()
-            } else 0
-            
-            SuccessRateCard(
-                successRate = successRate,
-                successfulAttempts = successfulAttempts,
-                totalAttempts = attempts.size
-            )
         }
         
         // Favorite Gym
@@ -132,14 +126,24 @@ fun OverallStatsCard(
     }
 }
 
-
+@OptIn(ExperimentalMaterial3Api::class)
-
 @Composable
-fun SuccessRateCard(
+fun ProgressChartCard(
-    successRate: Int,
+    progressData: List<ProgressDataPoint>,
-    successfulAttempts: Int,
+    problems: List<com.atridad.openclimb.data.model.Problem>,
-    totalAttempts: Int
 ) {
+    // Find all grading systems that have been used
+    val usedSystems = remember(problems) {
+        problems.map { it.difficulty.system }.distinct().filter { system ->
+            problems.any { it.difficulty.system == system }
+        }
+    }
+    
+    var selectedSystem by remember { 
+        mutableStateOf(usedSystems.firstOrNull() ?: DifficultySystem.V_SCALE) 
+    }
+    var expanded by remember { mutableStateOf(false) }
+    
     Card(
         modifier = Modifier.fillMaxWidth()
     ) {
@@ -148,38 +152,120 @@ fun SuccessRateCard(
                 .fillMaxWidth()
                 .padding(16.dp)
         ) {
-            Text(
-                text = "Success Rate",
-                style = MaterialTheme.typography.titleMedium,
-                fontWeight = FontWeight.Bold
-            )
-            
-            Spacer(modifier = Modifier.height(8.dp))
-            
             Row(
                 modifier = Modifier.fillMaxWidth(),
                 horizontalArrangement = Arrangement.SpaceBetween,
                 verticalAlignment = Alignment.CenterVertically
             ) {
                 Text(
-                    text = "$successRate%",
+                    text = "Progress Over Time",
-                    style = MaterialTheme.typography.displaySmall,
+                    style = MaterialTheme.typography.titleMedium,
                     fontWeight = FontWeight.Bold,
-                    color = MaterialTheme.colorScheme.primary
+                    modifier = Modifier.weight(1f)
                 )
                 
-                Column(horizontalAlignment = Alignment.End) {
+                // Scale selector dropdown
-                    Text(
+                if (usedSystems.size > 1) {
-                        text = "$successfulAttempts successful",
+                    ExposedDropdownMenuBox(
-                        style = MaterialTheme.typography.bodyMedium
+                        expanded = expanded,
-                    )
+                        onExpandedChange = { expanded = !expanded }
-                    Text(
+                    ) {
-                        text = "out of $totalAttempts attempts",
+                        OutlinedTextField(
-                        style = MaterialTheme.typography.bodySmall,
+                            value = when (selectedSystem) {
-                        color = MaterialTheme.colorScheme.onSurfaceVariant
+                                DifficultySystem.V_SCALE -> "V-Scale"
-                    )
+                                DifficultySystem.FONT -> "Font"
+                                DifficultySystem.YDS -> "YDS"
+                                DifficultySystem.CUSTOM -> "Custom"
+                            },
+                            onValueChange = {},
+                            readOnly = true,
+                            trailingIcon = { ExposedDropdownMenuDefaults.TrailingIcon(expanded = expanded) },
+                            modifier = Modifier
+                                .menuAnchor(type = MenuAnchorType.PrimaryNotEditable, enabled = true)
+                                .width(120.dp),
+                            textStyle = MaterialTheme.typography.bodyMedium
+                        )
+                        ExposedDropdownMenu(
+                            expanded = expanded,
+                            onDismissRequest = { expanded = false }
+                        ) {
+                            usedSystems.forEach { system ->
+                                DropdownMenuItem(
+                                    text = { 
+                                        Text(when (system) {
+                                            DifficultySystem.V_SCALE -> "V-Scale"
+                                            DifficultySystem.FONT -> "Font"
+                                            DifficultySystem.YDS -> "YDS"
+                                            DifficultySystem.CUSTOM -> "Custom"
+                                        })
+                                    },
+                                    onClick = {
+                                        selectedSystem = system
+                                        expanded = false
+                                    }
+                                )
+                            }
+                        }
+                    }
                 }
             }
+            
+            Spacer(modifier = Modifier.height(12.dp))
+            
+            // Filter progress data by selected scale
+            val filteredProgressData = remember(progressData, selectedSystem) {
+                progressData.filter { it.difficultySystem == selectedSystem }
+            }
+            
+            if (filteredProgressData.isNotEmpty()) {
+                val chartData = remember(filteredProgressData) {
+                    // Convert progress data to chart data points ordered by session
+                    filteredProgressData
+                        .sortedBy { it.date }
+                        .mapIndexed { index, p ->
+                            ChartDataPoint(
+                                x = (index + 1).toFloat(),
+                                y = p.maxGradeNumeric.toFloat(),
+                                label = "Session ${index + 1}"
+                            )
+                        }
+                }
+                
+                LineChart(
+                    data = chartData,
+                    modifier = Modifier.fillMaxWidth().height(220.dp),
+                    xAxisFormatter = { value -> 
+                        "S${value.toInt()}" // S1, S2, S3, etc.
+                    },
+                    yAxisFormatter = { value -> 
+                        numericToGrade(selectedSystem, value.toInt())
+                    }
+                )
+                
+                Spacer(modifier = Modifier.height(8.dp))
+                
+                Text(
+                    text = "X: session number, Y: max ${when(selectedSystem) {
+                        DifficultySystem.V_SCALE -> "V-grade"
+                        DifficultySystem.FONT -> "Font grade" 
+                        DifficultySystem.YDS -> "YDS grade"
+                        DifficultySystem.CUSTOM -> "custom grade"
+                    }} achieved",
+                    style = MaterialTheme.typography.bodySmall,
+                    color = MaterialTheme.colorScheme.onSurfaceVariant
+                )
+            } else {
+                Text(
+                    text = "No progress data available for ${when(selectedSystem) {
+                        DifficultySystem.V_SCALE -> "V-Scale"
+                        DifficultySystem.FONT -> "Font"
+                        DifficultySystem.YDS -> "YDS"
+                        DifficultySystem.CUSTOM -> "Custom"
+                    }} system",
+                    style = MaterialTheme.typography.bodyMedium,
+                    color = MaterialTheme.colorScheme.onSurfaceVariant
+                )
+            }
         }
     }
 }
@@ -253,3 +339,207 @@ fun RecentActivityCard(
         }
     }
 }
+
+data class ProgressDataPoint(
+    val date: String,
+    val maxGrade: String,
+    val maxGradeNumeric: Int,
+    val climbType: ClimbType,
+    val difficultySystem: DifficultySystem
+)
+
+fun calculateProgressOverTime(
+    sessions: List<com.atridad.openclimb.data.model.ClimbSession>,
+    problems: List<com.atridad.openclimb.data.model.Problem>,
+    attempts: List<com.atridad.openclimb.data.model.Attempt>
+): List<ProgressDataPoint> {
+    if (sessions.isEmpty() || problems.isEmpty() || attempts.isEmpty()) {
+        return emptyList()
+    }
+    
+    val sessionProgress = sessions.mapNotNull { session ->
+        val sessionAttempts = attempts.filter { it.sessionId == session.id }
+        if (sessionAttempts.isEmpty()) return@mapNotNull null
+        val attemptedProblemIds = sessionAttempts.map { it.problemId }.distinct()
+        val attemptedProblems = problems.filter { it.id in attemptedProblemIds }
+        if (attemptedProblems.isEmpty()) return@mapNotNull null
+        val highestGradeProblem = attemptedProblems.maxByOrNull { problem ->
+            gradeToNumeric(problem.difficulty.system, problem.difficulty.grade)
+        }
+        if (highestGradeProblem != null) {
+            ProgressDataPoint(
+                date = session.date,
+                maxGrade = highestGradeProblem.difficulty.grade,
+                maxGradeNumeric = gradeToNumeric(highestGradeProblem.difficulty.system, highestGradeProblem.difficulty.grade),
+                climbType = highestGradeProblem.climbType,
+                difficultySystem = highestGradeProblem.difficulty.system
+            )
+        } else null
+    }
+    return sessionProgress.sortedBy { it.date }
+}
+
+fun gradeToNumeric(system: DifficultySystem, grade: String): Int {
+    return when (system) {
+        DifficultySystem.V_SCALE -> {
+            when (grade) {
+                "VB" -> 0
+                else -> grade.removePrefix("V").toIntOrNull() ?: 0
+            }
+        }
+        DifficultySystem.FONT -> {
+            when (grade) {
+                "3" -> 3
+                "4A" -> 4
+                "4B" -> 5
+                "4C" -> 6
+                "5A" -> 7
+                "5B" -> 8
+                "5C" -> 9
+                "6A" -> 10
+                "6A+" -> 11
+                "6B" -> 12
+                "6B+" -> 13
+                "6C" -> 14
+                "6C+" -> 15
+                "7A" -> 16
+                "7A+" -> 17
+                "7B" -> 18
+                "7B+" -> 19
+                "7C" -> 20
+                "7C+" -> 21
+                "8A" -> 22
+                "8A+" -> 23
+                "8B" -> 24
+                "8B+" -> 25
+                "8C" -> 26
+                "8C+" -> 27
+                else -> 0
+            }
+        }
+        DifficultySystem.YDS -> {
+            when (grade) {
+                "5.0" -> 50
+                "5.1" -> 51
+                "5.2" -> 52
+                "5.3" -> 53
+                "5.4" -> 54
+                "5.5" -> 55
+                "5.6" -> 56
+                "5.7" -> 57
+                "5.8" -> 58
+                "5.9" -> 59
+                "5.10a" -> 60
+                "5.10b" -> 61
+                "5.10c" -> 62
+                "5.10d" -> 63
+                "5.11a" -> 64
+                "5.11b" -> 65
+                "5.11c" -> 66
+                "5.11d" -> 67
+                "5.12a" -> 68
+                "5.12b" -> 69
+                "5.12c" -> 70
+                "5.12d" -> 71
+                "5.13a" -> 72
+                "5.13b" -> 73
+                "5.13c" -> 74
+                "5.13d" -> 75
+                "5.14a" -> 76
+                "5.14b" -> 77
+                "5.14c" -> 78
+                "5.14d" -> 79
+                "5.15a" -> 80
+                "5.15b" -> 81
+                "5.15c" -> 82
+                "5.15d" -> 83
+                else -> 0
+            }
+        }
+        DifficultySystem.CUSTOM -> {
+            // Custom grades are numeric strings, so parse them directly
+            grade.toIntOrNull() ?: 0
+        }
+    }
+}
+
+fun numericToGrade(system: DifficultySystem, numeric: Int): String {
+    return when (system) {
+        DifficultySystem.V_SCALE -> {
+            when (numeric) {
+                0 -> "VB"
+                else -> "V$numeric"
+            }
+        }
+        DifficultySystem.FONT -> {
+            when (numeric) {
+                3 -> "3"
+                4 -> "4A"
+                5 -> "4B"
+                6 -> "4C"
+                7 -> "5A"
+                8 -> "5B"
+                9 -> "5C"
+                10 -> "6A"
+                11 -> "6A+"
+                12 -> "6B"
+                13 -> "6B+"
+                14 -> "6C"
+                15 -> "6C+"
+                16 -> "7A"
+                17 -> "7A+"
+                18 -> "7B"
+                19 -> "7B+"
+                20 -> "7C"
+                21 -> "7C+"
+                22 -> "8A"
+                23 -> "8A+"
+                24 -> "8B"
+                25 -> "8B+"
+                26 -> "8C"
+                27 -> "8C+"
+                else -> numeric.toString()
+            }
+        }
+        DifficultySystem.YDS -> {
+            when (numeric) {
+                50 -> "5.0"
+                51 -> "5.1"
+                52 -> "5.2"
+                53 -> "5.3"
+                54 -> "5.4"
+                55 -> "5.5"
+                56 -> "5.6"
+                57 -> "5.7"
+                58 -> "5.8"
+                59 -> "5.9"
+                60 -> "5.10a"
+                61 -> "5.10b"
+                62 -> "5.10c"
+                63 -> "5.10d"
+                64 -> "5.11a"
+                65 -> "5.11b"
+                66 -> "5.11c"
+                67 -> "5.11d"
+                68 -> "5.12a"
+                69 -> "5.12b"
+                70 -> "5.12c"
+                71 -> "5.12d"
+                72 -> "5.13a"
+                73 -> "5.13b"
+                74 -> "5.13c"
+                75 -> "5.13d"
+                76 -> "5.14a"
+                77 -> "5.14b"
+                78 -> "5.14c"
+                79 -> "5.14d"
+                80 -> "5.15a"
+                81 -> "5.15b"
+                82 -> "5.15c"
+                83 -> "5.15d"
+                else -> numeric.toString()
+            }
+        }
+        DifficultySystem.CUSTOM -> numeric.toString()
+    }
+}

app/src/main/java/com/atridad/openclimb/ui/screens/DetailScreens.kt

@@ -3,6 +3,7 @@ package com.atridad.openclimb.ui.screens
 import androidx.compose.foundation.BorderStroke
 import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.*
+import androidx.compose.foundation.text.KeyboardOptions
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.LazyRow
 import androidx.compose.foundation.lazy.items
@@ -25,6 +26,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.semantics.Role
 import androidx.compose.ui.text.font.FontWeight
+import androidx.compose.ui.text.input.KeyboardType
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.window.Dialog
 import androidx.lifecycle.viewModelScope
@@ -35,7 +37,6 @@ import com.atridad.openclimb.ui.theme.CustomIcons
 import com.atridad.openclimb.ui.viewmodel.ClimbViewModel
 import java.time.LocalDateTime
 import java.time.format.DateTimeFormatter
-import kotlin.math.roundToInt
 import kotlinx.coroutines.flow.first
 import kotlinx.coroutines.launch
 
@@ -291,8 +292,8 @@ fun SessionDetailScreen(
 
                     // Show stop icon for active sessions, delete icon for completed sessions
                     if (session?.status == SessionStatus.ACTIVE) {
-                        IconButton(onClick = { 
+                        IconButton(onClick = {
-                            session?.let { s ->
+                            session.let { s ->
                                 viewModel.endSession(context, s.id)
                                 onNavigateBack()
                             }
@@ -426,11 +427,6 @@ fun SessionDetailScreen(
                                     label = "Completed",
                                     value = completedProblems.size.toString()
                                 )
-                                StatItem(
-                                    label = "Success Rate",
-                                    value = "${((successfulAttempts.size.toDouble() / attempts.size) * 100).toInt()}%"
-                                )
-                                
                             }
                             
                             // Show grade range(s) with better layout
@@ -622,10 +618,6 @@ fun ProblemDetailScreen(
     // Calculate stats
     val successfulAttempts =
         attempts.filter { it.result in listOf(AttemptResult.SUCCESS, AttemptResult.FLASH) }
-    val successRate =
-        if (attempts.isNotEmpty()) {
-            (successfulAttempts.size.toDouble() / attempts.size * 100).toInt()
-        } else 0
 
     val attemptsWithSessions =
         attempts
@@ -793,7 +785,6 @@ fun ProblemDetailScreen(
                                     label = "Successful",
                                     value = successfulAttempts.size.toString()
                                 )
-                                StatItem(label = "Success Rate", value = "$successRate%")
                             }
 
                             Spacer(modifier = Modifier.height(12.dp))
@@ -926,14 +917,6 @@ fun GymDetailScreen(
             problems.any { problem -> problem.id == attempt.problemId }
         }
 
-    val successfulAttempts =
-        gymAttempts.filter { it.result in listOf(AttemptResult.SUCCESS, AttemptResult.FLASH) }
-
-    val successRate =
-        if (gymAttempts.isNotEmpty()) {
-            (successfulAttempts.size.toDouble() / gymAttempts.size * 100).toInt()
-        } else 0
-
     val uniqueProblemsClimbed = gymAttempts.map { it.problemId }.toSet().size
     val totalSessions = sessions.size
     val activeSessions = sessions.count { it.status == SessionStatus.ACTIVE }
@@ -1042,19 +1025,7 @@ fun GymDetailScreen(
                                         color = MaterialTheme.colorScheme.onSurfaceVariant
                                     )
                                 }
-                                Column(horizontalAlignment = Alignment.CenterHorizontally) {
+
-                                    Text(
-                                        text = "$successRate%",
-                                        style = MaterialTheme.typography.headlineSmall,
-                                        fontWeight = FontWeight.Bold,
-                                        color = MaterialTheme.colorScheme.primary
-                                    )
-                                    Text(
-                                        text = "Success Rate",
-                                        style = MaterialTheme.typography.bodySmall,
-                                        color = MaterialTheme.colorScheme.onSurfaceVariant
-                                    )
-                                }
                             }
 
                             Spacer(modifier = Modifier.height(12.dp))
@@ -1581,76 +1552,6 @@ private fun formatDate(dateString: String): String {
     }
 }
 
-/**
- * Calculate average grade for a specific set of problems, respecting their difficulty systems
- */
-private fun calculateAverageGrade(problems: List<Problem>): String? {
-    if (problems.isEmpty()) return null
-    
-    // Group problems by difficulty system
-    val problemsBySystem = problems.groupBy { it.difficulty.system }
-    
-    val averages = mutableListOf<String>()
-    
-    problemsBySystem.forEach { (system, systemProblems) ->
-        when (system) {
-            DifficultySystem.V_SCALE -> {
-                val gradeValues = systemProblems.mapNotNull { problem ->
-                    when {
-                        problem.difficulty.grade == "VB" -> 0
-                        else -> problem.difficulty.grade.removePrefix("V").toIntOrNull()
-                    }
-                }
-                if (gradeValues.isNotEmpty()) {
-                    val avg = gradeValues.average().roundToInt()
-                    averages.add(if (avg == 0) "VB" else "V$avg")
-                }
-            }
-            DifficultySystem.FONT -> {
-                val gradeValues = systemProblems.mapNotNull { problem ->
-                    // Extract numeric part from Font grades (e.g., "6A" -> 6, "7C+" -> 7)
-                    problem.difficulty.grade.filter { it.isDigit() }.toIntOrNull()
-                }
-                if (gradeValues.isNotEmpty()) {
-                    val avg = gradeValues.average().roundToInt()
-                    averages.add("$avg")
-                }
-            }
-            DifficultySystem.YDS -> {
-                val gradeValues = systemProblems.mapNotNull { problem ->
-                    // Extract numeric part from YDS grades (e.g., "5.10a" -> 5.10)
-                    val grade = problem.difficulty.grade
-                    if (grade.startsWith("5.")) {
-                        grade.substring(2).toDoubleOrNull()
-                    } else null
-                }
-                if (gradeValues.isNotEmpty()) {
-                    val avg = gradeValues.average()
-                    averages.add("5.${String.format("%.1f", avg)}")
-                }
-            }
-            DifficultySystem.CUSTOM -> {
-                // For custom systems, try to extract numeric values
-                val gradeValues = systemProblems.mapNotNull { problem ->
-                    problem.difficulty.grade.filter { it.isDigit() || it == '.' || it == '-' }.toDoubleOrNull()
-                }
-                if (gradeValues.isNotEmpty()) {
-                    val avg = gradeValues.average()
-                    averages.add(String.format("%.1f", avg))
-                }
-            }
-        }
-    }
-    
-    return if (averages.isNotEmpty()) {
-        if (averages.size == 1) {
-            averages.first()
-        } else {
-            averages.joinToString(" / ")
-        }
-    } else null
-}
-
 @OptIn(ExperimentalMaterial3Api::class)
 @Composable
 fun EnhancedAddAttemptDialog(
@@ -1860,8 +1761,12 @@ fun EnhancedAddAttemptDialog(
                                         color = MaterialTheme.colorScheme.onSurface
                                     )
 
-                                    TextButton(onClick = { showCreateProblem = false }) {
+                                    IconButton(onClick = { showCreateProblem = false }) {
-                                        Text("← Back", color = MaterialTheme.colorScheme.primary)
+                                        Icon(
+                                            Icons.AutoMirrored.Filled.ArrowBack,
+                                            contentDescription = "Back",
+                                            tint = MaterialTheme.colorScheme.onSurfaceVariant
+                                        )
                                     }
                                 }
 
@@ -1955,9 +1860,14 @@ fun EnhancedAddAttemptDialog(
                                 if (selectedDifficultySystem == DifficultySystem.CUSTOM) {
                                     OutlinedTextField(
                                         value = newProblemGrade,
-                                        onValueChange = { newProblemGrade = it },
+                                        onValueChange = { newValue ->
+                                            // Only allow integers for custom scales
+                                            if (newValue.isEmpty() || newValue.all { it.isDigit() }) {
+                                                newProblemGrade = newValue
+                                            }
+                                        },
                                         label = { Text("Grade *") },
-                                        placeholder = { Text("Enter custom grade") },
+                                        placeholder = { Text("Enter numeric grade (e.g. 5, 10, 15)") },
                                         modifier = Modifier.fillMaxWidth(),
                                         singleLine = true,
                                         colors =
@@ -1968,6 +1878,7 @@ fun EnhancedAddAttemptDialog(
                                                     MaterialTheme.colorScheme.outline
                                             ),
                                         isError = newProblemGrade.isBlank(),
+                                        keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Number),
                                         supportingText =
                                             if (newProblemGrade.isBlank()) {
                                                 {
@@ -1976,7 +1887,14 @@ fun EnhancedAddAttemptDialog(
                                                         color = MaterialTheme.colorScheme.error
                                                     )
                                                 }
-                                            } else null
+                                            } else {
+                                                {
+                                                    Text(
+                                                        "Custom grades must be whole numbers",
+                                                        color = MaterialTheme.colorScheme.onSurfaceVariant
+                                                    )
+                                                }
+                                            }
                                     )
                                 } else {
                                     var expanded by remember { mutableStateOf(false) }
@@ -2001,7 +1919,7 @@ fun EnhancedAddAttemptDialog(
                                             colors =
                                                 ExposedDropdownMenuDefaults
                                                     .outlinedTextFieldColors(),
-                                            modifier = Modifier.menuAnchor().fillMaxWidth(),
+                                            modifier = Modifier.menuAnchor(androidx.compose.material3.MenuAnchorType.PrimaryNotEditable, enabled = true).fillMaxWidth(),
                                             isError = newProblemGrade.isBlank(),
                                             supportingText =
                                                 if (newProblemGrade.isBlank()) {

app/src/main/java/com/atridad/openclimb/ui/screens/SettingsScreen.kt

@@ -26,12 +26,16 @@ fun SettingsScreen(
 ) {
     val uiState by viewModel.uiState.collectAsState()
     val context = LocalContext.current
+    
+    // State for reset confirmation dialog
+    var showResetDialog by remember { mutableStateOf(false) }
+    
     val packageInfo = remember {
         context.packageManager.getPackageInfo(context.packageName, 0)
     }
     val appVersion = packageInfo.versionName
     
-    // File picker launcher for import - accepts both ZIP and JSON files
+    // File picker launcher for import - only accepts ZIP files
     val importLauncher = rememberLauncherForActivityResult(
         contract = ActivityResultContracts.GetContent()
     ) { uri ->
@@ -46,9 +50,13 @@ fun SettingsScreen(
                     } else null
                 } ?: "import_file"
                 
-                val extension = fileName.substringAfterLast(".", "")
+                // Only allow ZIP files
-                val tempFileName = if (extension.isNotEmpty()) "temp_import.$extension" else "temp_import"
+                if (!fileName.lowercase().endsWith(".zip")) {
-                val tempFile = File(context.cacheDir, tempFileName)
+                    viewModel.setError("Only ZIP files are supported for import. Please use a ZIP file exported from OpenClimb.")
+                    return@let
+                }
+                
+                val tempFile = File(context.cacheDir, "temp_import.zip")
                 
                 inputStream?.use { input ->
                     tempFile.outputStream().use { output ->
@@ -62,7 +70,7 @@ fun SettingsScreen(
         }
     }
     
-    // File picker launcher for export - save location (ZIP format with images)
+    // File picker launcher for export - ZIP format with images
     val exportZipLauncher = rememberLauncherForActivityResult(
         contract = ActivityResultContracts.CreateDocument("application/zip")
     ) { uri ->
@@ -75,19 +83,6 @@ fun SettingsScreen(
         }
     }
     
-    // JSON export launcher
-    val exportJsonLauncher = rememberLauncherForActivityResult(
-        contract = ActivityResultContracts.CreateDocument("application/json")
-    ) { uri ->
-        uri?.let {
-            try {
-                viewModel.exportDataToUri(context, uri)
-            } catch (e: Exception) {
-                viewModel.setError("Failed to save file: ${e.message}")
-            }
-        }
-    }
-    
     LazyColumn(
         modifier = Modifier
             .fillMaxSize()
@@ -179,19 +174,13 @@ fun SettingsScreen(
                         )
                     ) {
                         ListItem(
-                            headlineContent = { Text("Export Data Only") },
+                            headlineContent = { Text("Import Data") },
-                            supportingContent = { Text("Export climbing data to JSON without images") },
+                            supportingContent = { Text("Import climbing data from ZIP file (recommended format)") },
-                            leadingContent = { Icon(Icons.Default.Share, contentDescription = null) },
+                            leadingContent = { Icon(Icons.Default.Add, contentDescription = null) },
                             trailingContent = {
                                 TextButton(
                                     onClick = { 
-                                        val defaultFileName = "openclimb_export_${
+                                        importLauncher.launch("application/zip")
-                                            java.time.LocalDateTime.now()
-                                                .toString()
-                                                .replace(":", "-")
-                                                .replace(".", "-")
-                                        }.json"
-                                        exportJsonLauncher.launch(defaultFileName)
                                     },
                                     enabled = !uiState.isLoading
                                 ) {
@@ -201,7 +190,7 @@ fun SettingsScreen(
                                             strokeWidth = 2.dp
                                         )
                                     } else {
-                                        Text("Export JSON")
+                                        Text("Import")
                                     }
                                 }
                             }
@@ -213,17 +202,17 @@ fun SettingsScreen(
                     Card(
                         shape = RoundedCornerShape(12.dp),
                         colors = CardDefaults.cardColors(
-                            containerColor = MaterialTheme.colorScheme.surfaceVariant.copy(alpha = 0.3f)
+                            containerColor = MaterialTheme.colorScheme.errorContainer.copy(alpha = 0.3f)
                         )
                     ) {
                         ListItem(
-                            headlineContent = { Text("Import Data") },
+                            headlineContent = { Text("Reset All Data") },
-                            supportingContent = { Text("Import climbing data from ZIP or JSON file") },
+                            supportingContent = { Text("Permanently delete all gyms, problems, sessions, attempts, and images") },
-                            leadingContent = { Icon(Icons.Default.Add, contentDescription = null) },
+                            leadingContent = { Icon(Icons.Default.Delete, contentDescription = null, tint = MaterialTheme.colorScheme.error) },
                             trailingContent = {
                                 TextButton(
                                     onClick = { 
-                                        importLauncher.launch("*/*")
+                                        showResetDialog = true
                                     },
                                     enabled = !uiState.isLoading
                                 ) {
@@ -233,7 +222,7 @@ fun SettingsScreen(
                                             strokeWidth = 2.dp
                                         )
                                     } else {
-                                        Text("Import")
+                                        Text("Reset", color = MaterialTheme.colorScheme.error)
                                     }
                                 }
                             }
@@ -390,4 +379,51 @@ fun SettingsScreen(
             }
         }
     }
+    
+    // Reset confirmation dialog
+    if (showResetDialog) {
+        AlertDialog(
+            onDismissRequest = { showResetDialog = false },
+            title = { Text("Reset All Data") },
+            text = {
+                Column {
+                    Text("Are you sure you want to reset all data?")
+                    Spacer(modifier = Modifier.height(8.dp))
+                    Text(
+                        text = "This will permanently delete:",
+                        style = MaterialTheme.typography.bodySmall,
+                        fontWeight = FontWeight.Medium
+                    )
+                    Spacer(modifier = Modifier.height(4.dp))
+                    Text(
+                        text = "• All gyms and their information\n• All problems and their images\n• All climbing sessions\n• All attempts and progress data",
+                        style = MaterialTheme.typography.bodySmall,
+                        color = MaterialTheme.colorScheme.error
+                    )
+                    Spacer(modifier = Modifier.height(8.dp))
+                    Text(
+                        text = "This action cannot be undone. Consider exporting your data first.",
+                        style = MaterialTheme.typography.bodySmall,
+                        fontWeight = FontWeight.Medium,
+                        color = MaterialTheme.colorScheme.error
+                    )
+                }
+            },
+            confirmButton = {
+                TextButton(
+                    onClick = {
+                        viewModel.resetAllData()
+                        showResetDialog = false
+                    }
+                ) {
+                    Text("Reset All Data", color = MaterialTheme.colorScheme.error)
+                }
+            },
+            dismissButton = {
+                TextButton(onClick = { showResetDialog = false }) { 
+                    Text("Cancel") 
+                }
+            }
+        )
+    }
 }

app/src/main/java/com/atridad/openclimb/ui/viewmodel/ClimbViewModel.kt

@@ -242,23 +242,7 @@ class ClimbViewModel(
     fun getAttemptsByProblem(problemId: String): Flow<List<Attempt>> = 
         repository.getAttemptsByProblem(problemId)
 
-    fun exportDataToUri(context: Context, uri: android.net.Uri) {
+
-        viewModelScope.launch {
-            try {
-                _uiState.value = _uiState.value.copy(isLoading = true)
-                repository.exportAllDataToUri(context, uri)
-                _uiState.value = _uiState.value.copy(
-                    isLoading = false,
-                    message = "Data exported successfully"
-                )
-            } catch (e: Exception) {
-                _uiState.value = _uiState.value.copy(
-                    isLoading = false,
-                    error = "Export failed: ${e.message}"
-                )
-            }
-        }
-    }
 
     fun exportDataToZipUri(context: Context, uri: android.net.Uri) {
         viewModelScope.launch {
@@ -283,13 +267,13 @@ class ClimbViewModel(
             try {
                 _uiState.value = _uiState.value.copy(isLoading = true)
                 
-                // Check if it's a ZIP file or JSON file
+                // Only support ZIP format for reliability
-                if (file.name.lowercase().endsWith(".zip")) {
+                if (!file.name.lowercase().endsWith(".zip")) {
-                    repository.importDataFromZip(file)
+                    throw Exception("Only ZIP files are supported for import. Please use a ZIP file exported from OpenClimb.")
-                } else {
-                    repository.importDataFromJson(file)
                 }
                 
+                repository.importDataFromZip(file)
+                
                 _uiState.value = _uiState.value.copy(
                     isLoading = false,
                     message = "Data imported successfully from ${file.name}"
@@ -316,6 +300,26 @@ class ClimbViewModel(
         _uiState.value = _uiState.value.copy(error = message)
     }
     
+    fun resetAllData() {
+        viewModelScope.launch {
+            try {
+                _uiState.value = _uiState.value.copy(isLoading = true)
+                
+                repository.resetAllData()
+                
+                _uiState.value = _uiState.value.copy(
+                    isLoading = false,
+                    message = "All data has been reset successfully"
+                )
+            } catch (e: Exception) {
+                _uiState.value = _uiState.value.copy(
+                    isLoading = false,
+                    error = "Reset failed: ${e.message}"
+                )
+            }
+        }
+    }
+    
     // Share operations
     suspend fun generateSessionShareCard(
         context: Context,

app/src/main/java/com/atridad/openclimb/utils/SessionShareUtils.kt

@@ -285,11 +285,7 @@ object SessionShareUtils {
                 drawStatItemFitting(canvas, width / 2f, rangesY, "Grade Range", singleRange, statLabelPaint, statValuePaint, width - 200f)
             }
             
-            // Success rate arc
+
-            val successRate = if (stats.totalAttempts > 0) {
-                (stats.successfulAttempts.toFloat() / stats.totalAttempts) * 100f
-            } else 0f
-            drawSuccessRateArc(canvas, width / 2f, height - 300f, successRate, statLabelPaint, statValuePaint)
             
             // App branding
             val brandingPaint = Paint().apply {
@@ -372,52 +368,7 @@ object SessionShareUtils {
         return "${sorted.first().grade} - ${sorted.last().grade}"
     }
     
-    private fun drawSuccessRateArc(
-        canvas: Canvas,
-        centerX: Float,
-        centerY: Float,
-        successRate: Float,
-        labelPaint: Paint,
-        valuePaint: Paint
-    ) {
-        val radius = 70f
-        val strokeWidth = 14f
 
-        // Background arc
-        val bgPaint = Paint().apply {
-            color = "#30FFFFFF".toColorInt()
-            style = Paint.Style.STROKE
-            this.strokeWidth = strokeWidth
-            isAntiAlias = true
-            strokeCap = Paint.Cap.ROUND
-        }
-
-        // Success arc
-        val successPaint = Paint().apply {
-            color = "#4CAF50".toColorInt()
-            style = Paint.Style.STROKE
-            this.strokeWidth = strokeWidth
-            isAntiAlias = true
-            strokeCap = Paint.Cap.ROUND
-        }
-
-        val rect = RectF(centerX - radius, centerY - radius, centerX + radius, centerY + radius)
-
-        // Draw background arc (full circle)
-        canvas.drawArc(rect, -90f, 360f, false, bgPaint)
-
-        // Draw success arc
-        val sweepAngle = (successRate / 100f) * 360f
-        canvas.drawArc(rect, -90f, sweepAngle, false, successPaint)
-
-        // Draw percentage text
-        val percentText = "${successRate.roundToInt()}%"
-        canvas.drawText(percentText, centerX, centerY + 8f, valuePaint)
-
-        // Draw label below the arc (outside the ring) for better readability
-        val belowLabelPaint = Paint(labelPaint)
-        canvas.drawText("Success Rate", centerX, centerY + radius + 36f, belowLabelPaint)
-    }
     
     private fun formatSessionDate(dateString: String): String {
         return try {

app/src/main/java/com/atridad/openclimb/utils/ZipExportImportUtils.kt

@@ -15,6 +15,7 @@ object ZipExportImportUtils {
     
     private const val DATA_JSON_FILENAME = "data.json"
     private const val IMAGES_DIR_NAME = "images"
+    private const val METADATA_FILENAME = "metadata.txt"
     
     /**
      * Creates a ZIP file containing the JSON data and all referenced images
@@ -38,37 +39,67 @@ object ZipExportImportUtils {
         val timestamp = LocalDateTime.now().toString().replace(":", "-").replace(".", "-")
         val zipFile = File(exportDir, "openclimb_export_$timestamp.zip")
         
-        ZipOutputStream(FileOutputStream(zipFile)).use { zipOut ->
+        try {
-            // Add JSON data file
+            ZipOutputStream(FileOutputStream(zipFile)).use { zipOut ->
-            val json = Json { prettyPrint = true }
+                // Add metadata file first
-            val jsonString = json.encodeToString(com.atridad.openclimb.data.repository.ClimbDataExport.serializer(), exportData)
+                val metadata = createMetadata(exportData, referencedImagePaths)
-            
+                val metadataEntry = ZipEntry(METADATA_FILENAME)
-            val jsonEntry = ZipEntry(DATA_JSON_FILENAME)
+                zipOut.putNextEntry(metadataEntry)
-            zipOut.putNextEntry(jsonEntry)
+                zipOut.write(metadata.toByteArray())
-            zipOut.write(jsonString.toByteArray())
+                zipOut.closeEntry()
-            zipOut.closeEntry()
+                
-            
+                // Add JSON data file
-            // Add images
+                val json = Json { 
-            referencedImagePaths.forEach { imagePath ->
+                    prettyPrint = true 
-                try {
+                    ignoreUnknownKeys = true
-                    val imageFile = ImageUtils.getImageFile(context, imagePath)
-                    if (imageFile.exists()) {
-                        val imageEntry = ZipEntry("$IMAGES_DIR_NAME/${imageFile.name}")
-                        zipOut.putNextEntry(imageEntry)
-                        
-                        FileInputStream(imageFile).use { imageInput ->
-                            imageInput.copyTo(zipOut)
-                        }
-                        zipOut.closeEntry()
-                    }
-                } catch (e: Exception) {
-                    // Log error but continue with other images
-                    e.printStackTrace()
                 }
+                val jsonString = json.encodeToString(com.atridad.openclimb.data.repository.ClimbDataExport.serializer(), exportData)
+                
+                val jsonEntry = ZipEntry(DATA_JSON_FILENAME)
+                zipOut.putNextEntry(jsonEntry)
+                zipOut.write(jsonString.toByteArray())
+                zipOut.closeEntry()
+                
+                // Add images with validation
+                var successfulImages = 0
+                referencedImagePaths.forEach { imagePath ->
+                    try {
+                        val imageFile = ImageUtils.getImageFile(context, imagePath)
+                        if (imageFile.exists() && imageFile.length() > 0) {
+                            val imageEntry = ZipEntry("$IMAGES_DIR_NAME/${imageFile.name}")
+                            zipOut.putNextEntry(imageEntry)
+                            
+                            FileInputStream(imageFile).use { imageInput ->
+                                imageInput.copyTo(zipOut)
+                            }
+                            zipOut.closeEntry()
+                            successfulImages++
+                        } else {
+                            android.util.Log.w("ZipExportImportUtils", "Image file not found or empty: $imagePath")
+                        }
+                    } catch (e: Exception) {
+                        android.util.Log.e("ZipExportImportUtils", "Failed to add image $imagePath: ${e.message}")
+                    }
+                }
+                
+                // Log export summary
+                android.util.Log.i("ZipExportImportUtils", "Export completed: ${successfulImages}/${referencedImagePaths.size} images included")
             }
+            
+            // Validate the created ZIP file
+            if (!zipFile.exists() || zipFile.length() == 0L) {
+                throw IOException("Failed to create ZIP file: file is empty or doesn't exist")
+            }
+            
+            return zipFile
+            
+        } catch (e: Exception) {
+            // Clean up failed export
+            if (zipFile.exists()) {
+                zipFile.delete()
+            }
+            throw IOException("Failed to create export ZIP: ${e.message}")
         }
-        
-        return zipFile
     }
     
     /**
@@ -84,37 +115,73 @@ object ZipExportImportUtils {
         exportData: com.atridad.openclimb.data.repository.ClimbDataExport,
         referencedImagePaths: Set<String>
     ) {
-        context.contentResolver.openOutputStream(uri)?.use { outputStream ->
+        try {
-            ZipOutputStream(outputStream).use { zipOut ->
+            context.contentResolver.openOutputStream(uri)?.use { outputStream ->
-                // Add JSON data file
+                ZipOutputStream(outputStream).use { zipOut ->
-                val json = Json { prettyPrint = true }
+                    // Add metadata file first
-                val jsonString = json.encodeToString(com.atridad.openclimb.data.repository.ClimbDataExport.serializer(), exportData)
+                    val metadata = createMetadata(exportData, referencedImagePaths)
-                
+                    val metadataEntry = ZipEntry(METADATA_FILENAME)
-                val jsonEntry = ZipEntry(DATA_JSON_FILENAME)
+                    zipOut.putNextEntry(metadataEntry)
-                zipOut.putNextEntry(jsonEntry)
+                    zipOut.write(metadata.toByteArray())
-                zipOut.write(jsonString.toByteArray())
+                    zipOut.closeEntry()
-                zipOut.closeEntry()
+                    
-                
+                    // Add JSON data file
-                // Add images
+                    val json = Json { 
-                referencedImagePaths.forEach { imagePath ->
+                        prettyPrint = true 
-                    try {
+                        ignoreUnknownKeys = true
-                        val imageFile = ImageUtils.getImageFile(context, imagePath)
-                        if (imageFile.exists()) {
-                            val imageEntry = ZipEntry("$IMAGES_DIR_NAME/${imageFile.name}")
-                            zipOut.putNextEntry(imageEntry)
-                            
-                            FileInputStream(imageFile).use { imageInput ->
-                                imageInput.copyTo(zipOut)
-                            }
-                            zipOut.closeEntry()
-                        }
-                    } catch (e: Exception) {
-                        // Log error but continue with other images
-                        e.printStackTrace()
                     }
+                    val jsonString = json.encodeToString(com.atridad.openclimb.data.repository.ClimbDataExport.serializer(), exportData)
+                    
+                    val jsonEntry = ZipEntry(DATA_JSON_FILENAME)
+                    zipOut.putNextEntry(jsonEntry)
+                    zipOut.write(jsonString.toByteArray())
+                    zipOut.closeEntry()
+                    
+                    // Add images with validation
+                    var successfulImages = 0
+                    referencedImagePaths.forEach { imagePath ->
+                        try {
+                            val imageFile = ImageUtils.getImageFile(context, imagePath)
+                            if (imageFile.exists() && imageFile.length() > 0) {
+                                val imageEntry = ZipEntry("$IMAGES_DIR_NAME/${imageFile.name}")
+                                zipOut.putNextEntry(imageEntry)
+                                
+                                FileInputStream(imageFile).use { imageInput ->
+                                    imageInput.copyTo(zipOut)
+                                }
+                                zipOut.closeEntry()
+                                successfulImages++
+                            }
+                        } catch (e: Exception) {
+                            android.util.Log.e("ZipExportImportUtils", "Failed to add image $imagePath: ${e.message}")
+                        }
+                    }
+                    
+                    android.util.Log.i("ZipExportImportUtils", "Export to URI completed: ${successfulImages}/${referencedImagePaths.size} images included")
                 }
-            }
+            } ?: throw IOException("Could not open output stream")
-        } ?: throw IOException("Could not open output stream")
+            
+        } catch (e: Exception) {
+            throw IOException("Failed to create export ZIP to URI: ${e.message}")
+        }
+    }
+    
+    private fun createMetadata(
+        exportData: com.atridad.openclimb.data.repository.ClimbDataExport,
+        referencedImagePaths: Set<String>
+    ): String {
+        return buildString {
+            appendLine("OpenClimb Export Metadata")
+            appendLine("=======================")
+            appendLine("Export Date: ${exportData.exportedAt}")
+            appendLine("Version: ${exportData.version}")
+            appendLine("Gyms: ${exportData.gyms.size}")
+            appendLine("Problems: ${exportData.problems.size}")
+            appendLine("Sessions: ${exportData.sessions.size}")
+            appendLine("Attempts: ${exportData.attempts.size}")
+            appendLine("Referenced Images: ${referencedImagePaths.size}")
+            appendLine("Format: ZIP with embedded JSON data and images")
+        }
     }
     
     /**
@@ -133,50 +200,89 @@ object ZipExportImportUtils {
      */
     fun extractImportZip(context: Context, zipFile: File): ImportResult {
         var jsonContent = ""
+        var metadataContent = ""
         val importedImagePaths = mutableMapOf<String, String>()
+        var foundRequiredFiles = mutableSetOf<String>()
         
-        ZipInputStream(FileInputStream(zipFile)).use { zipIn ->
+        try {
-            var entry = zipIn.nextEntry
+            ZipInputStream(FileInputStream(zipFile)).use { zipIn ->
-            
+                var entry = zipIn.nextEntry
-            while (entry != null) {
+                
-                when {
+                while (entry != null) {
-                    entry.name == DATA_JSON_FILENAME -> {
+                    when {
-                        // Read JSON data
+                        entry.name == METADATA_FILENAME -> {
-                        jsonContent = zipIn.readBytes().toString(Charsets.UTF_8)
+                            // Read metadata for validation
+                            metadataContent = zipIn.readBytes().toString(Charsets.UTF_8)
+                            foundRequiredFiles.add("metadata")
+                            android.util.Log.i("ZipExportImportUtils", "Found metadata: ${metadataContent.lines().take(3).joinToString()}")
+                        }
+                        
+                        entry.name == DATA_JSON_FILENAME -> {
+                            // Read JSON data
+                            jsonContent = zipIn.readBytes().toString(Charsets.UTF_8)
+                            foundRequiredFiles.add("data")
+                        }
+                        
+                        entry.name.startsWith("$IMAGES_DIR_NAME/") && !entry.isDirectory -> {
+                            // Extract image file
+                            val originalFilename = entry.name.substringAfter("$IMAGES_DIR_NAME/")
+                            
+                            try {
+                                // Create temporary file to hold the extracted image
+                                val tempFile = File.createTempFile("import_image_", "_$originalFilename", context.cacheDir)
+                                
+                                FileOutputStream(tempFile).use { output ->
+                                    zipIn.copyTo(output)
+                                }
+                                
+                                // Validate the extracted image
+                                if (tempFile.exists() && tempFile.length() > 0) {
+                                    // Import the image to permanent storage
+                                    val newPath = ImageUtils.importImageFile(context, tempFile)
+                                    if (newPath != null) {
+                                        importedImagePaths[originalFilename] = newPath
+                                        android.util.Log.d("ZipExportImportUtils", "Successfully imported image: $originalFilename -> $newPath")
+                                    } else {
+                                        android.util.Log.w("ZipExportImportUtils", "Failed to import image: $originalFilename")
+                                    }
+                                } else {
+                                    android.util.Log.w("ZipExportImportUtils", "Extracted image is empty: $originalFilename")
+                                }
+                                
+                                // Clean up temp file
+                                tempFile.delete()
+                                
+                            } catch (e: Exception) {
+                                android.util.Log.e("ZipExportImportUtils", "Failed to process image $originalFilename: ${e.message}")
+                            }
+                        }
+                        
+                        else -> {
+                            android.util.Log.d("ZipExportImportUtils", "Skipping ZIP entry: ${entry.name}")
+                        }
                     }
                     
-                    entry.name.startsWith("$IMAGES_DIR_NAME/") && !entry.isDirectory -> {
+                    zipIn.closeEntry()
-                        // Extract image file
+                    entry = zipIn.nextEntry
-                        val originalFilename = entry.name.substringAfter("$IMAGES_DIR_NAME/")
-                        
-                        // Create temporary file to hold the extracted image
-                        val tempFile = File.createTempFile("import_image_", "_$originalFilename", context.cacheDir)
-                        
-                        FileOutputStream(tempFile).use { output ->
-                            zipIn.copyTo(output)
-                        }
-                        
-                        // Import the image to permanent storage
-                        val newPath = ImageUtils.importImageFile(context, tempFile)
-                        if (newPath != null) {
-                            importedImagePaths[originalFilename] = newPath
-                        }
-                        
-                        // Clean up temp file
-                        tempFile.delete()
-                    }
                 }
-                
-                zipIn.closeEntry()
-                entry = zipIn.nextEntry
             }
+            
+            // Validate that we found the required files
+            if (!foundRequiredFiles.contains("data")) {
+                throw IOException("Invalid ZIP file: data.json not found")
+            }
+            
+            if (jsonContent.isBlank()) {
+                throw IOException("Invalid ZIP file: data.json is empty")
+            }
+            
+            android.util.Log.i("ZipExportImportUtils", "Import extraction completed: ${importedImagePaths.size} images processed")
+            
+            return ImportResult(jsonContent, importedImagePaths)
+            
+        } catch (e: Exception) {
+            throw IOException("Failed to extract import ZIP: ${e.message}")
         }
-        
-        if (jsonContent.isEmpty()) {
-            throw IOException("No data.json file found in the ZIP archive")
-        }
-        
-        return ImportResult(jsonContent, importedImagePaths)
     }
 
     /**

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

@@ -1,3 +1,4 @@
 <resources>
     <string name="app_name">OpenClimb</string>
+    <string name="session_tracking_service_description">Tracks active climbing sessions and displays session information in the notification area</string>
 </resources>

gradle/libs.versions.toml

@@ -1,24 +1,24 @@
 [versions]
 agp = "8.12.1"
-kotlin = "2.0.21"
+kotlin = "2.2.10"
-coreKtx = "1.15.0"
+coreKtx = "1.17.0"
 junit = "4.13.2"
 junitVersion = "1.3.0"
 espressoCore = "3.7.0"
-androidxTestCore = "1.6.0"
+androidxTestCore = "1.7.0"
-androidxTestExt = "1.2.0"
+androidxTestExt = "1.3.0"
-androidxTestRunner = "1.6.0"
+androidxTestRunner = "1.7.0"
-androidxTestRules = "1.6.0"
+androidxTestRules = "1.7.0"
-lifecycleRuntimeKtx = "2.9.2"
+lifecycleRuntimeKtx = "2.9.3"
 activityCompose = "1.10.1"
-composeBom = "2024.09.00"
+composeBom = "2025.08.01"
-room = "2.6.1"
+room = "2.7.2"
-navigation = "2.8.4"
+navigation = "2.9.3"
-viewmodel = "2.9.2"
+viewmodel = "2.9.3"
-kotlinxSerialization = "1.7.1"
+kotlinxSerialization = "1.9.0"
-kotlinxCoroutines = "1.9.0"
+kotlinxCoroutines = "1.10.2"
 coil = "2.7.0"
-ksp = "2.0.21-1.0.25"
+ksp = "2.2.10-2.0.2"
 
 [libraries]
 androidx-core-ktx = { group = "androidx.core", name = "core-ktx", version.ref = "coreKtx" }
@@ -64,8 +64,7 @@ mockk = { group = "io.mockk", name = "mockk", version = "1.13.8" }
 # Image Loading
 coil-compose = { group = "io.coil-kt", name = "coil-compose", version.ref = "coil" }
 
-# Charts - MPAndroidChart for now, will be replaced with Vico when stable
+
-mpandroidchart = { group = "com.github.PhilJay", name = "MPAndroidChart", version = "v3.1.0" }
 
 [plugins]
 android-application = { id = "com.android.application", version.ref = "agp" }