CM-53929 - Fix formatting

Author: DmytroLynda

cycode/cli/apps/report/sbom/path/path_command.py

@@ -39,7 +39,11 @@ def path_command(
     try:
         is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
         documents = get_relevant_documents(
-            progress_bar, SbomReportProgressBarSection.PREPARE_LOCAL_FILES, consts.SCA_SCAN_TYPE, (str(path),), is_cycodeignore_allowed=is_cycodeignore_allowed
+            progress_bar,
+            SbomReportProgressBarSection.PREPARE_LOCAL_FILES,
+            consts.SCA_SCAN_TYPE,
+            (str(path),),
+            is_cycodeignore_allowed=is_cycodeignore_allowed,
         )
         # TODO(MarshalX): combine perform_pre_scan_documents_actions with get_relevant_document.
         #  unhardcode usage of context in perform_pre_scan_documents_actions

cycode/cli/apps/scan/code_scanner.py

@@ -23,7 +23,11 @@
 from cycode.cli.models import CliError, Document, LocalScanResult
 from cycode.cli.utils.progress_bar import ScanProgressBarSection
 from cycode.cli.utils.scan_batch import run_parallel_batched_scan
-from cycode.cli.utils.scan_utils import generate_unique_scan_id, is_cycodeignore_allowed_by_scan_config, set_issue_detected_by_scan_results
+from cycode.cli.utils.scan_utils import (
+    generate_unique_scan_id,
+    is_cycodeignore_allowed_by_scan_config,
+    set_issue_detected_by_scan_results,
+)
 from cycode.cyclient.models import ZippedFileScanResult
 from cycode.logger import get_logger
 
@@ -43,7 +47,13 @@ def scan_disk_files(ctx: typer.Context, paths: tuple[str, ...]) -> None:
 
     try:
         is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
-        documents = get_relevant_documents(progress_bar, ScanProgressBarSection.PREPARE_LOCAL_FILES, scan_type, paths, is_cycodeignore_allowed=is_cycodeignore_allowed)
+        documents = get_relevant_documents(
+            progress_bar,
+            ScanProgressBarSection.PREPARE_LOCAL_FILES,
+            scan_type,
+            paths,
+            is_cycodeignore_allowed=is_cycodeignore_allowed,
+        )
         add_sca_dependencies_tree_documents_if_needed(ctx, scan_type, documents)
         scan_documents(ctx, documents, get_scan_parameters(ctx, paths))
     except Exception as e:

cycode/cli/apps/scan/commit_range_scanner.py

@@ -41,7 +41,11 @@
 from cycode.cli.utils.git_proxy import git_proxy
 from cycode.cli.utils.path_utils import get_path_by_os
 from cycode.cli.utils.progress_bar import ScanProgressBarSection
-from cycode.cli.utils.scan_utils import generate_unique_scan_id, is_cycodeignore_allowed_by_scan_config, set_issue_detected_by_scan_results
+from cycode.cli.utils.scan_utils import (
+    generate_unique_scan_id,
+    is_cycodeignore_allowed_by_scan_config,
+    set_issue_detected_by_scan_results,
+)
 from cycode.cyclient.models import ZippedFileScanResult
 from cycode.logger import get_logger
 
@@ -189,9 +193,11 @@ def _scan_sca_commit_range(ctx: typer.Context, repo_path: str, commit_range: str
     )
     from_commit_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SCA_SCAN_TYPE, from_commit_documents)
     to_commit_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SCA_SCAN_TYPE, to_commit_documents)
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
-    from_commit_documents = filter_documents_with_cycodeignore(from_commit_documents, repo_path, is_cycodeignore_allowed)
+    from_commit_documents = filter_documents_with_cycodeignore(
+        from_commit_documents, repo_path, is_cycodeignore_allowed
+    )
     to_commit_documents = filter_documents_with_cycodeignore(to_commit_documents, repo_path, is_cycodeignore_allowed)
 
     perform_sca_pre_commit_range_scan_actions(
@@ -208,9 +214,11 @@ def _scan_secret_commit_range(
     diff_documents_to_scan = excluder.exclude_irrelevant_documents_to_scan(
         consts.SECRET_SCAN_TYPE, commit_diff_documents_to_scan
     )
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
-    diff_documents_to_scan = filter_documents_with_cycodeignore(diff_documents_to_scan, repo_path, is_cycodeignore_allowed)
+    diff_documents_to_scan = filter_documents_with_cycodeignore(
+        diff_documents_to_scan, repo_path, is_cycodeignore_allowed
+    )
 
     scan_documents(
         ctx, diff_documents_to_scan, get_scan_parameters(ctx, (repo_path,)), is_git_diff=True, is_commit_range=True
@@ -229,10 +237,10 @@ def _scan_sast_commit_range(ctx: typer.Context, repo_path: str, commit_range: st
         to_commit_rev,
         reverse_diff=False,
     )
-    
+
     commit_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SAST_SCAN_TYPE, commit_documents)
     diff_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SAST_SCAN_TYPE, diff_documents)
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
     commit_documents = filter_documents_with_cycodeignore(commit_documents, repo_path, is_cycodeignore_allowed)
     diff_documents = filter_documents_with_cycodeignore(diff_documents, repo_path, is_cycodeignore_allowed)
@@ -267,15 +275,17 @@ def _scan_sca_pre_commit(ctx: typer.Context, repo_path: str) -> None:
         progress_bar_section=ScanProgressBarSection.PREPARE_LOCAL_FILES,
         repo_path=repo_path,
     )
