feat(hooks): Rename all hooks to make each one use an unique attribute (for AI performances), and introduce beforeFetch hooks.

BREAKING CHANGES: Rename all hooks to make each one use an unique attribute (for AI performances), and introduce beforeFetch hooks.

Author: vincentdesmares

lib/injectHooks.js

@@ -2,112 +2,154 @@
 exports.__esModule = true;
 exports.injectHooks = void 0;
 function injectListHooks(declaration, injectFunctions) {
-    var _a;
-    if (!declaration.list) {
-        declaration.list = {};
-    }
     // If the list is resolved by a custom function, we don't need to inject hooks
-    if ((_a = declaration === null || declaration === void 0 ? void 0 : declaration.list) === null || _a === void 0 ? void 0 : _a.resolve) {
+    if ('type' in declaration ||
+        ((declaration === null || declaration === void 0 ? void 0 : declaration.list) && 'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.list))) {
         return;
     }
-    var beforeList = Array.isArray(declaration.list.before)
-        ? declaration.list.before
-        : declaration.list.before
-            ? [declaration.list.before]
+    if (!declaration.list) {
+        declaration.list = {};
+    }
+    var beforeList = Array.isArray(declaration.list.beforeList)
+        ? declaration.list.beforeList
+        : declaration.list.beforeList
+            ? [declaration.list.beforeList]
             : [];
-    declaration.list.before = injectFunctions.listBefore
+    declaration.list.beforeList = injectFunctions.listBefore
         ? injectFunctions.listBefore(declaration.model, beforeList)
         : beforeList;
-    var afterList = Array.isArray(declaration.list.after)
-        ? declaration.list.after
-        : declaration.list.after
-            ? [declaration.list.after]
+    var afterList = Array.isArray(declaration.list.afterList)
+        ? declaration.list.afterList
+        : declaration.list.afterList
+            ? [declaration.list.afterList]
             : [];
-    declaration.list.after = injectFunctions.listAfter
+    declaration.list.afterList = injectFunctions.listAfter
         ? injectFunctions.listAfter(declaration.model, afterList)
         : afterList;
 }
 function injectUpdateHooks(declaration, injectFunctions) {
-    var _a, _b;
-    if (!((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('update')))
+    var _a;
+    if ('actions' in declaration && !((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('update')))
         return;
     // If the update is resolved by a custom function, we don't need to inject hooks
-    if ((_b = declaration === null || declaration === void 0 ? void 0 : declaration.update) === null || _b === void 0 ? void 0 : _b.resolve) {
+    if ('update' in declaration &&
+        (declaration === null || declaration === void 0 ? void 0 : declaration.update) &&
+        'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.update)) {
+        return;
+    }
+    if ('type' in declaration ||
+        ((declaration === null || declaration === void 0 ? void 0 : declaration.update) && 'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.update))) {
+        return;
+    }
+    if ('update' in declaration &&
+        (declaration === null || declaration === void 0 ? void 0 : declaration.update) &&
+        'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.update)) {
         return;
     }
     if (!declaration.update) {
         declaration.update = {};
     }
-    var beforeUpdate = Array.isArray(declaration.update.before)
-        ? declaration.update.before
-        : declaration.update.before
-            ? [declaration.update.before]
+    declaration.update = declaration.update;
+    var beforeUpdateFetch = Array.isArray(declaration.update.beforeUpdateFetch)
+        ? declaration.update.beforeUpdateFetch
+        : declaration.update.beforeUpdateFetch
+            ? [declaration.update.beforeUpdateFetch]
+            : [];
+    declaration.update.beforeUpdateFetch = injectFunctions.updateBeforeFetch
+        ? injectFunctions.updateBeforeFetch(declaration.model, beforeUpdateFetch)
+        : beforeUpdateFetch;
+    var beforeUpdate = Array.isArray(declaration.update.beforeUpdate)
+        ? declaration.update.beforeUpdate
+        : declaration.update.beforeUpdate
+            ? [declaration.update.beforeUpdate]
             : [];
-    declaration.update.before = injectFunctions.updateBefore
+    declaration.update.beforeUpdate = injectFunctions.updateBefore
         ? injectFunctions.updateBefore(declaration.model, beforeUpdate)
         : beforeUpdate;
-    var afterUpdate = Array.isArray(declaration.update.after)
-        ? declaration.update.after
-        : declaration.update.after
-            ? [declaration.update.after]
+    var afterUpdate = Array.isArray(declaration.update.afterUpdate)
+        ? declaration.update.afterUpdate
+        : declaration.update.afterUpdate
+            ? [declaration.update.afterUpdate]
             : [];
-    declaration.update.after = injectFunctions.updateAfter
+    declaration.update.afterUpdate = injectFunctions.updateAfter
         ? injectFunctions.updateAfter(declaration.model, afterUpdate)
         : afterUpdate;
 }
 function injectCreateHooks(declaration, injectFunctions) {
-    var _a, _b;
-    if (!((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('create')))
+    var _a;
+    if ('actions' in declaration && !((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('create')))
         return;
     // If the create is resolved by a custom function, we don't need to inject hooks
-    if ((_b = declaration === null || declaration === void 0 ? void 0 : declaration.create) === null || _b === void 0 ? void 0 : _b.resolve) {
+    if ('create' in declaration &&
+        (declaration === null || declaration === void 0 ? void 0 : declaration.create) &&
+        'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.create)) {
+        return;
+    }
+    if ('type' in declaration ||
+        ((declaration === null || declaration === void 0 ? void 0 : declaration.create) && 'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration.create))) {
         return;
     }
     if (!declaration.create) {
         declaration.create = {};
     }
-    var beforeCreate = Array.isArray(declaration.create.before)
-        ? declaration.create.before
-        : declaration.create.before
-            ? [declaration.create.before]
+    declaration.create = declaration.create;
+    var beforeCreate = Array.isArray(declaration.create.beforeCreate)
+        ? declaration.create.beforeCreate
+        : declaration.create.beforeCreate
+            ? [declaration.create.beforeCreate]
             : [];
-    declaration.create.before = injectFunctions.createBefore
+    declaration.create.beforeCreate = injectFunctions.createBefore
         ? injectFunctions.createBefore(declaration.model, beforeCreate)
         : beforeCreate;
-    var afterCreate = Array.isArray(declaration.create.after)
-        ? declaration.create.after
-        : declaration.create.after
-            ? [declaration.create.after]
+    var afterCreate = Array.isArray(declaration.create.afterCreate)
+        ? declaration.create.afterCreate
+        : declaration.create.afterCreate
+            ? [declaration.create.afterCreate]
             : [];
-    declaration.create.after = injectFunctions.createAfter
+    declaration.create.afterCreate = injectFunctions.createAfter
         ? injectFunctions.createAfter(declaration.model, afterCreate)
         : afterCreate;
 }
 function injectDeleteHooks(declaration, injectFunctions) {
-    var _a, _b;
-    if (!((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('delete')))
+    var _a;
+    if ('actions' in declaration && !((_a = declaration.actions) === null || _a === void 0 ? void 0 : _a.includes('delete')))
         return;
     // If the delete is resolved by a custom function, we don't need to inject hooks
-    if ((_b = declaration === null || declaration === void 0 ? void 0 : declaration["delete"]) === null || _b === void 0 ? void 0 : _b.resolve) {
+    if ('delete' in declaration &&
+        (declaration === null || declaration === void 0 ? void 0 : declaration["delete"]) &&
+        'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration["delete"])) {
+        return;
+    }
+    if ('type' in declaration ||
+        ((declaration === null || declaration === void 0 ? void 0 : declaration["delete"]) && 'resolve' in (declaration === null || declaration === void 0 ? void 0 : declaration["delete"]))) {
         return;
     }
     if (!declaration["delete"]) {
         declaration["delete"] = {};
     }
-    var beforeDelete = Array.isArray(declaration["delete"].before)
-        ? declaration["delete"].before
-        : declaration["delete"].before
-            ? [declaration["delete"].before]
+    declaration["delete"] = declaration["delete"];
+    var beforeDelete = Array.isArray(declaration["delete"].beforeDelete)
+        ? declaration["delete"].beforeDelete
+        : declaration["delete"].beforeDelete
+            ? [declaration["delete"].beforeDelete]
             : [];
-    declaration["delete"].before = injectFunctions.deleteBefore
+    declaration["delete"].beforeDelete = injectFunctions.deleteBefore
         ? injectFunctions.deleteBefore(declaration.model, beforeDelete)
         : beforeDelete;
-    var afterDelete = Array.isArray(declaration["delete"].after)
-        ? declaration["delete"].after
-        : declaration["delete"].after
-            ? [declaration["delete"].after]
+    var beforeDeleteFetch = Array.isArray(declaration["delete"].beforeDeleteFetch)
+        ? declaration["delete"].beforeDeleteFetch
+        : declaration["delete"].beforeDeleteFetch
+            ? [declaration["delete"].beforeDeleteFetch]
+            : [];
+    declaration["delete"].beforeDeleteFetch = injectFunctions.deleteBeforeFetch
+        ? injectFunctions.deleteBeforeFetch(declaration.model, beforeDeleteFetch)
+        : beforeDeleteFetch;
+    var afterDelete = Array.isArray(declaration["delete"].afterDelete)
+        ? declaration["delete"].afterDelete
+        : declaration["delete"].afterDelete
+            ? [declaration["delete"].afterDelete]
             : [];
-    declaration["delete"].after = injectFunctions.deleteAfter
+    declaration["delete"].afterDelete = injectFunctions.deleteAfter
         ? injectFunctions.deleteAfter(declaration.model, afterDelete)
         : afterDelete;
 }

lib/mutationResolvers/create.js

@@ -129,11 +129,11 @@ function generateMutationCreate(modelName, inputType, outputType, model, graphql
                         return [7 /*endfinally*/];
                     case 8:
                         if (!(graphqlModelDeclaration.create &&
-                            'before' in graphqlModelDeclaration.create &&
-                            graphqlModelDeclaration.create.before)) return [3 /*break*/, 16];
-                        beforeCreate = Array.isArray(graphqlModelDeclaration.create.before)
-                            ? graphqlModelDeclaration.create.before
-                            : [graphqlModelDeclaration.create.before];
+                            'beforeCreate' in graphqlModelDeclaration.create &&
+                            graphqlModelDeclaration.create.beforeCreate)) return [3 /*break*/, 16];
+                        beforeCreate = Array.isArray(graphqlModelDeclaration.create.beforeCreate)
+                            ? graphqlModelDeclaration.create.beforeCreate
+                            : [graphqlModelDeclaration.create.beforeCreate];
                         _f.label = 9;
                     case 9:
                         _f.trys.push([9, 14, 15, 16]);
@@ -220,11 +220,11 @@ function generateMutationCreate(modelName, inputType, outputType, model, graphql
                             return [2 /*return*/, undefined];
                         }
                         if (!(graphqlModelDeclaration.create &&
-                            'after' in graphqlModelDeclaration.create &&
-                            graphqlModelDeclaration.create.after)) return [3 /*break*/, 33];
-                        afterList = Array.isArray(graphqlModelDeclaration.create.after)
-                            ? graphqlModelDeclaration.create.after
-                            : [graphqlModelDeclaration.create.after];
+                            'afterCreate' in graphqlModelDeclaration.create &&
+                            graphqlModelDeclaration.create.afterCreate)) return [3 /*break*/, 33];
+                        afterList = Array.isArray(graphqlModelDeclaration.create.afterCreate)
+                            ? graphqlModelDeclaration.create.afterCreate
+                            : [graphqlModelDeclaration.create.afterCreate];
                         createdEntity = newEntity;
                         hookData = { data: createdEntity.get({ plain: true }) };
                         _f.label = 24;