Additional fixes for recording transcriber

Author: raivisdejus

buzz/transcriber/recording_transcriber.py

@@ -302,7 +302,7 @@ def start(self):
                         next_text: str = result.get("text")
 
                         # Update initial prompt between successive recording chunks
-                        initial_prompt += next_text
+                        initial_prompt = next_text
 
                         logging.debug(
                             "Received next result, length = %s, time taken = %s",
@@ -315,7 +315,11 @@ def start(self):
 
         except PortAudioError as exc:
             self.error.emit(str(exc))
-            logging.exception("")
+            logging.exception("PortAudio error during recording")
+            return
+        except Exception as exc:
+            logging.exception("Unexpected error during recording")
+            self.error.emit(str(exc))
             return
 
         self.finished.emit()
@@ -361,7 +365,11 @@ def stop_recording(self):
         self.is_running = False
         if self.process and self.process.poll() is None:
             self.process.terminate()
-            self.process.wait(timeout=5)
+            try:
+                self.process.wait(timeout=5)
+            except subprocess.TimeoutExpired:
+                self.process.kill()
+                logging.warning("Whisper server process had to be killed after timeout")
 
     def start_local_whisper_server(self):
         # Reduce verbose HTTP client logging from OpenAI/httpx
@@ -466,4 +474,7 @@ def start_local_whisper_server(self):
     def __del__(self):
         if self.process and self.process.poll() is None:
             self.process.terminate()
-            self.process.wait(timeout=5)
+            try:
+                self.process.wait(timeout=5)
+            except subprocess.TimeoutExpired:
+                self.process.kill()

buzz/widgets/audio_meter_widget.py

@@ -76,4 +76,4 @@ def update_amplitude(self, amplitude: float):
         self.current_amplitude = max(
             amplitude, self.current_amplitude * self.SMOOTHING_FACTOR
         )
-        self.repaint()
+        self.update()

buzz/widgets/recording_transcriber_widget.py

@@ -213,7 +213,7 @@ def __init__(
         self.presentation_options_bar.hide()
         self.copy_actions_bar = self.create_copy_actions_bar()
         layout.addWidget(self.copy_actions_bar)  # Add at the bottom
-        self.copy_actions_bar.hide() 
+        self.copy_actions_bar.hide()
 
     def create_presentation_options_bar(self) -> QWidget:
         """Crete the presentation options bar widget"""
@@ -297,15 +297,15 @@ def create_copy_actions_bar(self) -> QWidget:
         layout = QHBoxLayout(bar)
         layout.setContentsMargins(5, 5, 5, 5)
         layout.setSpacing(10)
-        
+
         layout.addStretch()  # Push button to the right
-        
+
         self.copy_transcript_button = QPushButton(_("Copy"), bar)
         self.copy_transcript_button.setToolTip(_("Copy transcription to clipboard"))
         self.copy_transcript_button.clicked.connect(self.on_copy_transcript_clicked)
         layout.addWidget(self.copy_transcript_button)
-        
-        return bar  
+
+        return bar
 
     def on_copy_transcript_clicked(self):
         """Handle copy transcript button click"""
@@ -340,7 +340,7 @@ def on_copy_transcript_clicked(self):
 
         self.copy_transcript_button.setText(_("Copied!"))
         QTimer.singleShot(2000, lambda: self.copy_transcript_button.setText(_("Copy")))
-                
+
     def on_show_presentation_clicked(self):
         """Handle click on 'Show in new window' button"""
         if self.presentation_window is None or not self.presentation_window.isVisible():
@@ -870,6 +870,7 @@ def stop_recording(self):
 
     def on_transcriber_finished(self):
         self.reset_record_button()
+        # Restart amplitude listener now that the transcription stream is closed
         self.reset_recording_amplitude_listener()
 
     def on_transcriber_error(self, error: str):