@octokit/request-error - npm Package Compare versions

Comparing version 5.0.1 to 6.0.0-beta.1

dist-node/index.js

@@ -1,41 +0,16 @@
-"use strict";
-var __create = Object.create;
-var __defProp = Object.defineProperty;
-var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __export = (target, all) => {
-  for (var name in all)
-    __defProp(target, name, { get: all[name], enumerable: true });
-};
-var __copyProps = (to, from, except, desc) => {
-  if (from && typeof from === "object" || typeof from === "function") {
-    for (let key of __getOwnPropNames(from))
-      if (!__hasOwnProp.call(to, key) && key !== except)
-        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
-  }
-  return to;
-};
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
-  // If the importer is in node compatibility mode or this is not an ESM
-  // file that has been converted to a CommonJS file using a Babel-
-  // compatible transform (i.e. "__esModule" has not been set), then set
-  // "default" to the CommonJS "module.exports" for node compatibility.
-  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
-  mod
-));
-var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
-
 // pkg/dist-src/index.js
-var dist_src_exports = {};
-__export(dist_src_exports, {
-  RequestError: () => RequestError
-});
-module.exports = __toCommonJS(dist_src_exports);
-var import_deprecation = require("deprecation");
-var import_once = __toESM(require("once"));
-var logOnceCode = (0, import_once.default)((deprecation) => console.warn(deprecation));
-var logOnceHeaders = (0, import_once.default)((deprecation) => console.warn(deprecation));
 var RequestError = class extends Error {
+  name;
+  /**
+   * http status code
+   */
+  status;
+  /**
+   * Request options that lead to the error.
+   */
+  request;
+  /**
+   * Response object if a response was received
+   */
+  response;
   constructor(message, statusCode, options) {
@@ -48,9 +23,4 @@ super(message);
     this.status = statusCode;
-    let headers;
-    if ("headers" in options && typeof options.headers !== "undefined") {
-      headers = options.headers;
-    }
     if ("response" in options) {
       this.response = options.response;
-      headers = options.response.headers;
     }
@@ -68,27 +38,6 @@ const requestCopy = Object.assign({}, options.request);
     this.request = requestCopy;
-    Object.defineProperty(this, "code", {
-      get() {
-        logOnceCode(
-          new import_deprecation.Deprecation(
-            "[@octokit/request-error] `error.code` is deprecated, use `error.status`."
-          )
-        );
-        return statusCode;
-      }
-    });
-    Object.defineProperty(this, "headers", {
-      get() {
-        logOnceHeaders(
-          new import_deprecation.Deprecation(
-            "[@octokit/request-error] `error.headers` is deprecated, use `error.response.headers`."
-          )
-        );
-        return headers || {};
-      }
-    });
   }
 };
-// Annotate the CommonJS export names for ESM import in node:
-0 && (module.exports = {
+export {
   RequestError
-});
+};

dist-src/index.js

@@ -1,6 +0,15 @@
-import { Deprecation } from "deprecation";
-import once from "once";
-const logOnceCode = once((deprecation) => console.warn(deprecation));
-const logOnceHeaders = once((deprecation) => console.warn(deprecation));
 class RequestError extends Error {
+  name;
+  /**
+   * http status code
+   */
+  status;
+  /**
+   * Request options that lead to the error.
+   */
+  request;
+  /**
+   * Response object if a response was received
+   */
+  response;
   constructor(message, statusCode, options) {
@@ -13,9 +22,4 @@ super(message);
     this.status = statusCode;
-    let headers;
-    if ("headers" in options && typeof options.headers !== "undefined") {
-      headers = options.headers;
-    }
     if ("response" in options) {
       this.response = options.response;
-      headers = options.response.headers;
     }
@@ -33,22 +37,2 @@ const requestCopy = Object.assign({}, options.request);
     this.request = requestCopy;
-    Object.defineProperty(this, "code", {
-      get() {
-        logOnceCode(
-          new Deprecation(
-            "[@octokit/request-error] `error.code` is deprecated, use `error.status`."
-          )
-        );
-        return statusCode;
-      }
-    });
-    Object.defineProperty(this, "headers", {
-      get() {
-        logOnceHeaders(
-          new Deprecation(
-            "[@octokit/request-error] `error.headers` is deprecated, use `error.response.headers`."
-          )
-        );
-        return headers || {};
-      }
-    });
   }
@@ -55,0 +39,0 @@ }

