Fix type errors and disable typescript incremental mode

.github/workflows/ci.yml

@@ -42,13 +42,6 @@ jobs:
     #     path: ${{ env.SCCACHE_PATH }}
     #     key: sccache-${{ runner.os }}-${{ hashFiles('pnpm-lock.yaml', 'patches/**') }}
 
-    - name: Restore cached tsconfig.tsbuildinfo
-      uses: actions/cache/restore@668228422ae6a00e4ad889ee87cd7109ec5666a7 # v5.0.4
-      id: cache-lint
-      with:
-        path: tsconfig.tsbuildinfo
-        key: lint-${{ runner.os }}-${{ hashFiles('pnpm-lock.yaml', 'patches/**', 'tsconfig.json') }}
-
     - name: Install Desktop node_modules
       run: pnpm install
       env:
@@ -84,13 +77,6 @@ jobs:
 
     - run: git diff --exit-code
 
-    - name: Update cached tsconfig.tsbuildinfo
-      uses: actions/cache/save@668228422ae6a00e4ad889ee87cd7109ec5666a7 # v5.0.4
-      if: github.ref == 'refs/heads/main'
-      with:
-        path: tsconfig.tsbuildinfo
-        key: ${{ steps.cache-lint.outputs.cache-primary-key }}
-
   macos:
     name: MacOS
     needs: lint

ts/components/Preferences.dom.tsx

@@ -1108,7 +1108,9 @@ export function Preferences({
             <Select
               id={themeSelectId}
               disabled={themeSetting === undefined}
-              onChange={onThemeChange}
+              onChange={value => {
+                onThemeChange(value as ThemeType);
+              }}
               options={[
                 {
                   text: i18n('icu:themeSystem'),
@@ -1579,7 +1581,9 @@ export function Preferences({
               <Select
                 ariaLabel={i18n('icu:Preferences--notification-content')}
                 disabled={!hasNotifications}
-                onChange={onNotificationContentChange}
+                onChange={value => {
+                  onNotificationContentChange(value as NotificationSettingType);
+                }}
                 options={[
                   {
                     text: i18n('icu:nameAndMessage'),
@@ -2158,7 +2162,9 @@ export function Preferences({
               )}
             >
               <Select
-                onChange={onSentMediaQualityChange}
+                onChange={value => {
+                  onSentMediaQualityChange(value as SentMediaQualityType);
+                }}
                 options={[
                   {
                     text: i18n('icu:sentMediaQualityStandard'),

ts/types/Storage.d.ts

@@ -8,13 +8,13 @@ export type { StorageAccessType } from './StorageKeys.std.ts';
 export type StorageInterface = {
   onready: (callback: () => void) => void;
 
-  get: <K extends keyof StorageAccessType, V extends StorageAccessType[K]>(
+  get<K extends keyof StorageAccessType, V extends StorageAccessType[K]>(
     key: K
-  ) => V | undefined;
-  get: <K extends keyof StorageAccessType, V extends StorageAccessType[K]>(
+  ): V | undefined;
+  get<K extends keyof StorageAccessType, V extends StorageAccessType[K]>(
     key: K,
     defaultValue: V
-  ) => V;
+  ): V;
 
   put: <K extends keyof StorageAccessType>(
     key: K,

ts/window.d.ts

@@ -256,32 +256,29 @@ declare global {
 
   // oxlint-disable-next-line typescript/consistent-type-definitions
   interface String {
-    split: (splitter: string | RegExp) => [string, ...Array<string>];
-    split: (splitter: string | RegExp, limit: 0) => [];
-    split: (splitter: string | RegExp, limit: 1) => [string];
-    split: (splitter: string | RegExp, limit: 2) => [string, string?];
-    split: (splitter: string | RegExp, limit: 3) => [string, string?, string?];
-    split: (
+    split(splitter: string | RegExp): [string, ...Array<string>];
+    split(splitter: string | RegExp, limit: 0): [];
+    split(splitter: string | RegExp, limit: 1): [string];
+    split(splitter: string | RegExp, limit: 2): [string, string?];
+    split(splitter: string | RegExp, limit: 3): [string, string?, string?];
+    split(
       splitter: string | RegExp,
       limit: 4
-    ) => [string, string?, string?, string?];
-    split: (
+    ): [string, string?, string?, string?];
+    split(
       splitter: string | RegExp,
       limit: 5
-    ) => [string, string?, string?, string?, string?];
-    split: (
-      splitter: string | RegExp,
-      limit: number
-    ) => [string, ...Array<string>];
-    split: (
+    ): [string, string?, string?, string?, string?];
+    split(splitter: string | RegExp, limit: number): [string, ...Array<string>];
+    split(
       splitter: string | RegExp,
       limit?: number
-    ) => [string, ...Array<string>];
-    split: <T>(splitter: StringSplitSplitter<T>, limit?: number) => T;
-    split: (
+    ): [string, ...Array<string>];
+    split<T>(splitter: StringSplitSplitter<T>, limit?: number): T;
+    split(
       splitter: string | RegExp | StringSplitSplitter<Array<string>>,
       limit?: number
-    ) => [string, ...Array<string>];
+    ): [string, ...Array<string>];
   }
 }
 

tsconfig.json

@@ -23,7 +23,7 @@
      */
 
     /* Save .tsbuildinfo files to allow for incremental compilation of projects. */
-    "incremental": true,
+    "incremental": false,
     /* Enable constraints that allow a TypeScript project to be used with project references. */
     "composite": false,
     /* Specify the path to .tsbuildinfo incremental compilation file. */