chore: revert out of scope files

Author: parker-snyk

lib/analyzer/image-inspector.ts

@@ -48,6 +48,7 @@ async function pullWithDockerBinary(
   password: string | undefined,
   platform: string | undefined,
 ): Promise<boolean> {
+  let pullAndSaveSuccessful = false;
   try {
     if (username || password) {
       debug(
@@ -56,12 +57,13 @@ async function pullWithDockerBinary(
     }
     await docker.pullCli(targetImage, { platform });
     await docker.save(targetImage, saveLocation);
-    return true;
+    return (pullAndSaveSuccessful = true);
   } catch (err) {
     debug(`couldn't pull ${targetImage} using docker binary: ${err.message}`);
+
     handleDockerPullError(err.stderr, platform);
 
-    return false;
+    return pullAndSaveSuccessful;
   }
 }
 
@@ -79,9 +81,10 @@ function handleDockerPullError(err: string, platform?: string) {
     "manifest unknown",
   ];
   if (unknownManifestConditions.some((value) => err.includes(value))) {
-    throw new Error(
-      `The image does not exist for ${platform ?? "the current platform"}`,
-    );
+    if (platform) {
+      throw new Error(`The image does not exist for ${platform}`);
+    }
+    throw new Error(`The image does not exist for the current platform`);
   }
 
   if (err.includes("invalid reference format")) {

lib/inputs/file-pattern/static.ts

@@ -5,20 +5,25 @@ import { ExtractAction, ExtractedLayers } from "../../extractor/types";
 import { streamToString } from "../../stream-utils";
 import { ManifestFile } from "../../types";
 
-/**
- * Return false if any exclusion pattern matches,
- * return true if any inclusion pattern matches
- */
 function generatePathMatcher(
   globsInclude: string[],
   globsExclude: string[],
 ): (filePath: string) => boolean {
   return (filePath: string): boolean => {
-    if (globsExclude.some((glob) => minimatch(filePath, glob))) {
-      return false;
+    let exclude = false;
+    for (const g of globsExclude) {
+      if (!exclude && minimatch(filePath, g)) {
+        exclude = true;
+      }
     }
-
-    return globsInclude.some((glob) => minimatch(filePath, glob));
+    if (!exclude) {
+      for (const g of globsInclude) {
+        if (minimatch(filePath, g)) {
+          return true;
+        }
+      }
+    }
+    return false;
   };
 }
 

lib/python-parser/requirements-parser.ts

@@ -15,18 +15,18 @@ export function getRequirements(fileContent: string): PythonRequirement[] {
 
 function parseLine(line: string): PythonRequirement | null {
   line = line.trim();
-  // there's no point in calling the regex if the line is a comment
-  if (line.length === 0 || line.startsWith("#")) {
+  if (line.length === 0) {
     return null;
   }
   const parsedLine = VERSION_PARSE_REGEX.exec(line);
   if (!parsedLine?.groups) {
     return null;
   }
   const { name, extras, specifier, version } = parsedLine.groups;
+  const correctedSpecifier = specifierValidRange(specifier, version);
   return {
     name: name.toLowerCase(),
-    specifier: specifierValidRange(specifier, version),
+    specifier: correctedSpecifier,
     version,
     extras: parseExtraNames(extras),
   } as PythonRequirement;