adding formatting

Author: Saumya-R

scripts/workflow/checkout_repos.py

@@ -40,7 +40,7 @@ def load_repos_config(config_file="./repos.json"):
         sys.exit(1)
 
     try:
-        with open(config_path, 'r') as f:
+        with open(config_path, "r") as f:
             repos = json.load(f)
         return repos
     except (json.JSONDecodeError, IOError) as e:
@@ -58,7 +58,7 @@ def is_commit_hash(ref):
     Returns:
         True if ref matches commit hash pattern
     """
-    return bool(re.match(r'^[0-9a-fA-F]{40}$', ref))
+    return bool(re.match(r"^[0-9a-fA-F]{40}$", ref))
 
 
 def checkout_repo(name, url, ref, path):
@@ -85,29 +85,19 @@ def checkout_repo(name, url, ref, path):
             print(f"  Detected commit hash. Cloning and then checking out.")
 
             # Clone the repository
-            subprocess.run(
-                ["git", "clone", url, path],
-                check=True,
-                capture_output=True
-            )
+            subprocess.run(["git", "clone", url, path], check=True, capture_output=True)
 
             # Checkout specific commit
-            subprocess.run(
-                ["git", "-C", path, "checkout", ref],
-                check=True,
-                capture_output=True
-            )
+            subprocess.run(["git", "-C", path, "checkout", ref], check=True, capture_output=True)
         else:
             print(f"Checking out {name} ({ref}) to {path}")
             print(f"  Detected branch/tag. Cloning with --branch.")
 
             # Clone with shallow copy and specific branch/tag
             # Add 'v' prefix if not already present (common convention)
-            branch_ref = ref if ref.startswith('v') else f'v{ref}'
+            branch_ref = ref if ref.startswith("v") else f"v{ref}"
             subprocess.run(
-                ["git", "clone", "--depth", "1", "--branch", branch_ref, url, path],
-                check=True,
-                capture_output=True
+                ["git", "clone", "--depth", "1", "--branch", branch_ref, url, path], check=True, capture_output=True
             )
 
         return True
@@ -120,18 +110,18 @@ def checkout_repo(name, url, ref, path):
 def main():
     """Main entry point."""
     # Load repository configurations
-    repos = load_repos_config('./repos.json')
+    repos = load_repos_config("./repos.json")
     repo_count = len(repos)
 
     # Track successfully checked out repositories
     repo_paths = []
 
     # Checkout each repository
     for i, repo in enumerate(repos):
-        name = repo.get('name', f'repo-{i}')
-        url = repo.get('url', '')
-        ref = repo.get('version', '')
-        path = repo.get('path', '')
+        name = repo.get("name", f"repo-{i}")
+        url = repo.get("url", "")
+        ref = repo.get("version", "")
+        path = repo.get("path", "")
 
         if not all([url, ref, path]):
             print(f"Warning: Skipping {name} - missing required fields", file=sys.stderr)
@@ -141,13 +131,13 @@ def main():
             repo_paths.append(path)
 
     # Output all paths (comma-separated for GitHub Actions compatibility)
-    repo_paths_output = ','.join(repo_paths)
+    repo_paths_output = ",".join(repo_paths)
 
     # Write to GITHUB_OUTPUT if available
-    github_output = os.environ.get('GITHUB_OUTPUT')
+    github_output = os.environ.get("GITHUB_OUTPUT")
     if github_output:
         try:
-            with open(github_output, 'a') as f:
+            with open(github_output, "a") as f:
                 f.write(f"repo_paths={repo_paths_output}\n")
         except IOError as e:
             print(f"Warning: Failed to write GITHUB_OUTPUT: {e}", file=sys.stderr)
@@ -159,5 +149,5 @@ def main():
     return 0 if len(repo_paths) == repo_count else 1
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     sys.exit(main())

scripts/workflow/parse_repos.py

@@ -16,6 +16,7 @@
 """
 
 import json
+import os
 import sys
 import subprocess
 from pathlib import Path
@@ -24,16 +25,8 @@
 def install_dependencies():
     """Ensure jq is installed (for reference, though we use Python's json)."""
     try:
-        subprocess.run(
-            ["sudo", "apt-get", "update"],
-            check=True,
-            capture_output=True
-        )
-        subprocess.run(
-            ["sudo", "apt-get", "install", "-y", "jq"],
-            check=True,
-            capture_output=True
-        )
+        subprocess.run(["sudo", "apt-get", "update"], check=True, capture_output=True)
+        subprocess.run(["sudo", "apt-get", "install", "-y", "jq"], check=True, capture_output=True)
     except subprocess.CalledProcessError as e:
         print(f"Warning: Failed to install jq: {e}", file=sys.stderr)
 
