@wyxos/zephyr 0.2.20 → 0.2.21

package/package.json

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

package/src/dependency-scanner.mjs

@@ -31,7 +31,7 @@ function isLocalPathOutsideRepo(depPath, rootDir) {
   // Check if resolved path is outside the repository root
   // Use path.relative to check if the path goes outside
   const relative = path.relative(normalizedRoot, normalizedResolved)
-  
+
   // If relative path starts with .., it's outside the repo
   // Also check if the resolved path doesn't start with the root + separator (for absolute paths)
   return relative.startsWith('..') || !normalizedResolved.startsWith(normalizedRoot + path.sep)
@@ -256,31 +256,8 @@ async function commitDependencyUpdates(rootDir, updatedFiles, promptFn, logFn) {
     return false
   }
 
-  const statusBefore = await getGitStatus(rootDir)
-
-  // Avoid accidentally committing unrelated staged changes
-  if (hasStagedChanges(statusBefore)) {
-    if (logFn) {
-      logFn('Staged changes detected. Skipping auto-commit of dependency updates.')
-    }
-    return false
-  }
-
   const fileList = updatedFiles.map((f) => path.basename(f)).join(', ')
 
-  const { shouldCommit } = await promptFn([
-    {
-      type: 'confirm',
-      name: 'shouldCommit',
-      message: `Commit dependency updates now? (${fileList})`,
-      default: true
-    }
-  ])
-
-  if (!shouldCommit) {
-    return false
-  }
-
   // Stage the updated files
   for (const file of updatedFiles) {
     try {
@@ -290,11 +267,6 @@ async function commitDependencyUpdates(rootDir, updatedFiles, promptFn, logFn) {
     }
   }
 
-  const newStatus = await getGitStatus(rootDir)
-  if (!hasStagedChanges(newStatus)) {
-    return false
-  }
-
   // Build commit message
   const commitMessage = `chore: update local file dependencies to online versions (${fileList})`
 
@@ -302,7 +274,7 @@ async function commitDependencyUpdates(rootDir, updatedFiles, promptFn, logFn) {
     logFn('Committing dependency updates...')
   }
 
-  await runCommand('git', ['commit', '-m', commitMessage], { cwd: rootDir })
+  await runCommand('git', ['commit', '-m', commitMessage, '--', ...updatedFiles], { cwd: rootDir })
 
   if (logFn) {
     logFn('Dependency updates committed.')
@@ -371,6 +343,9 @@ async function validateLocalDependencies(rootDir, promptFn, logFn = null) {
     throw new Error('Release cancelled: local file dependencies must be updated before release.')
   }
 
+  // If we cannot commit the update, do not proceed (otherwise release-node will fail later with a dirty tree).
+  // We allow users to opt-in to committing together with existing staged changes via prompt.
+
   // Track which files were updated
   const updatedFiles = new Set()
 
@@ -437,7 +412,12 @@ async function validateLocalDependencies(rootDir, promptFn, logFn = null) {
 
   // Commit the changes if any files were updated
   if (updatedFiles.size > 0) {
-    await commitDependencyUpdates(rootDir, Array.from(updatedFiles), promptFn, logFn)
+    const committed = await commitDependencyUpdates(rootDir, Array.from(updatedFiles), promptFn, logFn)
+    if (!committed) {
+      throw new Error(
+        'Release cancelled: dependency updates were applied but were not committed. Commit/stash your changes and rerun.'
+      )
+    }
   }
 }
 

package/src/release-node.mjs

@@ -126,13 +126,25 @@ async function runTests(skipTests, pkg, rootDir = process.cwd()) {
   logStep('Running test suite...')
 
   try {
+    const testRunScript = pkg?.scripts?.['test:run'] ?? ''
+    const testScript = pkg?.scripts?.test ?? ''
+    const usesNodeTest = (script) => /\bnode\b.*\s--test\b/.test(script)
+
     // Prefer test:run if available, otherwise use test with --run and --reporter flags
     if (hasScript(pkg, 'test:run')) {
-      // Pass reporter flag to test:run script
-      await runCommand('npm', ['run', 'test:run', '--', '--reporter=dot'], { cwd: rootDir })
+      if (usesNodeTest(testRunScript)) {
+        await runCommand('npm', ['run', 'test:run'], { cwd: rootDir })
+      } else {
+        // Pass reporter flag to test:run script
+        await runCommand('npm', ['run', 'test:run', '--', '--reporter=dot'], { cwd: rootDir })
+      }
     } else {
-      // For test script, pass --run and --reporter flags (works with vitest)
-      await runCommand('npm', ['test', '--', '--run', '--reporter=dot'], { cwd: rootDir })
+      if (usesNodeTest(testScript)) {
+        await runCommand('npm', ['test'], { cwd: rootDir })
+      } else {
+        // For test script, pass --run and --reporter flags (works with vitest)
+        await runCommand('npm', ['test', '--', '--run', '--reporter=dot'], { cwd: rootDir })
+      }
     }
 
     logSuccess('Tests passed.')
@@ -217,7 +229,13 @@ async function runLibBuild(skipBuild, pkg, rootDir = process.cwd()) {
   return hasLibChanges
 }
 
-async function ensureNpmAuth(rootDir = process.cwd()) {
+async function ensureNpmAuth(pkg, rootDir = process.cwd()) {
+  const isPrivate = pkg?.publishConfig?.access === 'restricted'
+  if (isPrivate) {
+    logStep('Skipping npm authentication check (package is private/restricted).')
+    return
+  }
+
   logStep('Confirming npm authentication...')
   try {
     const result = await runCommand('npm', ['whoami'], { capture: true, cwd: rootDir })
@@ -452,7 +470,7 @@ export async function releaseNode() {
     await runLint(skipLint, pkg, rootDir)
     await runTests(skipTests, pkg, rootDir)
     await runLibBuild(skipBuild, pkg, rootDir)
-    await ensureNpmAuth(rootDir)
+    await ensureNpmAuth(pkg, rootDir)
 
     const updatedPkg = await bumpVersion(releaseType, rootDir)
     await runBuild(skipBuild, updatedPkg, rootDir)