@lobb-js/studio 0.47.0 → 0.49.0

package/dist/popup.js

@@ -0,0 +1,104 @@
+import { mount, unmount } from "svelte";
+import DataTable from "./components/dataTable/dataTable.svelte";
+import Popup from "./components/popup/popup.svelte";
+import { createStudioContextMap } from "./context";
+let activePopup = null;
+// Props that produced the currently-mounted popup. Tracked so we can
+// push them onto `popupHistory` when this popup is swapped out by a
+// `replaceLast: true` opener, enabling back navigation.
+let activePopupProps = null;
+// Stack of popup-props that were swapped out via `replaceLast: true`.
+// The back arrow re-opens the top of the stack; closing via X or the
+// backdrop wipes the whole stack (a deliberate full-reset, not back).
+const popupHistory = [];
+// Internal mount path used by both `openPopup` (user-initiated) and
+// `goBackPopup` (history-driven). The `fromBack` flag suppresses
+// pushing the outgoing popup onto the history stack — when the user
+// hits Back, the popup they're leaving shouldn't reappear if they hit
+// Back again.
+function mountPopup(studioContext, props, opts = {}) {
+    const targetElement = document.querySelector("main");
+    if (!targetElement)
+        throw new Error("main html element doesn't exist");
+    if (props.replaceLast && activePopup) {
+        // Swap: archive the outgoing popup's props (unless this open is
+        // itself a back-navigation) and unmount it. Outro plays in
+        // parallel with the new popup's intro.
+        if (!opts.fromBack && activePopupProps) {
+            popupHistory.push(activePopupProps);
+        }
+        const toUnmount = activePopup;
+        activePopup = null;
+        activePopupProps = null;
+        unmount(toUnmount, { outro: true });
+    }
+    else if (!props.replaceLast) {
+        // A fresh, non-stacking popup starts a new context — drop any
+        // accumulated back history from the previous chain.
+        popupHistory.length = 0;
+    }
+    const canGoBack = popupHistory.length > 0;
+    const mounted = mount(Popup, {
+        target: targetElement,
+        context: createStudioContextMap(studioContext),
+        props: {
+            component: props.component,
+            componentProps: props.componentProps,
+            title: props.title,
+            size: props.size,
+            hideHeader: props.hideHeader,
+            canGoBack,
+            onBack: () => goBackPopup(studioContext),
+            onClose: async () => {
+                if (activePopup === mounted) {
+                    activePopup = null;
+                    activePopupProps = null;
+                    // X / backdrop is a full reset — nuke the back
+                    // stack so reopening anything starts fresh.
+                    popupHistory.length = 0;
+                }
+                await unmount(mounted, { outro: true });
+            },
+        },
+    });
+    activePopup = mounted;
+    activePopupProps = props;
+}
+export function openPopup(studioContext, props) {
+    mountPopup(studioContext, props);
+}
+export function goBackPopup(studioContext) {
+    const prev = popupHistory.pop();
+    if (!prev)
+        return;
+    // Force `replaceLast: true` so the swap+outro path runs, and tell
+    // `mountPopup` not to push the current popup onto history.
+    mountPopup(studioContext, { ...prev, replaceLast: true }, { fromBack: true });
+}
+export function openDataTablePopup(studioContext, props) {
+    // DataTable reads `searchParams` once during its initial $state setup,
+    // so sort/limit need to be folded into the searchParams object before
+    // we mount. Built here in plain TS — no Svelte reactivity to escape.
+    const searchParams = {};
+    if (props.sort)
+        searchParams.sort = props.sort;
+    if (props.limit != null)
+        searchParams.limit = String(props.limit);
+    openPopup(studioContext, {
+        component: DataTable,
+        componentProps: {
+            collectionName: props.collectionName,
+            filter: props.filter,
+            searchParams,
+            showHeader: props.showHeader,
+            showFooter: props.showFooter,
+            showFilter: props.showFilter ?? false,
+            tabs: props.tabs,
+            view: props.view,
+        },
+        title: props.title ?? props.collectionName,
+        size: props.size,
+        hideHeader: props.hideHeader,
+        replaceLast: props.replaceLast,
+    });
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lobb-js/studio",
   "license": "UNLICENSED",
-  "version": "0.47.0",
+  "version": "0.49.0",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -37,16 +37,16 @@
     "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
-    "test": "playwright test",
-    "test:ui": "playwright test --ui",
+    "test": "vitest run tests",
+    "test:watch": "vitest tests",
     "test:unit": "vitest",
     "test-storybook": "vitest --project=storybook",
     "prepublishOnly": "./scripts/prepublish.sh",
     "postpublish": "./scripts/postpublish.sh"
   },
   "devDependencies": {
-    "@lobb-js/core": "^0.40.0",
     "@chromatic-com/storybook": "^4.1.2",
+    "@lobb-js/core": "^0.41.0",
     "@playwright/test": "^1.60.0",
     "@storybook/addon-a11y": "^10.0.1",
     "@storybook/addon-docs": "^10.0.1",
@@ -74,7 +74,8 @@
     "tw-animate-css": "^1.4.0",
     "typescript": "~5.9.3",
     "vite": "^8.0.13",
-    "vitest": "^4.0.5"
+    "vitest": "^4.0.5",
+    "vitest-browser-svelte": "^2.1.1"
   },
   "peerDependencies": {
     "svelte": "^5.0.0",
@@ -91,7 +92,7 @@
     "@codemirror/view": "^6.39.12",
     "@dagrejs/dagre": "^1.1.5",
     "@internationalized/date": "^3.12.0",
-    "@lobb-js/sdk": "^0.4.0",
+    "@lobb-js/sdk": "^0.5.0",
     "@lucide/svelte": "^0.563.1",
     "@tailwindcss/vite": "^4.3.0",
     "@tiptap/core": "^3.0.0",

package/src/lib/actions.ts

@@ -3,8 +3,6 @@ import ConfirmationDialog from "./components/confirmationDialog/confirmationDial
 import CreateDetailView from "./components/detailView/create/createDetailView.svelte";
 import UpdateDetailView from "./components/detailView/update/updateDetailView.svelte";
 import DataTableDrawer from "./components/dataTableDrawer/dataTableDrawer.svelte";
-import DataTable from "./components/dataTable/dataTable.svelte";
-import Popup from "./components/popup/popup.svelte";
 import type { CreateDetailViewProp } from "./components/detailView/create/createDetailView.svelte";
 import type { UpdateDetailViewProp } from "./components/detailView/update/updateDetailView.svelte";
 import type { StudioContext } from "./context";
@@ -22,49 +20,6 @@ export interface OpenDataTableDrawerProps {
     tabs?: CollectionTab[];
 }
 
-export interface OpenPopupProps {
-    // Component rendered inside the popup. It receives `componentProps`
-    // spread onto it plus `onClose` so it can close the popup itself.
-    component: any;
-    componentProps?: Record<string, any>;
-    title?: string;
-    // `default` is the wide data-table popup (max-w-7xl, fixed 85vh).
-    // `sm` is the narrow detail-view variant (max-w-2xl, content-height).
-    size?: "default" | "sm";
-    // When true, the popup renders no chrome at all (no title bar, no
-    // close button). The body uses the `onClose` prop the popup passes
-    // it to render its own close affordance.
-    hideHeader?: boolean;
-    // Close the previously-opened popup before mounting this one —
-    // drill-down semantics. Only popups participate; drawers don't.
-    replaceLast?: boolean;
-}
-
-export interface OpenDataTablePopupProps {
-    collectionName: string;
-    filter?: Record<string, any>;
-    sort?: Record<string, "asc" | "desc">;
-    limit?: number;
-    title?: string;
-    showHeader?: boolean;
-    showFooter?: boolean;
-    showFilter?: boolean;
-    tabs?: CollectionTab[];
-    view?: { id: string; [key: string]: any };
-    replaceLast?: boolean;
-    size?: "default" | "sm";
-    hideHeader?: boolean;
-}
-
-let activePopup: any = null;
-
-function closeActivePopupIfRequested(replaceLast: boolean | undefined) {
-    if (!replaceLast || !activePopup) return;
-    const toUnmount = activePopup;
-    activePopup = null;
-    unmount(toUnmount, { outro: true });
-}
-
 export function showDialog(title: string, description: string): Promise<boolean> {
     return new Promise((resolve) => {
         const targetElement = document.querySelector("main");
@@ -145,53 +100,8 @@ export function openDataTableDrawer(studioContext: StudioContext, props: OpenDat
     });
 }
 
-export function openPopup(studioContext: StudioContext, props: OpenPopupProps) {
-    const targetElement = document.querySelector("main");
-    if (!targetElement) throw new Error("main html element doesn't exist");
-
-    closeActivePopupIfRequested(props.replaceLast);
-
-    const mounted = mount(Popup, {
-        target: targetElement,
-        context: createStudioContextMap(studioContext),
-        props: {
-            component: props.component,
-            componentProps: props.componentProps,
-            title: props.title,
-            size: props.size,
-            hideHeader: props.hideHeader,
-            onClose: async () => {
-                if (activePopup === mounted) activePopup = null;
-                await unmount(mounted, { outro: true });
-            },
-        },
-    });
-    activePopup = mounted;
-}
-
-export function openDataTablePopup(studioContext: StudioContext, props: OpenDataTablePopupProps) {
-    // DataTable reads `searchParams` once during its initial $state setup,
-    // so sort/limit need to be folded into the searchParams object before
-    // we mount. Built here in plain TS — no Svelte reactivity to escape.
-    const searchParams: Record<string, any> = {};
-    if (props.sort) searchParams.sort = props.sort;
-    if (props.limit != null) searchParams.limit = String(props.limit);
-
-    openPopup(studioContext, {
-        component: DataTable,
-        componentProps: {
-            collectionName: props.collectionName,
-            filter: props.filter,
-            searchParams,
-            showHeader: props.showHeader,
-            showFooter: props.showFooter,
-            showFilter: props.showFilter ?? false,
-            tabs: props.tabs,
-            view: props.view,
-        },
-        title: props.title ?? props.collectionName,
-        size: props.size,
-        hideHeader: props.hideHeader,
-        replaceLast: props.replaceLast,
-    });
-}
+// Popup helpers (openPopup, openDataTablePopup, goBackPopup) live in
+// `./popup` — they share enough state (active overlay, back-history)
+// to warrant their own module.
+export { openPopup, openDataTablePopup, goBackPopup } from "./popup";
+export type { OpenPopupProps, OpenDataTablePopupProps } from "./popup";

package/src/lib/components/detailView/detailView.svelte

@@ -1,17 +1,11 @@
 <script lang="ts">
-    import { CircleHelp } from "lucide-svelte";
-    import * as Tooltip from "../ui/tooltip";
     import { getStudioContext } from "../../context";
-    import ExtensionsComponents from "../extensionsComponents.svelte";
-    import { getExtensionUtils } from "../../extensions/extensionUtils";
-    import { getField, getFieldIcon } from "../dataTable/utils";
     import FieldInput from "./fieldInput.svelte";
-    import { getFieldTypeLabel } from "../../utils";
 
     interface Props {
         collectionName: string;
         entry: Record<string, any>;
-        fieldsErrors?: Record<string, string[]>;
+        fieldsErrors?: Record<string, any>;
         changedFields?: string[];
     }
 
@@ -22,9 +16,7 @@
         changedFields = [],
     }: Props = $props();
 
-    const { lobb, ctx } = getStudioContext();
-    // Singleton collections only ever have one row, and `id` is auto-
-    // generated for that row — there's no value to showing it in the form.
+    const { ctx } = getStudioContext();
     const fieldNames = $derived(
         Object.keys(ctx.meta.collections[collectionName].fields).filter(
             (fieldName) =>
@@ -35,58 +27,16 @@
 
 <div class="grid grid-cols-2 gap-4 p-4">
     {#each fieldNames as fieldName}
-        {#if !ctx.meta.collections[collectionName].fields[fieldName]?.ui?.hidden}
-            {@const field = getField(ctx, fieldName, collectionName)}
-            {@const FieldIcon = getFieldIcon(ctx, fieldName, collectionName)}
-            {@const description = ctx.meta.collections[collectionName].fields[fieldName]?.description}
-            {@const fieldDef = ctx.meta.collections[collectionName].fields[fieldName]}
-            {@const isFullWidth = field.type === "text" || field.type === "polymorphic" || fieldDef?.ui?.input?.type === "richtext" || fieldDef?.ui?.span === 2}
-            <div class="flex flex-col gap-2 {isFullWidth ? 'col-span-2' : 'col-span-1'}">
-                <div class="flex flex-1 items-end justify-between gap-2 text-xs">
-                    <div class="flex items-center gap-1.5">
-                        <ExtensionsComponents
-                            name="detailView.field.label"
-                            utils={getExtensionUtils(lobb, ctx)}
-                            {collectionName}
-                            {fieldName}
-                            bind:value={entry[fieldName]}
-                        >
-                            <div class="flex items-center gap-1.5">
-                                <div class="h-fit">{field.label}</div>
-                                <div class="flex h-fit items-center gap-1 text-[0.7rem] text-muted-foreground">
-                                    <FieldIcon size="12" />
-                                    {getFieldTypeLabel(field.type)}
-                                </div>
-                            </div>
-                        </ExtensionsComponents>
-                        {#if description}
-                            <Tooltip.Root>
-                                <Tooltip.Trigger>
-                                    <CircleHelp size="12" class="text-muted-foreground" />
-                                </Tooltip.Trigger>
-                                <Tooltip.Content class="max-w-64 text-xs">
-                                    {description}
-                                </Tooltip.Content>
-                            </Tooltip.Root>
-                        {/if}
-                    </div>
-                    <div>
-                        <ExtensionsComponents
-                            name="dvFields.topRight.{collectionName}.{fieldName}"
-                            utils={getExtensionUtils(lobb, ctx)}
-                            bind:value={entry[fieldName]}
-                        />
-                    </div>
-                </div>
-                <FieldInput
-                    {collectionName}
-                    {fieldName}
-                    bind:value={entry[fieldName]}
-                    bind:entry
-                    errorMessages={fieldsErrors[fieldName]}
-                    changed={changedFields.includes(fieldName)}
-                />
-            </div>
+        {@const fieldDef = ctx.meta.collections[collectionName].fields[fieldName]}
+        {#if !(fieldDef?.ui?.hidden) || (fieldDef as any)?.embedded}
+            <FieldInput
+                {collectionName}
+                {fieldName}
+                bind:value={entry[fieldName]}
+                bind:entry
+                errorMessages={fieldsErrors[fieldName]}
+                changed={changedFields.includes(fieldName)}
+            />
         {/if}
     {/each}
 </div>