@dyrected/core 2.5.16 → 2.5.17

package/dist/index.cjs

@@ -1170,19 +1170,27 @@ function parseMongoWhere(where) {
 }
 
 // src/utils/hooks.ts
-async function runCollectionHooks(hooks, args) {
+async function runCollectionHooks(hooks, args, options = {}) {
   if (!hooks || hooks.length === 0) {
     return args.data ?? args.doc ?? void 0;
   }
   let currentPayload = args.data ?? args.doc ?? void 0;
   for (const hook of hooks) {
-    const result = await hook({
-      ...args,
-      data: args.data !== void 0 ? currentPayload : void 0,
-      doc: args.doc !== void 0 ? currentPayload : void 0
-    });
-    if (result !== void 0) {
-      currentPayload = result;
+    try {
+      const result = await hook({
+        ...args,
+        data: args.data !== void 0 ? currentPayload : void 0,
+        doc: args.doc !== void 0 ? currentPayload : void 0
+      });
+      if (result !== void 0) {
+        currentPayload = result;
+      }
+    } catch (err) {
+      if (options.isolated) {
+        console.error("[dyrected/core] Side-effect hook failed (error isolated \u2014 DB write was successful):", err);
+      } else {
+        throw err;
+      }
     }
   }
   return currentPayload;
@@ -1683,7 +1691,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "create"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -1741,7 +1749,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "create"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -1799,7 +1807,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "update"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -1914,7 +1922,7 @@ var CollectionController = class {
       doc,
       user,
       req: c.req
-    });
+    }, { isolated: true });
     return c.json({ message: "Deleted" });
   }
   async deleteMany(c) {
@@ -1971,7 +1979,7 @@ var CollectionController = class {
           doc,
           user,
           req: c.req
-        });
+        }, { isolated: true });
       } catch (err) {
         failed.push({ id, error: err?.message ?? "Unknown error" });
       }
