scp-next 1.0.6 → 1.0.8

package/README.md

@@ -44,8 +44,7 @@ scp-next run <job> [source] [destination] [options]
 CLI commands use `<source> <destination>`. Programmatic upload and download APIs use
 `localPath` and `remotePath`.
 
-Destination paths follow familiar `cp`/`scp` behavior. If the destination exists as a directory or ends with a path separator, `scp-next` places the source inside that directory using the source basename.
-Use `--create-directories` when a destination directory may be missing.
+Destination paths follow familiar `cp`/`scp` behavior. If the destination exists as a directory or ends with a path separator, `scp-next` places the source inside that directory using the source basename. Missing destination directories are created by default.
 
 ## Upload Examples
 
@@ -54,8 +53,7 @@ scp-next upload ./dist /var/www/example \
   --host example.com \
   --username deploy \
   --private-key-file ~/.ssh/id_ed25519 \
-  --recursive \
-  --create-directories
+  --recursive
 ```
 
 Password authentication:
@@ -77,8 +75,7 @@ Prefer `SCP_NEXT_PASSWORD`, SSH agents, or protected private-key files for share
 scp-next download /var/log/example.log ./logs/example.log \
   --host example.com \
   --username deploy \
-  --private-key-file ~/.ssh/id_ed25519 \
-  --create-directories
+  --private-key-file ~/.ssh/id_ed25519
 ```
 
 ## CLI Options
@@ -96,7 +93,8 @@ scp-next download /var/log/example.log ./logs/example.log \
 | `--profile <name>`                    | Named server profile from the configuration file.                      |
 | `--recursive`                         | Transfer directories recursively.                                      |
 | `--overwrite`                         | Allow replacing existing destination files.                            |
-| `--create-directories`                | Create missing destination directories when supported.                 |
+| `--create-directories`                | Create missing destination directories. Enabled by default.            |
+| `--no-create-directories`             | Disable automatic destination directory creation.                      |
 | `--dry-run`                           | Resolve and validate the operation without connecting or transferring. |
 | `--timeout <milliseconds>`            | Connection and operation timeout in milliseconds.                      |
 | `--verbose`                           | Print non-sensitive diagnostic details.                                |

package/dist/cli/index.js

@@ -564,7 +564,7 @@ async function validateDownloadOptions(options) {
   }
   assertRemotePath(options.remotePath, "remotePath");
   validateTimeout(options.timeout);
-  await assertDownloadDestination(options.localPath, options.createDirectories);
+  await assertDownloadDestination(options.localPath, options.createDirectories ?? true);
   if (!options.dryRun) {
     await validateServerOptions(options);
   }
@@ -580,6 +580,9 @@ function percentage(transferredBytes, totalBytes) {
 function mergeOptions(serverOptions, transferOptions) {
   return { ...serverOptions, ...transferOptions ?? {} };
 }
+function shouldCreateDirectories(options) {
+  return options.createDirectories ?? true;
+}
 function hasTrailingPathSeparator(inputPath) {
   return /[\\/]$/.test(inputPath);
 }
@@ -596,6 +599,22 @@ async function isExistingLocalDirectory(localPath) {
     return false;
   }
 }
