seal the mbean traits, tricksily

Author: samspills

core/jvm/src/main/scala/cats/effect/unsafe/metrics/ComputePoolSampler.scala

@@ -25,7 +25,7 @@ package metrics
  *   the monitored compute work stealing thread pool
  */
 private[unsafe] final class ComputePoolSampler(compute: WorkStealingThreadPool[_])
-    extends ComputePoolSamplerMBean {
+    extends UnsealedComputePoolSamplerMBean {
   def getWorkerThreadCount(): Int = compute.getWorkerThreadCount()
   def getActiveThreadCount(): Int = compute.getActiveThreadCount()
   def getSearchingThreadCount(): Int = compute.getSearchingThreadCount()

core/jvm/src/main/scala/cats/effect/unsafe/metrics/ComputePoolSamplerMBean.scala

@@ -19,7 +19,7 @@ package cats.effect.unsafe.metrics
 /**
  * An MBean interface for monitoring a `WorkStealingThreadPool` backed compute thread pool.
  */
-trait ComputePoolSamplerMBean {
+sealed trait ComputePoolSamplerMBean {
 
   /**
    * Returns the number of `WorkerThread` instances backing the `WorkStealingThreadPool`.
@@ -76,3 +76,5 @@ trait ComputePoolSamplerMBean {
    */
   def getSuspendedFiberCount(): Long
 }
+
+private[unsafe] trait UnsealedComputePoolSamplerMBean extends ComputePoolSamplerMBean

core/jvm/src/main/scala/cats/effect/unsafe/metrics/CpuStarvation.scala

@@ -26,7 +26,7 @@ private[metrics] class CpuStarvation private (
     counter: AtomicLong,
     currentClockDrift: AtomicLong,
     maxClockDrift: AtomicLong)
-    extends CpuStarvationMBean {
+    extends UnsealedCpuStarvationMBean {
 
   override def getCpuStarvationCount(): Long = counter.get()
 

core/jvm/src/main/scala/cats/effect/unsafe/metrics/CpuStarvationMbean.scala

@@ -19,7 +19,7 @@ package cats.effect.metrics
 /**
  * An MBean interfaces for monitoring when CPU starvation occurs.
  */
-trait CpuStarvationMBean {
+sealed trait CpuStarvationMBean {
 
   /**
    * Returns the number of times CPU starvation has occurred.
@@ -45,3 +45,5 @@ trait CpuStarvationMBean {
    */
   def getCurrentClockDriftMs(): Long
 }
+
+private[unsafe] trait UnsealedCpuStarvationMBean extends CpuStarvationMBean

core/jvm/src/main/scala/cats/effect/unsafe/metrics/JvmCpuStarvationMetrics.scala

@@ -27,7 +27,7 @@ import java.lang.management.ManagementFactory
 import javax.management.{MBeanServer, ObjectName}
 
 private[effect] class JvmCpuStarvationMetrics private (mbean: CpuStarvation)
-    extends CpuStarvationMetrics {
+    extends UnsealedCpuStarvationMetrics {
   override def incCpuStarvationCount: IO[Unit] = mbean.incStarvationCount
 
   override def recordClockDrift(drift: FiniteDuration): IO[Unit] = mbean.recordDrift(drift)

core/jvm/src/main/scala/cats/effect/unsafe/metrics/LiveFiberSnapshotTrigger.scala

@@ -27,7 +27,7 @@ import scala.collection.mutable.ArrayBuffer
  *   the backing fiber monitor
  */
 private[unsafe] final class LiveFiberSnapshotTrigger(monitor: FiberMonitor)
-    extends LiveFiberSnapshotTriggerMBean {
+    extends UnsealedLiveFiberSnapshotTriggerMBean {
   def liveFiberSnapshot(): Array[String] = {
     val buffer = new ArrayBuffer[String]
     monitor.liveFiberSnapshot(buffer += _)

core/jvm/src/main/scala/cats/effect/unsafe/metrics/LiveFiberSnapshotTriggerMBean.scala

@@ -19,7 +19,7 @@ package cats.effect.unsafe.metrics
 /**
  * An MBean interface for triggering live fiber snapshots.
  */
-trait LiveFiberSnapshotTriggerMBean {
+sealed trait LiveFiberSnapshotTriggerMBean {
 
   /**
    * Obtains a snapshot of the fibers currently live on the [[IORuntime]] which this fiber
@@ -30,3 +30,6 @@ trait LiveFiberSnapshotTriggerMBean {
    */
   def liveFiberSnapshot(): Array[String]
 }
+
+private[unsafe] trait UnsealedLiveFiberSnapshotTriggerMBean
+    extends LiveFiberSnapshotTriggerMBean

core/jvm/src/main/scala/cats/effect/unsafe/metrics/LocalQueueSampler.scala

@@ -25,7 +25,7 @@ package metrics
  *   the monitored local queue
  */
 private[unsafe] final class LocalQueueSampler(queue: LocalQueue)
-    extends LocalQueueSamplerMBean {
+    extends UnsealedLocalQueueSamplerMBean {
   def getFiberCount(): Int = queue.getFiberCount()
   def getHeadIndex(): Int = queue.getHeadIndex()
   def getTailIndex(): Int = queue.getTailIndex()

core/jvm/src/main/scala/cats/effect/unsafe/metrics/LocalQueueSamplerMBean.scala

@@ -19,7 +19,7 @@ package cats.effect.unsafe.metrics
 /**
  * An MBean interface for monitoring a single `WorkerThread` `LocalQueue`.
  */
-trait LocalQueueSamplerMBean {
+sealed trait LocalQueueSamplerMBean {
 
   /**
    * Returns the number of fibers enqueued on the monitored `LocalQueue`.
@@ -127,3 +127,5 @@ trait LocalQueueSamplerMBean {
    */
   def getTailTag(): Int
 }
+
+private[unsafe] trait UnsealedLocalQueueSamplerMBean extends LocalQueueSamplerMBean