@open-mercato/ui 0.6.7-develop.6860.1.f796b22e80 → 0.6.7

package/dist/backend/DataTable.js

@@ -46,7 +46,6 @@ import { useT } from "@open-mercato/shared/lib/i18n/context";
 import { flash } from "./FlashMessages.js";
 import { useConfirmDialog } from "./confirm-dialog/index.js";
 import { ComponentReplacementHandles } from "@open-mercato/shared/modules/widgets/component-registry";
-import { dataTableExtensionSpotId, extensionSpotChildId } from "@open-mercato/shared/modules/widgets/extension-points";
 import { insertByInjectionPlacement } from "@open-mercato/shared/modules/widgets/injection-position";
 import { useVirtualizer } from "@tanstack/react-virtual";
 import { getDefaultOperator, isAdvancedFilterState, flatToTree } from "@open-mercato/shared/lib/query/advanced-filter";
@@ -75,7 +74,6 @@ import {
 } from "@dnd-kit/sortable";
 import { CSS } from "@dnd-kit/utilities";
 import { createLogger } from "@open-mercato/shared/lib/logger";
-import { clearAllPerspectiveState, PERSPECTIVE_COOKIE_PREFIX, PERSPECTIVE_STORAGE_PREFIX } from "./perspectiveState.js";
 const logger = createLogger("ui").child({ component: "DataTable" });
 let refreshScheduled = false;
 function scheduleRouterRefresh(router) {
@@ -201,6 +199,8 @@ function resolveExportSections(config) {
   addSection("full", config.full, "Full data export");
   return sections;
 }
+const PERSPECTIVE_COOKIE_PREFIX = "om_table_perspective";
+const PERSPECTIVE_STORAGE_PREFIX = "om_table_perspective_snapshot";
 const COLUMN_MIN_WIDTH = 60;
 const COLUMN_MAX_WIDTH = 900;
 function formatDurationLabel(durationMs) {
@@ -932,7 +932,7 @@ function DataTable({
     if (injectionSpotId?.startsWith("data-table:")) return injectionSpotId.slice("data-table:".length);
     return null;
   }, [injectionSpotId, perspective?.tableId, extensionTableIdProp]);
-  const resolvedInjectionSpotId = injectionSpotId ?? (perspective?.tableId ? dataTableExtensionSpotId(perspective.tableId) : null) ?? (extensionTableIdProp ? dataTableExtensionSpotId(extensionTableIdProp) : null);
+  const resolvedInjectionSpotId = injectionSpotId ?? (perspective?.tableId ? `data-table:${perspective.tableId}` : null) ?? (extensionTableIdProp ? `data-table:${extensionTableIdProp}` : null);
   const resolvedReplacementHandle = replacementHandle ?? ComponentReplacementHandles.dataTable(extensionTableId ?? "unknown");
   const baseInjectionContext = React.useMemo(
     () => {
@@ -944,32 +944,32 @@ function DataTable({
     [injectionContext, perspective?.tableId, extensionTableId, title]
   );
   const headerInjectionSpotId = React.useMemo(
-    () => resolvedInjectionSpotId ? extensionSpotChildId(resolvedInjectionSpotId, "header") : null,
+    () => resolvedInjectionSpotId ? `${resolvedInjectionSpotId}:header` : null,
     [resolvedInjectionSpotId]
   );
   const toolbarInjectionSpotId = React.useMemo(
-    () => resolvedInjectionSpotId ? extensionSpotChildId(resolvedInjectionSpotId, "toolbar") : null,
+    () => resolvedInjectionSpotId ? `${resolvedInjectionSpotId}:toolbar` : null,
     [resolvedInjectionSpotId]
   );
   const searchTrailingInjectionSpotId = React.useMemo(
-    () => resolvedInjectionSpotId ? extensionSpotChildId(resolvedInjectionSpotId, "search-trailing") : null,
+    () => resolvedInjectionSpotId ? `${resolvedInjectionSpotId}:search-trailing` : null,
     [resolvedInjectionSpotId]
   );
   const footerInjectionSpotId = React.useMemo(
-    () => resolvedInjectionSpotId ? extensionSpotChildId(resolvedInjectionSpotId, "footer") : null,
+    () => resolvedInjectionSpotId ? `${resolvedInjectionSpotId}:footer` : null,
     [resolvedInjectionSpotId]
   );
   const { widgets: columnWidgets } = useInjectionDataWidgets(
-    extensionTableId ? dataTableExtensionSpotId(extensionTableId, "columns") : "__disabled__:columns"
+    extensionTableId ? `data-table:${extensionTableId}:columns` : "__disabled__:columns"
   );
   const { widgets: rowActionWidgets } = useInjectionDataWidgets(
-    extensionTableId ? dataTableExtensionSpotId(extensionTableId, "row-actions") : "__disabled__:row-actions"
+    extensionTableId ? `data-table:${extensionTableId}:row-actions` : "__disabled__:row-actions"
   );
   const { widgets: bulkActionWidgets } = useInjectionDataWidgets(
-    extensionTableId ? dataTableExtensionSpotId(extensionTableId, "bulk-actions") : "__disabled__:bulk-actions"
+    extensionTableId ? `data-table:${extensionTableId}:bulk-actions` : "__disabled__:bulk-actions"
   );
   const { widgets: filterWidgets } = useInjectionDataWidgets(
-    extensionTableId ? dataTableExtensionSpotId(extensionTableId, "filters") : "__disabled__:filters"
+    extensionTableId ? `data-table:${extensionTableId}:filters` : "__disabled__:filters"
   );
   const injectedColumnDefs = React.useMemo(() => {
     const entries = [];
@@ -1231,19 +1231,18 @@ function DataTable({
       return changed ? filtered : prev;
     });
   }, [table, mergedColumns]);
-  const visibilitySeededByStoredSettings = React.useRef(Boolean(mergedInitialSettings?.columnVisibility));
-  const autoHiddenColumnIds = React.useRef(/* @__PURE__ */ new Set());
+  const initialVisibilityApplied = React.useRef(Boolean(mergedInitialSettings?.columnVisibility));
   React.useEffect(() => {
-    if (visibilitySeededByStoredSettings.current) return;
+    if (initialVisibilityApplied.current) return;
     const hidden = {};
     table.getAllLeafColumns().forEach((column) => {
       const hiddenMeta = column.columnDef?.meta?.hidden;
-      if (!hiddenMeta || autoHiddenColumnIds.current.has(column.id)) return;
-      hidden[column.id] = false;
-      autoHiddenColumnIds.current.add(column.id);
+      if (hiddenMeta) hidden[column.id] = false;
     });
-    if (!Object.keys(hidden).length) return;
-    setColumnVisibility((prev) => ({ ...hidden, ...prev }));
+    if (Object.keys(hidden).length) {
+      setColumnVisibility((prev) => ({ ...hidden, ...prev }));
+    }
+    initialVisibilityApplied.current = true;
   }, [table, mergedColumns]);
   const getCurrentSettings = React.useCallback(() => {
     const visibility = {};
@@ -2556,7 +2555,6 @@ function DataTable({
 }
 export {
   DataTable,
-  clearAllPerspectiveState,
   readPerspectiveSnapshot,
   sanitizePerspectiveSettings,
   withDataTableNamespaces,