@fieldnotes/core 0.52.1 → 0.53.0

package/dist/index.cjs

@@ -983,6 +983,12 @@ function getElementBounds(element) {
   }
   return null;
 }
+function getElementVisualBounds(element) {
+  const bounds = getElementBounds(element);
+  if (!bounds) return null;
+  const rotation = element.rotation ?? 0;
+  return rotation === 0 ? bounds : rotatedAABB(bounds, rotation);
+}
 function getArrowBoundsAnalytical(from, to, bend) {
   if (bend === 0) {
     const minX2 = Math.min(from.x, to.x);
@@ -2754,10 +2760,7 @@ var ElementStore = class {
   // Spatial index stores the rotation-expanded AABB so rotated elements remain
   // broad-phase hit-test/marquee candidates; precise tests run against local bounds.
   indexBounds(element) {
-    const bounds = getElementBounds(element);
-    if (!bounds) return null;
-    const angle = element.rotation ?? 0;
-    return angle === 0 ? bounds : rotatedAABB(bounds, angle);
+    return getElementVisualBounds(element);
   }
   add(element, meta) {
     element = sanitizeRichText(element);
@@ -7078,7 +7081,7 @@ var RenderLoop = class {
       if (this.renderer.isDomElement(element)) {
         activeHybridOrder = null;
         const layerOpacity = this.layerManager.getLayer?.(element.layerId)?.opacity ?? 1;
-        const elBounds = getElementBounds(element);
+        const elBounds = getElementVisualBounds(element);
         if (elBounds && !boundsIntersect(elBounds, cullingRect)) {
           this.domNodeManager.hideDomNode(element.id);
         } else {
@@ -7135,7 +7138,7 @@ var RenderLoop = class {
         offCtx.save();
         this.marginViewport.applyRenderTransform(offCtx);
         for (const element of elements) {
-          const elBounds = getElementBounds(element);
+          const elBounds = getElementVisualBounds(element);
           if (elBounds && !boundsIntersect(elBounds, cullingRect)) continue;
           this.renderer.renderCanvasElement(offCtx, element);
         }
@@ -7210,7 +7213,7 @@ var RenderLoop = class {
       hybridCtx.translate(this.camera.position.x, this.camera.position.y);
       hybridCtx.scale(this.camera.zoom, this.camera.zoom);
       for (const element of elements) {
-        const elBounds = getElementBounds(element);
+        const elBounds = getElementVisualBounds(element);
         if (elBounds && !boundsIntersect(elBounds, cullingRect)) continue;
         hybridCtx.save();
         hybridCtx.globalAlpha = this.layerManager.getLayer?.(element.layerId)?.opacity ?? 1;
@@ -8181,6 +8184,7 @@ var Viewport = class {
   noteEditor;
   arrowLabelEditor;
   historyRecorder;
+  transactionDepth = 0;
   selectionOps;
   toolContext;
   marginViewport;
@@ -8294,6 +8298,35 @@ var Viewport = class {
   get shortcuts() {
     return this.inputHandler.shortcuts;
   }
+  /**
+   * Groups synchronous local store and layer mutations into one undo step.
+   * Nested calls join the outer transaction. If the callback throws, mutations
+   * already applied remain undoable and the original error is rethrown.
+   */
+  transaction(operation) {
+    const isOuterTransaction = this.transactionDepth === 0;
+    if (isOuterTransaction) {
+      this.inputHandler.flushPendingHistory();
+      this.historyRecorder.begin();
+    }
+    this.transactionDepth += 1;
+    try {
+      return operation();
+    } finally {
+      this.transactionDepth -= 1;
+      if (isOuterTransaction) this.historyRecorder.commit();
+    }
+  }
+  /** Removes existing elements as one undoable operation and returns the number removed. */
+  removeElements(ids) {
+    const existingIds = [...new Set(ids)].filter((id) => this.store.getById(id) !== void 0);
+    if (existingIds.length === 0) return 0;
+    this.transaction(() => {
+      for (const id of existingIds) this.store.remove(id);
+    });
+    this.requestRender();
+    return existingIds.length;
+  }
   undo() {
     this.inputHandler.flushPendingHistory();
     this.historyRecorder.pause();
@@ -11209,7 +11242,7 @@ var LaserTool = class {
 };
 
 // src/index.ts
-var VERSION = "0.52.1";
+var VERSION = "0.53.0";
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   ArrowTool,