@codacy/verity-cli 0.31.1-experimental.b9b3862 → 0.31.1-experimental.f2f59c0

package/CHANGELOG.md

@@ -5,6 +5,31 @@ All notable changes to Verity are documented here. This project follows
 
 ## [Unreleased]
 
+### ⌨️ The setup questions answer to arrow keys
+
+- **`verity init` no longer asks you to type a number.** Analysis intensity and
+  the review moments are arrow-key lists: `↑↓` to move, `space` to toggle a
+  moment, `enter` to confirm — with the recommended answer already under the
+  cursor and the default set already ticked, so **Enter alone still gives exactly
+  what it gave before**.
+- **Digits still work.** This prompt used to be "type a number", and muscle memory
+  should not be punished for an interface improvement — a digit moves the cursor
+  rather than confirming, so it composes with the arrows instead of being a
+  second, hidden way to answer. `j`/`k` move too.
+- **An empty moment selection is refused, not accepted.** Unticking everything and
+  pressing Enter says so instead of installing a review tool that reviews nothing
+  — the same invariant the typed prompt held by falling back.
+- **Three rungs of degradation, same default at each.** Arrow keys on a real
+  terminal; the typed number prompt when the terminal cannot do raw mode; the
+  default when there is no terminal at all or stdin ends. An interface improvement
+  must not become a new way for a setup to get stuck — CI, pipes and agent Bash
+  calls behave exactly as before.
+- **The terminal is restored on every exit,** including Ctrl+C, which restores
+  before exiting: raw mode left on outlives the process and makes the user's next
+  shell prompt unusable.
+
+## [Unreleased]
+
 ### 🤝 Joining a project that already has a Standard
 
 - **`verity init` now offers the Standard the service already holds** for this

package/README.md

