Enable noPropertyAccessFromIndexSignature and noUncheckedIndexedAccess

package.json

@@ -54,6 +54,7 @@
   "scripts": {
     "lint": "eslint . --ext .ts,.js",
     "test": "jest",
-    "build": "node tools/build.js"
+    "build": "node tools/build.js",
+    "tsc": "tsc --noEmit"
   }
 }

src/UniversalRouter.test.ts

@@ -514,7 +514,7 @@ test('respects baseUrl', async () => {
   expect(action.mock.calls[0][0]).toHaveProperty('pathname', '/base/a/b/c')
   expect(action.mock.calls[0][0]).toHaveProperty('path', '/c')
   expect(action.mock.calls[0][0]).toHaveProperty('baseUrl', '/base/a/b')
-  expect(action.mock.calls[0][0]).toHaveProperty('route', routes.children[0].children[0])
+  expect(action.mock.calls[0][0]).toHaveProperty('route', routes.children[0]?.children[0])
   expect(action.mock.calls[0][0]).toHaveProperty('router', router)
 
   let err

src/UniversalRouter.ts

@@ -190,7 +190,7 @@
       if (matchResult && route.children) {
         while (childIndex < route.children.length) {
           if (!childMatches) {
-            const childRoute = route.children[childIndex]
+            const childRoute = route.children[childIndex]!
             childRoute.parent = route
 
             childMatches = matchRoute<R, C>(
@@ -320,7 +320,7 @@
       })
     }
 
-    context.next = next
+    context['next'] = next
 
     return Promise.resolve()
       .then(() => next(true, this.root))

src/UniversalRouterSync.test.ts

@@ -511,7 +511,7 @@ test('respects baseUrl', () => {
   expect(action.mock.calls[0][0]).toHaveProperty('pathname', '/base/a/b/c')
   expect(action.mock.calls[0][0]).toHaveProperty('path', '/c')
   expect(action.mock.calls[0][0]).toHaveProperty('baseUrl', '/base/a/b')
-  expect(action.mock.calls[0][0]).toHaveProperty('route', routes.children[0].children[0])
+  expect(action.mock.calls[0][0]).toHaveProperty('route', routes.children[0]?.children[0])
   expect(action.mock.calls[0][0]).toHaveProperty('router', router)
 
   let err

src/UniversalRouterSync.ts

@@ -191,7 +191,7 @@
       if (matchResult && route.children) {
         while (childIndex < route.children.length) {
           if (!childMatches) {
-            const childRoute = route.children[childIndex]
+            const childRoute = route.children[childIndex]!
             childRoute.parent = route
 
             childMatches = matchRoute<R, C>(
@@ -318,7 +318,7 @@
       return next(resume, parent, result)
     }
 
-    context.next = next
+    context['next'] = next
 
     try {
       return next(true, this.root)

src/generateUrls.ts

@@ -57,7 +57,7 @@
 
   if (routes) {
     for (let i = 0; i < routes.length; i++) {
-      const childRoute = routes[i]
+      const childRoute = routes[i]!
       const childName = childRoute.name
       childRoute.parent = route
       cacheRoutes(
@@ -117,7 +117,7 @@
       const keys: Keys = Object.create(null)
       for (let i = 0; i < tokens.length; i++) {
         const token = tokens[i]
-        if (typeof token !== 'string') {
+        if (token && typeof token !== 'string') {
           keys[token.name] = true
         }
       }
@@ -132,8 +132,8 @@
       const keys = Object.keys(params)
       for (let i = 0; i < keys.length; i++) {
         const key = keys[i]
-        if (!regexp.keys[key]) {
-          queryParams[key] = params[key]
+        if (key && !regexp.keys[key]) {
+          queryParams[key] = params[key] ?? ''
         }
       }
       const query = opts.stringifyQueryParams(queryParams)

tsconfig.json

@@ -8,6 +8,8 @@
     "strict": true,
     "declaration": true,
     "noEmitOnError": true,
+    "noPropertyAccessFromIndexSignature": true,
+    "noUncheckedIndexedAccess": true,
     "noUnusedLocals": true,
     "noUnusedParameters": true,
     "sourceMap": true,