@rpcbase/cli 0.122.0 → 0.124.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpcbase/cli",
-  "version": "0.122.0",
+  "version": "0.124.0",
   "type": "module",
   "main": "./index.js",
   "bin": {

package/src/cmd-deploy/detect-compose-changes.js

@@ -76,10 +76,12 @@ const runRsync = (cmd) => {
 }
 
 // Pull remote compose files into a temp dir, mirror local ones into another, then hash trees to detect diffs.
-// This is informational only; callers should treat errors as non-blocking.
-export const detectComposeChanges = ({ keyPath, user, host, deployDir, log, extraExcludes = [] }) => {
+// Callers await this check; errors bubble up except for benign rsync statuses handled in runRsync.
+export const detectComposeChanges = async ({ keyPath, user, host, deployDir, log, extraExcludes = [] }) => {
   const includeArgs = ["--include='*/'", "--include='*compose*.yml'", "--exclude='*'"]
   const excludeArgs = baseExcludeList
+    // dev compose files are not relevant for prod deploy checks
+    .concat(["compose.dev.yml"])
     .concat(extraExcludes)
     .map((p) => `--exclude='${p}'`)
     .join(" ")

package/src/cmd-deploy/index.js

@@ -88,22 +88,14 @@ export const deploy = async (argv) => {
     console.log(`${user}@${host}`, res)
     await ssh(`mkdir -p ~/apps/${deployDir}`)
 
-    let infrastructureChanged = false
-    let infrastructureDiff = ""
-    try {
-      ({ hasChanges: infrastructureChanged, output: infrastructureDiff } = detectComposeChanges({
-        keyPath,
-        user,
-        host,
-        deployDir,
-        log,
-        extraExcludes: argv.ignore || [],
-      }))
-    } catch (diffError) {
-      console.warn(
-        `Compose diff check skipped (non-blocking): ${diffError instanceof Error ? diffError.message : diffError}`,
-      )
-    }
+    const { hasChanges: infrastructureChanged, output: infrastructureDiff } = await detectComposeChanges({
+      keyPath,
+      user,
+      host,
+      deployDir,
+      log,
+      extraExcludes: argv.ignore || [],
+    })
 
     if (infrastructureChanged) {
       console.log("Infrastructure compose files differ between local and remote (info only; deploy proceeds).")