fix: replace unsafe mateEnd.get with descriptive error in allele frequency calculation

The mateEnd.get call in numOverlappingTemplates would throw a bare
NoSuchElementException when the BAM file lacks MC (mate CIGAR) tags.

Replace with getOrElse that raises an IllegalStateException with a
clear message directing users to add MC tags (e.g. via samtools
fixmate). Keep checkAsPair unchanged since it does not access mateEnd.

Author: nh13

src/main/scala/com/fulcrumgenomics/sv/tools/AggregateSvPileup.scala

@@ -437,8 +437,11 @@ case class AggregatedBreakpointPileup(id: String,
       val rec = samIter.next()
       if (!overlappers.contains(rec.name)) {
         if (checkAsPair(rec)) {
+          val mateEnd = rec.mateEnd.getOrElse(throw new IllegalArgumentException(
+            s"Record '${rec.name}' is paired with a mapped mate on the same contig but has no mate end position. " +
+            "Ensure the BAM file contains the MC (mate CIGAR) tag, e.g. by running samtools fixmate."))
           if (breakends.positions.exists(
-            pos => pos >= Math.min(rec.start, rec.mateStart) && pos <= Math.max(rec.end, rec.mateEnd.get))) {
+            pos => pos >= Math.min(rec.start, rec.mateStart) && pos <= Math.max(rec.end, mateEnd))) {
             overlappers.add(rec.name)
           }
         } else if (breakends.positions.exists(pos => pos >= rec.start && pos <= rec.end)) {