npm-pkg-lint 5.1.6 → 5.1.8

package/dist/index.js

@@ -15,7 +15,11 @@ var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require
   throw Error('Dynamic require of "' + x + '" is not supported');
 });
 var __commonJS = (cb, mod) => function __require2() {
-  return mod || (0, cb[__getOwnPropNames(cb)[0]])((mod = { exports: {} }).exports, mod), mod.exports;
+  try {
+    return mod || (0, cb[__getOwnPropNames(cb)[0]])((mod = { exports: {} }).exports, mod), mod.exports;
+  } catch (e) {
+    throw mod = 0, e;
+  }
 };
 var __copyProps = (to2, from, except, desc) => {
   if (from && typeof from === "object" || typeof from === "function") {
@@ -4687,20 +4691,22 @@ var require_range = __commonJS({
       return comp;
     };
     var isX = (id) => !id || id.toLowerCase() === "x" || id === "*";
+    var invalidXRangeOrder = (M2, m2, p2) => isX(M2) && !isX(m2) || isX(m2) && p2 && !isX(p2);
     var replaceTildes = (comp, options) => {
       return comp.trim().split(/\s+/).map((c) => replaceTilde(c, options)).join(" ");
     };
     var replaceTilde = (comp, options) => {
       const r = options.loose ? re2[t.TILDELOOSE] : re2[t.TILDE];
+      const z2 = options.includePrerelease ? "-0" : "";
       return comp.replace(r, (_2, M2, m2, p2, pr2) => {
         debug("tilde", comp, _2, M2, m2, p2, pr2);
         let ret;
         if (isX(M2)) {
           ret = "";
         } else if (isX(m2)) {
-          ret = `>=${M2}.0.0 <${+M2 + 1}.0.0-0`;
+          ret = `>=${M2}.0.0${z2} <${+M2 + 1}.0.0-0`;
         } else if (isX(p2)) {
-          ret = `>=${M2}.${m2}.0 <${M2}.${+m2 + 1}.0-0`;
+          ret = `>=${M2}.${m2}.0${z2} <${M2}.${+m2 + 1}.0-0`;
         } else if (pr2) {
           debug("replaceTilde pr", pr2);
           ret = `>=${M2}.${m2}.${p2}-${pr2} <${M2}.${+m2 + 1}.0-0`;
@@ -4767,6 +4773,9 @@ var require_range = __commonJS({
       const r = options.loose ? re2[t.XRANGELOOSE] : re2[t.XRANGE];
       return comp.replace(r, (ret, gtlt, M2, m2, p2, pr2) => {
         debug("xRange", comp, ret, gtlt, M2, m2, p2, pr2);
+        if (invalidXRangeOrder(M2, m2, p2)) {
+          return comp;
+        }
         const xM = isX(M2);
         const xm = xM || isX(m2);
         const xp = xm || isX(p2);
@@ -7652,7 +7661,7 @@ var blacklist = [
   filename("gulpfile.esm.js"),
   filename("Herebyfile.js"),
   filename("Herebyfile.mjs"),
-  /* ci */
+  /* CI */
   /^\.azure-.*$/,
   directory(".circleci"),
   directory(".github"),
@@ -10657,7 +10666,9 @@ async function getFileList(filename2) {
   await Ft({
     file: filename2,
     strict: true,
-    onentry: (entry) => entries.push(entry)
+    onReadEntry(entry) {
+      entries.push(entry);
+    }
   });
   return entries.map((entry) => {
     const filename3 = entry.path;
@@ -10695,11 +10706,11 @@ function blacklistedFiles(filelist) {
 function normalizeRequiredFiles(src) {
   if (typeof src === "string") {
     return [src];
-  } else if (Array.isArray(src)) {
+  }
+  if (Array.isArray(src)) {
     return src;
-  } else {
-    return Object.values(src).filter((it2) => it2 !== false);
   }
+  return Object.values(src).filter((it2) => it2 !== false);
 }
 function* yieldRequiredFiles(src, template) {
   const files = normalizeRequiredFiles(src);
@@ -11590,7 +11601,7 @@ function getProtocol(version2, field) {
   }
   return null;
 }
-var githubShorthandRe = /^[A-Za-z][\w-]*\/[\w.-]+(?:#.+)?$/;
+var githubShorthandRe = /^[A-Z][\w-]*\/[\w.-]+(?:#.+)?$/i;
 var gitUrlRe = /^[^@]+@[^:]+:.+/;
 function isGithubShorthand(version2) {
   return !version2.includes(":") && githubShorthandRe.test(version2);
@@ -11716,7 +11727,7 @@ function* outdatedEngines(pkg, pkgAst, ignoreNodeVersion) {
     if (!descriptor.eol) {
       break;
     }
-    const expanded = version2.replaceAll(/[*Xx]/g, "999");
+    const expanded = version2.replaceAll(/[*X]/gi, "999");
     if (!import_semver2.default.satisfies(expanded, range)) {
       continue;
     }
@@ -11947,6 +11958,7 @@ async function verifyEngineConstraint(pkg) {
 // src/validators/validation-error.ts
 var ValidationError = class extends Error {
   validator;
+  /* eslint-disable-next-line unicorn/custom-error-definition -- technical debt */
   constructor(validator, message) {
     super(message);
     this.name = "ValidationError";
@@ -11956,7 +11968,7 @@ var ValidationError = class extends Error {
 
 // src/validators/non-empty.ts
 function isEmpty(value) {
-  return value === void 0 || value === null || value === "";
+  return [void 0, null, ""].includes(value);
 }
 function nonempty(key, value) {
   if (isEmpty(value)) {
@@ -12084,7 +12096,8 @@ function validRepoUrl(key, value) {
 function validUrl(key, value) {
   if (typeof value === "string" && /^https:\/\/.+$/.test(value)) {
     return;
-  } else if (value && typeof value === "object" && "url" in value) {
+  }
+  if (value && typeof value === "object" && "url" in value) {
     validUrl(`${key}.url`, value.url);
   } else {
     throw new ValidationError(validUrl.name, `"${key}" must be a valid url (https only)`);
@@ -12313,7 +12326,7 @@ async function verifyShebang(pkg, tarball) {
   const contents = await getFileContent(tarball, binaries);
   for (const [filePath, content] of Object.entries(contents)) {
     const text = content.toString("utf-8");
-    if (/^#!.+?\r?\n/.test(text)) {
+    if (/^#!.+\r?\n/.test(text)) {
       continue;
     }
     results.push({
@@ -12473,9 +12486,8 @@ async function run() {
     result.messages.sort((a, b2) => {
       if (a.line !== b2.line) {
         return a.line - b2.line;
-      } else {
-        return a.column - b2.column;
       }
+      return a.column - b2.column;
     });
   }
   const output = stylish(results);