refactor: routine key editing

packages/entity/src/trait-components/behaved.jsx

@@ -29,6 +29,7 @@ const Behaved = ({
     editingRef.current?.focus();
   });
   const [isEditing, setIsEditing] = useState(false);
+  const [previousKey, setPreviousKey] = useState(false);
   const previousEditing = usePrevious(isEditing);
   useEffect(() => {
     setTimeout(() => {
@@ -41,33 +42,39 @@ const Behaved = ({
   const tabPanels = [];
   Object.entries(json.params.routines).forEach(([key, expressions]) => {
     const routinePath = join(path, 'params/routines', key);
+    const confirmEdit = () => {
+      if (isEditing && previousKey !== isEditing) {
+        patch({
+          op: 'move',
+          from: join(path, 'params/routines', previousKey),
+          path: join(path, 'params/routines', isEditing),
+        });
+      }
+      setIsEditing(false);
+      setPreviousKey(false);
+    };
     tabs.push(
       <Tab
         key={key}
         onDoubleClick={() => {
+          setPreviousKey(key);
           setIsEditing(key);
         }}
       >
         {
-          key === isEditing
+          key === previousKey
             ? (
               <input
                 type="text"
-                value={key}
+                value={isEditing}
-                onBlur={() => {
+                onBlur={confirmEdit}
-                  setIsEditing(false);
-                }}
                 onChange={({target: {value}}) => {
-                  patch({
+                  setPreviousKey(key);
-                    op: 'move',
-                    from: routinePath,
-                    path: join(path, 'params/routines', value),
-                  });
                   setIsEditing(value);
                 }}
                 onKeyPress={({key}) => {
                   if ('Enter' === key) {
-                    setIsEditing(false);
+                    confirmEdit();
                   }
                 }}
                 ref={editingRef}
@@ -124,6 +131,7 @@ const Behaved = ({
                   },
                 });
                 setIsEditing(name);
+                setPreviousKey(name);
               }}
               type="button"
             >