Merge pull request #639 from 3flex/ts-eslint-strict

Enable typescript-eslint strict checks

eslint.config.mjs

@@ -11,7 +11,7 @@ export default [
     eslint.configs.recommended,
     jsdoc.configs['flat/recommended'],
     eslintPluginPrettierRecommended,
-    ...tseslint.configs.recommended,
+    ...tseslint.configs.strict,
     promise.configs['flat/recommended'],
     importPlugin.flatConfigs.errors,
     importPlugin.flatConfigs.warnings,

src/components/commandHandler.ts

@@ -21,6 +21,7 @@ import { reportPlaybackProgress } from './jellyfinActions';
 import { PlaybackManager } from './playbackManager';
 import { DocumentManager } from './documentManager';
 
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export abstract class CommandHandler {
     private static playerManager: framework.PlayerManager;
     private static supportedCommands: SupportedCommands = {

src/components/credentialManager.ts

@@ -74,6 +74,7 @@ export class credentialManager {
      */
     remove(serverId: string): boolean {
         if (serverId in this.credentialStore) {
+            // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
             delete this.credentialStore[serverId];
 
             return true;

src/components/documentManager.ts

@@ -4,6 +4,7 @@ import { parseISO8601Date, TicksPerSecond, ticksToSeconds } from '../helpers';
 import { JellyfinApi } from './jellyfinApi';
 import { deviceIds, getActiveDeviceId } from './castDevices';
 
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export abstract class DocumentManager {
     // Duration between each backdrop switch in ms
     private static backdropPeriodMs: number | null = 30000;

src/components/jellyfinApi.ts

@@ -1,6 +1,7 @@
 import { version as packageVersion } from '../../package.json';
 import { ajax } from './fetchhelper';
 
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export abstract class JellyfinApi {
     // userId that we are connecting as currently
     public static userId: string | undefined;

src/components/playbackManager.ts

@@ -51,6 +51,7 @@ export interface PlaybackState {
     runtimeTicks: number;
 }
 
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export abstract class PlaybackManager {
     private static playerManager: framework.PlayerManager;
     private static activePlaylist: Array<BaseItemDto>;