@siteed/audio-studio 3.0.3 → 3.0.5

package/CHANGELOG.md

@@ -8,6 +8,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 
+## [3.0.5] - 2026-04-25
+### Changed
+- fix(audio-studio): don't start notification on prepareRecording (Android) (#364) ([5d40d7e](https://github.com/deeeed/audiolab/commit/5d40d7e730f2b74459d319979fd9c112891b10f2))
+- docs: update api references for v3.0.4 ([12421ee](https://github.com/deeeed/audiolab/commit/12421ee6146187888492b957c0eee428fcab3e8b))
+- chore(audio-studio): release @siteed/audio-studio@3.0.4 ([3a5b7da](https://github.com/deeeed/audiolab/commit/3a5b7da8f4289a599d928dce01f262ab97398143))
+## [3.0.4] - 2026-04-25
+### Changed
+- fix(audio-studio): move module ops off main thread + fix iOS stop crash (#363) ([1a07d2d](https://github.com/deeeed/audiolab/commit/1a07d2d9222bfac1083b00d6de992fcb2a3be42d))
+- chore(audio-studio): release @siteed/audio-studio@3.0.3 ([9d4d14e](https://github.com/deeeed/audiolab/commit/9d4d14e8dffd4378a71f2d3dfb5d7140b8248c0f))
+- fix(audio-studio): convert trim ranges from Double to Long on Android (#347) (#360) ([6099e0e](https://github.com/deeeed/audiolab/commit/6099e0e81f3c8d7b1c109b415ad63e924f3f043b))
+- feat(audio-studio): web trimAudio, WASM refactor, new utilities (#344) ([3d4862a](https://github.com/deeeed/audiolab/commit/3d4862a57a5eb2496169d27f758d7628d7d3659a))
+- feat(sherpa-voice): comprehensive logging + fix Android prod model extraction (#342) ([5b20594](https://github.com/deeeed/audiolab/commit/5b20594c729a9deb703cbc2e23c07daa616149ab))
+- fix(audio-studio): remove blocking CDN poll from publish.sh, sync wasmConfig to 3.0.2 ([290409a](https://github.com/deeeed/audiolab/commit/290409aae8d1160e1952a5ee1961ce3864fbb0c8))
+- chore(audio-studio): release @siteed/audio-studio@3.0.2 ([ebd3300](https://github.com/deeeed/audiolab/commit/ebd330098d2756e6d5a555126e2dfc3fb038d155))
 ## [3.0.3] - 2026-04-12
 ### Changed
 - fix(audio-studio): convert trim ranges from Double to Long on Android (#347) (#360) ([6099e0e](https://github.com/deeeed/audiolab/commit/6099e0e81f3c8d7b1c109b415ad63e924f3f043b))
@@ -335,7 +349,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Audio features extraction during recording
 - Consistent WAV PCM recording format across all platforms
 
-[unreleased]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.3...HEAD
+[unreleased]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.5...HEAD
+[3.0.5]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.4...@siteed/audio-studio@3.0.5
+[3.0.4]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.3...@siteed/audio-studio@3.0.4
 [3.0.3]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.2...@siteed/audio-studio@3.0.3
 [3.0.2]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.2-beta.2...@siteed/audio-studio@3.0.2
 [3.0.1]: https://github.com/deeeed/audiolab/compare/@siteed/audio-studio@3.0.0...@siteed/audio-studio@3.0.1

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

@@ -455,49 +455,55 @@ class AudioRecorderManager(
     }
 
     private fun initializePhoneStateListener() {
-        try {
-            LogUtils.d(CLASS_NAME, "Initializing phone state listener...")
-
-            if (permissionUtils.checkPhoneStatePermission()) {
-                LogUtils.d(CLASS_NAME, "Phone state permission granted")
-
-                val localTelephonyManager = telephonyManager
-                if (localTelephonyManager != null) {
-                    try {
-                        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
-                            // API 31+: Use modern TelephonyCallback which reliably fires on Android 12+
-                            val callback = object : TelephonyCallback(), TelephonyCallback.CallStateListener {
-                                override fun onCallStateChanged(state: Int) {
-                                    handleCallStateChanged(state)
+        // The legacy PhoneStateListener (API < 31) requires a Looper on the
+        // thread that constructs it; calling listen() also expects the same.
+        // prepareRecording now runs on Dispatchers.IO (no Looper), so we
+        // marshal the whole registration to the main thread's Looper. The
+        // API 31+ path uses context.mainExecutor and is thread-safe, but we
+        // route everything through main for consistency.
+        android.os.Handler(android.os.Looper.getMainLooper()).post {
+            try {
+                LogUtils.d(CLASS_NAME, "Initializing phone state listener...")
+
+                if (permissionUtils.checkPhoneStatePermission()) {
+                    LogUtils.d(CLASS_NAME, "Phone state permission granted")
+
+                    val localTelephonyManager = telephonyManager
+                    if (localTelephonyManager != null) {
+                        try {
+                            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+                                val callback = object : TelephonyCallback(), TelephonyCallback.CallStateListener {
+                                    override fun onCallStateChanged(state: Int) {
+                                        handleCallStateChanged(state)
+                                    }
                                 }
-                            }
-                            telephonyCallback = callback
-                            localTelephonyManager.registerTelephonyCallback(context.mainExecutor, callback)
-                            LogUtils.d(CLASS_NAME, "Successfully registered TelephonyCallback (API 31+)")
-                        } else {
-                            // Legacy: PhoneStateListener for API < 31
-                            phoneStateListener = object : PhoneStateListener() {
-                                @Deprecated("Deprecated in API 31")
-                                override fun onCallStateChanged(state: Int, phoneNumber: String?) {
-                                    handleCallStateChanged(state)
+                                telephonyCallback = callback
+                                localTelephonyManager.registerTelephonyCallback(context.mainExecutor, callback)
+                                LogUtils.d(CLASS_NAME, "Successfully registered TelephonyCallback (API 31+)")
+                            } else {
+                                phoneStateListener = object : PhoneStateListener() {
+                                    @Deprecated("Deprecated in API 31")
+                                    override fun onCallStateChanged(state: Int, phoneNumber: String?) {
+                                        handleCallStateChanged(state)
+                                    }
                                 }
+                                localTelephonyManager.listen(phoneStateListener, PhoneStateListener.LISTEN_CALL_STATE)
+                                LogUtils.d(CLASS_NAME, "Successfully registered PhoneStateListener (legacy)")
                             }
-                            localTelephonyManager.listen(phoneStateListener, PhoneStateListener.LISTEN_CALL_STATE)
-                            LogUtils.d(CLASS_NAME, "Successfully registered PhoneStateListener (legacy)")
+                        } catch (e: SecurityException) {
+                            LogUtils.w(CLASS_NAME, "Missing permission for phone state listener: ${e.message}")
+                        } catch (e: Exception) {
+                            LogUtils.e(CLASS_NAME, "Failed to register phone state listener", e)
                         }
-                    } catch (e: SecurityException) {
-                        LogUtils.w(CLASS_NAME, "Missing permission for phone state listener: ${e.message}")
-                    } catch (e: Exception) {
-                        LogUtils.e(CLASS_NAME, "Failed to register phone state listener", e)
+                    } else {
+                        LogUtils.w(CLASS_NAME, "TelephonyManager is null, phone call interruption handling disabled (device may not have telephony service)")
                     }
                 } else {
-                    LogUtils.w(CLASS_NAME, "TelephonyManager is null, phone call interruption handling disabled (device may not have telephony service)")
+                    LogUtils.w(CLASS_NAME, "READ_PHONE_STATE permission not granted, phone call interruption handling disabled")
                 }
-            } else {
-                LogUtils.w(CLASS_NAME, "READ_PHONE_STATE permission not granted, phone call interruption handling disabled")
+            } catch (e: Exception) {
+                LogUtils.e(CLASS_NAME, "Failed to initialize phone state listener", e)
             }
-        } catch (e: Exception) {
-            LogUtils.e(CLASS_NAME, "Failed to initialize phone state listener", e)
         }
     }
 
@@ -903,12 +909,6 @@ class AudioRecorderManager(
                 LogUtils.d(CLASS_NAME, "Skipping primary file creation - primary output is disabled")
             }
 
-            if (recordingConfig.showNotification && enableBackgroundAudio) {
-                notificationManager.initialize(recordingConfig)
-                notificationManager.startUpdates(System.currentTimeMillis())
-                AudioRecordingService.startService(context)
-            }
-
             acquireWakeLock()
             audioProcessor.resetCumulativeAmplitudeRange()
             return true
@@ -954,20 +954,28 @@ class AudioRecorderManager(
 
             audioRecord?.startRecording()
             isPaused.set(false)
-            _isRecording.set(true)
             isFirstChunk = true
-
-            if (!isPaused.get()) {
-                recordingStartTime = System.currentTimeMillis()
+            recordingStartTime = System.currentTimeMillis()
+
+            // Start notification + foreground service before flipping isRecording (#298, #288).
+            // Previously the notification block fired in initializeRecordingResources, which is
+            // also reached from prepareRecording, so the notification timer started on prepare.
+            // Mirrors iOS fix in AudioStreamManager.swift. Service start is shared by both
+            // showNotification and keepAwake gates and must precede _isRecording=true so
+            // getStatus() can't observe (isRecording=true && !isServiceRunning).
+            val needsService = (recordingConfig.showNotification || recordingConfig.keepAwake) &&
+                enableBackgroundAudio
+            if (recordingConfig.showNotification && enableBackgroundAudio) {
+                notificationManager.initialize(recordingConfig)
+                notificationManager.startUpdates(recordingStartTime)
             }
-
-            recordingThread = Thread { recordingProcess() }.apply { start() }
-            
-            // Start service if keepAwake is true, but only if background audio is enabled (#288)
-            if (recordingConfig.keepAwake && enableBackgroundAudio) {
+            if (needsService) {
                 AudioRecordingService.startService(context)
             }
-            
+
+            _isRecording.set(true)
+            recordingThread = Thread { recordingProcess() }.apply { start() }
+
             return true
 
         } catch (e: Exception) {