.github/workflows/continuous-integration.yml

@@ -1,12 +1,44 @@
 name: Continuous Integration
-on: [push, pull_request]
+on:
+  # branches pushed by collaborators
+  push:
+    branches:
+      - master
+  # pull request from non-collaborators
+  pull_request: {}
+  # nightly
+  schedule:
+    - cron:  '0 0 * * *'
 jobs:
-  test:
-    name: "Test #${{ matrix.flavor }}: node v${{ matrix.node }}, ${{ matrix.typescript }}"
+  lint-build:
+    name: "Lint & Build"
     runs-on: ubuntu-latest
+    steps:
+      # checkout code
+      - uses: actions/checkout@v2
+      # install node
+      - name: Use Node.js 14
+        uses: actions/setup-node@v1
+        with:
+          node-version: 14
+      # lint, build, test
+      - run: npm install
+      - run: npm run lint
+      - run: npm run build
+      - name: Upload package artifact
+        uses: actions/upload-artifact@v1
+        with:
+          name: ts-node-packed.tgz
+          path: tests/ts-node-packed.tgz
+
+  test:
+    needs: lint-build
+    name: "Test #${{ matrix.flavor }}: ${{ matrix.os }}, node v${{ matrix.node }}, ${{ matrix.typescript }}"
+    runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
+        os: [ubuntu-latest, windows-latest]
         flavor: [1, 2, 3, 4, 5, 6, 7]
         include:
           - flavor: 1
@@ -40,7 +72,28 @@ jobs:
           node-version: ${{ matrix.node }}
       # lint, build, test
       - run: npm install
-      - run: npm run build
-      - run: npm rm tslint
-      - run: npm install ${{ matrix.typescript }}
+      - run: npm run build-nopack
+      - name: Download package artifact
+        uses: actions/download-artifact@v1
+        with:
+          name: ts-node-packed.tgz
+          path: tests/
+      - run: npm install ${{ matrix.typescript }} --force
       - run: npm run test-cov
