Skip to content

Commit c81618a

Browse files
committed
properly included the enable diff param to the main module
1 parent 8a07994 commit c81618a

File tree

4 files changed

+7
-2
lines changed

4 files changed

+7
-2
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
66

77
[project]
88
name = "socketsecurity"
9-
version = "2.2.53"
9+
version = "2.2.54"
1010
requires-python = ">= 3.10"
1111
license = {"file" = "LICENSE"}
1212
dependencies = [

socketsecurity/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
__author__ = 'socket.dev'
2-
__version__ = '2.2.53'
2+
__version__ = '2.2.54'
33
USER_AGENT = f'SocketPythonCLI/{__version__}'

socketsecurity/config.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@ def from_args(cls, args_list: Optional[List[str]] = None) -> 'CliConfig':
107107
'commit_sha': args.commit_sha,
108108
'generate_license': args.generate_license,
109109
'enable_debug': args.enable_debug,
110+
'enable_diff': args.enable_diff,
110111
'allow_unverified': args.allow_unverified,
111112
'enable_json': args.enable_json,
112113
'enable_sarif': args.enable_sarif,

socketsecurity/socketcli.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -375,6 +375,7 @@ def main_code():
375375
# Case 4: Not a git repo (ignore_commit_files was auto-set to True)
376376
files_to_check = []
377377
# If --enable-diff is set, force diff mode for non-git repos
378+
log.debug(f"Case 4: Non-git repo - config.enable_diff={config.enable_diff}, type={type(config.enable_diff)}")
378379
if config.enable_diff:
379380
force_diff_mode = True
380381
log.debug("Non-git repo with --enable-diff: forcing diff mode")
@@ -401,9 +402,11 @@ def main_code():
401402

402403
# Case 3: If no supported files or files are empty, force API mode (no PR comments)
403404
# BUT: Don't force API mode if we're in force_diff_mode
405+
log.debug(f"files_to_check={files_to_check}, has_supported_files={has_supported_files}, force_diff_mode={force_diff_mode}, config.enable_diff={config.enable_diff}")
404406
if not has_supported_files and not force_diff_mode:
405407
force_api_mode = True
406408
log.debug("No supported manifest files found, forcing API mode")
409+
log.debug(f"force_api_mode={force_api_mode}")
407410

408411
# Determine scan behavior
409412
should_skip_scan = False # Always perform scan, but behavior changes based on supported files
@@ -465,6 +468,7 @@ def main_code():
465468
diff.report_url = ""
466469

467470
# Handle SCM-specific flows
471+
log.debug(f"Flow decision: scm={scm is not None}, force_diff_mode={force_diff_mode}, force_api_mode={force_api_mode}, enable_diff={config.enable_diff}")
468472
if scm is not None and scm.check_event_type() == "comment":
469473
# FIXME: This entire flow should be a separate command called "filter_ignored_alerts_in_comments"
470474
# It's not related to scanning or diff generation - it just:

0 commit comments

Comments
 (0)