@parhelia/core 0.1.12902 → 0.1.12904

package/dist/editor/client/EditorShell.js

@@ -1,20 +1,18 @@
 "use client";
 import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
-import React, { useState, useEffect, useRef, useCallback, useMemo, startTransition, } from "react";
+import React, { useState, useEffect, useRef, useCallback, useSyncExternalStore, useMemo, startTransition, } from "react";
 import { toast } from "sonner";
 import { EditContextProvider, FieldsEditContextProvider, OperationsContextProvider, useEditContext, } from "./editContext";
-import { aggregateModifiedFieldsFromStores, areModifiedFieldSnapshotsEqual, FieldModificationStore, } from "./fieldModificationStore";
+import { fieldModificationStore } from "./fieldModificationStore";
 import { SplashScreenAgentProvider } from "../../splash-screen/SplashScreenAgentContext";
 import { useRouter, useSearchParams, usePathname } from "./navigation";
 import { findComponent, getComponentById } from "../componentTreeHelper";
 import { getOperationsContext } from "./operations";
 import { handleErrorResult } from "./helpers";
-import { isManualShowMeFocusTransfer, isManualShowMeTarget, } from "./manualShowMeFocus";
 import { executeFieldAction as executeFieldServerAction, connectSocket, getEditHistory, getRunningOperations, reconnectSession, releaseFieldLocks, validateItems, } from "../services/editService";
 import { emitAgentDocumentsChanged } from "../services/agentService";
 import { onSocketReconnect as onAgentSocketReconnect } from "../services/agentSubscriptionRegistry";
 import { useEditorWebSocket } from "./hooks/useEditorWebSocket";
-import { getEditorRealtime } from "./realtime/realtimeTransport";
 import { createEditorSocketDiagnostics, } from "./socketDiagnostics";
 import { localStorageService } from "../services/localStorageService";
 import { useSocketMessageHandler } from "./hooks/useSocketMessageHandler";
@@ -22,7 +20,6 @@ import "react-json-view-lite/dist/index.css";
 import { MediaSelector, } from "../media-selector/MediaSelector";
 import { getComponentCommands } from "../commands/componentCommands";
 import { getLanguagesAndVersions, } from "../services/contentService";
-import { getAllFavorites } from "../services/favouritesService";
 import ConfirmationDialog from "../ConfirmationDialog";
 import { getItemDescriptor } from "../utils";
 import { EditContextMenu } from "../ContextMenu";
@@ -30,13 +27,10 @@ import { InlineAiTrigger } from "../ai/InlineAiTrigger";
 import { EditorFormHintPopover } from "../page-viewer/EditorFormHintPopover";
 import { FieldEditorPopup } from "../FieldEditorPopup";
 import { ConcurrentUserLimitDialog } from "../ConcurrentUserLimitDialog";
-import { SessionExpiredOverlay } from "./SessionExpiredOverlay";
 import { post } from "../services/serviceHelper";
-import { CONTENT_TREE_SIDEBAR_ID } from "../sidebar/contentTreeConstants";
 import { PageViewerFrame } from "../page-viewer/PageViewerFrame";
 import { useItemsRepository } from "./itemsRepository";
 import { Spinner } from "../ui/Spinner";
-import { createSettleRegistry } from "../ui/animationSettle";
 import { cleanId } from "../utils/id-helper";
 import { useDebouncedCallback } from "use-debounce";
 import { useKeyboardNavigation } from "../utils/keyboardNavigation";
@@ -44,14 +38,11 @@ import { Toaster } from "../../components/ui/sonner";
 import { Tour } from "../../tour/Tour";
 import { GuidanceOverlay } from "../ai/GuidanceOverlay";
 import { AgentDialogHandler } from "../ai/dialogs";
-import { setActiveAgentConversation } from "../ai/useActiveAgentConversation";
 import { usePageViewContext, } from "../page-viewer/pageViewContext";
 import { QuickItemSwitcher } from "../QuickItemSwitcher";
 import { useEditorSlotContext, } from "../views/editorSlotContext";
 import { getComments, getAvailableCommentTags, } from "../services/reviewsService";
 import { useReviews } from "../reviews/useReviews";
-import { isSuggestedEditRemoved } from "../reviews/suggestedEditState";
-import { COMMENT_TRANSIENT_ELEMENT_KEY, COMMENT_TRANSIENT_SELECTION_RECT, } from "../reviews/commentTransientSelection";
 import uuid from "react-uuid";
 import { flushSync } from "react-dom";
 import { getSuggestedEdits } from "../services/suggestedEditsService";
@@ -59,44 +50,19 @@ import { requestQuota } from "../services/aiService";
 import { useQuota } from "./hooks/useQuota";
 import { useEditorUrlSync } from "./hooks/useEditorUrlSync";
 import { useMediaQuery } from "./hooks/useMediaQuery";
+import { FullscreenControls } from "./ui/FullscreenControls";
 import { EditorChrome } from "./ui/EditorChrome";
 import { DevModeIndicator } from "./ui/DevModeIndicator";
 import { useWorkbox } from "./hooks/useWorkbox";
 import { useMediaSelector } from "./hooks/useMediaSelector";
 import { useGlobalEditorKeyDown } from "./hooks/useGlobalEditorKeyDown";
 import { useStartupChecks } from "../settings/status/index";
-import { EditorLoadingOverlay } from "../EditorLoadingOverlay";
 import { FeatureGate, LicenseFeatures, LicenseProvider, LicenseOverlay, } from "../../licensing";
 // Sentinel written to the `sidebar` URL param when the user has explicitly closed
 // every sidebar. Distinguishes "no preference yet" (param absent) from "user wants
 // nothing open" (param present with this value) so reload can honor the user's intent.
 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() &&
-        typeof range.startOffset === "number" &&
-        typeof range.endOffset === "number" &&
-        range.startOffset !== range.endOffset &&
-        range.fieldId &&
-        range.itemId);
-}
-function selectionRangeMatchesField(range, field) {
-    if (!range || !field)
-        return false;
-    return (cleanId(range.fieldId) === cleanId(field.fieldId) &&
-        cleanId(range.itemId) === cleanId(field.item.id) &&
-        (!range.language || range.language === field.item.language) &&
-        (typeof range.version !== "number" || range.version === field.item.version));
-}
 function reconcileSlotSettingsState(previous, slots, defaults) {
     const next = {};
     let changed = false;
@@ -139,7 +105,7 @@ function AgentsSlotContextBridge({ slot }) {
 function AgentsSlotContextBridgeHost({ slots }) {
     return (_jsx(_Fragment, { children: slots.map((slot) => (_jsx(AgentsSlotContextBridge, { slot: slot }, `agents-slot-bridge-${slot.slotId}`))) }));
 }
-export function EditorShell({ configuration, className, item: loadItemDescriptor, sessionId, userInfo, userPreferences, initialLicenseStatus, initialLicenseStatusLoaded, parheliaSettings, parheliaSettingsLoaded, setUserPreferences, children, }) {
+export function EditorShell({ configuration, className, item: loadItemDescriptor, sessionId, userInfo, userPreferences, initialLicenseStatus, initialLicenseStatusLoaded, parheliaSettings, setUserPreferences, children, }) {
     const router = useRouter();
     const pathname = usePathname();
     const searchParams = useSearchParams();
@@ -160,15 +126,12 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const contextMenuRef = useRef(null);
     const editContextRef = useRef(undefined);
     const [currentOverlay, setCurrentOverlay] = useState();
-    const [contextMenuSource, setContextMenuSource] = useState();
     const [item, setItem] = useState();
     const [focusedField, setFocusedField] = useState();
     const focusedFieldRef = useRef(undefined);
     const [selectedRange, setSelectedRange] = useState();
-    const lastTextSelectionRangeRef = useRef(undefined);
     const [validating, setValidating] = useState(false);
     const [inserting, setInserting] = useState();
-    const insertingSequenceRef = useRef(0);
     const [showFullscreenHint, setShowFullscreenHint] = useState(false);
     // Fullscreen is a shell-level UI mode. Keep it in EditorShell state so the shell can rerender
     // immediately when toggled, while still syncing the active slot's PageViewContext.fullscreen.
@@ -213,7 +176,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             return [];
         }
     });
-    const [remoteCarets, setRemoteCarets] = useState({});
     // Block editing until the server acknowledges our client-info for the current item
     const [isClientInfoAcknowledged, setIsClientInfoAcknowledged] = useState(false);
     if (typeof window !== "undefined")
@@ -225,13 +187,14 @@ 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 initialWorkspaceId = isReviewsSidebarRequest
+    const [workspaceId, setWorkspaceId] = useState(
+    // If workspace=reviews, use "editor" workspace (reviews is a sidebar, not a workspace)
+    isReviewsSidebarRequest
         ? "editor"
         : (rawWorkspace ??
             configuration.editor.defaultWorkspace ??
             configuration.editor.workspaces?.[0]?.id ??
-            "home");
-    const [workspaceId, setWorkspaceId] = useState(initialWorkspaceId);
+            "home"));
     const workspaceIdRef = useRef(workspaceId);
     useEffect(() => {
         workspaceIdRef.current = workspaceId;
@@ -249,7 +212,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                     : sidebarParam.split(",").filter(Boolean);
         }
         else {
-            sidebars = getInitialOpenSidebars(configuration, initialWorkspaceId);
+            sidebars = [...(configuration.editor.defaultOpenSidebars ?? [])];
         }
         // If workspace=reviews was requested, ensure reviews sidebar is open
         if (isReviewsSidebarRequest && !sidebars.includes("reviews")) {
@@ -387,10 +350,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const [componentDesignerRendering, setComponentDesignerRendering] = useState();
     const [insertMode, setInsertMode] = useState(false);
     const [ignoreBlur, setIgnoreBlur] = useState(false);
-    // Timestamp of the most recent pointer-down on a User Manual "Show me" control.
-    // Used by the field-blur guard so clicking a field-gated Show me button keeps
-    // the focused field (and thus the `.focused-field` anchor + field actions).
-    const manualShowMePointerDownAtRef = useRef(0);
     const [currentItemDescriptor, setCurrentItemDescriptor] = useState();
     const currentItemDescriptorRef = useRef(undefined);
     const latestGlobalLoadKeyRef = useRef(null);
@@ -413,25 +372,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         activeSlotIdRef.current = activeSlotId;
     }, [activeSlotId]);
     const [slotContexts, setSlotContexts] = useState(() => new Map());
