address the review concerns

Author: bajrangCoder

src/lib/openFolder.js

@@ -664,34 +664,40 @@ function execOperation(type, action, url, $target, name) {
 		newName = helpers.fixFilename(newName);
 		if (!newName) return;
 		startLoading();
-		const isNestedPath = newName.split("/").filter(Boolean).length > 1;
-		let newUrl;
+		try {
+			const isNestedPath = newName.split("/").filter(Boolean).length > 1;
+			let newUrl;
 
-		if (action === "new file") {
-			newUrl = await helpers.createFileStructure(url, newName);
-		} else {
-			newUrl = await helpers.createFileStructure(url, newName, false);
-		}
-		if (!newUrl) return;
+			if (action === "new file") {
+				newUrl = await helpers.createFileStructure(url, newName);
+			} else {
+				newUrl = await helpers.createFileStructure(url, newName, false);
+			}
+			if (!newUrl) return;
 
-		if (isNestedPath) {
-			openFolder.find(url)?.reload();
-			await FileList.refresh();
-			toast(strings.success);
-			return;
-		}
+			if (isNestedPath) {
+				openFolder.find(url)?.reload();
+				await FileList.refresh();
+				toast(strings.success);
+				return;
+			}
 
-		newName = Url.basename(newUrl.uri);
-		if ($target.unclasped) {
-			if (newUrl.type === "file") {
-				appendTile($target, createFileTile(newName, newUrl.uri));
-			} else if (newUrl.type === "folder") {
-				appendList($target, createFolderTile(newName, newUrl.uri));
+			newName = Url.basename(newUrl.uri);
+			if ($target.unclasped) {
+				if (newUrl.type === "file") {
+					appendTile($target, createFileTile(newName, newUrl.uri));
+				} else if (newUrl.type === "folder") {
+					appendList($target, createFolderTile(newName, newUrl.uri));
+				}
 			}
-		}
 
-		FileList.append(url, newUrl.uri);
-		toast(strings.success);
+			FileList.append(url, newUrl.uri);
+			toast(strings.success);
+		} catch (error) {
+			helpers.error(error);
+		} finally {
+			stopLoading();
+		}
 	}
 
 	async function paste() {

src/utils/helpers.js

@@ -457,18 +457,15 @@ export default {
 			const existingEntry = entries.find((entry) => entry.name === name);
 
 			if (existingEntry) {
-				const actualType =
-					existingEntry.isDirectory || existingEntry.isFile === false
-						? "folder"
-						: "file";
+				const actualType = existingEntry.isDirectory ? "folder" : "file";
 				if (actualType !== expectedType) {
 					throw new Error(
 						`${name} already exists as a ${actualType}, expected ${expectedType}.`,
 					);
 				}
 
 				return {
-					url: existingEntry.url || targetUri,
+					url: existingEntry.url || existingEntry.uri || targetUri,
 					created: false,
 					type: expectedType,
 				};