@solidactions/cli 1.22.0 → 1.23.0

package/README.md

@@ -115,7 +115,7 @@ Use `solidactions <command> --help` for full flag details on any command.
 
 | Command | Key Flags | Description |
 |---------|-----------|-------------|
-| `env set <key> <value>` | `-s`, `-y`, `--staging-value`, `--dev-value` | Set global variable (warns before overwriting) |
+| `env set <key> <value> --global` | `-s`, `-y`, `--staging-value`, `--dev-value`, `--global` | Set global variable (warns before overwriting) |
 | `env set <project> <key> <value>` | `-e`, `-s`, `-y` | Set project variable (warns before overwriting) |
 | `env list [project]` | `-e` | List variables |
 | `env delete <key-or-project> [key]` | `-y` | Delete a variable |

package/dist/commands/env-map.js

@@ -25,7 +25,7 @@ async function envMap(projectName, projectKey, globalKey, options = {}) {
         const globalVar = variables.find((v) => v.key === globalKey);
         if (!globalVar) {
             console.error(chalk_1.default.red(`Global variable "${globalKey}" not found.`));
-            console.log(chalk_1.default.gray('Create it with: solidactions env set ' + globalKey + ' <value>'));
+            console.log(chalk_1.default.gray('Create it with: solidactions env set ' + globalKey + ' <value> --global'));
             process.exit(1);
         }
         // Check if project key already has a mapping

package/dist/commands/env-set.js

@@ -17,15 +17,33 @@ function isNonTty() {
 }
 /** Printed before a 2-arg (global-scope) write — Jordan's runs 3-7 footgun. */
 exports.GLOBAL_ENV_SCOPE_NOTE = [
-    'Note: no project specified — creating a GLOBAL variable.',
+    'Note: --global specified — creating a GLOBAL variable.',
     "  Global variables are NOT visible to a project's plain `env:` YAML declarations",
     '  unless you map them (`solidactions env map …`).',
     '  For a project variable, use: solidactions env set <project> KEY value',
 ].join('\n');
 async function envSet(keyOrProject, valueOrKey, valueIfProject, options = {}) {
-    const config = await (0, api_1.requireConfigWithWorkspace)();
     // Detect mode based on arguments
     const isProjectMode = valueIfProject !== undefined;
+    // Jordan Wall #4 (Sweep C, → 1.23.0): a 2-positional-arg call used to fall
+    // through to GLOBAL scope silently, so a typo'd `env set KEY VALUE` (meant
+    // for a project) wrote a global var the workflow never reads. Global
+    // writes now require an explicit --global. Runs before any
+    // config/network work so a plain argument mistake never needs a login.
+    if (!isProjectMode && !options.global) {
+        process.stderr.write(chalk_1.default.red('env set needs either a project or --global — pick one:\n' +
+            '  solidactions env set KEY VALUE --global      (global variable)\n' +
+            '  solidactions env set <project> KEY VALUE     (project variable)\n'));
+        process.exit(1);
+    }
+    // --global never takes a project positional — reject the ambiguous combo too.
+    if (isProjectMode && options.global) {
+        process.stderr.write(chalk_1.default.red('env set: --global does not take a project argument — pick one:\n' +
+            '  solidactions env set KEY VALUE --global      (global variable)\n' +
+            '  solidactions env set <project> KEY VALUE     (project variable)\n'));
+        process.exit(1);
+    }
+    const config = await (0, api_1.requireConfigWithWorkspace)();
     if (isProjectMode) {
         // Project mode: solidactions env set <project> <key> <value>
         const projectName = keyOrProject;

package/dist/index.js

@@ -244,6 +244,7 @@ env
     .argument('[value]', 'Variable value (when first arg is project)')
     .option('-s, --secret', 'Mark as encrypted secret')
     .option('-e, --env <environment>', 'Target environment (production/staging/dev)', 'dev')
+    .option('--global', 'Set a global variable (no project) — required for the 2-arg form')
     .option('--staging-value <value>', 'Staging environment value (global only)')
     .option('--dev-value <value>', 'Dev environment value (global only)')
     .option('--staging-inherit', 'Staging inherits from production (global only)')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solidactions/cli",
-  "version": "1.22.0",
+  "version": "1.23.0",
   "description": "SolidActions CLI - Deploy and manage workflow automation",
   "main": "dist/index.js",
   "bin": {