@fieldnotes/core 0.38.5 → 0.38.7

package/dist/index.cjs

@@ -924,6 +924,11 @@ var KeyboardActions = class {
       this.nudgeTxId = recorder?.currentTransactionId ?? null;
     } else {
       clearTimeout(this.nudgeTimer);
+      const recorder = this.deps.getHistoryRecorder();
+      if (recorder?.currentTransactionId !== this.nudgeTxId) {
+        recorder?.begin();
+        this.nudgeTxId = recorder?.currentTransactionId ?? null;
+      }
     }
     const moved = sel.tool.nudgeSelection(dx * step, dy * step, sel.ctx);
     this.nudgeTimer = setTimeout(() => this.flushPendingNudge(), 400);
@@ -934,7 +939,7 @@ var KeyboardActions = class {
     clearTimeout(this.nudgeTimer);
     this.nudgeTimer = null;
     const recorder = this.deps.getHistoryRecorder();
-    if (this.nudgeTxId === null || recorder?.currentTransactionId === this.nudgeTxId) {
+    if (recorder?.currentTransactionId === this.nudgeTxId) {
       recorder?.commit();
     }
     this.nudgeTxId = null;
@@ -1245,6 +1250,9 @@ function bindingMatches(p, e, allowShift) {
   if (e.altKey !== p.alt) return false;
   return p.digit ? e.code === `Digit${p.key}` : e.key.toLowerCase() === p.key;
 }
+function sameBinding(a, b) {
+  return a.key === b.key && a.mod === b.mod && a.ctrl === b.ctrl && a.meta === b.meta && a.shift === b.shift && a.alt === b.alt;
+}
 function toArray(bindings) {
   if (bindings === null) return [];
   return Array.isArray(bindings) ? [...bindings] : [bindings];
@@ -1273,6 +1281,16 @@ var ShortcutMap = class {
   rebind(action, bindings) {
     const list = toArray(bindings);
     const parsedList = list.map(parseBinding);
+    for (const p of parsedList) {
+      for (const [otherAction, otherList] of this.parsed) {
+        if (otherAction === action) continue;
+        if (otherList.some((q) => sameBinding(p, q))) {
+          console.warn(
+            `[fieldnotes] shortcut binding for "${action}" conflicts with "${otherAction}"; first registered wins`
+          );
+        }
+      }
+    }
     this.raw.set(action, list);
     this.parsed.set(action, parsedList);
   }
@@ -9124,7 +9142,7 @@ var TemplateTool = class {
 };
 
 // src/index.ts
-var VERSION = "0.38.5";
+var VERSION = "0.38.7";
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   ArrowTool,