require to use extensions in import specifiers (#459)

* require to use extensions in import specifiers

* Apply suggestions from code review

* remove unrelated file

.eslintrc

@@ -4,9 +4,11 @@
   ],
   "env": {
     "es6": true,
-    "node": true,
-    "browser": true,
-    "jasmine": true
+    "node": false,
+    "jest": false,
+    "browser": false,
+    "shared-node-browser": true,
+    "jasmine": false
   },
   "parserOptions": {
     "ecmaVersion": 2020,
@@ -21,6 +23,60 @@
     "no-param-reassign": ["off"],
     "import/no-extraneous-dependencies": ["error", {
       "devDependencies": ["test/**/*.js"]
-    }]
-  }
+    }],
+    "import/extensions": ["error", "ignorePackages"]
+  },
+  "overrides": [
+    {
+      "files": ["lib/browser/**/*.js", "demos/**/*.js"],
+      "env": {
+        "browser": true
+      }
+    },
+    {
+      "files": ["lib/node/**/*.js"],
+      "env": {
+        "node": true
+      },
+      "rules": {
+        "no-restricted-globals": [
+          "error",
+          {
+            "name": "__filename",
+            "message": "Use import.meta.url instead"
+          },
+          {
+            "name": "__dirname",
+            "message": "Not available in ESM"
+          },
+          {
+            "name": "exports",
+            "message": "Not available in ESM"
+          },
+          {
+            "name": "module",
+            "message": "Not available in ESM"
+          },
+          {
+            "name": "require",
+            "message": "Use import instead"
+          }
+        ]
+      }
+    },
+    {
+      "files": ["test/**/*.js", "demos/**/*.js"],
+      "env": {
+        "browser": true,
+        "node": true,
+        "jasmine": true
+      },
+      "parserOptions": {
+        "sourceType": "script"
+      },
+      "rules": {
+        "import/extensions": "off"
+      }
+    }
+  ]
 }

.gitignore

@@ -1,3 +1,5 @@
+.eslintcache
+.vscode/settings.json
 node_modules
 demos/reactnative/.expo
 lib.es5

demos/browser/demo.js

@@ -1,6 +1,8 @@
 /* global tus */
 /* eslint-disable no-console, no-alert */
 
+'use strict'
+
 let upload          = null
 let uploadIsRunning = false
 const toggleBtn       = document.querySelector('#toggle-btn')

demos/browser/video.js

@@ -1,6 +1,8 @@
 /* global tus */
 /* eslint-disable no-console, no-alert */
 
+'use strict'
+
 let stopRecording   = null
 let upload          = null
 const recordBtn       = document.querySelector('#record-btn')

demos/cordova/hooks/before_prepare/copy_tus_files.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+'use strict'
+
 const fs = require('fs')
 const path = require('path')
 

demos/cordova/www/js/index.js

@@ -1,6 +1,8 @@
 /* global tus Camera */
 /* eslint-disable no-alert */
 
+'use strict'
+
 let upload = null
 let uploadIsRunning = false
 let file = null

demos/nodejs/index.js

@@ -1,5 +1,7 @@
 /* eslint-disable no-console */
 
+'use strict'
+
 const fs = require('fs')
 const tus = require('../..')
 

lib/browser/fileReader.js

@@ -1,8 +1,8 @@
-import isReactNative from './isReactNative'
-import uriToBlob from './uriToBlob'
+import isReactNative from './isReactNative.js'
+import uriToBlob from './uriToBlob.js'
 
