Update more dependencies (#687)

* Update more dependencies

Bumps almost everything up, removing almost all warnings from yarn audit. Doesn't touch cheerio or jest, as they require more attention and QA still.

* Adjust more dependencies, tweak build files

Author: johnholdun

.eslintrc

@@ -24,7 +24,17 @@
       "error",
       { "max": 1, "maxEOF": 0, "maxBOF": 0 }
     ],
-    "import/no-unresolved": false
+    "import/no-unresolved": false,
+    "import/no-extraneous-dependencies": [
+      "error",
+      {
+        "devDependencies": [
+          "**/*.test.js",
+          "scripts/proxy-browser-test.js",
+          "rollup.config*js"
+        ]
+      }
+    ]
   },
   "settings": {
     "import/resolver": {

package.json

@@ -101,6 +101,7 @@
     "remark-cli": "^7.0.0",
     "remark-lint": "^6.0.4",
     "remark-preset-lint-recommended": "^3.0.2",
+    "request": "^2.88.2",
     "requirejs": "^2.3.6",
     "rollup": "^1.1.0",
     "rollup-plugin-babel": "^4.0.1",
@@ -122,12 +123,10 @@
     "jquery": "^3.5.0",
     "moment": "^2.23.0",
     "moment-parseformat": "3.0.0",
-    "moment-timezone": "0.5.26",
-    "postman-request": "^2.88.1-postman.7.1",
-    "request-promise": "^4.2.2",
+    "moment-timezone": "0.5.37",
+    "postman-request": "^2.88.1-postman.31",
     "string-direction": "^0.1.2",
-    "turndown": "^5.0.3",
-    "url": "^0.11.0",
+    "turndown": "^7.1.1",
     "valid-url": "^1.0.9",
     "wuzzy": "^0.1.4",
     "yargs-parser": "^15.0.1"

scripts/proxy-browser-test.js

@@ -1,9 +1,8 @@
-/* eslint-disable */
-const express = require('express'); // eslint-disable-line import/no-extraneous-dependencies
+const express = require('express');
 const request = require('request');
 
 const app = express();
-var server;
+let server;
 
 const start = () => {
   app.use('/:url', (req, res) => {
@@ -23,24 +22,28 @@ const start = () => {
     };
     req.pipe(request(options)).pipe(res);
   });
-
   server = app.listen(process.env.PORT || 3000);
 };
 
 const stop = () => {
-  server && server.close();
+  if (server) {
+    server.close();
+  }
 };
 
 start();
 require('child_process').exec(
-  'node ./node_modules/karma/bin/karma start ./scripts/karma.conf.js' +
-    (process.env.CI ? ' --CI' : ''),
+  `node ./node_modules/karma/bin/karma start ./scripts/karma.conf.js${
+    process.env.CI ? ' --CI' : ''
+  }`,
   { stdio: [0, 1, 2] },
   (err, stdout) => {
     if (err) {
+      // eslint-disable-next-line no-console
       console.log('stdout', stdout);
       process.exit(1);
     } else {
+      // eslint-disable-next-line no-console
       console.log('stdout', stdout);
     }
     stop();