refactor: migrate platform usage to ScanConfig and enhance logging

Replace args.platform with config.platform in container validation.
Enhance config logging to show more field values.
Add comprehensive multi-field test coverage.

Completes core scanning field migration within invoke_sbomgen scope.

Author: bluesentinelsec

entrypoint/entrypoint/orchestrator.py

@@ -16,7 +16,7 @@ def execute(args) -> int:
     # NEW: Create structured config from legacy args (strangler fig pattern)
     from entrypoint.data_model import ScanConfig, ArtifactType
     config = ScanConfig.from_args(args)
-    logging.info(f"Created config: artifact_type={config.artifact_type.value}, artifact_path={config.artifact_path}")
+    logging.info(f"Created config: artifact_type={config.artifact_type.value}, artifact_path={config.artifact_path}, sbomgen_version={config.sbomgen_version}, timeout={config.timeout}s")
 
     # OLD: Keep existing logic unchanged for now
     logging.info(f"downloading and installing inspector-sbomgen version {config.sbomgen_version}")
@@ -209,11 +209,11 @@ def invoke_sbomgen(args) -> int:
 
     if config.artifact_type == ArtifactType.CONTAINER:
 
-        if args.platform:
-            platform_arg = args.platform.lower()
+        if config.platform:
+            platform_arg = config.platform.lower()
             if not is_valid_container_platform(platform_arg):
                 logging.fatal(
-                    f"received invalid container image platform: '{args.platform}'. Platform should be of the form 'os/cpu/variant' such as 'linux/amd64' or 'linux/arm64/v8'")
+                    f"received invalid container image platform: '{config.platform}'. Platform should be of the form 'os/cpu/variant' such as 'linux/amd64' or 'linux/arm64/v8'")
             sbomgen_args.append("--platform")
             sbomgen_args.append(platform_arg)
 

entrypoint/tests/test_data_model.py

@@ -178,3 +178,21 @@ def test_scan_config_skip_scanners_list_access(self):
     def test_scan_config_skip_files_list_access(self):
         config = ScanConfig(skip_files=["./media", "/tmp/foo"])
         self.assertEqual(config.skip_files, ["./media", "/tmp/foo"])
+
+    def test_scan_config_platform_access(self):
+        config = ScanConfig(platform="linux/amd64")
+        self.assertEqual(config.platform, "linux/amd64")
+
+    def test_scan_config_comprehensive_access(self):
+        config = ScanConfig(
+            artifact_type=ArtifactType.CONTAINER,
+            artifact_path="./test",
+            sbomgen_version="1.8.0",
+            timeout=300,
+            platform="linux/amd64"
+        )
+        self.assertEqual(config.artifact_type, ArtifactType.CONTAINER)
+        self.assertEqual(config.artifact_path, "./test")
+        self.assertEqual(config.sbomgen_version, "1.8.0")
+        self.assertEqual(config.timeout, 300)
+        self.assertEqual(config.platform, "linux/amd64")