opencode-onboard 0.4.2 → 0.4.4

package/src/utils/__tests__/copy.test.js

@@ -1,117 +1,117 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
-import path from 'path'
-import os from 'os'
-import fse from 'fs-extra'
-
-// Use real fs-extra for file system tests (temp dirs)
-import { copyContent, findAiFiles } from '../copy.js'
-
-const tmpDir = () => fse.mkdtempSync(path.join(os.tmpdir(), 'ob-test-'))
-const aiFiles = [
-  'AGENTS.md',
-  'CLAUDE.md',
-  '.cursorrules',
-  '.clinerules',
-  '.github/copilot-instructions.md',
-]
-
-describe('copy utils', () => {
-  describe('findAiFiles()', () => {
-    let dir
-
-    beforeEach(() => {
-      dir = tmpDir()
-    })
-
-    afterEach(async () => {
-      await fse.remove(dir)
-    })
-
-    it('returns empty array when no AI files exist', async () => {
-      const found = await findAiFiles(dir, aiFiles)
-      expect(found).toEqual([])
-    })
-
-    it('detects AGENTS.md', async () => {
-      await fse.writeFile(path.join(dir, 'AGENTS.md'), '# agents')
-      const found = await findAiFiles(dir, aiFiles)
-      expect(found).toHaveLength(1)
-      expect(found[0]).toContain('AGENTS.md')
-    })
-
-    it('detects CLAUDE.md', async () => {
-      await fse.writeFile(path.join(dir, 'CLAUDE.md'), '# claude')
-      const found = await findAiFiles(dir, aiFiles)
-      expect(found).toHaveLength(1)
-      expect(found[0]).toContain('CLAUDE.md')
-    })
-
-    it('detects multiple AI files at once', async () => {
-      await fse.writeFile(path.join(dir, 'AGENTS.md'), '')
-      await fse.writeFile(path.join(dir, '.cursorrules'), '')
-      await fse.writeFile(path.join(dir, '.clinerules'), '')
-      const found = await findAiFiles(dir, aiFiles)
-      expect(found).toHaveLength(3)
-    })
-
-    it('detects nested copilot-instructions.md', async () => {
-      const ghDir = path.join(dir, '.github')
-      await fse.ensureDir(ghDir)
-      await fse.writeFile(path.join(ghDir, 'copilot-instructions.md'), '')
-      const found = await findAiFiles(dir, aiFiles)
-      expect(found).toHaveLength(1)
-      expect(found[0]).toContain('copilot-instructions.md')
-    })
-  })
-
-  describe('copyContent()', () => {
-    let src, dest
-
-    beforeEach(async () => {
-      src = tmpDir()
-      dest = tmpDir()
-    })
-
-    afterEach(async () => {
-      await fse.remove(src)
-      await fse.remove(dest)
-    })
-
-    it('copies files that match neither platform exclusion', async () => {
-      await fse.writeFile(path.join(src, 'AGENTS.md'), '# agents')
-      await copyContent(src, dest, 'github')
-      expect(await fse.pathExists(path.join(dest, 'AGENTS.md'))).toBe(true)
-    })
-
-    it('always excludes .bootstrap folder', async () => {
-      await fse.ensureDir(path.join(src, '.bootstrap'))
-      await fse.writeFile(path.join(src, '.bootstrap', 'secret.md'), 'internal')
-
-      await copyContent(src, dest, 'github')
-
-      expect(await fse.pathExists(path.join(dest, '.bootstrap', 'secret.md'))).toBe(false)
-    })
-
-    it('does not overwrite existing files', async () => {
-      await fse.writeFile(path.join(src, 'AGENTS.md'), 'new content')
-      await fse.writeFile(path.join(dest, 'AGENTS.md'), 'original content')
-
-      await copyContent(src, dest, 'github')
-
-      const content = await fse.readFile(path.join(dest, 'AGENTS.md'), 'utf-8')
-      expect(content).toBe('original content')
-    })
-
-    it('copies github-specific files when platform is github', async () => {
-      await fse.writeFile(path.join(src, 'agent-gh.md'), 'github agent')
-      await copyContent(src, dest, 'github')
-      expect(await fse.pathExists(path.join(dest, 'agent-gh.md'))).toBe(true)
-    })
-
-    it('copies azure-specific files when platform is azure', async () => {
-      await fse.writeFile(path.join(src, 'agent-az.md'), 'azure agent')
-      await copyContent(src, dest, 'azure')
-      expect(await fse.pathExists(path.join(dest, 'agent-az.md'))).toBe(true)
-    })
-  })
-})
+import { describe, it, expect, beforeEach, afterEach } from 'vitest'
+import path from 'path'
+import os from 'os'
+import fse from 'fs-extra'
+
+// Use real fs-extra for file system tests (temp dirs)
+import { copyContent, findAiFiles } from '../copy.js'
+
+const tmpDir = () => fse.mkdtempSync(path.join(os.tmpdir(), 'ob-test-'))
+const aiFiles = [
+  'AGENTS.md',
+  'CLAUDE.md',
+  '.cursorrules',
+  '.clinerules',
+  '.github/copilot-instructions.md',
+]
+
+describe('copy utils', () => {
+  describe('findAiFiles()', () => {
+    let dir
+
+    beforeEach(() => {
+      dir = tmpDir()
+    })
+
+    afterEach(async () => {
+      await fse.remove(dir)
+    })
+
+    it('returns empty array when no AI files exist', async () => {
+      const found = await findAiFiles(dir, aiFiles)
+      expect(found).toEqual([])
+    })
+
+    it('detects AGENTS.md', async () => {
+      await fse.writeFile(path.join(dir, 'AGENTS.md'), '# agents')
+      const found = await findAiFiles(dir, aiFiles)
+      expect(found).toHaveLength(1)
+      expect(found[0]).toContain('AGENTS.md')
+    })
+
+    it('detects CLAUDE.md', async () => {
+      await fse.writeFile(path.join(dir, 'CLAUDE.md'), '# claude')
+      const found = await findAiFiles(dir, aiFiles)
+      expect(found).toHaveLength(1)
+      expect(found[0]).toContain('CLAUDE.md')
+    })
+
+    it('detects multiple AI files at once', async () => {
+      await fse.writeFile(path.join(dir, 'AGENTS.md'), '')
+      await fse.writeFile(path.join(dir, '.cursorrules'), '')
+      await fse.writeFile(path.join(dir, '.clinerules'), '')
+      const found = await findAiFiles(dir, aiFiles)
+      expect(found).toHaveLength(3)
+    })
+
+    it('detects nested copilot-instructions.md', async () => {
+      const ghDir = path.join(dir, '.github')
+      await fse.ensureDir(ghDir)
+      await fse.writeFile(path.join(ghDir, 'copilot-instructions.md'), '')
+      const found = await findAiFiles(dir, aiFiles)
+      expect(found).toHaveLength(1)
+      expect(found[0]).toContain('copilot-instructions.md')
+    })
+  })
+
+  describe('copyContent()', () => {
+    let src, dest
+
+    beforeEach(async () => {
+      src = tmpDir()
+      dest = tmpDir()
+    })
+
+    afterEach(async () => {
+      await fse.remove(src)
+      await fse.remove(dest)
+    })
+
+    it('copies files that match neither platform exclusion', async () => {
+      await fse.writeFile(path.join(src, 'AGENTS.md'), '# agents')
+      await copyContent(src, dest, 'github')
+      expect(await fse.pathExists(path.join(dest, 'AGENTS.md'))).toBe(true)
+    })
+
+    it('always excludes .bootstrap folder', async () => {
+      await fse.ensureDir(path.join(src, '.bootstrap'))
+      await fse.writeFile(path.join(src, '.bootstrap', 'secret.md'), 'internal')
+
+      await copyContent(src, dest, 'github')
+
+      expect(await fse.pathExists(path.join(dest, '.bootstrap', 'secret.md'))).toBe(false)
+    })
+
+    it('does not overwrite existing files', async () => {
+      await fse.writeFile(path.join(src, 'AGENTS.md'), 'new content')
+      await fse.writeFile(path.join(dest, 'AGENTS.md'), 'original content')
+
+      await copyContent(src, dest, 'github')
+
+      const content = await fse.readFile(path.join(dest, 'AGENTS.md'), 'utf-8')
+      expect(content).toBe('original content')
+    })
+
+    it('copies github-specific files when platform is github', async () => {
+      await fse.writeFile(path.join(src, 'agent-gh.md'), 'github agent')
+      await copyContent(src, dest, 'github')
+      expect(await fse.pathExists(path.join(dest, 'agent-gh.md'))).toBe(true)
+    })
+
+    it('copies azure-specific files when platform is azure', async () => {
+      await fse.writeFile(path.join(src, 'agent-az.md'), 'azure agent')
+      await copyContent(src, dest, 'azure')
+      expect(await fse.pathExists(path.join(dest, 'agent-az.md'))).toBe(true)
+    })
+  })
+})

