@invarn/cibuild 2.6.9 → 2.7.1

package/dist/src/yaml/step-validator.js

@@ -7,6 +7,7 @@
  */
 import { getStepExecutor, hasStep, getStepMetadata } from './steps/registry.js';
 import { EnvResolver } from './env-resolver.js';
+import { shellAssignedNames } from './shell-assignments.js';
 import { detectPlatformInfo } from './platform-detector.js';
 import { declaresMultiplatformApp } from './meta-helpers.js';
 // POSIX env vars the shell always provides — don't flag these as undeclared.
@@ -30,71 +31,6 @@ const POSIX_ENV_ALLOWLIST = new Set([
     'CIBUILD_SCRATCH',
     'JAVA_HOME',
 ]);
-/**
- * Names a shell snippet assigns to itself, in every form a step script uses.
- *
- * The auto-discovery below treats every `$UPPERCASE` it finds in a step's
- * inputs as a value the pipeline must supply. For a `script` step the inputs
- * *are* the shell body, so a variable the script computes reads exactly like
- * one it needs handed to it — and the run is refused for a value nothing was
- * ever supposed to provide.
- *
- * That is not hypothetical. A generated multiplatform pipeline whose prepare
- * step opens with
- *
- *     INTEGRATION="$KMM_IOS_INTEGRATION"
- *     IOS_DIR=$(dirname "$IOS_PROJECT_PATH")
- *     for WORKSPACE in "$IOS_DIR"/*.xcworkspace; do …
- *
- * was refused before step 1 on "3 required value(s) missing — INTEGRATION,
- * IOS_DIR, WORKSPACE", while the two variables it genuinely reads were both
- * declared and present. The shell was correct; the reader was not.
- *
- * Only the assignment *target* is collected, never the right-hand side, so
- * `INTEGRATION="$KMM_IOS_INTEGRATION"` still requires `KMM_IOS_INTEGRATION`.
- * That is what keeps this from becoming a hole: a script cannot excuse itself
- * from a value it reads by mentioning it.
- *
- * Assignment order is deliberately not modelled. A snippet that reads `$FOO`
- * before setting it is a bug in the snippet, and guessing about it here would
- * trade a false refusal for a different false refusal; "the script sets this
- * somewhere" is the question worth asking, and erring toward letting the build
- * run is the right direction for a check that cannot prompt.
- */
-export function shellAssignedNames(obj) {
-    const names = new Set();
-    const collect = (text) => {
-        // `NAME=…`, `export NAME=…`, `local/readonly/declare/typeset NAME=…`, and
-        // `NAME+=…`. Anchored to the start of a command — line start, or after a
-        // separator — so `--flag=X` and `foo BAR=1` inside a longer word are not
-        // mistaken for assignments.
-        const assignment = /(?:^|[\n;&|(]|\|\||&&)[ \t]*(?:(?:export|readonly|local|declare|typeset)[ \t]+)?([A-Z_][A-Z_0-9]*)\+?=/g;
-        // `for NAME in …`, `select NAME in …`
-        const loop = /(?:^|[\n;&|(])[ \t]*(?:for|select)[ \t]+([A-Z_][A-Z_0-9]*)[ \t]+in\b/g;
-        // `read NAME`, `read -r NAME OTHER`, `while read -r NAME; do`
-        const read = /(?:^|[\n;&|(]|\|\||&&)[ \t]*(?:while[ \t]+)?read[ \t]+((?:-\S+[ \t]+)*)([A-Z_][A-Z_0-9]*(?:[ \t]+[A-Z_][A-Z_0-9]*)*)/g;
-        for (const re of [assignment, loop]) {
-            let m;
-            while ((m = re.exec(text)) !== null)
-                names.add(m[1]);
-        }
-        let m;
-        while ((m = read.exec(text)) !== null) {
-            for (const n of m[2].trim().split(/[ \t]+/))
-                names.add(n);
-        }
-    };
-    const walk = (value) => {
-        if (typeof value === 'string')
-            collect(value);
-        else if (Array.isArray(value))
-            value.forEach(walk);
-        else if (value && typeof value === 'object')
-            Object.values(value).forEach(walk);
-    };
-    walk(obj);
-    return names;
-}
 /**
  * StepValidator orchestrates pre-execution validation for a workflow.
  * It iterates through all steps, collects validation requirements,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invarn/cibuild",
-  "version": "2.6.9",
+  "version": "2.7.1",
   "description": "CI Build CLI — local pipeline orchestration and validation",
   "type": "module",
   "main": "dist/cli.cjs",