cleanup and build process

Author: fubhy

.github/workflows/tests-and-checks.yml

@@ -25,8 +25,8 @@ jobs:
       - name: Generate Prisma Types
         run: cd apps/server && pnpm prisma generate
       - name: Typecheck
-        run: pnpm ts:check
-      - name: Linting & Formatting
         run: pnpm check
+      - name: Linting & Formatting
+        run: pnpm lint
       - name: Test
         run: pnpm test

.gitignore

@@ -1,5 +1,6 @@
 dist
 build
+publish
 output
 
 # Logs
@@ -10,6 +11,9 @@ yarn-debug.log*
 yarn-error.log*
 lerna-debug.log*
 
+# Local database
+*.db
+
 # Runtime data
 pids
 *.pid

apps/events/.gitignore

@@ -5,16 +5,7 @@
   "type": "module",
   "scripts": {
     "dev": "vite --force",
-    "build": "tsc -b && vite build",
-    "lint": "pnpm biome lint src/*",
-    "lint:fix": "pnpm biome lint --write src/*",
-    "format": "pnpm biome format src/*",
-    "format:fix": "pnpm biome format --write src/*",
-    "check": "pnpm biome check src/*",
-    "check:fix": "pnpm biome check --write src/*",
-    "preview": "vite preview",
-    "test": "vitest run",
-    "ts:check": "tsc --noEmit"
+    "preview": "vite preview"
   },
   "dependencies": {
     "@automerge/automerge": "^v2.2.9-alpha.3",
@@ -39,8 +30,7 @@
     "tailwind-merge": "^2.5.3",
     "tailwindcss-animate": "^1.0.7",
     "uuid": "^11.0.3",
-    "viem": "^2.21.56",
-    "vite-plugin-node-polyfills": "^0.22.0"
+    "viem": "^2.21.56"
   },
   "devDependencies": {
     "@biomejs/biome": "1.9.4",
@@ -55,6 +45,7 @@
     "globals": "^15.10.0",
     "postcss": "^8.4.47",
     "tailwindcss": "^3.4.13",
+    "vite-plugin-node-polyfills": "^0.22.0",
     "vite-plugin-top-level-await": "^1.4.4",
     "vite-plugin-wasm": "^3.3.0"
   }

apps/events/package.json

@@ -8,7 +8,7 @@ function DoGraphLogin() {
   useEffect(() => {
     console.log('Logging in to The Graph');
     login();
-  }, []);
+  }, [login]);
   return <div />;
 }
 
@@ -17,27 +17,26 @@ function Auth({ children }: { children: React.ReactNode }) {
   const { wallets } = useWallets();
   const [signer, setSigner] = useState<Identity.Signer | null>(null);
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: todo [this is kinda ugly]
   useEffect(() => {
-    const getSigner = async () => {
-      const embeddedWallet = wallets.find((wallet) => wallet.walletClientType === 'privy') || wallets[0];
-      const provider = await embeddedWallet.getEthersProvider();
-      const newSigner = provider.getSigner();
-      if (embeddedWallet.walletClientType === 'privy') {
-        newSigner.signMessage = async (message) => {
-          // @ts-expect-error signMessage is a string in this case
-          const signature = await signMessage(message); //, uiConfig);
+    if (wallets.length > 0) {
+      (async () => {
+        const embeddedWallet = wallets.find((wallet) => wallet.walletClientType === 'privy') || wallets[0];
+        const provider = await embeddedWallet.getEthersProvider();
+        const newSigner = provider.getSigner();
 
-          return signature;
-        };
-      }
-      setSigner(newSigner);
-    };
+        if (embeddedWallet.walletClientType === 'privy') {
+          newSigner.signMessage = async (message) => {
+            // @ts-expect-error signMessage is a string in this case
+            const signature = await signMessage(message); //, uiConfig);
+            return signature;
+          };
+        }
 
-    if (wallets.length > 0) {
-      getSigner();
+        setSigner(newSigner);
+      })();
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [wallets]);
+  }, [wallets, setSigner, signMessage]);
 
   return (
     <>

apps/events/src/components/auth.tsx

@@ -2,6 +2,7 @@ import { createRoot } from 'react-dom/client';
 import { Boot } from './Boot.js';
 import './index.css';
 
+// biome-ignore lint/style/noNonNullAssertion: root element is always there
 createRoot(document.getElementById('root')!).render(
   // <StrictMode>
   <Boot />,

apps/events/src/main.tsx

@@ -30,9 +30,11 @@ export const Route = createRootRoute({
           <footer className="flex flex-col gap-2 sm:flex-row py-6 w-full shrink-0 items-center px-4 md:px-6 border-t">
             <p className="text-xs text-gray-500 dark:text-gray-400">© 2023 Acme Events. All rights reserved.</p>
             <nav className="sm:ml-auto flex gap-4 sm:gap-6">
+              {/* biome-ignore lint/a11y/useValidAnchor: todo */}
               <a className="text-xs hover:underline underline-offset-4" href="#">
                 Terms of Service
               </a>
+              {/* biome-ignore lint/a11y/useValidAnchor: todo */}
               <a className="text-xs hover:underline underline-offset-4" href="#">
                 Privacy
               </a>

apps/events/src/routes/__root.tsx

@@ -14,19 +14,17 @@ function Login() {
   const disableLogin = !ready || (ready && authenticated);
   const { authenticated: graphAuthenticated } = Identity.useGraphLogin();
   const router = useRouter();
-  const redirectToPlayground = () => {
-    router.navigate({
-      to: '/playground',
-    });
-  };
 
   useEffect(() => {
     console.log('Privy Authenticated', authenticated);
     console.log('The Graph Authenticated', graphAuthenticated);
     if (authenticated && graphAuthenticated) {
-      redirectToPlayground();
+      router.navigate({
+        to: '/playground',
+      });
     }
-  }, [authenticated, graphAuthenticated]);
+  }, [authenticated, router, graphAuthenticated]);
+
   return (
     <div className="flex flex-1 justify-center items-center flex-col gap-4">
       {(!ready || !authenticated) && (

apps/events/src/routes/login.lazy.tsx

@@ -63,7 +63,7 @@ const App = ({
     console.log('Authenticated: ', isAuthenticated());
     console.log('Session Token: ', getSessionToken());
     console.log('Identity: ', getIdentity());
-  }, [accountId]);
+  }, [accountId, isAuthenticated, getSessionToken, getIdentity]);
   return (
     <>
       <div>

apps/events/src/routes/playground.tsx

@@ -1,5 +1,6 @@
 {
   "extends": "../../tsconfig.base.json",
+  "include": ["src"],
   "compilerOptions": {
     "target": "ES2021",
     "useDefineForClassFields": true,
@@ -34,6 +35,5 @@
       "@graphprotocol/hypergraph": ["../../packages/hypergraph/src/index.js"],
       "@graphprotocol/hypergraph/*": ["../../packages/hypergraph/src/*.js"]
     }
-  },
-  "include": ["src"]
+  }
 }