npm-pkg-lint 5.1.7 → 5.1.9

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") {
@@ -4693,15 +4697,16 @@ var require_range = __commonJS({
     };
     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`;
@@ -7656,7 +7661,7 @@ var blacklist = [
   filename("gulpfile.esm.js"),
   filename("Herebyfile.js"),
   filename("Herebyfile.mjs"),
-  /* ci */
+  /* CI */
   /^\.azure-.*$/,
   directory(".circleci"),
   directory(".github"),
@@ -8738,7 +8743,7 @@ var bn = (s3, t) => {
   t = t.slice((e + " ").length);
   let i = t.split("="), r = i.shift();
   if (!r) return s3;
-  let n = r.replace(/^SCHILY\.(dev|ino|nlink)/, "$1"), o = i.join("=");
+  let n = r.replace(/^SCHILY\.(dev|ino|nlink)/, "$1"), o = i.join("=").replace(/\0.*/, "");
   return s3[n] = /^([A-Z]+\.)?([mac]|birth|creation)time$/.test(n) ? new Date(Number(o) * 1e3) : /^[0-9]+$/.test(o) ? +o : o, s3;
 };
 var _n = process.env.TESTING_TAR_FAKE_PLATFORM || process.platform;
@@ -10661,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;
@@ -10699,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);
@@ -11473,17 +11480,21 @@ function* validateOrder2(pkgAst, value, path8) {
   const keys = Object.keys(value);
   if (keys.includes("require")) {
     for (const esm of ["import", "module"]) {
-      if (keys.includes(esm) && keys.indexOf(esm) > keys.indexOf("require")) {
-        const { line, column } = jsonLocation(pkgAst, "member", "exports", ...path8, esm);
-        const property = path8.map((it2) => `["${it2}"]`).join("");
-        yield {
-          ruleId: ruleId5,
-          severity: severity3,
-          message: `"${esm}" must come before "require" in "exports${property}"`,
-          line,
-          column
-        };
+      if (!keys.includes(esm)) {
+        continue;
+      }
+      if (keys.indexOf(esm) <= keys.indexOf("require")) {
+        continue;
       }
+      const { line, column } = jsonLocation(pkgAst, "member", "exports", ...path8, esm);
+      const property = path8.map((it2) => `["${it2}"]`).join("");
+      yield {
+        ruleId: ruleId5,
+        severity: severity3,
+        message: `"${esm}" must come before "require" in "exports${property}"`,
+        line,
+        column
+      };
     }
   }
   for (const key of keys) {
@@ -11594,7 +11605,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);
@@ -11720,7 +11731,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;
     }
@@ -11879,7 +11890,8 @@ async function* getDeepDependencies(pkg, visited, dependency) {
   if (!pkgData) {
     return;
   }
-  for (let [key, version2] of Object.entries(pkgData.dependencies ?? {})) {
+  const { dependencies = {} } = pkgData;
+  for (let [key, version2] of Object.entries(dependencies)) {
     if (version2.startsWith("npm:")) {
       const [newKey, newVersion] = version2.slice("npm:".length).split("@", 2);
       key = newKey;
@@ -11951,6 +11963,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";
@@ -11960,7 +11973,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)) {
@@ -12088,7 +12101,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)`);
@@ -12317,7 +12331,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({
@@ -12477,9 +12491,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);