dist-types/index.d.ts

@@ -1,3 +0,3 @@
-import type { RequestOptions, ResponseHeaders, OctokitResponse } from "@octokit/types";
-import type { RequestErrorOptions } from "./types";
+import type { RequestOptions, OctokitResponse } from "@octokit/types";
+import type { RequestErrorOptions } from "./types.js";
 /**
@@ -13,8 +13,2 @@ * Error with extra properties to help with debugging
     /**
-     * http status code
-     *
-     * @deprecated `error.code` is deprecated in favor of `error.status`
-     */
-    code: number;
-    /**
      * Request options that lead to the error.
@@ -24,8 +18,2 @@ */
     /**
-     * error response headers
-     *
-     * @deprecated `error.headers` is deprecated in favor of `error.response.headers`
-     */
-    headers: ResponseHeaders;
-    /**
      * Response object if a response was received
@@ -32,0 +20,0 @@ */

dist-web/index.js

 // pkg/dist-src/index.js
-import { Deprecation } from "deprecation";
-import once from "once";
-var logOnceCode = once((deprecation) => console.warn(deprecation));
-var logOnceHeaders = once((deprecation) => console.warn(deprecation));
 var RequestError = class extends Error {
+  name;
+  /**
+   * http status code
+   */
+  status;
+  /**
+   * Request options that lead to the error.
+   */
+  request;
+  /**
+   * Response object if a response was received
+   */
+  response;
   constructor(message, statusCode, options) {
@@ -14,9 +23,4 @@ super(message);
     this.status = statusCode;
-    let headers;
-    if ("headers" in options && typeof options.headers !== "undefined") {
-      headers = options.headers;
-    }
     if ("response" in options) {
       this.response = options.response;
-      headers = options.response.headers;
     }
@@ -34,22 +38,2 @@ const requestCopy = Object.assign({}, options.request);
     this.request = requestCopy;
-    Object.defineProperty(this, "code", {
-      get() {
-        logOnceCode(
-          new Deprecation(
-            "[@octokit/request-error] `error.code` is deprecated, use `error.status`."
-          )
-        );
-        return statusCode;
-      }
-    });
-    Object.defineProperty(this, "headers", {
-      get() {
-        logOnceHeaders(
-          new Deprecation(
-            "[@octokit/request-error] `error.headers` is deprecated, use `error.response.headers`."
-          )
-        );
-        return headers || {};
-      }
-    });
   }
@@ -56,0 +40,0 @@ };

package.json

 {
   "name": "@octokit/request-error",
-  "version": "5.0.1",
+  "version": "6.0.0-beta.1",
   "publishConfig": {
     "access": "public"
   },
+  "type": "module",
   "description": "Error class for Octokit request errors",
@@ -23,10 +24,10 @@ "repository": "github:octokit/request-error.js",
   "devDependencies": {
-    "@octokit/tsconfig": "^2.0.0",
+    "@octokit/tsconfig": "^3.0.0",
     "@types/jest": "^29.0.0",
-    "@types/node": "^18.0.0",
+    "@types/node": "^20.0.0",
     "@types/once": "^1.4.0",
-    "esbuild": "^0.19.0",
+    "esbuild": "^0.20.0",
     "glob": "^10.2.6",
     "jest": "^29.0.0",
-    "prettier": "3.0.3",
+    "prettier": "3.2.5",
     "ts-jest": "^29.0.0",
@@ -42,8 +43,16 @@ "typescript": "^5.0.0"
   ],
-  "main": "dist-node/index.js",
-  "browser": "dist-web/index.js",
-  "types": "dist-types/index.d.ts",
-  "module": "dist-src/index.js",
+  "exports": {
+    ".": {
+      "node": {
+        "types": "./dist-types/index.d.ts",
+        "import": "./dist-node/index.js"
+      },
+      "browser": {
+        "types": "./dist-types/index.d.ts",
+        "import": "./dist-web/index.js"
+      }
+    }
+  },
   "sideEffects": false,
   "unpkg": "dist-web/index.js"
 }

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

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

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

Is type module

Yes

Worsened metrics

Total package byte prevSize

12062

decreased by-34.98%

Lines of code

152

decreased by-38.21%

Number of low quality alerts

2

increased by100%

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes