@aloud/runner 0.2.2 → 0.2.3

package/dist/cli.js

@@ -2666,7 +2666,7 @@ function alive(pid) {
 }
 
 // src/version.ts
-var RUNNER_VERSION = "0.2.2";
+var RUNNER_VERSION = "0.2.3";
 var RUNNER_VERSION_HEADER = "x-aloud-runner-version";
 
 // src/protocol/client.ts
@@ -7658,6 +7658,25 @@ async function login(argv) {
   const server = stringOption(argv, "--server") ?? DEFAULT_SERVER;
   let token = stringOption(argv, "--token");
   if (!token) {
+    if (!process.stdin.isTTY) {
+      process.stderr.write(
+        [
+          "",
+          "There is no terminal attached here, so there is nowhere to paste a token.",
+          "",
+          "If you are an agent: stop and hand this back. The person runs `aloud login` in their",
+          "own terminal and pastes the token at the prompt. Do not ask them to paste it to you.",
+          "",
+          "To connect without a terminal, set both of these instead and skip login entirely:",
+          `  export ALOUD_SERVER=${server}`,
+          "  export ALOUD_RUNNER_TOKEN=utar_...",
+          "",
+          `A token comes from ${server}/app/settings/runners and is shown once.`,
+          ""
+        ].join("\n")
+      );
+      return 1;
+    }
     process.stdout.write(`
 Open ${server}/app/settings/runners and create a runner.
 `);
@@ -7735,7 +7754,12 @@ async function setup() {
     steps.push(`npm install -g @aloud/runner@latest    # ${RUNNER_VERSION} is installed, ${latest} is out`);
   }
   if (!credentials) {
-    steps.push("aloud login    # prompts for a token. The person, not you, pastes it in here.");
+    steps.push(
+      "aloud login    # needs a terminal. If you are an agent, hand this step to the person:"
+    );
+    steps.push(
+      "               # they run it themselves and paste the token at the prompt."
+    );
   }
   if (!running) {
     steps.push("aloud start    # in a background shell. It never exits.");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aloud/runner",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Run Aloud usability studies in a real browser on your own machine, so a study can reach localhost and anything else behind your network.",
   "license": "ISC",
   "repository": {

package/src/cli.ts

@@ -106,6 +106,30 @@ async function login(argv: readonly string[]): Promise<number> {
   let token = stringOption(argv, "--token");
 
   if (!token) {
+    // An agent runs commands in a shell with nothing attached to stdin, so the prompt below would
+    // read end-of-file and the token would come back empty. That used to surface as "that does not
+    // look like a runner token", which blames the wrong thing and tells nobody what to do. This is
+    // the single step of setup that a person has to perform, so it is worth saying so precisely.
+    if (!process.stdin.isTTY) {
+      process.stderr.write(
+        [
+          "",
+          "There is no terminal attached here, so there is nowhere to paste a token.",
+          "",
+          "If you are an agent: stop and hand this back. The person runs `aloud login` in their",
+          "own terminal and pastes the token at the prompt. Do not ask them to paste it to you.",
+          "",
+          "To connect without a terminal, set both of these instead and skip login entirely:",
+          `  export ALOUD_SERVER=${server}`,
+          "  export ALOUD_RUNNER_TOKEN=utar_...",
+          "",
+          `A token comes from ${server}/app/settings/runners and is shown once.`,
+          "",
+        ].join("\n"),
+      );
+      return 1;
+    }
+
     process.stdout.write(`\nOpen ${server}/app/settings/runners and create a runner.\n`);
     process.stdout.write("It shows you a token once. Paste it here.\n\n");
     const rl = createInterface({ input: process.stdin, output: process.stdout });
@@ -206,7 +230,12 @@ async function setup(): Promise<number> {
     steps.push(`npm install -g @aloud/runner@latest    # ${RUNNER_VERSION} is installed, ${latest} is out`);
   }
   if (!credentials) {
-    steps.push("aloud login    # prompts for a token. The person, not you, pastes it in here.");
+    steps.push(
+      "aloud login    # needs a terminal. If you are an agent, hand this step to the person:",
+    );
+    steps.push(
+      "               # they run it themselves and paste the token at the prompt.",
+    );
   }
   if (!running) {
     steps.push("aloud start    # in a background shell. It never exits.");

package/src/version.ts

@@ -10,7 +10,7 @@
  * package.json beside it to read, and importing one into the source trips the composite build's
  * rootDir. `version.test.ts` asserts this matches, so the drift this invites cannot survive CI.
  */
-export const RUNNER_VERSION = "0.2.2";
+export const RUNNER_VERSION = "0.2.3";
 
 /** The header the server reads it from. */
 export const RUNNER_VERSION_HEADER = "x-aloud-runner-version";