safe-global / safe-smart-account / 18092332097

94%

push

github

Use nullish coalescing in buildSafeTransaction to preserve intentional falsy values (#1043)

Description
- Summary: Replace logical OR fallbacks with nullish coalescing in
buildSafeTransaction to avoid overwriting intentional falsy values
(e.g., 0).
- Why: || treats 0/"" as falsy and replaces them with defaults, which
can mask valid inputs and cause subtle bugs.
- Change:
-
value/data/operation/safeTxGas/baseGas/gasPrice/gasToken/refundReceiver
now use ?? with safe defaults.

304 of 336 branches covered (90.48%)

Branch coverage included in aggregate %.

449 of 461 relevant lines covered (97.4%)

109.0 hits per line