@vita-mojo/lambda-logger 0.0.45 → 0.0.46-VMOS-14435-707bfce-34-rc.0

package/package.json

@@ -1,6 +1,11 @@
 {
   "name": "@vita-mojo/lambda-logger",
-  "version": "0.0.45",
+  "version": "0.0.46-VMOS-14435-707bfce-34-rc.0",
   "type": "commonjs",
+  "dependencies": {
+    "tslib": "^2.8.1",
+    "lambda-log": "^3.1.0"
+  },
+  "types": "./src/index.d.ts",
   "main": "./src/index.js"
-}
+}

package/src/lib/lambda-logger.js

@@ -2,7 +2,30 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.lambdaLogger = exports.LambdaLogger = void 0;
 const log = require("lambda-log");
-const R = require("ramda");
+/**
+ * Replaces `R.isEmpty`.
+ *
+ * Deliberately narrower than "has no own enumerable keys". Only strings,
+ * arrays, plain objects, `arguments` and typed arrays have an empty form;
+ * every other object counts as NOT empty. An Error has no own enumerable keys,
+ * so `Object.keys(err).length === 0` would call it empty and drop the payload
+ * - and `error(msg: string | Error, meta?: object)` invites exactly that call.
+ * Losing an error from an error log is the worst failure this file can have.
+ */
+const isEmptyPayload = (value) => {
+    if (typeof value === 'string' || Array.isArray(value))
+        return value.length === 0;
+    if (typeof value !== 'object' || value === null)
+        return false;
+    const tag = Object.prototype.toString.call(value);
+    if (tag === '[object Object]')
+        return Object.keys(value).length === 0;
+    if (ArrayBuffer.isView(value))
+        return value.byteLength === 0;
+    if (tag === '[object Arguments]')
+        return value.length === 0;
+    return false;
+};
 class LambdaLogger extends log.LambdaLog {
     constructor(options) {
         super(options);
@@ -20,7 +43,14 @@ class LambdaLogger extends log.LambdaLog {
         this.updateMeta({ data: Object.assign(Object.assign({}, this.options.meta.data), dataFields) });
     }
     log(level, msg, data, tags = []) {
-        if (R.isEmpty(data)) {
+        // The undefined check matters as much as the emptiness one. `data` is
+        // optional, and an omitted argument used to fall through to the branch
+        // below, where lambda-log merged `{ data: undefined }` over the meta --
+        // silently erasing anything setData/updateData had established. Passing
+        // `{}` preserved it, so omitting the argument behaved WORSE than passing
+        // an empty one. null is covered too, for JavaScript callers the optional
+        // type does not constrain.
+        if (data === undefined || data === null || isEmptyPayload(data)) {
             return super.log(level, msg, {}, tags);
         }
         return super.log(level, msg, { data }, tags);

package/src/lib/lambda-logger.js.map

@@ -1 +1 @@
-{"version":3,"file":"lambda-logger.js","sourceRoot":"","sources":["../../../../../packages/lambda-logger/src/lib/lambda-logger.ts"],"names":[],"mappings":";;;AAAA,kCAAkC;AAClC,2BAA2B;AAI3B,MAAa,YAAa,SAAQ,GAAG,CAAC,SAAS;IAC7C,YAAY,OAA4B;QACtC,KAAK,CAAC,OAAO,CAAC,CAAC;IACjB,CAAC;IAED,OAAO,CAAC,IAAyB;QAC/B,IAAI,CAAC,OAAO,CAAC,IAAI,GAAG,IAAI,CAAC;IAC3B,CAAC;IAED,UAAU,CAAC,UAA+B;QACxC,IAAI,CAAC,OAAO,CAAC,IAAI,mCAAQ,IAAI,CAAC,OAAO,CAAC,IAAI,GAAK,UAAU,CAAE,CAAC;IAC9D,CAAC;IAED,OAAO,CAAC,IAAyB;QAC/B,IAAI,CAAC,UAAU,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;IAC5B,CAAC;IAED,UAAU,CAAC,UAA+B;QACxC,IAAI,CAAC,UAAU,CAAC,EAAE,IAAI,kCAAO,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,GAAK,UAAU,CAAE,EAAE,CAAC,CAAC;IAC1E,CAAC;IAEQ,GAAG,CAAC,KAAa,EAAE,GAAQ,EAAE,IAA0B,EAAE,IAAI,GAAG,EAAE;QACzE,IAAI,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC;YACpB,OAAO,KAAK,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,CAAC,CAAC;QACzC,CAAC;QAED,OAAO,KAAK,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE,IAAI,EAAE,EAAE,IAAI,CAAC,CAAC;IAC/C,CAAC;CACF;AA5BD,oCA4BC;AAED,MAAM,OAAO,GAAG;IACd,QAAQ,EAAE,UAAU;IACpB,UAAU,EAAE,SAAS;IACrB,OAAO,EAAE,MAAM;CAChB,CAAC;AAEW,QAAA,YAAY,GAAkB,IAAI,YAAY,CAAC,OAAO,CAAC,CAAC"}
+{"version":3,"file":"lambda-logger.js","sourceRoot":"","sources":["../../../../../packages/lambda-logger/src/lib/lambda-logger.ts"],"names":[],"mappings":";;;AAAA,kCAAkC;AAIlC;;;;;;;;;GASG;AACH,MAAM,cAAc,GAAG,CAAC,KAAc,EAAW,EAAE;IACjD,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;QACnD,OAAO,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC;IAC5B,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI;QAAE,OAAO,KAAK,CAAC;IAE9D,MAAM,GAAG,GAAG,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClD,IAAI,GAAG,KAAK,iBAAiB;QAAE,OAAO,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC;IACtE,IAAI,WAAW,CAAC,MAAM,CAAC,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC,UAAU,KAAK,CAAC,CAAC;IAC7D,IAAI,GAAG,KAAK,oBAAoB;QAC9B,OAAQ,KAA4B,CAAC,MAAM,KAAK,CAAC,CAAC;IAEpD,OAAO,KAAK,CAAC;AACf,CAAC,CAAC;AAEF,MAAa,YAAa,SAAQ,GAAG,CAAC,SAAS;IAC7C,YAAY,OAA4B;QACtC,KAAK,CAAC,OAAO,CAAC,CAAC;IACjB,CAAC;IAED,OAAO,CAAC,IAAyB;QAC/B,IAAI,CAAC,OAAO,CAAC,IAAI,GAAG,IAAI,CAAC;IAC3B,CAAC;IAED,UAAU,CAAC,UAA+B;QACxC,IAAI,CAAC,OAAO,CAAC,IAAI,mCAAQ,IAAI,CAAC,OAAO,CAAC,IAAI,GAAK,UAAU,CAAE,CAAC;IAC9D,CAAC;IAED,OAAO,CAAC,IAAyB;QAC/B,IAAI,CAAC,UAAU,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;IAC5B,CAAC;IAED,UAAU,CAAC,UAA+B;QACxC,IAAI,CAAC,UAAU,CAAC,EAAE,IAAI,kCAAO,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,GAAK,UAAU,CAAE,EAAE,CAAC,CAAC;IAC1E,CAAC;IAEQ,GAAG,CAAC,KAAa,EAAE,GAAQ,EAAE,IAA0B,EAAE,IAAI,GAAG,EAAE;QACzE,sEAAsE;QACtE,uEAAuE;QACvE,wEAAwE;QACxE,wEAAwE;QACxE,yEAAyE;QACzE,yEAAyE;QACzE,2BAA2B;QAC3B,IAAI,IAAI,KAAK,SAAS,IAAI,IAAI,KAAK,IAAI,IAAI,cAAc,CAAC,IAAI,CAAC,EAAE,CAAC;YAChE,OAAO,KAAK,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,CAAC,CAAC;QACzC,CAAC;QAED,OAAO,KAAK,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE,IAAI,EAAE,EAAE,IAAI,CAAC,CAAC;IAC/C,CAAC;CACF;AAnCD,oCAmCC;AAED,MAAM,OAAO,GAAG;IACd,QAAQ,EAAE,UAAU;IACpB,UAAU,EAAE,SAAS;IACrB,OAAO,EAAE,MAAM;CAChB,CAAC;AAEW,QAAA,YAAY,GAAkB,IAAI,YAAY,CAAC,OAAO,CAAC,CAAC"}