@appium/support - npm Package Compare versions

Comparing version 4.2.6 to 4.3.0

build/lib/logging.d.ts

@@ -22,5 +22,5 @@ /** @type {import('@appium/types').AppiumLoggerLevel[]} */
  *
- * @param {import('npmlog').Logger} logger
+ * @param {import('@appium/logger').Logger} logger
  */
-export function patchLogger(logger: import('npmlog').Logger): void;
+export function patchLogger(logger: import('@appium/logger').Logger): void;
 /**
@@ -27,0 +27,0 @@ *

build/lib/logging.js

@@ -8,3 +8,3 @@ "use strict";
 exports.loadSecureValuesPreprocessingRules = exports.getLogger = exports.patchLogger = exports.log = exports.LEVELS = void 0;
-const npmlog_1 = __importDefault(require("npmlog"));
+const logger_1 = __importDefault(require("@appium/logger"));
 const lodash_1 = __importDefault(require("lodash"));
@@ -25,3 +25,3 @@ const util_1 = require("./util");
  *
- * @param {import('npmlog').Logger} logger
+ * @param {import('@appium/logger').Logger} logger
  */
@@ -36,3 +36,3 @@ function patchLogger(logger) {
  *
- * @returns {[import('npmlog').Logger, boolean]}
+ * @returns {[import('@appium/logger').Logger, boolean]}
  */
@@ -53,3 +53,3 @@ function _getLogger() {
         // otherwise, either use the global, or a new `npmlog` object
-        logger = global._global_npmlog || npmlog_1.default;
+        logger = global._global_npmlog || logger_1.default;
         // The default value is 10000, which causes excessive memory usage
@@ -56,0 +56,0 @@ logger.maxRecordSize = MAX_LOG_RECORDS_COUNT;

build/lib/mjpeg.js

@@ -156,8 +156,11 @@ "use strict";
     stop() {
-        if (!this.consumer) {
-            return;
+        if (this.consumer) {
+            this.consumer.unpipe(this);
         }
-        this.responseStream.unpipe(this.consumer);
-        this.consumer.unpipe(this);
-        this.responseStream.destroy();
+        if (this.responseStream) {
+            if (this.consumer) {
+                this.responseStream.unpipe(this.consumer);
+            }
+            this.responseStream.destroy();
+        }
         this.clear();
@@ -164,0 +167,0 @@ }

lib/logging.js

 // @ts-check
 
-import npmlog from 'npmlog';
+import globalLog from '@appium/logger';
 import _ from 'lodash';
@@ -23,3 +23,3 @@ import {unleakString} from './util';
  *
- * @param {import('npmlog').Logger} logger
+ * @param {import('@appium/logger').Logger} logger
  */
@@ -34,3 +34,3 @@ function patchLogger(logger) {
  *
- * @returns {[import('npmlog').Logger, boolean]}
+ * @returns {[import('@appium/logger').Logger, boolean]}
  */
@@ -51,3 +51,3 @@ function _getLogger() {
     // otherwise, either use the global, or a new `npmlog` object
-    logger = global._global_npmlog || npmlog;
+    logger = global._global_npmlog || globalLog;
     // The default value is 10000, which causes excessive memory usage
@@ -54,0 +54,0 @@ logger.maxRecordSize = MAX_LOG_RECORDS_COUNT;

lib/mjpeg.js

@@ -175,9 +175,11 @@ import _ from 'lodash';
   stop() {
-    if (!this.consumer) {
-      return;
+    if (this.consumer) {
+      this.consumer.unpipe(this);
     }
-
-    this.responseStream.unpipe(this.consumer);
-    this.consumer.unpipe(this);
-    this.responseStream.destroy();
+    if (this.responseStream) {
+      if (this.consumer) {
+        this.responseStream.unpipe(this.consumer);
+      }
+      this.responseStream.destroy();
+    }
     this.clear();
@@ -184,0 +186,0 @@ }

package.json

 {
   "name": "@appium/support",
-  "version": "4.2.6",
+  "version": "4.3.0",
   "description": "Support libs used across appium packages",
@@ -45,3 +45,3 @@ "keywords": [
     "@appium/tsconfig": "^0.3.3",
-    "@appium/types": "^0.18.0",
+    "@appium/types": "^0.19.0",
     "@colors/colors": "1.6.0",
@@ -56,3 +56,2 @@ "@types/archiver": "6.0.2",
     "@types/ncp": "2.0.8",
-    "@types/npmlog": "7.0.0",
     "@types/pluralize": "0.0.33",
@@ -64,3 +63,3 @@ "@types/semver": "7.5.8",
     "@types/uuid": "9.0.8",
-    "@types/which": "3.0.3",
+    "@types/which": "3.0.4",
     "archiver": "7.0.1",
@@ -83,3 +82,2 @@ "axios": "1.7.2",
     "ncp": "2.0.0",
-    "npmlog": "7.0.1",
     "opencv-bindings": "4.5.5",
@@ -96,4 +94,4 @@ "pkg-dir": "5.0.0",
     "supports-color": "8.1.1",
-    "teen_process": "2.1.3",
-    "type-fest": "4.18.3",
+    "teen_process": "2.1.4",
+    "type-fest": "4.19.0",
     "uuid": "9.0.1",
@@ -113,3 +111,3 @@ "which": "4.0.0",
   },
-  "gitHead": "a4138c6ec9524594519f29938008500b7cdcc0b8"
+  "gitHead": "339acd345fafa1292c8d286261553fcc6b440e46"
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

433921

increased by0.06%

Dependency count

52

decreased by-3.7%

Lines of code

8838

increased by0.07%

Dependency changes

• + Added@appium/logger@1.6.1(transitive)

• + Added@appium/types@0.19.2(transitive)

• + Added@types/which@3.0.4(transitive)

• + Addedis-core-module@2.16.0(transitive)

• + Addedresolve@1.22.9(transitive)

• + Addedteen_process@2.1.4(transitive)

• + Addedtype-fest@4.19.0(transitive)

• - Removed@types/npmlog@7.0.0

• - Removednpmlog@7.0.1

• - Removed@appium/types@0.18.0(transitive)

• - Removed@types/npmlog@7.0.0(transitive)

• - Removed@types/which@3.0.3(transitive)

• - Removedaproba@2.0.0(transitive)

• - Removedare-we-there-yet@4.0.2(transitive)

• - Removedcolor-support@1.1.3(transitive)

• - Removedgauge@5.0.2(transitive)

• - Removedhas-unicode@2.0.1(transitive)

• - Removedis-core-module@2.16.1(transitive)

• - Removednpmlog@7.0.1(transitive)

• - Removedresolve@1.22.10(transitive)

• - Removedteen_process@2.1.3(transitive)

• - Removedtype-fest@4.18.3(transitive)

• - Removedwide-align@1.1.5(transitive)

• Updated@appium/types@^0.19.0

• Updated@types/which@3.0.4

• Updatedteen_process@2.1.4

• Updatedtype-fest@4.19.0