Audit use of `slices.Collect(..)`

[rowanseymour]

When updating to go 1.23 and switching to the iterator based functions in maps and slices we just replaced something like maps.Keys(..) with slices.Collect(maps.Keys(..)) but it would be better to actually use iterators where we can, especially for things like chunking thru really big slices.