@orsetra/shared-ui 1.0.37 → 1.0.39

package/components/ui/alert-banner.tsx

@@ -10,10 +10,10 @@ const alertBannerVariants = cva(
   {
     variants: {
       variant: {
-        success: "bg-green-50 text-green-800 border-b border-green-200",
-        error: "bg-red-50 text-red-800 border-b border-red-200",
-        warning: "bg-amber-50 text-amber-800 border-b border-amber-200",
-        info: "bg-blue-50 text-blue-800 border-b border-blue-200",
+        success: "bg-green-50 text-green-800",
+        error: "bg-red-50 text-red-800",
+        warning: "bg-amber-50 text-amber-800",
+        info: "bg-blue-50 text-blue-800",
       },
     },
     defaultVariants: {

package/components/ui/side-panel.tsx

@@ -5,7 +5,7 @@ import * as DialogPrimitive from "@radix-ui/react-dialog"
 import { X } from "lucide-react"
 import { cn } from "../../lib/utils"
 
-const SidePanel = DialogPrimitive.Root
+const SidePanelRoot = DialogPrimitive.Root
 
 const SidePanelTrigger = DialogPrimitive.Trigger
 
@@ -28,6 +28,80 @@ const SidePanelOverlay = React.forwardRef<
 ))
 SidePanelOverlay.displayName = DialogPrimitive.Overlay.displayName
 
+interface SidePanelProps {
+  open: boolean
+  onOpenChange: (open: boolean) => void
+  title: string
+  description?: string
+  children: React.ReactNode
+  actions: React.ReactNode
+  side?: "left" | "right"
+  width?: string
+  onClose?: () => void
+}
+
+const SidePanel = ({
+  open,
+  onOpenChange,
+  title,
+  description,
+  children,
+  actions,
+  side = "right",
+  width = "max-w-lg",
+  onClose,
+}: SidePanelProps) => {
+  return (
+    <SidePanelRoot open={open} onOpenChange={onOpenChange}>
+      <SidePanelPortal>
+        <SidePanelOverlay />
+        <DialogPrimitive.Content
+          className={cn(
+            "fixed z-50 bg-white shadow-lg transition ease-in-out data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:duration-300 data-[state=open]:duration-500 flex flex-col",
+            side === "right" &&
+              "inset-y-0 right-0 h-full w-full border-l data-[state=closed]:slide-out-to-right data-[state=open]:slide-in-from-right",
+            side === "left" &&
+              "inset-y-0 left-0 h-full w-full border-r data-[state=closed]:slide-out-to-left data-[state=open]:slide-in-from-left",
+            width
+          )}
+        >
+          {/* Header fixe */}
+          <div className="flex items-center justify-between border-b border-ibm-gray-20 px-6 py-4 flex-shrink-0">
+            <div className="flex-1">
+              <DialogPrimitive.Title className="text-lg font-semibold text-ibm-gray-100">
+                {title}
+              </DialogPrimitive.Title>
+              {description && (
+                <DialogPrimitive.Description className="text-sm text-ibm-gray-60">
+                  {description}
+                </DialogPrimitive.Description>
+              )}
+            </div>
+            <DialogPrimitive.Close
+              className="rounded-sm opacity-70 ring-offset-white transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ibm-blue-60 focus:ring-offset-2 disabled:pointer-events-none data-[state=open]:bg-ibm-gray-10"
+              onClick={onClose}
+            >
+              <X className="h-5 w-5 text-ibm-gray-70" />
+              <span className="sr-only">Close</span>
+            </DialogPrimitive.Close>
+          </div>
+
+          {/* Body scrollable */}
+          <div className="flex-1 overflow-y-auto px-6 py-6">
+            {children}
+          </div>
+
+          {/* Footer fixe */}
+          <div className="flex items-center justify-end gap-3 border-t border-ibm-gray-20 px-6 py-4 flex-shrink-0">
+            {actions}
+          </div>
+        </DialogPrimitive.Content>
+      </SidePanelPortal>
+    </SidePanelRoot>
+  )
+}
+
+// Composants legacy pour compatibilité ascendante
 interface SidePanelContentProps
   extends React.ComponentPropsWithoutRef<typeof DialogPrimitive.Content> {
   side?: "left" | "right"
@@ -59,34 +133,19 @@ const SidePanelContent = React.forwardRef<
 ))
 SidePanelContent.displayName = DialogPrimitive.Content.displayName
 
-interface SidePanelHeaderProps extends React.HTMLAttributes<HTMLDivElement> {
-  onClose?: () => void
-  showCloseButton?: boolean
-}
-
-const SidePanelHeader = React.forwardRef<HTMLDivElement, SidePanelHeaderProps>(
-  ({ className, children, onClose, showCloseButton = true, ...props }, ref) => (
-    <div
-      ref={ref}
-      className={cn(
-        "flex items-center justify-between border-b border-ibm-gray-20 px-6 py-4",
-        className
-      )}
-      {...props}
-    >
-      <div className="flex-1">{children}</div>
-      {showCloseButton && (
-        <DialogPrimitive.Close
-          className="rounded-sm opacity-70 ring-offset-white transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ibm-blue-60 focus:ring-offset-2 disabled:pointer-events-none data-[state=open]:bg-ibm-gray-10"
-          onClick={onClose}
-        >
-          <X className="h-5 w-5 text-ibm-gray-70" />
-          <span className="sr-only">Close</span>
-        </DialogPrimitive.Close>
-      )}
-    </div>
-  )
-)
+const SidePanelHeader = React.forwardRef<
+  HTMLDivElement,
+  React.HTMLAttributes<HTMLDivElement>
+>(({ className, ...props }, ref) => (
+  <div
+    ref={ref}
+    className={cn(
+      "flex items-center justify-between border-b border-ibm-gray-20 px-6 py-4 flex-shrink-0",
+      className
+    )}
+    {...props}
+  />
+))
 SidePanelHeader.displayName = "SidePanelHeader"
 
 const SidePanelTitle = React.forwardRef<