@@ -2073,7 +2081,7 @@ var GlobalController = class {
       user,
       req: c.req,
       operation: "update"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.global.hooks?.afterRead, {
       doc: updated,
       req: c.req,

package/dist/index.d.cts

@@ -1,5 +1,5 @@
-import { D as DyrectedConfig, F as Field, C as CollectionConfig, P as Prettify, I as InferDocShape, S as SystemDocFields, A as AuthDocFields, U as UploadDocFields, G as GlobalConfig } from './app-DO1s9YW1.cjs';
-export { a as AccessFunction, b as AdminConfig, c as AuthenticatedUser, B as BaseDocument, d as Block, e as CollectionAfterChangeHook, f as CollectionAfterDeleteHook, g as CollectionAfterReadHook, h as CollectionBeforeChangeHook, i as CollectionBeforeDeleteHook, j as CollectionBeforeReadHook, k as DatabaseAdapter, l as DynamicOptionItem, m as DynamicOptionsConfig, n as DynamicOptionsResolver, o as DynamicOptionsResolverArgs, p as DyrectedContext, q as FieldAfterReadHook, r as FieldBeforeChangeHook, s as FieldHook, t as FieldType, u as FileData, v as GlobalAfterChangeHook, w as GlobalAfterReadHook, x as GlobalBeforeChangeHook, y as GlobalBeforeReadHook, H as HookFunction, z as HookRequestContext, E as ImageService, J as PaginatedResult, K as StorageAdapter, L as UploadConfig, M as createDyrectedApp } from './app-DO1s9YW1.cjs';
+import { D as DyrectedConfig, F as Field, C as CollectionConfig, P as Prettify, I as InferDocShape, S as SystemDocFields, A as AuthDocFields, U as UploadDocFields, G as GlobalConfig } from './app-BElen1tP.cjs';
+export { a as AccessFunction, b as AdminConfig, c as AuthenticatedUser, B as BaseDocument, d as Block, e as CollectionAfterChangeHook, f as CollectionAfterDeleteHook, g as CollectionAfterReadHook, h as CollectionBeforeChangeHook, i as CollectionBeforeDeleteHook, j as CollectionBeforeReadHook, k as DatabaseAdapter, l as DynamicOptionItem, m as DynamicOptionsConfig, n as DynamicOptionsResolver, o as DynamicOptionsResolverArgs, p as DyrectedContext, q as FieldAfterReadHook, r as FieldBeforeChangeHook, s as FieldHook, t as FieldType, u as FileData, v as GlobalAfterChangeHook, w as GlobalAfterReadHook, x as GlobalBeforeChangeHook, y as GlobalBeforeReadHook, H as HookFunction, z as HookRequestContext, E as ImageService, J as PaginatedResult, K as StorageAdapter, L as UploadConfig, M as createDyrectedApp } from './app-BElen1tP.cjs';
 import 'hono/types';
 import 'hono';
 
@@ -91,6 +91,11 @@ type AnyHookFn = (args: any) => any;
  * Each hook receives the output of the previous hook merged back into the
  * original `args`. When a hook returns a non-undefined value it becomes the
  * `data` / `doc` for the next hook in the chain.
+ *
+ * Pass `{ isolated: true }` for hooks that run **after** a DB write has already
+ * been committed (`afterChange`, `afterDelete`). In isolated mode each hook is
+ * wrapped in a try/catch so a failing side-effect (email, webhook, etc.) never
+ * surfaces as an HTTP 500 to the caller — the write already succeeded.
  */
 declare function runCollectionHooks(hooks: AnyHookFn[] | undefined, args: {
     data?: unknown;
@@ -99,6 +104,8 @@ declare function runCollectionHooks(hooks: AnyHookFn[] | undefined, args: {
     req?: unknown;
     operation?: "create" | "update" | "delete";
     [key: string]: unknown;
+}, options?: {
+    isolated?: boolean;
 }): Promise<any>;
 /**
  * Execute field-level `beforeChange` hooks recursively on a data payload.

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { D as DyrectedConfig, F as Field, C as CollectionConfig, P as Prettify, I as InferDocShape, S as SystemDocFields, A as AuthDocFields, U as UploadDocFields, G as GlobalConfig } from './app-DO1s9YW1.js';
-export { a as AccessFunction, b as AdminConfig, c as AuthenticatedUser, B as BaseDocument, d as Block, e as CollectionAfterChangeHook, f as CollectionAfterDeleteHook, g as CollectionAfterReadHook, h as CollectionBeforeChangeHook, i as CollectionBeforeDeleteHook, j as CollectionBeforeReadHook, k as DatabaseAdapter, l as DynamicOptionItem, m as DynamicOptionsConfig, n as DynamicOptionsResolver, o as DynamicOptionsResolverArgs, p as DyrectedContext, q as FieldAfterReadHook, r as FieldBeforeChangeHook, s as FieldHook, t as FieldType, u as FileData, v as GlobalAfterChangeHook, w as GlobalAfterReadHook, x as GlobalBeforeChangeHook, y as GlobalBeforeReadHook, H as HookFunction, z as HookRequestContext, E as ImageService, J as PaginatedResult, K as StorageAdapter, L as UploadConfig, M as createDyrectedApp } from './app-DO1s9YW1.js';
+import { D as DyrectedConfig, F as Field, C as CollectionConfig, P as Prettify, I as InferDocShape, S as SystemDocFields, A as AuthDocFields, U as UploadDocFields, G as GlobalConfig } from './app-BElen1tP.js';
+export { a as AccessFunction, b as AdminConfig, c as AuthenticatedUser, B as BaseDocument, d as Block, e as CollectionAfterChangeHook, f as CollectionAfterDeleteHook, g as CollectionAfterReadHook, h as CollectionBeforeChangeHook, i as CollectionBeforeDeleteHook, j as CollectionBeforeReadHook, k as DatabaseAdapter, l as DynamicOptionItem, m as DynamicOptionsConfig, n as DynamicOptionsResolver, o as DynamicOptionsResolverArgs, p as DyrectedContext, q as FieldAfterReadHook, r as FieldBeforeChangeHook, s as FieldHook, t as FieldType, u as FileData, v as GlobalAfterChangeHook, w as GlobalAfterReadHook, x as GlobalBeforeChangeHook, y as GlobalBeforeReadHook, H as HookFunction, z as HookRequestContext, E as ImageService, J as PaginatedResult, K as StorageAdapter, L as UploadConfig, M as createDyrectedApp } from './app-BElen1tP.js';
 import 'hono/types';
 import 'hono';
 
@@ -91,6 +91,11 @@ type AnyHookFn = (args: any) => any;
  * Each hook receives the output of the previous hook merged back into the
  * original `args`. When a hook returns a non-undefined value it becomes the
  * `data` / `doc` for the next hook in the chain.
+ *
+ * Pass `{ isolated: true }` for hooks that run **after** a DB write has already
+ * been committed (`afterChange`, `afterDelete`). In isolated mode each hook is
+ * wrapped in a try/catch so a failing side-effect (email, webhook, etc.) never
+ * surfaces as an HTTP 500 to the caller — the write already succeeded.
  */
 declare function runCollectionHooks(hooks: AnyHookFn[] | undefined, args: {
     data?: unknown;
@@ -99,6 +104,8 @@ declare function runCollectionHooks(hooks: AnyHookFn[] | undefined, args: {
     req?: unknown;
     operation?: "create" | "update" | "delete";
     [key: string]: unknown;
+}, options?: {
+    isolated?: boolean;
 }): Promise<any>;
 /**
  * Execute field-level `beforeChange` hooks recursively on a data payload.

package/dist/index.js

@@ -4,7 +4,7 @@ import {
   executeFieldBeforeChange,
   normalizeConfig,
   runCollectionHooks
-} from "./chunk-2JMA3M5S.js";
+} from "./chunk-YNJ3YC4N.js";
 
 // src/utils/setup-prompt.ts
 function buildEnvironmentSection(frameworkLabel, isSelfHosted, config) {

package/dist/server.cjs

@@ -269,19 +269,27 @@ async function verifyPassword(plain, stored) {
 }
 
 // src/utils/hooks.ts
-async function runCollectionHooks(hooks, args) {
+async function runCollectionHooks(hooks, args, options = {}) {
   if (!hooks || hooks.length === 0) {
     return args.data ?? args.doc ?? void 0;
   }
   let currentPayload = args.data ?? args.doc ?? void 0;
   for (const hook of hooks) {
-    const result = await hook({
-      ...args,
-      data: args.data !== void 0 ? currentPayload : void 0,
-      doc: args.doc !== void 0 ? currentPayload : void 0
-    });
-    if (result !== void 0) {
-      currentPayload = result;
+    try {
+      const result = await hook({
+        ...args,
+        data: args.data !== void 0 ? currentPayload : void 0,
+        doc: args.doc !== void 0 ? currentPayload : void 0
+      });
+      if (result !== void 0) {
+        currentPayload = result;
+      }
+    } catch (err) {
+      if (options.isolated) {
+        console.error("[dyrected/core] Side-effect hook failed (error isolated \u2014 DB write was successful):", err);
+      } else {
+        throw err;
+      }
     }
   }
   return currentPayload;
@@ -554,7 +562,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "create"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -612,7 +620,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "create"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -670,7 +678,7 @@ var CollectionController = class {
       user,
       req: c.req,
       operation: "update"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.collection.hooks?.afterRead, {
       doc,
       req: c.req,
@@ -785,7 +793,7 @@ var CollectionController = class {
       doc,
       user,
       req: c.req
-    });
+    }, { isolated: true });
     return c.json({ message: "Deleted" });
   }
   async deleteMany(c) {
@@ -842,7 +850,7 @@ var CollectionController = class {
           doc,
           user,
           req: c.req
-        });
+        }, { isolated: true });
       } catch (err) {
         failed.push({ id, error: err?.message ?? "Unknown error" });
       }
@@ -944,7 +952,7 @@ var GlobalController = class {
       user,
       req: c.req,
       operation: "update"
-    });
+    }, { isolated: true });
     const readDoc = await runCollectionHooks(this.global.hooks?.afterRead, {
       doc: updated,
       req: c.req,

package/dist/server.d.cts

@@ -1,5 +1,5 @@
-import { p as DyrectedContext, D as DyrectedConfig, C as CollectionConfig, G as GlobalConfig } from './app-DO1s9YW1.cjs';
-export { M as createDyrectedApp } from './app-DO1s9YW1.cjs';
+import { p as DyrectedContext, D as DyrectedConfig, C as CollectionConfig, G as GlobalConfig } from './app-BElen1tP.cjs';
+export { M as createDyrectedApp } from './app-BElen1tP.cjs';
 import * as hono from 'hono';
 import { Hono, Context } from 'hono';
 import * as hono_utils_types from 'hono/utils/types';

package/dist/server.d.ts

@@ -1,5 +1,5 @@
-import { p as DyrectedContext, D as DyrectedConfig, C as CollectionConfig, G as GlobalConfig } from './app-DO1s9YW1.js';
-export { M as createDyrectedApp } from './app-DO1s9YW1.js';
+import { p as DyrectedContext, D as DyrectedConfig, C as CollectionConfig, G as GlobalConfig } from './app-BElen1tP.js';
+export { M as createDyrectedApp } from './app-BElen1tP.js';
 import * as hono from 'hono';
 import { Hono, Context } from 'hono';
 import * as hono_utils_types from 'hono/utils/types';

package/dist/server.js

@@ -6,7 +6,7 @@ import {
   PreviewController,
   createDyrectedApp,
   registerRoutes
-} from "./chunk-2JMA3M5S.js";
+} from "./chunk-YNJ3YC4N.js";
 export {
   AuthController,
   CollectionController,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dyrected/core",
-  "version": "2.5.16",
+  "version": "2.5.17",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",