chore(ui): fix tsconfig

See also
- P4sca1/cron-schedule#289 (comment)
- microsoft/TypeScript#50794

Author: aentwist

Diff for: ui/src/api.ts

@@ -2,8 +2,8 @@ import { spawn } from "child_process";
 import type { BrowserWindow } from "electron";
 import { app, ipcMain } from "electron";
 import path from "path";
-import type { CommandData, Message, RunData, SettingsData } from "./types";
-import { CommandType } from "./types";
+import type { CommandData, Message, RunData, SettingsData } from "./types.ts";
+import { CommandType } from "./types.ts";
 
 export class Api {
   private static instance = new Api();

Diff for: ui/src/main.ts

@@ -1,9 +1,9 @@
 import { BrowserWindow, app, ipcMain, shell } from "electron";
 import fs from "fs/promises";
 import path from "path";
-import { Api } from "./api";
-import type { RootState } from "./stores";
-import { registerUpdateHandlers, startUpdatePolling } from "./update";
+import { Api } from "./api.ts";
+import type { RootState } from "./stores/index.ts";
+import { registerUpdateHandlers, startUpdatePolling } from "./update.ts";
 
 // Handle creating/removing shortcuts on Windows when installing/uninstalling.
 if (require("electron-squirrel-startup")) {

Diff for: ui/src/preload.ts

@@ -2,9 +2,9 @@
 // https://www.electronjs.org/docs/latest/tutorial/process-model#preload-scripts
 
 import { contextBridge, ipcRenderer } from "electron";
-import type { RootState } from "./stores";
-import type { Message, RunData, SettingsData } from "./types";
-import { CommandType } from "./types";
+import type { RootState } from "./stores/index.ts";
+import type { Message, RunData, SettingsData } from "./types.ts";
+import { CommandType } from "./types.ts";
 
 const electronApi = {
   saveState: (state: Partial<RootState>): Promise<void> =>

Diff for: ui/tsconfig.app.json

@@ -1,32 +1,20 @@
 {
-  "compilerOptions": {
-    "target": "ES2021",
-    "useDefineForClassFields": true,
-    "lib": ["ES2021", "DOM", "DOM.Iterable"],
-    "module": "commonjs",
-    "skipLibCheck": true,
-    "esModuleInterop": true,
-    "noImplicitAny": true,
-    "sourceMap": true,
-    "baseUrl": ".",
-    "outDir": "dist",
-
-    /* Bundler mode */
-    "moduleResolution": "node",
-    // "allowImportingTsExtensions": true,
-    "resolveJsonModule": true,
-    // "isolatedModules": true,
-    "noEmit": true,
-    "jsx": "react-jsx",
-    // https://github.com/emotion-js/emotion/issues/1249#issuecomment-828088254
-    "jsxImportSource": "@emotion/react",
-    "types": ["vitest/globals"],
+  "extends": "./tsconfig.base.json",
+  "include": ["src/**/*"],
+  "exclude": [
+    "src/main.ts",
+    "src/preload.ts",
+    "src/update.ts",
+    "src/api.ts",
 
-    /* Linting */
-    "strict": true,
-    "noUnusedLocals": true,
-    "noUnusedParameters": true,
-    "noFallthroughCasesInSwitch": true
-  },
-  "include": ["src/**/*"]
+    "src/**/*.test.*",
+    "src/App.tesst.tsx"
+  ],
+  "compilerOptions": {
+    "module": "esnext",
+    "moduleResolution": "bundler",
+    "isolatedModules": true,
+    "noEmit": true, // might not be needed
+    "lib": ["esnext"]
+  }
 }

Diff for: ui/tsconfig.base.json

@@ -0,0 +1,18 @@
+{
+  "compilerOptions": {
+    // recommended
+    "target": "es2016",
+    "module": "esnext",
+    "esModuleInterop": true,
+    "strict": true,
+    "skipLibCheck": true,
+
+    // other common options
+    "resolveJsonModule": true,
+
+    // both app and node (test) need jsx
+    "jsx": "react-jsx",
+    // https://github.com/emotion-js/emotion/issues/1249#issuecomment-828088254
+    "jsxImportSource": "@emotion/react"
+  }
+}

Diff for: ui/tsconfig.node.json

@@ -1,10 +1,21 @@
 {
+  "extends": "./tsconfig.base.json",
+  "include": [
+    "src/main.ts",
+    "src/preload.ts",
+    "src/update.ts",
+    "src/api.ts",
+
+    "src/**/*.test.*",
+
+    "*.config.*",
+    "forge.env.d.ts"
+  ],
   "compilerOptions": {
-    "skipLibCheck": true,
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "allowSyntheticDefaultImports": true,
-    "strict": true
-  },
-  "include": ["vite.*.ts", "forge.*.ts"]
+    "module": "nodenext",
+    "moduleResolution": "nodenext",
+    "noEmit": true,
+    "allowImportingTsExtensions": true,
+    "types": ["vitest/globals"]
+  }
 }

Diff for: ui/vite.main.config.ts

@@ -5,7 +5,7 @@ import {
   getBuildConfig,
   getBuildDefine,
   pluginHotRestart,
-} from "./vite.base.config";
+} from "./vite.base.config.ts";
 
 // https://vitejs.dev/config
 export default defineConfig((env) => {

Diff for: ui/vite.preload.config.ts

@@ -1,6 +1,10 @@
 import type { ConfigEnv, UserConfig } from "vite";
 import { defineConfig, mergeConfig } from "vite";
-import { external, getBuildConfig, pluginHotRestart } from "./vite.base.config";
+import {
+  external,
+  getBuildConfig,
+  pluginHotRestart,
+} from "./vite.base.config.ts";
 
 // https://vitejs.dev/config
 export default defineConfig((env) => {

Diff for: ui/vite.renderer.config.ts

@@ -1,6 +1,6 @@
 import type { ConfigEnv, UserConfig } from "vite";
 import { defineConfig } from "vite";
-import { pluginExposeRenderer } from "./vite.base.config";
+import { pluginExposeRenderer } from "./vite.base.config.ts";
 
 // https://vitejs.dev/config
 export default defineConfig((env) => {