-import FileSource from './sources/FileSource'
-import StreamSource from './sources/StreamSource'
+import FileSource from './sources/FileSource.js'
+import StreamSource from './sources/StreamSource.js'
 
 export default class FileReader {
   openFile (input, chunkSize) {

lib/browser/fileSignature.js

@@ -1,4 +1,4 @@
-import isReactNative from './isReactNative'
+import isReactNative from './isReactNative.js'
 
 // TODO: Differenciate between input types
 

lib/browser/index.js

@@ -1,12 +1,12 @@
-import BaseUpload from '../upload'
-import NoopUrlStorage from '../noopUrlStorage'
-import { enableDebugLog } from '../logger'
-import DetailedError from '../error'
+import BaseUpload from '../upload.js'
+import NoopUrlStorage from '../noopUrlStorage.js'
+import { enableDebugLog } from '../logger.js'
+import DetailedError from '../error.js'
 
-import { canStoreURLs, WebStorageUrlStorage } from './urlStorage'
-import DefaultHttpStack from './httpStack'
-import FileReader from './fileReader'
-import fingerprint from './fileSignature'
+import { canStoreURLs, WebStorageUrlStorage } from './urlStorage.js'
+import DefaultHttpStack from './httpStack.js'
+import FileReader from './fileReader.js'
+import fingerprint from './fileSignature.js'
 
 const defaultOptions = {
   ...BaseUpload.defaultOptions,

lib/browser/sources/FileSource.js

@@ -1,5 +1,5 @@
-import isCordova from './isCordova'
-import readAsByteArray from './readAsByteArray'
+import isCordova from './isCordova.js'
+import readAsByteArray from './readAsByteArray.js'
 
 export default class FileSource {
   // Make this.size a method

lib/node/fileReader.js

@@ -1,9 +1,9 @@
 import { ReadStream } from 'fs'
 import isStream from 'is-stream'
 
-import BufferSource from './sources/BufferSource'
-import getFileSource from './sources/FileSource'
-import StreamSource from './sources/StreamSource'
+import BufferSource from './sources/BufferSource.js'
+import getFileSource from './sources/FileSource.js'
+import StreamSource from './sources/StreamSource.js'
 
 export default class FileReader {
   openFile (input, chunkSize) {

lib/node/index.js

@@ -1,13 +1,13 @@
-import BaseUpload from '../upload'
-import NoopUrlStorage from '../noopUrlStorage'
-import { enableDebugLog } from '../logger'
-import DetailedError from '../error'
+import BaseUpload from '../upload.js'
+import NoopUrlStorage from '../noopUrlStorage.js'
+import { enableDebugLog } from '../logger.js'
+import DetailedError from '../error.js'
 
-import { FileUrlStorage, canStoreURLs } from './urlStorage'
-import DefaultHttpStack from './httpStack'
-import FileReader from './fileReader'
-import fingerprint from './fileSignature'
-import StreamSource from './sources/StreamSource'
+import { FileUrlStorage, canStoreURLs } from './urlStorage.js'
+import DefaultHttpStack from './httpStack.js'
+import FileReader from './fileReader.js'
+import fingerprint from './fileSignature.js'
+import StreamSource from './sources/StreamSource.js'
 
 const defaultOptions = {
   ...BaseUpload.defaultOptions,

lib/upload.js

@@ -1,8 +1,8 @@
 import { Base64 } from 'js-base64'
 import URL from 'url-parse'
-import DetailedError from './error'
-import { log } from './logger'
-import uuid from './uuid'
+import DetailedError from './error.js'
+import { log } from './logger.js'
+import uuid from './uuid.js'
 
 const defaultOptions = {
   endpoint: null,
@@ -924,8 +924,8 @@ function sendRequest (req, body, options) {
 function isOnline () {
   let online = true
   if (typeof window !== 'undefined'
-        && 'navigator' in window
-        && window.navigator.onLine === false) {
+        && 'navigator' in window // eslint-disable-line no-undef
+        && window.navigator.onLine === false) { // eslint-disable-line no-undef
     online = false
   }
 

package.json

@@ -99,6 +99,6 @@
     "test-browserstack": "karma start test/karma/browserstack.conf.js",
     "test-node": "jasmine test/spec/node-index.js",
     "test-types": "tsd",
-    "lint": "eslint ."
+    "lint": "eslint . --cache"
   }
 }

test/karma/base.conf.js

@@ -1,5 +1,7 @@
 // Karma base configuration
 
+'use strict'
+
 module.exports = (config) => {
   config.set({
     // base path that will be used to resolve all patterns (eg. files, exclude)

test/karma/browserstack.conf.js

@@ -1,3 +1,5 @@
+'use strict'
+
 // Karma configuration for testing using BrowserStack
 const baseConfig = require('./base.conf')
 

test/karma/puppeteer.conf.js

@@ -1,3 +1,5 @@
+'use strict'
+
 // Karma configuration for testing using Puppeteer
 const baseConfig = require('./base.conf')
 

test/spec/browser-index.js

@@ -1,7 +1,9 @@
+'use strict'
+
 // The regenerator runtime is needed since the test use functions
 // with the async/await keywords. See
 // https://babeljs.io/docs/en/babel-plugin-transform-regenerator
-import 'regenerator-runtime/runtime'
+require('regenerator-runtime/runtime')
 
 beforeEach(() => {
   // Clear localStorage before every test to prevent stored URLs to

test/spec/helpers/assertUrlStorage.js

@@ -1,3 +1,5 @@
+'use strict'
+
 module.exports = async function assertUrlStorage (urlStorage) {
   // In the beginning of the test, the storage should be empty.
   let result = await urlStorage.findAllUploads()

test/spec/helpers/utils.js

@@ -1,4 +1,7 @@
 /* eslint-disable max-classes-per-file */
+
+'use strict'
+
 const isBrowser  = typeof window !== 'undefined'
 const isNode     = !isBrowser
 

test/spec/node-index.js

@@ -1,3 +1,5 @@
+'use strict'
+
 require('./test-common')
 require('./test-node-specific')
 require('./test-parallel-uploads')

test/spec/test-browser-specific.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const assertUrlStorage = require('./helpers/assertUrlStorage')
 const { TestHttpStack, waitableFunction, wait } = require('./helpers/utils')
 const tus = require('../..')

test/spec/test-common.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const { TestHttpStack, waitableFunction, wait, getBlob } = require('./helpers/utils')
 const tus = require('../..')
 

test/spec/test-end-to-end.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const axios = require('axios')
 const { getBlob } = require('./helpers/utils')
 const tus = require('../..')

test/spec/test-node-specific.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const stream = require('stream')
 const temp = require('temp')
 const fs = require('fs')

test/spec/test-parallel-uploads.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const { TestHttpStack, waitableFunction, wait, getBlob } = require('./helpers/utils')
 const tus = require('../..')
 

test/spec/test-terminate.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const { TestHttpStack, getBlob } = require('./helpers/utils')
 const tus = require('../..')