@lobb-js/lobb-ext-storage 0.15.1 → 0.16.0

package/dist/index.js

@@ -1,5 +1,5 @@
 import FileExplorerPage from "./lib/components/pages/fileExplorerPage.svelte";
-import ForeignKeyComponent from "./lib/components/foreignKeyComponent.svelte";
+import FilePickerField from "./lib/components/FilePickerField.svelte";
 import ChildrenFileExplorer from "./lib/components/childrenFileExplorer.svelte";
 import ExplorerNotSupported from "./lib/components/explorerNotSupported.svelte";
 export default function extension(utils) {
@@ -7,7 +7,7 @@ export default function extension(utils) {
         name: "storage",
         components: {
             "pages.file_manager": FileExplorerPage,
-            "detailView.fields.foreignKey.storage_fs": ForeignKeyComponent,
+            "detailView.fields.foreignKey.storage_fs": FilePickerField,
             "detailView.create.subRecords.storage_fs": ExplorerNotSupported,
             "detailView.update.subRecords.storage_fs": ChildrenFileExplorer,
             "listView.entry.children.storage_fs": ChildrenFileExplorer,

package/dist/lib/components/FilePickerField.svelte

@@ -0,0 +1,48 @@
+<script lang="ts">
+    import type { ExtensionProps } from "@lobb-js/studio";
+    import FileExplorer, { type Entry } from "./fileExplorer.svelte";
+    import FileIcon from "./fileIcon.svelte";
+
+    interface Props extends ExtensionProps {
+        value: Entry;
+        header?: any;
+        isDisabled?: boolean;
+        errorMessages?: any;
+    }
+
+    let { utils, value = $bindable(), header, isDisabled = false, errorMessages = [], ...props }: Props = $props();
+
+    const FilePlus2 = $derived(utils.components.Icons.FilePlus2);
+    const Drawer = $derived(utils.components.Drawer);
+    const FieldWrapper = $derived(utils.components.FieldWrapper);
+
+    let openDrawer = $state(false);
+
+    async function onHideHandle() {
+        openDrawer = false;
+    }
+
+    async function handleOnFileSelect(entry: Entry) {
+        value = entry;
+        openDrawer = false;
+    }
+</script>
+
+<FieldWrapper span={2} {isDisabled} {errorMessages} {header}>
+    {#snippet children()}
+        <button onclick={() => openDrawer = true} class="flex flex-col items-center gap-2 w-full border p-4 rounded-md bg-muted">
+            {#if value}
+                <FileIcon entry={value} {utils} />
+            {:else}
+                <FilePlus2 class="text-muted-foreground" />
+                <div class="text-sm text-muted-foreground">Click to add an asset</div>
+            {/if}
+        </button>
+
+        {#if openDrawer}
+            <Drawer onHide={onHideHandle}>
+                <FileExplorer onFileSelect={handleOnFileSelect} utils={utils} {...props} />
+            </Drawer>
+        {/if}
+    {/snippet}
+</FieldWrapper>

package/dist/lib/components/FilePickerField.svelte.d.ts

@@ -0,0 +1,14 @@
+import { SvelteComponentTyped } from "svelte";
+declare const __propDef: {
+    props: Record<string, never>;
+    events: {
+        [evt: string]: CustomEvent<any>;
+    };
+    slots: {};
+};
+export type FilePickerFieldProps = typeof __propDef.props;
+export type FilePickerFieldEvents = typeof __propDef.events;
+export type FilePickerFieldSlots = typeof __propDef.slots;
+export default class FilePickerField extends SvelteComponentTyped<FilePickerFieldProps, FilePickerFieldEvents, FilePickerFieldSlots> {
+}
+export {};

package/dist/lib/components/fileExplorer.svelte

@@ -66,7 +66,7 @@
 		contextMenuOpen = false;
 		const newFolderName = prompt("Please enter a name for the new folder:");
 		if (newFolderName) {
-			const response = await utils.lobb.createOne("storage_fs", {
+			const response = await utils.lobb.runAction("storage_create", {
 				data: { name: newFolderName, path: location, type: "directory", ...foreignKeyObject },
 			});
 			const result = await response.json();
@@ -89,8 +89,8 @@
 
 		async function uploadFile(file: File) {
 			if (file) {
-				const response = await utils.lobb.createOne(
-					"storage_fs",
+				const response = await utils.lobb.runAction(
+					"storage_create",
 					{ data: { path: location, ...foreignKeyObject } },
 					file,
 				);
@@ -133,10 +133,9 @@
 		if (!entryId) {
 			throw new Error("The id of the entry is (null) for some reason");
 		}
-		const response = await utils.lobb.deleteOne(
-			"storage_fs",
-			entryId,
-		);
+		const response = await utils.lobb.runAction("storage_delete", {
+			id: String(entryId),
+		});
 		const result = await response.json();
 		if (result.status >= 400) {
 			return;

package/dist/lib/components/fileIcon.svelte

@@ -16,7 +16,7 @@
     class="pointer-events-none flex flex-col items-center gap-2 min-w-28"
 >
     {#if entry.file_mime_type.startsWith("image/")}
-        <div class="rounded-md w-24 h-24 bg-center bg-contain bg-no-repeat" style="background-image: url('{utils.lobb.lobbUrl}/api/collections/storage_fs/{entry.id}?action=view');"></div>
+        <div class="rounded-md w-24 h-24 bg-center bg-contain bg-no-repeat" style="background-image: url('{utils.lobb.lobbUrl}/api/actions/storage_read?id={entry.id}&action=view');"></div>
     {:else}
         <div
             class="pointer-events-none relative"

package/extensions/storage/index.ts

@@ -1,6 +1,5 @@
 import type { Extension } from "@lobb-js/core";
 import type { ExtensionConfig } from "./config/extensionConfigSchema.ts";
-export type { StorageFsCreateOne, StorageFsFindOne } from "./types.ts";
 
 import packageJson from "../../package.json" with { type: "json" };
 import { collections } from "./collections/index.ts";
@@ -8,6 +7,7 @@ import { meta } from "./meta.ts";
 import { migrations } from "./migrations.ts";
 import { openapi } from "./openapi.ts";
 import { getWorkflows } from "./workflows/index.ts";
+import { getStorageActions } from "./workflows/actions.ts";
 
 export default function storage(extensionConfig: ExtensionConfig): Extension {
   return {
@@ -15,6 +15,7 @@ export default function storage(extensionConfig: ExtensionConfig): Extension {
     name: "storage",
     icon: "Folders",
     collections: (_lobb: any) => collections(extensionConfig),
+    actions: getStorageActions(extensionConfig),
     migrations: migrations,
     meta: meta,
     workflows: getWorkflows(extensionConfig),

package/extensions/storage/migrations.ts

@@ -1,59 +1,7 @@
-import type { Lobb, Migrations } from "@lobb-js/core";
-
-/**
- * Backfill `parent_id` for any row in storage_fs that still has it null.
- * Resolves the parent by walking the existing `path` string and matching
- * the (parent_path, name) of every directory record. Idempotent — running
- * it twice is safe; rows that already have parent_id are skipped.
- */
-export async function backfillStorageFsParentId(lobb: Lobb): Promise<void> {
-  const rows = (await lobb.collectionService.findAll({
-    collectionName: "storage_fs",
-    params: { limit: 1000000 },
-  })).data;
-  if (rows.length === 0) return;
-
-  // Index every directory by (path, name) so we can resolve any child's
-  // parent in one lookup. This is the same key that resolveParentIdFromPath
-  // walks at runtime, just precomputed.
-  const dirByKey = new Map<string, number>();
-  for (const r of rows as any[]) {
-    if (r.type === "directory") {
-      dirByKey.set(`${r.path}::${r.name}`, r.id);
-    }
-  }
-
-  function parentIdFor(row: any): number | null {
-    const path: string = row.path ?? "/";
-    if (path === "/" || path === "") return null;
-    const segments = path.split("/").filter(Boolean);
-    if (segments.length === 0) return null;
-    const lastName = segments[segments.length - 1];
-    // No-trailing-slash convention: root is "/", deeper levels are
-    // "/foo" or "/foo/bar". Older data may have either form; this
-    // resolves only the normalized one. Loose-format rows get retried
-    // on the next pass.
-    const parentDirPath =
-      segments.length > 1 ? "/" + segments.slice(0, -1).join("/") : "/";
-    const key = `${parentDirPath}::${lastName}`;
-    return dirByKey.get(key) ?? null;
-  }
-
-  for (const row of rows as any[]) {
-    if (row.parent_id != null) continue;
-    const pid = parentIdFor(row);
-    if (pid != null) {
-      await lobb.collectionStore.updateOne({
-        collectionName: "storage_fs",
-        id: row.id,
-        data: { parent_id: pid },
-      });
-    }
-  }
-}
+import type { Migrations } from "@lobb-js/core";
 
 // No built-in migrations for the storage extension. The `parent_id` column
-// is set on every new write by the service; existing rows from environments
-// that predate parent_id can be reconciled by calling
-// `backfillStorageFsParentId` from a host-app migration when needed.
+// is set on every new write by the service; no backfill ships with the
+// package — host apps that need to reconcile legacy rows can write their
+// own migration against `storage_fs`.
 export const migrations: Migrations = {};

package/extensions/storage/studio/index.ts

@@ -1,7 +1,7 @@
 import type { Extension, ExtensionUtils } from "@lobb-js/studio";
 
 import FileExplorerPage from "./lib/components/pages/fileExplorerPage.svelte";
-import ForeignKeyComponent from "./lib/components/foreignKeyComponent.svelte";
+import FilePickerField from "./lib/components/FilePickerField.svelte";
 import ChildrenFileExplorer from "./lib/components/childrenFileExplorer.svelte";
 import ExplorerNotSupported from "./lib/components/explorerNotSupported.svelte";
 
@@ -10,7 +10,7 @@ export default function extension(utils: ExtensionUtils): Extension {
     name: "storage",
     components: {
       "pages.file_manager": FileExplorerPage,
-      "detailView.fields.foreignKey.storage_fs": ForeignKeyComponent,
+      "detailView.fields.foreignKey.storage_fs": FilePickerField,
       "detailView.create.subRecords.storage_fs": ExplorerNotSupported,
       "detailView.update.subRecords.storage_fs": ChildrenFileExplorer,
       "listView.entry.children.storage_fs": ChildrenFileExplorer,

package/extensions/storage/studio/lib/components/FilePickerField.svelte

@@ -0,0 +1,48 @@
+<script lang="ts">
+    import type { ExtensionProps } from "@lobb-js/studio";
+    import FileExplorer, { type Entry } from "./fileExplorer.svelte";
+    import FileIcon from "./fileIcon.svelte";
+
+    interface Props extends ExtensionProps {
+        value: Entry;
+        header?: any;
+        isDisabled?: boolean;
+        errorMessages?: any;
+    }
+
+    let { utils, value = $bindable(), header, isDisabled = false, errorMessages = [], ...props }: Props = $props();
+
+    const FilePlus2 = $derived(utils.components.Icons.FilePlus2);
+    const Drawer = $derived(utils.components.Drawer);
+    const FieldWrapper = $derived(utils.components.FieldWrapper);
+
+    let openDrawer = $state(false);
+
+    async function onHideHandle() {
+        openDrawer = false;
+    }
+
+    async function handleOnFileSelect(entry: Entry) {
+        value = entry;
+        openDrawer = false;
+    }
+</script>
+
+<FieldWrapper span={2} {isDisabled} {errorMessages} {header}>
+    {#snippet children()}
+        <button onclick={() => openDrawer = true} class="flex flex-col items-center gap-2 w-full border p-4 rounded-md bg-muted">
+            {#if value}
+                <FileIcon entry={value} {utils} />
+            {:else}
+                <FilePlus2 class="text-muted-foreground" />
+                <div class="text-sm text-muted-foreground">Click to add an asset</div>
+            {/if}
+        </button>
+
+        {#if openDrawer}
+            <Drawer onHide={onHideHandle}>
+                <FileExplorer onFileSelect={handleOnFileSelect} utils={utils} {...props} />
+            </Drawer>
+        {/if}
+    {/snippet}
+</FieldWrapper>

package/extensions/storage/studio/lib/components/fileExplorer.svelte

@@ -66,7 +66,7 @@
 		contextMenuOpen = false;
 		const newFolderName = prompt("Please enter a name for the new folder:");
 		if (newFolderName) {
-			const response = await utils.lobb.createOne("storage_fs", {
+			const response = await utils.lobb.runAction("storage_create", {
 				data: { name: newFolderName, path: location, type: "directory", ...foreignKeyObject },
 			});
 			const result = await response.json();
@@ -89,8 +89,8 @@
 
 		async function uploadFile(file: File) {
 			if (file) {
-				const response = await utils.lobb.createOne(
-					"storage_fs",
+				const response = await utils.lobb.runAction(
+					"storage_create",
 					{ data: { path: location, ...foreignKeyObject } },
 					file,
 				);
@@ -133,10 +133,9 @@
 		if (!entryId) {
 			throw new Error("The id of the entry is (null) for some reason");
 		}
-		const response = await utils.lobb.deleteOne(
-			"storage_fs",
-			entryId,
-		);
+		const response = await utils.lobb.runAction("storage_delete", {
+			id: String(entryId),
+		});
 		const result = await response.json();
 		if (result.status >= 400) {
 			return;

package/extensions/storage/studio/lib/components/fileIcon.svelte

@@ -16,7 +16,7 @@
     class="pointer-events-none flex flex-col items-center gap-2 min-w-28"
 >
     {#if entry.file_mime_type.startsWith("image/")}
-        <div class="rounded-md w-24 h-24 bg-center bg-contain bg-no-repeat" style="background-image: url('{utils.lobb.lobbUrl}/api/collections/storage_fs/{entry.id}?action=view');"></div>
+        <div class="rounded-md w-24 h-24 bg-center bg-contain bg-no-repeat" style="background-image: url('{utils.lobb.lobbUrl}/api/actions/storage_read?id={entry.id}&action=view');"></div>
     {:else}
         <div
             class="pointer-events-none relative"

package/extensions/storage/utils.ts

@@ -1,22 +1,6 @@
-import type { Context } from "hono";
 import type { Lobb } from "@lobb-js/core";
 import { LobbError } from "@lobb-js/core";
 
-export async function getFileFromRequestBody(
-  c: Context,
-): Promise<File | undefined> {
-  const contentType = c.req.header("Content-Type");
-  if (contentType?.startsWith("multipart/form-data")) {
-    const formData = await c.req.formData();
-    const entry = formData.get("file");
-    if (entry instanceof File) {
-      return entry;
-    }
-  }
-  // return undefined if no file or wrong content type
-  return undefined;
-}
-
 // ── parent_id / path helpers ──────────────────────────────────────────────
 //
 // The storage_fs table uses parent_id as the source of truth for the

package/extensions/storage/workflows/actionController.ts

@@ -0,0 +1,87 @@
+import type { Workflow } from "@lobb-js/core";
+import type { Context } from "hono";
+
+// The two storage operations that need the HTTP layer (a multipart file, or a
+// binary response) are served by adapting the *action* endpoints rather than
+// touching the collection CRUD routes (which stay clean and predictable).
+//
+//   POST /api/actions/storage_create   (multipart)        → upload a file
+//   GET  /api/actions/storage_read?action=view|download   → stream file bytes
+//
+// Everything else flows through the default JSON action handling.
+
+export function getActionControllerWorkflows(): Workflow[] {
+  return [
+    {
+      name: "storageActionController",
+      eventName: "core.actions.controller.override",
+      handler: async (input, ctx) => {
+        const name = input.name as string;
+        const context = input.context as Context;
+
+        // Multipart upload → pull the File off the request and pass it to the action.
+        if (name === "storage_create") {
+          const contentType = context.req.header("Content-Type");
+          if (!contentType?.startsWith("multipart/form-data")) return; // JSON create → default flow
+
+          const force = context.req.query("force") === "true";
+          const body = await context.req.parseBody({ all: true });
+          const fileEntry = body["file"];
+
+          if (!fileEntry) {
+            throw new ctx.LobbError({ code: "BAD_REQUEST", message: "No files were provided." });
+          }
+          if (Array.isArray(fileEntry)) {
+            throw new ctx.LobbError({ code: "BAD_REQUEST", message: "Only one file can be uploaded at a time." });
+          }
+          if (!(fileEntry instanceof File)) {
+            throw new ctx.LobbError({ code: "BAD_REQUEST", message: "The file shouldnt be a string" });
+          }
+
+          const extraData: Record<string, string> = {};
+          for (const [key, val] of Object.entries(body)) {
+            if (key !== "file" && key !== "payload" && typeof val === "string") {
+              extraData[key] = val;
+            }
+          }
+
+          const result = await ctx.lobb.runAction("storage_create", {
+            file: fileEntry,
+            force,
+            data: extraData,
+          });
+
+          return context.json(result, 201);
+        }
+
+        // GET file view/download → stream raw bytes (browser-embeddable).
+        if (name === "storage_read") {
+          const action = context.req.query("action");
+          if (action !== "view" && action !== "download") return; // non-binary read → default flow
+
+          const id = context.req.query("id");
+          if (!id) {
+            throw new ctx.LobbError({ code: "BAD_REQUEST", message: "The (id) query param is required." });
+          }
+
+          const result = await ctx.lobb.runAction("storage_read", { id, readFile: true });
+
+          if (!result.data) {
+            throw new ctx.LobbError({ code: "NOT_FOUND", message: "The record is not found" });
+          }
+
+          const headers: Record<string, string> = {
+            "Content-Type": result.data.file_mime_type,
+          };
+          if (action === "download") {
+            headers["Content-Disposition"] = `attachment; filename="${result.data.name}"`;
+          }
+
+          return new Response(result.file.buffer as ArrayBuffer, { status: 200, headers });
+        }
+
+        return;
+      },
+    },
+  ];
+}

package/extensions/storage/workflows/{services.ts → actions.ts}

@@ -1,4 +1,4 @@
-import type { Workflow } from "@lobb-js/core";
+import type { ActionsConfig } from "@lobb-js/core";
 import { LobbError } from "@lobb-js/core";
 import type { ExtensionConfig } from "../config/extensionConfigSchema.ts";
 import { initStorageAdapter } from "../adapters/index.ts";
@@ -9,43 +9,20 @@ import {
   resolveParentIdFromPath,
 } from "../utils.ts";
 
-// All writes funnel through these service overrides so the invariant
-// `path === parent.path + parent.name + "/"` is maintained on every change.
-// Reads stay free — both `parent_id` and `path` are on the row.
+// The storage_fs hierarchy invariant — `path === parent.path + parent.name` —
+// is maintained by these actions on every write. `parent_id` is the source of
+// truth; `path` is a derived cache for fast subtree queries and breadcrumbs.
 //
-// `parent_id` is the source of truth for the hierarchy; `path` is a cached
-// view used for fast subtree queries and breadcrumbs. Callers may pass
-// either (or both) to create/update — we resolve to parent_id and recompute
-// path from the tree.
-
-export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow[] {
-  return [
-    {
-      name: "storageFindOneService",
-      eventName: "core.service.findOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName === "storage_fs") {
-          const result = await ctx.lobb.collectionStore.findOne({
-            collectionName: "storage_fs",
-            id: input.id,
-          });
-
-          if (result.data?.type === "file" && input.readFile) {
-            const adapter = initStorageAdapter(extensionConfig);
-            const file = await adapter.readFile(result.data.id);
-            return { ...result, file: new Uint8Array(file) };
-          }
-
-          return result;
-        }
-      },
-    },
-    {
-      name: "storageCreateOneServiceOverride",
-      eventName: "core.service.createOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName !== "storage_fs") return;
+// Complex/multi-step operations (upload, create, rename/move, cascade delete)
+// live here as actions. Reads stay free on the collection routes; only file
+// view/download and multipart upload are adapted from the collection route into
+// `storage_read` / `storage_create` by the controller workflows.
 
+export function getStorageActions(extensionConfig: ExtensionConfig): ActionsConfig {
+  return {
+    // Create a file (when `file` is present) or a directory / metadata record.
+    storage_create: {
+      handler: async (input: any, ctx) => {
         const raw = input.data ?? {};
         const explicitParentId = raw.parent_id ?? null;
         const pathInput: string = raw.path ?? "/";
@@ -89,8 +66,7 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
           return result;
         }
 
-        // Non-file writes (directories or pure metadata records) — let the
-        // generic store handle it after we stamp parent_id + path.
+        // Non-file writes (directories or pure metadata records).
         return await ctx.lobb.collectionStore.createOne({
           collectionName: "storage_fs",
           data: {
@@ -105,99 +81,28 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
         });
       },
     },
-    {
-      name: "storageDeleteOneService",
-      eventName: "core.service.deleteOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName !== "storage_fs") return;
-
-        const entry = (await ctx.lobb.collectionStore.findOne({
-          collectionName: "storage_fs",
-          id: input.id,
-        })).data;
-        if (!entry) {
-          return ctx.lobb.collectionStore.deleteOne({
-            collectionName: "storage_fs",
-            id: input.id,
-          });
-        }
-
-        const adapter = initStorageAdapter(extensionConfig);
-
-        if (entry.type === "file") {
-          await adapter.removeFile(entry.id);
-        } else if (entry.type === "directory") {
-          // Cascade leaf-first so each deleted row has no remaining children
-          // referencing it — Lobb's core FK-children check would otherwise
-          // block the delete. findDescendants returns BFS (shallow → deep);
-          // reverse to walk deep → shallow.
-          const descendants = await findDescendants(entry.id, ctx.lobb);
-          for (const d of [...descendants].reverse()) {
-            if (d.type === "file") {
-              await adapter.removeFile(d.id);
-            }
-            await ctx.lobb.collectionStore.deleteOne({
-              collectionName: "storage_fs",
-              id: String(d.id),
-            });
-          }
-        }
 
-        return ctx.lobb.collectionStore.deleteOne({
+    // Read a record, optionally with the file bytes (for view/download).
+    storage_read: {
+      handler: async (input: any, ctx) => {
+        const result = await ctx.lobb.collectionStore.findOne({
           collectionName: "storage_fs",
           id: input.id,
         });
-      },
-    },
-    {
-      name: "storageDeleteManyService",
-      eventName: "core.service.deleteMany.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName !== "storage_fs") return;
-
-        const matched = (await ctx.lobb.collectionStore.findAll({
-          collectionName: "storage_fs",
-          params: { filter: input.filter },
-        })).data;
 
-        const adapter = initStorageAdapter(extensionConfig);
-        // Collect every row to remove, BFS shallow→deep. We delete in
-        // reverse order so leaves are dropped before their parents — that
-        // keeps Lobb's children FK check happy.
-        const ordered: any[] = [];
-        const seen = new Set<number>();
-        for (const entry of matched) {
-          if (seen.has(entry.id)) continue;
-          seen.add(entry.id);
-          ordered.push(entry);
-          if (entry.type === "directory") {
-            for (const d of await findDescendants(entry.id, ctx.lobb)) {
-              if (seen.has(d.id)) continue;
-              seen.add(d.id);
-              ordered.push(d);
-            }
-          }
-        }
-
-        for (const row of [...ordered].reverse()) {
-          if (row.type === "file") await adapter.removeFile(row.id);
-          await ctx.lobb.collectionStore.deleteOne({
-            collectionName: "storage_fs",
-            id: String(row.id),
-          });
+        if (result.data?.type === "file" && input.readFile) {
+          const adapter = initStorageAdapter(extensionConfig);
+          const file = await adapter.readFile(result.data.id);
+          return { ...result, file: new Uint8Array(file) };
         }
 
-        // affectedCount reflects what the caller asked to delete (the
-        // matched rows). Cascaded descendants are an implementation detail.
-        return { affectedCount: matched.length };
+        return result;
       },
     },
-    {
-      name: "storageUpdateOneService",
-      eventName: "core.service.updateOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName !== "storage_fs") return;
 
+    // Rename / move / update metadata, cascading `path` to all descendants.
+    storage_update: {
+      handler: async (input: any, ctx) => {
         const current = (await ctx.lobb.collectionStore.findOne({
           collectionName: "storage_fs",
           id: input.id,
@@ -212,17 +117,13 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
 
         const data: any = { ...input.data };
 
-        // Translate any path-based reparenting into a parent_id update so
-        // the rest of this handler has a single canonical signal to work
-        // with. Explicit parent_id wins if both are present.
+        // Translate any path-based reparenting into a parent_id update.
         if (data.path != null && data.parent_id == null) {
           data.parent_id = await resolveParentIdFromPath(data.path, ctx.lobb);
         }
         // The path field is derived — never let a caller overwrite it raw.
         delete data.path;
 
-        // If the row's location in the tree is changing (name or parent),
-        // recompute its own path and cascade the change to every descendant.
         const nameChanged = data.name != null && data.name !== current.name;
         const parentChanged =
           Object.prototype.hasOwnProperty.call(data, "parent_id") &&
@@ -232,8 +133,7 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
           const newParentId = parentChanged ? (data.parent_id ?? null) : current.parent_id;
           const newName = nameChanged ? data.name : current.name;
 
-          // Cycle prevention: the new parent must not be the row itself,
-          // and walking up from it must never hit the row's own id.
+          // Cycle prevention.
           if (parentChanged && newParentId != null) {
             if (Number(newParentId) === Number(input.id)) {
               throw new LobbError({
@@ -266,15 +166,11 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
             data: { ...data, path: newSelfPath },
           });
 
-          // Cascade: for a directory, every descendant's `path` is anchored
-          // to this row's name + its own path. Recompute by walking the
-          // tree once in memory — no DB round trip per descendant.
+          // Cascade descendant paths.
           if (current.type === "directory") {
             const descendants = await findDescendants(Number(input.id), ctx.lobb);
             const byId = new Map<number, any>();
             for (const d of descendants) byId.set(d.id, d);
-            // The directory's OWN full path (= its parent's path + its name)
-            // is what every direct child stores in `path`.
             const newDirFullPath = newSelfPath === "/" ? "/" + newName : newSelfPath + "/" + newName;
             const pathOf = (id: number | null): string => {
               if (id == null) return "/";
@@ -303,5 +199,79 @@ export function getServicesWorkflows(extensionConfig: ExtensionConfig): Workflow
         });
       },
     },
-  ];
+
+    // Delete a file or directory, cascading leaf-first and removing files from disk.
+    storage_delete: {
+      handler: async (input: any, ctx) => {
+        const entry = (await ctx.lobb.collectionStore.findOne({
+          collectionName: "storage_fs",
+          id: input.id,
+        })).data;
+        if (!entry) {
+          return ctx.lobb.collectionStore.deleteOne({
+            collectionName: "storage_fs",
+            id: input.id,
+          });
+        }
+
+        const adapter = initStorageAdapter(extensionConfig);
+
+        if (entry.type === "file") {
+          await adapter.removeFile(entry.id);
+        } else if (entry.type === "directory") {
+          const descendants = await findDescendants(entry.id, ctx.lobb);
+          for (const d of [...descendants].reverse()) {
+            if (d.type === "file") {
+              await adapter.removeFile(d.id);
+            }
+            await ctx.lobb.collectionStore.deleteOne({
+              collectionName: "storage_fs",
+              id: String(d.id),
+            });
+          }
+        }
+
+        return ctx.lobb.collectionStore.deleteOne({
+          collectionName: "storage_fs",
+          id: input.id,
+        });
+      },
+    },
+
+    // Delete every row matching `filter`, cascading descendants, removing files.
+    storage_delete_many: {
+      handler: async (input: any, ctx) => {
+        const matched = (await ctx.lobb.collectionStore.findAll({
+          collectionName: "storage_fs",
+          params: { filter: input.filter },
+        })).data;
+
+        const adapter = initStorageAdapter(extensionConfig);
+        const ordered: any[] = [];
+        const seen = new Set<number>();
+        for (const entry of matched) {
+          if (seen.has(entry.id)) continue;
+          seen.add(entry.id);
+          ordered.push(entry);
+          if (entry.type === "directory") {
+            for (const d of await findDescendants(entry.id, ctx.lobb)) {
+              if (seen.has(d.id)) continue;
+              seen.add(d.id);
+              ordered.push(d);
+            }
+          }
+        }
+
+        for (const row of [...ordered].reverse()) {
+          if (row.type === "file") await adapter.removeFile(row.id);
+          await ctx.lobb.collectionStore.deleteOne({
+            collectionName: "storage_fs",
+            id: String(row.id),
+          });
+        }
+
+        return { affectedCount: matched.length };
+      },
+    },
+  };
 }

package/extensions/storage/workflows/index.ts

@@ -1,6 +1,5 @@
 import type { ExtensionConfig } from "../config/extensionConfigSchema.ts";
-import { getControllersWorkflows } from "./controllers.ts";
-import { getServicesWorkflows } from "./services.ts";
+import { getActionControllerWorkflows } from "./actionController.ts";
 import { init } from "../init.ts";
 
 export function getWorkflows(extensionConfig: ExtensionConfig) {
@@ -12,7 +11,6 @@ export function getWorkflows(extensionConfig: ExtensionConfig) {
         await init(extensionConfig);
       },
     },
-    ...getServicesWorkflows(extensionConfig),
-    ...getControllersWorkflows(),
+    ...getActionControllerWorkflows(),
   ];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lobb-js/lobb-ext-storage",
-  "version": "0.15.1",
+  "version": "0.16.0",
   "license": "UNLICENSED",
   "type": "module",
   "publishConfig": {
@@ -32,14 +32,14 @@
     "package": "svelte-package --input extensions/storage/studio"
   },
   "dependencies": {
-    "@lobb-js/core": "^0.40.0",
+    "@lobb-js/core": "^0.41.0",
     "browser-fs-access": "^0.35.0",
     "hono": "^4.7.0",
     "openapi-types": "^12.1.3",
     "path-browserify": "^1.0.1"
   },
   "devDependencies": {
-    "@lobb-js/studio": "^0.46.0",
+    "@lobb-js/studio": "^0.49.0",
     "@lucide/svelte": "^0.563.1",
     "@sveltejs/adapter-node": "^5.5.4",
     "@sveltejs/kit": "^2.60.1",

package/dist/lib/components/foreignKeyComponent.svelte

@@ -1,44 +0,0 @@
-<script lang="ts">
-    import type { ExtensionProps } from "@lobb-js/studio";
-    import FileExplorer, { type Entry } from "./fileExplorer.svelte";
-    import FileIcon from "./fileIcon.svelte";
-
-    interface Props extends ExtensionProps {
-        value: Entry;
-    }
-
-	let { utils, value = $bindable(), ...props }: Props = $props();
-
-	const { FilePlus2 } = utils.components.Icons;
-	const { Drawer } = utils.components;
-
-    let openDrawer = $state(false);
-
-    function handleClick() {
-        openDrawer = true;
-    }
-
-    async function onHideHandle() {
-        openDrawer = false;
-    }
-
-    async function handleOnFileSelect(entry: Entry) {
-        value = entry;
-        openDrawer = false;
-    }
-</script>
-
-<button onclick={handleClick} class="flex flex-col items-center gap-2 w-full border p-4 rounded-md bg-muted">
-    {#if value}
-        <FileIcon entry={value} {utils} />
-    {:else}
-        <FilePlus2 class="text-muted-foreground" />
-        <div class="text-sm text-muted-foreground">Click to add an asset</div>
-    {/if}
-</button>
-
-{#if openDrawer}
-    <Drawer onHide={onHideHandle}>
-        <FileExplorer onFileSelect={handleOnFileSelect} utils={utils} {...props} />
-    </Drawer>
-{/if}

package/dist/lib/components/foreignKeyComponent.svelte.d.ts

@@ -1,14 +0,0 @@
-import { SvelteComponentTyped } from "svelte";
-declare const __propDef: {
-    props: Record<string, never>;
-    events: {
-        [evt: string]: CustomEvent<any>;
-    };
-    slots: {};
-};
-export type ForeignKeyComponentProps = typeof __propDef.props;
-export type ForeignKeyComponentEvents = typeof __propDef.events;
-export type ForeignKeyComponentSlots = typeof __propDef.slots;
-export default class ForeignKeyComponent extends SvelteComponentTyped<ForeignKeyComponentProps, ForeignKeyComponentEvents, ForeignKeyComponentSlots> {
-}
-export {};

package/extensions/storage/studio/lib/components/foreignKeyComponent.svelte

@@ -1,44 +0,0 @@
-<script lang="ts">
-    import type { ExtensionProps } from "@lobb-js/studio";
-    import FileExplorer, { type Entry } from "./fileExplorer.svelte";
-    import FileIcon from "./fileIcon.svelte";
-
-    interface Props extends ExtensionProps {
-        value: Entry;
-    }
-
-	let { utils, value = $bindable(), ...props }: Props = $props();
-
-	const { FilePlus2 } = utils.components.Icons;
-	const { Drawer } = utils.components;
-
-    let openDrawer = $state(false);
-
-    function handleClick() {
-        openDrawer = true;
-    }
-
-    async function onHideHandle() {
-        openDrawer = false;
-    }
-
-    async function handleOnFileSelect(entry: Entry) {
-        value = entry;
-        openDrawer = false;
-    }
-</script>
-
-<button onclick={handleClick} class="flex flex-col items-center gap-2 w-full border p-4 rounded-md bg-muted">
-    {#if value}
-        <FileIcon entry={value} {utils} />
-    {:else}
-        <FilePlus2 class="text-muted-foreground" />
-        <div class="text-sm text-muted-foreground">Click to add an asset</div>
-    {/if}
-</button>
-
-{#if openDrawer}
-    <Drawer onHide={onHideHandle}>
-        <FileExplorer onFileSelect={handleOnFileSelect} utils={utils} {...props} />
-    </Drawer>
-{/if}

package/extensions/storage/types.ts

@@ -1,11 +0,0 @@
-import type { ExposedServiceOutput, ServiceOp } from "@lobb-js/core";
-
-export type StorageFsCreateOne = ServiceOp<
-  { file: File; data?: { path?: string; parent_id?: number | null }; force?: boolean },
-  ExposedServiceOutput
->;
-
-export type StorageFsFindOne = ServiceOp<
-  { id: string; readFile: boolean },
-  { data: { file_mime_type: string; name: string } | null; file: Uint8Array }
->;

package/extensions/storage/workflows/controllers.ts

@@ -1,103 +0,0 @@
-import type { Lobb, Workflow } from "@lobb-js/core";
-import type { Context } from "hono";
-import type { StorageFsCreateOne, StorageFsFindOne } from "../types.ts";
-
-export function getControllersWorkflows(): Workflow[] {
-  return [
-    {
-      name: "storagePreCreateController",
-      eventName: "core.controllers.createOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName === "storage_fs") {
-          const context = input.context as Context;
-          const lobb = context.get("lobb") as Lobb;
-          const contentType = context.req.header("Content-Type");
-
-          if (contentType?.startsWith("multipart/form-data")) {
-            const body = await context.req.parseBody({ all: true });
-            const fileEntry = body["file"];
-
-            if (!fileEntry) {
-              throw new ctx.LobbError({
-                code: "BAD_REQUEST",
-                message: "No files were provided.",
-              });
-            }
-
-            if (Array.isArray(fileEntry)) {
-              throw new ctx.LobbError({
-                code: "BAD_REQUEST",
-                message: "Only one file can be uploaded at a time.",
-              });
-            }
-
-            if (!(fileEntry instanceof File)) {
-              throw new ctx.LobbError({
-                code: "BAD_REQUEST",
-                message: "The file shouldnt be a string",
-              });
-            }
-
-            const force = context.req.query("force") === "true";
-
-            const extraData: Record<string, string> = {};
-            for (const [key, val] of Object.entries(body)) {
-              if (key !== "file" && key !== "payload" && typeof val === "string") {
-                extraData[key] = val;
-              }
-            }
-
-            const result = await ctx.lobb.collectionService.createOne<StorageFsCreateOne>({
-              collectionName: "storage_fs",
-              file: fileEntry,
-              force,
-              data: extraData,
-            });
-
-            return context.json(result, 201);
-          }
-        }
-      },
-    },
-    {
-      name: "storageFindOneController",
-      eventName: "core.controllers.findOne.override",
-      handler: async (input, ctx) => {
-        if (input.collectionName === "storage_fs") {
-          const context = input.context as Context;
-          const action = context.req.query("action");
-
-          if (action === "view" || action === "download") {
-            const result = await ctx.lobb.collectionService.findOne<StorageFsFindOne>({
-              collectionName: "storage_fs",
-              id: input.id,
-              readFile: true,
-            });
-
-            if (!result.data) {
-              throw new ctx.LobbError({
-                code: "NOT_FOUND",
-                message: "The record is not found",
-              });
-            }
-
-            const headers: Record<string, string> = {
-              "Content-Type": result.data.file_mime_type,
-            };
-            if (action === "download") {
-              headers["Content-Disposition"] =
-                `attachment; filename="${result.data.name}"`;
-            }
-
-            return new Response(result.file.buffer as ArrayBuffer, { status: 200, headers });
-          } else if (action) {
-            throw new ctx.LobbError({
-              code: "BAD_REQUEST",
-              message: `The passed (${action}) action query param is not supported`,
-            });
-          }
-        }
-      },
-    },
-  ];
-}