fix linting errors

Author: heyzec

src/commons/application/Application.tsx

@@ -1,7 +1,11 @@
 import React from 'react';
 import { useDispatch } from 'react-redux';
 import { Outlet } from 'react-router-dom';
-import Messages, { MessageType, MessageTypeNames, sendToWebview } from 'src/features/vscode/messages';
+import Messages, {
+  MessageType,
+  MessageTypeNames,
+  sendToWebview
+} from 'src/features/vscode/messages';
 
 import NavigationBar from '../navigationBar/NavigationBar';
 import Constants from '../utils/Constants';
@@ -78,8 +82,8 @@ const Application: React.FC = () => {
     if (!window.confirm) {
       // Polyfill confirm() to instead show as VS Code notification
       // TODO: Pass text as a new Message to the webview
-      window.confirm = (text) => {
-        console.log(`Confirmation automatically accepted: ${text ?? "No text provided"}`);
+      window.confirm = text => {
+        console.log(`Confirmation automatically accepted: ${text ?? 'No text provided'}`);
         return true;
       };
     }
@@ -102,7 +106,12 @@ const Application: React.FC = () => {
           if (message.token) {
             const token = JSON.parse(message.token.trim());
             console.log(`FRONTEND: WebviewStarted: ${token}`);
-            dispatch(SessionActions.setTokens({accessToken: token.accessToken, refreshToken: token.refreshToken}));
+            dispatch(
+              SessionActions.setTokens({
+                accessToken: token.accessToken,
+                refreshToken: token.refreshToken
+              })
+            );
             dispatch(SessionActions.fetchUserAndCourse());
           }
           break;
@@ -121,6 +130,7 @@ const Application: React.FC = () => {
           break;
       }
     });
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
 
   return (

src/commons/assessmentWorkspace/AssessmentWorkspace.tsx

@@ -215,27 +215,7 @@ const AssessmentWorkspace: React.FC<AssessmentWorkspaceProps> = props => {
     if (!assessment) {
       return;
     }
-    // // ------------- PLEASE NOTE, EVERYTHING BELOW THIS SEEMS TO BE UNUSED -------------
-    // // checkWorkspaceReset does exactly the same thing.
-    // let questionId = props.questionId;
-    // if (props.questionId >= assessment.questions.length) {
-    //   questionId = assessment.questions.length - 1;
-    // }
-
-    // const question = assessment.questions[questionId];
-
-    // let answer = '';
-    // if (question.type === QuestionTypes.programming) {
-    //   if (question.answer) {
-    //     answer = (question as IProgrammingQuestion).answer as string;
-    //   } else {
-    //     answer = (question as IProgrammingQuestion).solutionTemplate;
-    //   }
-    // }
-
-    // // TODO: Hardcoded to make use of the first editor tab. Refactoring is needed for this workspace to enable Folder mode.
-    // handleEditorValueChange(0, answer);
-    // // eslint-disable-next-line react-hooks/exhaustive-deps
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
 
   /**

src/features/vscode/messages.ts

@@ -14,9 +14,9 @@ const Messages = createMessages({
   NewEditor: (assessmentName: string, questionId: number, code: string) => ({
     assessmentName,
     questionId,
-    code,
+    code
   }),
-  Text: (code: string) => ({ code }),
+  Text: (code: string) => ({ code })
 });
 
 export default Messages;
@@ -32,20 +32,18 @@ type BaseMessage<T extends string, P extends object> = {
 
 // A helper function to create messages dynamically from schema (hoisted!)
 function createMessages<T extends Record<string, (...args: any[]) => object>>(
-  creators: T,
+  creators: T
 ): {
-  [K in Extract<keyof T, string>]: (
-    ...args: Parameters<T[K]>
-  ) => BaseMessage<K, ReturnType<T[K]>>;
+  [K in Extract<keyof T, string>]: (...args: Parameters<T[K]>) => BaseMessage<K, ReturnType<T[K]>>;
 } {
   return Object.fromEntries(
     Object.entries(creators).map(([key, creator]) => [
       key,
       (...args: any[]) => ({
         type: key,
-        ...creator(...args),
-      }),
-    ]),
+        ...creator(...args)
+      })
+    ])
   ) as any;
 }
 
@@ -61,14 +59,14 @@ export type MessageType = MessageTypes[keyof MessageTypes];
 export const MessageTypeNames = (() =>
   ({
     ...Object.keys(Messages)
-      .filter((k) => isNaN(Number(k)))
+      .filter(k => isNaN(Number(k)))
       .reduce(
         (acc, cur) => ({
           ...acc,
-          [cur]: cur,
+          [cur]: cur
         }),
-        {},
-      ),
+        {}
+      )
   }) as {
     [k in keyof typeof Messages]: k;
   })();
@@ -77,19 +75,19 @@ export const MessageTypeNames = (() =>
 // Wrapper functions
 // ================================================================================
 
-export const FRONTEND_ELEMENT_ID = "frontend";
+export const FRONTEND_ELEMENT_ID = 'frontend';
 
 export function sendToWebview(message: MessageType) {
-  window.parent.postMessage(message, "*");
+  window.parent.postMessage(message, '*');
 }
 export function sendToFrontend(document: Document, message: MessageType) {
   const iframe: HTMLIFrameElement = document.getElementById(
-    FRONTEND_ELEMENT_ID,
+    FRONTEND_ELEMENT_ID
   ) as HTMLIFrameElement;
   const contentWindow = iframe.contentWindow;
   if (!contentWindow) {
     return;
   }
   // TODO: Don't hardcode this!
-  contentWindow.postMessage(message, "http://localhost:8000");
+  contentWindow.postMessage(message, 'http://localhost:8000');
 }