velocious 1.0.566 → 1.0.567

package/README.md

@@ -1180,6 +1180,8 @@ Evaluate inline JavaScript (Rails-style runner) with initialized app/database co
 npx velocious runner "const users = await db.query('SELECT COUNT(*) AS count FROM users'); console.log(users[0].count)"
 ```
 
+Successful CLI commands exit with status `0`. If command execution rejects, Velocious marks the process failed and always runs final database cleanup. Successful cleanup preserves and rethrows the original command error unchanged; if cleanup also fails, an `AggregateError` retains the command error first and as its `cause`, followed by the cleanup error. Cleanup-only failures also exit nonzero. These failures therefore remain nonzero even when application code installs an `uncaughtException` listener that reports and consumes the error. See [CLI process exit behavior](docs/cli.md#process-exit-behavior).
+
 By default, migrations write `db/structure-<identifier>.sql` files for each database in non-test environments. Test skips these automatic writes unless you explicitly opt in. Configure allow/deny lists in your configuration:
 
 ```js

package/bin/velocious.js

@@ -39,10 +39,37 @@ const cli = new Cli({
   processArgs
 })
 
+let commandError
+let commandFailed = false
+
 try {
   await cli.execute()
-} finally {
+} catch (error) {
+  commandError = error
+  commandFailed = true
+  process.exitCode = 1
+}
+
+let cleanupError
+let cleanupFailed = false
+
+try {
   await configuration.closeDatabaseConnections()
+} catch (error) {
+  cleanupError = error
+  cleanupFailed = true
+  process.exitCode = 1
+}
+
+if (commandFailed && cleanupFailed) {
+  throw new AggregateError(
+    [commandError, cleanupError],
+    "Velocious CLI command execution and database cleanup both failed",
+    {cause: commandError}
+  )
 }
 
+if (commandFailed) throw commandError
+if (cleanupFailed) throw cleanupError
+
 process.exit(0)

package/build/bin/velocious.js

@@ -39,10 +39,37 @@ const cli = new Cli({
   processArgs
 })
 
+let commandError
+let commandFailed = false
+
 try {
   await cli.execute()
-} finally {
+} catch (error) {
+  commandError = error
+  commandFailed = true
+  process.exitCode = 1
+}
+
+let cleanupError
+let cleanupFailed = false
+
+try {
   await configuration.closeDatabaseConnections()
+} catch (error) {
+  cleanupError = error
+  cleanupFailed = true
+  process.exitCode = 1
+}
+
+if (commandFailed && cleanupFailed) {
+  throw new AggregateError(
+    [commandError, cleanupError],
+    "Velocious CLI command execution and database cleanup both failed",
+    {cause: commandError}
+  )
 }
 
+if (commandFailed) throw commandError
+if (cleanupFailed) throw cleanupError
+
 process.exit(0)

package/package.json

@@ -3,7 +3,7 @@
     "velocious": "build/bin/velocious.js"
   },
   "name": "velocious",
-  "version": "1.0.566",
+  "version": "1.0.567",
   "main": "build/index.js",
   "types": "build/index.d.ts",
   "files": [