@@ -56,44 +49,39 @@ def parse_known_good(json_file="./known_good.json"):
         sys.exit(1)
 
     try:
-        with open(json_path, 'r') as f:
+        with open(json_path, "r") as f:
             data = json.load(f)
     except json.JSONDecodeError as e:
         print(f"Error: Failed to parse JSON: {e}", file=sys.stderr)
         sys.exit(1)
 
     # Extract target_sw modules
-    modules = data.get('modules', {}).get('target_sw', {})
+    modules = data.get("modules", {}).get("target_sw", {})
 
     # Transform modules into repository objects
     repos = []
     module_outputs = {}
 
     for name, config in modules.items():
-        repo_url = config.get('repo', '')
-        version = config.get('version', '')
-        branch = config.get('branch', '')
-        hash_val = config.get('hash', '')
+        repo_url = config.get("repo", "")
+        version = config.get("version", "")
+        branch = config.get("branch", "")
+        hash_val = config.get("hash", "")
 
         # Use version, branch, or hash (in that order of preference)
         ref = version or branch or hash_val
 
-        repo_obj = {
-            'name': name,
-            'url': repo_url,
-            'version': ref,
-            'path': f'repos/{name}'
-        }
+        repo_obj = {"name": name, "url": repo_url, "version": ref, "path": f"repos/{name}"}
         repos.append(repo_obj)
 
         # Store module outputs for GITHUB_OUTPUT compatibility
-        module_outputs[f'{name}_url'] = repo_url
+        module_outputs[f"{name}_url"] = repo_url
         if version:
-            module_outputs[f'{name}_version'] = version
+            module_outputs[f"{name}_version"] = version
         if branch:
-            module_outputs[f'{name}_branch'] = branch
+            module_outputs[f"{name}_branch"] = branch
         if hash_val:
-            module_outputs[f'{name}_hash'] = hash_val
+            module_outputs[f"{name}_hash"] = hash_val
 
     return repos, len(modules), module_outputs
 
@@ -103,7 +91,7 @@ def write_repos_json(repos, output_file="./repos.json"):
     output_path = Path(output_file)
 
     try:
-        with open(output_path, 'w') as f:
+        with open(output_path, "w") as f:
             json.dump(repos, f, indent=2)
         print(f"Generated {output_file}:")
         print(json.dumps(repos, indent=2))
@@ -120,11 +108,11 @@ def write_github_output(outputs):
     Args:
         outputs: Dictionary of key-value pairs to output
     """
-    github_output = Path(os.environ.get('GITHUB_OUTPUT', '/dev/null'))
+    github_output = Path(os.environ.get("GITHUB_OUTPUT", "/dev/null"))
 
     if github_output.exists() or github_output.parent.exists():
         try:
-            with open(github_output, 'a') as f:
+            with open(github_output, "a") as f:
                 for key, value in outputs.items():
                     f.write(f"{key}={value}\n")
         except IOError as e:
@@ -139,18 +127,18 @@ def main():
     # install_dependencies()
 
     # Parse known_good.json
-    repos, module_count, module_outputs = parse_known_good('./known_good.json')
+    repos, module_count, module_outputs = parse_known_good("./known_good.json")
 
     # Write repos.json
     write_repos_json(repos)
 
     # Write GitHub Actions outputs
-    github_outputs = {'MODULE_COUNT': str(module_count)}
+    github_outputs = {"MODULE_COUNT": str(module_count)}
     github_outputs.update(module_outputs)
     write_github_output(github_outputs)
 
     print("Parse complete!")
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()

scripts/workflow/recategorize_guidelines.py

@@ -68,15 +68,17 @@ def recategorize_sarif():
             [
                 "python3",
                 RECATEGORIZE_SCRIPT,
-                "--coding-standards-schema-file", CODING_STANDARDS_SCHEMA,
-                "--sarif-schema-file", SARIF_SCHEMA,
+                "--coding-standards-schema-file",
+                CODING_STANDARDS_SCHEMA,
+                "--sarif-schema-file",
+                SARIF_SCHEMA,
                 CODING_STANDARDS_CONFIG,
                 SARIF_FILE,
                 str(output_file),
             ],
             check=True,
             capture_output=True,
-            text=True
+            text=True,
         )
 
         print("Recategorization completed successfully")
@@ -120,5 +122,5 @@ def main():
     sys.exit(0)
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()