Stop recognition when the user closes the pop-up

src/VXController.js

@@ -19,11 +19,19 @@ const state = mobx.observable({
 
 // View
 {
+  const actions = {
+    stop() {
+      if (isActive()) {
+        recognition.stop()
+      }
+    },
+  }
+
   const view = payload => {
     views.forEach(v => {
       try {
         if (currentSettings) {
-          v(payload, currentSettings)
+          v(payload, currentSettings, actions)
         }
       } catch (e) {
         console.error(`Output module "${v.name}" failed`, e)

src/VXOutputModules.js

@@ -5,6 +5,8 @@ export const PopupOutputModule = (() => {
   let activePopup
   let state
   let autoFocus
+  let stop
+
   function broadcastState() {
     if (state) {
       chrome.runtime.sendMessage({ state }, () => {
@@ -39,6 +41,7 @@ export const PopupOutputModule = (() => {
     chrome.runtime.onMessage.addListener(messageListener)
     const windowRemovedListener = function(removedWindowId) {
       if (removedWindowId === windowId) {
+        if (stop) stop()
         chrome.windows.onRemoved.removeListener(windowRemovedListener)
         activePopup = null
       }
@@ -79,13 +82,17 @@ export const PopupOutputModule = (() => {
   return /** @type {VXOutput} */ (function PopupOutputModule(
     nextState,
     settings,
+    actions,
   ) {
     if (settings.outputPopup !== 'on') return
     autoFocus = settings.outputPopupAutoFocus === 'on'
-
+    stop = actions.stop
     state = nextState
+
     if (nextState.showing && !activePopup) {
-      activePopup = createPopup()
+      if (nextState.status !== 'ended') {
+        activePopup = createPopup()
+      }
     } else if (!nextState.showing && activePopup) {
       activePopup.dispose()
       activePopup = null

src/types.ts

@@ -1,11 +1,14 @@
 interface VXOutput {
   (
     state: {
-      status: string
+      status: 'idle' | 'listening' | 'starting' | 'error' | 'ended'
       finalTranscript: string
       interimTranscript: string
       showing: string
     },
     settings: typeof import('./VXDefaultSettings').defaultSettings,
+    actions: {
+      stop(): void
+    },
   ): void
 }