-    
+
     git_head_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SCA_SCAN_TYPE, git_head_documents)
     pre_committed_documents = excluder.exclude_irrelevant_documents_to_scan(
         consts.SCA_SCAN_TYPE, pre_committed_documents
     )
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
     git_head_documents = filter_documents_with_cycodeignore(git_head_documents, repo_path, is_cycodeignore_allowed)
-    pre_committed_documents = filter_documents_with_cycodeignore(pre_committed_documents, repo_path, is_cycodeignore_allowed)
+    pre_committed_documents = filter_documents_with_cycodeignore(
+        pre_committed_documents, repo_path, is_cycodeignore_allowed
+    )
 
     perform_sca_pre_hook_range_scan_actions(repo_path, git_head_documents, pre_committed_documents)
 
@@ -308,7 +318,7 @@ def _scan_secret_pre_commit(ctx: typer.Context, repo_path: str) -> None:
         )
 
     documents_to_scan = excluder.exclude_irrelevant_documents_to_scan(consts.SECRET_SCAN_TYPE, documents_to_scan)
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
     documents_to_scan = filter_documents_with_cycodeignore(documents_to_scan, repo_path, is_cycodeignore_allowed)
 
@@ -323,14 +333,16 @@ def _scan_sast_pre_commit(ctx: typer.Context, repo_path: str, **_) -> None:
         progress_bar_section=ScanProgressBarSection.PREPARE_LOCAL_FILES,
         repo_path=repo_path,
     )
-    
+
     pre_committed_documents = excluder.exclude_irrelevant_documents_to_scan(
         consts.SAST_SCAN_TYPE, pre_committed_documents
     )
     diff_documents = excluder.exclude_irrelevant_documents_to_scan(consts.SAST_SCAN_TYPE, diff_documents)
-    
+
     is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
-    pre_committed_documents = filter_documents_with_cycodeignore(pre_committed_documents, repo_path, is_cycodeignore_allowed)
+    pre_committed_documents = filter_documents_with_cycodeignore(
+        pre_committed_documents, repo_path, is_cycodeignore_allowed
+    )
     diff_documents = filter_documents_with_cycodeignore(diff_documents, repo_path, is_cycodeignore_allowed)
 
     _scan_commit_range_documents(ctx, pre_committed_documents, diff_documents, scan_parameters=scan_parameters)

cycode/cli/apps/scan/repository/repository_command.py

@@ -61,7 +61,7 @@ def repository_command(
             )
 
         documents_to_scan = excluder.exclude_irrelevant_documents_to_scan(scan_type, documents_to_scan)
-        
+
         is_cycodeignore_allowed = is_cycodeignore_allowed_by_scan_config(ctx)
         documents_to_scan = filter_documents_with_cycodeignore(documents_to_scan, str(path), is_cycodeignore_allowed)
 

cycode/cli/apps/scan/scan_command.py

