fix: sort aggregated pileup IDs numerically instead of lexicographically

[nh13]

Summary

• Compound breakpoint IDs in AggregatedBreakpointPileup were sorted as strings, producing non-intuitive ordering (e.g. "112_456_5_7_8_9" instead of "5_7_8_9_112_456")
• Change pileupIds() and add() to use sortBy(_.toInt) for numeric ordering
• Update test expectations to match new numeric ordering

Test plan

• Existing tests pass with updated expectations (./mill tools.test)

[clintval]

May also help with determinism. Thanks!

[coderabbitai]

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 21 minutes and 58 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/aggregate-pileup-id-sorting

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}