@lavalogic/scoria 0.38.2 → 0.38.4

package/dist/Components/Table/Misc/TableConfigurationModal.svelte

@@ -14,7 +14,7 @@
 	import { Size } from '../../../Types/Internal/Size.js';
 	import type { TabOption } from '../../../Types/Internal/TabOption.js';
 	import { Variant } from '../../../Types/Internal/Variant.js';
-	import { getContext } from 'svelte';
+	import { getContext, onMount } from 'svelte';
 	import { TableContext } from '../Types/Context/TableContext.svelte.js';
 	import type { Primitive } from '../Types/Context/TableInitOptions.js';
 	import type { DatatableView } from '../Types/Persistence/DatatableView.js';
@@ -48,6 +48,10 @@
 	// errors inline in its own body.
 	let errorMessage = $state<string | null>(null);
 
+	// True while the open-time fetch of saved views is in flight, so the
+	// My-tab shows "Loading…" rather than the "nothing saved yet" message.
+	let loadingViews = $state(false);
+
 	// Whether remote saved views are available at all. A table created
 	// without `datatableUuid` / `remoteLayouts` opts out of remote views;
 	// the modal then renders a graceful "unavailable" empty state and
@@ -88,11 +92,34 @@
 		errorMessage = null;
 		try {
 			await action();
-		} catch {
-			errorMessage = failureMessage;
+		} catch (e) {
+			// Surface the underlying error rather than swallowing it: the
+			// generic message alone has made failures hard to diagnose. The
+			// full error (with stack) goes to the console; its message is
+			// appended to the banner so it is visible without devtools.
+			console.error('[TableConfigurationModal] action failed:', e);
+			errorMessage = e instanceof Error ? `${failureMessage} (${e.message})` : failureMessage;
 		}
 	}
 
+	// Fetch the saved views fresh every time the modal opens. The boot-time
+	// fetch can miss (it races table construction / auth readiness), so the
+	// modal cannot rely on `_viewState.savedViews` already being populated -
+	// it pulls `GET /datatable-state` itself on open. A failure surfaces in
+	// the inline error banner via `guard` rather than being swallowed.
+	onMount(() => {
+		if (!remoteAvailable) {
+			return;
+		}
+		loadingViews = true;
+		void guard(
+			() => tableContext.refreshSavedViews(),
+			'Could not load saved layouts and filters. Please try again.'
+		).finally(() => {
+			loadingViews = false;
+		});
+	});
+
 	/** Apply a saved view to the live table. Pure local call - no await. */
 	function applyView(view: DatatableView): void {
 		errorMessage = null;
@@ -301,7 +328,9 @@
 		{/if}
 
 		{#if args.tab === 'mine'}
-			{#if myViews.length === 0}
+			{#if loadingViews}
+				<p class="muted">Loading saved {sectionNounPlural.toLowerCase()}…</p>
+			{:else if myViews.length === 0}
 				<p class="muted">
 					You have not saved any {sectionNounPlural.toLowerCase()} yet. Use the "Save a new {sectionNoun}"
 					tab to create one.
@@ -458,10 +487,6 @@
 					}}
 				/>
 				<Button
-					type="submit"
-					variant={Variant.Secondary}
-					size={Size.MediumSmall}
-					nowrap
 					disabled={!newViewName.trim()}
 					onclick={saveNewView}>Save Current</Button
 				>

package/dist/Components/Table/Types/Context/TableContext.svelte.d.ts

@@ -940,8 +940,10 @@ export declare class TableContext<T extends object, RowIdType extends Primitive>
      * against the now-known body of any persisted saved-view ref.
      *
      * No-op when remote saved views are disabled (`datatableUuid` /
-     * `remoteLayouts` absent). An adapter rejection is routed to
-     * `devCatch` and never breaks the table.
+     * `remoteLayouts` absent). An adapter rejection is propagated to the
+     * caller: the boot-time call site catches it with `devCatch`, while
+     * the Table Configuration modal routes it to its inline error banner
+     * so a failed fetch is visible rather than silently swallowed.
      */
     readonly refreshSavedViews: () => Promise<void>;
     /**

package/dist/Components/Table/Types/Context/TableContext.svelte.js

@@ -2439,8 +2439,10 @@ export class TableContext {
      * against the now-known body of any persisted saved-view ref.
      *
      * No-op when remote saved views are disabled (`datatableUuid` /
-     * `remoteLayouts` absent). An adapter rejection is routed to
-     * `devCatch` and never breaks the table.
+     * `remoteLayouts` absent). An adapter rejection is propagated to the
+     * caller: the boot-time call site catches it with `devCatch`, while
+     * the Table Configuration modal routes it to its inline error banner
+     * so a failed fetch is visible rather than silently swallowed.
      */
     refreshSavedViews = async () => {
         const adapter = this.remoteLayouts;
@@ -2448,18 +2450,13 @@ export class TableContext {
         if (!adapter || !datatableUuid) {
             return;
         }
-        try {
-            const views = await adapter.listViews(datatableUuid);
-            this._viewState.setSavedViews('layout', views.filter((view) => view.kind === 'layout'));
-            this._viewState.setSavedViews('filter', views.filter((view) => view.kind === 'filter'));
-            // A persisted saved-view ref restored at boot may have had an
-            // unknown body until now; reconcile both baselines against it.
-            this._reconcileBaselineForActiveView();
-            this._reconcileBaselineForActiveFilterView();
-        }
-        catch (e) {
-            devCatch(e);
-        }
+        const views = await adapter.listViews(datatableUuid);
+        this._viewState.setSavedViews('layout', views.filter((view) => view.kind === 'layout'));
+        this._viewState.setSavedViews('filter', views.filter((view) => view.kind === 'filter'));
+        // A persisted saved-view ref restored at boot may have had an
+        // unknown body until now; reconcile both baselines against it.
+        this._reconcileBaselineForActiveView();
+        this._reconcileBaselineForActiveFilterView();
     };
     /**
      * Drop the persisted live-layout snapshot for this table. The next

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lavalogic/scoria",
   "description": "Svelte components used for the FloWMS Web Frontend",
-  "version": "0.38.2",
+  "version": "0.38.4",
   "publishConfig": {
     "access": "restricted"
   },