+async function ensureLocalDirectory(localPath, createParents = false) {
+  try {
+    const stats = await stat3(localPath);
+    if (stats.isDirectory()) {
+      return;
+    }
+    throw new TransferError(`Local destination already exists and is not a directory: ${localPath}`, {
+      context: { localPath }
+    });
+  } catch (error) {
+    if (error instanceof TransferError) {
+      throw error;
+    }
+    await mkdir2(localPath, { recursive: createParents });
+  }
+}
 var ScpNextClientImpl = class {
   serverOptions;
   transport;
@@ -705,7 +724,7 @@ var ScpNextClientImpl = class {
     const remoteKind = await this.transport.exists(remotePath);
     const isDirectoryLikeDestination = remoteKind === "d" || remotePath.endsWith("/");
     if (!isDirectoryLikeDestination) {
-      await this.ensureRemoteParent(remotePath, options.createDirectories);
+      await this.ensureRemoteParent(remotePath, shouldCreateDirectories(options));
       await this.assertRemoteOverwrite(remotePath, options.overwrite);
       return remotePath;
     }
@@ -716,7 +735,7 @@ var ScpNextClientImpl = class {
     }
     const remoteDirectory = remotePath.replace(/\/+$/g, "") || "/";
     if (!remoteKind) {
-      if (!options.createDirectories) {
+      if (!shouldCreateDirectories(options)) {
         throw new TransferError(`Remote destination directory does not exist: ${remoteDirectory}`, {
           context: { remotePath }
         });
@@ -742,7 +761,7 @@ var ScpNextClientImpl = class {
     if (isDirectoryLikeDestination) {
       const remoteDirectory = trimmedRemoteDirectory(remotePath);
       if (!remoteKind) {
-        if (!options.createDirectories) {
+        if (!shouldCreateDirectories(options)) {
           throw new TransferError(`Remote destination directory does not exist: ${remoteDirectory}`, {
             context: { remotePath }
           });
@@ -754,7 +773,7 @@ var ScpNextClientImpl = class {
       return destinationRoot;
     }
     if (!remoteKind) {
-      if (!options.createDirectories) {
+      if (!shouldCreateDirectories(options)) {
         throw new TransferError(`Remote destination directory does not exist: ${remotePath}`, {
           context: { remotePath }
         });
@@ -783,7 +802,7 @@ var ScpNextClientImpl = class {
   async downloadSingleFile(options, remotePath) {
     const localPath = await this.resolveLocalDestinationForRemoteFile(options, remotePath);
     await this.assertLocalOverwrite(localPath, options.overwrite);
-    if (options.createDirectories) {
+    if (shouldCreateDirectories(options)) {
       await mkdir2(path4.dirname(localPath), { recursive: true });
     }
     await this.transport.downloadFile(remotePath, localPath, (step) => {
@@ -805,7 +824,7 @@ var ScpNextClientImpl = class {
     const totalBytes = files.reduce((sum, file) => sum + file.size, 0);
     let completedBytes = 0;
     let completedFiles = 0;
-    await mkdir2(localRoot, { recursive: Boolean(options.createDirectories) });
+    await ensureLocalDirectory(localRoot, shouldCreateDirectories(options));
     for (const file of files) {
       const localFile = path4.join(localRoot, ...file.relativePath.split("/"));
       await this.assertLocalOverwrite(localFile, options.overwrite);
@@ -835,7 +854,7 @@ var ScpNextClientImpl = class {
       return localDestination;
     }
     if (!await isExistingLocalDirectory(localDestination)) {
-      if (!options.createDirectories) {
+      if (!shouldCreateDirectories(options)) {
         throw new TransferError(`Local destination directory does not exist: ${localDestination}`, {
           context: { localPath: options.localPath }
         });
@@ -851,7 +870,7 @@ var ScpNextClientImpl = class {
       return localDestination;
     }
     if (!await isExistingLocalDirectory(localDestination)) {
-      if (!options.createDirectories) {
+      if (!shouldCreateDirectories(options)) {
         throw new TransferError(`Local destination directory does not exist: ${localDestination}`, {
           context: { localPath: options.localPath }
         });
@@ -1117,7 +1136,7 @@ var DEFAULT_TRANSFER = {
   port: 22,
   recursive: false,
   overwrite: false,
-  createDirectories: false,
+  createDirectories: true,
   dryRun: false
 };
 function assignDefined(target, source) {
@@ -1307,6 +1326,7 @@ function addTransferOptions(command) {
   command.option("--recursive", "Transfer directories recursively");
   command.option("--overwrite", "Overwrite existing destination files");
   command.option("--create-directories", "Create missing destination directories");
+  command.option("--no-create-directories", "Do not create missing destination directories");
   command.option("--dry-run", "Resolve and validate without connecting or transferring");
   command.addOption(new Option("--timeout <milliseconds>", "Connection/operation timeout").argParser(parseInteger2));
   command.option("--verbose", "Print verbose diagnostic output");