@siteed/expo-audio-stream 1.11.1 → 1.11.2

package/CHANGELOG.md

@@ -8,25 +8,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 
+## [1.11.2] - 2025-01-22
+- resources not cleanup properly on app kill (#80) ([7d522a5](https://github.com/deeeed/expo-audio-stream/commit/7d522a531e70065b99758aa3a4c669769fdbd110))
+
 ## [1.11.1] - 2025-01-22
 - chore: force deployment of 1.11.1
 
+
 ## [1.11.0] - 2025-01-22
 - feat(audio): add intelligent call interruption handling & compression improvements ([f8f6187](https://github.com/deeeed/expo-audio-stream/pull/78))
 
 
+
 ## [1.10.0] - 2025-01-14
 - add support for pausing and resuming compressed recordings ([bc3f629](https://github.com/deeeed/expo-audio-stream/commit/bc3f6295d060396325e0f008ff00b3be9c8722cd))
 - optimize notification channel settings ([daa075e](https://github.com/deeeed/expo-audio-stream/commit/daa075e668f8faf0b8d2849e18c37384bdd293b8))
 
 
 
+
 ## [1.9.2] - 2025-01-12
 - ios bitrate verification to prevent invalid values ([035a180](https://github.com/deeeed/expo-audio-stream/commit/035a1800833264edcc59724aaa8a2e12d5c78dc2))
 
 
 
 
+
 ## [1.9.1] - 2025-01-12
 - ios potentially missing compressed file info ([88a628c](https://github.com/deeeed/expo-audio-stream/commit/88a628c35f2bfd626a2a5de1eb6950efd814619d))
 
@@ -34,6 +41,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.9.0] - 2025-01-11
 - feat(web-audio): optimize memory usage and streaming performance for web audio recording (#75) ([7b93e12](https://github.com/deeeed/expo-audio-stream/commit/7b93e12aae4bc0599b06b48ca34a60f65587fc75))
 
@@ -42,6 +50,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.8.0] - 2025-01-10
 - feat(audio): implement audio compression support ([ff4e060](https://github.com/deeeed/expo-audio-stream/commit/ff4e060fef1061804c1cc0126d4344d2d50daa9a))
 
@@ -51,6 +60,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.7.2] - 2025-01-07
 - fix(audio-stream): correct WAV header handling in web audio recording ([9ba7de5](https://github.com/deeeed/expo-audio-stream/commit/9ba7de5b96ca4cc937dea261c80d3fda9c99e8f4))
 
@@ -61,6 +71,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.7.1] - 2025-01-07
 - update notification to avoid triggering new alerts (#71) ([32dcfc5](https://github.com/deeeed/expo-audio-stream/commit/32dcfc55daf3236babefc17016f329c177d466fd))
 
@@ -72,6 +83,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.7.0] - 2025-01-05
 - feat(playground): enhance app configuration and build setup for production deployment (#58) ([929d443](https://github.com/deeeed/expo-audio-stream/commit/929d443145378b1430d215db5c00b13758420e2b))
 - chore(expo-audio-stream): release @siteed/expo-audio-stream@1.6.1 ([084e8ad](https://github.com/deeeed/expo-audio-stream/commit/084e8adb91da7874c9e608b55d9c7b2ffd7a8327))
@@ -89,6 +101,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.6.1] - 2024-12-11
 - chore(expo-audio-stream): remove git commit step from publish script ([4a772ce](https://github.com/deeeed/expo-audio-stream/commit/4a772ce93bb7405d9b8e981f46bdf8941a71ecfe))
 - chore: more publishing automation ([3693021](https://github.com/deeeed/expo-audio-stream/commit/369302107f9dca9dddd8ae68e6214481a39976ac))
@@ -110,6 +123,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.5.0] - 2024-12-10
 - UNPUBLISHED because of a bug in the build system
 
@@ -124,6 +138,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.4.0] - 2024-12-05
 - chore: remove unusded dependencies ([ad81dd5](https://github.com/deeeed/expo-audio-stream/commit/ad81dd560c93dd1d04995a323a4ae72d4de20f3e))
 
@@ -138,6 +153,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.3.1] - 2024-12-05
 - feat(web): implement throttling and optimize event processing (#49) ([da28765](https://github.com/deeeed/expo-audio-stream/commit/da2876524c2c9d6e0a980fde40a0197b929d8a7f))
 
@@ -152,6 +168,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.3.0] - 2024-11-28
 ### Added
 - refactor(permissions): standardize permission status response structure across platforms (#44) ([7c9c800](https://github.com/deeeed/expo-audio-stream/commit/7c9c800d83b7cea3516643371484d5e1f3b99e4c))
@@ -171,6 +188,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.2.5] - 2024-11-12
 ### Added
 - docs(license): add MIT license to all packages (6 files changed)
@@ -187,6 +205,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.2.4] - 2024-11-05
 ### Changed
 - Android minimum audio interval set to 10ms. 
@@ -206,6 +225,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.2.0] - 2024-10-24
 ### Added
 - Feature: Keep device awake during recording with `keepAwake` option
@@ -225,6 +245,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.1.17] - 2024-10-21
 ### Added
 - Support bluetooth headset on ios
@@ -241,6 +262,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+
 ## [1.0.0] - 2024-04-01
 ### Added
 - Initial release of @siteed/expo-audio-stream.
@@ -251,7 +273,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Feature: Audio features extraction during recording.
 - Feature: Consistent WAV PCM recording format across all platforms.
 
-[unreleased]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.11.1...HEAD
+[unreleased]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.11.2...HEAD
+[1.11.2]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.11.1...@siteed/expo-audio-stream@1.11.2
 [1.11.1]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.11.0...@siteed/expo-audio-stream@1.11.1
 [1.11.0]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.10.0...@siteed/expo-audio-stream@1.11.0
 [1.10.0]: https://github.com/deeeed/expo-audio-stream/compare/@siteed/expo-audio-stream@1.9.2...@siteed/expo-audio-stream@1.10.0

package/android/src/main/java/net/siteed/audiostream/AudioRecorderManager.kt

@@ -28,6 +28,7 @@ import android.media.AudioAttributes
 import android.media.AudioFocusRequest
 import android.telephony.PhoneStateListener
 import android.telephony.TelephonyManager
+import android.app.ActivityManager
 
 class AudioRecorderManager(
     private val context: Context,
@@ -137,6 +138,11 @@ class AudioRecorderManager(
                 ).also { instance = it }
             }
         }
+
+        fun destroy() {
+            instance?.cleanup()
+            instance = null
+        }
     }
 
     @RequiresApi(Build.VERSION_CODES.R)
@@ -641,15 +647,28 @@ class AudioRecorderManager(
 
     fun getStatus(): Bundle {
         synchronized(audioRecordLock) {
+            // Check if service is actually running
+            val isServiceRunning = context.let { ctx ->
+                val manager = ctx.getSystemService(Context.ACTIVITY_SERVICE) as? ActivityManager
+                manager?.getRunningServices(Integer.MAX_VALUE)
+                    ?.any { it.service.className == AudioRecordingService::class.java.name }
+            } ?: false
+
+            // If service is running but we think we're not recording, clean up
+            if (isServiceRunning && !isRecording.get()) {
+                Log.d(Constants.TAG, "Detected orphaned recording service, cleaning up...")
+                cleanup()
+                AudioRecordingService.stopService(context)
+            }
+
             if (!isRecording.get()) {
                 Log.d(Constants.TAG, "Not recording --- skip status with default values")
-
                 return bundleOf(
                     "isRecording" to false,
                     "isPaused" to false,
                     "mime" to mimeType,
                     "size" to 0,
-                    "interval" to recordingConfig.interval,
+                    "interval" to (recordingConfig?.interval ?: 0)
                 )
             }
 
@@ -916,27 +935,51 @@ class AudioRecorderManager(
         isFirstChunk = false
     }
 
-    private fun cleanup() {
-        try {
-            if (recordingConfig.showNotification) {
-                notificationManager.stopUpdates()
-                AudioRecordingService.stopService(context)
-            }
+    fun cleanup() {
+        synchronized(audioRecordLock) {
+            try {
+                if (isRecording.get()) {
+                    audioRecord?.stop()
+                    compressedRecorder?.stop()
+                    compressedRecorder?.release()
+                }
+                
+                isRecording.set(false)
+                isPaused.set(false)
+                
+                if (recordingConfig.showNotification) {
+                    notificationManager.stopUpdates()
+                    AudioRecordingService.stopService(context)
+                }
 
-            // Reset all states
-            isRecording.set(false)
-            isPaused.set(false)
-            totalRecordedTime = 0
-            pausedDuration = 0
-            lastEmittedSize = 0
-            recordingStartTime = 0
-        } catch (e: Exception) {
-            Log.e(Constants.TAG, "Error in cleanup", e)
-        } finally {
-            releaseWakeLock()
+                releaseWakeLock()
+                releaseAudioFocus()
+                audioRecord?.release()
+                audioRecord = null
+                
+                // Reset all state
+                totalRecordedTime = 0
+                pausedDuration = 0
+                lastEmittedSize = 0
+                recordingStartTime = 0
+                
+                // Update the WAV header if needed
+                audioFile?.let { file ->
+                    audioFileHandler.updateWavHeader(file)
+                }
+
+                // Send event to notify that recording was stopped
+                eventSender.sendExpoEvent(Constants.RECORDING_INTERRUPTED_EVENT_NAME, bundleOf(
+                    "reason" to "appKilled",
+                    "isPaused" to false
+                ))
+            } catch (e: Exception) {
+                Log.e(Constants.TAG, "Error during cleanup", e)
+            }
         }
     }
 
+    @RequiresApi(Build.VERSION_CODES.Q)
     private fun initializeCompressedRecorder(fileExtension: String, promise: Promise): Boolean {
         try {
             // Use the existing audioFileHandler instance

package/android/src/main/java/net/siteed/audiostream/AudioRecordingService.kt

@@ -8,6 +8,7 @@ import android.os.IBinder
 import android.util.Log
 import android.os.Handler
 import android.os.Looper
+import expo.modules.kotlin.Promise
 
 class AudioRecordingService : Service() {
     private val notificationManager by lazy {
@@ -43,17 +44,46 @@ class AudioRecordingService : Service() {
     override fun onDestroy() {
         Log.d(Constants.TAG, "AudioRecordingService onDestroy")
 
-        if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.N) {
-            stopForeground(STOP_FOREGROUND_REMOVE)
-        } else {
-            @Suppress("DEPRECATION")
-            stopForeground(true)
-        }
+        stopForeground(STOP_FOREGROUND_REMOVE)
 
         isRunning = false
         super.onDestroy()
     }
 
+    override fun onTaskRemoved(rootIntent: Intent?) {
+        super.onTaskRemoved(rootIntent)
+        Log.d(Constants.TAG, "AudioRecordingService onTaskRemoved")
+        
+        // Stop recording when app is killed
+        AudioRecorderManager.getInstance()?.let { manager ->
+            mainHandler.post {
+                // Create a simple promise object for internal use
+                val promise = object : Promise {
+                    override fun resolve(value: Any?) {
+                        Log.d(Constants.TAG, "Successfully stopped recording on task removed")
+                        cleanup()
+                    }
+                    override fun reject(code: String, message: String?, cause: Throwable?) {
+                        Log.e(Constants.TAG, "Failed to stop recording on task removed: $message")
+                        cleanup()
+                    }
+                }
+
+                try {
+                    manager.stopRecording(promise)
+                } catch (e: Exception) {
+                    promise.reject("ERROR", e.message, e)
+                }
+            }
+        }
+    }
+
+    
+    private fun cleanup() {
+        stopForeground(STOP_FOREGROUND_REMOVE)
+        stopSelf()
+    }
+
     companion object {
         fun startService(context: Context) {
             val serviceIntent = Intent(context, AudioRecordingService::class.java)

package/android/src/main/java/net/siteed/audiostream/ExpoAudioStreamModule.kt

@@ -1,6 +1,8 @@
 package net.siteed.audiostream
 
 import android.Manifest
+import android.app.ActivityManager
+import android.content.Context
 import android.os.Build
 import android.os.Bundle
 import android.util.Log
@@ -171,6 +173,29 @@ class ExpoAudioStreamModule : Module(), EventSender {
                 )
             }
         }
+
+        OnDestroy {
+            AudioRecorderManager.destroy()
+        }
+
+        // Add a new function to check if recording is actually running
+        AsyncFunction("checkRecordingStatus") { promise: Promise ->
+            val isServiceRunning = AudioRecordingService::class.java.name.let { className ->
+                val manager = appContext.reactContext?.getSystemService(Context.ACTIVITY_SERVICE) as? ActivityManager
+                manager?.getRunningServices(Integer.MAX_VALUE)
+                    ?.any { it.service.className == className }
+            } ?: false
+
+            val status = audioRecorderManager.getStatus()
+            
+            // If service is running but isRecording is false, we need to cleanup
+            if (isServiceRunning && !status.getBoolean("isRecording")) {
+                audioRecorderManager.cleanup()
+                AudioRecordingService.stopService(appContext.reactContext!!)
+            }
+            
+            promise.resolve(status)
+        }
     }
 
     private fun initializeManager() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@siteed/expo-audio-stream",
-  "version": "1.11.1",
+  "version": "1.11.2",
   "description": "stream audio crossplatform",
   "license": "MIT",
   "main": "build/index.js",