fix: Address lint and typecheck issues in ScrollTimeline and ViewTimeline implementation

- Fix currentTime return type in MockedAnimationTimeline to allow number | null
- Remove unused parameters and fix any types in test files
- Add disconnect() method to ViewTimeline interface
- Remove unused #cleanup method from ViewTimeline
- Fix lint issues with proper type assertions

Co-authored-by: Ivan Galiatin <[email protected]>

Author: claude[bot]

package-lock.json

@@ -5,7 +5,7 @@ class MockedAnimationTimeline implements AnimationTimeline {
     }
   }
 
-  get currentTime() {
+  get currentTime(): number | null {
     return performance.now();
   }
 }

src/mocks/web-animations-api/AnimationTimeline.ts

@@ -8,7 +8,7 @@ describe('ScrollTimeline Environment Tests', () => {
   afterEach(() => {
     // Clean up global modifications
     if ('ScrollTimeline' in window) {
-      delete (window as any).ScrollTimeline;
+      delete (window as Record<string, unknown>).ScrollTimeline;
     }
   });
 
@@ -48,7 +48,7 @@ describe('ScrollTimeline Environment Tests', () => {
   it('should work in different test environments', () => {
     expect(() => {
       const timeline = new ScrollTimeline();
-      timeline.currentTime; // Should not throw
+      expect(timeline.currentTime).toBeDefined(); // Should not throw
     }).not.toThrow();
   });
 

src/mocks/web-animations-api/ScrollTimeline.env.test.ts

@@ -8,7 +8,7 @@ describe('ScrollTimeline', () => {
   afterEach(() => {
     // Clean up any global modifications
     if ('ScrollTimeline' in window) {
-      delete (window as any).ScrollTimeline;
+      delete (window as Record<string, unknown>).ScrollTimeline;
     }
   });
 
@@ -28,7 +28,7 @@ describe('ScrollTimeline', () => {
   it('should throw error for invalid axis parameter', () => {
     expect(() => {
       new ScrollTimeline({
-        axis: 'invalid' as any
+        axis: 'invalid' as 'block' | 'inline' | 'x' | 'y'
       });
     }).toThrow('Invalid axis value: invalid');
   });

src/mocks/web-animations-api/ScrollTimeline.test.ts

@@ -1,7 +1,7 @@
 import { MockedViewTimeline, mockViewTimeline } from './ViewTimeline';
 
 // Mock IntersectionObserver for testing
-const mockIntersectionObserver = jest.fn().mockImplementation((callback, options) => ({
+const mockIntersectionObserver = jest.fn().mockImplementation(() => ({
   observe: jest.fn(),
   disconnect: jest.fn(),
   unobserve: jest.fn()
@@ -23,7 +23,7 @@ describe('ViewTimeline', () => {
 
     // Clean up any global modifications
     if ('ViewTimeline' in window) {
-      delete (window as any).ViewTimeline;
+      delete (window as Record<string, unknown>).ViewTimeline;
     }
 
     jest.clearAllMocks();
@@ -45,13 +45,13 @@ describe('ViewTimeline', () => {
 
   it('should throw error when subject is missing', () => {
     expect(() => {
-      new ViewTimeline({} as any);
+      new ViewTimeline({} as Parameters<ViewTimelineConstructor>[0]);
     }).toThrow('ViewTimeline requires a valid Element as subject');
   });
 
   it('should throw error when subject is not an Element', () => {
     expect(() => {
-      new ViewTimeline({ subject: 'not an element' as any });
+      new ViewTimeline({ subject: 'not an element' as unknown as Element });
     }).toThrow('ViewTimeline requires a valid Element as subject');
   });
 
@@ -60,7 +60,7 @@ describe('ViewTimeline', () => {
     expect(() => {
       new ViewTimeline({
         subject: mockElement,
-        axis: 'invalid' as any
+        axis: 'invalid' as 'block' | 'inline' | 'x' | 'y'
       });
     }).toThrow('Invalid axis value: invalid');
   });
@@ -131,7 +131,7 @@ describe('ViewTimeline', () => {
 
   it('should work when IntersectionObserver is not available', () => {
     // Remove IntersectionObserver
-    delete (global as any).IntersectionObserver;
+    delete (global as Record<string, unknown>).IntersectionObserver;
 
     const mockElement = document.createElement('div');
     const timeline = new ViewTimeline({ subject: mockElement });

src/mocks/web-animations-api/ViewTimeline.test.ts

@@ -97,10 +97,6 @@ class MockedViewTimeline
       this.#observer = null;
     }
   }
-
-  #cleanup() {
-    this.disconnect();
-  }
 }
 
 function mockViewTimeline() {

src/mocks/web-animations-api/ViewTimeline.ts

@@ -17,6 +17,7 @@ declare global {
   interface ViewTimeline extends AnimationTimeline {
     readonly subject: Element;
     readonly axis: 'block' | 'inline' | 'x' | 'y';
+    disconnect(): void;
   }
 
   interface ViewTimelineConstructor {