Remove some extension-only messages from app definitions

app/src/bridge/components/receiverSelector.ts

@@ -48,7 +48,7 @@ export function startReceiverSelector (data: string) {
 
             if (!jsonData.mediaType) {
                 sendMessage({
-                    subject: "main:receiverSelector/stop"
+                    subject: "main:receiverSelector/stopped"
                   , data: jsonData
                 });
 
@@ -74,7 +74,7 @@ export function startReceiverSelector (data: string) {
             selectorAppOpen = false;
 
             sendMessage({
-                subject: "main:receiverSelector/close"
+                subject: "main:receiverSelector/cancelled"
             });
         }
     });

app/src/bridge/index.ts

@@ -51,8 +51,8 @@ decodeTransform.on("data", (message: Message) => {
         }
 
         case "bridge:stopReceiverApp": {
-            stopReceiverApp(message.data.receiver.host
+            const { host, port } = message.data.receiver;
-                          , message.data.receiver.port);
+            stopReceiverApp(host, port);
             break;
         }
 
@@ -66,7 +66,8 @@ decodeTransform.on("data", (message: Message) => {
 
         // Media server
         case "bridge:mediaServer/start": {
-            startMediaServer(message.data.filePath, message.data.port);
+            const { filePath, port } = message.data;
+            startMediaServer(filePath, port);
             break;
         }
         case "bridge:mediaServer/stop": {

app/src/bridge/messaging.ts

@@ -7,150 +7,148 @@ import { Receiver
 
 
 type MessageDefinitions = {
-      "shim:serviceUp": { id: Receiver["id"] }
+    // Session messages
-    , "shim:serviceDown": { id: Receiver["id"] }
+    "shim:session/stopped": {}
-  
+  , "shim:session/connected": {
-    , "shim:launchApp": { receiver: Receiver }
+        sessionId: string
-  
+      , namespaces: Array<{ name: string }>
-      // Session messages
+      , displayName: string
-    , "shim:session/stopped": {}
+      , statusText: string
-    , "shim:session/connected": {
+    }
-          sessionId: string
+  , "shim:session/updateStatus": { volume: any /* Volume */ }
-        , namespaces: Array<{ name: string }>
+  , "shim:session/impl_addMessageListener": {
-        , displayName: string
+        namespace: string
-        , statusText: string
+      , data: string
-      }
+    }
-    , "shim:session/updateStatus": { volume: any /* Volume */ }
+  , "shim:session/impl_sendMessage": {
-    , "shim:session/impl_addMessageListener": {
+        messageId: string
-          namespace: string
+      , error: boolean
-        , data: string
+    }
-      }
+  , "shim:session/impl_setReceiverMuted": {
-    , "shim:session/impl_sendMessage": {
+        volumeId: string
-          messageId: string
+      , error: boolean
-        , error: boolean
+    }
-      }
+  , "shim:session/impl_setReceiverVolumeLevel": {
-    , "shim:session/impl_setReceiverMuted": {
+        volumeId: string
-          volumeId: string
+      , error: boolean
-        , error: boolean
+    }
-      }
+  , "shim:session/impl_stop": {
-    , "shim:session/impl_setReceiverVolumeLevel": {
+        stopId: string
-          volumeId: string
+      , error: boolean
-        , error: boolean
+    }
-      }
-    , "shim:session/impl_stop": {
-          stopId: string
-        , error: boolean
-      }
-  
-      // Bridge session messages
-    , "bridge:session/initialize": {
-          address: string
-        , port: number
-        , appId: string
-        , sessionId: string
-        , _id: string
-      }
-    , "bridge:session/close": {}
-    , "bridge:session/impl_leave": {
-          id: string
-        , _id: string
-      }
-    , "bridge:session/impl_sendMessage": {
-          namespace: string
-        , message: any
-        , messageId: string
-        , _id: string
-      }
-    , "bridge:session/impl_setReceiverMuted": {
-          muted: boolean
-        , volumeId: string
-        , _id: string
-      }
-    , "bridge:session/impl_setReceiverVolumeLevel": {
-          newLevel: number
-        , volumeId: string
-        , _id: string
-      }
-    , "bridge:session/impl_stop": {
-          stopId: string;
-          _id: string;
-      }
-    , "bridge:session/impl_addMessageListener": {
-          namespace: string;
-          _id: string;
-      }
-  
-      // Media messages
-    , "shim:media/update": {
-          currentTime: number
-        , _lastCurrentTime: number
-        , customData: any
-        , playbackRate: number
-        , playerState: string
-        , repeatMode: string
-        , _volumeLevel: number
-        , _volumeMuted: boolean
-        , media: unknown // MediaInfo
-        , mediaSessionId: number
-      }
-    , "shim:media/sendMediaMessageResponse": {
-          messageId: string
-        , error: boolean
-      }
-  
-      // Bridge media messages
-    , "bridge:media/initialize": {
-          sessionId: string
-        , mediaSessionId: number
-        , _internalSessionId: string
-        , _id: string
-      }
-    , "bridge:media/sendMediaMessage": {
-          message: any
-        , messageId: string
-        , _id: string
-      }
-  
-      // Bridge messages
-    , "main:receiverSelector/selected": ReceiverSelectionCast
-    , "main:receiverSelector/error": string
-    , "main:receiverSelector/close": {}
-    , "main:receiverSelector/stop": ReceiverSelectionStop
-  
-      /**
-       * getInfo uses the old :/ form for compat with old bridge
-       * versions.
-       */
-    , "bridge:getInfo": string
-    , "bridge:/getInfo": string
 
-    , "bridge:initialize": { shouldWatchStatus: boolean }
+    // Bridge session messages
+  , "bridge:session/initialize": {
+        address: string
+      , port: number
+      , appId: string
+      , sessionId: string
+      , _id: string
+    }
+  , "bridge:session/close": {}
+  , "bridge:session/impl_leave": {
+        id: string
+      , _id: string
+    }
+  , "bridge:session/impl_sendMessage": {
+        namespace: string
+      , message: any
+      , messageId: string
+      , _id: string
+    }
+  , "bridge:session/impl_setReceiverMuted": {
+        muted: boolean
+      , volumeId: string
+      , _id: string
+    }
+  , "bridge:session/impl_setReceiverVolumeLevel": {
+        newLevel: number
+      , volumeId: string
+      , _id: string
+    }
+  , "bridge:session/impl_stop": {
+        stopId: string;
+        _id: string;
+    }
+  , "bridge:session/impl_addMessageListener": {
+        namespace: string;
+        _id: string;
+    }
+
+    // Media messages
+  , "shim:media/update": {
+        currentTime: number
+      , _lastCurrentTime: number
+      , customData: any
+      , playbackRate: number
+      , playerState: string
+      , repeatMode: string
+      , _volumeLevel: number
+      , _volumeMuted: boolean
+      , media: unknown // MediaInfo
+      , mediaSessionId: number
+    }
+  , "shim:media/sendMediaMessageResponse": {
+        messageId: string
+      , error: boolean
+    }
+
+    // Bridge media messages
+  , "bridge:media/initialize": {
+        sessionId: string
+      , mediaSessionId: number
+      , _internalSessionId: string
+      , _id: string
+    }
+
+  , "bridge:media/sendMediaMessage": {
+        message: any
+      , messageId: string
+      , _id: string
+    }
+
+      // Bridge messages
+  , "main:receiverSelector/selected": ReceiverSelectionCast
+  , "main:receiverSelector/stopped": ReceiverSelectionStop
+  , "main:receiverSelector/cancelled": {}
+  , "main:receiverSelector/error": string
+
+    /**
+     * getInfo uses the old :/ form for compat with old bridge
+     * versions.
+     */
+  , "bridge:getInfo": string
+  , "bridge:/getInfo": string
+
+  , "bridge:initialize": {
+        shouldWatchStatus: boolean
+    }
+
+  , "bridge:receiverSelector/open": any
+  , "bridge:receiverSelector/close": {}
+
+  , "bridge:stopReceiverApp": { receiver: Receiver }
+
+  , "bridge:mediaServer/start": {
+        filePath: string
+      , port: number
+    }
+  , "bridge:mediaServer/stop": {}
+  , "mediaCast:mediaServer/started": {
+        mediaPath: string
+      , subtitlePaths: string[]
+      , localAddress: string
+    }
+  , "mediaCast:mediaServer/stopped": {}
+  , "mediaCast:mediaServer/error": {}
+
+  , "main:serviceUp": Receiver
+  , "main:serviceDown": { id: string }
   
-    , "bridge:receiverSelector/open": any
+  , "main:receiverStatus": {
-    , "bridge:receiverSelector/close": {}
+        id: string
-  
+      , status: ReceiverStatus
-    , "bridge:stopReceiverApp": { receiver: Receiver }
+    }
-  
-    , "bridge:mediaServer/start": {
-          filePath: string
-        , port: number
-      }
-    , "bridge:mediaServer/stop": {}
-    , "mediaCast:mediaServer/started": {
-          mediaPath: string
-        , subtitlePaths: string[]
-        , localAddress: string
-      }
-    , "mediaCast:mediaServer/stopped": {}
-    , "mediaCast:mediaServer/error": {}
-  
-    , "main:serviceUp": Receiver
-    , "main:serviceDown": { id: string }
-  
-    , "main:receiverStatus": {
-          id: string
-        , status: ReceiverStatus
-      }
 }
 
 

ext/src/background/receiverSelector/NativeReceiverSelector.ts

@@ -114,12 +114,7 @@ export default class NativeReceiverSelector extends ReceiverSelector {
 
                 break;
             }
-            case "main:receiverSelector/error": {
+            case "main:receiverSelector/cancelled": {
-                logger.error("Native receiver selector error", message.data);
-                this.dispatchEvent(new CustomEvent("error"));
-                break;
-            }
-            case "main:receiverSelector/close": {
                 if (!this.wasReceiverSelected) {
                     this.dispatchEvent(new CustomEvent("cancelled"));
                 }
@@ -134,12 +129,17 @@ export default class NativeReceiverSelector extends ReceiverSelector {
 
                 break;
             }
-            case "main:receiverSelector/stop": {
+            case "main:receiverSelector/stopped": {
                 this.dispatchEvent(new CustomEvent("stop", {
                     detail: message.data
                 }));
                 break;
             }
+            case "main:receiverSelector/error": {
+                logger.error("Native receiver selector error", message.data);
+                this.dispatchEvent(new CustomEvent("error"));
+                break;
+            }
         }
     }
 }

ext/src/messaging.ts

@@ -53,6 +53,10 @@ type ExtMessageDefinitions = {
   , "main:sessionCreated": {}
 
   , "shim:initialized": BridgeInfo
+  , "shim:serviceUp": { id: Receiver["id"] }
+  , "shim:serviceDown": { id: Receiver["id"] }
+
+  , "shim:launchApp": { receiver: Receiver }
 }
 
 /**
@@ -61,13 +65,8 @@ type ExtMessageDefinitions = {
  *   app/bridge/messaging.ts > MessagesBase
  */
 type AppMessageDefinitions = {
-    "shim:serviceUp": { id: Receiver["id"] }
-  , "shim:serviceDown": { id: Receiver["id"] }
-
-  , "shim:launchApp": { receiver: Receiver }
-
     // Session messages
-  , "shim:session/stopped": {}
+    "shim:session/stopped": {}
   , "shim:session/connected": {
         sessionId: string
       , namespaces: Array<{ name: string }>
@@ -167,9 +166,9 @@ type AppMessageDefinitions = {
 
     // Bridge messages
   , "main:receiverSelector/selected": ReceiverSelectionCast
+  , "main:receiverSelector/stopped": ReceiverSelectionStop
+  , "main:receiverSelector/cancelled": {}
   , "main:receiverSelector/error": string
-  , "main:receiverSelector/close": {}
-  , "main:receiverSelector/stop": ReceiverSelectionStop
 
     /**
      * getInfo uses the old :/ form for compat with old bridge
@@ -178,7 +177,9 @@ type AppMessageDefinitions = {
   , "bridge:getInfo": string
   , "bridge:/getInfo": string
 
-  , "bridge:initialize": { shouldWatchStatus: boolean }
+  , "bridge:initialize": {
+        shouldWatchStatus: boolean
+    }
 
   , "bridge:receiverSelector/open": any
   , "bridge:receiverSelector/close": {}