@@ -165,11 +165,11 @@ def scan_command(
 
     # Get remote URL from current working directory
     remote_url = _try_get_git_remote_url(os.getcwd())
-    
+
     remote_scan_config = scan_client.get_scan_configuration_safe(scan_type, remote_url)
     if remote_scan_config:
         excluder.apply_scan_config(str(scan_type), remote_scan_config)
-    
+
     ctx.obj['scan_config'] = remote_scan_config
 
     if export_type and export_file:

cycode/cli/files_collector/documents_walk_ignore.py

@@ -29,7 +29,7 @@ def _log_ignored_files(repo_path: str, dirpath: str, ignored_dirnames: list[str]
     """Log ignored files for debugging (similar to walk_ignore function)."""
     rel_dirpath = '' if dirpath == repo_path else os.path.relpath(dirpath, repo_path)
     display_dir = rel_dirpath or '.'
-    
+
     for is_dir, names in (
         (True, ignored_dirnames),
         (False, ignored_filenames),
@@ -44,14 +44,14 @@ def _log_ignored_files(repo_path: str, dirpath: str, ignored_dirnames: list[str]
 def _build_allowed_paths_set(ignore_filter_manager: IgnoreFilterManager, repo_path: str) -> set[str]:
     """Build set of allowed file paths using walk_with_ignored."""
     allowed_paths = set()
-    
+
     for dirpath, dirnames, filenames, ignored_dirnames, ignored_filenames in ignore_filter_manager.walk_with_ignored():
         _log_ignored_files(repo_path, dirpath, ignored_dirnames, ignored_filenames)
-        
+
         for filename in filenames:
             file_path = os.path.join(dirpath, filename)
             allowed_paths.add(file_path)
-    
+
     return allowed_paths
 
 
@@ -63,22 +63,20 @@ def _get_document_check_path(document: 'Document', repo_path: str) -> str:
             check_path = document.path
         else:
             check_path = os.path.join(repo_path, document.path)
-    
+
     return os.path.normpath(check_path)
 
 
 def _filter_documents_by_allowed_paths(
-    documents: list['Document'], 
-    allowed_paths: set[str], 
-    repo_path: str
+    documents: list['Document'], allowed_paths: set[str], repo_path: str
 ) -> list['Document']:
     """Filter documents by checking if their paths are in the allowed set."""
     filtered_documents = []
-    
+
     for document in documents:
         try:
             check_path = _get_document_check_path(document, repo_path)
-            
+
             if check_path in allowed_paths:
                 filtered_documents.append(document)
             else:
@@ -87,44 +85,42 @@ def _filter_documents_by_allowed_paths(
         except Exception as e:
             logger.debug('Error processing document %s: %s', document.path, e)
             filtered_documents.append(document)
-    
+
     return filtered_documents
 
 
 def filter_documents_with_cycodeignore(
-    documents: list['Document'], 
-    repo_path: str, 
-    is_cycodeignore_allowed: bool = True
+    documents: list['Document'], repo_path: str, is_cycodeignore_allowed: bool = True
 ) -> list['Document']:
     """Filter documents based on .cycodeignore patterns.
-    
+
     This function uses .cycodeignore file in the repository root to filter out
     documents whose paths match any of those patterns.
-    
+
     Args:
         documents: List of Document objects to filter
         repo_path: Path to the repository root
         is_cycodeignore_allowed: Whether .cycodeignore filtering is allowed by scan configuration
-        
+
     Returns:
         List of Document objects that don't match any .cycodeignore patterns
     """
     if not is_cycodeignore_allowed:
         logger.debug('.cycodeignore filtering is not allowed by scan configuration')
         return documents
-    
+
     cycodeignore_path = _get_cycodeignore_path(repo_path)
-    
+
     if not os.path.exists(cycodeignore_path):
         return documents
-    
+
     logger.info('Using %s for filtering documents', cycodeignore_path)
-    
+
     ignore_filter_manager = _create_ignore_filter_manager(repo_path, cycodeignore_path)
-    
+
     allowed_paths = _build_allowed_paths_set(ignore_filter_manager, repo_path)
-    
+
     filtered_documents = _filter_documents_by_allowed_paths(documents, allowed_paths, repo_path)
-    
+
     logger.debug('Filtered %d documents using .cycodeignore patterns', len(documents) - len(filtered_documents))
     return filtered_documents

cycode/cli/files_collector/path_documents.py

@@ -18,18 +18,15 @@
 
 
 def _get_all_existing_files_in_directory(
-    path: str, 
-    *, 
-    walk_with_ignore_patterns: bool = True,
-    is_cycodeignore_allowed: bool = True
+    path: str, *, walk_with_ignore_patterns: bool = True, is_cycodeignore_allowed: bool = True
 ) -> list[str]:
     files: list[str] = []
 
     if walk_with_ignore_patterns:
         walk_func = lambda p: walk_ignore(p, is_cycodeignore_allowed=is_cycodeignore_allowed)
     else:
         walk_func = os.walk
-        
+
     for root, _, filenames in walk_func(path):
         for filename in filenames:
             files.append(os.path.join(root, filename))
@@ -51,12 +48,12 @@ def _get_relevant_files_in_path(path: str, *, is_cycodeignore_allowed: bool = Tr
 
 
 def _get_relevant_files(
-    progress_bar: 'BaseProgressBar', 
-    progress_bar_section: 'ProgressBarSection', 
-    scan_type: str, 
+    progress_bar: 'BaseProgressBar',
+    progress_bar_section: 'ProgressBarSection',
+    scan_type: str,
     paths: tuple[str, ...],
     *,
-    is_cycodeignore_allowed: bool = True
+    is_cycodeignore_allowed: bool = True,
 ) -> list[str]:
     all_files_to_scan = []
     for path in paths:
@@ -110,7 +107,9 @@ def get_relevant_documents(
     is_git_diff: bool = False,
     is_cycodeignore_allowed: bool = True,
 ) -> list[Document]:
-    relevant_files = _get_relevant_files(progress_bar, progress_bar_section, scan_type, paths, is_cycodeignore_allowed=is_cycodeignore_allowed)
+    relevant_files = _get_relevant_files(
+        progress_bar, progress_bar_section, scan_type, paths, is_cycodeignore_allowed=is_cycodeignore_allowed
+    )
 
     documents: list[Document] = []
     for file in relevant_files:

cycode/cli/files_collector/walk_ignore.py

@@ -29,12 +29,12 @@ def _walk_to_top(path: str) -> Iterable[str]:
 def _collect_top_level_ignore_files(path: str, *, is_cycodeignore_allowed: bool = True) -> list[str]:
     ignore_files = []
     top_paths = reversed(list(_walk_to_top(path)))  # we must reverse it to make top levels more prioritized
-    
+
     supported_files = set(_SUPPORTED_IGNORE_PATTERN_FILES)
     if is_cycodeignore_allowed:
         supported_files.add(consts.CYCODEIGNORE_FILENAME)
         logger.debug('.cycodeignore files included due to scan configuration')
-    
+
     for dir_path in top_paths:
         for ignore_file in supported_files:
             ignore_file_path = os.path.join(dir_path, ignore_file)
@@ -44,7 +44,9 @@ def _collect_top_level_ignore_files(path: str, *, is_cycodeignore_allowed: bool
     return ignore_files
 
 
-def walk_ignore(path: str, *, is_cycodeignore_allowed: bool = True) -> Generator[tuple[str, list[str], list[str]], None, None]:
+def walk_ignore(
+    path: str, *, is_cycodeignore_allowed: bool = True
+) -> Generator[tuple[str, list[str], list[str]], None, None]:
     ignore_file_paths = _collect_top_level_ignore_files(path, is_cycodeignore_allowed=is_cycodeignore_allowed)
     ignore_filter_manager = IgnoreFilterManager.build(
         path=path,
@@ -64,4 +66,4 @@ def walk_ignore(path: str, *, is_cycodeignore_allowed: bool = True) -> Generator
                     full_path = os.path.join(full_path, '*')
                 logger.debug('Ignoring match %s', full_path)
 
-        yield dirpath, dirnames, filenames
+        yield dirpath, dirnames, filenames

cycode/cyclient/scan_client.py

@@ -284,15 +284,17 @@ def get_scan_configuration(self, scan_type: str, remote_url: Optional[str] = Non
         params = {}
         if remote_url:
             params['remote_url'] = remote_url
-            
+
         response = self.scan_cycode_client.get(
             url_path=self.get_scan_configuration_path(scan_type),
             params=params,
             hide_response_content_log=self._hide_response_log,
         )
         return models.ScanConfigurationSchema().load(response.json())
 
-    def get_scan_configuration_safe(self, scan_type: str, remote_url: Optional[str] = None) -> Optional['models.ScanConfiguration']:
+    def get_scan_configuration_safe(
+        self, scan_type: str, remote_url: Optional[str] = None
+    ) -> Optional['models.ScanConfiguration']:
         try:
             return self.get_scan_configuration(scan_type, remote_url)
         except RequestHttpError as e: