fix: ESLint warnings for @typescript-eslint/no-explicit-any (#296)

* Initial plan

* Initial analysis: ESLint warnings identified

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

* Fix all ESLint warnings

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

* Use ReturnType for setInterval return type

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

* Fix bug in isArrayOptions type guard

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

* Revert package-lock.json and yarn.lock changes

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

* Remove package-lock.json from .gitignore

Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: limonte <6059356+limonte@users.noreply.github.com>

Author: Copilot

projects/ngx-sweetalert2-demo/src/app/app.component.ts

@@ -17,7 +17,7 @@ export class AppComponent {
 
     private readonly dynamicTextChunks = "This dynamic content is controlled by Angular".split(" ");
 
-    private dynamicTextChunksIntervalHandle?: any;
+    private dynamicTextChunksIntervalHandle?: ReturnType<typeof setInterval>;
 
     private currentTextChunkOffset = 0;
 

projects/ngx-sweetalert2-demo/src/test.ts

@@ -4,7 +4,7 @@ import "zone.js/testing";
 import { getTestBed } from "@angular/core/testing";
 import { BrowserDynamicTestingModule, platformBrowserDynamicTesting } from "@angular/platform-browser-dynamic/testing";
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any -- Ambient declaration for require
 declare const require: any;
 
 //=> First, initialize the Angular testing environment

projects/ngx-sweetalert2/src/lib/swal-portal.component.ts

@@ -15,5 +15,5 @@ import { CommonModule } from "@angular/common";
 })
 export class SwalPortalComponent {
     @Input()
-    public template: TemplateRef<any> | null = null;
+    public template: TemplateRef<unknown> | null = null;
 }

projects/ngx-sweetalert2/src/lib/swal-portal.directive.ts

@@ -44,7 +44,7 @@ export class SwalPortalDirective implements OnInit, OnDestroy {
     private readonly resolver = inject(ComponentFactoryResolver);
     private readonly injector = inject(Injector);
     private readonly app = inject(ApplicationRef);
-    private readonly templateRef = inject<TemplateRef<any>>(TemplateRef);
+    private readonly templateRef = inject<TemplateRef<unknown>>(TemplateRef);
     private readonly sweetAlert2Loader = inject(SweetAlert2LoaderService);
     private readonly swalTargets = inject(SwalPortalTargets);
     private readonly swalComponent = inject(SwalComponent, { host: true });

projects/ngx-sweetalert2/src/lib/swal.component.ts

@@ -243,6 +243,7 @@ export class SwalComponent implements OnInit, AfterViewInit, OnChanges, OnDestro
      *     }
      */
     @Output()
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- EventEmitter can emit various types depending on input/preConfirm configuration
     public readonly confirm = new EventEmitter<any>();
 
     /**
@@ -403,6 +404,7 @@ export class SwalComponent implements OnInit, AfterViewInit, OnChanges, OnDestro
 
         return result;
 
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic hook function that accepts any function signature
         function composeHook<T extends (...args: any[]) => void>(
             userHook: T | undefined,
             libHook: T,

projects/ngx-sweetalert2/src/lib/swal.directive.ts

@@ -55,8 +55,9 @@ export class SwalDirective implements OnInit, OnDestroy {
             this.swalOptions = options;
         }
 
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Type guard function needs to accept any type to check if it's an array
         function isArrayOptions(value: any): value is SweetAlertArrayOptions {
-            return Array.isArray(options);
+            return Array.isArray(value);
         }
     }
 
@@ -75,6 +76,7 @@ export class SwalDirective implements OnInit, OnDestroy {
      *     }
      */
     @Output()
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- EventEmitter can emit various types depending on input/preConfirm configuration
     public readonly confirm = new EventEmitter<any>();
 
     /**

projects/ngx-sweetalert2/src/lib/sweetalert2-loader.service.ts

@@ -30,6 +30,7 @@ export class SweetAlert2LoaderService {
         this.swalPromiseCache = libPromise.then((value) => (isDefaultExport(value) ? value : value.default));
 
         function isLoader(value: SwalProvider): value is SwalModuleLoader {
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Checking for version property that doesn't exist in the type definition
             return typeof value === "function" && (value as any).version === undefined;
         }
 

projects/ngx-sweetalert2/src/test.ts

@@ -5,7 +5,7 @@ import { getTestBed } from "@angular/core/testing";
 import { platformBrowserDynamicTesting } from "@angular/platform-browser-dynamic/testing";
 import { BrowserDynamicTestingModule } from "@angular/platform-browser-dynamic/testing";
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any -- Ambient declaration for require
 declare const require: any;
 
 //=> First, initialize the Angular testing environment