fix: lint errors + remove unnecessary document page

kevinjosethomas · kevinjosethomas · commit 59074c91d2eb · 2025-02-04T21:03:41.000-08:00
diff --git a/.eslintrc b/.eslintrc
@@ -26,6 +26,8 @@
     "@next/next/no-duplicate-head": "off",
     "@next/next/no-typos": "off",
     "@next/next/no-before-interactive-script-outside-document": "off",
+    "@next/next/no-styled-jsx-in-document": "off",
+    "@next/next/no-head-import-in-document": "off",
   },
   "overrides": [
     {
diff --git a/src/api/analysis/analysis.ts b/src/api/analysis/analysis.ts
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import {
   Player,
   MoveMap,
diff --git a/src/components/Analysis/AnalysisGameList.tsx b/src/components/Analysis/AnalysisGameList.tsx
@@ -51,15 +51,17 @@ export const AnalysisGameList: React.FC<AnalysisGameListProps> = ({
 
   const listKeys = useMemo(() => {
     return analysisTournamentList
-      ? Array.from(analysisTournamentList.keys()).sort((a, b) =>
-          b.split('---')[1].localeCompare(a.split('---')[1]),
+      ? Array.from(analysisTournamentList.keys()).sort(
+          (a, b) =>
+            b?.split('---')?.[1]?.localeCompare(a?.split('---')?.[1] ?? '') ??
+            0,
         )
       : []
   }, [analysisTournamentList])
 
   const initialOpenIndex = useMemo(() => {
     if (analysisTournamentList && currentId) {
-      return listKeys.map((m) => m.split('---')[0]).indexOf(currentId[0])
+      return listKeys.map((m) => m?.split('---')?.[0]).indexOf(currentId[0])
     } else {
       return null
     }
diff --git a/src/components/Misc/ModalContainer.tsx b/src/components/Misc/ModalContainer.tsx
@@ -14,7 +14,7 @@ export const ModalContainer: React.FC<Props> = ({
   return (
     // Modal always has dismiss button so not required for a11y
     // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
-    (<motion.div
+    <motion.div
       initial={{ opacity: 0 }}
       animate={{ opacity: 1 }}
       exit={{ opacity: 0 }}
@@ -33,6 +33,6 @@ export const ModalContainer: React.FC<Props> = ({
       >
         {children}
       </motion.div>
-    </motion.div>)
-  );
+    </motion.div>
+  )
 }
diff --git a/src/components/Openings/PlayOpening.tsx b/src/components/Openings/PlayOpening.tsx
@@ -207,24 +207,24 @@ function Switcher({
   setSelected: (selected: { id: string; name: string }) => void
 }) {
   return (
-    (<div className="flex w-full flex-col gap-0.5 px-4">
+    <div className="flex w-full flex-col gap-0.5 px-4">
       <p className="text-sm text-secondary">{label}</p>
       <div className="flex w-full flex-row overflow-hidden rounded-sm bg-background-2/80">
         {options.map((option, index) => (
           // eslint-disable-next-line jsx-a11y/click-events-have-key-events
-          (<div
+          <div
             key={index}
             role="button"
             tabIndex={0}
             onClick={() => setSelected(option)}
             className={`flex flex-1 cursor-pointer items-center justify-center px-2 py-1.5 ${selected.id === option.id ? 'bg-human-4/60' : 'hover:bg-human-4/10'}`}
           >
             <p className="select-none text-sm">{option.name}</p>
-          </div>)
+          </div>
         ))}
       </div>
-    </div>)
-  );
+    </div>
+  )
 }
 
 function ColorPicker({
diff --git a/src/pages/_app.tsx b/src/pages/_app.tsx
@@ -39,6 +39,10 @@ function MaiaPlatform({ Component, pageProps }: AppProps) {
     >
       <Head>
         <link rel="icon" type="image/png" href="/favicon.png" />
+        <link
+          rel="stylesheet"
+          href="https://fonts.googleapis.com/css2?family=Material+Symbols+Outlined&display=swap"
+        />
       </Head>
       <div className={`${OpenSans.className} app-container`}>
         <Header />
diff --git a/src/pages/_document.tsx b/src/pages/_document.tsx
diff --git a/src/pages/analysis/[...id].tsx b/src/pages/analysis/[...id].tsx
@@ -453,7 +453,7 @@ const Analysis: React.FC<Props> = ({
                 ) : screen.id === 'export' ? (
                   <div className="flex flex-col p-4">
                     <ExportGame
-                      game={analyzedGame as any as PlayedGame}
+                      game={analyzedGame as unknown as PlayedGame}
                       whitePlayer={analyzedGame.whitePlayer.name}
                       blackPlayer={analyzedGame.blackPlayer.name}
                       event="Analysis"
diff --git a/src/pages/leaderboard.tsx b/src/pages/leaderboard.tsx
@@ -1,6 +1,5 @@
-import React from 'react'
 import Head from 'next/head'
-import { useCallback, useEffect, useState } from 'react'
+import React, { useCallback, useEffect, useState } from 'react'
 
 import {
   BrainIcon,

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@`
`26`	`26`	`"@next/next/no-duplicate-head": "off",`
`27`	`27`	`"@next/next/no-typos": "off",`
`28`	`28`	`"@next/next/no-before-interactive-script-outside-document": "off",`
	`29`	`+ "@next/next/no-styled-jsx-in-document": "off",`
	`30`	`+ "@next/next/no-head-import-in-document": "off",`
`29`	`31`	`},`
`30`	`32`	`"overrides": [`
`31`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+/* eslint-disable @typescript-eslint/no-explicit-any */`
`1`	`2`	`import {`
`2`	`3`	`Player,`
`3`	`4`	`MoveMap,`