@wyxos/zephyr 0.2.5 → 0.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wyxos/zephyr",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "A streamlined deployment tool for web applications with intelligent Laravel project detection",
   "type": "module",
   "main": "./src/index.mjs",

package/src/release-node.mjs

@@ -221,15 +221,43 @@ async function runTests(skipTests, pkg, rootDir = process.cwd()) {
 
   logStep('Running test suite...')
 
-  // Prefer test:run if available, otherwise use test with --run flag
-  if (hasScript(pkg, 'test:run')) {
-    await runCommand('npm', ['run', 'test:run'], { cwd: rootDir })
-  } else {
-    // For test script, try to pass --run flag (works with vitest)
-    await runCommand('npm', ['test', '--', '--run'], { cwd: rootDir })
-  }
+  let dotInterval = null
+  try {
+    // Capture output and show dots as progress
+    process.stdout.write('  ')
+    dotInterval = setInterval(() => {
+      process.stdout.write('.')
+    }, 200)
+
+    // Prefer test:run if available, otherwise use test with --run flag
+    if (hasScript(pkg, 'test:run')) {
+      await runCommand('npm', ['run', 'test:run'], { capture: true, cwd: rootDir })
+    } else {
+      // For test script, try to pass --run flag (works with vitest)
+      await runCommand('npm', ['test', '--', '--run'], { capture: true, cwd: rootDir })
+    }
 
-  logSuccess('Tests passed.')
+    if (dotInterval) {
+      clearInterval(dotInterval)
+      dotInterval = null
+    }
+    process.stdout.write('\n')
+    logSuccess('Tests passed.')
+  } catch (error) {
+    // Clear dots and show error output
+    if (dotInterval) {
+      clearInterval(dotInterval)
+      dotInterval = null
+    }
+    process.stdout.write('\n')
+    if (error.stdout) {
+      console.error(error.stdout)
+    }
+    if (error.stderr) {
+      console.error(error.stderr)
+    }
+    throw error
+  }
 }
 
 async function runBuild(skipBuild, pkg, rootDir = process.cwd()) {
@@ -337,7 +365,10 @@ async function publishPackage(pkg, rootDir = process.cwd()) {
   const publishArgs = ['publish', '--ignore-scripts'] // Skip prepublishOnly since we already built lib
 
   if (pkg.name.startsWith('@')) {
-    publishArgs.push('--access', 'public')
+    // For scoped packages, determine access level from publishConfig
+    // Default to 'public' for scoped packages if not specified (free npm accounts require public for scoped packages)
+    const access = pkg.publishConfig?.access || 'public'
+    publishArgs.push('--access', access)
   }
 
   logStep(`Publishing ${pkg.name}@${pkg.version} to npm...`)

package/src/ssh-utils.mjs

@@ -92,11 +92,11 @@ export async function connectToServer(config, rootDir) {
  * @param {NodeSSH} ssh - SSH client instance
  * @param {string} label - Human-readable label for the command
  * @param {string} command - Command to execute
- * @param {Object} options - Options: { cwd, allowFailure, printStdout, bootstrapEnv, rootDir, writeToLogFile }
+ * @param {Object} options - Options: { cwd, allowFailure, printStdout, bootstrapEnv, rootDir, writeToLogFile, env }
  * @returns {Promise<Object>} Command result
  */
 export async function executeRemoteCommand(ssh, label, command, options = {}) {
-  const { cwd, allowFailure = false, printStdout = true, bootstrapEnv = true, rootDir = null, writeToLogFile = null } = options
+  const { cwd, allowFailure = false, printStdout = true, bootstrapEnv = true, rootDir = null, writeToLogFile = null, env = {} } = options
 
   logProcessing(`\n→ ${label}`)
 
@@ -120,14 +120,27 @@ export async function executeRemoteCommand(ssh, label, command, options = {}) {
   ].join('; ')
 
   const escapeForDoubleQuotes = (value) => value.replace(/(["\\$`])/g, '\\$1')
+  const escapeForSingleQuotes = (value) => value.replace(/'/g, "'\\''")
+
+  // Build environment variable exports
+  let envExports = ''
+  if (Object.keys(env).length > 0) {
+    const envPairs = Object.entries(env).map(([key, value]) => {
+      const escapedValue = escapeForSingleQuotes(String(value))
+      return `${key}='${escapedValue}'`
+    })
+    envExports = envPairs.join(' ') + ' '
+  }
 
   let wrappedCommand = command
   let execOptions = { cwd }
 
   if (bootstrapEnv && cwd) {
     const cwdForShell = escapeForDoubleQuotes(cwd)
-    wrappedCommand = `${profileBootstrap}; cd "${cwdForShell}" && ${command}`
+    wrappedCommand = `${profileBootstrap}; cd "${cwdForShell}" && ${envExports}${command}`
     execOptions = {}
+  } else if (Object.keys(env).length > 0) {
+    wrappedCommand = `${envExports}${command}`
   }
 
   const result = await ssh.execCommand(wrappedCommand, execOptions)
@@ -191,7 +204,13 @@ export async function readRemoteFile(ssh, filePath, remoteCwd) {
 }
 
 /**
- * Download file from remote server via SFTP
+ * Download file from remote server via SFTP with progress
+ * 
+ * Note: Currently uses single-stream download (most reliable).
+ * Multi-streaming is technically possible with ssh2-sftp-client's fastGet,
+ * but it's unreliable on many servers and can cause data corruption.
+ * Single-stream ensures data integrity at the cost of potentially slower speeds.
+ * 
  * @param {NodeSSH} ssh - SSH client instance
  * @param {string} remotePath - Path to file on remote server
  * @param {string} localPath - Local path to save file
@@ -206,8 +225,26 @@ export async function downloadRemoteFile(ssh, remotePath, localPath, remoteCwd)
 
   logProcessing(`Downloading ${absoluteRemotePath} to ${localPath}...`)
 
-  await ssh.getFile(localPath, absoluteRemotePath)
+  let transferred = 0
+  const startTime = Date.now()
+
+  // Single-stream download (most reliable for data integrity)
+  await ssh.getFile(localPath, absoluteRemotePath, null, {
+    step: (totalTransferred, chunk, total) => {
+      transferred = totalTransferred
+      const percent = total > 0 ? Math.round((transferred / total) * 100) : 0
+      const elapsed = (Date.now() - startTime) / 1000
+      const speed = elapsed > 0 ? (transferred / elapsed / 1024 / 1024).toFixed(2) : 0
+      const sizeMB = (transferred / 1024 / 1024).toFixed(2)
+      const totalMB = total > 0 ? (total / 1024 / 1024).toFixed(2) : '?'
+
+      // Update progress on same line
+      process.stdout.write(`\r  Progress: ${percent}% (${sizeMB}MB / ${totalMB}MB) - ${speed} MB/s`)
+    }
+  })
 
+  // Clear progress line and show completion
+  process.stdout.write('\r' + ' '.repeat(80) + '\r')
   logSuccess(`Downloaded ${absoluteRemotePath} to ${localPath}`)
 }