Review fixes

Author: emeeks

src/components/charts/shared/statisticalOverlays.ts

@@ -201,17 +201,17 @@ function buildPrecomputed(
 
   if (groupByField) {
     // Group-aware boundary duplication: collect points per group, find
-    // segment transitions within each group, then emit all points in
-    // original order with bridge points inserted at the right positions.
+    // segment transitions within each group, then append bridge points
+    // after the tagged data. Ordering within groups is handled by the
+    // downstream pipeline which sorts by x-accessor per group.
     const groups = new Map<string, Record<string, any>[]>()
     for (const d of tagged) {
       const key = d[groupByField] ?? "__default"
       if (!groups.has(key)) groups.set(key, [])
       groups.get(key)!.push(d)
     }
 
-    // For each group, find segment boundaries and collect bridge points
-    // keyed by their position in the original tagged array (insert-after index).
+    // For each group, find segment boundaries and collect bridge points.
     const bridgePoints: Record<string, any>[] = []
     for (const [, groupPoints] of groups) {
       for (let j = 0; j < groupPoints.length - 1; j++) {

src/components/stream/StreamGeoFrame.tsx

@@ -684,20 +684,22 @@ const StreamGeoFrame = forwardRef<StreamGeoFrameHandle, StreamGeoFrameProps>(
 
           if (hoveredNode && hoveredNode.type === "point") {
             const pn = hoveredNode as PointSceneNode
-            const nodeColor = resolveNodeColor(hoveredNode)
+            // Respect hoverAnnotation.pointColor for consistency with XY frame
+            const hoverConfig = typeof hoverAnnotation === "object" ? hoverAnnotation : undefined
+            const pointColor = hoverConfig?.pointColor || resolveNodeColor(hoveredNode)
             ictx.beginPath()
             ictx.arc(pn.x, pn.y, pn.r + 3, 0, Math.PI * 2)
-            if (nodeColor) {
+            if (pointColor) {
               ictx.save()
               ictx.globalAlpha = 0.4
-              ictx.fillStyle = nodeColor
+              ictx.fillStyle = pointColor
               ictx.fill()
               ictx.restore()
             } else {
               ictx.fillStyle = "rgba(255, 255, 255, 0.4)"
               ictx.fill()
             }
-            ictx.strokeStyle = nodeColor || "rgba(0, 0, 0, 0.5)"
+            ictx.strokeStyle = pointColor || "rgba(0, 0, 0, 0.5)"
             ictx.lineWidth = 2
             ictx.stroke()
           }