+      - name: Coveralls
+        if: ${{ always() }}
+        uses: coverallsapp/github-action@master
+        with:
+          github-token: ${{ secrets.github_token }}
+          flag-name: run-${{ matrix.os }}-${{ matrix.flavor }}
+          parallel: true
+  finish:
+    needs: test
+    runs-on: ubuntu-latest
+    if: ${{ always() }}
+    steps:
+      - name: Coveralls Finished
+        uses: coverallsapp/github-action@master
+        with:
+          github-token: ${{ secrets.github_token }}
+          parallel-finished: true

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-node",
-  "version": "8.10.1",
+  "version": "8.10.2",
   "description": "TypeScript execution environment and REPL for node.js, with source map support",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -22,14 +22,16 @@
   "scripts": {
     "lint": "tslint \"src/**/*.ts\" --project tsconfig.json",
     "lint-fix": "tslint \"src/**/*.ts\" --project tsconfig.json --fix",
-    "clean": "rimraf dist && rimraf tsconfig.schema.json && rimraf tsconfig.schemastore-schema.json",
-    "build": "npm run clean && npm run build-tsc && npm run build-configSchema",
+    "clean": "rimraf dist && rimraf tsconfig.schema.json && rimraf tsconfig.schemastore-schema.json && rimraf tests/ts-node-packed.tgz",
+    "build": "npm run build-nopack && npm run build-pack",
+    "build-nopack": "npm run clean && npm run build-tsc && npm run build-configSchema",
     "build-tsc": "tsc",
     "build-configSchema": "typescript-json-schema --topRef --refs --validationKeywords allOf --out tsconfig.schema.json tsconfig.json TsConfigSchema && node --require ./register ./scripts/create-merged-schema",
+    "build-pack": "node ./scripts/build-pack.js",
     "test-spec": "mocha dist/**/*.spec.js -R spec --bail",
     "test-cov": "istanbul cover node_modules/mocha/bin/_mocha -- \"dist/**/*.spec.js\" -R spec --bail",
     "test": "npm run build && npm run lint && npm run test-cov",
-    "prepare": "npm run build"
+    "prepare": "npm run build-nopack"
   },
   "engines": {
     "node": ">=6.0.0"
@@ -60,7 +62,7 @@
     "@types/chai": "^4.0.4",
     "@types/diff": "^4.0.2",
     "@types/mocha": "^5.2.7",
-    "@types/node": "^13.13.1",
+    "@types/node": "13.13.5",
     "@types/proxyquire": "^1.3.28",
     "@types/react": "^16.0.2",
     "@types/semver": "^7.1.0",
@@ -76,8 +78,9 @@
     "semver": "^7.1.3",
     "tslint": "^6.1.0",
     "tslint-config-standard": "^9.0.0",
-    "typescript": "^3.7.2",
-    "typescript-json-schema": "^0.42.0"
+    "typescript": "3.8.3",
+    "typescript-json-schema": "^0.42.0",
+    "util.promisify": "^1.0.1"
   },
   "peerDependencies": {
     "typescript": ">=2.7"

scripts/build-pack.js

@@ -0,0 +1,20 @@
+// Written in JS to support Windows
+// Would otherwise be written as inline bash in package.json script
+
+const { exec } = require('child_process')
+const { mkdtempSync, writeFileSync, readFileSync, unlinkSync, rmdirSync, readdirSync } = require('fs')
+const { join } = require('path')
+
+const testDir = join(__dirname, '../tests')
+const tarballPath = join(testDir, 'ts-node-packed.tgz')
+const tempDir = mkdtempSync(join(testDir, 'tmp'))
+exec(`npm pack "${join(__dirname, '..')}"`, { cwd: tempDir }, (err, stdout) => {
+  if (err) {
+    console.error(err)
+    process.exit(1)
+  }
+  const tempTarballPath = join(tempDir, readdirSync(tempDir)[0])
+  writeFileSync(tarballPath, readFileSync(tempTarballPath))
+  unlinkSync(tempTarballPath)
+  rmdirSync(tempDir)
+})

src/esm.ts

@@ -1,6 +1,6 @@
 import { register, getExtensions, RegisterOptions } from './index'
-import { parse as parseUrl, format as formatUrl, UrlWithStringQuery } from 'url'
-import { posix as posixPath } from 'path'
+import { parse as parseUrl, format as formatUrl, UrlWithStringQuery, fileURLToPath, pathToFileURL } from 'url'
+import { extname } from 'path'
 import * as assert from 'assert'
 const { createResolve } = require('../dist-raw/node-esm-resolve-implementation')
 
@@ -65,13 +65,12 @@ export function registerAndCreateEsmHooks (opts?: RegisterOptions) {
     const { pathname } = parsed
     assert(pathname !== null, 'ESM getFormat() hook: URL should never have null pathname')
 
+    const nativePath = fileURLToPath(url)
+
     // If file has .ts, .tsx, or .jsx extension, then ask node how it would treat this file if it were .js
-    const ext = posixPath.extname(pathname!)
+    const ext = extname(nativePath)
     if (ext === '.ts' || ext === '.tsx' || ext === '.jsx') {
-      return defer(formatUrl({
-        ...parsed,
-        pathname: pathname + '.js'
-      }))
+      return defer(formatUrl(pathToFileURL(nativePath + '.js')))
     }
 
     return defer()
@@ -88,20 +87,13 @@ export function registerAndCreateEsmHooks (opts?: RegisterOptions) {
     if (!isFileUrlOrNodeStyleSpecifier(parsed)) {
       return defer()
     }
-    const { pathname } = parsed
-    if (pathname === null || !posixPath.isAbsolute(pathname)) {
-      // If we are meant to handle this URL, then it has already been resolved to an absolute path by our resolver hook
-      return defer()
-    }
-
-    // Assigning to a new variable so it's clear that we have stopped thinking of it as a URL, and started using it like a native FS path
-    const fileName = pathname
+    const nativePath = fileURLToPath(url)
 
-    if (tsNodeInstance.ignored(fileName)) {
+    if (tsNodeInstance.ignored(nativePath)) {
       return defer()
     }
 
-    const emittedJs = tsNodeInstance.compile(sourceAsString, fileName)
+    const emittedJs = tsNodeInstance.compile(sourceAsString, nativePath)
 
     return { source: emittedJs }
   }

src/externs.d.ts

@@ -0,0 +1,4 @@
+declare module 'util.promisify' {
+  const _export: typeof import('util').promisify
+  export = _export
+}

src/index.spec.ts

@@ -5,16 +5,28 @@ import semver = require('semver')
 import ts = require('typescript')
 import proxyquire = require('proxyquire')
 import { register, create, VERSION } from './index'
+import { unlinkSync, existsSync, statSync } from 'fs'
+import * as promisify from 'util.promisify'
+
+const execP = promisify(exec)
 
 const TEST_DIR = join(__dirname, '../tests')
 const PROJECT = join(TEST_DIR, 'tsconfig.json')
-const BIN_PATH = join(__dirname, '../dist/bin')
-const BIN_SCRIPT_PATH = join(__dirname, '../dist/bin-script')
+const BIN_PATH = join(TEST_DIR, 'node_modules/.bin/ts-node')
+const BIN_SCRIPT_PATH = join(TEST_DIR, 'node_modules/.bin/ts-node-script')
 
 const SOURCE_MAP_REGEXP = /\/\/# sourceMappingURL=data:application\/json;charset=utf\-8;base64,[\w\+]+=*$/
 
+// Pack and install ts-node locally, necessary to test package "exports"
+before(async function () {
+  this.timeout(30000)
+  await execP(`npm install`, { cwd: TEST_DIR })
+  const packageLockPath = join(TEST_DIR, 'package-lock.json')
+  existsSync(packageLockPath) && unlinkSync(packageLockPath)
+})
+
 describe('ts-node', function () {
-  const cmd = `node "${BIN_PATH}" --project "${PROJECT}"`
+  const cmd = `"${BIN_PATH}" --project "${PROJECT}"`
 
   this.timeout(10000)
 
@@ -35,7 +47,7 @@ describe('ts-node', function () {
     })
 
     it('should register via cli', function (done) {
-      exec(`node -r ../register hello-world.ts`, {
+      exec(`node -r ts-node/register hello-world.ts`, {
         cwd: TEST_DIR
       }, function (err, stdout) {
         expect(err).to.equal(null)
@@ -73,7 +85,7 @@ describe('ts-node', function () {
     })
 
     it('should provide registered information on register', function (done) {
-      exec(`node -r ../register env.ts`, {
+      exec(`node -r ts-node/register env.ts`, {
         cwd: TEST_DIR
       }, function (err, stdout) {
         expect(err).to.equal(null)
@@ -177,7 +189,7 @@ describe('ts-node', function () {
         }
 
         expect(err.message).to.contain([
-          `${join(__dirname, '../tests/throw.ts')}:3`,
+          `${join(__dirname, '../tests/throw.ts')}:100`,
           '  bar () { throw new Error(\'this is a demo\') }',
           '                 ^',
           'Error: this is a demo'
@@ -194,7 +206,7 @@ describe('ts-node', function () {
         }
 
         expect(err.message).to.contain([
-          `${join(__dirname, '../tests/throw.ts')}:3`,
+          `${join(__dirname, '../tests/throw.ts')}:100`,
           '  bar () { throw new Error(\'this is a demo\') }',
           '                 ^'
         ].join('\n'))
@@ -250,7 +262,7 @@ describe('ts-node', function () {
     })
 
     it('should pipe into an eval script', function (done) {
-      const cp = exec(`${cmd} --transpile-only -pe 'process.stdin.isTTY'`, function (err, stdout) {
+      const cp = exec(`${cmd} --transpile-only -pe "process.stdin.isTTY"`, function (err, stdout) {
         expect(err).to.equal(null)
         expect(stdout).to.equal('undefined\n')
 
@@ -357,18 +369,23 @@ describe('ts-node', function () {
 
     describe('issue #884', function () {
       it('should compile', function (done) {
-        exec(`node "${BIN_PATH}" --project tests/issue-884/tsconfig.json tests/issue-884`, function (err, stdout) {
-          expect(err).to.equal(null)
-          expect(stdout).to.equal('')
+        // TODO disabled because it consistently fails on Windows on TS 2.7
+        if (process.platform === 'win32' && semver.satisfies(ts.version, '2.7')) {
+          this.skip()
+        } else {
+          exec(`"${BIN_PATH}" --project tests/issue-884/tsconfig.json tests/issue-884`, function (err, stdout) {
+            expect(err).to.equal(null)
+            expect(stdout).to.equal('')
 
-          return done()
-        })
+            return done()
+          })
+        }
       })
     })
 
     describe('issue #986', function () {
       it('should not compile', function (done) {
-        exec(`node "${BIN_PATH}" --project tests/issue-986/tsconfig.json tests/issue-986`, function (err, stdout, stderr) {
+        exec(`"${BIN_PATH}" --project tests/issue-986/tsconfig.json tests/issue-986`, function (err, stdout, stderr) {
           expect(err).not.to.equal(null)
           expect(stderr).to.contain('Cannot find name \'TEST\'') // TypeScript error.
           expect(stdout).to.equal('')
@@ -378,7 +395,7 @@ describe('ts-node', function () {
       })
 
       it('should compile with `--files`', function (done) {
-        exec(`node "${BIN_PATH}" --files --project tests/issue-986/tsconfig.json tests/issue-986`, function (err, stdout, stderr) {
+        exec(`"${BIN_PATH}" --files --project tests/issue-986/tsconfig.json tests/issue-986`, function (err, stdout, stderr) {
           expect(err).not.to.equal(null)
           expect(stderr).to.contain('ReferenceError: TEST is not defined') // Runtime error.
           expect(stdout).to.equal('')
@@ -390,25 +407,29 @@ describe('ts-node', function () {
 
     if (semver.gte(ts.version, '2.7.0')) {
       it('should support script mode', function (done) {
-        exec(`node ${BIN_SCRIPT_PATH} tests/scope/a/log`, function (err, stdout) {
+        exec(`${BIN_SCRIPT_PATH} tests/scope/a/log`, function (err, stdout) {
           expect(err).to.equal(null)
           expect(stdout).to.equal('.ts\n')
 
           return done()
         })
       })
       it('should read tsconfig relative to realpath, not symlink, in scriptMode', function (done) {
-        exec(`node ${BIN_SCRIPT_PATH} tests/main-realpath/symlink/symlink.tsx`, function (err, stdout) {
-          expect(err).to.equal(null)
-          expect(stdout).to.equal('')
+        if (statSync(join(TEST_DIR, 'main-realpath/symlink/symlink.tsx')).isSymbolicLink()) {
+          exec(`${BIN_SCRIPT_PATH} tests/main-realpath/symlink/symlink.tsx`, function (err, stdout) {
+            expect(err).to.equal(null)
+            expect(stdout).to.equal('')
 
-          return done()
-        })
+            return done()
+          })
+        } else {
+          this.skip()
+        }
       })
     }
 
     describe('should read ts-node options from tsconfig.json', function () {
-      const BIN_EXEC = `node "${join(__dirname, '../dist/bin')}" --project tests/tsconfig-options/tsconfig.json`
+      const BIN_EXEC = `"${BIN_PATH}" --project tests/tsconfig-options/tsconfig.json`
 
       it('should override compiler options from env', function (done) {
         exec(`${BIN_EXEC} tests/tsconfig-options/log-options.js`, {
@@ -419,7 +440,7 @@ describe('ts-node', function () {
         }, function (err, stdout) {
           expect(err).to.equal(null)
           const { config } = JSON.parse(stdout)
-          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/env-typeroots')])
+          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/env-typeroots').replace(/\\/g, '/')])
           return done()
         })
       })
@@ -428,7 +449,7 @@ describe('ts-node', function () {
         exec(`${BIN_EXEC} tests/tsconfig-options/log-options.js`, function (err, stdout) {
           expect(err).to.equal(null)
           const { options, config } = JSON.parse(stdout)
-          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots')])
+          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots').replace(/\\/g, '/')])
           expect(config.options.types).to.deep.equal(['tsconfig-tsnode-types'])
           expect(options.pretty).to.equal(undefined)
           expect(options.skipIgnore).to.equal(false)
@@ -438,10 +459,10 @@ describe('ts-node', function () {
       })
 
       it('should have flags override `tsconfig.json`', function (done) {
-        exec(`${BIN_EXEC} --skip-ignore --compiler-options '{"types": ["flags-types"]}' tests/tsconfig-options/log-options.js`, function (err, stdout) {
+        exec(`${BIN_EXEC} --skip-ignore --compiler-options "{\\"types\\":[\\"flags-types\\"]}" tests/tsconfig-options/log-options.js`, function (err, stdout) {
           expect(err).to.equal(null)
           const { options, config } = JSON.parse(stdout)
-          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots')])
+          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots').replace(/\\/g, '/')])
           expect(config.options.types).to.deep.equal(['flags-types'])
           expect(options.pretty).to.equal(undefined)
           expect(options.skipIgnore).to.equal(true)
@@ -460,7 +481,7 @@ describe('ts-node', function () {
         }, function (err, stdout) {
           expect(err).to.equal(null)
           const { options, config } = JSON.parse(stdout)
-          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots')])
+          expect(config.options.typeRoots).to.deep.equal([join(__dirname, '../tests/tsconfig-options/tsconfig-typeroots').replace(/\\/g, '/')])
           expect(config.options.types).to.deep.equal(['tsconfig-tsnode-types'])
           expect(options.pretty).to.equal(true)
           expect(options.skipIgnore).to.equal(false)
@@ -481,7 +502,7 @@ describe('ts-node', function () {
       })
 
       it('should give ts error for invalid node_modules', function (done) {
-        exec(`${cmd} --compiler-host --skip-ignore tests/from-node-modules`, function (err, stdout) {
+        exec(`${cmd} --compiler-host --skip-ignore tests/from-node-modules/from-node-modules`, function (err, stdout) {
           if (err === null) return done('Expected an error')
 
           expect(err.message).to.contain('Unable to compile file from external library')
@@ -599,7 +620,7 @@ describe('ts-node', function () {
       } catch (error) {
         expect(error.stack).to.contain([
           'Error: this is a demo',
-          `    at Foo.bar (${join(__dirname, '../tests/throw.ts')}:3:18)`
+          `    at Foo.bar (${join(__dirname, '../tests/throw.ts')}:100:18)`
         ].join('\n'))
 
         done()

src/index.ts

@@ -420,7 +420,7 @@ export function create (rawOptions: CreateOptions = {}): Register {
   sourceMapSupport.install({
     environment: 'node',
     retrieveFile (path: string) {
-      return outputCache.get(path)?.content || ''
+      return outputCache.get(normalizeSlashes(path))?.content || ''
     }
   })
 

tests/.gitignore

@@ -1 +1,3 @@
-!node_modules/
+!from-node-modules/node_modules/
+package-lock.json
+ts-node-packed.tgz

tests/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "ts-node": "file:ts-node-packed.tgz"
+  }
+}

tests/throw.ts

@@ -1,4 +1,101 @@
+// intentional whitespace to prove that sourcemaps are working.  Throw should happen on line 100.
+// 100 lines is meant to be far more space than the helper functions would take.
 class Foo {
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
   constructor () { this.bar() }
   bar () { throw new Error('this is a demo') }
 }