@shellui/core 0.0.11 → 0.0.13

package/package.json

@@ -1,30 +1,31 @@
 {
   "name": "@shellui/core",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "description": "ShellUI Core - Core React application runtime",
-  "main": "src/index.ts",
-  "types": "src/index.ts",
+  "main": "dist/index.js",
+  "types": "dist/index.d.ts",
   "type": "module",
   "exports": {
     ".": {
-      "types": "./src/index.ts",
-      "import": "./src/index.ts",
-      "default": "./src/index.ts"
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js",
+      "default": "./dist/index.js"
     },
     "./types": {
-      "types": "./src/features/config/types.ts",
-      "import": "./src/features/config/types.ts",
-      "default": "./src/features/config/types.ts"
+      "types": "./dist/types.d.ts",
+      "import": "./dist/types.js",
+      "default": "./dist/types.js"
     },
     "./constants/urls": {
-      "types": "./src/constants/urls.ts",
-      "import": "./src/constants/urls.ts",
-      "default": "./src/constants/urls.ts"
+      "types": "./dist/constants/urls.d.ts",
+      "import": "./dist/constants/urls.js",
+      "default": "./dist/constants/urls.js"
     },
-    "./style.css": "./src/index.css",
+    "./style.css": "./dist/style.css",
     "./package.json": "./package.json"
   },
   "files": [
+    "dist",
     "src",
     "postcss.config.js",
     "tailwind.config.js",
@@ -63,7 +64,7 @@
     "workbox-strategies": "^7.1.0",
     "workbox-cacheable-response": "^7.1.0",
     "workbox-expiration": "^7.1.0",
-    "@shellui/sdk": "0.0.11"
+    "@shellui/sdk": "0.0.13"
   },
   "peerDependencies": {
     "react": "^18.0.0 || ^19.0.0",
@@ -86,6 +87,7 @@
     "vite-plugin-dts": "^4.5.0"
   },
   "scripts": {
+    "build": "vite build",
     "clean": "rm -rf dist",
     "test": "echo \"No tests specified for @shellui/core\""
   }

package/src/features/config/ConfigProvider.ts

@@ -31,7 +31,17 @@ export function ConfigProvider(props: ConfigProviderProps): ReturnType<typeof cr
       // Vite's define inserts the string value directly, so we only need to parse once
       // Access it directly (no typeof check) so Vite can statically analyze and replace it
       // @ts-expect-error - __SHELLUI_CONFIG__ is injected by Vite at build time
-      const configValue: unknown = __SHELLUI_CONFIG__;
+      let configValue: unknown = __SHELLUI_CONFIG__;
+      
+      // In development, if __SHELLUI_CONFIG__ is undefined, it might not have been replaced by Vite
+      // This can happen if the Vite define configuration isn't working properly
+      if (configValue === undefined && typeof window !== 'undefined') {
+        // Try to get it from window (fallback for dev mode issues)
+        const g = window as unknown as { __SHELLUI_CONFIG__?: unknown };
+        if (g.__SHELLUI_CONFIG__ !== undefined) {
+          configValue = g.__SHELLUI_CONFIG__;
+        }
+      }
 
       // After Vite replacement, configValue will be a JSON string
       // Example: "{\"title\":\"shellui\"}" -> parse -> {title: "shellui"}
@@ -61,8 +71,9 @@ export function ConfigProvider(props: ConfigProviderProps): ReturnType<typeof cr
       }
 
       // Fallback: try to read from globalThis (for edge cases or if define didn't work)
+      // This handles cases where Vite's define didn't work or config needs to be injected at runtime
       const g = globalThis as unknown as { __SHELLUI_CONFIG__?: unknown };
-      if (typeof g.__SHELLUI_CONFIG__ !== 'undefined') {
+      if (typeof g.__SHELLUI_CONFIG__ !== 'undefined' && configValue === undefined) {
         const fallbackValue = g.__SHELLUI_CONFIG__;
         const parsedConfig: ShellUIConfig =
           typeof fallbackValue === 'string'
@@ -80,9 +91,12 @@ export function ConfigProvider(props: ConfigProviderProps): ReturnType<typeof cr
       }
 
       // Return empty config if __SHELLUI_CONFIG__ is undefined (fallback for edge cases)
-      logger.warn(
-        'Config not found. Using empty config. Make sure shellui.config.ts is properly loaded during build.',
-      );
+      // This ensures the provider always provides a value, preventing "useConfig must be used within ConfigProvider" errors
+      if (process.env.NODE_ENV === 'development') {
+        logger.warn(
+          'Config not found. Using empty config. Make sure shellui.config.ts is properly loaded and Vite define is configured correctly.',
+        );
+      }
       return {} as ShellUIConfig;
     } catch (err) {
       logger.error('Failed to load ShellUI config:', { error: err });
@@ -91,6 +105,7 @@ export function ConfigProvider(props: ConfigProviderProps): ReturnType<typeof cr
     }
   });
 
+  // Always provide a value - never null - to prevent "useConfig must be used within ConfigProvider" errors
   const value: ConfigContextValue = { config };
   return createElement(ConfigContext.Provider, { value }, props.children);
 }

package/src/features/config/useConfig.ts

@@ -9,7 +9,20 @@ import { ConfigContext, type ConfigContextValue } from './ConfigProvider';
 export function useConfig(): ConfigContextValue {
   const context = useContext(ConfigContext);
   if (context === null) {
-    throw new Error('useConfig must be used within a ConfigProvider');
+    // This error should never happen if ConfigProvider is properly wrapping the app
+    // If you see this error, check that:
+    // 1. Your component is rendered inside a <ConfigProvider> tree
+    // 2. The ConfigProvider is mounted before components that use useConfig()
+    // 3. Vite's define configuration is working correctly (check __SHELLUI_CONFIG__)
+    const error = new Error(
+      'useConfig must be used within a ConfigProvider. ' +
+      'Make sure your app is wrapped with <ConfigProvider> and that Vite define is configured correctly.'
+    );
+    if (typeof window !== 'undefined' && process.env.NODE_ENV === 'development') {
+      console.error('[ShellUI] ConfigProvider error:', error);
+      console.error('[ShellUI] Check that __SHELLUI_CONFIG__ is defined:', typeof (window as any).__SHELLUI_CONFIG__);
+    }
+    throw error;
   }
   return context;
 }

package/src/index.ts

@@ -2,8 +2,8 @@
  * ShellUI Core - Main entry point
  * Exports the App component, types, and utilities for use in config files and other packages.
  *
- * CSS is imported as a side effect. Consumers must import '@shellui/core/style.css' 
- * (or include it in their build) for styles to apply.
+ * CSS is imported as a side effect and extracted to dist/style.css during the library build.
+ * Consumers must import '@shellui/core/style.css' (or include it in their build) for styles to apply.
  */
 import './index.css';