xo: Fix @typescript-eslint/consistent-type-definitions.

Signed-off-by: Anders Kaseorg <[email protected]>

Author: andersk

app/common/dnd-util.ts

@@ -13,10 +13,10 @@ export type DndSettings = {
 
 type SettingName = keyof DndSettings;
 
-interface Toggle {
+type Toggle = {
   dnd: boolean;
   newSettings: Partial<DndSettings>;
-}
+};
 
 export function toggle(): Toggle {
   const dnd = !ConfigUtil.getConfigItem("dnd", false);

app/common/logger-util.ts

@@ -6,9 +6,9 @@ import process from "node:process";
 import {initSetUp} from "./default-util";
 import {app} from "./remote";
 
-interface LoggerOptions {
+type LoggerOptions = {
   file?: string;
-}
+};
 
 initSetUp();
 

app/common/messages.ts

@@ -1,7 +1,7 @@
-interface DialogBoxError {
+type DialogBoxError = {
   title: string;
   content: string;
-}
+};
 
 export function invalidZulipServerError(domain: string): string {
   return `${domain} does not appear to be a valid Zulip server. Make sure that

app/common/typed-ipc.ts

@@ -1,7 +1,7 @@
 import type {DndSettings} from "./dnd-util";
 import type {MenuProps, ServerConf} from "./types";
 
-export interface MainMessage {
+export type MainMessage = {
   "clear-app-settings": () => void;
   "configure-spell-checker": () => void;
   "fetch-user-agent": () => string;
@@ -22,15 +22,15 @@ export interface MainMessage {
   "update-badge": (messageCount: number) => void;
   "update-menu": (props: MenuProps) => void;
   "update-taskbar-icon": (data: string, text: string) => void;
-}
+};
 
-export interface MainCall {
+export type MainCall = {
   "get-server-settings": (domain: string) => ServerConf;
   "is-online": (url: string) => boolean;
   "save-server-icon": (iconURL: string) => string;
-}
+};
 
-export interface RendererMessage {
+export type RendererMessage = {
   back: () => void;
   "copy-zulip-url": () => void;
   destroytray: () => void;
@@ -79,4 +79,4 @@ export interface RendererMessage {
   zoomActualSize: () => void;
   zoomIn: () => void;
   zoomOut: () => void;
-}
+};

app/common/types.ts

@@ -1,8 +1,8 @@
-export interface MenuProps {
+export type MenuProps = {
   tabs: TabData[];
   activeTabIndex?: number;
   enableMenu?: boolean;
-}
+};
 
 export type NavItem =
   | "General"
@@ -11,16 +11,16 @@ export type NavItem =
   | "Organizations"
   | "Shortcuts";
 
-export interface ServerConf {
+export type ServerConf = {
   url: string;
   alias: string;
   icon: string;
-}
+};
 
 export type TabRole = "server" | "function";
 
-export interface TabData {
+export type TabData = {
   role: TabRole;
   name: string;
   index: number;
-}
+};

app/renderer/js/clipboard-decrypter.ts

@@ -16,11 +16,11 @@ import crypto from "node:crypto";
 // don’t leak anything from the user’s clipboard other than the token
 // intended for us.
 
-export interface ClipboardDecrypter {
+export type ClipboardDecrypter = {
   version: number;
   key: Uint8Array;
   pasted: Promise<string>;
-}
+};
 
 export class ClipboardDecrypterImpl implements ClipboardDecrypter {
   version: number;

app/renderer/js/components/functional-tab.ts

@@ -5,9 +5,9 @@ import {generateNodeFromHtml} from "./base";
 import type {TabProps} from "./tab";
 import Tab from "./tab";
 
-export interface FunctionalTabProps extends TabProps {
+export type FunctionalTabProps = {
   $view: Element;
-}
+} & TabProps;
 
 export default class FunctionalTab extends Tab {
   $view: Element;

app/renderer/js/components/server-tab.ts

@@ -9,9 +9,9 @@ import type {TabProps} from "./tab";
 import Tab from "./tab";
 import type WebView from "./webview";
 
-export interface ServerTabProps extends TabProps {
+export type ServerTabProps = {
   webview: Promise<WebView>;
-}
+} & TabProps;
 
 export default class ServerTab extends Tab {
   webview: Promise<WebView>;

app/renderer/js/components/tab.ts

@@ -1,6 +1,6 @@
 import type {TabRole} from "../../../common/types";
 
-export interface TabProps {
+export type TabProps = {
   role: TabRole;
   icon?: string;
   name: string;
@@ -12,7 +12,7 @@ export interface TabProps {
   onHoverOut?: () => void;
   materialIcon?: string;
   onDestroy?: () => void;
-}
+};
 
 export default abstract class Tab {
   abstract $el: Element;

app/renderer/js/components/webview.ts

@@ -19,7 +19,7 @@ import {contextMenu} from "./context-menu";
 
 const shouldSilentWebview = ConfigUtil.getConfigItem("silent", false);
 
-interface WebViewProps {
+type WebViewProps = {
   $root: Element;
   rootWebContents: WebContents;
   index: number;
@@ -32,7 +32,7 @@ interface WebViewProps {
   preload?: string;
   onTitleChange: () => void;
   hasPermission?: (origin: string, permission: string) => boolean;
-}
+};
 
 export default class WebView {
   static templateHtml(props: WebViewProps): Html {