Minor cleanups

thomas-chauchefoin-tob · claude · thomas-chauchefoin-tob · commit 56f0423eec50 · 2026-02-20T00:35:18.000+01:00
Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/fickling/cli.py b/fickling/cli.py
@@ -175,7 +175,7 @@ def main(argv: list[str] | None = None) -> int:
                             "Do not unpickle this file if it is from an untrusted source!\n\n"
                         )
 
-            return [EXIT_UNSAFE, EXIT_CLEAN][was_safe]
+            return EXIT_CLEAN if was_safe else EXIT_UNSAFE
 
         else:
             var_id = 0
diff --git a/fickling/constants.py b/fickling/constants.py
@@ -1,5 +1,3 @@
-"""Fickling constants module - shared constants to avoid circular imports."""
-
 # ClamAV-compatible exit codes for CI/CD integration
 EXIT_CLEAN = 0  # No issues found
 EXIT_UNSAFE = 1  # Potentially malicious content detected

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ def main(argv: list[str] \| None = None) -> int:`
`175`	`175`	`"Do not unpickle this file if it is from an untrusted source!\n\n"`
`176`	`176`	`)`
`177`	`177`
`178`		`- return [EXIT_UNSAFE, EXIT_CLEAN][was_safe]`
	`178`	`+ return EXIT_CLEAN if was_safe else EXIT_UNSAFE`
`179`	`179`
`180`	`180`	`else:`
`181`	`181`	`var_id = 0`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-"""Fickling constants module - shared constants to avoid circular imports."""`
`2`		`-`
`3`	`1`	`# ClamAV-compatible exit codes for CI/CD integration`
`4`	`2`	`EXIT_CLEAN = 0 # No issues found`
`5`	`3`	`EXIT_UNSAFE = 1 # Potentially malicious content detected`