octwin-cli 0.8.4 → 0.8.6

package/dist/lib/outcome.js

@@ -0,0 +1,79 @@
+/**
+ * `Outcome` — what one `octwin validate` check ACTUALLY did, as a value.
+ *
+ * ## Why this exists
+ *
+ * `cmdValidate` used to hand-write its own `✓` per check, inside the `if` that had the
+ * check's data, with a `skipped[]` array and a `skipReasons` Map maintained alongside. It
+ * was correct because someone was careful, and the next check added was one `if` away from
+ * lying again. It had lied before, twice, in the same shape:
+ *
+ *   - the KB-dependent block printed ONE `✓` above six checks whose own `✓`s lived inside
+ *     their `if`s, so a run with no pulled KB read as "one check, passed". An entire backlog
+ *     batch reached production that way;
+ *   - `yamlDocs()` parsed inside `try { … } catch { return [] }`, so a file that failed to
+ *     parse was silently DROPPED and every later check skipped it — `validate` printed
+ *     all-green over a pack the platform's importer then refused with the same message, from
+ *     the same `yaml` version this CLI already had.
+ *
+ * **An unparseable file is not "no findings", it is "no idea", and those must never print the
+ * same.** So a check no longer prints; it RETURNS one of three things, and a single printer
+ * renders all of them. A check that returns nothing cannot reach the `passed` branch, which
+ * makes the false-`✓` unrepresentable rather than merely discouraged.
+ *
+ * ## Why `not-run` carries the lookup rather than a message
+ *
+ * `describeKbLookup` phrases it at print time, so the wording lives in the module that owns
+ * the three KB states and cannot drift into a doubled "SKIPPED — SKIPPED —". It also lets the
+ * printer GROUP by reason: when the KB is absent every check skips for the identical reason,
+ * and six copies of one sentence is how a reader learns to scroll past the ⚠ block — the same
+ * failure as not printing it at all.
+ */
+import { describeKbLookup } from './kb-path.js';
+/**
+ * Run every check in order and print exactly what each returned.
+ *
+ * Returns the labels that could not run, so the caller can decide what a partial pass means
+ * (`--require-kb` fails on any; an interactive run says which, last, where it is read).
+ *
+ * The first `findings` outcome terminates via `die`, matching the previous behaviour: an
+ * author fixes one class at a time, and printing six screens of unrelated findings buries the
+ * first. Order therefore matters and is the caller's to choose.
+ */
+export function reportChecks(checks, io) {
+    const notRun = [];
+    for (const check of checks) {
+        const outcome = check.run();
+        switch (outcome.kind) {
+            case 'passed':
+                io.log(`✓ ${outcome.line}`);
+                break;
+            case 'findings': {
+                const n = outcome.lines.length;
+                io.err(`✗ ${n} ${outcome.noun}${n === 1 ? '' : 's'}:`);
+                for (const line of outcome.lines)
+                    io.err(`  ✗ ${line}`);
+                io.die(outcome.hint);
+            }
+            // eslint-disable-next-line no-fallthrough -- `die` returns never; the case above cannot exit.
+            case 'not-run':
+                notRun.push({ label: check.label, lookup: outcome.lookup });
+                break;
+        }
+    }
+    // One ⚠ per distinct reason, naming every check it cost. Grouped by the lookup's IDENTITY.
+    const byReason = new Map();
+    for (const { label, lookup } of notRun) {
+        const key = lookup.state === 'ok'
+            ? 'ok'
+            : `${lookup.state}|${'dir' in lookup ? lookup.dir : ''}|${'reason' in lookup ? lookup.reason : ''}`;
+        const bucket = byReason.get(key) ?? { lookup, labels: [] };
+        bucket.labels.push(label);
+        byReason.set(key, bucket);
+    }
+    for (const { lookup, labels } of byReason.values()) {
+        const what = labels.length === 1 ? labels[0] : `${labels.length} checks (${labels.join(', ')})`;
+        io.log(`⚠ ${describeKbLookup(lookup, what)}`);
+    }
+    return notRun.map(n => n.label);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "octwin-cli",
-  "version": "0.8.4",
+  "version": "0.8.6",
   "description": "Octwin external-pack developer CLI (by CEQUENS) — scaffold, validate, deploy, and check pure-YAML packs on your tenant.",
   "type": "module",
   "bin": {