Merge pull request #15 from pinojs/feat/source-maps

Add source maps support

.travis.yml

@@ -5,5 +5,6 @@ node_js:
 - '8'
 - '10'
 - '12'
+- '13'
 script:
 - npm run coverage

examples/.gitignore

@@ -0,0 +1,5 @@
+node_modules
+*.log
+.nyc_output
+module-ts.js
+module-ts.js.map

examples/module-ts.ts

@@ -0,0 +1,12 @@
+const pino = process.env.NODE_ENV === 'development' ? require('../')(require('pino')()) : require('pino')()
+
+function add(n: number, m: number) {
+  return n + m
+}
+
+async function main() {
+  pino.info(add(2, 3))
+}
+
+main()
+

examples/package.json

@@ -0,0 +1,17 @@
+{
+  "name": "examples",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "scripts": {
+    "compile": "tsc -sourcemap module-ts.ts",
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "keywords": [],
+  "author": "",
+  "license": "ISC",
+  "dependencies": {
+    "@types/node": "^12.7.5",
+    "typescript": "^3.6.3"
+  }
+}

index.js

@@ -1,5 +1,6 @@
 'use strict'
 
+require('source-map-support/register')
 const NODEJS_VERSION = parseInt(process.version.slice(1).split('.')[0], 10)
 const STACKTRACE_OFFSET = NODEJS_VERSION && NODEJS_VERSION > 6 ? 2 : 3
 const LINE_OFFSET = 7

package.json

@@ -11,7 +11,7 @@
       "patterns": [
         "."
       ],
-      "extensions": "js"
+      "extensions": ["js", "ts"]
     }
   },
   "devDependencies": {
@@ -20,7 +20,8 @@
     "pino-debug": "^1.3.0",
     "standard": "^8.6.0",
     "tap": "^12.0.1",
-    "through2": "^2.0.3"
+    "through2": "^2.0.3",
+    "typescript": "^3.7.2"
   },
   "peerDependencies": {
     "pino": "*"
@@ -32,9 +33,17 @@
     "lint-fix": "standard --fix index.js examples/*js tests/*.js",
     "pretest": "npm run lint-fix",
     "test": "tap tests/*.js",
-    "test-watch": "tap tests/*.js",
-    "coverage": "tap --cov tests/*.js",
+    "test-watch": "tap tests/*.js tests/*.ts",
+    "coverage": "npm run pretest && tap --node-arg=\"-r\" --node-arg=\"source-map-support/register\" --cov tests/*.js tests/*.ts",
     "example": "env NODE_ENV=development node examples/index.js",
     "watch": "npm-watch"
+  },
+  "dependencies": {
+    "source-map-support": "^0.5.13"
+  },
+  "standard": {
+    "ignore": [
+      "examples/module-ts.js"
+    ]
   }
 }

tests/.gitignore

@@ -0,0 +1,2 @@
+test-ts.js
+test-ts.js.map

tests/test-ts.ts

@@ -0,0 +1,18 @@
+import tap = require('tap')
+import pino = require('pino')
+import through2 = require('through2')
+import pinoCaller = require('../')
+
+tap.test('pino caller works with sourcemaps (typescript)', function (t) {
+  t.plan(3)
+
+  const pinoInstance = pinoCaller(pino(through2(function (chunk, enc, callback) {
+    const res = JSON.parse(chunk.toString('utf8'))
+    const regex = /Test.<anonymous> \(\/(.)*tests\/test-ts.ts/
+    t.ok(res.caller, 'caller property is set')
+    t.equal(typeof res.caller, 'string', 'caller property is a string')
+    t.ok(regex.test(res.caller), 'caller property matches the test regex')
+  })))
+
+  pinoInstance.info('test')
+})