@visualautomate/cli 0.5.1 → 0.5.2

package/README.md

@@ -59,6 +59,10 @@ state    {"runs":1}
 A run passes when it succeeds and, if the test file names an `expectOutput`,
 leaves by that port — the same verdict as the GitHub check.
 
+The verdict is green when it passed and red when it did not, in the sandbox as
+well as out here. `NO_COLOR` turns that off, and output that is not a terminal —
+a pipe, a file, a CI log — is plain anyway; `FORCE_COLOR` turns it back on.
+
 ### In a GitHub repository
 
 `test` works wherever the platform tests your code on a push:

package/dist/index.js

@@ -1823,12 +1823,12 @@ function entryIn(dir) {
   return ENTRIES.map((file2) => join(dir, file2)).find((file2) => existsSync(file2)) ?? null;
 }
 function matchFolders(folders, name) {
-  const wanted = name.trim().toLowerCase();
-  const exact = folders.filter((folder) => folder.toLowerCase() === wanted);
+  const wanted2 = name.trim().toLowerCase();
+  const exact = folders.filter((folder) => folder.toLowerCase() === wanted2);
   if (exact.length > 0) return exact;
-  const prefix = folders.filter((folder) => folder.toLowerCase().startsWith(wanted));
+  const prefix = folders.filter((folder) => folder.toLowerCase().startsWith(wanted2));
   if (prefix.length > 0) return prefix;
-  return folders.filter((folder) => folder.toLowerCase().includes(wanted));
+  return folders.filter((folder) => folder.toLowerCase().includes(wanted2));
 }
 function findTargets(cwd, only) {
   const dir = resolve(cwd);
@@ -2087,6 +2087,38 @@ async function savePersisted(dir, files, state) {
 
 // src/docker.ts
 import { spawnSync } from "node:child_process";
+
+// src/colour.ts
+var CODES = {
+  reset: "\x1B[0m",
+  bold: "\x1B[1m",
+  dim: "\x1B[2m",
+  red: "\x1B[31m",
+  green: "\x1B[32m",
+  yellow: "\x1B[33m",
+  blue: "\x1B[34m"
+};
+function wanted() {
+  const env = process.env;
+  if (env.NO_COLOR !== void 0 && env.NO_COLOR !== "") return false;
+  if (env.FORCE_COLOR !== void 0 && env.FORCE_COLOR !== "" && env.FORCE_COLOR !== "0") return true;
+  return Boolean(process.stdout.isTTY);
+}
+var ON = wanted();
+function paint(code) {
+  return (text) => ON ? `${code}${text}${CODES.reset}` : text;
+}
+var red = paint(CODES.red);
+var green = paint(CODES.green);
+var yellow = paint(CODES.yellow);
+var blue = paint(CODES.blue);
+var dim = paint(CODES.dim);
+var bold = paint(CODES.bold);
+function colourEnv() {
+  return ON ? ["--env", "FORCE_COLOR=1"] : ["--env", "NO_COLOR=1"];
+}
+
+// src/docker.ts
 var IMAGE = "ghcr.io/visualautomate/plugin-sandbox";
 var TIERS = {
   standard: { cpus: "0.25", memory: "1g" },
@@ -2119,6 +2151,9 @@ function dockerArgs(run) {
     "ALL",
     "--security-opt",
     "no-new-privileges",
+    // So a run inside the container is as readable as one out here: its
+    // stdout is a pipe to Docker, which would otherwise turn colour off.
+    ...colourEnv(),
     ...run.user ? ["--user", `${run.user.uid}:${run.user.gid}`] : [],
     run.image,
     ...run.args
@@ -2142,16 +2177,18 @@ function dockerProblem() {
 function explainExit(code, tier) {
   if (code === 137) {
     const next = Object.keys(TIERS)[Object.keys(TIERS).indexOf(tier) + 1];
-    return `\u2717 stopped   the step used more than the ${TIERS[tier].memory.replace("g", " GB")} of memory the ${tier} tier has, and the container was killed. On the platform it would stop the same way.` + (next ? ` To check it fits a bigger machine: --tier ${next}` : "");
+    return red(
+      `\u2717 stopped   the step used more than the ${TIERS[tier].memory.replace("g", " GB")} of memory the ${tier} tier has, and the container was killed. On the platform it would stop the same way.` + (next ? ` To check it fits a bigger machine: --tier ${next}` : "")
+    );
   }
-  if (code === 125) return "\u2717 Docker could not start the container. Is the image available? Try: docker pull " + IMAGE;
+  if (code === 125) return red("\u2717 Docker could not start the container. Is the image available? Try: docker pull " + IMAGE);
   return null;
 }
 function ensureDocker(say, waitMs = 12e4) {
   const problem = dockerProblem();
   if (!problem) return null;
   if (problem.includes("not installed")) return problem;
-  say("Docker is not running. Starting Docker Desktop\u2026");
+  say(yellow("Docker is not running. Starting Docker Desktop\u2026"));
   const started = spawnSync("docker", ["desktop", "start"], { encoding: "utf8" });
   if (started.status !== 0) {
     say("Could not start it from here \u2014 start Docker Desktop yourself; this will carry on when it is up.");
@@ -2185,7 +2222,7 @@ function runInDocker(run) {
 }
 
 // src/index.ts
-var VERSION = true ? "0.5.1" : "0.0.0-dev";
+var VERSION = true ? "0.5.2" : "0.0.0-dev";
 var DEFAULT_API_URL = "https://visualautomate.com";
 function configPath() {
   return join3(homedir(), ".visualautomate", "config.json");
@@ -2212,7 +2249,7 @@ async function tokenFor(flags) {
   return { token, apiUrl };
 }
 function fail(message) {
-  console.error(message);
+  console.error(red(message));
   throw new SilentExit();
 }
 var SilentExit = class extends Error {
@@ -2289,8 +2326,8 @@ async function cmdLogin(flags) {
   });
   const who = await whoami(apiUrl, token);
   await writeConfig({ token, apiUrl, email: who.email });
-  ok(`
-\u2713 Signed in as ${who.email}`);
+  ok(green(`
+\u2713 Signed in as ${who.email}`));
   ok(`  Token stored in ${configPath()}`);
 }
 async function whoami(apiUrl, token) {
@@ -2376,9 +2413,9 @@ visualautomate push     # publish it to your account
 \`test.json\` is the input, config and state \`test\` runs with.
 `
   );
-  ok(`
+  ok(green(`
 \u2713 Created ${name} from the "${templateName}" template
-`);
+`));
   ok("  cd " + name);
   ok("  visualautomate test");
   ok("  visualautomate push\n");
@@ -2412,12 +2449,12 @@ function line(indent, text) {
 async function runTarget(target, flags, several) {
   const say = (text) => line(several, text);
   if (several) ok(`
-\u25B8 ${target.name}`);
+${bold(`\u25B8 ${target.name}`)}`);
   try {
     const code = await codeFor(target);
     const check = validatePluginCode(code);
     if (!check.valid) {
-      say(`\u2717 refused   ${check.error}`);
+      say(`${red("\u2717 refused")}   ${check.error}`);
       return false;
     }
     const settings = await readTestFile(target.dir, flags.input);
@@ -2431,7 +2468,7 @@ async function runTarget(target, flags, several) {
         const manifest = parseJsonc(await readFile3(manifestPath, "utf8")).value;
         outputs = manifest.outputs ?? [];
       } catch (error) {
-        say(`\u2717 manifest.json is not valid JSON: ${error instanceof Error ? error.message : String(error)}`);
+        say(red(`\u2717 manifest.json is not valid JSON: ${error instanceof Error ? error.message : String(error)}`));
         return false;
       }
     }
@@ -2457,33 +2494,37 @@ async function runTarget(target, flags, several) {
     const port = result.output?.output ?? null;
     const expected = settings.expectOutput;
     const passed = result.status === "success" && (!expected || port === expected);
-    say(`${passed ? "\u2713 passed" : "\u2717 failed"}  ${result.status}, ${result.durationMs}ms`);
-    if (result.error) say(`error    ${result.error}`);
+    const verdict = passed ? green(bold("\u2713 passed")) : red(bold("\u2717 failed"));
+    say(`${verdict}  ${dim(`${result.status}, ${result.durationMs}ms`)}`);
+    if (result.error) say(`${dim("error")}    ${red(result.error)}`);
     if (result.output) {
-      say(`output   ${port}${expected ? ` (expected ${expected})` : ""}`);
-      say(`data     ${JSON.stringify(result.output.data, null, 2).replace(/\n/g, "\n         ")}`);
+      const port_ = passed || !expected ? port : red(String(port));
+      say(`${dim("output")}   ${port_}${expected ? dim(` (expected ${expected})`) : ""}`);
+      say(`${dim("data")}     ${JSON.stringify(result.output.data, null, 2).replace(/\n/g, "\n         ")}`);
     }
-    if (result.files.size > 0) say(`files    ${[...result.files.keys()].join(", ")}`);
-    if (result.state.size > 0) say(`state    ${JSON.stringify(Object.fromEntries(result.state))}`);
-    for (const entry of result.logs ?? []) say(`log      ${entry}`);
+    if (result.files.size > 0) say(`${dim("files")}    ${[...result.files.keys()].join(", ")}`);
+    if (result.state.size > 0) say(`${dim("state")}    ${JSON.stringify(Object.fromEntries(result.state))}`);
+    for (const entry of result.logs ?? []) say(`${dim("log")}      ${entry}`);
     for (const [host, enforced] of denied) {
       say(
-        enforced ? `egress   blocked ${host} \u2014 not in allowedDomains` : `! egress ${host} is not allowed on the platform unless the plugin declares it. Add "allowedDomains": ["${host}"] to the test file to check with the same rule.`
+        enforced ? `${dim("egress")}   ${red(`blocked ${host}`)} \u2014 not in allowedDomains` : yellow(`! egress ${host} is not allowed on the platform unless the plugin declares it. `) + `Add "allowedDomains": ["${host}"] to the test file to check with the same rule.`
       );
     }
     if (persist) {
       const refused = await savePersisted(target.dir, result.files, result.state);
-      say(`kept     storage and state in ${relative3(process.cwd(), join3(target.dir, ".visualautomate")) || ".visualautomate"}`);
-      for (const path of refused) say(`! storage path "${path}" points outside the storage folder and was not written`);
+      say(`${dim("kept")}     ${dim(`storage and state in ${relative3(process.cwd(), join3(target.dir, ".visualautomate")) || ".visualautomate"}`)}`);
+      for (const path of refused) say(yellow(`! storage path "${path}" points outside the storage folder and was not written`));
     }
     if (port && outputs.length > 0 && !outputs.includes(port)) {
       say(
-        `! This returned output "${port}", which manifest.json does not list (it has ${outputs.map((o) => `"${o}"`).join(", ")}). On a canvas nothing would follow it.`
+        yellow(
+          `! This returned output "${port}", which manifest.json does not list (it has ${outputs.map((o) => `"${o}"`).join(", ")}). On a canvas nothing would follow it.`
+        )
       );
     }
     return passed;
   } catch (error) {
-    say(`\u2717 ${error instanceof Error ? error.message : String(error)}`);
+    say(red(`\u2717 ${error instanceof Error ? error.message : String(error)}`));
     return false;
   }
 }
@@ -2492,8 +2533,11 @@ async function runTargets(targets, flags) {
   for (const target of targets) {
     if (await runTarget(target, flags, targets.length > 1)) passed++;
   }
-  if (targets.length > 1) ok(`
-${passed} of ${targets.length} passed`);
+  if (targets.length > 1) {
+    const tally = `${passed} of ${targets.length} passed`;
+    ok(`
+${passed === targets.length ? green(bold(tally)) : red(bold(tally))}`);
+  }
   return passed === targets.length;
 }
 function packageLoader(dir) {
@@ -2559,7 +2603,7 @@ function sayWhatRuns(targets, picked) {
     return;
   }
   ok(`Running ${targets.length} modules: ${targets.map((t) => t.name.replace(/^modules\//, "")).join(", ")}`);
-  if (!picked) ok(`One of them: vsa dev ${targets[0].name.replace(/^modules\//, "")}`);
+  if (!picked) ok(dim(`One of them: vsa dev ${targets[0].name.replace(/^modules\//, "")}`));
 }
 async function cmdTest(args, flags) {
   const targets = targetsOrFail(args, flags);
@@ -2575,11 +2619,11 @@ async function cmdDev(args, flags) {
   if (inSandbox) prepareSandbox(flags);
   const header = () => {
     process.stdout.write("\x1Bc");
-    ok(`vsa dev \u2014 ${(/* @__PURE__ */ new Date()).toLocaleTimeString()} \u2014 Ctrl+C to stop`);
+    ok(`${bold("vsa dev")} ${dim(`\u2014 ${(/* @__PURE__ */ new Date()).toLocaleTimeString()} \u2014 Ctrl+C to stop`)}`);
   };
   const run = async (chosen) => inSandbox ? runTargetsInDocker(chosen, targets, flags) : runTargets(chosen, flags);
   header();
-  ok(inSandbox ? `in ${imageFor(flags)}, tier ${flags.tier || "standard"}` : "on this machine, not in the sandbox");
+  ok(dim(inSandbox ? `in ${imageFor(flags)}, tier ${flags.tier || "standard"}` : "on this machine, not in the sandbox"));
   sayWhatRuns(targets, Boolean(picked));
   await run(targets);
   const ignored = [`${sep2}node_modules${sep2}`, `${sep2}.git${sep2}`, `${sep2}dist${sep2}`, `${sep2}.visualautomate${sep2}`];
@@ -2598,14 +2642,14 @@ async function cmdDev(args, flags) {
       targets = findTargets(root, picked);
     } catch (error) {
       header();
-      ok(`\u2717 ${error instanceof Error ? error.message : String(error)}`);
+      ok(red(`\u2717 ${error instanceof Error ? error.message : String(error)}`));
       running = false;
       return;
     }
     const hit = changed.map((path) => targetForPath(targets, path));
     const chosen = hit.some((t) => t === null) ? targets : [...new Set(hit)];
     header();
-    ok(`changed: ${changed.map((p) => relative3(root, p)).join(", ")}`);
+    ok(dim(`changed: ${changed.map((p) => relative3(root, p)).join(", ")}`));
     await run(chosen);
     running = false;
   };
@@ -2707,8 +2751,8 @@ ${text.slice(0, 500)}`);
   if (!res.ok || !data.success) {
     fail(`Push refused: ${data.error ?? `status ${res.status}`}`);
   }
-  ok(`
-\u2713 Published (id: ${data.pluginId})`);
+  ok(green(`
+\u2713 Published (id: ${data.pluginId})`));
   ok(`  A version snapshot was saved, so you can roll back from the dashboard.`);
 }
 function printHelp() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@visualautomate/cli",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Build and publish VisualAutomate plugins.",
   "license": "MIT",
   "type": "module",