add noUncheckedIndexedAccess flag to tsconfig

fekoneko · Dec 4, 2024 · 0e5ea29 · 0e5ea29
1 parent 8fadf2c
commit 0e5ea29
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 10 deletions.
diff --git a/src/components/collection-explorer/work-viewer/ImageWorkViewer.tsx b/src/components/collection-explorer/work-viewer/ImageWorkViewer.tsx
@@ -61,6 +61,7 @@ export const ImageWorkViewer: FC<ImageWorkViewerProps> = ({ work }) => {
   }, [work]);
 
   const asset = work.imageAssets[pageNumber];
+  if (!asset) return null;
 
   return (
     <div

diff --git a/src/components/collection-explorer/works-list/WorksListCardContents.tsx b/src/components/collection-explorer/works-list/WorksListCardContents.tsx
@@ -13,12 +13,14 @@ export const WorksListCardContent: FC<WorksListCardContentProps> = memo(({ work
     {!!work.imageAssets?.length && (
       <div className="relative size-full">
         <div className="relative flex size-full items-center transition-all [clip-path:rect(0_100%_100%_0_round_0.5rem)] hover:z-20 hover:translate-x-1 hover:[clip-path:rect(-100%_300%_300%_-100%_round_0.5rem)]">
-          <ImageView
-            src={convertFileSrc(work.imageAssets[0].path)}
-            width={work.imageAssets[0].dimensions.width}
-            height={work.imageAssets[0].dimensions.height}
-            className="pointer-events-none absolute w-full rounded-lg transition-transform [:hover>&]:scale-[1.2] [:hover>&]:shadow-md"
-          />
+          {work.imageAssets[0] && (
+            <ImageView
+              src={convertFileSrc(work.imageAssets[0].path)}
+              width={work.imageAssets[0].dimensions.width}
+              height={work.imageAssets[0].dimensions.height}
+              className="pointer-events-none absolute w-full rounded-lg transition-transform [:hover>&]:scale-[1.2] [:hover>&]:shadow-md"
+            />
+          )}
         </div>
         <p className="absolute right-0 top-0 -mr-2 -mt-0.5 rounded-lg bg-paper px-2 text-text shadow-md transition-colors [:hover>&]:invisible">
           x{work.imageAssets.length}

diff --git a/src/components/header/CollectionButton.tsx b/src/components/header/CollectionButton.tsx
@@ -10,7 +10,7 @@ export const CollectionButton: FC = () => {
   const recentSelectRef = useRef<HTMLSelectElement>(null);
 
   useEffect(() => {
-    if (!recentPaths || recentPaths.length === 0 || collectionName !== null) return;
+    if (!recentPaths || recentPaths[0] === undefined || collectionName !== null) return;
     switchCollection(recentPaths[0]);
   }, [recentPaths, collectionName, switchCollection]);
 
@@ -51,7 +51,7 @@ export const CollectionButton: FC = () => {
           <select
             ref={recentSelectRef}
             className="mt-0.5 w-[16.5px] cursor-pointer bg-transparent [zoom:1.4] focus:outline-none"
-            onChange={(e) => switchCollection(recentPaths[e.target.selectedIndex])}
+            onChange={(e) => switchCollection(recentPaths[e.target.selectedIndex]!)}
           >
             {recentPaths.map((path) => (
               <option key={path} className="bg-background px-3 text-text [zoom:0.65]">

diff --git a/tsconfig.app.json b/tsconfig.app.json
@@ -21,7 +21,8 @@
     "strict": true,
     "noUnusedLocals": true,
     "noUnusedParameters": true,
-    "noFallthroughCasesInSwitch": true
+    "noFallthroughCasesInSwitch": true,
+    "noUncheckedIndexedAccess": true
   },
   "include": ["src"]
 }
diff --git a/tsconfig.bun.json b/tsconfig.bun.json
@@ -9,7 +9,8 @@
     "moduleResolution": "bundler",
     "allowSyntheticDefaultImports": true,
     "strict": true,
-    "noEmit": true
+    "noEmit": true,
+    "noUncheckedIndexedAccess": true
   },
   "include": ["plugins", "vite.config.ts"]
 }