Make type imports explicit

app/src/bridge/components/cast/Session.ts

@@ -1,11 +1,11 @@
 "use strict";
 
-import { Channel } from "castv2";
+import type { Channel } from "castv2";
 
 import messaging from "../../messaging";
 
-import { ReceiverDevice } from "../../messagingTypes";
-import { ReceiverMessage } from "./types";
+import type { ReceiverDevice } from "../../messagingTypes";
+import type { ReceiverMessage } from "./types";
 
 import CastClient, { NS_CONNECTION, NS_HEARTBEAT } from "./client";
 

app/src/bridge/components/cast/client.ts

@@ -2,7 +2,7 @@
 
 import { Channel, Client } from "castv2";
 
-import { ReceiverMessage, SenderMessage } from "./types";
+import type { ReceiverMessage, SenderMessage } from "./types";
 
 export const NS_CONNECTION = "urn:x-cast:com.google.cast.tp.connection";
 export const NS_HEARTBEAT = "urn:x-cast:com.google.cast.tp.heartbeat";

app/src/bridge/components/cast/discovery.ts

@@ -2,7 +2,7 @@
 
 import mdns from "mdns";
 
-import { ReceiverDevice } from "../../messagingTypes";
+import type { ReceiverDevice } from "../../messagingTypes";
 
 /**
  * Chromecast TXT record

app/src/bridge/components/cast/remote.ts

@@ -2,7 +2,7 @@
 
 import CastClient from "./client";
 
-import {
+import type {
     MediaStatus,
     ReceiverMessage,
     ReceiverMediaMessage,

app/src/bridge/messaging.ts

@@ -4,14 +4,14 @@ import { TypedEmitter } from "tiny-typed-emitter";
 
 import { DecodeTransform, EncodeTransform } from "../transforms";
 
-import {
+import type {
     MediaStatus,
     ReceiverStatus,
     SenderMediaMessage,
     SenderMessage
 } from "./components/cast/types";
 
-import {
+import type {
     ReceiverDevice,
     CastSessionCreatedDetails,
     CastSessionUpdatedDetails

app/src/bridge/messagingTypes.ts

@@ -1,6 +1,6 @@
 "use strict";
 
-import {
+import type {
     Image,
     ReceiverStatus,
     SenderApplication,

app/src/transforms.ts

@@ -1,7 +1,7 @@
 "use strict";
 
 import { Transform, TransformCallback } from "stream";
-import { Message } from "./bridge/messaging";
+import type { Message } from "./bridge/messaging";
 
 type ResponseHandlerFunction = (message: Message) => Promise<unknown>;
 

ext/src/background/deviceManager.ts

@@ -4,9 +4,10 @@ import bridge from "../lib/bridge";
 import logger from "../lib/logger";
 import { TypedEventTarget } from "../lib/TypedEventTarget";
 
-import { Message, Port } from "../messaging";
-import { ReceiverDevice } from "../types";
-import {
+import type { Message, Port } from "../messaging";
+import type { ReceiverDevice } from "../types";
+
+import type {
     MediaStatus,
     ReceiverStatus,
     SenderMediaMessage,

ext/src/cast/eventMessaging.ts

@@ -1,7 +1,7 @@
 "use strict";
 
 import logger from "../lib/logger";
-import { Message } from "../messaging";
+import type { Message } from "../messaging";
 
 type EventMessengerListener = (message: Message) => void;
 

ext/src/cast/export.ts

@@ -1,9 +1,9 @@
 "use strict";
 
-import { Message } from "../messaging";
+import type { Message } from "../messaging";
 
-import { BridgeInfo } from "../lib/bridge";
-import { TypedMessagePort } from "../lib/TypedMessagePort";
+import type { BridgeInfo } from "../lib/bridge";
+import type { TypedMessagePort } from "../lib/TypedMessagePort";
 
 import CastSDK from "./sdk";
 

ext/src/cast/sdk/Session.ts

@@ -6,7 +6,7 @@ import logger from "../../lib/logger";
 
 import eventMessaging from "../eventMessaging";
 
-import {
+import type {
     ErrorCallback,
     LoadSuccessCallback,
     MediaListener,
@@ -24,10 +24,10 @@ import {
 } from "./types";
 
 import { SessionStatus } from "./enums";
-import { Image, Receiver, SenderApplication } from "./classes";
+import type { Image, Receiver, SenderApplication } from "./classes";
 
 import { MediaCommand } from "./media/enums";
-import { LoadRequest, QueueLoadRequest, QueueItem } from "./media/classes";
+import type { LoadRequest, QueueLoadRequest, QueueItem } from "./media/classes";
 import Media, { NS_MEDIA } from "./media/Media";
 
 /**

ext/src/cast/sdk/classes.ts

@@ -1,6 +1,6 @@
 "use strict";
 
-import Session from "./Session";
+import type Session from "./Session";
 
 import {
     AutoJoinPolicy,

ext/src/cast/sdk/index.ts

@@ -2,11 +2,11 @@
 
 import logger from "../../lib/logger";
 
-import { Message } from "../../messaging";
+import type { Message } from "../../messaging";
 import eventMessaging from "../eventMessaging";
 
-import { ReceiverDevice } from "../../types";
-import { ErrorCallback, SuccessCallback } from "../types";
+import type { ReceiverDevice } from "../../types";
+import type { ErrorCallback, SuccessCallback } from "../types";
 
 import {
     AutoJoinPolicy,

ext/src/cast/sdk/media/Media.ts

@@ -30,8 +30,12 @@ import {
 import { PlayerState, RepeatMode } from "./enums";
 import { ErrorCode } from "../enums";
 
-import { ErrorCallback, SuccessCallback, UpdateListener } from "../../types";
-import { SenderMediaMessage } from "../types";
+import type {
+    ErrorCallback,
+    SuccessCallback,
+    UpdateListener
+} from "../../types";
+import type { SenderMediaMessage } from "../types";
 import { getEstimatedTime } from "../../utils";
 
 export const NS_MEDIA = "urn:x-cast:com.google.cast.media";

ext/src/cast/sdk/media/classes.ts

@@ -1,6 +1,6 @@
 "use strict";
 
-import { Image, Volume } from "../classes";
+import type { Image, Volume } from "../classes";
 
 import {
     ContainerType,

ext/src/cast/sdk/types.ts

@@ -5,9 +5,9 @@
  *   app/src/bridge/components/cast/types.ts
  */
 
