refactor: address gemini review on PR #52

- computePointSnap now accumulates all guides at the same best
  distance, mirroring computeSnap's behaviour: when two neighbour
  objects share an edge with the dragged endpoint, both guide
  lines render instead of only the first-encountered one.
- BarcodeObject's text-group wrapper guards on .length > 0 instead
  of truthy. textNodes / textElements are arrays initialised to []
  / undefined respectively; an empty array is truthy and would
  otherwise spawn an empty <Group> + ref callback every render.

Author: u8array

src/components/Canvas/BarcodeObject.tsx

@@ -446,7 +446,7 @@ export function BarcodeObject({
             strokeWidth={isSelected ? 2 : 0}
             strokeScaleEnabled={false}
           />
-          {textNodes && <Group ref={excludeGroupFromBbox}>{textNodes}</Group>}
+          {textNodes.length > 0 && <Group ref={excludeGroupFromBbox}>{textNodes}</Group>}
         </Group>
       );
     }
@@ -695,7 +695,9 @@ export function BarcodeObject({
             strokeWidth={isSelected ? 2 : 0}
             strokeScaleEnabled={false}
           />
-          {textElements && <Group ref={excludeGroupFromBbox}>{textElements}</Group>}
+          {Array.isArray(textElements) && textElements.length > 0 && (
+            <Group ref={excludeGroupFromBbox}>{textElements}</Group>
+          )}
         </Group>
       );
     }

src/lib/snapGuides.ts

@@ -23,20 +23,28 @@ export function computePointSnap(
   ): { value: number; guides: SnapGuide[] } => {
     let bestDelta = Infinity;
     let bestValue = drag;
-    let bestGuide: SnapGuide | null = null;
+    let bestGuides: SnapGuide[] = [];
     const consider = (target: number, perpFrom: number, perpTo: number) => {
       const d = Math.abs(target - drag);
-      if (d > threshold || d >= bestDelta) return;
-      bestDelta = d;
-      bestValue = target;
+      if (d > threshold || d > bestDelta) return;
       const guideOrientation: 'H' | 'V' = axis === 'x' ? 'V' : 'H';
-      bestGuide = {
+      const guide: SnapGuide = {
         orientation: guideOrientation,
         type: 'align',
         pos: target,
         from: Math.min(dragPerp, perpFrom) - GUIDE_PADDING_PX,
         to: Math.max(dragPerp, perpTo) + GUIDE_PADDING_PX,
       };
+      if (d < bestDelta) {
+        // Strictly closer — replace the best candidate.
+        bestDelta = d;
+        bestValue = target;
+        bestGuides = [guide];
+      } else if (target === bestValue) {
+        // Same edge value at the same distance — accumulate so each
+        // contributing object draws its own guide line.
+        bestGuides.push(guide);
+      }
     };
     for (const o of others) {
       const startEdge = axis === 'x' ? o.x : o.y;
@@ -54,7 +62,7 @@ export function computePointSnap(
       consider(startEdge, perpStart, perpEnd);
       consider(endEdge, perpStart, perpEnd);
     }
-    return { value: bestValue, guides: bestGuide ? [bestGuide] : [] };
+    return { value: bestValue, guides: bestGuides };
   };
 
   const xRes = snapAxisPt(point.x, point.y, 'x');