chore: noUncheckedIndexAccess

Author: tmm

src/dev/devtools.tsx

@@ -160,7 +160,8 @@ export function routes(
         }
 
         try {
-          initialData = (await getInitialData(frameUrl)) as Bootstrap['data']
+          if (frameUrl)
+            initialData = (await getInitialData(frameUrl)) as Bootstrap['data']
         } catch (error) {
           if (error instanceof HTTPException) throw error
         }

src/utils/getImagePaths.ts

@@ -4,7 +4,7 @@ export function getImagePaths(path: string) {
 
   // skip the first split result as it's empty
   for (let i = 1; i < pathParts.length; i++) {
-    if (!pathParts[i].startsWith(':') || !pathParts[i].endsWith('?')) continue
+    if (!pathParts[i]?.startsWith(':') || !pathParts[i]?.endsWith('?')) continue
     imagePaths.push(`${pathParts.slice(0, i).join('/')}/image`)
   }
   imagePaths.push(`${path}/image`)

src/utils/parsePath.ts

@@ -1,5 +1,5 @@
-export function parsePath(path_: string) {
-  let path = path_.split('?')[0]
+export function parsePath(path_: string): string {
+  let path = path_.split('?')[0]!
   if (path.endsWith('/')) path = path.slice(0, -1)
   return path
 }

tsconfig.base.json

@@ -11,6 +11,7 @@
     "useDefineForClassFields": true, // Not enabled by default in `strict` mode unless we bump `target` to ES2022.
     "noFallthroughCasesInSwitch": true, // Not enabled by default in `strict` mode.
     "noImplicitReturns": true, // Not enabled by default in `strict` mode.
+    "noUncheckedIndexedAccess": true,
     "useUnknownInCatchVariables": true, // TODO: This would normally be enabled in `strict` mode but would require some adjustments to the codebase.
     "noImplicitOverride": true, // Not enabled by default in `strict` mode.
     "noUnusedLocals": true, // Not enabled by default in `strict` mode.