Fix TypeScript errors and warnings

Author: webstackdev

package-lock.json

@@ -173,6 +173,7 @@
     "tslib": "2.8.1",
     "typescript": "5.9.3",
     "typescript-eslint": "8.46.1",
+    "unist-util-inspect": "^8.1.0",
     "unist-util-visit": "^5.0.0",
     "vitest": "3.2.4",
     "vitest-axe": "0.1.0"

package.json

@@ -25,7 +25,7 @@ import { writeFile } from 'fs/promises'
 type IconGenerator = (options: Metadata) => Promise<Buffer>
 
 const faviconPath = `src/assets/favicon.svg`
-const generateIcoFavicon: IconGenerator = ({ width, height, density }) => {
+const generateIcoFavicon: IconGenerator = async ({ width, height, density }) => {
   if (!width || !height || !density)
     throw new Error(`Required option not passed to generateIcoFavicon`)
   const faviconDimensions = [32, 64]

scripts/build/favicon.ts

@@ -149,7 +149,7 @@ describe('NewsletterForm class works', () => {
     elements.emailInput.value = 'test@example.com'
     elements.consentCheckbox.checked = true
     const submitEvent = new Event('submit')
-    await elements.form.dispatchEvent(submitEvent)
+    elements.form.dispatchEvent(submitEvent)
 
     // Wait for async operations
     await vi.runAllTimersAsync()
@@ -180,7 +180,7 @@ describe('NewsletterForm class works', () => {
     elements.emailInput.value = 'test@example.com'
     elements.consentCheckbox.checked = true
     const submitEvent = new Event('submit')
-    await elements.form.dispatchEvent(submitEvent)
+    elements.form.dispatchEvent(submitEvent)
 
     // Wait for async operations
     await vi.runAllTimersAsync()
@@ -202,7 +202,7 @@ describe('NewsletterForm class works', () => {
     elements.emailInput.value = 'test@example.com'
     elements.consentCheckbox.checked = true
     const submitEvent = new Event('submit')
-    await elements.form.dispatchEvent(submitEvent)
+    elements.form.dispatchEvent(submitEvent)
 
     // Wait for async operations
     await vi.runAllTimersAsync()

src/components/CallToAction/Newsletter/__tests__/client.spec.ts

@@ -30,14 +30,6 @@ Object.defineProperty(document, 'cookie', {
 })
 
 // Mock DOM elements
-const mockElement = (id: string, checked = false) =>
-  ({
-    id,
-    checked,
-    addEventListener: vi.fn(),
-    style: {},
-  }) as unknown as HTMLInputElement
-
 const mockButton = (id: string): HTMLButtonElement => {
   const button = document.createElement('button')
   button.id = id

src/components/Cookies/Customize/__tests__/client.spec.ts

@@ -120,9 +120,9 @@ const purposeList = purposes.map(p => purposeDescriptions[p] || p)
       if (!checkbox) return
 
       const purposes = JSON.parse(
-        container.dataset.purposes || '[]'
+        container.dataset['purposes'] || '[]'
       ) as ConsentPurpose[]
-      const formId = container.dataset.formId
+      const formId = container.dataset['formId']
 
       initGDPRConsent(checkbox.id, purposes, formId)
     })

src/components/GDPR/Consent/index.astro

@@ -8,5 +8,5 @@ export interface ImageProps {
   image?: {
     src: ImageMetadata
     alt: string
-  }
+  } | undefined
 }

src/components/Hero/@types/image.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest'
+import { describe, it, expect, beforeEach, vi, afterEach, type Mock } from 'vitest'
 import { JSDOM } from 'jsdom'
 import { readFileSync } from 'fs'
 import { resolve } from 'path'
@@ -82,7 +82,7 @@ describe('HeroLoader', () => {
     })
 
     it('should register with loader', () => {
-      ;(registerScript as vi.Mock).mockClear()
+      ;(registerScript as Mock).mockClear()
       registerScript(HeroLoader)
       expect(registerScript).toHaveBeenCalledWith(HeroLoader)
     })

src/components/Hero/__tests__/client.spec.ts

@@ -138,13 +138,13 @@ export class HeroLoader extends LoadableScript {
         },
         '-=1'
       )
-      .staggerFrom(
+      .from(
         '.monitorContentGroup path',
-        1,
         {
+          duration: 1,
           scaleX: 0,
-        },
-        0.1
+          stagger: 0.1,
+        }
       )
       .from('.monitorLogo', {
         scale: 0,
@@ -248,13 +248,13 @@ export class HeroLoader extends LoadableScript {
         },
         '-=2'
       )
-      .staggerFrom(
+      .from(
         '.laptopContentGroup path',
-        1,
         {
+          duration: 1,
           scaleX: 0,
-        },
-        0.1
+          stagger: 0.1,
+        }
       )
       .to('.laptopTrackpad', {
         duration: 0.3,
@@ -322,12 +322,13 @@ export class HeroLoader extends LoadableScript {
         delay: 2,
         ease: Anticipate.out,
       })
-      .staggerFrom(
+      .from(
         ['.tabletButton', '.tabletCamera'],
-        0.5,
         {
+          duration: 0.5,
           scale: 0,
           ease: 'back',
+          stagger: 0,
         },
         '-=1'
       )
@@ -375,14 +376,14 @@ export class HeroLoader extends LoadableScript {
       .set(['.tabletGroup', '.tabletContentGroup'], {
         alpha: 0,
       })
-      .staggerFrom(
+      .from(
         ['.phoneButton', '.phoneCamera', '.phoneSpeaker'],
-        1,
         {
+          duration: 1,
           scale: 0,
           ease: 'back',
-        },
-        0.1
+          stagger: 0.1,
+        }
       )
       .to('.phoneGroup', {
         duration: 2,

src/components/Hero/client.ts

@@ -12,7 +12,7 @@ import heroSvg from '@assets/images/site/hero.svg?raw'
   </div>
 </div>
 
-<script type="module">
+<script>
   import { registerScript } from '../../Scripts/loader'
   import { HeroLoader } from '../client'