Revert "Add guardrail in case manifest file does not exist (#93)" (#94)

This reverts commit c2183b5.

Author: dacoburn

pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "socketsecurity"
-version = "2.1.15"
+version = "2.1.14"
 requires-python = ">= 3.10"
 license = {"file" = "LICENSE"}
 dependencies = [

requirements-dev.txt

@@ -1,2 +1,2 @@
 __author__ = 'socket.dev'
-__version__ = '2.1.15'
+__version__ = '2.1.14'

requirements.txt

@@ -42,7 +42,7 @@ class CliConfig:
     enable_sarif: bool = False
     disable_overview: bool = False
     disable_security_issue: bool = False
-    files: str = None
+    files: str = "[]"
     ignore_commit_files: bool = False
     disable_blocking: bool = False
     integration_type: IntegrationType = "api"

socketsecurity/__init__.py

@@ -860,10 +860,9 @@ def get_source_data(package: Package, packages: dict) -> list:
             manifests = ""
             if not hasattr(package, "manifestFiles"):
                 return introduced_by
-            if hasattr(package, "manifestFiles"):
-                for manifest_data in package.manifestFiles:
-                    manifest_file = manifest_data.get("file")
-                    manifests += f"{manifest_file};"
+            for manifest_data in package.manifestFiles:
+                manifest_file = manifest_data.get("file")
+                manifests += f"{manifest_file};"
             manifests = manifests.rstrip(";")
             source = ("direct", manifests)
             introduced_by.append(source)

socketsecurity/config.py

@@ -130,7 +130,7 @@ def main_code():
     files_to_check = set(json.loads(config.files))  # Start with manually specified files
 
     # Add git changes if this is a repo and we're not ignoring commit files
-    if is_repo and not config.ignore_commit_files and not files_to_check:
+    if is_repo and not config.ignore_commit_files:
         files_to_check.update(git_repo.changed_files)
 
     # Determine if we need to scan based on manifest files
@@ -260,7 +260,7 @@ def main_code():
         output_handler.handle_output(diff)
 
         # Handle license generation
-    if should_skip_scan and diff.id != "no_diff_id" and config.generate_license:
+    if diff is not None and diff.id != "no_diff_id" and config.generate_license:
         all_packages = {}
         for purl in diff.packages:
             package = diff.packages[purl]