@snowtop/ent 0.1.0-alpha35 → 0.1.0-alpha42

package/action/action.d.ts

@@ -49,7 +49,7 @@ export interface Action<TEnt extends Ent<TViewer>, TBuilder extends Builder<TEnt
     changeset(): Promise<Changeset>;
     builder: TBuilder;
     getPrivacyPolicy(): PrivacyPolicy<TEnt>;
-    getTriggers?(): Trigger<TEnt, TBuilder, TViewer, TInput, TExistingEnt>[];
+    getTriggers?(): (Trigger<TEnt, TBuilder, TViewer, TInput, TExistingEnt> | Trigger<TEnt, TBuilder, TViewer, TInput, TExistingEnt>[])[];
     getObservers?(): Observer<TEnt, TBuilder, TViewer, TInput, TExistingEnt>[];
     getValidators?(): Validator<TEnt, TBuilder, TViewer, TInput, TExistingEnt>[];
     getInput(): TInput;

package/action/orchestrator.js

@@ -333,22 +333,46 @@ class Orchestrator {
         ]);
     }
     async triggers(action, builder, triggers) {
-        await Promise.all(triggers.map(async (trigger) => {
-            let ret = await trigger.changeset(builder, action.getInput());
-            if (Array.isArray(ret)) {
-                ret = await Promise.all(ret);
-            }
-            if (Array.isArray(ret)) {
-                for (const v of ret) {
-                    if (typeof v === "object") {
-                        this.changesets.push(v);
-                    }
+        let groups = [];
+        let lastArray = 0;
+        let prevWasArray = false;
+        for (let i = 0; i < triggers.length; i++) {
+            let t = triggers[i];
+            if (Array.isArray(t)) {
+                if (!prevWasArray) {
+                    // @ts-ignore
+                    groups.push(triggers.slice(lastArray, i));
                 }
+                groups.push(t);
+                prevWasArray = true;
+                lastArray++;
             }
-            else if (ret) {
-                this.changesets.push(ret);
+            else {
+                if (i === triggers.length - 1) {
+                    // @ts-ignore
+                    groups.push(triggers.slice(lastArray, i + 1));
+                }
+                prevWasArray = false;
             }
-        }));
+        }
+        for (const triggers of groups) {
+            await Promise.all(triggers.map(async (trigger) => {
+                let ret = await trigger.changeset(builder, action.getInput());
+                if (Array.isArray(ret)) {
+                    ret = await Promise.all(ret);
+                }
+                if (Array.isArray(ret)) {
+                    for (const v of ret) {
+                        if (typeof v === "object") {
+                            this.changesets.push(v);
+                        }
+                    }
+                }
+                else if (ret) {
+                    this.changesets.push(ret);
+                }
+            }));
+        }
     }
     async validators(validators, action, builder) {
         let promises = [];

package/core/context.js

@@ -39,7 +39,7 @@ class ContextCache {
         const key = this.getkey(options);
         let rows = m.get(key);
         if (rows) {
-            (0, logger_1.log)("query", {
+            (0, logger_1.log)("cache", {
                 "cache-hit": key,
                 "tableName": options.tableName,
             });
@@ -54,7 +54,7 @@ class ContextCache {
         const key = this.getkey(options);
         let row = m.get(key);
         if (row) {
-            (0, logger_1.log)("query", {
+            (0, logger_1.log)("cache", {
                 "cache-hit": key,
                 "tableName": options.tableName,
             });

package/core/ent.js

@@ -38,7 +38,7 @@ class cacheMap {
     get(key) {
         const ret = this.m.get(key);
         if (ret) {
-            (0, logger_1.log)("query", {
+            (0, logger_1.log)("cache", {
                 "dataloader-cache-hit": key,
                 "tableName": this.options.tableName,
             });

package/core/loaders/loader.js

@@ -37,12 +37,12 @@ class cacheMap {
             // might be a lot?
             // TODO this is not the best log format
             // was designed for ObjectLoader time. Now we have different needs e.g. count, assoc etc
-            (0, logger_1.log)("query", {
+            (0, logger_1.log)("cache", {
                 "dataloader-cache-hit": key,
                 "tableName": this.options.tableName,
             });
             // } else {
-            //   log("query", {
+            //   log("cache", {
             //     "dataloader-cache-miss": key,
             //     "tableName": options.tableName,
             //   });
@@ -50,21 +50,21 @@ class cacheMap {
         return ret;
     }
     set(key, value) {
-        // log("query", {
+        // log("cache", {
         //   "dataloader-cache-set": key,
         //   "tableName": options.tableName,
         // });
         return this.m.set(key, value);
     }
     delete(key) {
-        // log("query", {
+        // log("cache", {
         //   "dataloader-cache-delete": key,
         //   "tableName": options.tableName,
         // });
         return this.m.delete(key);
     }
     clear() {
-        // log("query", {
+        // log("cache", {
         //   "dataloader-cache-clear": true,
         //   "tableName": options.tableName,
         // });

package/core/logger.d.ts

@@ -1,4 +1,4 @@
-declare type logType = "query" | "warn" | "info" | "error" | "debug";
+declare type logType = "query" | "warn" | "info" | "error" | "debug" | "cache";
 export declare function setLogLevels(levels: logType | logType[]): void;
 export declare function clearLogLevels(): void;
 export declare function log(level: logType, msg: any): void;

package/core/logger.js

@@ -7,6 +7,7 @@ var m = {
     info: "log",
     error: "error",
     debug: "debug",
+    cache: "log",
 };
 var logLevels = new Map();
 function setLogLevels(levels) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@snowtop/ent",
-  "version": "0.1.0-alpha35",
+  "version": "0.1.0-alpha42",
   "description": "snowtop ent framework",
   "main": "index.js",
   "types": "index.d.ts",

package/schema/struct_field.js

@@ -83,6 +83,12 @@ class StructField extends field_1.BaseField {
             }
             promises.push(field.valid(val));
         }
+        for (const k in obj) {
+            // extra undefined fields are invalid
+            if (this.options.fields[k] === undefined) {
+                return false;
+            }
+        }
         if (!valid) {
             return valid;
         }

package/testutils/builder.d.ts

@@ -80,9 +80,12 @@ export declare class SimpleBuilder<T extends Ent, TExistingEnt extends TMaybleNu
     orchestrator: Orchestrator<T, Data, Viewer>;
     fields: Map<string, any>;
     nodeType: string;
+    m: Map<string, any>;
     constructor(viewer: Viewer, schema: BuilderSchema<T>, fields: Map<string, any>, operation: WriteOperation, existingEnt: TExistingEnt, action?: Action<T, SimpleBuilder<T>, Viewer, Data> | undefined);
     getInput(): Data;
     updateInput(input: Data): void;
+    storeData(k: string, v: any): void;
+    getStoredData(k: string): any;
     build(): Promise<Changeset>;
     editedEnt(): Promise<T | null>;
     editedEntX(): Promise<T>;
@@ -100,7 +103,7 @@ export declare class SimpleAction<T extends Ent, TExistingEnt extends TMaybleNul
     builder: SimpleBuilder<T, TExistingEnt>;
     viewerForEntLoad: viewerEntLoadFunc | undefined;
     constructor(viewer: Viewer, schema: BuilderSchema<T>, fields: Map<string, any>, operation: WriteOperation | undefined, existingEnt: TExistingEnt);
-    getTriggers(): Trigger<T, SimpleBuilder<T>>[];
+    getTriggers(): (Trigger<T, SimpleBuilder<T>> | Array<Trigger<T, SimpleBuilder<T>>>)[];
     getValidators(): Validator<T, SimpleBuilder<T>>[];
     getObservers(): Observer<T, SimpleBuilder<T>>[];
     getPrivacyPolicy(): PrivacyPolicy<Ent<Viewer<Ent<any> | null, ID | null>>, Viewer<Ent<any> | null, ID | null>>;

package/testutils/builder.js

@@ -150,6 +150,7 @@ class SimpleBuilder {
         this.schema = schema;
         this.operation = operation;
         this.existingEnt = existingEnt;
+        this.m = new Map();
         // create dynamic placeholder
         // TODO: do we need to use this as the node when there's an existingEnt
         // same for generated builders.
@@ -230,6 +231,14 @@ class SimpleBuilder {
             }
         }
     }
+    // store data in Builder that can be retrieved by another validator, trigger, observer later in the action
+    storeData(k, v) {
+        this.m.set(k, v);
+    }
+    // retrieve data stored in this Builder with key
+    getStoredData(k) {
+        return this.m.get(k);
+    }
     build() {
         return this.orchestrator.build();
     }

package/tsc/transform_action.d.ts

@@ -11,10 +11,12 @@ export declare class TransformAction implements TransformFile {
         rawString?: undefined;
         traversed?: undefined;
         imports?: undefined;
+        removeImports?: undefined;
     } | {
         rawString: string;
         traversed: boolean;
         imports: Map<string, string[]>;
+        removeImports: string[];
         node?: undefined;
     } | undefined;
 }

package/tsc/transform_action.js

@@ -25,11 +25,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.TransformAction = void 0;
 const typescript_1 = __importDefault(require("typescript"));
 const ast_1 = require("../tsc/ast");
-const action_1 = require("../action");
 const viewer_1 = require("../core/viewer");
 const path = __importStar(require("path"));
 const snake_case_1 = require("snake-case");
-function findInput(file, classInfo, sourceFile) {
+// returns input and importPath
+function getBaseFileInfo(file, classInfo, sourceFile) {
     // @ts-ignore
     const importStatements = sourceFile.statements.filter((stmt) => typescript_1.default.isImportDeclaration(stmt));
     for (const imp of importStatements) {
@@ -49,13 +49,19 @@ function findInput(file, classInfo, sourceFile) {
                 .filter((imp) => imp.trim() && imp.endsWith("Input"))
                 .map((v) => v.trim());
             if (inputs.length === 1) {
-                return inputs[0];
+                return {
+                    input: inputs[0],
+                    importPath: impInfo.importPath,
+                };
             }
             if (inputs.length && classInfo.name.endsWith("Action")) {
                 const prefix = classInfo.name.slice(0, classInfo.name.length - 6);
                 inputs = inputs.filter((imp) => imp.slice(0, imp.length - 5) === prefix);
                 if (inputs.length === 1) {
-                    return inputs[0];
+                    return {
+                        input: inputs[0],
+                        importPath: impInfo.importPath,
+                    };
                 }
             }
         }
@@ -66,17 +72,20 @@ let m = {
     triggers: {
         m: "getTriggers",
         i: "Trigger",
+        suffix: "Triggers",
     },
     observers: {
         m: "getObservers",
         i: "Observer",
+        suffix: "Observers",
     },
     validators: {
         m: "getValidators",
         i: "Validator",
+        suffix: "Validators",
     },
 };
-function getConversionInfo(mm) {
+function getConversionInfo(mm, actionName) {
     if (mm.kind !== typescript_1.default.SyntaxKind.PropertyDeclaration) {
         return null;
     }
@@ -89,6 +98,8 @@ function getConversionInfo(mm) {
         text,
         method: v.m,
         interface: v.i,
+        // CreateFooActionTriggers etc
+        methodType: actionName + v.suffix,
     };
 }
 class TransformAction {
@@ -109,21 +120,19 @@ class TransformAction {
         // require action
         const p = require(path.join(process.cwd(), "./" + file.slice(0, -3)));
         const action = new p.default(new viewer_1.LoggedOutViewer(), {});
+        const actionName = action.constructor.name;
         const builder = action.builder.constructor.name;
         const nodeName = action.builder.ent.name;
-        const existingEnt = action.builder.operation === action_1.WriteOperation.Insert
-            ? `${nodeName} | null`
-            : nodeName;
         const viewer = this.customInfo.viewerInfo.name;
-        const input = findInput(file, classInfo, sourceFile);
-        if (!input) {
+        const baseInfo = getBaseFileInfo(file, classInfo, sourceFile);
+        if (!baseInfo) {
             return;
         }
         let klassContents = "";
         let traversed = false;
         let newImports = [];
         for (const mm of node.members) {
-            const conv = getConversionInfo(mm);
+            const conv = getConversionInfo(mm, actionName);
             if (conv !== null) {
                 const property = mm;
                 // if invalid, bounce
@@ -132,10 +141,10 @@ class TransformAction {
                 }
                 traversed = true;
                 const pp = property.initializer.getFullText(sourceFile).trim();
-                const code = `${conv.method}(): ${conv.interface}<${nodeName}, ${builder}<${input}, ${existingEnt}>, ${viewer}, ${input}, ${existingEnt}>[] {
+                const code = `${conv.method}(): ${conv.methodType} {
             return ${pp}
           }`;
-                newImports.push(conv.interface);
+                newImports.push(conv.methodType);
                 klassContents += (0, ast_1.getPreText)(contents, mm, sourceFile) + code;
             }
             else {
@@ -149,10 +158,9 @@ class TransformAction {
                 [viewer],
             ],
             [
-                (0, ast_1.transformRelative)(file, "src/ent", this.customInfo.relativeImports),
-                [nodeName],
+                (0, ast_1.transformRelative)(file, baseInfo.importPath, this.customInfo.relativeImports),
+                newImports,
             ],
-            ["@snowtop/ent/action", newImports],
             [
                 (0, ast_1.transformRelative)(file, builderPath, this.customInfo.relativeImports),
                 [builder],
@@ -163,6 +171,8 @@ class TransformAction {
             rawString: classInfo.wrapClassContents(klassContents),
             traversed,
             imports,
+            removeImports: ["Trigger", "Observer", "Validator"],
+            // not removing FooBuilder incase it's still somehow used in type of inline builders
         };
     }
 }