Started resolving typing issues

Author: nandi95

src/config.ts

@@ -21,15 +21,15 @@ interface Plugin<Instance, O> {
 class Pluggable<Instance = DOMWrapper<Element>> {
   installedPlugins: Plugin<Instance, any>[] = []
 
-  install<O>(handler: (instance: Instance) => Record<string, any>)
+  install<O>(handler: (instance: Instance) => Record<string, any>): void
   install<O>(
     handler: (instance: Instance, options: O) => Record<string, any>,
     options: O
-  )
+  ): void
   install<O>(
     handler: (instance: Instance, options?: O) => Record<string, any>,
     options?: O
-  ) {
+  ): void {
     if (typeof handler !== 'function') {
       console.error('plugin.install must receive a function')
       handler = () => ({})

src/createDomEvent.ts

@@ -1,3 +1,4 @@
+// @ts-expect-error No DefinitelyTyped package exists
 import eventTypes from 'dom-event-types'
 
 interface TriggerOptions {
@@ -9,7 +10,7 @@ interface TriggerOptions {
 
 interface EventParams {
   eventType: string
-  modifiers: string[]
+  modifiers: KeyNameArray
   options?: TriggerOptions
 }
 
@@ -21,8 +22,8 @@ const mouseKeyModifiers = ['left', 'middle', 'right']
 /**
  * Groups modifiers into lists
  */
-function generateModifiers(modifiers: string[], isOnClick: boolean) {
-  const keyModifiers: string[] = []
+function generateModifiers(modifiers: KeyNameArray, isOnClick: boolean) {
+  const keyModifiers: KeyNameArray = []
   const systemModifiers: string[] = []
 
   for (let i = 0; i < modifiers.length; i++) {
@@ -50,6 +51,7 @@ function generateModifiers(modifiers: string[], isOnClick: boolean) {
   }
 }
 
+export type KeyNameArray = Array<keyof typeof keyCodesByKeyName>
 export const keyCodesByKeyName = {
   backspace: 8,
   tab: 9,
@@ -98,10 +100,13 @@ function getEventProperties(eventParams: EventParams) {
 
   // convert `shift, ctrl` to `shiftKey, ctrlKey`
   // allows trigger('keydown.shift.ctrl.n') directly
-  const systemModifiersMeta = systemModifiers.reduce((all, key) => {
-    all[`${key}Key`] = true
-    return all
-  }, {})
+  const systemModifiersMeta = systemModifiers.reduce(
+    (all: Record<string, boolean>, key) => {
+      all[`${key}Key`] = true
+      return all
+    },
+    {}
+  )
 
   // get the keyCode for backwards compat
   const keyCode =
@@ -148,8 +153,12 @@ function createDOMEvent(eventString: String, options?: TriggerOptions) {
   // split eventString like `keydown.ctrl.shift.c` into `keydown` and array of modifiers
   const [eventType, ...modifiers] = eventString.split('.')
 
-  const eventParams: EventParams = { eventType, modifiers, options }
-  const event: Event = createEvent(eventParams)
+  const eventParams: EventParams = {
+    eventType,
+    modifiers: modifiers as KeyNameArray,
+    options
+  }
+  const event: Event & TriggerOptions = createEvent(eventParams)
   const eventPrototype = Object.getPrototypeOf(event)
 
   // attach custom options to the event, like `relatedTarget` and so on.

src/domWrapper.ts

@@ -158,10 +158,10 @@ export class DOMWrapper<ElementType extends Element> {
     }
 
     element.selected = true
-    let parentElement = element.parentElement
+    let parentElement = element.parentElement!
 
     if (parentElement.tagName === 'OPTGROUP') {
-      parentElement = parentElement.parentElement
+      parentElement = parentElement.parentElement!
     }
 
     return new DOMWrapper(parentElement).trigger('change')

src/stubs.ts

@@ -17,7 +17,7 @@ import { matchName } from './utils/matchName'
 import { ComponentInternalInstance } from '@vue/runtime-core'
 
 interface StubOptions {
-  name?: string
+  name: string
   props?: any
   propsDeclaration?: any
 }

src/types.ts

@@ -141,3 +141,5 @@ export type GlobalMountOptions = {
    */
   renderStubDefaultSlot?: boolean
 }
+
+export type VueElement = Element & { __vue_app__?: any }

src/utils.ts

@@ -42,8 +42,8 @@ export const mergeDeep = (
   target: Record<string, any>,
   source: Record<string, any>
 ) => {
-  const isObject = (obj: unknown): obj is Object =>
-    obj && typeof obj === 'object'
+  const isObject = (obj: unknown): obj is Record<string, any> =>
+    !!obj && typeof obj === 'object'
 
   if (!isObject(target) || !isObject(source)) {
     return source
@@ -101,6 +101,6 @@ export function textContent(element: Element): string {
   // we check if the element is a comment first
   // to return an empty string in that case, instead of the comment content
   return element.nodeType !== Node.COMMENT_NODE
-    ? element.textContent?.trim()
+    ? element.textContent?.trim() ?? ''
     : ''
 }

src/utils/compileSlots.ts

@@ -1,5 +1,6 @@
 import { compile } from '@vue/compiler-dom'
 import * as vue from 'vue'
+import type { SetupContext } from 'vue'
 
 export function processSlot(source = '', Vue = vue) {
   let template = source.trim()
@@ -25,14 +26,14 @@ export function processSlot(source = '', Vue = vue) {
     components: {
       SlotWrapper: {
         inheritAttrs: false,
-        setup(_, { slots, attrs }) {
+        setup(_: Record<string, any>, ctx: SetupContext) {
           return () => {
-            const names = Object.keys(slots)
+            const names = Object.keys(ctx.slots)
             if (names.length === 0) {
               return []
             } else {
               const slotName = names[0]
-              return slots[slotName](attrs)
+              return ctx.slots[slotName]!(ctx.attrs)
             }
           }
         }

src/utils/find.ts

@@ -35,7 +35,7 @@ export function matches(
       return true
     }
 
-    let componentName
+    let componentName: string | undefined
     if ('name' in nodeType || 'displayName' in nodeType) {
       // match normal component definitions or functional components
       componentName = nodeType.name || nodeType.displayName

src/utils/isElementVisible.ts

@@ -26,7 +26,7 @@ function isAttributeVisible<T extends Element>(element: T) {
   )
 }
 
-export function isElementVisible<T extends Element>(element: T) {
+export function isElementVisible<T extends Element>(element: T): boolean {
   return (
     element.nodeName !== '#comment' &&
     isStyleVisible(element) &&

src/utils/matchName.ts

@@ -1,11 +1,11 @@
 import { camelize, capitalize } from './vueShared'
 
-export function matchName(target: string, sourceName: string) {
+export function matchName(target: string, sourceName: string): boolean {
   const camelized = camelize(target)
   const capitalized = capitalize(camelized)
 
   return (
-    sourceName &&
+    !!sourceName &&
     (sourceName === target ||
       sourceName === camelized ||
       sourceName === capitalized ||