feat: improved copy api

Author: bajrangCoder

src/lib/editorFile.js

@@ -748,7 +748,9 @@ export default class EditorFile {
 					editorManager.container.parentElement.appendChild(this.content);
 				}
 			}
-			editor?.selection.clearSelection();
+			if (activeFile && activeFile.type === "editor") {
+				activeFile.session.selection.clearSelection();
+			}
 		}
 
 		this.#tab.classList.add("active");

src/lib/editorManager.js

@@ -660,7 +660,9 @@ async function EditorManager($header, $body) {
 					$container.parentElement.appendChild(file.content);
 				}
 			}
-			editor?.selection.clearSelection();
+			if (manager.activeFile && manager.activeFile.type === "editor") {
+				manager.activeFile.session.selection.clearSelection();
+			}
 		}
 
 		$header.text = file.filename;

src/plugins/sftp/src/com/foxdebug/sftp/Sftp.java

@@ -599,51 +599,57 @@ public void run() {
               String source = args.optString(0);
               String destination = args.optString(1);
 
-              if (ssh == null || sftp == null) {
-                callback.error("Not connected");
-                return;
-              }
-
-              try {
-                copyRecursively(source, destination);
-                callback.success();
-              } catch (Exception e) {
-                callback.error("Copy failed: " + errMessage(e));
-              }
-            } catch (Exception e) {
-              callback.error(errMessage(e));
-            }
-          }
-
-          private void copyRecursively(String source, String destination)
-            throws SftpStatusException, SshException, IOException {
-            SftpFileAttributes attrs = sftp.stat(source);
+              if (ssh != null && sftp != null) {
+                try {
+                  SftpFileAttributes sourceAttrs = sftp.stat(source);
 
-            if (attrs.isDirectory()) {
-              try {
-                sftp.mkdir(destination);
-              } catch (SftpStatusException e) {
-                // Directory may already exist - ignore
-              }
+                  if (sourceAttrs.isFile()) {
+                    // file copy
+                    try (java.io.InputStream in = sftp.getInputStream(source)) {
+                      sftp.put(in, destination);
+                    }
+                  } else if (sourceAttrs.isDirectory()) {
+                    // directory copy
+                    sftp.mkdir(destination);
+
+                    for (SftpFile file : sftp.ls(source)) {
+                      String filename = file.getFilename();
+                      if (!filename.equals(".") && !filename.equals("..")) {
+                        String sourcePath = source + "/" + filename;
+                        String destPath = destination + "/" + filename;
+
+                        if (file.getAttributes().isDirectory()) {
+                          // Recursively copy subdirectories
+                          JSONArray subDirArgs = new JSONArray();
+                          subDirArgs.put(sourcePath);
+                          subDirArgs.put(destPath);
+                          copy(subDirArgs, callback);
+                        } else {
+                          // Copy files within directory
+                          try (
+                            java.io.InputStream in = sftp.getInputStream(
+                              sourcePath
+                            )
+                          ) {
+                            sftp.put(in, destPath);
+                          }
+                        }
+                      }
+                    }
+                  }
 
-              // Copy directory contents
-              for (SftpFile file : sftp.ls(source)) {
-                String filename = file.getFilename();
-                if (!filename.equals(".") && !filename.equals("..")) {
-                  String srcPath = source + "/" + filename;
-                  String destPath = destination + "/" + filename;
-                  copyRecursively(srcPath, destPath);
+                  callback.success();
+                  return;
+                } catch (SftpStatusException e) {
+                  callback.error("Source does not exist: " + errMessage(e));
+                  return;
                 }
               }
-            } else {
-              // Copy file using buffered streams
-              try (
-                java.io.InputStream in = sftp.getInputStream(source);
-                java.io.BufferedInputStream bis =
-                  new java.io.BufferedInputStream(in)
-              ) {
-                sftp.put(bis, destination);
-              } catch (TransferCancelledException e) {}
+              callback.error("Not connected");
+            } catch (
+              SshException | IOException | TransferCancelledException e
+            ) {
+              callback.error(errMessage(e));
             }
           }
         }

src/plugins/sftp/www/sftp.js

@@ -42,8 +42,8 @@ module.exports = {
   rename: function (oldpath, newpath, onSuccess, onFail) {
     cordova.exec(onSuccess, onFail, 'Sftp', 'rename', [oldpath, newpath]);
   },
-  copy: function (sourcePath, destinationPath, onSuccess, onFail) {
-    cordova.exec(onSuccess, onFail, 'Sftp', 'copy', [sourcePath, destinationPath]);
+  copy: function (source, destination, onSuccess, onFail) {
+    cordova.exec(onSuccess, onFail, 'Sftp', 'copy', [source, destination]);
   },
   pwd: function (onSuccess, onFail) {
     cordova.exec(onSuccess, onFail, 'Sftp', 'pwd', []);