fix: loop_select conditions (PRO-587)

[AlastairHolmes]

No description provided.

[linear]

PRO-587 Review loop_select exit conditions

See TODO comments

[codecov]

Codecov Report

Merging #4061 (4d1bfad) into main (0168147) will decrease coverage by 0%.
Report is 5 commits behind head on main.
The diff coverage is 82%.

@@          Coverage Diff           @@
##            main   #4061    +/-   ##
======================================
- Coverage     72%     71%    -0%     
======================================
  Files        376     376            
  Lines      59433   59662   +229     
  Branches   59433   59662   +229     
======================================
+ Hits       42506   42547    +41     
- Misses     14816   14965   +149     
- Partials    2111    2150    +39     

Files	Coverage Δ
engine/src/witness/common/chain_source/shared.rs	0% <0%> (ø)
...chain_source/chunked_by_vault/deposit_addresses.rs	0% <0%> (ø)
...nked_chain_source/chunked_by_vault/egress_items.rs	0% <0%> (ø)
utilities/src/with_std/loop_select.rs	71% <90%> (-1%)	⬇️

... and 20 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more