test: enable ts strict mode

Author: danielroe

src/transform.ts

@@ -28,6 +28,8 @@ export const MagicRegExpTransformPlugin = createUnplugin(() => {
       if (pathname.match(/\.((c|m)?j|t)sx?$/g)) {
         return true
       }
+
+      return false
     },
     transform(code, id) {
       if (!code.includes('magic-regexp')) return
@@ -36,7 +38,7 @@ export const MagicRegExpTransformPlugin = createUnplugin(() => {
       if (!statements.length) return
 
       const contextMap: Context = { ...magicRegExp }
-      const wrapperNames = []
+      const wrapperNames: string[] = []
       let namespace: string
 
       for (const i of statements.flatMap(i => parseStaticImport(i))) {
@@ -46,7 +48,7 @@ export const MagicRegExpTransformPlugin = createUnplugin(() => {
         }
         if (i.namedImports) {
           for (const key in i.namedImports) {
-            contextMap[i.namedImports[key]] = magicRegExp[key]
+            contextMap[i.namedImports[key]] = magicRegExp[key as keyof typeof magicRegExp]
           }
           if (i.namedImports.createRegExp) {
             wrapperNames.push(i.namedImports.createRegExp)
@@ -59,8 +61,9 @@ export const MagicRegExpTransformPlugin = createUnplugin(() => {
       const s = new MagicString(code)
 
       walk(this.parse(code), {
-        enter(node: SimpleCallExpression) {
-          if (node.type !== 'CallExpression') return
+        enter(_node) {
+          if (_node.type !== 'CallExpression') return
+          const node = _node as SimpleCallExpression
           if (
             // Normal call
             !wrapperNames.includes((node.callee as any).name) &&

test/index.test.ts

@@ -7,7 +7,7 @@ import { createInput } from '../src/core/internal'
 describe('magic-regexp', () => {
   it('works as a normal regexp', () => {
     const regExp = createRegExp('in', [global])
-    expect('thing'.match(regExp)[0]).toMatchInlineSnapshot('"in"')
+    expect('thing'.match(regExp)?.[0]).toMatchInlineSnapshot('"in"')
     expect(regExp.test('thing')).toBeTruthy()
     expect(regExp.lastIndex).toMatchInlineSnapshot('4')
   })
@@ -27,13 +27,13 @@ describe('inputs', () => {
   it('before', () => {
     const regExp = createRegExp(char.before('foo'))
     expect(regExp).toMatchInlineSnapshot('/\\.\\(\\?=foo\\)/')
-    expect('bafoo'.match(regExp)[0]).toMatchInlineSnapshot('"a"')
+    expect('bafoo'.match(regExp)?.[0]).toMatchInlineSnapshot('"a"')
     expect(regExp.test('foo')).toBeFalsy()
   })
   it('after', () => {
     const regExp = createRegExp(char.after('foo'))
     expect(regExp).toMatchInlineSnapshot('/\\(\\?<=foo\\)\\./')
-    expect('fooafoo'.match(regExp)[0]).toMatchInlineSnapshot('"a"')
+    expect('fooafoo'.match(regExp)?.[0]).toMatchInlineSnapshot('"a"')
     expect(regExp.test('foo')).toBeFalsy()
   })
   it('notBefore', () => {
@@ -59,21 +59,21 @@ describe('inputs', () => {
   it('capture groups', () => {
     const pattern = anyOf(anyOf('foo', 'bar').as('test'), exactly('baz').as('test2'))
 
-    expect('football'.match(createRegExp(pattern)).groups).toMatchInlineSnapshot(`
+    expect('football'.match(createRegExp(pattern))?.groups).toMatchInlineSnapshot(`
       {
         "test": "foo",
         "test2": undefined,
       }
     `)
-    expect('fobazzer'.match(createRegExp(pattern)).groups).toMatchInlineSnapshot(`
+    expect('fobazzer'.match(createRegExp(pattern))?.groups).toMatchInlineSnapshot(`
       {
         "test": undefined,
         "test2": "baz",
       }
     `)
-    expectTypeOf<Record<'test' | 'test2', string | undefined> | undefined>(
-      'fobazzer'.match(createRegExp(pattern))?.groups
-    )
+    expectTypeOf('fobazzer'.match(createRegExp(pattern))?.groups).toMatchTypeOf<
+      Record<'test' | 'test2', string | undefined> | undefined
+    >()
     // @ts-expect-error
     'fobazzer'.match(createRegExp(pattern))?.groups.other
 
@@ -84,7 +84,7 @@ describe('inputs', () => {
           "test2": "baz",
         }
       `)
-      expectTypeOf<Record<'test' | 'test2', string | undefined>>(match.groups)
+      expectTypeOf(match.groups).toMatchTypeOf<Record<'test' | 'test2', string | undefined>>()
     }
   })
 })

test/transform.test.ts

@@ -78,7 +78,7 @@ describe('transformer', () => {
 })
 
 const transform = (code: string | string[], id = 'some-id.js') => {
-  const plugin = MagicRegExpTransformPlugin.vite()
+  const plugin = MagicRegExpTransformPlugin.vite() as any
   return plugin.transform.call(
     { parse: (code: string) => parse(code, { ecmaVersion: 2022, sourceType: 'module' }) },
     Array.isArray(code) ? code.join('\n') : code,

tsconfig.json

@@ -4,6 +4,7 @@
     "module": "ESNext",
     "moduleResolution": "Node",
     "esModuleInterop": true,
+    "strict": true,
     "skipLibCheck": true,
     "paths": {
       "magic-regexp/transform": ["./src/transform"],