feat: Added unsafe way to use the config with the possibility to have…

… a null config and a safe way Signed-off-by: Tom Lanser <[email protected]>
animo · Jan 4, 2023 · 61119c7 · 61119c7
1 parent 9ac1380
commit 61119c7
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/packages/toolbox-ui/src/ToolboxApp.tsx b/packages/toolbox-ui/src/ToolboxApp.tsx
@@ -7,7 +7,7 @@ import React from 'react'
 import { RouterProvider } from 'react-router-dom'
 
 import { GlobalErrorHandler } from './components/GlobalErrorHandler'
-import { useConfig } from './contexts/ConfigProvider'
+import { useConfig, useConfigUnsafe } from './contexts/ConfigProvider'
 
 interface ToolboxAppProps {
   router: RouterProviderProps['router']
@@ -31,9 +31,9 @@ const toolboxTheme = (colorScheme: ColorScheme): MantineThemeOverride => ({
 })
 
 export const ToolboxApp = ({ router }: ToolboxAppProps) => {
-  const { config, setColorScheme, loading } = useConfig()
+  const { config, setColorScheme } = useConfigUnsafe()
 
-  const colorScheme = loading ? 'light' : config.colorScheme
+  const colorScheme = config?.colorScheme ?? 'light'
   const toggleColorScheme = (value?: ColorScheme) => setColorScheme(value || colorScheme === 'dark' ? 'light' : 'dark')
 
   return (

diff --git a/packages/toolbox-ui/src/contexts/AgentManagerContext.tsx b/packages/toolbox-ui/src/contexts/AgentManagerContext.tsx
@@ -3,7 +3,7 @@ import type { ReactNode } from 'react'
 
 import React, { createContext, useContext, useMemo, useState } from 'react'
 
-import { useConfig } from './ConfigProvider'
+import { useConfig, useConfigUnsafe } from './ConfigProvider'
 
 export interface IAgentContext {
   agents: AgentConfigRecord[]
@@ -33,7 +33,7 @@ interface AgentManagerProviderProps {
 }
 
 export const AgentManagerProvider = ({ children }: AgentManagerProviderProps) => {
-  const { config, addAgent, loading } = useConfig()
+  const { config, addAgent, loading } = useConfigUnsafe()
   const [currentAgentId, setCurrentAgentId] = useState<string>()
 
   const logout = () => {

diff --git a/packages/toolbox-ui/src/contexts/ConfigProvider.tsx b/packages/toolbox-ui/src/contexts/ConfigProvider.tsx
@@ -22,6 +22,10 @@ export const useConfig = () => {
   }
 }
 
+export const useConfigUnsafe = () => {
+  return useContext(configContext)
+}
+
 interface ConfigProviderProps {
   children: ReactNode
   configRepository: ConfigFileRepository