fix: improve components

src/components/lume/LumeDashboard/LumeDashboard.tsx

@@ -4,6 +4,7 @@ import Logo from "../../../assets/lume-logo.png";
 import { cva } from "class-variance-authority";
 import { cn } from "../../utils";
 import { useState, useEffect } from "react";
+import React from "react";
 
 const SYNCSTATE_TO_TEXT: Record<Network["syncState"], string> = {
   done: "Synced",
@@ -11,8 +12,11 @@ const SYNCSTATE_TO_TEXT: Record<Network["syncState"], string> = {
   syncing: "Syncing",
 };
 
-const LumeDashboard = () => {
+export const LumeDashboardTrigger = Dialog.Trigger;
-  const { networks } = useLume();
+LumeDashboardTrigger.displayName = "LumeDashboardTrigger";
+
+const LumeDashboard = ({children}: React.PropsWithChildren) => {
+  const { lume: { networks } } = useLume();
 
   const [uniqueNetworkTypes, setUniqueNetworkTypes] = useState<string[]>([]);
 
@@ -22,9 +26,29 @@ const LumeDashboard = () => {
     setUniqueNetworkTypes(uniqueTypes);
   }, [networks]);
 
+  const DefaultTrigger = () => (
+    <LumeDashboardTrigger asChild>
+      <button className="bg-primary text-primary-foreground p-2 px-4 text-sm font-semibold font-mono rounded-md">
+        Open Dashboard
+      </button>
+    </LumeDashboardTrigger>
+  );
+  const GivenTrigger = React.Children.toArray(children)
+    .filter((c) => {
+      console.log({component: c})
+      if(typeof c === 'object'){
+        //@ts-expect-error -- I dont know what the type of this should be, i just know that this works
+        return c.type?.displayName === "LumeDashboardTrigger"
+      }
+
+      return false
+    })
+    .at(0);
+  const Trigger = GivenTrigger ? () => GivenTrigger : DefaultTrigger;
+
   return (
     <Dialog.Root>
-      <Dialog.Trigger>Open</Dialog.Trigger>
+      <Trigger />
       <Dialog.Portal>
         <Dialog.Overlay className="fixed z-40 inset-0 bg-black bg-opacity-50 backdrop-blur-sm" />
         <Dialog.Content className="fixed p-5 z-50 right-0 bottom-0 top-0 w-[300px] bg-neutral-950 text-white border-black border-l data-[state=closed]:slide-out-to-right data-[state=open]:slide-in-from-right ease-in-out data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:duration-300 data-[state=open]:duration-500">

src/components/lume/LumeIdentity/LumeIdentity.tsx

@@ -1,4 +1,4 @@
-import React, { ReactElement, type FC, JSXElementConstructor, ReactPortal } from "react";
+import React, { type FC } from "react";
 import { Button } from "../../ui/button";
 import {
   SwitchableComponent,
@@ -128,10 +128,6 @@ export default function Wrapped({ children }: React.PropsWithChildren) {
     .at(0);
   const Trigger = GivenTrigger ? () => GivenTrigger : DefaultTrigger;
 
-  console.log({
-    Trigger, GivenTrigger
-  })
-
   return (
     <Dialog.Root>
       <Trigger />

src/components/lume/LumeIdentity/LumeIdentityContext.ts

@@ -1,23 +1,23 @@
-import { useState } from "react";
 import {
   login,
   // loginComplete,
   // logoutComplete,
 } from "@lumeweb/libkernel/kernel";
+import { useLume } from "../LumeProvider";
 
 export function useLumeIndentity() {
-  const [loggedIn, setLoggedIn] = useState(false);
+  const {isLoggedIn, setIsLoggedIn} = useLume();
 
   return {
-    isSignedIn: loggedIn,
+    isSignedIn: isLoggedIn,
     async signIn(key: Uint8Array) {
       await login(key);
       // await loginComplete(); # this function is buggy `auth.ts:42 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'promise') `
-      setLoggedIn(true);
+      setIsLoggedIn(true);
     },
     async signOut() {
       // await logoutComplete();
-      setLoggedIn(false);
+      setIsLoggedIn(false);
     },
   };
 }

src/components/lume/LumeProvider.tsx

@@ -30,13 +30,15 @@ type LumeObject = {
   networks: Network[];
 };
 
-type LumeContext = {
+type LumeContextType = {
+  isLoggedIn: boolean,
+  setIsLoggedIn: (value: boolean) => void,
   lume: LumeObject;
 };
 
 const networkRegistry = createNetworkRegistryClient();
 
-const LumeContext = createContext<LumeContext | undefined>(undefined);
+const LumeContext = createContext<LumeContextType | undefined>(undefined);
 
 const LumeProvider = ({ children }: { children: ReactNode }) => {
   const [lume, setLume] = useState<LumeObject>({ networks: [] });
@@ -121,9 +123,10 @@ const LumeProvider = ({ children }: { children: ReactNode }) => {
       statusUnsubs.current.forEach((unsub) => unsub());
     };
   }, []);
+  const [isLoggedIn, setIsLoggedIn] = useState(false);
 
   return (
-    <LumeContext.Provider value={{ lume }}>{children}</LumeContext.Provider>
+    <LumeContext.Provider value={{ lume, isLoggedIn, setIsLoggedIn }}>{children}</LumeContext.Provider>
   );
 };
 
@@ -136,6 +139,5 @@ export function useLume() {
     throw new Error("useLume must be used within a LumeProvider");
   }
 
-  const { lume } = ctx;
+  return ctx;
-  return lume;
 }

src/index.ts

@@ -2,6 +2,6 @@ export {
   default as LumeProvider,
   useLume,
 } from "./components/lume/LumeProvider";
-export { default as LumeDashboard } from "./components/lume/LumeDashboard/LumeDashboard";
+export { default as LumeDashboard, LumeDashboardTrigger } from "./components/lume/LumeDashboard/LumeDashboard";
 export { default as LumeIdentity, LumeIdentityTrigger } from "./components/lume/LumeIdentity/LumeIdentity";
 import "../styles/globals.css";