@orsetra/shared-ui 1.9.0 → 1.9.1

package/components/layout/sidebar/main-sidebar-flyout.tsx

@@ -3,13 +3,24 @@
 import * as React from "react"
 import { cn } from "../../../lib/utils"
 import type { MainMenuItem, SubMenuItem } from "./data"
+import { Home, ExternalLink, type LucideIcon } from "lucide-react"
+
+export interface FlyoutDefaultLink {
+  label:        string
+  href:         string
+  description?: string
+  icon?:        LucideIcon
+}
 
 interface MainSidebarFlyoutProps {
-  item: MainMenuItem
-  /** Sub-items with hrefs already resolved by the parent */
-  subItems: SubMenuItem[]
-  onMouseEnter: () => void
-  onMouseLeave: () => void
+  item:                 MainMenuItem | null
+  subItems:             SubMenuItem[]
+  onMouseEnter:         () => void
+  onMouseLeave:         () => void
+  main_base_url?:       string
+  defaultTitle?:        string
+  defaultDescription?:  string
+  defaultLinks?:        FlyoutDefaultLink[]
 }
 
 export function MainSidebarFlyout({
@@ -17,8 +28,21 @@ export function MainSidebarFlyout({
   subItems,
   onMouseEnter,
   onMouseLeave,
+  main_base_url = "/",
+  defaultTitle = "Camel X Platform",
+  defaultDescription,
+  defaultLinks = [],
 }: MainSidebarFlyoutProps) {
-  const Icon = item.icon
+  const homePath = React.useMemo(() => {
+    if (!main_base_url) return "/"
+    try {
+      return (main_base_url.startsWith("http://") || main_base_url.startsWith("https://"))
+        ? new URL(main_base_url).pathname
+        : main_base_url
+    } catch {
+      return "/"
+    }
+  }, [main_base_url])
 
   return (
     <div
@@ -26,46 +50,107 @@ export function MainSidebarFlyout({
       onMouseLeave={onMouseLeave}
       className="h-full flex flex-col flex-shrink-0 overflow-hidden bg-gray-50 border-r border-ui-border shadow-[4px_0_20px_-4px_rgba(0,0,0,0.10)] w-72 lg:w-80"
     >
-      {/* ── Section header ───────────────────────────────────────────────────── */}
-      <div className="flex items-center px-4 lg:px-5 pt-5 pb-1 flex-shrink-0 gap-2.5">
-        <Icon className="h-5 w-5 flex-shrink-0" style={{ color: "#4589ff" }} />
-        <h2 className="text-sm font-semibold text-text-primary tracking-wide truncate flex-1">
-          {item.label}
-        </h2>
-      </div>
+      {item ? (
+        /* ── Hovered item: sub-menu list ─────────────────────────────────── */
+        <>
+          <div className="flex items-center px-4 lg:px-5 pt-5 pb-1 flex-shrink-0 gap-2.5">
+            <item.icon className="h-5 w-5 flex-shrink-0" style={{ color: "#4589ff" }} />
+            <h2 className="text-sm font-semibold text-text-primary tracking-wide truncate flex-1">
+              {item.label}
+            </h2>
+          </div>
 
-      {/* ── Section description ───────────────────────────────────────────────── */}
-      {item.description && (
-        <p className="px-4 lg:px-5 pt-1 pb-4 text-xs text-text-secondary leading-relaxed">
-          {item.description}
-        </p>
-      )}
+          {item.description && (
+            <p className="px-4 lg:px-5 pt-1 pb-4 text-xs text-text-secondary leading-relaxed">
+              {item.description}
+            </p>
+          )}
+
+          <nav className="flex-1 overflow-y-auto px-2 lg:px-3 py-2 space-y-0.5">
+            {subItems.map((sub) => {
+              const SubIcon = sub.icon
+              return (
+                <a
+                  key={sub.id}
+                  href={sub.href}
+                  className={cn(
+                    "flex items-start transition-colors border-l-4",
+                    "px-2 lg:px-3 py-2 gap-x-3",
+                    "hover:bg-ui-background border-transparent hover:border-interactive",
+                  )}
+                >
+                  <SubIcon className="h-4 w-4 flex-shrink-0 mt-0.5" style={{ color: "#4589ff" }} />
+                  <div className="min-w-0 flex-1">
+                    <p className="text-sm font-medium text-text-primary leading-snug truncate">{sub.name}</p>
+                    {sub.description && (
+                      <p className="text-xs text-text-secondary leading-relaxed mt-0.5 line-clamp-2">{sub.description}</p>
+                    )}
+                  </div>
+                </a>
+              )
+            })}
+          </nav>
+        </>
+      ) : (
+        /* ── Default state: welcome + CTA + doc links ───────────────────── */
+        <div className="flex flex-col h-full">
+          {/* Header */}
+          <div className="px-5 pt-6 pb-5 border-b border-ui-border flex-shrink-0">
+            <div className="flex items-center gap-2.5 mb-3">
+              <div className="h-8 w-8 rounded-md bg-interactive flex items-center justify-center flex-shrink-0">
+                <Home className="h-4 w-4 text-white" />
+              </div>
+              <span className="text-sm font-semibold text-text-primary truncate">{defaultTitle}</span>
+            </div>
+
+            {defaultDescription && (
+              <p className="text-xs text-text-secondary leading-relaxed mb-4">
+                {defaultDescription}
+              </p>
+            )}
 
-      {/* ── Nav items ────────────────────────────────────────────────────────── */}
-      <nav className="flex-1 overflow-y-auto px-2 lg:px-3 py-2 space-y-0.5">
-        {subItems.map((sub) => {
-          const SubIcon = sub.icon
-          return (
             <a
-              key={sub.id}
-              href={sub.href}
-              className={cn(
-                "flex items-start transition-colors border-l-4",
-                "px-2 lg:px-3 py-2 gap-x-3",
-                "hover:bg-ui-background border-transparent hover:border-interactive",
-              )}
+              href={homePath}
+              className="flex items-center justify-center gap-2 w-full px-3 py-2 bg-interactive hover:bg-interactive-hover text-white text-sm font-medium transition-colors"
             >
-              <SubIcon className="h-4 w-4 flex-shrink-0 mt-0.5" style={{ color: "#4589ff" }} />
-              <div className="min-w-0 flex-1">
-                <p className="text-sm font-medium text-text-primary leading-snug truncate">{sub.name}</p>
-                {sub.description && (
-                  <p className="text-xs text-text-secondary leading-relaxed mt-0.5 line-clamp-2">{sub.description}</p>
-                )}
-              </div>
+              <Home className="h-4 w-4 flex-shrink-0" />
+              Tableau de bord
             </a>
-          )
-        })}
-      </nav>
+          </div>
+
+          {/* Doc / resource links */}
+          {defaultLinks.length > 0 && (
+            <nav className="flex-1 overflow-y-auto px-2 lg:px-3 py-3 space-y-0.5">
+              <p className="px-2 pb-1 text-[10px] font-semibold text-text-secondary uppercase tracking-widest">
+                Ressources
+              </p>
+              {defaultLinks.map((link, i) => {
+                const LinkIcon = link.icon ?? ExternalLink
+                const isExternal = link.href.startsWith("http://") || link.href.startsWith("https://")
+                return (
+                  <a
+                    key={i}
+                    href={link.href}
+                    {...(isExternal ? { target: "_blank", rel: "noopener noreferrer" } : {})}
+                    className="flex items-start gap-x-3 px-2 lg:px-3 py-2 border-l-4 border-transparent hover:bg-ui-background hover:border-interactive transition-colors"
+                  >
+                    <LinkIcon className="h-4 w-4 flex-shrink-0 mt-0.5 text-text-secondary" />
+                    <div className="min-w-0 flex-1">
+                      <p className="text-sm font-medium text-text-primary leading-snug flex items-center gap-1.5">
+                        {link.label}
+                        {isExternal && <ExternalLink className="h-3 w-3 text-text-secondary flex-shrink-0" />}
+                      </p>
+                      {link.description && (
+                        <p className="text-xs text-text-secondary leading-relaxed mt-0.5 line-clamp-2">{link.description}</p>
+                      )}
+                    </div>
+                  </a>
+                )
+              })}
+            </nav>
+          )}
+        </div>
+      )}
     </div>
   )
 }

package/components/layout/sidebar/main-sidebar.tsx

@@ -7,7 +7,9 @@ import { Logo } from "../../ui/logo"
 import { type MainMenuItem, type SidebarMenus } from "./data"
 import { X, Menu, ChevronDown, ChevronRight } from "lucide-react"
 import { useIsMobile } from "../../../hooks/use-mobile"
-import { MainSidebarFlyout } from "./main-sidebar-flyout"
+import { MainSidebarFlyout, type FlyoutDefaultLink } from "./main-sidebar-flyout"
+
+export type { FlyoutDefaultLink }
 
 export type SidebarMode = 'expanded' | 'minimized' | 'hidden'
 
@@ -21,6 +23,9 @@ interface MainSidebarProps {
   onSecondarySidebarOpen?: () => void
   sidebarMenus?: SidebarMenus
   mainMenuItems?: MainMenuItem[]
+  flyoutDefaultTitle?:        string
+  flyoutDefaultDescription?:  string
+  flyoutDefaultLinks?:        FlyoutDefaultLink[]
 }
 
 export function MainSidebar({
@@ -33,6 +38,9 @@ export function MainSidebar({
   onSecondarySidebarOpen,
   sidebarMenus = {},
   mainMenuItems = [],
+  flyoutDefaultTitle,
+  flyoutDefaultDescription,
+  flyoutDefaultLinks = [],
 }: MainSidebarProps) {
   const isMinimized = mode === 'minimized'
   const isMobile = useIsMobile()
@@ -66,6 +74,40 @@ export function MainSidebar({
     setHoveredItem(item)
   }
 
+  const applyTemplateVars = (href: string): string => {
+    // Guard: SSR has no window/localStorage
+    if (typeof window === "undefined") return href
+    try {
+      const projRaw = localStorage.getItem("current-business-unit")
+      const envRaw  = localStorage.getItem("current-environment")
+      // Parse carefully: value may be null (not yet set), invalid JSON, or a non-object
+      const currentProject = typeof projRaw === "string"
+        ? (JSON.parse(projRaw)?.state?.currentProject ?? "")
+        : ""
+      const currentEnv = typeof envRaw === "string"
+        ? (JSON.parse(envRaw)?.state?.currentEnv ?? "")
+        : ""
+      // Only replace when the value is a non-empty string; if empty, leave the
+      // placeholder in the URL so the link is visibly broken rather than silently
+      // producing a protocol-relative URL (e.g. "//path") that navigates incorrectly.
+      let result = href
+      if (currentProject && typeof currentProject === "string") {
+        result = result.replace(/\$\{currentProject\}/g, currentProject)
+      }
+      if (currentEnv && typeof currentEnv === "string") {
+        result = result.replace(/\$\{currentEnv\}/g, currentEnv)
+      }
+      return result
+    } catch {
+      // localStorage blocked (Safari private mode), JSON parse error, etc. — return unchanged
+      return href
+    }
+  }
+
+  const safeUrlPathname = (url: string): string => {
+    try { return new URL(url).pathname } catch { return "/" }
+  }
+
   const handleMenuClick = (item: MainMenuItem) => {
     const menuId = item.id
     const hasSubMenu = (sidebarMenus[menuId]?.length ?? 0) > 0
@@ -81,24 +123,34 @@ export function MainSidebar({
     if (!isMinimized && onSecondarySidebarOpen) onSecondarySidebarOpen()
 
     if (item.href) {
-      const path = item.href.startsWith('http://') || item.href.startsWith('https://')
-        ? new URL(item.href).pathname
-        : item.href
-      window.location.href = path
+      try {
+        const resolved = applyTemplateVars(item.href)
+        const path = resolved.startsWith('http://') || resolved.startsWith('https://')
+          ? safeUrlPathname(resolved)
+          : resolved
+        window.location.href = path
+      } catch {
+        // navigation error — don't crash the sidebar
+      }
     }
 
     if (!isMinimized) onToggle()
   }
 
   const buildSubItemHref = (menuId: string, href: string): string => {
-    if (href.startsWith('http://') || href.startsWith('https://')) {
-      return new URL(href).pathname
+    try {
+      const resolved = applyTemplateVars(href)
+      if (resolved.startsWith('http://') || resolved.startsWith('https://')) {
+        return safeUrlPathname(resolved)
+      }
+      const cleanHref = resolved.replace(/^\//, '')
+      const cleanBase = (main_base_url.startsWith('http://') || main_base_url.startsWith('https://'))
+        ? safeUrlPathname(main_base_url).replace(/\/$/, '')
+        : (main_base_url ?? "").replace(/\/$/, '')
+      return `${cleanBase}/${menuId}/${cleanHref}`
+    } catch {
+      return "/"
     }
-    const cleanHref = href.replace(/^\//, '')
-    const cleanBase = (main_base_url.startsWith('http://') || main_base_url.startsWith('https://'))
-      ? new URL(main_base_url).pathname.replace(/\/$/, '')
-      : main_base_url.replace(/\/$/, '')
-    return `${cleanBase}/${menuId}/${cleanHref}`
   }
 
   const handleSubMenuClick = (e: React.MouseEvent, menuId: string, href: string) => {
@@ -157,9 +209,14 @@ export function MainSidebar({
               const isFlyActive  = hoveredItem?.id === item.id
 
               const itemHref = !hasSubMenu && item.href
-                ? (item.href.startsWith('http://') || item.href.startsWith('https://')
-                    ? new URL(item.href).pathname
-                    : item.href)
+                ? (() => {
+                    try {
+                      const resolved = applyTemplateVars(item.href)
+                      return resolved.startsWith('http://') || resolved.startsWith('https://')
+                        ? safeUrlPathname(resolved)
+                        : resolved
+                    } catch { return item.href }
+                  })()
                 : undefined
 
               const itemCls = cn(
@@ -231,15 +288,23 @@ export function MainSidebar({
         </div>
 
         {/* ── Flyout — desktop only, flex child to the right of nav ─────────── */}
-        {!isMobile && hoveredItem && (
+        {/* Shown when the sidebar is open (default state) or an item is hovered (sub-menu state) */}
+        {!isMobile && (hoveredItem || (isOpen && !isMinimized)) && (
           <MainSidebarFlyout
             item={hoveredItem}
-            subItems={(sidebarMenus[hoveredItem.id] ?? []).map(sub => ({
-              ...sub,
-              href: buildSubItemHref(hoveredItem.id, sub.href),
-            }))}
+            subItems={hoveredItem
+              ? (sidebarMenus[hoveredItem.id] ?? []).map(sub => ({
+                  ...sub,
+                  href: buildSubItemHref(hoveredItem.id, sub.href),
+                }))
+              : []
+            }
             onMouseEnter={cancelCloseTimer}
             onMouseLeave={startCloseTimer}
+            main_base_url={main_base_url}
+            defaultTitle={flyoutDefaultTitle}
+            defaultDescription={flyoutDefaultDescription}
+            defaultLinks={flyoutDefaultLinks}
           />
         )}
 

package/package.json

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