From 34a1715923d4c8d908efdf44390620f583d87486 Mon Sep 17 00:00:00 2001
From: Keith Stevens <fozziethebeat@gmail.com>
Date: Tue, 10 Jan 2023 13:02:17 +0900
Subject: [PATCH] Cleaning up a suite of eslint warnings

---
 website/src/components/Buttons/Skip.tsx       | 5 -----
 website/src/components/CollapsableText.tsx    | 2 +-
 website/src/components/FlaggableElement.tsx   | 4 ++--
 website/src/components/Messages.tsx           | 8 ++++++--
 website/src/components/Tasks/CreateTask.tsx   | 3 +--
 website/src/components/Tasks/EvaluateTask.tsx | 2 +-
 website/src/components/Tasks/Task.tsx         | 8 ++++----
 website/src/lib/oasst_api_client.ts           | 4 ++--
 website/src/pages/api/set_label.ts            | 3 +--
 website/src/pages/auth/verify.tsx             | 1 -
 website/src/pages/create/summarize_story.tsx  | 2 +-
 website/src/pages/evaluate/rate_summary.tsx   | 2 +-
 12 files changed, 20 insertions(+), 24 deletions(-)

diff --git a/website/src/components/Buttons/Skip.tsx b/website/src/components/Buttons/Skip.tsx
index 8440e348..bcd0eb79 100644
--- a/website/src/components/Buttons/Skip.tsx
+++ b/website/src/components/Buttons/Skip.tsx
@@ -1,10 +1,6 @@
 import {
   Button,
   ButtonProps,
-  Menu,
-  MenuButton,
-  MenuItem,
-  MenuList,
   Modal,
   ModalBody,
   ModalCloseButton,
@@ -16,7 +12,6 @@ import {
   useDisclosure,
 } from "@chakra-ui/react";
 import { useState } from "react";
-import { FaChevronDown } from "react-icons/fa";
 
 interface SkipButtonProps extends ButtonProps {
   onSkip: (reason: string) => void;
diff --git a/website/src/components/CollapsableText.tsx b/website/src/components/CollapsableText.tsx
index 1f34c508..5dcab595 100644
--- a/website/src/components/CollapsableText.tsx
+++ b/website/src/components/CollapsableText.tsx
@@ -12,7 +12,7 @@ import React from "react";
 
 export const CollapsableText = ({ text, maxLength = 220 }) => {
   const { isOpen, onOpen, onClose } = useDisclosure();
-  if (typeof text != "string" || text.length <= maxLength) {
+  if (typeof text !== "string" || text.length <= maxLength) {
     return text;
   } else {
     return (
diff --git a/website/src/components/FlaggableElement.tsx b/website/src/components/FlaggableElement.tsx
index 2f3f25b2..9606f425 100644
--- a/website/src/components/FlaggableElement.tsx
+++ b/website/src/components/FlaggableElement.tsx
@@ -69,14 +69,14 @@ export const FlaggableElement = (props) => {
   const handleCheckboxState = (isChecked, idx) => {
     setCheckboxValues(
       checkboxValues.map((val, i) => {
-        return i == idx ? isChecked : val;
+        return i === idx ? isChecked : val;
       })
     );
   };
   const handleSliderState = (newVal, idx) => {
     setSliderValues(
       sliderValues.map((val, i) => {
-        return i == idx ? newVal : val;
+        return i === idx ? newVal : val;
       })
     );
   };
diff --git a/website/src/components/Messages.tsx b/website/src/components/Messages.tsx
index b14b0aaa..ef02d4e0 100644
--- a/website/src/components/Messages.tsx
+++ b/website/src/components/Messages.tsx
@@ -43,7 +43,7 @@ export const Messages = ({
   return <Grid gap={2}>{items}</Grid>;
 };
 
-export const MessageView = forwardRef(({ is_assistant, text, message_id }: Message, ref) => {
+export const MessageView = forwardRef(({ is_assistant, text }: Message, ref) => {
   const { colorMode } = useColorMode();
 
   const bgColor = useMemo(() => {
@@ -54,7 +54,11 @@ export const MessageView = forwardRef(({ is_assistant, text, message_id }: Messa
     }
   }, [colorMode, is_assistant]);
 
-  return <div className={`${bgColor} p-4 rounded-md text-white whitespace-pre-wrap`}>{text}</div>;
+  return (
+    <div ref={ref} className={`${bgColor} p-4 rounded-md text-white whitespace-pre-wrap`}>
+      {text}
+    </div>
+  );
 });
 
 MessageView.displayName = "MessageView";
diff --git a/website/src/components/Tasks/CreateTask.tsx b/website/src/components/Tasks/CreateTask.tsx
index 7af262e1..e02dcdeb 100644
--- a/website/src/components/Tasks/CreateTask.tsx
+++ b/website/src/components/Tasks/CreateTask.tsx
@@ -1,10 +1,9 @@
 import { useState } from "react";
-
 import { Messages } from "src/components/Messages";
 import { TaskControls } from "src/components/Survey/TaskControls";
 import { TrackedTextarea } from "src/components/Survey/TrackedTextarea";
 import { TwoColumnsWithCards } from "src/components/Survey/TwoColumnsWithCards";
-import { TaskType } from "./TaskTypes";
+import { TaskType } from "src/components/Tasks/TaskTypes";
 
 export interface CreateTaskProps {
   // we need a task type
diff --git a/website/src/components/Tasks/EvaluateTask.tsx b/website/src/components/Tasks/EvaluateTask.tsx
index 3871b2d9..d0a1f404 100644
--- a/website/src/components/Tasks/EvaluateTask.tsx
+++ b/website/src/components/Tasks/EvaluateTask.tsx
@@ -48,7 +48,7 @@ export const EvaluateTask = ({ tasks, trigger, onSkipTask, onNextTask, mainBgCla
 
       <TaskControlsOverridable
         tasks={tasks}
-        isValid={ranking.length == tasks[0].task[sortables].length}
+        isValid={ranking.length === tasks[0].task[sortables].length}
         prepareForSubmit={() => setRanking(tasks[0].task[sortables].map((_, idx) => idx))}
         onSubmitResponse={submitResponse}
         onSkipTask={(task, reason) => {
diff --git a/website/src/components/Tasks/Task.tsx b/website/src/components/Tasks/Task.tsx
index 153e0a93..777f5dd5 100644
--- a/website/src/components/Tasks/Task.tsx
+++ b/website/src/components/Tasks/Task.tsx
@@ -1,8 +1,8 @@
-import { CreateTask } from "./CreateTask";
-import { EvaluateTask } from "./EvaluateTask";
-import { TaskCategory, TaskTypes } from "./TaskTypes";
-import useSWRMutation from "swr/mutation";
+import { CreateTask } from "src/components/Tasks/CreateTask";
+import { EvaluateTask } from "src/components/Tasks/EvaluateTask";
+import { TaskCategory, TaskTypes } from "src/components/Tasks/TaskTypes";
 import poster from "src/lib/poster";
+import useSWRMutation from "swr/mutation";
 
 export const Task = ({ tasks, trigger, mutate, mainBgClasses }) => {
   const task = tasks[0].task;
diff --git a/website/src/lib/oasst_api_client.ts b/website/src/lib/oasst_api_client.ts
index 1bbb13a8..6a5ca58e 100644
--- a/website/src/lib/oasst_api_client.ts
+++ b/website/src/lib/oasst_api_client.ts
@@ -30,7 +30,7 @@ export class OasstApiClient {
       body: JSON.stringify(body),
     });
 
-    if (resp.status == 204) {
+    if (resp.status === 204) {
       return null;
     }
 
@@ -57,7 +57,7 @@ export class OasstApiClient {
       },
     });
 
-    if (resp.status == 204) {
+    if (resp.status === 204) {
       return null;
     }
 
diff --git a/website/src/pages/api/set_label.ts b/website/src/pages/api/set_label.ts
index 4db5ddaf..cfda114b 100644
--- a/website/src/pages/api/set_label.ts
+++ b/website/src/pages/api/set_label.ts
@@ -1,5 +1,4 @@
 import { getToken } from "next-auth/jwt";
-import prisma from "src/lib/prismadb";
 
 /**
  * Sets the Label in the Backend.
@@ -15,7 +14,7 @@ const handler = async (req, res) => {
   }
 
   // Parse out the local message_id, task ID and the interaction contents.
-  const { message_id, post_id, label_map, text } = await JSON.parse(req.body);
+  const { message_id, label_map, text } = await JSON.parse(req.body);
 
   const interactionRes = await fetch(`${process.env.FASTAPI_URL}/api/v1/text_labels`, {
     method: "POST",
diff --git a/website/src/pages/auth/verify.tsx b/website/src/pages/auth/verify.tsx
index b4d7d739..876aa677 100644
--- a/website/src/pages/auth/verify.tsx
+++ b/website/src/pages/auth/verify.tsx
@@ -1,7 +1,6 @@
 import { useColorMode } from "@chakra-ui/react";
 import Head from "next/head";
 import { getCsrfToken, getProviders } from "next-auth/react";
-import { AuthLayout } from "src/components/AuthLayout";
 
 export default function Verify() {
   const { colorMode } = useColorMode();
diff --git a/website/src/pages/create/summarize_story.tsx b/website/src/pages/create/summarize_story.tsx
index 8620a8f5..61415962 100644
--- a/website/src/pages/create/summarize_story.tsx
+++ b/website/src/pages/create/summarize_story.tsx
@@ -63,7 +63,7 @@ const SummarizeStory = () => {
     return <LoadingScreen text="Loading..." />;
   }
 
-  if (tasks.length == 0) {
+  if (tasks.length === 0) {
     return <div className="p-6 bg-slate-100 text-gray-800">No tasks found...</div>;
   }
 
diff --git a/website/src/pages/evaluate/rate_summary.tsx b/website/src/pages/evaluate/rate_summary.tsx
index 0d2352a2..e9da4a63 100644
--- a/website/src/pages/evaluate/rate_summary.tsx
+++ b/website/src/pages/evaluate/rate_summary.tsx
@@ -60,7 +60,7 @@ const RateSummary = () => {
     return <LoadingScreen text="Loading..." />;
   }
 
-  if (tasks.length == 0) {
+  if (tasks.length === 0) {
     return (
       <div className={`p-12 ${mainBgClasses}`}>
         <div className="flex h-full">