refactor: dashboard api base url env (#374)

Author: peppescg

index.html

@@ -9,7 +9,7 @@
     <div id="root"></div>
     <script>
       window.APP_CONFIG = {
-        BASE_API_URL: '${BASE_API_URL}',
+        DASHBOARD_API_BASE_URL: '${DASHBOARD_API_BASE_URL}',
       }
     </script>
     <script type="module" src="/src/main.tsx"></script>

src/global.d.ts

@@ -1,5 +1,5 @@
 export interface AppConfig {
-  BASE_API_URL?: string
+  DASHBOARD_API_BASE_URL?: string
 }
 
 declare global {

src/hooks/useSse.ts

@@ -8,7 +8,7 @@ import {
 import { invalidateQueries } from '@/lib/react-query-utils'
 import { getAppConfig } from '@/lib/utils'
 
-const baseApiUrl = getAppConfig().BASE_API_URL
+const baseApiUrl = getAppConfig().DASHBOARD_API_BASE_URL
 
 export function useSse() {
   const location = useLocation()

src/lib/__tests__/utils.test.ts

@@ -4,9 +4,9 @@ import { AppConfig } from '@/global'
 describe('getAppConfig', () => {
   const mockViteBaseApiUrl = 'https://api.mock.com'
 
-  it('default base api url if ${BASE_API_URL}" not configured', () => {
+  it('default base api url if ${DASHBOARD_API_BASE_URL}" not configured', () => {
     const mockAppConfig: AppConfig = {
-      BASE_API_URL: '${BASE_API_URL}',
+      DASHBOARD_API_BASE_URL: '${DASHBOARD_API_BASE_URL}',
     }
 
     Object.defineProperty(window, 'APP_CONFIG', {
@@ -16,15 +16,15 @@ describe('getAppConfig', () => {
 
     const expectedConfig: AppConfig = {
       ...mockAppConfig,
-      BASE_API_URL: 'https://mock.codegate.ai',
+      DASHBOARD_API_BASE_URL: 'https://mock.codegate.ai',
     }
 
     expect(getAppConfig()).toEqual(expectedConfig)
   })
 
-  it('replace base api url if ${BASE_API_URL}" is configured', () => {
+  it('replace base api url if ${DASHBOARD_API_BASE_URL}" is configured', () => {
     const mockAppConfig: AppConfig = {
-      BASE_API_URL: mockViteBaseApiUrl,
+      DASHBOARD_API_BASE_URL: mockViteBaseApiUrl,
     }
 
     Object.defineProperty(window, 'APP_CONFIG', {
@@ -34,7 +34,7 @@ describe('getAppConfig', () => {
 
     const expectedConfig: AppConfig = {
       ...mockAppConfig,
-      BASE_API_URL: mockViteBaseApiUrl,
+      DASHBOARD_API_BASE_URL: mockViteBaseApiUrl,
     }
 
     expect(getAppConfig()).toEqual(expectedConfig)

src/lib/utils.ts

@@ -73,17 +73,17 @@ export function sanitizeQuestionPrompt({
 }
 
 export function getAppConfig(): AppConfig {
-  const baseApiUrl = window.APP_CONFIG?.BASE_API_URL
+  const baseApiUrl = window.APP_CONFIG?.DASHBOARD_API_BASE_URL
 
-  if (!baseApiUrl || baseApiUrl === '${BASE_API_URL}') {
+  if (!baseApiUrl || baseApiUrl === '${DASHBOARD_API_BASE_URL}') {
     return {
       ...window.APP_CONFIG,
-      BASE_API_URL: import.meta.env.VITE_BASE_API_URL,
+      DASHBOARD_API_BASE_URL: import.meta.env.VITE_BASE_API_URL,
     }
   }
 
   return {
     ...window.APP_CONFIG,
-    BASE_API_URL: baseApiUrl,
+    DASHBOARD_API_BASE_URL: baseApiUrl,
   }
 }

src/main.tsx

@@ -17,7 +17,7 @@ import { getAppConfig } from './lib/utils.ts'
 
 // Initialize the API client
 client.setConfig({
-  baseUrl: getAppConfig().BASE_API_URL,
+  baseUrl: getAppConfig().DASHBOARD_API_BASE_URL,
 })
 
 createRoot(document.getElementById('root')!).render(

src/vite-env.d.ts

@@ -1,5 +1,5 @@
 /// <reference types="vite/client" />
 interface ImportBaseApiEnv {
-  readonly BASE_API_URL: string
+  readonly DASHBOARD_API_BASE_URL: string
   readonly VITE_BASE_API_URL: string
 }