@parhelia/core 0.1.12889 → 0.1.12891

package/dist/editor/client/EditorShell.js

@@ -71,6 +71,13 @@ import { FeatureGate, LicenseFeatures, LicenseProvider, LicenseOverlay, } from "
 const SIDEBAR_NONE_SENTINEL = "none";
 const sidebarUrlValue = (ids) => ids.length ? ids.join(",") : SIDEBAR_NONE_SENTINEL;
 // moved to hooks/useQuota
+function getInitialOpenSidebars(configuration, workspaceId) {
+    const workspace = configuration.editor.workspaces?.find((candidate) => candidate.id === workspaceId);
+    if (workspace?.supportsSidebars && workspace.defaultSidebars?.length) {
+        return [...workspace.defaultSidebars];
+    }
+    return [...(configuration.editor.defaultOpenSidebars ?? [])];
+}
 function isUsableTextSelectionRange(range) {
     return !!(range &&
         range.text?.trim() &&
@@ -215,14 +222,13 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     // (Next.js App Router uses startTransition internally for URL changes)
     const rawWorkspace = useMemo(() => searchParams.get("workspace"), [searchParams]);
     const isReviewsSidebarRequest = rawWorkspace === "reviews";
-    const [workspaceId, setWorkspaceId] = useState(
-    // If workspace=reviews, use "editor" workspace (reviews is a sidebar, not a workspace)
-    isReviewsSidebarRequest
+    const initialWorkspaceId = isReviewsSidebarRequest
         ? "editor"
         : (rawWorkspace ??
             configuration.editor.defaultWorkspace ??
             configuration.editor.workspaces?.[0]?.id ??
-            "home"));
+            "home");
+    const [workspaceId, setWorkspaceId] = useState(initialWorkspaceId);
     const workspaceIdRef = useRef(workspaceId);
     useEffect(() => {
         workspaceIdRef.current = workspaceId;
@@ -240,7 +246,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                     : sidebarParam.split(",").filter(Boolean);
         }
         else {
-            sidebars = [...(configuration.editor.defaultOpenSidebars ?? [])];
+            sidebars = getInitialOpenSidebars(configuration, initialWorkspaceId);
         }
         // If workspace=reviews was requested, ensure reviews sidebar is open
         if (isReviewsSidebarRequest && !sidebars.includes("reviews")) {
@@ -3530,8 +3536,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         };
     }, [getSidebarsForWorkspace, workspaceId]);
     // Track the last workspaceId this effect saw so we only auto-apply
-    // `currentWorkspace.defaultSidebars` on an actual workspace change. On initial mount
-    // the user's explicit empty state (sidebar=none in URL) must not be overwritten.
+    // `currentWorkspace.defaultSidebars` on actual workspace changes. Initial
+    // workspace defaults are applied by the openSidebars state initializer, while
+    // explicit sidebar URL state such as sidebar=none is still respected.
     const prevWorkspaceIdForDefaultsRef = useRef(null);
     useEffect(() => {
         if (!currentWorkspace.supportsSidebars) {
@@ -5303,7 +5310,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                     setTimeout(() => {
                         setShowFullscreenHint(false);
                     }, 600);
-                }, "data-testid": "fullscreen-hint-overlay", children: _jsxs("div", { className: "bg-neutral-grey-100/60 fixed top-6 left-1/2 -translate-x-1/2 transform rounded-full px-6 py-2.5 text-sm font-medium text-white shadow-2xl backdrop-blur-md transition-all duration-500", children: ["Press", " ", _jsx("kbd", { className: "badge-pad-sm mx-1 rounded bg-white/20 text-xs font-semibold", children: "Ctrl + F11" }), " ", "to exit fullscreen"] }) }))] })) : (_jsxs(_Fragment, { children: [_jsx(EditorChrome, { className: className, currentWorkspace: currentWorkspace, centerPanelView: centerPanelView, editContext: editContext, showAgentsPanel: showAgentsPanel, handleSetShowAgentsPanel: handleSetShowAgentsPanel, workspaceId: workspaceId }), isTourActive && configuration.activeTour && (_jsx(Tour, { tourStopCallback: () => {
+                }, "data-testid": "fullscreen-hint-overlay", children: _jsxs("div", { className: "bg-neutral-grey-100/60 fixed top-6 left-1/2 -translate-x-1/2 transform rounded-full px-6 py-2.5 text-sm font-medium text-white shadow-2xl backdrop-blur-md transition-all duration-500", children: ["Press", " ", _jsx("kbd", { className: "badge-pad-sm mx-1 rounded bg-white/20 text-xs font-medium", children: "Ctrl + F11" }), " ", "to exit fullscreen"] }) }))] })) : (_jsxs(_Fragment, { children: [_jsx(EditorChrome, { className: className, currentWorkspace: currentWorkspace, centerPanelView: centerPanelView, editContext: editContext, showAgentsPanel: showAgentsPanel, handleSetShowAgentsPanel: handleSetShowAgentsPanel, workspaceId: workspaceId }), isTourActive && configuration.activeTour && (_jsx(Tour, { tourStopCallback: () => {
                     setIsTourActive(false);
                     // Remove tour state from URL
                     // Use history.replaceState instead of router.replace to avoid triggering React navigation