package/src/utils/exec-spinner.js

@@ -1,47 +1,47 @@
-import chalk from 'chalk'
-import ora from 'ora'
-
-// ── Screen / step state ──────────────────────────────────────────────────────
-
-const previousSteps = [];    // up to 2 completed steps, each is an array of lines
-let currentStepLines = [];   // lines accumulated in the current step
-let stepSpinner = null;      // ora spinner shown while step is working
-
-export function appendLine(line) {
-  currentStepLines.push(line);
-}
-
-export function stopSpinner() {
-  if (stepSpinner) {
-    stepSpinner.stop();
-    stepSpinner = null;
-  }
-}
-
-export function startSpinner(text = 'working...') {
-  stopSpinner();
-  stepSpinner = ora({ text: chalk.dim(text), color: 'red' }).start();
-}
-
-export function redraw() {
-  if (process.stdout.isTTY) console.clear();
-
-  // Show up to 2 previous steps dimmed
-  for (const stepLines of previousSteps) {
-    for (const line of stepLines) {
-      process.stdout.write(chalk.dim(line) + '\n');
-    }
-    process.stdout.write('\n');
-  }
-
-  // Current step output
-  for (const line of currentStepLines) {
-    process.stdout.write(line + '\n');
-  }
-}
-
-export function rotateStep() {
-  previousSteps.push(currentStepLines);
-  if (previousSteps.length > 2) previousSteps.shift();
-  currentStepLines = [];
-}
+import chalk from 'chalk'
+import ora from 'ora'
+
+// ── Screen / step state ──────────────────────────────────────────────────────
+
+const previousSteps = [];    // up to 2 completed steps, each is an array of lines
+let currentStepLines = [];   // lines accumulated in the current step
+let stepSpinner = null;      // ora spinner shown while step is working
+
+export function appendLine(line) {
+  currentStepLines.push(line);
+}
+
+export function stopSpinner() {
+  if (stepSpinner) {
+    stepSpinner.stop();
+    stepSpinner = null;
+  }
+}
+
+export function startSpinner(text = 'working...') {
+  stopSpinner();
+  stepSpinner = ora({ text: chalk.dim(text), color: 'red' }).start();
+}
+
+export function redraw() {
+  if (process.stdout.isTTY) console.clear();
+
+  // Show up to 2 previous steps dimmed
+  for (const stepLines of previousSteps) {
+    for (const line of stepLines) {
+      process.stdout.write(`${chalk.dim(line)}\n`);
+    }
+    process.stdout.write('\n');
+  }
+
+  // Current step output
+  for (const line of currentStepLines) {
+    process.stdout.write(`${line}\n`);
+  }
+}
+
+export function rotateStep() {
+  previousSteps.push(currentStepLines);
+  if (previousSteps.length > 2) previousSteps.shift();
+  currentStepLines = [];
+}