-    const fieldModificationStoresBySlotRef = useRef(new Map());
-    const [fieldModificationStoreRegistryRevision, setFieldModificationStoreRegistryRevision,] = useState(0);
-    const getFieldModificationStoreForSlot = useCallback((slotId) => {
-        let store = fieldModificationStoresBySlotRef.current.get(slotId);
-        if (!store) {
-            store = new FieldModificationStore();
-            fieldModificationStoresBySlotRef.current.set(slotId, store);
-        }
-        return store;
-    }, []);
-    const getFieldModificationStores = useCallback(() => {
-        return Array.from(fieldModificationStoresBySlotRef.current.values());
-    }, []);
-    const ensureFieldModificationStoreRegistered = useCallback((slotId) => {
-        if (fieldModificationStoresBySlotRef.current.has(slotId))
-            return;
-        fieldModificationStoresBySlotRef.current.set(slotId, new FieldModificationStore());
-        setFieldModificationStoreRegistryRevision((previous) => previous + 1);
-    }, []);
     const promptSessionReconnect = useCallback((reason) => {
         const description = reason && reason !== "session-revoked"
             ? `${reason}. Click reconnect to continue in this browser.`
@@ -453,8 +393,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                         (typeof result.summary === "string" && result.summary.trim()) ||
                         "";
                     toast.error("Reconnect failed", {
-                        description: serverDetail ||
-                            "Could not claim this browser session. Try again.",
+                        description: serverDetail || "Could not claim this browser session. Try again.",
                     });
                 },
             },
@@ -501,32 +440,13 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     // The very first URL sync after initial load should replaceState (not pushState)
     // so the initial bare URL isn't left as a dead-end history entry.
     const isFirstUrlSyncRef = useRef(true);
+    const [inlineEditingFieldElement, setInlineEditingFieldElement] = useState();
     const [lockedField, setLockedField] = useState();
     const [itemLanguages, setItemLanguages] = useState([]);
     const [itemVersions, setItemVersions] = useState([]);
     const [isLoadingVersions, setIsLoadingVersions] = useState(false);
-    const [modifiedFields, setModifiedFields] = useState([]);
-    useEffect(() => {
-        const stores = getFieldModificationStores();
-        const recomputeModifiedFields = () => {
-            const next = aggregateModifiedFieldsFromStores(stores);
-            setModifiedFields((previous) => areModifiedFieldSnapshotsEqual(previous, next) ? previous : next);
-        };
-        const unsubscribes = stores.map((store) => store.subscribeAll(recomputeModifiedFields));
-        recomputeModifiedFields();
-        return () => {
-            unsubscribes.forEach((unsubscribe) => unsubscribe());
-        };
-    }, [
-        editorSlots,
-        fieldModificationStoreRegistryRevision,
-        getFieldModificationStores,
-    ]);
+    const modifiedFields = useSyncExternalStore((onStoreChange) => fieldModificationStore.subscribeAll(onStoreChange), () => fieldModificationStore.getAllFields(), () => fieldModificationStore.getAllFields());
     const [comments, setComments] = useState([]);
-    // True once loadComments has succeeded for the current item. Destructive
-    // decisions that depend on a suggestion's comment thread (e.g. auto-deleting
-    // a reverted no-op suggestion) must not run on the initial empty array.
-    const [commentsLoaded, setCommentsLoaded] = useState(false);
     const [suggestedEdits, setSuggestedEdits] = useState([]);
     const [showSuggestedEdits, setShowSuggestedEdits] = useState(false);
     const [showSuggestedEditsDiff, setShowSuggestedEditsDiff] = useState(false);
@@ -545,7 +465,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         localStorageService.setItem("editor.showResolvedComments", showResolvedComments);
     }, [showResolvedComments]);
     const [selectedComment, setSelectedComment] = useState();
-    const [selectedSuggestedEdit, setSelectedSuggestedEdit] = useState();
     const [browseHistory, setBrowseHistory] = useState(() => {
         // Convert UserBrowseHistoryEntry[] to HistoryEntry[]
         if (!userInfo.browseHistory)
@@ -591,79 +510,13 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     // moved into useWorkbox
     // Initialize tour state from URL parameter to survive navigation/remounts
     // Memoize to avoid triggering Router state updates during render
-    const tourRestoredFromUrl = useMemo(() => Boolean(configuration.activeTour) &&
-        searchParams.get("tour") === "active", [configuration.activeTour, searchParams]);
+    const tourRestoredFromUrl = useMemo(() => searchParams.get("tour") === "active", [searchParams]);
     const [isTourActive, setIsTourActive] = useState(tourRestoredFromUrl);
     // Track if tour was restored from URL to skip prepareAction on initial render
     const tourRestoredRef = useRef(tourRestoredFromUrl);
-    const [mode, setModeState] = useState("edit");
-    const modeRef = useRef("edit");
-    modeRef.current = mode;
-    const modeChangeParticipantsRef = useRef(new Set());
-    const loadSuggestedEditsRef = useRef(async () => { });
-    const [modeTransitionRevision, setModeTransitionRevision] = useState(0);
+    const [mode, setMode] = useState("edit");
     const [previewDate, setPreviewDate] = useState();
     const [layoutMode, setLayoutMode] = useState("final");
-    const registerModeChangeParticipant = useCallback((participant) => {
-        modeChangeParticipantsRef.current.add(participant);
-        return () => {
-            modeChangeParticipantsRef.current.delete(participant);
-        };
-    }, []);
-    const changeMode = useCallback(async (nextMode, options) => {
-        const previousMode = modeRef.current;
-        if (nextMode === previousMode)
-            return;
-        const participants = Array.from(modeChangeParticipantsRef.current);
-        for (const participant of participants) {
-            await participant.beforeModeChange?.(nextMode, previousMode, options);
-        }
-        for (const participant of participants) {
-            participant.clearInlineDedupe?.(nextMode, previousMode, options);
-        }
-        flushSync(() => {
-            modeRef.current = nextMode;
-            setModeState(nextMode);
-            setModeTransitionRevision((revision) => revision + 1);
-            if (nextMode !== "preview") {
-                setPreviewDate(undefined);
-            }
-            if (nextMode === "suggestions") {
-                setShowSuggestedEdits(true);
-                setWorkspaceId("editor");
-                setShowComments(true);
-            }
-        });
-        if (nextMode === "suggestions") {
-            void loadSuggestedEditsRef.current();
-        }
-    }, []);
-    const setMode = useCallback((nextMode) => {
-        const resolvedMode = typeof nextMode === "function" ? nextMode(modeRef.current) : nextMode;
-        void changeMode(resolvedMode);
-    }, [changeMode]);
-    const closeFeedbackSuggestionsMode = useCallback(() => {
-        setShowComments(false);
-        setShowSuggestedEdits(false);
-        setShowSuggestedEditsDiff(false);
-        if (modeRef.current === "suggestions") {
-            void changeMode("edit", { reason: "feedback-sidebar-close" });
-        }
-    }, [changeMode]);
-    const activateCommentsModeForWriting = useCallback(async (reason) => {
-        setShowComments(true);
-        setShowSuggestedEdits(false);
-        setShowSuggestedEditsDiff(false);
-        if (modeRef.current === "suggestions") {
-            await changeMode("edit", { reason });
-        }
-    }, [changeMode]);
-    const workspaceShowsModeControls = !!configuration.editor.workspaces?.find((workspace) => workspace.id === workspaceId)?.toolbarFactory;
-    useEffect(() => {
-        if (workspaceShowsModeControls || mode !== "suggestions")
-            return;
-        void changeMode("edit", { reason: "mode-controls-hidden" });
-    }, [changeMode, mode, workspaceId, workspaceShowsModeControls]);
     useEffect(() => {
         if (mode !== "preview") {
             setPreviewDate(undefined);
@@ -682,9 +535,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         return userPreferences.showAgentsPanel ?? false;
     });
     const [editorFormHiddenDefault, setEditorFormHiddenDefault] = useState(userPreferences.editorFormHidden ?? false);
-    // Stable registry that lets layout owners (Splitter panels) report when a
-    // reveal surface finished animating, and lets callers await that signal.
-    const settleRegistry = useMemo(() => createSettleRegistry(), []);
     useEffect(() => {
         setSlotSettingsById((prev) => reconcileSlotSettingsState(prev, editorSlots, {
             showComponentNavigator: showComponentNavigatorDefault,
@@ -702,10 +552,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const [selectedHelpSectionId, setSelectedHelpSectionId] = useState(null);
     const [showAgentsWorkspaceEditor, setShowAgentsWorkspaceEditor] = useState(false);
     const [selectedAgentsWorkspaceAgentId, setSelectedAgentsWorkspaceAgentId] = useState(null);
-    const selectedAgentsWorkspaceAgentIdRef = useRef(selectedAgentsWorkspaceAgentId);
-    useEffect(() => {
-        selectedAgentsWorkspaceAgentIdRef.current = selectedAgentsWorkspaceAgentId;
-    }, [selectedAgentsWorkspaceAgentId]);
     const [activeEditorTab, setActiveEditorTab] = useState(null);
     const showLayoutParam = useMemo(() => searchParams.get("showLayout") ??
         searchParams.get("showLayoutComponents"), [searchParams]);
@@ -776,6 +622,19 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             setSelectedHelpSectionId(helpParam === "contents" || helpParam === "true" ? null : helpParam);
         }
     }, [searchParams, isInitialLoad]);
+    useEffect(() => {
+        if (mode === "suggestions") {
+            // Ensure we're in the editor workspace and open the feedback sidebar
+            setWorkspaceId("editor");
+            // Open the feedback sidebar if not already open
+            setOpenSidebars((prev) => {
+                if (!prev.includes("comments")) {
+                    return [...prev, "comments"];
+                }
+                return prev;
+            });
+        }
+    }, [mode]);
     useEffect(() => {
         if (focusedField &&
             selection.length > 0 &&
@@ -785,23 +644,11 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     }, [selection]);
     useEffect(() => {
         focusedFieldRef.current = focusedField;
-        const lastTextSelection = lastTextSelectionRangeRef.current;
-        if (focusedField &&
-            lastTextSelection &&
-            (focusedField.fieldId !== lastTextSelection.fieldId ||
-                focusedField.item.id !== lastTextSelection.itemId)) {
-            lastTextSelectionRangeRef.current = undefined;
-        }
     }, [focusedField]);
-    useEffect(() => {
-        if (isUsableTextSelectionRange(selectedRange)) {
-            lastTextSelectionRangeRef.current = selectedRange;
-        }
-    }, [selectedRange]);
     // Track the current page item descriptor for itemsRepository
     const [currentPageItem, setCurrentPageItem] = useState();
     // Create itemsRepository with current page item (will be updated when page loads)
-    const itemsRepository = useItemsRepository(addRecentEdit, currentPageItem, getFieldModificationStores);
+    const itemsRepository = useItemsRepository(addRecentEdit, currentPageItem);
     const fallbackPageViewContext = usePageViewContext({
         pageItemDescriptor: currentItemDescriptor,
         itemsRepository,
@@ -835,7 +682,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     }, []);
     const slotContextsRef = useRef(slotContexts);
     const lastActiveSlotContextRef = useRef(null);
-    slotContextsRef.current = slotContexts;
+    useEffect(() => {
+        slotContextsRef.current = slotContexts;
+    }, [slotContexts]);
     const getActiveSlotContext = useCallback(() => {
         const activeId = activeSlotIdRef.current;
         if (!activeId)
@@ -868,7 +717,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const isEditorFormHiddenForSlot = useCallback((slotId) => {
         if (!slotId)
             return editorFormHiddenDefault;
-        return (slotSettingsById[slotId]?.editorFormHidden ?? editorFormHiddenDefault);
+        return slotSettingsById[slotId]?.editorFormHidden ?? editorFormHiddenDefault;
     }, [editorFormHiddenDefault, slotSettingsById]);
     // Sync global compareMode from the active slot's compareMode for URL syncing
     useEffect(() => {
@@ -924,57 +773,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         socketMessageListeners.current.add(callback);
         return () => socketMessageListeners.current.delete(callback);
     }, []);
-    useEffect(() => {
-        const removeListener = addSocketMessageListener((message) => {
-            if (message.type !== "caret-position")
-                return;
-            const payload = message.payload;
-            const sessionIdValue = payload?.sessionId?.trim();
-            if (!sessionIdValue || sessionIdValue === sessionId)
-                return;
-            setRemoteCarets((current) => {
-                if (payload?.offset == null) {
-                    if (!current[sessionIdValue])
-                        return current;
-                    const next = { ...current };
-                    delete next[sessionIdValue];
-                    return next;
-                }
-                return {
-                    ...current,
-                    [sessionIdValue]: {
-                        sessionId: sessionIdValue,
-                        fieldId: payload.fieldId,
-                        item: payload.item,
-                        offset: payload.offset,
-                        value: payload.value,
-                    },
-                };
-            });
-        });
-        return () => {
-            try {
-                removeListener();
-            }
-            catch { }
-        };
-    }, [addSocketMessageListener, sessionId]);
-    useEffect(() => {
-        const activeSessionIds = new Set(activeSessions.map((s) => s.sessionId));
-        setRemoteCarets((current) => {
-            let changed = false;
-            const next = {};
-            Object.entries(current).forEach(([caretSessionId, caret]) => {
-                if (activeSessionIds.has(caretSessionId)) {
-                    next[caretSessionId] = caret;
-                }
-                else {
-                    changed = true;
-                }
-            });
-            return changed ? next : current;
-        });
-    }, [activeSessions]);
     // Listen for batch completion notifications
     useEffect(() => {
         const removeListener = addSocketMessageListener((message) => {
@@ -1156,9 +954,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         });
     }, [isInitialLoad, searchParams]);
     const startTour = useCallback(() => {
-        if (!configuration.activeTour) {
-            return;
-        }
         setIsTourActive(true);
         // Persist tour state to URL so it survives navigation/remounts
         // Use history.replaceState instead of router.replace to avoid triggering React navigation
@@ -1166,7 +961,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         params.set("tour", "active");
         const newUrl = `${window.location.pathname}?${params.toString()}`;
         window.history.replaceState(getCurrentHistoryState(), "", newUrl);
-    }, [configuration.activeTour, setIsTourActive]);
+    }, [setIsTourActive]);
     const isMobile = useMediaQuery("(max-width: 767px)");
     // On mobile, clear all locked sidebars and keep only the last-opened panel.
     // Handles viewport resize: desktop -> mobile unlocks everything and trims to one panel.
@@ -1301,14 +1096,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         showComponentNavigatorDefault,
         setUserPreferences,
     ]);
-    useEffect(() => {
-        if (workspaceId !== "editor")
-            return;
-        if (mode !== "edit") {
-            setEditorFormHintVisible(false);
-            setMobileEditorPanelOpenRaw(false);
-        }
-    }, [mode, workspaceId]);
     const markEditorFormHintSeen = useCallback(() => {
         setEditorFormHintSeenState(true);
         setUserPreferences({ editorFormHintSeen: true });
@@ -1338,19 +1125,17 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     }, [showHelpTerminal]);
     const toggleHelpTerminal = useCallback((options) => {
         const isCurrentlyOpen = showHelpTerminal;
-        const requestedTab = options?.activeTab;
-        if (isCurrentlyOpen &&
-            (!requestedTab || requestedTab === helpTerminalActiveTab)) {
-            // Close when toggling the currently active help view.
+        if (isCurrentlyOpen && !options?.activeTab) {
+            // Close if already open and no specific tab requested
             handleSetShowHelpTerminal(false);
         }
         else {
             setHelpTerminalInitialPrompt(options?.initialPrompt);
             setHelpTerminalProfileName(options?.profileName);
-            setHelpTerminalActiveTab(requestedTab);
+            setHelpTerminalActiveTab(options?.activeTab);
             handleSetShowHelpTerminal(true);
         }
-    }, [showHelpTerminal, helpTerminalActiveTab, handleSetShowHelpTerminal]);
+    }, [showHelpTerminal, handleSetShowHelpTerminal]);
     const handleSetShowAgentsWorkspaceEditor = useCallback((value) => {
         const newValue = typeof value === "function" ? value(showAgentsWorkspaceEditor) : value;
         setShowAgentsWorkspaceEditor(newValue);
@@ -1379,9 +1164,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setSidebarStacks((prev) => normalizeSidebarStacks(newOrder, prev));
     }, []);
     const ensureSidebarPinned = useCallback((sidebarId) => {
-        if (sidebarId === CONTENT_TREE_SIDEBAR_ID) {
-            return;
-        }
         const currentPinnedSidebars = pinnedSidebarsRef.current;
         if (currentPinnedSidebars.includes(sidebarId)) {
             return;
@@ -1427,7 +1209,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 // Force a reconnect to refresh presence after several failed nudges
                 console.warn("Session presence did not recover after retries. Forcing reconnect.");
                 try {
-                    getEditorRealtime().reconnect("recover-presence");
+                    globalThis.editorSocket?.close(4000, "recover-presence");
                 }
                 catch { }
             }
@@ -1476,13 +1258,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                     }));
                 }
             })
