Intent-Code Divergence
Medium
- Confidence
- 95% confidence
- Finding
- The comment at L296 says the concurrent write is safe because each goroutine writes to its own index, but the closure passed to errgroup.Go captures the loop variables `i` and `url` from the surrounding `for` loop without rebinding them. In Go versions before the newer loop-variable semantics, goroutines may observe the final values instead of per-iteration values, so the comment's safety claim actively contradicts the code as written.
