@haystackeditor/cli 0.15.1 → 0.15.2

package/dist/commands/setup.js

@@ -39,8 +39,11 @@ function buildAnswerMap(options) {
         answers.install_hooks = false;
     }
     if (options.yes) {
-        // Accept defaults non-interactively: don't open the toggle UI (keep every
-        // discovered item) and confirm the write. Only fill ids not already set.
+        // Accept the interactive wizard's defaults non-interactively: keep every
+        // discovered item (skip the toggle UI), confirm the write, and say yes to
+        // the same things the prompts default to (auto-merge, session tracking,
+        // hooks). Use --skip-entire / --no-auto-merge to opt out. Only fill ids not
+        // already set by an explicit flag or --answers.
         if (!('want_to_toggle' in answers))
             answers.want_to_toggle = false;
         if (!('confirm_write' in answers))
@@ -48,9 +51,11 @@ function buildAnswerMap(options) {
         if (!('auto_merge' in answers))
             answers.auto_merge = true;
         if (!('install_entire' in answers))
-            answers.install_entire = false;
+            answers.install_entire = true;
+        if (!('upgrade_entire' in answers))
+            answers.upgrade_entire = true;
         if (!('install_hooks' in answers))
-            answers.install_hooks = false;
+            answers.install_hooks = true;
     }
     return answers;
 }
@@ -730,14 +735,26 @@ function tryOpenBrowser(url) {
  *  the poll keeps waiting) but rethrows a genuine auth failure so the caller can
  *  bail and prompt re-auth. */
 async function isAppInstalled(token) {
-    return (await fetchHaystackInstallations(token)).length > 0;
+    try {
+        return (await fetchHaystackInstallations(token)).length > 0;
+    }
+    catch (err) {
+        // Auth failures are permanent — surface them. Anything else (5xx, network
+        // blip) is transient during the install poll: return false so the poll
+        // keeps waiting instead of crashing the wizard.
+        if (err instanceof InstallationsAuthError)
+            throw err;
+        return false;
+    }
 }
 async function stepEnsureAppInstalled(token) {
     console.log(chalk.bold('  Step 0: Verify GitHub App'));
-    // Initial check — distinguish a genuine auth failure (bail, prompt re-auth)
-    // from a transient probe error (refuse-to-fail and proceed).
+    // Initial check — use the raw fetch (not isAppInstalled, which swallows
+    // transient errors) so we can distinguish three outcomes: installed (done),
+    // genuine auth failure (bail + prompt re-auth), or a transient probe error
+    // (refuse-to-fail and proceed rather than brick setup over a blip).
     try {
-        if (await isAppInstalled(token)) {
+        if ((await fetchHaystackInstallations(token)).length > 0) {
             console.log(chalk.green('  ✓ Haystack App installed\n'));
             return;
         }
@@ -803,14 +820,24 @@ async function stepSelectRepos(token) {
         console.log(chalk.yellow('\n  No repositories found.\n'));
         process.exit(1);
     }
-    const selectedRepos = await ui.multiselect({
-        id: 'select_repos',
-        message: 'Select repositories to configure:',
-        choices: repos.map((r) => ({ name: r, value: r })),
-    });
-    if (selectedRepos.length === 0) {
-        console.log(chalk.yellow('\n  No repositories selected.\n'));
-        process.exit(1);
+    let selectedRepos = [];
+    for (;;) {
+        selectedRepos = await ui.multiselect({
+            id: 'select_repos',
+            message: 'Select repositories to configure:',
+            choices: repos.map((r) => ({ name: r, value: r })),
+        });
+        if (selectedRepos.length > 0)
+            break;
+        // Interactive: keep the picker open so the user can correct an empty pick
+        // (restores the old inquirer `validate` behavior). Non-interactive: a
+        // pre-supplied / agent-sent empty selection can't be fixed by re-prompting,
+        // so fail clearly instead of looping forever.
+        if (!ui.interactive) {
+            console.log(chalk.yellow('\n  No repositories selected. Pass --repo <owner/name>.\n'));
+            process.exit(1);
+        }
+        console.log(chalk.yellow('  Select at least one repository.'));
     }
     console.log(chalk.green(`  ${selectedRepos.length} repo(s) selected\n`));
     return selectedRepos;

package/dist/utils/prompter.js

@@ -50,7 +50,7 @@ class TtyPrompter {
     }
     async multiselect(q) {
         if (q.id in this.answers)
-            return this.answers[q.id];
+            return asArray(this.answers[q.id], q.id);
         const choices = q.choices.map((c) => isSeparator(c) ? new inquirer.Separator(c.separator) : { name: c.name, value: c.value, checked: c.checked });
         const { value } = await inquirer.prompt([
             { type: 'checkbox', name: 'value', message: q.message, choices: choices, pageSize: 20 },
@@ -77,9 +77,16 @@ class TtyPrompter {
         return false;
     }
     progress(message, detail) {
+        // Only emit carriage-return / clear-line ANSI to a real TTY; piped to a file
+        // or CI log (non-interactive, no --json) those sequences corrupt the output,
+        // so stay silent there.
+        if (!process.stdout.isTTY)
+            return;
         process.stdout.write(`\r${chalk.dim('  ' + message)}${detail ? chalk.dim(' — ' + detail) : ''}\x1b[K`);
     }
     clearProgress() {
+        if (!process.stdout.isTTY)
+            return;
         process.stdout.write('\r\x1b[K');
     }
     result(_data) {
@@ -179,7 +186,7 @@ class JsonPrompter {
     }
     async multiselect(q) {
         if (q.id in this.answers)
-            return this.answers[q.id];
+            return asArray(this.answers[q.id], q.id);
         const options = q.choices
             .filter((c) => !isSeparator(c))
             .map((c) => ({ value: c.value, label: c.name, default: c.checked ?? false }));
@@ -249,6 +256,15 @@ export function createPrompter(options = {}) {
 function sleep(ms) {
     return new Promise((resolve) => setTimeout(resolve, ms));
 }
+/** Validate a pre-supplied multiselect answer is actually an array before use,
+ *  so a bad --answers value (e.g. a string) fails fast with a clear message
+ *  instead of crashing downstream array operations. */
+function asArray(value, id) {
+    if (!Array.isArray(value)) {
+        throw new Error(`Answer for "${id}" must be a JSON array, got ${typeof value}.`);
+    }
+    return value;
+}
 function stringifyArg(a) {
     return typeof a === 'string' ? a : String(a);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@haystackeditor/cli",
-  "version": "0.15.1",
+  "version": "0.15.2",
   "description": "Set up Haystack for your project — automated PR review, triage, and merge queue",
   "type": "module",
   "bin": {