@abedmurrar/objection 4.0.0-beta.8 → 4.0.0-beta.9

package/lib/queryBuilder/RelationExpression.js

@@ -1,40 +1,9 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DuplicateRelationError = exports.RelationExpressionParseError = exports.RelationExpression = void 0;
-const parser = __importStar(require("./parsers/relationExpressionParser"));
+// A named import: `import * as` of a JavaScript module wraps it in an object
+// of getters (tsc's __importStar), which is slow to call through.
+const relationExpressionParser_1 = require("./parsers/relationExpressionParser");
 const objectUtils_1 = require("../utils/objectUtils");
 const RelationDoesNotExistError_1 = require("../model/RelationDoesNotExistError");
 class RelationExpressionParseError extends Error {
@@ -212,7 +181,7 @@ function parse(str) {
         return cloneNode(cachedNode);
     }
     else {
-        const node = parser.parse(str);
+        const node = (0, relationExpressionParser_1.parse)(str);
         parseCache.set(str, cloneNode(node));
         return node;
     }

package/lib/utils/objectUtils.js

@@ -25,7 +25,12 @@ exports.last = last;
 exports.get = get;
 exports.set = set;
 const cloneDeep_1 = require("./cloneDeep");
-Object.defineProperty(exports, "cloneDeep", { enumerable: true, get: function () { return cloneDeep_1.cloneDeep; } });
+// Re-exporting an imported binding makes tsc define the export as a getter,
+// which puts this module's whole exports object in V8's slow dictionary mode
+// and makes every `objectUtils.x()` call several times slower. A local
+// constant is exported as a plain property instead.
+const cloneDeep = cloneDeep_1.cloneDeep;
+exports.cloneDeep = cloneDeep;
 const SMALL_ARRAY_SIZE = 10;
 function isEmpty(item) {
     if (Array.isArray(item) || Buffer.isBuffer(item)) {

package/lib/utils/parseFieldExpression.js

@@ -1,40 +1,9 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseFieldExpression = parseFieldExpression;
-const jsonFieldExpressionParser = __importStar(require("../queryBuilder/parsers/jsonFieldExpressionParser"));
+// A named import: `import * as` of a JavaScript module wraps it in an object
+// of getters (tsc's __importStar), which is slow to call through.
+const jsonFieldExpressionParser_1 = require("../queryBuilder/parsers/jsonFieldExpressionParser");
 const cache = new Map();
 function parseFieldExpression(expr) {
     let parsedExpr = cache.get(expr);
@@ -42,7 +11,7 @@ function parseFieldExpression(expr) {
         return parsedExpr;
     }
     else {
-        const parserOutput = jsonFieldExpressionParser.parse(expr);
+        const parserOutput = (0, jsonFieldExpressionParser_1.parse)(expr);
         parsedExpr = preprocessParsedExpression(parserOutput);
         // We don't take a copy of the parsedExpr each time we
         // use if from cache. Instead to make sure it's never

package/lib/utils/promiseUtils/index.js

@@ -1,15 +1,25 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.try = exports.map = exports.mapAfterAllReturn = exports.afterReturn = exports.after = exports.isPromise = void 0;
-const isPromise_1 = require("./isPromise");
-Object.defineProperty(exports, "isPromise", { enumerable: true, get: function () { return isPromise_1.isPromise; } });
+exports.try = exports.mapAfterAllReturn = exports.map = exports.isPromise = exports.afterReturn = exports.after = void 0;
 const after_1 = require("./after");
-Object.defineProperty(exports, "after", { enumerable: true, get: function () { return after_1.after; } });
 const afterReturn_1 = require("./afterReturn");
-Object.defineProperty(exports, "afterReturn", { enumerable: true, get: function () { return afterReturn_1.afterReturn; } });
-const mapAfterAllReturn_1 = require("./mapAfterAllReturn");
-Object.defineProperty(exports, "mapAfterAllReturn", { enumerable: true, get: function () { return mapAfterAllReturn_1.mapAfterAllReturn; } });
+const isPromise_1 = require("./isPromise");
 const map_1 = require("./map");
-Object.defineProperty(exports, "map", { enumerable: true, get: function () { return map_1.promiseMap; } });
+const mapAfterAllReturn_1 = require("./mapAfterAllReturn");
 const try_1 = require("./try");
-Object.defineProperty(exports, "try", { enumerable: true, get: function () { return try_1.promiseTry; } });
+// Re-exporting imported bindings makes tsc define the exports as getters,
+// which puts this module's exports object in V8's slow dictionary mode and
+// makes every call through it several times slower. Local constants are
+// exported as plain properties instead.
+const isPromise = isPromise_1.isPromise;
+exports.isPromise = isPromise;
+const after = after_1.after;
+exports.after = after;
+const afterReturn = afterReturn_1.afterReturn;
+exports.afterReturn = afterReturn;
+const mapAfterAllReturn = mapAfterAllReturn_1.mapAfterAllReturn;
+exports.mapAfterAllReturn = mapAfterAllReturn;
+const map = map_1.promiseMap;
+exports.map = map;
+const promiseTryExport = try_1.promiseTry;
+exports.try = promiseTryExport;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abedmurrar/objection",
-  "version": "4.0.0-beta.8",
+  "version": "4.0.0-beta.9",
   "description": "An SQL-friendly ORM for Node.js. A fork of objection.js with strict TypeScript types and MySQL support.",
   "main": "lib/objection.js",
   "type": "commonjs",