Add debug logs

src/renderer/core/layout/operations/LayoutMutations.ts

@@ -4,6 +4,8 @@
  * Provides a clean API for layout operations that are CRDT-ready.
  * Operations are synchronous and applied directly to the store.
  */
+import log from 'loglevel'
+
 import { layoutStore } from '@/renderer/core/layout/store/LayoutStore'
 import {
   type LayoutMutations,
@@ -14,6 +16,8 @@ import {
   type Size
 } from '@/renderer/core/layout/types'
 
+const logger = log.getLogger('LayoutMutations')
+
 class LayoutMutationsImpl implements LayoutMutations {
   /**
    * Set the current mutation source
@@ -161,6 +165,11 @@ class LayoutMutationsImpl implements LayoutMutations {
     targetNodeId: string,
     targetSlot: number
   ): void {
+    logger.debug('Creating link:', {
+      linkId: Number(linkId),
+      from: `${sourceNodeId}[${sourceSlot}]`,
+      to: `${targetNodeId}[${targetSlot}]`
+    })
     layoutStore.applyOperation({
       type: 'createLink',
       entity: 'link',
@@ -179,6 +188,7 @@ class LayoutMutationsImpl implements LayoutMutations {
    * Delete a link
    */
   deleteLink(linkId: string | number): void {
+    logger.debug('Deleting link:', Number(linkId))
     layoutStore.applyOperation({
       type: 'deleteLink',
       entity: 'link',
@@ -198,6 +208,12 @@ class LayoutMutationsImpl implements LayoutMutations {
     parentId?: string | number,
     linkIds: (string | number)[] = []
   ): void {
+    logger.debug('Creating reroute:', {
+      rerouteId: Number(rerouteId),
+      position,
+      parentId: parentId != null ? Number(parentId) : undefined,
+      linkCount: linkIds.length
+    })
     layoutStore.applyOperation({
       type: 'createReroute',
       entity: 'reroute',
@@ -215,6 +231,7 @@ class LayoutMutationsImpl implements LayoutMutations {
    * Delete a reroute
    */
   deleteReroute(rerouteId: string | number): void {
+    logger.debug('Deleting reroute:', Number(rerouteId))
     layoutStore.applyOperation({
       type: 'deleteReroute',
       entity: 'reroute',
@@ -233,6 +250,11 @@ class LayoutMutationsImpl implements LayoutMutations {
     position: Point,
     previousPosition: Point
   ): void {
+    logger.debug('Moving reroute:', {
+      rerouteId: Number(rerouteId),
+      from: previousPosition,
+      to: position
+    })
     layoutStore.applyOperation({
       type: 'moveReroute',
       entity: 'reroute',

src/renderer/core/layout/store/LayoutStore.ts

@@ -4,6 +4,7 @@
  * Uses Yjs for efficient local state management and future collaboration.
  * CRDT ensures conflict-free operations for both single and multi-user scenarios.
  */
+import log from 'loglevel'
 import { type ComputedRef, type Ref, computed, customRef } from 'vue'
 import * as Y from 'yjs'
 
@@ -38,6 +39,8 @@ import {
 } from '@/renderer/core/layout/types'
 import { SpatialIndexManager } from '@/renderer/core/spatial/SpatialIndex'
 
+const logger = log.getLogger('LayoutStore')
+
 class LayoutStoreImpl implements LayoutStore {
   // Yjs document and shared data structures
   private ydoc = new Y.Doc()
@@ -384,6 +387,15 @@ class LayoutStoreImpl implements LayoutStore {
   updateSlotLayout(key: string, layout: SlotLayout): void {
     const existing = this.slotLayouts.get(key)
 
+    if (!existing) {
+      logger.debug('Adding slot:', {
+        nodeId: layout.nodeId,
+        type: layout.type,
+        index: layout.index,
+        bounds: layout.bounds
+      })
+    }
+
     if (existing) {
       // Update spatial index
       this.slotSpatialIndex.update(key, layout.bounds)
@@ -429,6 +441,14 @@ class LayoutStoreImpl implements LayoutStore {
   updateRerouteLayout(rerouteId: RerouteId, layout: RerouteLayout): void {
     const existing = this.rerouteLayouts.get(rerouteId)
 
+    if (!existing) {
+      logger.debug('Adding reroute layout:', {
+        rerouteId,
+        position: layout.position,
+        bounds: layout.bounds
+      })
+    }
+
     if (existing) {
       // Update spatial index
       this.rerouteSpatialIndex.update(String(rerouteId), layout.bounds) // Spatial index uses strings
@@ -498,6 +518,15 @@ class LayoutStoreImpl implements LayoutStore {
     }
 
     const existing = this.linkSegmentLayouts.get(key)
+    if (!existing) {
+      logger.debug('Adding link segment:', {
+        linkId,
+        rerouteId,
+        bounds: layout.bounds,
+        hasPath: !!layout.path
+      })
+    }
+
     if (existing) {
       // Update spatial index
       this.linkSegmentSpatialIndex.update(key, layout.bounds)
@@ -543,6 +572,14 @@ class LayoutStoreImpl implements LayoutStore {
     }
     const candidateKeys = this.linkSegmentSpatialIndex.query(searchArea)
 
+    if (candidateKeys.length > 0) {
+      logger.debug('Checking link segments at point:', {
+        point,
+        candidateCount: candidateKeys.length,
+        tolerance: hitWidth
+      })
+    }
+
     // Precise hit test only on candidates
     for (const key of candidateKeys) {
       const segmentLayout = this.linkSegmentLayouts.get(key)
@@ -559,6 +596,11 @@ class LayoutStoreImpl implements LayoutStore {
         )
 
         if (hit) {
+          logger.debug('Link segment hit:', {
+            linkId: segmentLayout.linkId,
+            rerouteId: segmentLayout.rerouteId,
+            point
+          })
           return {
             linkId: segmentLayout.linkId,
             rerouteId: segmentLayout.rerouteId
@@ -625,6 +667,13 @@ class LayoutStoreImpl implements LayoutStore {
     }
     const candidateRerouteKeys = this.rerouteSpatialIndex.query(searchArea)
 
+    if (candidateRerouteKeys.length > 0) {
+      logger.debug('Checking reroutes at point:', {
+        point,
+        candidateCount: candidateRerouteKeys.length
+      })
+    }
+
     // Check precise distance for candidates
     for (const rerouteKey of candidateRerouteKeys) {
       const rerouteId = Number(rerouteKey) as RerouteId // Convert string key back to numeric
@@ -635,6 +684,11 @@ class LayoutStoreImpl implements LayoutStore {
         const distance = Math.sqrt(dx * dx + dy * dy)
 
         if (distance <= rerouteLayout.radius) {
+          logger.debug('Reroute hit:', {
+            rerouteId: rerouteLayout.id,
+            position: rerouteLayout.position,
+            distance
+          })
           return rerouteLayout
         }
       }