-                .catch((error) => {
-                if (error instanceof TypeError &&
-                    error.message === "Failed to fetch") {
-                    return;
-                }
-                console.error("Keep Alive error:", error);
-            });
+                .catch((error) => console.error("Keep Alive error:", error));
         };
         const keepaliveIntervalMs = (() => {
             const param = new URLSearchParams(window.location.search).get("keepaliveIntervalMs");
@@ -1524,11 +1300,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 }
                 // Handle sidebar changes — clear when absent so state matches the URL
                 const sidebarParam = urlParams.get("sidebar");
-                const newSidebars = sidebarParam === SIDEBAR_NONE_SENTINEL
-                    ? []
-                    : sidebarParam
-                        ? sidebarParam.split(",").filter(Boolean)
-                        : [];
+                const newSidebars = sidebarParam
+                    ? sidebarParam.split(",").filter(Boolean)
+                    : [];
                 setOpenSidebars(newSidebars);
                 // Handle wizard ID changes
                 const wizardId = urlParams.get("wizardid");
@@ -1541,14 +1315,18 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 if (helpParam) {
                     setHelpTerminalActiveTab("manual");
                     setShowHelpTerminal(true);
-                    setSelectedHelpSectionId(helpParam === "contents" || helpParam === "true" ? null : helpParam);
+                    setSelectedHelpSectionId(helpParam === "contents" || helpParam === "true"
+                        ? null
+                        : helpParam);
                 }
                 else {
                     handleSetShowHelpTerminal(false);
                 }
                 // Handle mode changes — always set to avoid stale-closure mismatch
                 const urlMode = urlParams.get("mode");
-                setMode(urlMode ?? "edit");
+                if (urlMode) {
+                    setMode(urlMode);
+                }
                 // Handle fullscreen changes (shell-level state)
                 const isFullscreen = urlParams.get("fullscreen") === "true";
                 if (configuration.forceFullscreen) {
@@ -1626,9 +1404,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             return;
         }
         const tour = configuration.activeTour;
-        if (!tour) {
-            return;
-        }
         const key = tour === "default" ? "editor.tourShown" : "editor.tourShown." + tour;
         const tourShown = localStorageService.getString(key);
         if (!tourShown) {
@@ -1706,8 +1481,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const historyModeRef = useRef(historyMode);
     const itemRef = useRef(item);
     const historySearchQueryRef = useRef(historySearchQuery);
-    const showOnlyMyChangesRef = useRef(showOnlyMyChanges);
-    const filterByCurrentLanguageRef = useRef(filterByCurrentLanguage);
     useEffect(() => {
         historyModeRef.current = historyMode;
     }, [historyMode]);
@@ -1717,12 +1490,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     useEffect(() => {
         historySearchQueryRef.current = historySearchQuery;
     }, [historySearchQuery]);
-    useEffect(() => {
-        showOnlyMyChangesRef.current = showOnlyMyChanges;
-    }, [showOnlyMyChanges]);
-    useEffect(() => {
-        filterByCurrentLanguageRef.current = filterByCurrentLanguage;
-    }, [filterByCurrentLanguage]);
     // Mirror the backend LIKE filter on MainItemPath/Title/Description so that
     // ops arriving via WebSocket while a search is active don't bypass the filter.
     const matchesHistorySearchQuery = useCallback((op) => {
@@ -1815,27 +1582,21 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             allComments.sort((a, b) => a.position - b.position);
             return allComments;
         });
-        setCommentsLoaded(true);
     }, [currentItemDescriptor, searchParams]);
     // Assuming currentItemDescriptor, ui, state, handleErrorResult, and setSuggestedEdits
     // are available in your component context.
     const loadSuggestedEdits = useCallback(async () => {
         if (!item)
             return;
-        const reviewId = searchParams.get("reviewId");
-        const result = await getSuggestedEdits(item.descriptor.id, item.descriptor.language, item.descriptor.version, reviewId ?? undefined);
+        const result = await getSuggestedEdits(item.descriptor.id, item.descriptor.language, item.descriptor.version);
         if (handleErrorResult(result, ui, state))
             return;
-        // A refetch can race a locally reverted suggestion's server delete; don't
-        // resurrect suggestions the user already removed.
-        const edits = (result.data || []).filter((edit) => !isSuggestedEditRemoved(edit.id));
+        const edits = result.data || [];
         setSuggestedEdits(edits);
-    }, [item, searchParams]);
-    useEffect(() => {
-        loadSuggestedEditsRef.current = loadSuggestedEdits;
-    }, [loadSuggestedEdits]);
+    }, [item]);
     const loadFavorites = useCallback(async () => {
         try {
+            const { getAllFavorites } = await import("../services/favouritesService");
             const result = await getAllFavorites();
             if (result.data) {
                 setFavorites(result.data);
@@ -1846,9 +1607,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             setFavorites([]);
         }
     }, []);
-    const refreshFavorites = useCallback(async (opts) => {
-        await loadFavorites();
-    }, [loadFavorites]);
     const page = activePageViewContext.page;
     const currentHistoryScope = useMemo(() => {
         const currentItem = item?.descriptor;
@@ -2010,34 +1768,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             : filterByCurrentLanguage;
         const trimmedHistoryQuery = historySearchQuery.trim();
         const historyQuery = trimmedHistoryQuery.length > 0 ? trimmedHistoryQuery : undefined;
-        const requestMode = currentMode;
-        const requestSearchQuery = trimmedHistoryQuery;
-        const requestItem = item?.descriptor;
-        const requestLanguage = item?.descriptor?.language || currentItemDescriptor?.language;
-        const isStaleHistoryResponse = () => {
-            if (historyModeRef.current !== requestMode)
-                return true;
-            if ((historySearchQueryRef.current || "").trim() !== requestSearchQuery) {
-                return true;
-            }
-            if (showOnlyMyChangesRef.current !== shouldFilterBySession)
-                return true;
-            if (filterByCurrentLanguageRef.current !== shouldFilterByLanguage) {
-                return true;
-            }
-            const latestItem = itemRef.current?.descriptor;
-            if (requestMode !== "global") {
-                return (!latestItem ||
-                    !requestItem ||
-                    latestItem.id !== requestItem.id ||
-                    latestItem.language !== requestItem.language ||
-                    latestItem.version !== requestItem.version);
-            }
-            return (shouldFilterByLanguage &&
-                !!requestLanguage &&
-                !!latestItem?.language &&
-                latestItem.language !== requestLanguage);
-        };
         if (currentMode === "global") {
             // Global mode: optionally filter by session and/or language
             const currentLanguage = item?.descriptor?.language || currentItemDescriptor?.language;
@@ -2053,9 +1783,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 console.error("[EditorShell] Failed to load history:", result);
                 return;
             }
-            if (isStaleHistoryResponse()) {
-                return;
-            }
             setEditHistory((prev) => {
                 const next = normalizeEditHistoryPayload(result.data);
                 const scope = {
@@ -2143,9 +1870,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 console.error("[EditorShell] Failed to load item history:", result);
                 return;
             }
-            if (isStaleHistoryResponse()) {
-                return;
-            }
             let operations = normalizeEditHistoryPayload(result.data);
             // Client-side version filtering for current-version mode
             if (currentMode === "current-version") {
@@ -2245,9 +1969,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             globalThis.editorRefreshTimer = undefined;
         };
         if (mode === "immediate") {
-            if (refreshTimer) {
-                clearTimeout(refreshTimer);
-            }
             doRefresh();
             return;
         }
@@ -2263,8 +1984,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     useEffect(() => {
         if (!currentItemDescriptor)
             return;
-        // The previous item's comments don't speak for the new one.
-        setCommentsLoaded(false);
         loadComments();
         loadSuggestedEdits();
     }, [currentItemDescriptor]);
@@ -2443,7 +2162,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         else {
             current.set("compare", "true");
         }
-        current.set("mode", workspaceShowsModeControls ? mode : "edit");
+        current.set("mode", mode);
         // Handle fullscreen parameter
         if (fullscreen) {
             current.set("fullscreen", "true");
@@ -2467,13 +2186,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             current.delete("ccpanel");
             current.delete("providerId");
             current.delete("modelId");
-            current.delete("projectTemplateId");
-        }
-        if (viewName !== "taskboard") {
-            current.delete("tbView");
-            current.delete("tbProjectId");
-            current.delete("tbTaskId");
-            current.delete("tbWizard");
         }
         // Preserve reviewId parameter if it exists (for review links)
         // This ensures review links don't lose the reviewId when URL is synced
@@ -2551,6 +2263,15 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             setItem(item);
             if (!item)
                 return;
+            if (itemRef.current?.descriptor.id === currentItemDescriptor.id &&
+                itemRef.current?.descriptor.language ===
+                    currentItemDescriptor.language &&
+                itemRef.current?.descriptor.version !== item.version) {
+                showInfoToast({
+                    summary: "New version!",
+                    details: "New version of item loaded",
+                });
+            }
         }
         load();
     }, [itemsRepository.revision]);
@@ -2638,7 +2359,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             (activeSlotIdRef.current == null ||
                 existingSlotForItem.slotId === activeSlotIdRef.current)) {
             const isExistingSlotActive = existingSlotForItem.slotId === activeSlotIdRef.current;
-            ensureFieldModificationStoreRegistered(existingSlotForItem.slotId);
             if (isExistingSlotActive) {
                 latestGlobalLoadKeyRef.current = requestedItemKey;
             }
@@ -2697,7 +2417,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 : (activeSlotId ?? editorSlots[0]?.slotId ?? null));
         if (!targetSlotId)
             return undefined;
-        ensureFieldModificationStoreRegistered(targetSlotId);
         setEditorSlots((prev) => {
             if (shouldOpenNewSlot && !options?.targetSlotId) {
                 const existingIdx = prev.findIndex((s) => s.slotId === targetSlotId);
@@ -2775,7 +2494,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         activeSlotId,
         item,
         workspaceId,
-        ensureFieldModificationStoreRegistered,
     ]);
     const setActiveSlot = useCallback(async (slotId) => {
         if (activeSlotId === slotId)
@@ -2818,9 +2536,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         // Remove the slot
         setEditorSlots((prev) => prev.filter((s) => s.slotId !== slotId));
         unregisterSlotContext(slotId);
-        if (fieldModificationStoresBySlotRef.current.delete(slotId)) {
-            setFieldModificationStoreRegistryRevision((previous) => previous + 1);
-        }
         // Activate the next slot if we closed the active one
         if (isClosingActiveSlot) {
             if (nextActiveSlotId) {
@@ -2865,7 +2580,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     }, [fullscreen, configuration.forceFullscreen, searchParams, isMobile]);
     const state = {
         page,
-        currentItemDescriptor,
         configuration,
         selection,
         setSelection,
@@ -2883,17 +2597,12 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         showOnlyMyChanges,
         historyMode,
         refreshHistory,
-        comments,
-        commentsLoaded,
         suggestedEdits,
         setSuggestedEdits,
         mode,
         layoutMode,
         setFocusFieldComponentId,
         setInsertMode,
-        getActiveSlotContext,
-        getFieldModificationStoreForSlot,
-        getActiveSlotId,
     };
     useEffect(() => {
         if (currentOverlay !== "fields")
@@ -2901,12 +2610,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         if (currentOverlay !== "context-menu")
             contextMenuRef.current?.close({});
     }, [currentOverlay]);
-    const handleContextMenuOpenChange = useCallback((open) => {
-        if (open)
-            return;
-        setContextMenuSource(undefined);
-        setCurrentOverlay((overlay) => overlay === "context-menu" ? undefined : overlay);
-    }, []);
     useEffect(() => {
         loadItemVersions();
     }, [currentItemDescriptor]);
@@ -2995,10 +2698,10 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                         : op.canUndo;
                 const mergedCanRedo = isUndoUpdate
                     ? true
-                    : (op.canRedo ?? existingOp.canRedo);
+                    : op.canRedo ?? existingOp.canRedo;
                 const mergedUndone = isUndoUpdate
                     ? true
-                    : (op.undone ?? existingOp.undone);
+                    : op.undone ?? existingOp.undone;
                 const mergedOp = {
                     ...existingOp,
                     ...op,
@@ -3058,15 +2761,13 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         onOperationExecuted,
     };
     // moved to useMediaSelector
-    const { workboxItems, isWorkboxLoading, refreshWorkbox } = useWorkbox({
+    const { workboxItems } = useWorkbox({
         page,
         currentItemDescriptor,
         validate,
     });
-    // Refs for operations-context callbacks (needed because operationsContext is created later)
+    // Ref for markOperationComplete callback (needed because operationsContext is created later)
     const markOperationCompleteRef = useRef(null);
-    const addExecutingOperationRef = useRef(null);
-    const updateExecutingOperationProgressRef = useRef(null);
     // WebSocket message handler and connection
     const messageHandler = useSocketMessageHandler({
         sessionId,
@@ -3091,24 +2792,22 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         markOperationComplete: (operationId) => {
             markOperationCompleteRef.current?.(operationId);
         },
-        addExecutingOperation: (operation) => {
-            addExecutingOperationRef.current?.(operation);
-        },
-        updateExecutingOperationProgress: (operationId, progress, statusMessage) => {
-            updateExecutingOperationProgressRef.current?.(operationId, progress, statusMessage);
-        },
-        refreshWorkbox,
     });
     // Concurrent user limit error state
     const [concurrentUserLimitError, setConcurrentUserLimitError] = useState(null);
     concurrentUserLimitErrorRef.current = concurrentUserLimitError;
     const handleRetryConnection = useCallback(() => {
         setConcurrentUserLimitError(null);
-        // Force reconnection by triggering a new connection attempt. Works for either transport:
-        // WebSocket close lets the hook's reconnect loop pick up; SSE re-opens via the reconnect handler.
-        getEditorRealtime().reconnect("retry-connection");
+        // Force reconnection by triggering a new connection attempt
+        // The useEditorWebSocket hook will check availability again
+        const socket = globalThis.editorSocket;
+        if (socket) {
+            socket.close();
+            delete globalThis.editorSocket;
+        }
+        // The hook will automatically attempt to reconnect
     }, []);
-    useEditorWebSocket({
+    const { socketRef: socketInstanceRef } = useEditorWebSocket({
         sessionId,
         onMessage: messageHandler,
         onOpen: async () => {
@@ -3144,16 +2843,14 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         requestQuota: requestQuotaAndUpdate,
         sendClientInfo,
         setSocketDiagnostics,
-        enableWebSocketTransport: parheliaSettings?.enableWebSocketTransport ?? true,
-        enableSseFallback: parheliaSettings?.enableSseFallback ?? true,
-        transportSettingsReady: parheliaSettingsLoaded === true,
     });
     useEffect(() => {
         const hasMySession = activeSessions.some((s) => s.sessionId === sessionId);
-        if (hasMySession && getEditorRealtime().isOpen()) {
+        if (hasMySession &&
+            socketInstanceRef.current?.readyState === WebSocket.OPEN) {
             toast.dismiss("session-revoked");
         }
-    }, [activeSessions, sessionId, socketConnectionVersion]);
+    }, [activeSessions, sessionId, socketConnectionVersion, socketInstanceRef]);
     useEffect(() => {
         const handleRevoked = (event) => {
             const customEvent = event;
@@ -3164,36 +2861,37 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             window.removeEventListener("parhelia:session-revoked", handleRevoked);
         };
     }, [promptSessionReconnect]);
-    const sendSocketMessage = useCallback((message) => {
-        if (message.type === "caret-position" &&
-            message.payload?.source === "form-field") {
-            const payload = message.payload;
-            setRemoteCarets((current) => {
-                if (payload.offset == null) {
-                    if (!current[sessionId])
-                        return current;
-                    const next = { ...current };
-                    delete next[sessionId];
-                    return next;
-                }
-                if (!payload.fieldId || !payload.item) {
-                    return current;
-                }
-                return {
-                    ...current,
-                    [sessionId]: {
-                        sessionId,
-                        fieldId: payload.fieldId,
-                        item: payload.item,
-                        offset: payload.offset,
-                        value: payload.value,
+    // Show a single "session expired" toast with a Login action when any
+    // service call (or the keepalive check) detects that the user is no
+    // longer authenticated. Using a stable toast id coalesces the toast so a
+    // burst of failing calls does not spam the UI with duplicate toasts.
+    useEffect(() => {
+        if (typeof window === "undefined")
+            return;
+        const handleExpired = () => {
+            toast.error("Your session has expired", {
+                id: "session-expired",
+                description: "Please login again to continue editing.",
+                action: {
+                    label: "Login",
+                    onClick: () => {
+                        window.location.href = "/sitecore/login";
                     },
-                };
+                },
+                duration: Infinity,
             });
+        };
+        window.addEventListener("parhelia:session-expired", handleExpired);
+        return () => {
+            window.removeEventListener("parhelia:session-expired", handleExpired);
+        };
+    }, []);
+    const sendSocketMessage = useCallback((message) => {
+        if (socketInstanceRef.current &&
+            socketInstanceRef.current.readyState === WebSocket.OPEN) {
+            socketInstanceRef.current.send(JSON.stringify(message));
         }
-        // Routes over whichever transport is active (WebSocket preferred, SSE fallback).
-        getEditorRealtime().send(message);
-    }, [sessionId]);
+    }, [socketInstanceRef]);
     // URL update helper - defined early so it can be used by workspace/sidebar functions
     const updateUrl = useCallback((params) => {
         // Use the actual browser pathname (including any basePath like /parhelia)
@@ -3227,15 +2925,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 if (!result)
                     return;
             }
-            const targetWorkspace = configuration.editor.workspaces?.find((workspace) => workspace.id === targetWorkspaceId);
-            const agentToCarryIntoEditor = workspaceIdRef.current === "agents" && targetWorkspaceId === "editor"
-                ? selectedAgentsWorkspaceAgentIdRef.current
-                : null;
-            if (agentToCarryIntoEditor) {
-                setActiveAgentConversation(agentToCarryIntoEditor);
-                handleSetShowAgentsPanel(true);
-            }
-            if (targetWorkspace?.closeAgentsPanelOnEnter) {
+            // Close agents panel when entering page wizard so it doesn't stay open
+            if (targetWorkspaceId === "page-wizard") {
                 handleSetShowAgentsPanel(false);
             }
             // Track previous workspace before switching (use state directly)
@@ -3255,18 +2946,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                     version: currentDescriptor.version.toString(),
                 }
                 : {};
-            const taskboardUrlParams = targetWorkspaceId === "taskboard"
-                ? {}
-                : {
-                    tbView: undefined,
-                    tbProjectId: undefined,
-                    tbTaskId: undefined,
-                    tbWizard: undefined,
-                };
             updateUrl({
                 workspace: targetWorkspaceId,
                 ...currentItemUrlParams,
-                ...taskboardUrlParams,
                 ...options?.urlParams,
             });
             if (typeof document.startViewTransition === "function") {
@@ -3287,7 +2969,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         addNavigationEntry,
         updateUrl,
         handleSetShowAgentsPanel,
-        configuration.editor.workspaces,
     ]);
     // Helper: get all sidebar IDs that should be preserved (locked sidebars + their stack mates)
     const getPreservedSidebarIds = useCallback(() => {
@@ -3315,9 +2996,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             // Check if this sidebar is part of a multi-member stack
             const currentStacks = sidebarStacksRef.current;
             const stackContainingSidebar = currentStacks.find((stack) => stack.includes(sidebarId));
-            const isInMultiMemberStack = stackContainingSidebar &&
-                stackContainingSidebar.filter((id) => id !== CONTENT_TREE_SIDEBAR_ID)
-                    .length > 1;
+            const isInMultiMemberStack = stackContainingSidebar && stackContainingSidebar.length > 1;
             // Don't close via icon click if in multi-member stack (unless forceClose is true)
             if (isInMultiMemberStack && !options?.forceClose) {
                 return;
@@ -3331,9 +3010,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             if (sidebarId === "agents-panel") {
                 setUserPreferences({ showAgentsPanel: false });
             }
-            if (sidebarId === "feedback") {
-                closeFeedbackSuggestionsMode();
-            }
             startTransition(() => {
                 updateUrl({ sidebar: sidebarUrlValue(newSidebars) });
             });
@@ -3342,12 +3018,10 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         // Opening a new sidebar
         // Keep sidebars that are locked OR in a stack with a locked sidebar
         const preservedSet = getPreservedSidebarIds();
-        const newSidebars = sidebarId === CONTENT_TREE_SIDEBAR_ID
-            ? [...currentOpenSidebars, sidebarId]
-            : [
-                ...currentOpenSidebars.filter((id) => preservedSet.has(id) || id === CONTENT_TREE_SIDEBAR_ID), // Keep locked stacks and the dedicated tree panel
-                sidebarId, // Add the new one
-            ];
+        const newSidebars = [
+            ...currentOpenSidebars.filter((id) => preservedSet.has(id)), // Keep locked stacks
+            sidebarId, // Add the new one
+        ];
         openSidebarsRef.current = newSidebars;
         setOpenSidebars(newSidebars);
         ensureSidebarPinned(sidebarId);
@@ -3369,7 +3043,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         ensureSidebarPinned,
         setUserPreferences,
         isMobile,
-        closeFeedbackSuggestionsMode,
     ]);
     // Ensure a sidebar is open (without toggling it closed if already open)
     const openSidebar = useCallback((sidebarId, options) => {
@@ -3381,10 +3054,10 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         const preservedSet = options?.preserveOpenSidebars
             ? undefined
             : getPreservedSidebarIds();
-        const newSidebars = options?.preserveOpenSidebars || sidebarId === CONTENT_TREE_SIDEBAR_ID
+        const newSidebars = options?.preserveOpenSidebars
             ? [...currentOpenSidebars, sidebarId]
             : [
-                ...currentOpenSidebars.filter((id) => preservedSet?.has(id) || id === CONTENT_TREE_SIDEBAR_ID),
+                ...currentOpenSidebars.filter((id) => preservedSet?.has(id)),
                 sidebarId,
             ];
         openSidebarsRef.current = newSidebars;
@@ -3554,10 +3227,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 return false;
             if (!isTaskboardWorkspace && isTaskboardSidebar)
                 return false;
-            // Keep agents-panel broadly available, except in dedicated workspaces
-            // that own their full surface.
+            // Always show agents-panel regardless of workspace settings.
             if (s.id === "agents-panel") {
-                return targetWorkspaceId !== "translation-management";
+                return true;
             }
             // If no workspace settings or no sidebars defined for current workspace, show all.
             if (workspaceAllowedSidebarIds.length === 0) {
@@ -3584,9 +3256,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         };
     }, [getSidebarsForWorkspace, workspaceId]);
     // Track the last workspaceId this effect saw so we only auto-apply
-    // `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.
+    // `currentWorkspace.defaultSidebars` on an actual workspace change. On initial mount
+    // the user's explicit empty state (sidebar=none in URL) must not be overwritten.
     const prevWorkspaceIdForDefaultsRef = useRef(null);
     useEffect(() => {
         if (!currentWorkspace.supportsSidebars) {
@@ -3679,67 +3350,42 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         const result = await command.execute(context);
         return result;
     }, [editContextRef, pathname, router, searchParams, openDialog]);
-    const operationsContext = getOperationsContext({ ...state, refreshWorkbox }, ui);
+    const operationsContext = getOperationsContext(state, ui);
     const operations = operationsContext.ops;
-    // Update the refs so the socket handler can access the operations-context callbacks
+    // Update the ref for markOperationComplete so the socket handler can access it
     useEffect(() => {
         markOperationCompleteRef.current =
             operationsContext.context.markOperationComplete;
-        addExecutingOperationRef.current =
-            operationsContext.context.addExecutingOperation;
-        updateExecutingOperationProgressRef.current =
-            operationsContext.context.updateExecutingOperationProgress;
-    }, [
-        operationsContext.context.markOperationComplete,
-        operationsContext.context.addExecutingOperation,
-        operationsContext.context.updateExecutingOperationProgress,
-    ]);
+    }, [operationsContext.context.markOperationComplete]);
     // Replace selected text with new content (for inline AI editing)
-    const replaceSelectedText = useCallback(async (newText, selectionOverride) => {
-        const rangeToReplace = selectionOverride ?? selectedRange ?? lastTextSelectionRangeRef.current;
+    const replaceSelectedText = useCallback(async (newText) => {
+        if (!focusedField)
+            return;
         try {
             // If we have selected text and a replaceText callback, use it
-            if (rangeToReplace?.text && rangeToReplace.replaceText) {
-                rangeToReplace.replaceText(newText);
+            if (selectedRange?.text && selectedRange.replaceText) {
+                selectedRange.replaceText(newText);
                 setSelectedRange(undefined);
-                lastTextSelectionRangeRef.current = undefined;
                 return;
             }
             // If no selection but we have a replaceText callback, try to use it for insertion
             // (some editors might support insertion via the same callback)
-            if (!rangeToReplace?.text && rangeToReplace?.replaceText) {
-                rangeToReplace.replaceText(newText);
+            if (!selectedRange?.text && selectedRange?.replaceText) {
+                selectedRange.replaceText(newText);
                 setSelectedRange(undefined);
-                lastTextSelectionRangeRef.current = undefined;
                 return;
             }
-            const fieldToReplace = focusedField ??
-                (rangeToReplace?.fieldId &&
-                    rangeToReplace.itemId &&
-                    rangeToReplace.language &&
-                    typeof rangeToReplace.version === "number"
-                    ? {
-                        fieldId: rangeToReplace.fieldId,
-                        item: {
-                            id: rangeToReplace.itemId,
-                            language: rangeToReplace.language,
-                            version: rangeToReplace.version,
-                        },
-                    }
-                    : undefined);
-            if (!fieldToReplace)
-                return;
             // Fallback: indexOf-based replacement/insertion for plain text fields
-            const { fieldId, item } = fieldToReplace;
-            const modifiedField = getActiveSlotContext()?.fieldModificationStore.getField(fieldId, item.id, item.language, item.version);
+            const { fieldId, item } = focusedField;
+            const modifiedField = fieldModificationStore.getField(fieldId, item.id, item.language, item.version);
             const currentValue = modifiedField
                 ? modifiedField.value || ""
-                : (await itemsRepository.getField(fieldToReplace))
-                    ?.value || "";
+                : (await itemsRepository.getField(focusedField))?.value ||
+                    "";
             let newValue;
-            if (rangeToReplace?.text) {
+            if (selectedRange?.text) {
                 // Replace selected text
-                const { text: originalSelectedText } = rangeToReplace;
+                const { text: originalSelectedText } = selectedRange;
                 const actualPosition = currentValue.indexOf(originalSelectedText);
                 if (actualPosition === -1) {
                     console.error("Failed to find selected text in field value");
@@ -3752,7 +3398,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             }
             else {
                 // Insert at cursor position (use startOffset if available, otherwise append)
-                const insertPosition = rangeToReplace?.startOffset ?? currentValue.length;
+                const insertPosition = selectedRange?.startOffset ?? currentValue.length;
                 newValue =
                     currentValue.slice(0, insertPosition) +
                         newText +
@@ -3760,24 +3406,17 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             }
             // Update the field
             await operations.editField({
-                field: fieldToReplace,
+                field: focusedField,
                 value: newValue,
                 refresh: "immediate",
             });
             // Clear the selection after replacement/insertion
             setSelectedRange(undefined);
-            lastTextSelectionRangeRef.current = undefined;
         }
         catch (error) {
             console.error("Failed to replace/insert text:", error);
         }
-    }, [
-        selectedRange,
-        focusedField,
-        itemsRepository,
-        operations,
-        getActiveSlotContext,
-    ]);
+    }, [selectedRange, focusedField, itemsRepository, operations]);
     // Insert text at cursor position (for when there's no selection)
     const insertTextAtCursor = useCallback(async (newText) => {
         if (!focusedField)
@@ -3818,7 +3457,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             }
             // Fallback: use field value length (append)
             const { fieldId, item } = focusedField;
-            const modifiedField = getActiveSlotContext()?.fieldModificationStore.getField(fieldId, item.id, item.language, item.version);
+            const modifiedField = fieldModificationStore.getField(fieldId, item.id, item.language, item.version);
             const currentValue = modifiedField
                 ? modifiedField.value || ""
                 : (await itemsRepository.getField(focusedField))?.value ||
@@ -3837,13 +3476,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         catch (error) {
             console.error("Failed to insert text at cursor:", error);
         }
-    }, [
-        focusedField,
-        selectedRange,
-        itemsRepository,
-        operations,
-        getActiveSlotContext,
-    ]);
+    }, [focusedField, selectedRange, itemsRepository, operations]);
     // Cleanup pending debounced operations on unmount to prevent stuck "saving" states
     useEffect(() => {
         return () => {
@@ -3857,31 +3490,14 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             setTimeout(() => {
                 setIgnoreBlur(false);
             }, 20);
+            setFocusedField({ ...field });
             if (requestLock) {
                 const lockResult = (await operations.ensureLock(field)) || false;
-                if (!lockResult) {
-                    return false;
-                }
-                setFocusedField({ ...field });
-                return true;
+                return lockResult;
             }
-            setFocusedField({ ...field });
         }
         else {
-            // Keep the field selected when the blur was caused by clicking a User
-            // Manual "Show me" control. Those reveals (e.g. the Reset field action)
-            // gate availability on the `.focused-field` anchor, so dropping the
-            // focused field here would disable the button and hide the very element
-            // being highlighted. A mouse blur fires before the button becomes the
-            // active element, so the guard also accepts a recent pointer-down on the
-            // control (keyboard activation still leaves it as the active element).
-            const focusMovedToShowMe = typeof document !== "undefined" &&
-                isManualShowMeFocusTransfer({
-                    activeElement: document.activeElement,
-                    lastPointerDownAt: manualShowMePointerDownAtRef.current,
-                    now: Date.now(),
-                });
-            if (!ignoreBlur && !focusMovedToShowMe) {
+            if (!ignoreBlur) {
                 setFocusedField(undefined);
                 setLockedField(undefined); // Clear client-side lock state
                 releaseFieldLocks(sessionId);
@@ -4046,12 +3662,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setNavigationHistory,
     ]);
     useEffect(() => {
-        if (typeof window === "undefined") {
-            return;
-        }
-        const exposeViewportTestApi = process.env.PARHELIA_DEV_MODE === "true" ||
-            window.localStorage?.getItem("__parheliaViewportTestApi") === "true";
-        if (!exposeViewportTestApi) {
+        if (typeof window === "undefined" ||
+            process.env.PARHELIA_DEV_MODE !== "true") {
             return;
         }
         window.__parheliaQuickSwitcherTestApi = {
@@ -4260,24 +3872,18 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             };
         }
     }, [operations.onFieldBlur]);
-    // Record pointer-downs on User Manual "Show me" controls before the field
-    // blur fires, so the blur guard above can keep the focused field for
-    // field-gated reveals regardless of focus/activeElement timing.
     useEffect(() => {
-        if (typeof document === "undefined")
-            return;
-        const handlePointerDown = (event) => {
-            if (isManualShowMeTarget(event.target)) {
-                manualShowMePointerDownAtRef.current = Date.now();
-            }
-        };
-        document.addEventListener("pointerdown", handlePointerDown, true);
-        document.addEventListener("mousedown", handlePointerDown, true);
-        return () => {
-            document.removeEventListener("pointerdown", handlePointerDown, true);
-            document.removeEventListener("mousedown", handlePointerDown, true);
-        };
-    }, []);
+        if (mode === "suggestions") {
+            setShowSuggestedEdits(true);
+            // Refresh from the server to recover from any missed
+            // `suggested-edit-updated` WebSocket events (e.g. created by an
+            // agent tool while another tab was focused).
+            loadSuggestedEdits();
+        }
+        else {
+            setShowSuggestedEdits(false);
+        }
+    }, [mode, loadSuggestedEdits]);
     const isReadOnly = !item ||
         !item.canWriteItem ||
         !item.canWriteLanguage ||
@@ -4296,6 +3902,15 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         // Otherwise, only show workspaces that are in the settings
         return workspaceIdsFromSettings.includes(w.id) && !w.visible;
     });
+    // Handle initial mode setup from URL (only on initial load)
+    useEffect(() => {
+        if (!isInitialLoad)
+            return;
+        const queryMode = searchParams.get("mode");
+        if (queryMode) {
+            setMode(queryMode);
+        }
+    }, [searchParams, isInitialLoad]);
     // Add this new effect for initial fullscreen setup
     useEffect(() => {
         if (!isInitialLoad)
@@ -4305,19 +3920,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             activePageViewContext.setFullscreen(isFullscreen);
         }
     }, [searchParams, isInitialLoad, activePageViewContext]);
-    useEffect(() => {
-        if (typeof window === "undefined" ||
-            process.env.PARHELIA_DEV_MODE !== "true") {
-            return;
-        }
-        window.__parheliaViewportTestApi = {
-            setZoom: (zoom) => activePageViewContext.setZoom(zoom),
-            getZoom: () => activePageViewContext.zoom,
-        };
-        return () => {
-            delete window.__parheliaViewportTestApi;
-        };
-    }, [activePageViewContext]);
     // Simple registry for per-view context factories (e.g., GraphQL context)
     // Stored outside the EditContext object via ref to avoid re-renders
     if (!globalThis.__editorContextFactoriesRef) {
@@ -4386,8 +3988,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         clipboard,
         setClipboard,
         workboxItems,
-        isWorkboxLoading,
-        refreshWorkbox,
         requestRefresh,
         refreshCompletedFlag,
         openCreatePageDialog: () => {
@@ -4415,8 +4015,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setScrollIntoView,
         focusedField,
         setFocusedField: setFocusedFieldWithLock,
-        getComponentCommands: async (components, providerContext) => {
-            return await getComponentCommands(components, editContext, providerContext);
+        getComponentCommands: async (components) => {
+            return await getComponentCommands(components, editContext);
         },
         dragStart: (dragObject) => {
             setDragObject(dragObject);
@@ -4424,11 +4024,16 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         dragEnd: () => {
             setDragObject(undefined);
         },
-        droppedInPlaceholder: async (placeholderKey, index, spotPositionElement, spotPositionAnchor, insertOption, spotPosition, color) => {
-            const droppedDragObject = dragObject;
-            if ((!droppedDragObject && !insertOption) || !page)
+        droppedInPlaceholder: async (placeholderKey, index, spotPositionElement, spotPositionAnchor, insertOption) => {
+            if ((!dragObject && !insertOption) || !page)
                 return;
             setDragObject(undefined);
+            if (spotPositionElement && spotPositionAnchor) {
+                setInserting({
+                    positionElement: spotPositionElement,
+                    positionAnchor: spotPositionAnchor,
+                });
+            }
             const placeholderKeyComponents = placeholderKey.split("_");
             const parentId = placeholderKeyComponents.length === 1
                 ? undefined
@@ -4438,32 +4043,39 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             let op;
             if (!currentItemDescriptorRef.current)
                 return;
-            let insertionSequence;
-            if ((spotPosition || spotPositionElement) && spotPositionAnchor) {
-                insertionSequence = ++insertingSequenceRef.current;
-                setInserting({
-                    position: spotPosition,
-                    positionElement: spotPositionElement,
-                    positionAnchor: spotPositionAnchor,
-                    color,
-                });
-            }
-            let operationPromise;
             if (insertOption) {
                 // Clear any selected component type for insertion once the component
                 // has been dropped into a placeholder. This ensures that insert mode
                 // can exit cleanly and the editor header updates from "Add Component"
                 // to the actual component name after the operation completes.
                 setSelectedForInsertion("");
-                operationPromise = operations.addComponent(insertOption.typeId, placeholderKey, index, currentItemDescriptorRef.current);
+                operations
+                    .addComponent(insertOption.typeId, placeholderKey, index, currentItemDescriptorRef.current)
+                    .then((result) => {
+                    setInserting(undefined);
+                })
+                    .catch((err) => {
+                    setInserting(undefined);
+                });
+                return;
             }
-            else if (droppedDragObject?.type == "template") {
-                operationPromise = operations.addComponent(droppedDragObject.typeId, placeholderKey, index, currentItemDescriptorRef.current);
+            if (!dragObject)
+                return;
+            if (dragObject && dragObject.type == "template") {
+                operations
+                    .addComponent(dragObject.typeId, placeholderKey, index, currentItemDescriptorRef.current)
+                    .then((result) => {
+                    setInserting(undefined);
+                })
+                    .catch((err) => {
+                    setInserting(undefined);
+                });
+                return;
             }
-            else if (droppedDragObject?.type == "component" ||
-                (droppedDragObject?.type == "items" && droppedDragObject.items)) {
+            if (dragObject.type == "component" ||
+                (dragObject.type == "items" && dragObject.items)) {
                 const parentComponent = parentId && page ? getComponentById(parentId, page) : null;
-                if (droppedDragObject.type == "items" && droppedDragObject.items) {
+                if (dragObject.type == "items" && dragObject.items) {
                     op = {
                         type: "link-component",
                         mainItem: page.item.descriptor,
@@ -4477,52 +4089,33 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                         placeholderIndex: index,
                         date: new Date().toISOString(),
                         id: uuid(),
-                        linkedComponentItem: droppedDragObject.items[0],
+                        linkedComponentItem: dragObject.items[0],
                         title: "Link component",
                     };
                 }
                 else {
-                    const componentIds = droppedDragObject.components?.map((c) => c.id) ?? [];
-                    if (componentIds.length > 0) {
-                        op = {
-                            type: "move-component",
-                            mainItem: page.item.descriptor,
-                            parent: parentId && {
-                                id: parentId,
-                                language: page.item.descriptor.language,
-                                version: page.item.descriptor.version,
-                            },
-                            placeholderKey,
-                            placeholderIndex: index,
-                            componentIds,
-                            date: new Date().toISOString(),
-                            id: uuid(),
-                            title: "Move component",
-                        };
-                    }
+                    const componentIds = dragObject.components?.map((c) => c.id) ?? [];
+                    if (componentIds.length === 0)
+                        return;
+                    op = {
+                        type: "move-component",
+                        mainItem: page.item.descriptor,
+                        parent: parentId && {
+                            id: parentId,
+                            language: page.item.descriptor.language,
+                            version: page.item.descriptor.version,
+                        },
+                        placeholderKey,
+                        placeholderIndex: index,
+                        componentIds,
+                        date: new Date().toISOString(),
+                        id: uuid(),
+                        title: "Move component",
+                    };
                 }
             }
             if (op)
-                operationPromise = operations.executeEditOperation(op);
-            if (!operationPromise) {
-                if (insertionSequence !== undefined &&
-                    insertingSequenceRef.current === insertionSequence) {
-                    setInserting(undefined);
-                }
-                return;
-            }
-            try {
-                await operationPromise;
-            }
-            catch (err) {
-                console.error("Failed to drop component in placeholder", err);
-            }
-            finally {
-                if (insertionSequence !== undefined &&
-                    insertingSequenceRef.current === insertionSequence) {
-                    setInserting(undefined);
-                }
-            }
+                operations.executeEditOperation(op);
         },
         page,
         triggerFieldAction: async (fieldDescriptor, actionButton, parameters, event) => {
@@ -4658,9 +4251,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 : x));
         },
         activeFieldActions,
-        showContextMenu: (event, items, options) => {
+        showContextMenu: (event, items) => {
             contextMenuRef.current?.show(event, items);
-            setContextMenuSource(options?.source);
             setCurrentOverlay("context-menu");
         },
         updateContextMenu: (items) => {
@@ -4686,7 +4278,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         refreshHistory,
         isRefreshing,
         activeSessions,
-        remoteCarets,
         unlockField: async () => {
             setLockedField(undefined); // Clear client-side lock state
             await releaseFieldLocks(sessionId);
@@ -4749,7 +4340,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         insertMode,
         setInsertMode,
         currentOverlay,
-        contextMenuSource,
         setCurrentOverlay,
         lockedField,
         timings,
@@ -4787,7 +4377,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         unregisterSlotContext,
         getActiveSlotContext,
         getActiveSlotId,
-        getFieldModificationStoreForSlot,
         closeEditorSlot,
         setActiveSlot,
         revision,
@@ -4795,10 +4384,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         pageModelReadyToken,
         selectedComment,
         setSelectedComment,
-        selectedSuggestedEdit,
-        setSelectedSuggestedEdit,
         comments,
-        commentsLoaded,
         availableCommentTags,
         loadComments,
         setComments,
@@ -4806,60 +4392,27 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setShowComments,
         showResolvedComments,
         setShowResolvedComments,
-        addComment: async (options) => {
+        addComment: async () => {
             const descriptor = item?.descriptor;
             if (!descriptor)
                 return;
-            // Only keep a single empty draft comment at a time. Clicking
-            // "add comment" again while an unsaved draft already exists should
-            // focus that draft instead of stacking up more empty inputs.
-            const existingDraft = comments.find((c) => c.isNew);
-            if (existingDraft) {
-                await activateCommentsModeForWriting("comment-add");
-                setSelectedComment(existingDraft);
-                setSelectedSuggestedEdit(undefined);
-                return;
-            }
             // Snapshot selection context up-front so switching views doesn't change what we comment on.
-            const hintedFieldSnapshot = options?.field;
             const focusedFieldSnapshot = focusedFieldRef.current;
-            const selectedRangeCandidate = isUsableTextSelectionRange(selectedRange)
-                ? selectedRange
-                : lastTextSelectionRangeRef.current;
-            const selectedRangeSnapshot = !hintedFieldSnapshot ||
-                selectionRangeMatchesField(selectedRangeCandidate, hintedFieldSnapshot)
-                ? selectedRangeCandidate
-                : undefined;
-            const selectedRangeFieldSnapshot = isUsableTextSelectionRange(selectedRangeSnapshot)
-                ? {
-                    fieldId: selectedRangeSnapshot.fieldId,
-                    item: {
-                        id: selectedRangeSnapshot.itemId,
-                        language: selectedRangeSnapshot.language || descriptor.language,
-                        version: typeof selectedRangeSnapshot.version === "number"
-                            ? selectedRangeSnapshot.version
-                            : descriptor.version,
-                    },
-                }
-                : undefined;
-            const commentFieldSnapshot = selectedRangeFieldSnapshot ??
-                hintedFieldSnapshot ??
-                focusedFieldSnapshot;
-            const itemId = commentFieldSnapshot?.item.id ||
-                (selection.length > 0 ? selection[0] : undefined) ||
+            const selectionSnapshot = [...selection];
+            const selectedRangeSnapshot = selectedRange;
+            const itemId = focusedFieldSnapshot?.item.id ||
+                (selectionSnapshot.length > 0 ? selectionSnapshot[0] : undefined) ||
                 descriptor.id;
             if (!itemId)
                 return;
-            // Ensure the promoted comments surface is active when adding a comment.
-            await activateCommentsModeForWriting("comment-add");
-            const language = commentFieldSnapshot?.item.language || descriptor.language;
-            const version = typeof commentFieldSnapshot?.item.version === "number"
-                ? commentFieldSnapshot.item.version
-                : descriptor.version;
+            // Ensure the feedback sidebar is open when adding a comment (e.g. command palette / toolbar button).
+            openSidebar("feedback");
+            const language = descriptor.language;
+            const version = descriptor.version;
             const getFieldName = async () => {
-                if (!commentFieldSnapshot)
+                if (!focusedFieldSnapshot)
                     return "";
-                const field = await itemsRepository.getField(commentFieldSnapshot);
+                const field = await itemsRepository.getField(focusedFieldSnapshot);
                 return field?.name;
             };
             const getItemName = async () => {
@@ -4875,40 +4428,24 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                 isNew: true,
                 itemId,
                 itemName: await getItemName(),
-                fieldId: commentFieldSnapshot?.fieldId,
+                fieldId: focusedFieldSnapshot?.fieldId,
                 fieldName: await getFieldName(),
                 mainItemId: descriptor.id,
                 language,
                 version,
                 position: 0,
-                rangeStart: isUsableTextSelectionRange(selectedRangeSnapshot)
-                    ? selectedRangeSnapshot.startOffset
-                    : 0,
-                rangeEnd: isUsableTextSelectionRange(selectedRangeSnapshot)
-                    ? selectedRangeSnapshot.endOffset
-                    : 0,
+                rangeStart: selectedRangeSnapshot?.startOffset || 0,
+                rangeEnd: selectedRangeSnapshot?.endOffset || 0,
                 author: user?.name,
                 authorDisplayName: user?.fullName,
-                created: new Date().toISOString(),
+                date: new Date().toISOString(),
             };
-            if (selectedRangeSnapshot?.clientRect) {
-                newComment[COMMENT_TRANSIENT_SELECTION_RECT] =
-                    selectedRangeSnapshot.clientRect;
-            }
-            if (selectedRangeSnapshot?.elementKey) {
-                newComment[COMMENT_TRANSIENT_ELEMENT_KEY] =
-                    selectedRangeSnapshot.elementKey;
-            }
             setComments([newComment, ...comments]);
             setSelectedComment(newComment);
-            setSelectedSuggestedEdit(undefined);
-            lastTextSelectionRangeRef.current = undefined;
+            setShowComments(true);
         },
         mode,
-        changeMode,
         setMode,
-        registerModeChangeParticipant,
-        modeTransitionRevision,
         previewDate,
         setPreviewDate,
         layoutMode,
@@ -4933,8 +4470,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setComponentNavigatorOpenForSlot,
         showAgentsPanel,
         setShowAgentsPanel: handleSetShowAgentsPanel,
-        waitForSurfaceSettled: settleRegistry.waitForSurfaceSettled,
-        notifySurfaceSettled: settleRegistry.notifySettled,
         isEditorFormHiddenForSlot,
         setEditorFormHiddenForSlot,
         editorFormHintSeen,
@@ -4978,7 +4513,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         setCurrentWizardId,
         favorites,
         loadFavorites,
-        refreshFavorites,
         isQuickSwitcherVisible: quickSwitcherVisible,
         // Context factory registry methods
         registerContextFactory: (name, factory) => {
@@ -5023,8 +4557,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         dragObject,
         clipboard,
         workboxItems,
-        isWorkboxLoading,
-        refreshWorkbox,
         requestRefresh,
         refreshCompletedFlag,
         searchParams,
@@ -5059,7 +4591,6 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         componentDesignerRendering,
         insertMode,
         currentOverlay,
-        contextMenuSource,
         lockedField,
         timings,
         selectedRange,
@@ -5077,9 +4608,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         notifyPageModelReady,
         pageModelReadyToken,
         selectedComment,
-        selectedSuggestedEdit,
         comments,
-        commentsLoaded,
         availableCommentTags,
         loadComments,
         showComments,
@@ -5091,22 +4620,15 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         isRefreshing,
         favorites,
         loadFavorites,
-        refreshFavorites,
         currentWizardId,
         quickSwitcherVisible,
         quickSwitcherSelectedIndex,
         handleQuickSwitcherSelect,
         webSocketMessages,
         socketDiagnostics,
-        remoteCarets,
         factoriesRef,
         user,
         statusMessage,
-        activateCommentsModeForWriting,
-        changeMode,
-        setMode,
-        registerModeChangeParticipant,
-        modeTransitionRevision,
         previewDate,
         layoutMode,
         enableCompletions,
@@ -5122,11 +4644,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         handleSetShowComponentNavigator,
         showAgentsPanel,
         handleSetShowAgentsPanel,
-        settleRegistry,
         activeSlotId,
-        editorSlots,
-        slotContexts,
-        getFieldModificationStoreForSlot,
         isEditorFormHiddenForSlot,
         setEditorFormHiddenForSlot,
         editorFormHintSeen,
@@ -5163,6 +4681,9 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     const fieldsEditContext = {
         modifiedFields,
         recentEdits,
+        clearModifiedFields: () => {
+            fieldModificationStore.clear();
+        },
         clearRecentEdits: () => {
             setRecentEdits([]);
         },
@@ -5173,6 +4694,8 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
             setLockedField(undefined); // Clear client-side lock state
             await releaseFieldLocks(sessionId);
         },
+        inlineEditingFieldElement,
+        setInlineEditingFieldElement,
     };
     // Debug: Track editContext changes
     // const prevDependencies = useRef<any[]>([]);
@@ -5211,6 +4734,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     //     componentDesignerRendering,
     //     insertMode,
     //     currentOverlay,
+    //     inlineEditingFieldElement,
     //     lockedField,
     //     selectedRange,
     //     pageViewContext,
@@ -5298,6 +4822,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     //     'componentDesignerRendering',
     //     'insertMode',
     //     'currentOverlay',
+    //     'inlineEditingFieldElement',
     //     'lockedField',
     //     'selectedRange',
     //     'pageViewContext',
@@ -5386,7 +4911,7 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
     // Auto-open the mobile editor panel for new selection/intent changes, but
     // keep a manual close sticky for the exact same context.
     useEffect(() => {
-        if (!isMobile || workspaceId !== "editor" || mode !== "edit")
+        if (!isMobile || workspaceId !== "editor")
             return;
         if (activeEditorTab) {
             handleSetMobileEditorPanelOpen(true);
@@ -5404,17 +4929,19 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
         insertMode,
         isMobile,
         mobilePanelDismissToken,
-        mode,
         selection,
         workspaceId,
     ]);
+    useEffect(() => {
+        fieldsEditContext.clearModifiedFields();
+    }, [currentItemDescriptor]);
     if (!currentWorkspace)
         return null;
-    const editorUi = fullscreen ? (_jsxs(_Fragment, { children: [_jsx("div", { className: "fixed inset-0 flex", children: _jsx(PageViewerFrame, { compareView: compareMode, pageViewContext: activePageViewContext }) }), showFullscreenHint && !configuration.forceFullscreen && !isMobile && (_jsx("div", { className: "fixed inset-0 z-10000", onMouseMoveCapture: () => {
+    const editorUi = fullscreen ? (_jsxs(_Fragment, { children: [_jsxs("div", { className: "fixed inset-0 flex", children: [_jsx(PageViewerFrame, { compareView: compareMode, pageViewContext: activePageViewContext }), _jsx(FullscreenControls, { device: activePageViewContext.device, setDevice: (d) => activePageViewContext.setDevice(d), canExit: !configuration.forceFullscreen, onExit: () => setFullscreen(false), firstMobileDeviceName: configuration.devices[0]?.name })] }), showFullscreenHint && !configuration.forceFullscreen && !isMobile && (_jsx("div", { className: "fixed inset-0 z-10000", onMouseMoveCapture: () => {
                     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-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: () => {
+                }, "data-testid": "fullscreen-hint-overlay", children: _jsxs("div", { className: "fixed top-6 left-1/2 -translate-x-1/2 transform rounded-full bg-black/60 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: "mx-1 rounded bg-white/20 px-1.5 py-0.5 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 && (_jsx(Tour, { tourStopCallback: () => {
                     setIsTourActive(false);
                     // Remove tour state from URL
                     // Use history.replaceState instead of router.replace to avoid triggering React navigation
@@ -5426,14 +4953,12 @@ export function EditorShell({ configuration, className, item: loadItemDescriptor
                         : window.location.pathname;
                     window.history.replaceState(getCurrentHistoryState(), "", newUrl);
                 }, configuration: configuration, restoredFromUrl: tourRestoredRef.current })), _jsx(FeatureGate, { feature: LicenseFeatures.AI, children: _jsx(GuidanceOverlay, {}) }), _jsx(FeatureGate, { feature: LicenseFeatures.AI, children: _jsx(AgentDialogHandler, {}) })] }));
-    const showRefreshSpinner = editContext.isRefreshing && !activePageViewContext.iframeSupportsRefresh;
-    return (_jsx(LicenseProvider, { initialLicenseStatus: initialLicenseStatus, initialStatusLoaded: initialLicenseStatusLoaded, children: _jsx("div", { className: `editor h-full w-full`, children: _jsx(OperationsContextProvider, { value: operationsContext.context, children: _jsx(FieldsEditContextProvider, { value: fieldsEditContext, children: _jsx(EditContextProvider, { value: editContext, children: _jsxs(SplashScreenAgentProvider, { children: [_jsx(DevModeIndicator, {}), startupChecks.state === "loading" && _jsx(EditorLoadingOverlay, {}), showRefreshSpinner && (_jsx("div", { className: "text-neutral-grey-50 pointer-events-none fixed right-0 bottom-0 flex h-24 w-24 items-center justify-center opacity-50 select-none", children: _jsx(Spinner, {}) })), (currentWorkspace.id === "agents" ||
-                                    currentWorkspace.id === "taskboard" ||
-                                    currentWorkspace.id === "translation-management") &&
+    return (_jsx(LicenseProvider, { initialLicenseStatus: initialLicenseStatus, initialStatusLoaded: initialLicenseStatusLoaded, children: _jsx("div", { className: `editor h-full w-full`, children: _jsx(OperationsContextProvider, { value: operationsContext.context, children: _jsx(FieldsEditContextProvider, { value: fieldsEditContext, children: _jsx(EditContextProvider, { value: editContext, children: _jsxs(SplashScreenAgentProvider, { children: [_jsx(DevModeIndicator, {}), startupChecks.state === "loading" && (_jsx("div", { className: "fixed inset-0 z-50 flex items-center justify-center bg-white/70 backdrop-blur-[1px]", children: _jsx("div", { className: "flex items-center gap-3 rounded-md border border-gray-200 bg-white px-4 py-3 text-gray-700 shadow-sm", children: _jsx(Spinner, { size: "xl" }) }) })), editContext.isRefreshing && (_jsx("div", { className: "pointer-events-none fixed right-0 bottom-0 flex h-24 w-24 items-center justify-center text-gray-600 opacity-50 select-none", children: _jsx(Spinner, {}) })), (currentWorkspace.id === "agents" ||
+                                    currentWorkspace.id === "taskboard") &&
                                     showAgentsWorkspaceEditor && (_jsx(AgentsSlotContextBridgeHost, { slots: editorSlots })), startupChecks.state !== "loading" && (children || editorUi), startupChecks.state !== "loading" && dialog, _jsx(Toaster, { position: "top-center" }), " ", _jsx(ConfirmationDialog, { ref: confirmationDialogRef }), _jsx(ConcurrentUserLimitDialog, { open: concurrentUserLimitError !== null, onOpenChange: (open) => {
                                         if (!open) {
                                             setConcurrentUserLimitError(null);
                                         }
-                                    }, sessionId: sessionId, currentUsers: concurrentUserLimitError?.currentUsers ?? 0, maxUsers: concurrentUserLimitError?.maxUsers ?? 0, message: concurrentUserLimitError?.message ?? "", onRetry: handleRetryConnection, isAdministrator: userInfo.user.isAdministrator === true }), _jsx(QuickItemSwitcher, { visible: quickSwitcherVisible, entries: quickSwitcherEntries.slice(0, 5), selectedIndex: quickSwitcherSelectedIndex, onSelect: handleQuickSwitcherSelect, onClose: () => setQuickSwitcherVisible(false) }), _jsx(EditContextMenu, { ref: contextMenuRef, onOpenChange: handleContextMenuOpenChange }), _jsx(EditorFormHintPopover, {}), _jsx(FeatureGate, { feature: LicenseFeatures.AI, children: _jsx(InlineAiTrigger, {}) }), media.mediaSelectorVisible && (_jsx(MediaSelector, { language: editContext.currentItemDescriptor.language, visible: media.mediaSelectorVisible, onHide: media.handleHide, onMediaSelected: media.onMediaSelect, selectedIdPath: media.selectedMediaIdPath, mode: media.mediaSelectorMode, initialSearchTerm: media.initialSearchTerm })), _jsx(FieldEditorPopup, { ref: fieldEditorPopupRef }), _jsx(LicenseOverlay, {}), _jsx(SessionExpiredOverlay, {})] }) }) }) }) }) }));
+                                    }, sessionId: sessionId, currentUsers: concurrentUserLimitError?.currentUsers ?? 0, maxUsers: concurrentUserLimitError?.maxUsers ?? 0, message: concurrentUserLimitError?.message ?? "", onRetry: handleRetryConnection, isAdministrator: userInfo.user.isAdministrator === true }), _jsx(QuickItemSwitcher, { visible: quickSwitcherVisible, entries: quickSwitcherEntries.slice(0, 5), selectedIndex: quickSwitcherSelectedIndex, onSelect: handleQuickSwitcherSelect, onClose: () => setQuickSwitcherVisible(false) }), _jsx(EditContextMenu, { ref: contextMenuRef }), _jsx(EditorFormHintPopover, {}), _jsx(FeatureGate, { feature: LicenseFeatures.AI, children: _jsx(InlineAiTrigger, {}) }), media.mediaSelectorVisible && (_jsx(MediaSelector, { language: editContext.currentItemDescriptor.language, visible: media.mediaSelectorVisible, onHide: media.handleHide, onMediaSelected: media.onMediaSelect, selectedIdPath: media.selectedMediaIdPath, mode: media.mediaSelectorMode, initialSearchTerm: media.initialSearchTerm })), _jsx(FieldEditorPopup, { ref: fieldEditorPopupRef }), _jsx(LicenseOverlay, {})] }) }) }) }) }) }));
 }
 //# sourceMappingURL=EditorShell.js.map