@nmakarov/cli-toolkit 0.57.0 → 0.59.0

package/dist/index.js

@@ -3545,6 +3545,12 @@ var Db = class _Db {
       }
       if (!dbConnectionString) {
         if (!dbName) {
+          const env = context?.args?.env;
+          if (env && String(env).toLowerCase() !== "local") {
+            throw new ParamError(
+              `Db: dbName not set for env="${env}" (set --dbName or DB_NAME_${String(env).toUpperCase()})`
+            );
+          }
           dbName = "local";
         }
         const paramName = `dbConnectionString${capitalizeFirstLetter(dbName)}`;
@@ -5664,7 +5670,10 @@ async function reloadPm2(paths, options = {}) {
       logger.info(`pm2 ${appName}: reloading (old pid=${oldPid})`);
     }
   }
-  await runShell(`pm2 startOrReload "${paths.ecosystem}" --update-env`, { logger });
+  await runShell(
+    `ENV=production NODE_ENV=production pm2 startOrReload "${paths.ecosystem}" --update-env`,
+    { logger }
+  );
   logger.info("pm2 reloaded");
   if (appName) {
     const proc = await waitPm2(
@@ -5713,7 +5722,10 @@ async function startPm2(paths, options = {}) {
     logger.info(`[dryRun] would pm2 start ${paths.ecosystem} --update-env`);
     return;
   }
-  await runShell(`pm2 start "${paths.ecosystem}" --update-env`, { logger });
+  await runShell(
+    `ENV=production NODE_ENV=production pm2 start "${paths.ecosystem}" --update-env`,
+    { logger }
+  );
   logger.info("pm2 started");
   if (appName) {
     const proc = await waitPm2(
@@ -5968,11 +5980,12 @@ function resolvePm2Args(pm2) {
 function buildEnsureEnvScript() {
   return `/**
  * Written by cli-toolkit deploy (init-structure). Do not hand-edit.
- * Preloaded via pm2 node_args --require so Args sees ENV=production even when
- * a post-SIGKILL restart briefly omits ecosystem env (otherwise Db \u2192 local:6032).
+ * Preloaded via pm2 node_args --require. Always set (do not only fill when unset):
+ * \`pm2 startOrReload --update-env\` can inject a non-production ENV from the
+ * deploy shell / prior dump, and a conditional assign would leave it in place.
  */
-if (!process.env.ENV) process.env.ENV = "production";
-if (!process.env.NODE_ENV) process.env.NODE_ENV = "production";
+process.env.ENV = "production";
+process.env.NODE_ENV = "production";
 `;
 }
 function buildEcosystemConfig(service, paths) {