add sweep update and lint fixes

hero-org · Sep 3, 2023 · 6fab696 · 6fab696
1 parent 30c5def
commit 6fab696
Show file tree

Hide file tree

Showing 8 changed files with 35 additions and 28 deletions.
diff --git a/package.json b/package.json
@@ -14,7 +14,9 @@
     "build": "cross-env INLINE_RUNTIME_CHUNK=true NODE_OPTIONS=--max-old-space-size=8192 yarn vite build",
     "preview": "vite preview",
     "serve": "serve -s build",
-    "preinstall": "yarn global add serve"
+    "preinstall": "yarn global add serve",
+    "lint": "eslint --ext .js,.jsx,.ts,.tsx src",
+    "lint:fix": "eslint --fix --ext .js,.jsx,.ts,.tsx src"
   },
   "eslintConfig": {
     "extends": [

diff --git a/src/pages/Login/index.tsx b/src/pages/Login/index.tsx
@@ -1,7 +1,7 @@
+import React, { useEffect, useState } from 'react'
 import { Auth } from '@supabase/auth-ui-react';
 import { ThemeSupa } from '@supabase/auth-ui-shared'
 import { supabaseClient } from '@/common/helpers/supabase';
-import { useEffect, useState } from 'react';
 import { Session } from '@supabase/supabase-js';
 import { hydrate } from '@/stores/useAccountStore';
 import { useNavigate, useLocation } from "react-router-dom";

diff --git a/src/pages/Notifications/index.tsx b/src/pages/Notifications/index.tsx
@@ -1,6 +1,6 @@
+import React, { useEffect, useState } from 'react'
 
 import { classNames } from '@/common/helpers/css'
-import { useEffect, useState } from 'react'
 import { fetchCasts, getNeynarNotificationsEndpoint } from '@/common/helpers/neynar'
 import { useAccountStore } from '@/stores/useAccountStore'
 import { SelectableListWithHotkeys } from '@/common/components/SelectableListWithHotkeys'

diff --git a/src/pages/Settings/index.tsx b/src/pages/Settings/index.tsx
@@ -35,7 +35,7 @@ export default function Settings() {
     if (session) {
       resetStore();
       setUser(null);
-      const res = await supabaseClient.auth.signOut();
+      await supabaseClient.auth.signOut();
     }
 
     navigate('/login');
@@ -44,7 +44,7 @@ export default function Settings() {
   const displayEmail = user?.email ? `${user?.email.slice(0, 5)}...@${user?.email.split('@')[1]}` : '';
 
   const renderInfoSection = () => {
-    let commands: SimpleCommand[] = [
+    const commands: SimpleCommand[] = [
       { name: 'Command Palette', shortcut: 'cmd+k' },
       { name: 'Feed: go to previous cast in list', shortcut: 'k' },
       { name: 'Feed: go to next cast in list', shortcut: 'j' },

diff --git a/src/stores/useAccountStore.ts b/src/stores/useAccountStore.ts
@@ -188,7 +188,7 @@ const switchAccountTo = (idx: number) => {
 };
 
 const getAccountCommands = () => {
-  let accountCommands: CommandType[] = [];
+  const accountCommands: CommandType[] = [];
 
   for (let i = 0; i < 9; i++) {
     accountCommands.push({
@@ -208,7 +208,7 @@ const getAccountCommands = () => {
 };
 
 const getChannelCommands = () => {
-  let channelCommands: CommandType[] = [];
+  const channelCommands: CommandType[] = [];
 
   channelCommands.push({
     name: `Switch to follow feed`,

diff --git a/src/stores/useNavigationStore.ts b/src/stores/useNavigationStore.ts
@@ -38,19 +38,19 @@ const store = (set: StoreSet) => ({
       state.isCommandPaletteOpen = !state.isCommandPaletteOpen;
     });
   },
-  toAccounts: () => {
-    set((state) => {
-      console.log('useNavStore: toAccounts');
-      // state.mainNavigation = MAIN_NAVIGATION_ENUM.ACCOUNTS;
-      redirect(`/${MAIN_NAVIGATION_ENUM.ACCOUNTS}`)
-    });
-  },
-  toFeed: () => {
-    set((state) => {
-      // state.mainNavigation = MAIN_NAVIGATION_ENUM.FEED;
-      redirect(`/${MAIN_NAVIGATION_ENUM.FEED}`)
-    });
-  },
+  // toAccounts: () => {
+  //   set((state) => {
+  //     console.log('useNavStore: toAccounts');
+  //     // state.mainNavigation = MAIN_NAVIGATION_ENUM.ACCOUNTS;
+  //     redirect(`/${MAIN_NAVIGATION_ENUM.ACCOUNTS}`)
+  //   });
+  // },
+  // toFeed: () => {
+  //   set((state) => {
+  //     // state.mainNavigation = MAIN_NAVIGATION_ENUM.FEED;
+  //     redirect(`/${MAIN_NAVIGATION_ENUM.FEED}`)
+  //   });
+  // },
   // toNewPost: () => {
   //   set((state) => {
   //     state.mainNavigation = MAIN_NAVIGATION_ENUM.NEW_POST;
@@ -61,12 +61,12 @@ const store = (set: StoreSet) => ({
   //     state.mainNavigation = MAIN_NAVIGATION_ENUM.REPLIES;
   //   });
   // },
-  toSettings: () => {
-    set((state) => {
-      console.log('useNavStore: toSettings')
-      // state.mainNavigation = MAIN_NAVIGATION_ENUM.SETTINGS;
-      redirect(`/${MAIN_NAVIGATION_ENUM.FEED}`)
-    });
-  },
+  // toSettings: () => {
+  //   set((state) => {
+  //     console.log('useNavStore: toSettings')
+  //     // state.mainNavigation = MAIN_NAVIGATION_ENUM.SETTINGS;
+  //     redirect(`/${MAIN_NAVIGATION_ENUM.FEED}`)
+  //   });
+  // },
 });
 export const useNavigationStore = create<NavigationStore>()(devtools(mutative(store)));
diff --git a/src/stores/useNewPostStore.ts b/src/stores/useNewPostStore.ts
@@ -141,7 +141,7 @@ const store = (set: StoreSet) => ({
         })).then((res) => {
           if (res.error) {
             console.log('publishPostdraft error:', res.error);
-            return;
+            return
           }
           trackEventWithProperties('publish_post', { authorFid: account.platformAccountId });
           state.removePostDraft(draftIdx);

diff --git a/sweep.yaml b/sweep.yaml
@@ -15,3 +15,8 @@ docs:
   hub-nodejs: ["https://github.com/farcasterxyz/hub-monorepo/blob/main/packages/hub-nodejs/docs/README.md", "We use hub monorepo nodejs web to interact with the Farcaster protocol"]
   react-js: ["https://react.dev/reference/react", "We use react js as our frontend framework"]
   tauri: ["https://tauri.app/v1/", "Tauri is our vector database"]
+sandbox:
+  enabled: true
+  image: "Nodejs"
+  install: "yarn install"
+  formatter: "yarn run lint:fix"