@lvce-editor/explorer-view 2.46.0 → 2.48.0

package/dist/explorerViewWorkerMain.js

@@ -431,7 +431,7 @@ const create$4 = (method, params) => {
   };
 };
 const callbacks = Object.create(null);
-const set$2 = (id, fn) => {
+const set$3 = (id, fn) => {
   callbacks[id] = fn;
 };
 const get$2 = id => {
@@ -450,7 +450,7 @@ const registerPromise = () => {
     resolve,
     promise
   } = Promise.withResolvers();
-  set$2(id, resolve);
+  set$3(id, resolve);
   return {
     id,
     promise
@@ -779,7 +779,7 @@ const send = (transport, method, ...params) => {
   const message = create$4(method, params);
   transport.send(message);
 };
-const invoke$2 = (ipc, method, ...params) => {
+const invoke$3 = (ipc, method, ...params) => {
   return invokeHelper(ipc, method, params, false);
 };
 const invokeAndTransfer = (ipc, method, ...params) => {
@@ -812,7 +812,7 @@ const createRpc = ipc => {
       send(ipc, method, ...params);
     },
     invoke(method, ...params) {
-      return invoke$2(ipc, method, ...params);
+      return invoke$3(ipc, method, ...params);
     },
     invokeAndTransfer(method, ...params) {
       return invokeAndTransfer(ipc, method, ...params);
@@ -868,8 +868,13 @@ const WebWorkerRpcClient = {
   create: create$3
 };
 
+const CreateFolder$1 = 'createFolder'; // TODO use number
+const CreateFile$1 = 'createFile'; // TODO use number
+const Copy$2 = 'copy'; // TODO use number
+const Rename$2 = 'rename'; // TODO use number
+
 const rpcs = Object.create(null);
-const set$a = (id, rpc) => {
+const set$b = (id, rpc) => {
   rpcs[id] = rpc;
 };
 const get$1 = id => {
@@ -893,19 +898,19 @@ const create$2 = rpcId => {
       return rpc.invokeAndTransfer(method, ...params);
     },
     set(rpc) {
-      set$a(rpcId, rpc);
+      set$b(rpcId, rpc);
     }
   };
 };
 const RendererWorker$1 = 1;
 const {
-  invoke: invoke$5,
-  set: set$5
+  invoke: invoke$2,
+  set: set$2
 } = create$2(RendererWorker$1);
 const RendererWorker = {
   __proto__: null,
-  invoke: invoke$5,
-  set: set$5
+  invoke: invoke$2,
+  set: set$2
 };
 
 const {
@@ -932,9 +937,6 @@ const getRealPath = async path => {
 const stat = async dirent => {
   return invoke('FileSystem.stat', dirent);
 };
-const createFile = async uri => {
-  return invoke('FileSystem.writeFile', uri, '');
-};
 const writeFile = async (uri, content) => {
   return invoke('FileSystem.writeFile', uri, content);
 };
@@ -948,20 +950,29 @@ const copy$1 = async (oldUri, newUri) => {
   return invoke('FileSystem.copy', oldUri, newUri);
 };
 
-const createNestedPath = async (root, path, pathSeparator) => {
-  const parts = path.slice(root.length).split(pathSeparator);
-  let currentPath = '';
-  for (const part of parts) {
-    if (!part) continue;
-    currentPath = currentPath ? `${currentPath}${pathSeparator}${part}` : part;
-    try {
-      await mkdir(`${root}${currentPath}`);
-    } catch (error) {
-      // Ignore error if directory already exists
-      if (!(error instanceof Error && error.message.includes('already exists'))) {
-        throw error;
-      }
+const applyOperation = operation => {
+  if (operation.type === CreateFolder$1) {
+    return mkdir(operation.path);
+  }
+  if (operation.type === Copy$2) {
+    return copy$1(operation.from || '', operation.path);
+  }
+  if (operation.type === Rename$2) {
+    return rename$1(operation.from || '', operation.path);
+  }
+  return writeFile(operation.path, operation.text);
+};
+
+const applyFileOperations = async operations => {
+  try {
+    // TODO run operations in parallel if possible
+    for (const operation of operations) {
+      await applyOperation(operation);
     }
+    return '';
+  } catch (error) {
+    console.error(new VError(error, `Failed to apply file operations`));
+    return `${error}`;
   }
 };
 
@@ -988,21 +999,6 @@ const join2 = (path, childPath) => {
   return `${path}/${childPath}`;
 };
 
-const createNewDirentsAccept = async (newFileName, pathSeparator, absolutePath, root, createFn) => {
-  try {
-    // Create parent directories if they don't exist
-    if (newFileName.includes(pathSeparator)) {
-      const parentPath = dirname(pathSeparator, absolutePath);
-      await createNestedPath(root, parentPath, pathSeparator);
-    }
-    await createFn(absolutePath);
-    return '';
-  } catch (error) {
-    console.error(new VError(error, `Failed to create file`));
-    return `${error}`;
-  }
-};
-
 const createTree = (items, root) => {
   const tree = Object.create(null);
   const rootLength = root.length;
@@ -1130,6 +1126,39 @@ const getFileIcons = async (dirents, fileIconCache) => {
   };
 };
 
+const getFileOperationsNestedPath = (path, root, pathSeparator) => {
+  const parts = path.slice(root.length).split(pathSeparator);
+  const operations = [];
+  let currentPath = '';
+  for (const part of parts) {
+    if (!part) continue;
+    currentPath = currentPath ? `${currentPath}${pathSeparator}${part}` : part;
+    operations.push({
+      type: CreateFolder$1,
+      path: `${root}${currentPath}`
+    });
+  }
+  return operations;
+};
+const getFileOperationsCreate = (newFileName, newDirentType, pathSeparator, absolutePath, root) => {
+  const operations = [];
+  const parentPath = dirname(pathSeparator, absolutePath);
+  operations.push(...getFileOperationsNestedPath(parentPath, root, pathSeparator));
+  if (newDirentType === File) {
+    operations.push({
+      type: CreateFile$1,
+      path: absolutePath,
+      text: ''
+    });
+  } else if (newDirentType === Directory) {
+    operations.push({
+      type: CreateFolder$1,
+      path: absolutePath
+    });
+  }
+  return operations;
+};
+
 const getIndex = (dirents, uri) => {
   for (let i = 0; i < dirents.length; i++) {
     const dirent = dirents[i];
@@ -1477,7 +1506,7 @@ const validateFileName2 = name => {
   return '';
 };
 
-const acceptCreate = async (state, newDirentType, createFn) => {
+const acceptCreate = async (state, newDirentType) => {
   const {
     editingValue,
     minLineY,
@@ -1499,7 +1528,8 @@ const acceptCreate = async (state, newDirentType, createFn) => {
   }
   const parentFolder = getParentFolder(items, focusedIndex, root);
   const absolutePath = join2(parentFolder, newFileName);
-  const createErrorMessage = await createNewDirentsAccept(newFileName, pathSeparator, absolutePath, root, createFn);
+  const operations = getFileOperationsCreate(editingValue, newDirentType, pathSeparator, absolutePath, root);
+  const createErrorMessage = await applyFileOperations(operations);
   if (createErrorMessage) {
     return {
       ...state,
@@ -1534,11 +1564,11 @@ const acceptCreate = async (state, newDirentType, createFn) => {
 };
 
 const acceptCreateFile = async state => {
-  return acceptCreate(state, File, createFile);
+  return acceptCreate(state, File);
 };
 
 const acceptCreateFolder = async state => {
-  return acceptCreate(state, Directory, mkdir);
+  return acceptCreate(state, Directory);
 };
 
 const computeExplorerRenamedDirentUpdate = (root, parentPath, oldUri, children, tree, newUri) => {
@@ -1559,18 +1589,16 @@ const computeExplorerRenamedDirentUpdate = (root, parentPath, oldUri, children,
   return update;
 };
 
-const createNewDirentsRename = async (renamedDirent, editingValue, pathSeparator) => {
-  try {
-    // TODO this does not work with rename of nested file
-    const oldAbsolutePath = renamedDirent.path;
-    const oldParentPath = dirname2(oldAbsolutePath);
-    const newAbsolutePath = join2(oldParentPath, editingValue);
-    await rename$1(oldAbsolutePath, newAbsolutePath);
-  } catch (error) {
-    console.error(new VError(error, `Failed to rename file`));
-    return false;
-  }
-  return true;
+const getFileOperationsRename = (oldAbsolutePath, newFileName) => {
+  const operations = [];
+  const oldParentPath = dirname2(oldAbsolutePath);
+  const newAbsolutePath = join2(oldParentPath, newFileName);
+  operations.push({
+    type: Rename$2,
+    path: newAbsolutePath,
+    from: oldAbsolutePath
+  });
+  return operations;
 };
 
 const updateTree2 = (tree, update) => {
@@ -1600,9 +1628,13 @@ const acceptRename = async state => {
     };
   }
   const renamedDirent = items[editingIndex];
-  const successful = await createNewDirentsRename(renamedDirent, editingValue);
-  if (!successful) {
-    return state;
+  const operations = getFileOperationsRename(renamedDirent.path, editingValue);
+  const renameErrorMessage = await applyFileOperations(operations);
+  if (renameErrorMessage) {
+    return {
+      ...state,
+      editingErrorMessage: renameErrorMessage
+    };
   }
   const oldUri = renamedDirent.path;
   const dirname = dirname2(oldUri);
@@ -3227,23 +3259,6 @@ const refresh = async state => {
   };
 };
 
-const applyOperation = operation => {
-  if (operation.type === 'createFolder') {
-    return mkdir(operation.path);
-  }
-  if (operation.type === 'copy') {
-    return copy$1(operation.from || '', operation.path);
-  }
-  return writeFile(operation.path, operation.text);
-};
-
-const applyFileOperations = async operations => {
-  // TODO run operations in parallel if possible
-  for (const operation of operations) {
-    await applyOperation(operation);
-  }
-};
-
 const getChildHandles = async fileHandle => {
   // @ts-ignore
   const values = fileHandle.values();
@@ -3290,14 +3305,13 @@ const getFileOperations = (root, uploadTree) => {
       const fullPath = currentPath ? join2(currentPath, path) : path;
       if (typeof value === 'object') {
         operations.push({
-          type: 'createFolder',
-          path: join2(root, fullPath),
-          text: ''
+          type: CreateFolder$1,
+          path: join2(root, fullPath)
         });
         processTree(value, fullPath);
       } else if (typeof value === 'string') {
         operations.push({
-          type: 'createFile',
+          type: CreateFile$1,
           path: join2(root, fullPath),
           text: value
         });
@@ -3362,7 +3376,6 @@ const getFileOperationsElectron = async (root, paths, fileHandles) => {
     operations.push({
       type: 'copy',
       path: join2(root, name),
-      text: '',
       from: path
     });
   }
@@ -5174,7 +5187,7 @@ const selectDown = state => {
   }
   const newItems = items.map((item, i) => ({
     ...item,
-    selected: i === targetIndex + 1 ? true : item.selected
+    selected: i === targetIndex + 1 ? true : item.selected || i === focusedIndex
   }));
   return {
     ...state,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/explorer-view",
-  "version": "2.46.0",
+  "version": "2.48.0",
   "description": "Explorer Worker",
   "repository": {
     "type": "git",