Fix delay when finalizing sound recording

ThomasFarstrike · ThomasFarstrike · commit 9286260453ff · 2025-12-17T22:20:24.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,7 @@
 0.5.2
 =====
 - AudioFlinger: optimize WAV volume scaling for speed and immediately set volume
-- AudioFlinger: eliminate thread by using TaskManager (asyncio)
+- AudioFlinger: add support for I2S microphone recording to WAV
 - AppStore app: eliminate all thread by using TaskManager
 - AppStore app: add support for BadgeHub backend
 - OSUpdate app: show download speed
diff --git a/internal_filesystem/apps/com.micropythonos.soundrecorder/assets/sound_recorder.py b/internal_filesystem/apps/com.micropythonos.soundrecorder/assets/sound_recorder.py
@@ -36,7 +36,7 @@ class SoundRecorder(Activity):
 
     # Constants
     MAX_DURATION_MS = 60000  # 60 seconds max recording
-    RECORDINGS_DIR = "data/com.micropythonos.soundrecorder/recordings"
+    RECORDINGS_DIR = "data/recordings"
 
     # UI Widgets
     _status_label = None
diff --git a/internal_filesystem/lib/mpos/audio/stream_record.py b/internal_filesystem/lib/mpos/audio/stream_record.py
@@ -261,10 +261,8 @@ def record(self):
 
             print(f"RecordStream: max_bytes={max_bytes}, chunk_size={chunk_size}")
 
-            # Open file for appending audio data
-            with open(self.file_path, 'r+b') as f:
-                f.seek(44)  # Skip header
-
+            # Open file for appending audio data (append mode to avoid seek issues)
+            with open(self.file_path, 'ab') as f:
                 buf = bytearray(chunk_size)
 
                 while self._keep_running and self._bytes_recorded < max_bytes:
@@ -294,8 +292,11 @@ def record(self):
                         f.write(buf[:num_read])
                         self._bytes_recorded += num_read
 
-                # Update header with actual data size
-                print(f"RecordStream: Updating WAV header with data_size={self._bytes_recorded}")
+            # Close the file first, then reopen to update header
+            # This avoids the massive delay caused by seeking backwards in a large file
+            # on ESP32 with SD card (FAT filesystem buffering issue)
+            print(f"RecordStream: Updating WAV header with data_size={self._bytes_recorded}")
+            with open(self.file_path, 'r+b') as f:
                 self._update_wav_header(f, self._bytes_recorded)
 
             elapsed_ms = time.ticks_diff(time.ticks_ms(), start_time)
