@hexdspace/react 0.1.51 → 0.1.52

package/dist/index.js

@@ -3671,9 +3671,21 @@ import { useCallback as useCallback5, useEffect as useEffect7, useMemo as useMem
 var scopeBindings = /* @__PURE__ */ new Map();
 function notify(entry) {
   for (const notifySub of entry.subscribers.values()) {
-    notifySub(entry.ownerId);
+    notifySub(entry);
   }
 }
+function getScopeEntry(keyboardScopeId) {
+  const existing = scopeBindings.get(keyboardScopeId);
+  if (existing) return existing;
+  const entry = {
+    ownerId: null,
+    subscribers: /* @__PURE__ */ new Map(),
+    modifierActionsByOwner: /* @__PURE__ */ new Map(),
+    modifierActionsVersion: 0
+  };
+  scopeBindings.set(keyboardScopeId, entry);
+  return entry;
+}
 function useVimBindings(regionKey, keyboardScopeId, options = {}) {
   const vim = useVimController();
   const enabled = options.enabled ?? true;
@@ -3681,20 +3693,18 @@ function useVimBindings(regionKey, keyboardScopeId, options = {}) {
   const modifierMovementActions = options.modifierMovementActions;
   const ownerIdRef = useRef3(Symbol("vim-binding-owner"));
   const [isOwner, setIsOwner] = useState5(false);
-  const handleOwnerChange = useCallback5((ownerId) => {
-    setIsOwner(ownerId === ownerIdRef.current);
+  const [modifierActionsVersion, setModifierActionsVersion] = useState5(0);
+  const handleEntryChange = useCallback5((entry) => {
+    setIsOwner(entry.ownerId === ownerIdRef.current);
+    setModifierActionsVersion(entry.modifierActionsVersion);
   }, []);
   useEffect7(() => {
     if (!enabled) {
       setIsOwner(false);
       return;
     }
-    const entry = scopeBindings.get(keyboardScopeId) ?? {
-      ownerId: null,
-      subscribers: /* @__PURE__ */ new Map()
-    };
-    scopeBindings.set(keyboardScopeId, entry);
-    entry.subscribers.set(ownerIdRef.current, handleOwnerChange);
+    const entry = getScopeEntry(keyboardScopeId);
+    entry.subscribers.set(ownerIdRef.current, handleEntryChange);
     if (!entry.ownerId) {
       entry.ownerId = ownerIdRef.current;
     }
@@ -3704,18 +3714,38 @@ function useVimBindings(regionKey, keyboardScopeId, options = {}) {
       if (entry.ownerId === ownerIdRef.current) {
         const nextOwner = entry.subscribers.keys().next().value ?? null;
         entry.ownerId = nextOwner;
-        if (!nextOwner && entry.subscribers.size === 0) {
+        if (!nextOwner && entry.subscribers.size === 0 && entry.modifierActionsByOwner.size === 0) {
           scopeBindings.delete(keyboardScopeId);
           return;
         }
         notify(entry);
         return;
       }
-      if (entry.subscribers.size === 0) {
+      if (entry.subscribers.size === 0 && entry.modifierActionsByOwner.size === 0) {
+        scopeBindings.delete(keyboardScopeId);
+      }
+    };
+  }, [enabled, handleEntryChange, keyboardScopeId]);
+  useEffect7(() => {
+    if (!enabled) {
+      return;
+    }
+    const entry = getScopeEntry(keyboardScopeId);
+    entry.modifierActionsByOwner.set(ownerIdRef.current, modifierMovementActions ?? null);
+    entry.modifierActionsVersion += 1;
+    notify(entry);
+    return () => {
+      const existing = scopeBindings.get(keyboardScopeId);
+      if (!existing) return;
+      existing.modifierActionsByOwner.delete(ownerIdRef.current);
+      existing.modifierActionsVersion += 1;
+      if (!existing.ownerId && existing.subscribers.size === 0 && existing.modifierActionsByOwner.size === 0) {
         scopeBindings.delete(keyboardScopeId);
+        return;
       }
+      notify(existing);
     };
-  }, [enabled, handleOwnerChange, keyboardScopeId]);
+  }, [enabled, keyboardScopeId, modifierMovementActions]);
   useEffect7(() => {
     if (!enabled) {
       vim.setModifierMovementActions(regionKey, null);
@@ -3757,11 +3787,12 @@ function useVimBindings(regionKey, keyboardScopeId, options = {}) {
       shift: { label: "Shift", prefix: "Shift" },
       ctrlShift: { label: "Ctrl+Shift", prefix: "Control+Shift" }
     };
+    const modifierAvailability = getModifierAvailability(keyboardScopeId);
     const modifierBindings = (modifier, actions) => {
       if (!actions) return [];
       const { label, prefix } = modifierLabels[modifier];
       return movementEntries.flatMap((entry) => {
-        if (!actions?.[entry.dir]) return [];
+        if (!actions[entry.dir]) return [];
         const combos = [`${prefix}+${entry.char.toUpperCase()}`];
         if (entry.arrow) {
           combos.push(`${prefix}+${entry.arrow}`);
@@ -3817,13 +3848,44 @@ function useVimBindings(regionKey, keyboardScopeId, options = {}) {
         group: "vim"
       },
       { combos: ["Escape"], handler: (e) => vim.handleEscape(e), description: "vim: escape", group: "vim" },
-      ...modifierBindings("ctrl", modifierMovementActions?.ctrl),
-      ...modifierBindings("shift", modifierMovementActions?.shift),
-      ...modifierBindings("ctrlShift", modifierMovementActions?.ctrlShift)
+      ...modifierBindings("ctrl", modifierAvailability?.ctrl),
+      ...modifierBindings("shift", modifierAvailability?.shift),
+      ...modifierBindings("ctrlShift", modifierAvailability?.ctrlShift)
     ];
-  }, [allowArrowKeys, enabled, isOwner, modifierMovementActions, vim]);
+  }, [allowArrowKeys, enabled, isOwner, keyboardScopeId, modifierActionsVersion, vim]);
   useShortcut(keyboardScopeId, bindings, { ignoreTyping: true, preventDefault: true });
 }
+function getModifierAvailability(keyboardScopeId) {
+  const entry = scopeBindings.get(keyboardScopeId);
+  if (!entry) return null;
+  const result = {
+    ctrl: {},
+    shift: {},
+    ctrlShift: {}
+  };
+  for (const actions of entry.modifierActionsByOwner.values()) {
+    if (!actions) continue;
+    const ctrl = actions.ctrl;
+    if (ctrl) {
+      for (const dir of Object.keys(ctrl)) {
+        if (ctrl[dir]) result.ctrl[dir] = true;
+      }
+    }
+    const shift = actions.shift;
+    if (shift) {
+      for (const dir of Object.keys(shift)) {
+        if (shift[dir]) result.shift[dir] = true;
+      }
+    }
+    const ctrlShift = actions.ctrlShift;
+    if (ctrlShift) {
+      for (const dir of Object.keys(ctrlShift)) {
+        if (ctrlShift[dir]) result.ctrlShift[dir] = true;
+      }
+    }
+  }
+  return result;
+}
 
 // src/feature/vim-navigation/infra/web/react/hook/use-vim-region.tsx
 import { useEffect as useEffect9, useMemo as useMemo8, useRef as useRef4, useSyncExternalStore } from "react";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hexdspace/react",
-  "version": "0.1.51",
+  "version": "0.1.52",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",