package/src/utils/exec.js

@@ -1,131 +1,134 @@
-import chalk from 'chalk'
-import { execa } from 'execa'
-import ora from 'ora'
-import { appendLine, redraw, rotateStep, startSpinner, stopSpinner } from './exec-spinner.js'
-
-// ── Public API ───────────────────────────────────────────────────────────────
-
-/**
- * Run a shell command with a spinner.
- * Returns { success, stdout, stderr }
- */
-export async function run(command, args = [], { label, cwd = process.cwd() } = {}) {
-  const spinner = ora(label ?? `${command} ${args.join(' ')}`).start();
-  try {
-    const result = await execa(command, args, { cwd, reject: false });
-    if (result.exitCode === 0) {
-      spinner.succeed();
-    } else {
-      spinner.fail();
-    }
-    return { success: result.exitCode === 0, stdout: result.stdout, stderr: result.stderr };
-  } catch (err) {
-    spinner.fail();
-    return { success: false, stdout: '', stderr: err.message };
-  }
-}
-
-/**
- * Check if a command is available on PATH.
- * Returns true/false.
- */
-export async function commandExists(command) {
-  try {
-    const result = await execa(command, ['--version'], { reject: false });
-    return result.exitCode === 0;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Print a section header, clears screen, shows previous step dimmed, starts new step.
- */
-export function header(text) {
-  rotateStep();
-
-  const line1 = '';
-  const line2 = chalk.bold.hex('#fe3d57')(`━━ ${text}`);
-  const line3 = '';
-
-  appendLine(line1);
-  appendLine(line2);
-  appendLine(line3);
-
-  redraw();
-
-  startSpinner('working...');
-}
-
-/**
- * Restart the step spinner after prompts or logs.
- */
-export function loading(text = 'working...') {
-  startSpinner(text);
-}
-
-/**
- * Print a success line.
- */
-export function success(text) {
-  stopSpinner();
-  const line = chalk.green('✓ ') + text;
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print a warning line.
- */
-export function warn(text) {
-  stopSpinner();
-  const line = chalk.yellow('⚠ ') + text;
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an error line.
- */
-export function error(text) {
-  stopSpinner();
-  const line = chalk.red('✗ ') + text;
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an info line.
- */
-export function info(text) {
-  stopSpinner();
-  const line = chalk.dim('  ' + text);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an action prompt line (white bold, requires user interaction).
- */
-export function prompt(text) {
-  stopSpinner();
-  const line = chalk.bold('  ' + text);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print a code block.
- */
-export function code(lines) {
-  stopSpinner();
-  appendLine('');
-  console.log();
-  for (const line of lines) {
-    const formatted = chalk.bgGray.white('  ' + line + '  ');
-    appendLine(formatted);
-    console.log(formatted);
-  }
-  appendLine('');
-  console.log();
-}
+import chalk from 'chalk'
+import { execa } from 'execa'
+import ora from 'ora'
+import { appendLine, redraw, rotateStep, startSpinner, stopSpinner } from './exec-spinner.js'
+import { MARKERS } from './terminal.js';
+
+// ── Public API ───────────────────────────────────────────────────────────────
+
+/**
+ * Run a shell command with a spinner.
+ * Returns { success, stdout, stderr }
+ */
+export async function run(command, args = [], { label, cwd = process.cwd() } = {}) {
+  const spinner = ora(label ?? `${command} ${args.join(' ')}`).start();
+  try {
+    const result = await execa(command, args, { cwd, reject: false });
+    if (result.exitCode === 0) {
+      spinner.succeed();
+    } else {
+      spinner.fail();
+    }
+    return { success: result.exitCode === 0, stdout: result.stdout, stderr: result.stderr };
+  } catch (err) {
+    spinner.fail();
+    return { success: false, stdout: '', stderr: err.message };
+  }
+}
+
+/**
+ * Check if a command is available on PATH.
+ * Returns true/false.
+ */
+export async function commandExists(command) {
+  try {
+    const result = await execa(command, ['--version'], { reject: false });
+    return result.exitCode === 0;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Print a section header, clears screen, shows previous step dimmed, starts new step.
+ */
+export function header(text) {
+  rotateStep();
+
+  const line1 = '';
+  const line2 = chalk.bold.hex('#fe3d57')(`━━ ${text}`);
+  const line3 = '';
+
+  appendLine(line1);
+  appendLine(line2);
+  appendLine(line3);
+
+  redraw();
+
+  startSpinner('working...');
+}
+
+/**
+ * Restart the step spinner after prompts or logs.
+ */
+export function loading(text = 'working...') {
+  startSpinner(text);
+}
+
+/**
+ * Print a success line.
+ */
+export function success(text) {
+  stopSpinner();
+  const line = chalk.green(`${MARKERS.OK_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print a warning line.
+ */
+export function warn(text) {
+  stopSpinner();
+  const line = chalk.yellow(`${MARKERS.WARN_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an error line.
+ */
+export function error(text) {
+  stopSpinner();
+  const line = chalk.red(`${MARKERS.ERROR_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an info line.
+ */
+export function info(text) {
+  stopSpinner();
+  const line = chalk.dim(`${MARKERS.EMPTY}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an action prompt line (white bold, requires user interaction).
+ */
+export function prompt(text) {
+  stopSpinner();
+  const line = chalk.bold(`${MARKERS.EMPTY}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print a code block.
+ */
+export function code(lines) {
+  stopSpinner();
+  appendLine('');
+  console.log();
+  for (const line of lines) {
+    const formatted = chalk.bgGray.white(
+      `${MARKERS.EMPTY}${line}${MARKERS.EMPTY}`,
+    );
+    appendLine(formatted);
+    console.log(formatted);
+  }
+  appendLine('');
+  console.log();
+}

package/src/utils/terminal.js

@@ -0,0 +1,6 @@
+export const MARKERS = {
+  EMPTY: "  ",
+  OK_PREFIX: "✓ ",
+  WARN_PREFIX: "⚠ ",
+  ERROR_PREFIX: "✗ ",
+};