-import { SenderApplication, Volume, Image } from "./classes";
-import { MediaInfo, QueueItem } from "./media/classes";
-import {
+import type { SenderApplication, Volume, Image } from "./classes";
+import type { MediaInfo, QueueItem } from "./media/classes";
+import type {
     IdleReason,
     PlayerState,
     RepeatMode,

ext/src/cast/senders/media/index.ts

@@ -4,8 +4,8 @@ import logger from "../../../lib/logger";
 import options from "../../../lib/options";
 import cast, { ensureInit } from "../../export";
 
-import { Message } from "../../../messaging";
-import { ReceiverDevice } from "../../../types";
+import type { Message } from "../../../messaging";
+import type { ReceiverDevice } from "../../../types";
 
 import type Session from "../../sdk/Session";
 import type Media from "../../sdk/media/Media";

ext/src/cast/types.ts

@@ -1,7 +1,7 @@
 "use strict";
 
-import { Error as Error_ } from "./sdk/classes";
-import Media from "./sdk/media/Media";
+import type { Error as Error_ } from "./sdk/classes";
+import type Media from "./sdk/media/Media";
 
 export type SuccessCallback = () => void;
 export type ErrorCallback = (err: Error_) => void;

ext/src/lib/bridge.ts

@@ -3,7 +3,7 @@
 import semver from "semver";
 
 import logger from "./logger";
-import { Port } from "../messaging";
+import type { Port } from "../messaging";
 import * as nativeMessaging from "./nativeMessaging";
 import options from "./options";
 

ext/src/lib/nativeMessaging.ts

@@ -3,7 +3,7 @@
 import logger from "./logger";
 import options from "./options";
 
-import { Message, Port } from "../messaging";
+import type { Message, Port } from "../messaging";
 
 type DisconnectListener = (port: Port) => void;
 type MessageListener = (message: Message) => void;

ext/src/messaging.ts

@@ -1,9 +1,9 @@
 "use strict";
 
-import { TypedPort } from "./lib/TypedPort";
-import { BridgeInfo } from "./lib/bridge";
+import type { TypedPort } from "./lib/TypedPort";
+import type { BridgeInfo } from "./lib/bridge";
 
-import {
+import type {
     ReceiverSelection,
     ReceiverSelectionCast,
     ReceiverSelectionStop,
@@ -11,7 +11,7 @@ import {
     ReceiverSelectorReceiverMessage
 } from "./background/receiverSelector";
 
-import {
+import type {
     CastSessionCreatedDetails,
     CastSessionUpdatedDetails,
     MediaStatus,
@@ -19,9 +19,9 @@ import {
     SenderMediaMessage,
     SenderMessage
 } from "./cast/sdk/types";
-import { SessionRequest } from "./cast/sdk/classes";
+import type { SessionRequest } from "./cast/sdk/classes";
 
-import { ReceiverDevice, ReceiverSelectorMediaType } from "./types";
+import type { ReceiverDevice, ReceiverSelectorMediaType } from "./types";
 
 /**
  * Messages are JSON objects with a `subject` string key and a

ext/src/types.ts

@@ -1,7 +1,7 @@
 "use strict";
 
-import { SessionRequest } from "./cast/sdk/classes";
-import { MediaStatus, ReceiverStatus } from "./cast/sdk/types";
+import type { SessionRequest } from "./cast/sdk/classes";
+import type { MediaStatus, ReceiverStatus } from "./cast/sdk/types";
 
 export enum ReceiverDeviceCapabilities {
     NONE = 0,

ext/src/ui/options/Bridge.svelte

@@ -11,7 +11,7 @@
     } from "../../lib/bridge";
     import logger from "../../lib/logger";
 
-    import { Options } from "../../lib/options";
+    import type { Options } from "../../lib/options";
 
     const _ = browser.i18n.getMessage;
 

ext/src/ui/options/Whitelist.svelte

@@ -1,10 +1,12 @@
 <script lang="ts">
     import { tick } from "svelte";
 
-    import knownApps, { KnownApp } from "../../cast/knownApps";
-    import { WhitelistItemData } from "../../background/whitelist";
+    import type { WhitelistItemData } from "../../background/whitelist";
+
     import { REMOTE_MATCH_PATTERN_REGEX } from "../../lib/matchPattern";
-    import { Options } from "../../lib/options";
+    import type { Options } from "../../lib/options";
+
+    import knownApps, { KnownApp } from "../../cast/knownApps";
 
     const _ = browser.i18n.getMessage;
 

ext/src/ui/popup/Popup.svelte

@@ -430,7 +430,8 @@
                 {device}
                 {isMediaTypeAvailable}
                 isAnyMediaTypeAvailable={availableMediaTypes !==
-                    ReceiverSelectorMediaType.None && isDeviceCompatible(device)}
+                    ReceiverSelectorMediaType.None &&
+                    isDeviceCompatible(device)}
                 isAnyConnecting={isConnecting}
                 on:cast={ev => onReceiverCast(ev.detail.device)}
                 on:stop={ev => onReceiverStop(ev.detail.device)}

ext/src/ui/popup/Receiver.svelte

@@ -1,11 +1,14 @@
 <script lang="ts">
     import { createEventDispatcher, onMount } from "svelte";
 
-    import { PlayerState } from "../../cast/sdk/media/enums";
-    import { SenderMediaMessage, SenderMessage } from "../../cast/sdk/types";
-    import { ReceiverDevice } from "../../types";
+    import type { ReceiverDevice } from "../../types";
+    import type { Port } from "../../messaging";
 
-    import { Port } from "../../messaging";
+    import { PlayerState } from "../../cast/sdk/media/enums";
+    import type {
+        SenderMediaMessage,
+        SenderMessage
+    } from "../../cast/sdk/types";
 
     import LoadingIndicator from "../LoadingIndicator.svelte";
     import ReceiverMedia from "./ReceiverMedia.svelte";

ext/src/ui/popup/ReceiverMedia.svelte

@@ -1,20 +1,21 @@
 <script lang="ts">
     import { createEventDispatcher, onMount } from "svelte";
 
-    import { ReceiverDevice } from "../../types";
+    import type { ReceiverDevice } from "../../types";
+
     import { MediaStatus, _MediaCommand } from "../../cast/sdk/types";
-    import { Image, Volume } from "../../cast/sdk/classes";
+    import type { Image, Volume } from "../../cast/sdk/classes";
     import {
         MetadataType,
         PlayerState,
         StreamType,
         TrackType
     } from "../../cast/sdk/media/enums";
+    import { getEstimatedTime } from "../../cast/utils";
 
     const _ = browser.i18n.getMessage;
 
     import deviceStore from "./deviceStore";
-    import { getEstimatedTime } from "../../cast/utils";
 
     const dispatch = createEventDispatcher<{
         togglePlayback: void;

ext/src/ui/popup/deviceStore.ts

@@ -1,4 +1,4 @@
 import { writable } from "svelte/store";
-import { ReceiverDevice } from "../../types";
+import type { ReceiverDevice } from "../../types";
 
 export default writable<ReceiverDevice[]>([]);

tsconfig.json

@@ -7,7 +7,8 @@
         "removeComments": true,
         "resolveJsonModule": true,
         "target": "es6",
-        "strict": true
+        "strict": true,
+        "importsNotUsedAsValues": "error"
     },
     "exclude": ["node_modules/**/*", "**/node_modules/**/*", "dist/**/*"]
 }