Do not sync judge results with firebase (#166)

* do not sync judge results with firebase

this takes up too much space: roughly 3GB for 350k files. we should figure out a better way to do this later.

* oops

* skip sync output test

.gitignore

@@ -36,4 +36,7 @@ yarn-error.log*
 /playwright/.cache/
 # private folder where I store firebase admin key
 private
-.env
+.env
+
+# ignore downloaded firebase data
+/db_analysis/*.json

e2e/runs_code.spec.ts

@@ -49,7 +49,7 @@ test.describe('Basic Functionality', () => {
     await context2.close();
   });
 
-  test('should sync output', async ({ page, browser, isMobile }) => {
+  test.skip('should sync output', async ({ page, browser, isMobile }) => {
     const context2 = await browser.newContext();
     const page2 = await context2.newPage();
 

pages/[id].tsx

@@ -30,13 +30,15 @@ import { useNullableUserContext } from '../src/context/UserContext';
 import useUserPermission from '../src/hooks/useUserPermission';
 import { SettingsModal } from '../src/components/settings/SettingsModal';
 import { getSampleIndex } from '../src/components/JudgeInterface/Samples';
-import useJudgeResults from '../src/hooks/useJudgeResults';
 import JudgeResult from '../src/types/judge';
 import useUserFileConnection from '../src/hooks/useUserFileConnection';
 import useUpdateUserDashboard from '../src/hooks/useUpdateUserDashboard';
 import { ConfirmOverrideModal } from '../src/components/ConfirmOverrideModal';
 import Link from 'next/link';
 import Head from 'next/head';
+import useJudgeResults, {
+  JudgeResultsProvider,
+} from '../src/context/JudgeResultsContext';
 
 function EditorPage() {
   const { fileData, updateFileData } = useEditorContext();
@@ -51,6 +53,7 @@ function EditorPage() {
   const getMainEditorValue = useAtomValue(mainEditorValueAtom);
   const getInputEditorValue = useAtomValue(inputEditorValueAtom);
   const [judgeResults, setJudgeResults] = useJudgeResults();
+  const [isCodeRunning, setIsCodeRunning] = useState(false);
 
   useUserFileConnection();
   useUpdateUserDashboard();
@@ -74,11 +77,6 @@ function EditorPage() {
 
   const handleRunCode = () => {
     const problem = fileData.settings.problem;
-    const setIsRunning = (isRunning: boolean) => {
-      updateFileData({
-        isCodeRunning: isRunning,
-      });
-    };
     const fetchJudge = (
       code: string,
       input: string,
@@ -113,7 +111,7 @@ function EditorPage() {
         return;
       }
 
-      setIsRunning(true);
+      setIsCodeRunning(true);
       setResultAt(inputTabIndex, null);
 
       const code = getMainEditorValue();
@@ -133,7 +131,7 @@ function EditorPage() {
           );
           console.error(e);
         })
-        .finally(() => setIsRunning(false));
+        .finally(() => setIsCodeRunning(false));
     };
 
     const runAllSamples = async () => {
@@ -143,7 +141,7 @@ function EditorPage() {
       }
       const samples = problem.samples;
 
-      setIsRunning(true);
+      setIsCodeRunning(true);
       setResultAt(1, null);
 
       const code = getMainEditorValue();
@@ -206,7 +204,7 @@ function EditorPage() {
       } catch (e) {
         console.error(e);
       }
-      setIsRunning(false);
+      setIsCodeRunning(false);
     };
 
     if (inputTab === 'input') {
@@ -234,7 +232,7 @@ function EditorPage() {
             runButton={
               <RunButton
                 onClick={handleRunCode}
-                showLoading={fileData.isCodeRunning || loading}
+                showLoading={isCodeRunning || loading}
                 disabledForViewOnly={readOnly}
               />
             }
@@ -314,7 +312,9 @@ export default function FilePage() {
         fileNotFoundUI={fileNotFoundUI}
         permissionDeniedUI={permissionDeniedUI}
       >
-        <EditorPage />
+        <JudgeResultsProvider>
+          <EditorPage />
+        </JudgeResultsProvider>
       </EditorProvider>
       <ConfirmOverrideModal />
     </>

src/components/Workspace/Workspace.tsx

@@ -27,11 +27,12 @@ import { Output } from '../Output';
 import { TabBar } from '../TabBar';
 import { UserList } from '../UserList/UserList';
 import Samples from '../JudgeInterface/Samples';
-import useJudgeResults from '../../hooks/useJudgeResults';
 import { useEditorContext } from '../../context/EditorContext';
 import useUserPermission from '../../hooks/useUserPermission';
 import { useUserContext } from '../../context/UserContext';
 import { Sample } from '../JudgeInterface/Samples';
+import useJudgeResults from '../../context/JudgeResultsContext';
+
 export type ProblemData = {
   id: number;
   submittable: boolean;

src/components/settings/SettingsModal.tsx

@@ -20,11 +20,11 @@ import JudgeSettings from './JudgeSettings';
 
 import SignInSettings from './SignInSettings';
 import JudgeResult from '../../types/judge';
-import useJudgeResults from '../../hooks/useJudgeResults';
 import { EditorMode, useUserContext } from '../../context/UserContext';
 import { FileSettings, useEditorContext } from '../../context/EditorContext';
 import useUserPermission from '../../hooks/useUserPermission';
 import firebase from 'firebase/app';
+import useJudgeResults from '../../context/JudgeResultsContext';
 
 export interface SettingsDialogProps {
   isOpen: boolean;

src/context/EditorContext.tsx

@@ -38,10 +38,6 @@ export type FileData = {
     };
   };
   settings: FileSettings;
-  isCodeRunning: boolean;
-  state: {
-    judge_resuts: any; // ???
-  };
   chat: {
     [key: string]: Omit<ChatMessage, 'key'>;
   };

src/context/JudgeResultsContext.tsx

@@ -0,0 +1,30 @@
+import { createContext, useContext, useState } from 'react';
+import JudgeResult from '../types/judge';
+
+const JudgeResultsContext = createContext<
+  [(JudgeResult | null)[], (newResults: (JudgeResult | null)[]) => void] | null
+>(null);
+
+export default function useJudgeResults() {
+  const context = useContext(JudgeResultsContext);
+  if (context === null) {
+    throw new Error(
+      'useJudgeResults must be used within an JudgeResultsProvider'
+    );
+  }
+
+  return context;
+}
+
+export function JudgeResultsProvider({
+  children,
+}: {
+  children: React.ReactNode;
+}) {
+  const state = useState<(JudgeResult | null)[]>([]);
+  return (
+    <JudgeResultsContext.Provider value={state}>
+      {children}
+    </JudgeResultsContext.Provider>
+  );
+}