@izkac/forgekit 0.1.2 → 0.1.3

package/bin/forge.mjs

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 /**
  * Forge CLI — session orchestration, prefs, models, project init.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@izkac/forgekit",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Forgekit CLIs — forgekit (install), forge (workflow), review (code review)",
   "type": "module",
   "bin": {
@@ -42,5 +42,8 @@
     "claude",
     "codex"
   ],
-  "license": "MIT"
+  "license": "MIT",
+  "dependencies": {
+    "@inquirer/prompts": "^8.5.2"
+  }
 }

package/src/init.mjs

@@ -11,10 +11,9 @@
 
 import fs from 'node:fs';
 import path from 'node:path';
-import readline from 'node:readline/promises';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import { spawnSync } from 'node:child_process';
-import { stdin as input, stdout as output } from 'node:process';
+import { checkbox, confirm, input } from '@inquirer/prompts';
 import {
   DEFAULT_ADR_DIR,
   disableProjectAdr,
@@ -22,7 +21,6 @@ import {
   normalizeAdrDir,
   scaffoldAdr,
 } from './adr.mjs';
-import { parseMenuSelection } from './menu-select.mjs';
 import {
   DEFAULT_SPECS_DIR,
   hasOpenSpecConfig,
@@ -392,26 +390,15 @@ export function initProject(selected, opts) {
 }
 
 async function promptAgents() {
-  const rl = readline.createInterface({ input, output });
-  const map = { 1: 'cursor', 2: 'claude', 3: 'codex' };
-  const allIds = ['cursor', 'claude', 'codex'];
-  const allNum = '4';
-  try {
-    process.stdout.write(`Init Forge project wiring for which environments?\n`);
-    process.stdout.write(`(pick one or more — e.g. 1 or 1,3 — or ${allNum} for all)\n`);
-    process.stdout.write(`  1) Cursor\n`);
-    process.stdout.write(`  2) Claude Code\n`);
-    process.stdout.write(`  3) Codex CLI\n`);
-    process.stdout.write(`  ${allNum}) All\n`);
-    for (;;) {
-      const answer = await rl.question(`Choice(s) [1-${allNum}]: `);
-      const parsed = parseMenuSelection(answer, map, allIds, allNum);
-      if (parsed.ok) return parsed.ids;
-      process.stdout.write(`${parsed.error}\n`);
-    }
-  } finally {
-    rl.close();
-  }
+  return checkbox({
+    message: 'Init Forge project wiring for which environments?',
+    choices: [
+      { value: 'cursor', name: 'Cursor' },
+      { value: 'claude', name: 'Claude Code' },
+      { value: 'codex', name: 'Codex CLI' },
+    ],
+    required: true,
+  });
 }
 
 /**
@@ -419,19 +406,11 @@ async function promptAgents() {
  * @returns {Promise<boolean>} true = user accepted OpenSpec setup
  */
 async function promptOpenSpecSetup() {
-  const rl = readline.createInterface({ input, output });
-  try {
-    const yn = (
-      await rl.question(
-        'OpenSpec is not set up in this project. Install and set it up now? [Y/n] (n = built-in specs engine) ',
-      )
-    )
-      .trim()
-      .toLowerCase();
-    return !(yn === 'n' || yn === 'no');
-  } finally {
-    rl.close();
-  }
+  return confirm({
+    message:
+      'OpenSpec is not set up in this project. Install and set it up now? (No = built-in specs engine)',
+    default: true,
+  });
 }
 
 /**
@@ -490,25 +469,16 @@ async function resolveInitPlanEngine(opts) {
  * @returns {Promise<{ enabled: boolean, dir: string }>}
  */
 async function promptAdrForInit(defaultDir = DEFAULT_ADR_DIR) {
-  const rl = readline.createInterface({ input, output });
-  let enabled = false;
-  try {
-    const yn = (
-      await rl.question(
-        'Use Architecture Decision Records (ADRs) in this project? [y/N] ',
-      )
-    )
-      .trim()
-      .toLowerCase();
-    enabled = yn === 'y' || yn === 'yes';
-    if (!enabled) return { enabled: false, dir: defaultDir };
-    const dirAnswer = (
-      await rl.question(`ADR directory inside the repo [${defaultDir}]: `)
-    ).trim();
-    return { enabled: true, dir: normalizeAdrDir(dirAnswer || defaultDir) };
-  } finally {
-    rl.close();
-  }
+  const enabled = await confirm({
+    message: 'Use Architecture Decision Records (ADRs) in this project?',
+    default: false,
+  });
+  if (!enabled) return { enabled: false, dir: defaultDir };
+  const dir = await input({
+    message: 'ADR directory inside the repo',
+    default: defaultDir,
+  });
+  return { enabled: true, dir: normalizeAdrDir(dir.trim() || defaultDir) };
 }
 
 async function main(argv = process.argv.slice(2)) {
@@ -566,6 +536,7 @@ if (isDirect) {
   main()
     .then((code) => process.exit(code))
     .catch((err) => {
+      if (err?.name === 'ExitPromptError') process.exit(130);
       process.stderr.write(`${err.message || err}\n`);
       process.exit(1);
     });

package/src/install.mjs

@@ -16,9 +16,8 @@
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import readline from 'node:readline/promises';
 import { pathToFileURL } from 'node:url';
-import { stdin as input, stdout as output } from 'node:process';
+import { checkbox, confirm, input, select } from '@inquirer/prompts';
 import {
   ADR_SKILLS,
   DEFAULT_ADR_DIR,
@@ -28,7 +27,6 @@ import {
   scaffoldAdr,
   disableProjectAdr,
 } from './adr.mjs';
-import { parseMenuSelection } from './menu-select.mjs';
 import { saveUserPlanEngine } from './plan-engine.mjs';
 import { hashDirectory, packageVersion, resolveAsset } from './paths.mjs';
 
@@ -415,42 +413,27 @@ export function listInstallStatus(opts = {}) {
 }
 
 /**
- * @param {string} question
- * @param {Record<string, string>} map  number → id
- * @param {string[]} allIds
+ * @param {string} message
+ * @param {string[]} ids
  * @returns {Promise<string[]>}
  */
-async function promptMulti(question, map, allIds) {
-  const rl = readline.createInterface({ input, output });
-  try {
-    const entries = Object.entries(map);
-    const allNum = String(entries.length + 1);
-    process.stdout.write(`${question}\n`);
-    process.stdout.write(`(pick one or more — e.g. 1 or 1,3 — or ${allNum} for all)\n`);
-    for (const [num, id] of entries) {
-      const label = SKILLS[id]?.label ?? AGENTS[id]?.label ?? id;
-      process.stdout.write(`  ${num}) ${label}\n`);
-    }
-    process.stdout.write(`  ${allNum}) All\n`);
-    for (;;) {
-      const answer = await rl.question(`Choice(s) [1-${allNum}]: `);
-      const parsed = parseMenuSelection(answer, map, allIds, allNum);
-      if (parsed.ok) return parsed.ids;
-      process.stdout.write(`${parsed.error}\n`);
-    }
-  } finally {
-    rl.close();
-  }
+async function promptMulti(message, ids) {
+  return checkbox({
+    message,
+    choices: ids.map((id) => ({
+      value: id,
+      name: SKILLS[id]?.label ?? AGENTS[id]?.label ?? id,
+    })),
+    required: true,
+  });
 }
 
 async function promptSkills() {
-  const map = Object.fromEntries(SKILL_IDS.map((id, i) => [String(i + 1), id]));
-  return promptMulti('Install which skills?', map, SKILL_IDS);
+  return promptMulti('Install which skills?', SKILL_IDS);
 }
 
 async function promptAgents() {
-  const map = Object.fromEntries(AGENT_IDS.map((id, i) => [String(i + 1), id]));
-  return promptMulti('Install for which environments?', map, AGENT_IDS);
+  return promptMulti('Install for which environments?', AGENT_IDS);
 }
 
 /**
@@ -458,35 +441,21 @@ async function promptAgents() {
  * @returns {Promise<string>}
  */
 export async function promptAdrDir(defaultDir = DEFAULT_ADR_DIR) {
-  const rl = readline.createInterface({ input, output });
-  try {
-    const dirAnswer = (
-      await rl.question(`ADR directory inside each repo [${defaultDir}]: `)
-    ).trim();
-    return normalizeAdrDir(dirAnswer || defaultDir);
-  } finally {
-    rl.close();
-  }
+  const dir = await input({
+    message: 'ADR directory inside each repo',
+    default: defaultDir,
+  });
+  return normalizeAdrDir(dir.trim() || defaultDir);
 }
 
 /**
  * @returns {Promise<{ enabled: boolean, dir: string }>}
  */
 export async function promptAdrOptions() {
-  const rl = readline.createInterface({ input, output });
-  let enabled = false;
-  try {
-    const yn = (
-      await rl.question(
-        'Use Architecture Decision Records (ADRs) after OpenSpec archive? [y/N] ',
-      )
-    )
-      .trim()
-      .toLowerCase();
-    enabled = yn === 'y' || yn === 'yes';
-  } finally {
-    rl.close();
-  }
+  const enabled = await confirm({
+    message: 'Use Architecture Decision Records (ADRs) after OpenSpec archive?',
+    default: false,
+  });
   if (!enabled) return { enabled: false, dir: DEFAULT_ADR_DIR };
   const dir = await promptAdrDir(DEFAULT_ADR_DIR);
   return { enabled: true, dir };
@@ -496,19 +465,13 @@ export async function promptAdrOptions() {
  * @returns {Promise<boolean>} true = OpenSpec, false = built-in specs engine
  */
 export async function promptOpenSpec() {
-  const rl = readline.createInterface({ input, output });
-  try {
-    const yn = (
-      await rl.question(
-        'Plan with OpenSpec (vendor CLI)? [Y/n] (n = built-in specs engine) ',
-      )
-    )
-      .trim()
-      .toLowerCase();
-    return !(yn === 'n' || yn === 'no');
-  } finally {
-    rl.close();
-  }
+  return select({
+    message: 'Planning engine?',
+    choices: [
+      { value: true, name: 'OpenSpec (vendor CLI)' },
+      { value: false, name: 'Built-in specs engine' },
+    ],
+  });
 }
 
 /**
@@ -763,6 +726,7 @@ if (isDirect) {
   runInstall()
     .then((code) => process.exit(code))
     .catch((err) => {
+      if (err?.name === 'ExitPromptError') process.exit(130);
       process.stderr.write(`${err.message || err}\n`);
       process.exit(1);
     });

package/src/menu-select.mjs

@@ -1,49 +0,0 @@
-/**
- * Shared numbered-menu selection (one, many, or all).
- */
-
-/**
- * Parse a menu answer into selected ids.
- * Accepts one number, several (comma/space), or the All option.
- *
- * @param {string} answer
- * @param {Record<string, string>} map  number → id
- * @param {string[]} allIds
- * @param {string} allNum
- * @returns {{ ok: true, ids: string[] } | { ok: false, error: string }}
- */
-export function parseMenuSelection(answer, map, allIds, allNum) {
-  const raw = String(answer ?? '').trim();
-  if (!raw) {
-    return {
-      ok: false,
-      error: `Enter one or more numbers (e.g. 1 or 1,3) or ${allNum} for all`,
-    };
-  }
-  if (raw === allNum || /^all$/i.test(raw)) {
-    return { ok: true, ids: [...allIds] };
-  }
-  const tokens = raw.split(/[,\s]+/).filter(Boolean);
-  /** @type {string[]} */
-  const ids = [];
-  /** @type {string[]} */
-  const bad = [];
-  for (const t of tokens) {
-    if (t === allNum || /^all$/i.test(t)) {
-      return { ok: true, ids: [...allIds] };
-    }
-    const id = map[t];
-    if (!id) bad.push(t);
-    else if (!ids.includes(id)) ids.push(id);
-  }
-  if (bad.length) {
-    return {
-      ok: false,
-      error: `Unknown choice(s): ${bad.join(', ')}. Use listed numbers or ${allNum} for all`,
-    };
-  }
-  if (ids.length === 0) {
-    return { ok: false, error: 'Nothing selected' };
-  }
-  return { ok: true, ids };
-}

package/src/menu-select.test.mjs

@@ -1,52 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { parseMenuSelection } from './menu-select.mjs';
-
-const map = { 1: 'forge', 2: 'review', 3: 'adr', 4: 'git' };
-const allIds = ['forge', 'review', 'adr', 'git'];
-const allNum = '5';
-
-test('parseMenuSelection: single choice', () => {
-  assert.deepEqual(parseMenuSelection('1', map, allIds, allNum), {
-    ok: true,
-    ids: ['forge'],
-  });
-});
-
-test('parseMenuSelection: multiple comma-separated', () => {
-  assert.deepEqual(parseMenuSelection('1,3', map, allIds, allNum), {
-    ok: true,
-    ids: ['forge', 'adr'],
-  });
-});
-
-test('parseMenuSelection: multiple space-separated', () => {
-  assert.deepEqual(parseMenuSelection('2 4', map, allIds, allNum), {
-    ok: true,
-    ids: ['review', 'git'],
-  });
-});
-
-test('parseMenuSelection: all via number or word', () => {
-  assert.deepEqual(parseMenuSelection('5', map, allIds, allNum), {
-    ok: true,
-    ids: [...allIds],
-  });
-  assert.deepEqual(parseMenuSelection('all', map, allIds, allNum), {
-    ok: true,
-    ids: [...allIds],
-  });
-});
-
-test('parseMenuSelection: empty and unknown rejected', () => {
-  assert.equal(parseMenuSelection('', map, allIds, allNum).ok, false);
-  assert.equal(parseMenuSelection('9', map, allIds, allNum).ok, false);
-  assert.equal(parseMenuSelection('1,9', map, allIds, allNum).ok, false);
-});
-
-test('parseMenuSelection: dedupes repeats', () => {
-  assert.deepEqual(parseMenuSelection('1,1,2', map, allIds, allNum), {
-    ok: true,
-    ids: ['forge', 'review'],
-  });
-});