@@ -132,7 +191,7 @@ const SidePanelFooter = React.forwardRef<
   <div
     ref={ref}
     className={cn(
-      "flex items-center justify-end gap-3 border-t border-ibm-gray-20 px-6 py-4",
+      "flex items-center justify-end gap-3 border-t border-ibm-gray-20 px-6 py-4 flex-shrink-0",
       className
     )}
     {...props}
@@ -150,4 +209,5 @@ export {
   SidePanelDescription,
   SidePanelBody,
   SidePanelFooter,
+  SidePanelRoot,
 }

package/index.ts

@@ -1,8 +1,9 @@
 // Utilities
 export * from './lib/utils'
 export * from './lib/menu-utils'
+export * from './lib/error-utils'
 export { BaseService } from './lib/base-service'
-export { default as HttpClient, useHttpClient } from './lib/http-client'
+export { default as HttpClient, useHttpClient, ApiError } from './lib/http-client'
 
 // UI Components
 export * from './components/ui'

package/lib/error-utils.ts

@@ -0,0 +1,127 @@
+/**
+ * Utility functions for parsing API errors
+ */
+
+import { ApiError } from './http-client'
+
+/**
+ * Standard API error response format
+ */
+export interface ApiErrorResponse {
+  BusinessCode?: number
+  Message?: string
+  message?: string
+  error?: string
+  code?: number
+  details?: unknown
+}
+
+/**
+ * Extracts a user-friendly error message from various error formats
+ * 
+ * Supports:
+ * - ApiError from http-client
+ * - Standard Error objects
+ * - API responses with BusinessCode/Message format
+ * - API responses with message/error format
+ * 
+ * @param error - The error object to parse
+ * @param fallbackMessage - Default message if no error message can be extracted
+ * @returns A user-friendly error message
+ * 
+ * @example
+ * ```typescript
+ * try {
+ *   await apiCall()
+ * } catch (error) {
+ *   const message = parseApiError(error)
+ *   showError(message)
+ * }
+ * ```
+ */
+export function parseApiError(error: unknown, fallbackMessage = "An unexpected error occurred"): string {
+  // Handle ApiError from http-client
+  if (error instanceof ApiError) {
+    // Check raw response for BusinessCode/Message format
+    if (error.raw && typeof error.raw === 'object') {
+      const rawError = error.raw as ApiErrorResponse
+      if (rawError.Message) {
+        return rawError.Message
+      }
+      if (rawError.message) {
+        return rawError.message
+      }
+      if (rawError.error) {
+        return rawError.error
+      }
+    }
+    // Use ApiError message
+    return error.message
+  }
+
+  // Handle standard Error objects
+  if (error instanceof Error) {
+    return error.message
+  }
+
+  // Handle plain objects with error properties
+  if (error && typeof error === 'object') {
+    const errorObj = error as ApiErrorResponse
+    
+    // Check for BusinessCode/Message format (custom API format)
+    if (errorObj.Message) {
+      return errorObj.Message
+    }
+    
+    // Check for standard message/error properties
+    if (errorObj.message) {
+      return errorObj.message
+    }
+    
+    if (errorObj.error) {
+      return errorObj.error
+    }
+  }
+
+  // Handle string errors
+  if (typeof error === 'string') {
+    return error
+  }
+
+  // Fallback
+  return fallbackMessage
+}
+
+/**
+ * Checks if an error is an ApiError with a specific status code
+ * 
+ * @param error - The error to check
+ * @param status - The status code to match
+ * @returns true if the error is an ApiError with the specified status
+ * 
+ * @example
+ * ```typescript
+ * if (isApiErrorWithStatus(error, 404)) {
+ *   showError("Resource not found")
+ * }
+ * ```
+ */
+export function isApiErrorWithStatus(error: unknown, status: number): boolean {
+  return error instanceof ApiError && error.status === status
+}
+
+/**
+ * Checks if an error is a network error (no response from server)
+ * 
+ * @param error - The error to check
+ * @returns true if the error is a network error
+ */
+export function isNetworkError(error: unknown): boolean {
+  if (error instanceof Error) {
+    const message = error.message.toLowerCase()
+    return message.includes('network') || 
+           message.includes('fetch') || 
+           message.includes('connection')
+  }
+  return false
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orsetra/shared-ui",
-  "version": "1.0.37",
+  "version": "1.0.39",
   "description": "Shared UI components for Orsetra platform",
   "main": "./index.ts",
   "types": "./index.ts",