@@ -11,8 +11,9 @@ verity init
 ```
 
 `verity init` is the whole setup. It asks how deeply to review and when
-(on stop / before commit / before push), wires everything, offers the optional
-GitHub login — then either **adopts the Standard your team already has**, or
+(on stop / before commit / before push) with arrow-key lists — `↑↓` to move,
+`space` to toggle, `enter` to confirm, and Enter alone takes the recommended
+answers — wires everything, offers the optional GitHub login — then either **adopts the Standard your team already has**, or
 launches Claude Code to synthesize one from your codebase (`/verity-setup`).
 
 ### Joining a project that already uses Verity

package/bin/verity.js

@@ -19364,7 +19364,7 @@ function channelSilence(input) {
 // src/lib/cli-version.ts
 function cliVersion() {
   try {
-    return true ? "0.31.1-experimental.b9b3862" : "dev";
+    return true ? "0.31.1-experimental.f2f59c0" : "dev";
   } catch {
     return "dev";
   }
@@ -24075,12 +24075,168 @@ function registerMigrateCommand(program2) {
 }
 
 // src/lib/prompt.ts
-var readline2 = __toESM(require("node:readline/promises"));
+var readline3 = __toESM(require("node:readline/promises"));
+
+// src/lib/select.ts
+var readline2 = __toESM(require("node:readline"));
+function keyToAction(str, key) {
+  const name = key?.name;
+  if (key?.ctrl && (name === "c" || str === "")) return { type: "interrupt" };
+  if (key?.ctrl && (name === "d" || str === "")) return { type: "eof" };
+  switch (name) {
+    case "up":
+      return { type: "move", delta: -1 };
+    case "down":
+      return { type: "move", delta: 1 };
+    case "k":
+      return key?.ctrl ? null : { type: "move", delta: -1 };
+    case "j":
+      return key?.ctrl ? null : { type: "move", delta: 1 };
+    case "space":
+      return { type: "toggle" };
+    case "return":
+    case "enter":
+      return { type: "confirm" };
+    default:
+      break;
+  }
+  if (str === " ") return { type: "toggle" };
+  if (str === "\r" || str === "\n") return { type: "confirm" };
+  if (str && /^[1-9]$/.test(str)) return { type: "jump", index: Number(str) - 1 };
+  return null;
+}
+function applyKey(state, action, count, mode2) {
+  switch (action.type) {
+    case "move": {
+      const cursor = (state.cursor + action.delta + count) % count;
+      return { status: "open", state: { cursor, chosen: state.chosen } };
+    }
+    case "jump": {
+      if (action.index >= count) return { status: "open", state };
+      return { status: "open", state: { cursor: action.index, chosen: state.chosen } };
+    }
+    case "toggle": {
+      if (mode2 === "single") {
+        return { status: "open", state: { cursor: state.cursor, chosen: /* @__PURE__ */ new Set([state.cursor]) } };
+      }
+      const chosen = new Set(state.chosen);
+      if (chosen.has(state.cursor)) chosen.delete(state.cursor);
+      else chosen.add(state.cursor);
+      return { status: "open", state: { cursor: state.cursor, chosen } };
+    }
+    case "confirm": {
+      if (mode2 === "single") return { status: "confirmed", indices: [state.cursor] };
+      if (state.chosen.size === 0) {
+        return {
+          status: "open",
+          state: { ...state, hint: "Select at least one \u2014 space toggles the option under the cursor." }
+        };
+      }
+      return { status: "confirmed", indices: [...state.chosen].sort((a, b) => a - b) };
+    }
+    case "interrupt":
+      return { status: "interrupt" };
+    case "eof":
+      return { status: "eof" };
+  }
+}
+var GREEN3 = "\x1B[0;32m";
+var DIM4 = "\x1B[2m";
+var BOLD2 = "\x1B[1m";
+var RESET3 = "\x1B[0m";
+function renderSelect(question, choices, state, mode2, color = colorEnabled()) {
+  const paint = (text, code) => color ? `${code}${text}${RESET3}` : text;
+  const lines = ["", `  ${question}`];
+  const labelOf = (c) => `${c.label}${c.recommended ? " (recommended)" : ""}`;
+  const gutter = Math.max(...choices.map((c) => labelOf(c).length));
+  choices.forEach((choice, i) => {
+    const here = i === state.cursor;
+    const marker = state.chosen.has(i) ? "\u25C9" : "\u25CB";
+    const cursor = here ? "\u276F" : " ";
+    const label2 = labelOf(choice);
+    const pad = choice.hint ? " ".repeat(gutter - label2.length) : "";
+    const hint = choice.hint ? `${pad}   ${paint(choice.hint, DIM4)}` : "";
+    lines.push(`  ${cursor} ${marker} ${here ? paint(label2, BOLD2) : label2}${hint}`);
+  });
+  const keys = mode2 === "multi" ? "\u2191\u2193 move \xB7 space toggle \xB7 enter confirm" : "\u2191\u2193 move \xB7 enter confirm";
+  lines.push(`  ${paint(state.hint ?? keys, state.hint ? GREEN3 : DIM4)}`);
+  return lines;
+}
+function runSelect(opts) {
+  const input = opts.input ?? process.stdin;
+  const output = opts.output ?? process.stdout;
+  const { choices, mode: mode2 } = opts;
+  if (typeof input.setRawMode !== "function" || !input.isTTY) return Promise.resolve(null);
+  const initialIdx = choices.map((c, i) => opts.initial.includes(c.id) ? i : -1).filter((i) => i >= 0);
+  let state = {
+    cursor: initialIdx[0] ?? 0,
+    chosen: new Set(mode2 === "single" ? [initialIdx[0] ?? 0] : initialIdx)
+  };
+  return new Promise((resolve4) => {
+    let painted = 0;
+    let settled = false;
+    const draw = () => {
+      if (painted > 0) {
+        readline2.moveCursor(output, 0, -painted);
+        readline2.cursorTo(output, 0);
+        readline2.clearScreenDown(output);
+      }
+      const lines = renderSelect(opts.question, choices, state, mode2);
+      output.write(lines.join("\n") + "\n");
+      painted = lines.length;
+    };
+    const onKey = (str, key) => {
+      const action = keyToAction(str, key);
+      if (!action) return;
+      const outcome = applyKey(state, action, choices.length, mode2);
+      if (outcome.status === "open") {
+        state = outcome.state;
+        draw();
+        return;
+      }
+      if (outcome.status === "confirmed") {
+        state = { ...state, hint: void 0 };
+        draw();
+        finish(outcome.indices.map((i) => choices[i].id));
+        return;
+      }
+      if (outcome.status === "eof") {
+        finish(null);
+        return;
+      }
+      restore();
+      output.write("\n");
+      process.exit(130);
+    };
+    const restore = () => {
+      input.removeListener("keypress", onKey);
+      try {
+        input.setRawMode(false);
+      } catch {
+      }
+      input.pause();
+    };
+    const finish = (value) => {
+      if (settled) return;
+      settled = true;
+      restore();
+      resolve4(value);
+    };
+    readline2.emitKeypressEvents(input);
+    input.setRawMode(true);
+    input.resume();
+    input.on("keypress", onKey);
+    input.once("end", () => finish(null));
+    draw();
+  });
+}
+
+// src/lib/prompt.ts
 function interactive() {
   return !!process.stdin.isTTY && !!process.stdout.isTTY;
 }
 async function askLine(question, io = {}) {
-  const rl = readline2.createInterface({
+  const rl = readline3.createInterface({
     input: io.input ?? process.stdin,
     output: io.output ?? process.stdout
   });
@@ -24138,6 +24294,8 @@ function printOptions(question, choices) {
 }
 async function promptChoice(question, choices, fallback) {
   if (!interactive()) return fallback;
+  const picked = await runSelect({ question, choices, initial: [fallback], mode: "single" });
+  if (picked !== null) return picked[0] ?? fallback;
   printOptions(question, choices);
   const defaultIdx = choices.findIndex((c) => c.id === fallback);
   const answer = await ask(`  Choose [${defaultIdx + 1}]: `);
@@ -24151,6 +24309,8 @@ async function promptChoice(question, choices, fallback) {
 }
 async function promptMultiSelect(question, choices, fallback) {
   if (!interactive()) return [...fallback];
+  const picked = await runSelect({ question, choices, initial: fallback, mode: "multi" });
+  if (picked !== null) return picked.length > 0 ? picked : [...fallback];
   printOptions(question, choices);
   const defaultLabel = choices.map((c, i) => fallback.includes(c.id) ? String(i + 1) : null).filter(Boolean).join(",");
   const answer = await ask(`  Choose one or more, comma-separated [${defaultLabel}]: `);
@@ -24853,7 +25013,7 @@ function registerInitCommand(program2) {
         ...telemetryChoice ? { telemetry: telemetryChoice } : {},
         init: {
           completed_at: (/* @__PURE__ */ new Date()).toISOString(),
-          cli_version: true ? "0.31.1-experimental.b9b3862" : "dev"
+          cli_version: true ? "0.31.1-experimental.f2f59c0" : "dev"
         }
       });
     } catch (err) {
@@ -25524,8 +25684,8 @@ function registerTelemetryCommands(program2) {
 }
 
 // src/cli.ts
-program.name("verity").description("CLI for Verity quality gate service").version("0.31.1-experimental.b9b3862").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async (_thisCommand, actionCommand) => {
-  installStderrLog(actionCommand.name(), process.argv.slice(2), "0.31.1-experimental.b9b3862");
+program.name("verity").description("CLI for Verity quality gate service").version("0.31.1-experimental.f2f59c0").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async (_thisCommand, actionCommand) => {
+  installStderrLog(actionCommand.name(), process.argv.slice(2), "0.31.1-experimental.f2f59c0");
   setUserNamedServiceUrl(program.opts().serviceUrl);
   try {
     await foldLegacyLocalCredential();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/verity-cli",
-  "version": "0.31.1-experimental.b9b3862",
+  "version": "0.31.1-experimental.f2f59c0",
   "description": "CLI for Verity quality gate service",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://verity.md",