opencode-onboard 0.4.3 → 0.4.5

package/src/steps/source/source.test.js

@@ -1,124 +1,124 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
-import fs from 'node:fs'
-import path from 'node:path'
-import os from 'node:os'
-
-vi.mock('../../utils/exec.js', () => ({
-  header: vi.fn(),
-  info: vi.fn(),
-  success: vi.fn(),
-  warn: vi.fn(),
-}))
-
-vi.mock('@inquirer/prompts', () => ({
-  select: vi.fn(),
-  checkbox: vi.fn(),
-}))
-
-vi.mock('fs-extra', () => ({
-  default: {
-    readJson: vi.fn().mockResolvedValue({
-      message: 'Select source scope',
-      default: 'current',
-      choices: [
-        { name: 'Current folder', value: 'current' },
-        { name: 'Parent folder', value: 'parent' },
-        { name: 'Child folders', value: 'children' },
-      ],
-      parentSelectionMessage: 'Select sibling folders',
-      childrenSelectionMessage: 'Select child folders',
-    }),
-    readdir: vi.fn(),
-    stat: vi.fn().mockResolvedValue({ isDirectory: () => true }),
-  },
-}))
-
-import { select, checkbox } from '@inquirer/prompts'
-import fse from 'fs-extra'
-import { chooseSourceScope } from './index.js'
-
-describe('chooseSourceScope()', () => {
-  let tmpDir, originalCwd
-
-  beforeEach(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'source-test-'))
-    originalCwd = process.cwd()
-    process.chdir(tmpDir)
-    vi.clearAllMocks()
-  })
-
-  afterEach(() => {
-    process.chdir(originalCwd)
-    fs.rmSync(tmpDir, { recursive: true, force: true })
-  })
-
-  it('returns current folder when user selects current mode', async () => {
-    select.mockResolvedValue('current')
-
-    const result = await chooseSourceScope()
-
-    expect(result.sourceMode).toBe('current')
-    expect(result.sourceRoots).toContain(tmpDir)
-  })
-
-  it('lists parent folders when user selects parent mode', async () => {
-    select.mockResolvedValue('parent')
-    const parentDir = path.dirname(tmpDir)
-    const siblingDir = path.join(parentDir, 'sibling-project')
-    fs.mkdirSync(siblingDir, { recursive: true })
-    fse.readdir.mockResolvedValue(['sibling-project'])
-
-    await chooseSourceScope()
-
-    expect(checkbox).toHaveBeenCalled()
-  })
-
-  it('falls back to current when no siblings found', async () => {
-    select.mockResolvedValue('parent')
-    fse.readdir.mockResolvedValue([])
-
-    const result = await chooseSourceScope()
-
-    expect(result.sourceMode).toBe('current')
-  })
-
-  it('falls back to current when no folders selected', async () => {
-    select.mockResolvedValue('parent')
-    checkbox.mockResolvedValue([])
-
-    const result = await chooseSourceScope()
-
-    expect(result.sourceMode).toBe('current')
-  })
-
-  it('lists child folders when user selects children mode', async () => {
-    select.mockResolvedValue('children')
-    fse.readdir.mockResolvedValue(['packages', 'apps'])
-    checkbox.mockResolvedValue([path.join(tmpDir, 'packages')])
-
-    const result = await chooseSourceScope()
-
-    expect(checkbox).toHaveBeenCalled()
-    expect(result.sourceMode).toBe('children-selected')
-    expect(result.sourceRoots).toContain(path.join(tmpDir, 'packages'))
-  })
-
-  it('falls back to current when no child folders found', async () => {
-    select.mockResolvedValue('children')
-    fse.readdir.mockResolvedValue([])
-
-    const result = await chooseSourceScope()
-
-    expect(result.sourceMode).toBe('current')
-  })
-
-  it('falls back to current when no child folders selected', async () => {
-    select.mockResolvedValue('children')
-    fse.readdir.mockResolvedValue(['packages'])
-    checkbox.mockResolvedValue([])
-
-    const result = await chooseSourceScope()
-
-    expect(result.sourceMode).toBe('current')
-  })
-})
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import fs from 'node:fs'
+import path from 'node:path'
+import os from 'node:os'
+
+vi.mock('../../utils/exec.js', () => ({
+  header: vi.fn(),
+  info: vi.fn(),
+  success: vi.fn(),
+  warn: vi.fn(),
+}))
+
+vi.mock('@inquirer/prompts', () => ({
+  select: vi.fn(),
+  checkbox: vi.fn(),
+}))
+
+vi.mock('fs-extra', () => ({
+  default: {
+    readJson: vi.fn().mockResolvedValue({
+      message: 'Select source scope',
+      default: 'current',
+      choices: [
+        { name: 'Current folder', value: 'current' },
+        { name: 'Parent folder', value: 'parent' },
+        { name: 'Child folders', value: 'children' },
+      ],
+      parentSelectionMessage: 'Select sibling folders',
+      childrenSelectionMessage: 'Select child folders',
+    }),
+    readdir: vi.fn(),
+    stat: vi.fn().mockResolvedValue({ isDirectory: () => true }),
+  },
+}))
+
+import { select, checkbox } from '@inquirer/prompts'
+import fse from 'fs-extra'
+import { chooseSourceScope } from './index.js'
+
+describe('chooseSourceScope()', () => {
+  let tmpDir, originalCwd
+
+  beforeEach(() => {
+    tmpDir = fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'source-test-')))
+    originalCwd = process.cwd()
+    process.chdir(tmpDir)
+    vi.clearAllMocks()
+  })
+
+  afterEach(() => {
+    process.chdir(originalCwd)
+    fs.rmSync(tmpDir, { recursive: true, force: true })
+  })
+
+  it('returns current folder when user selects current mode', async () => {
+    select.mockResolvedValue('current')
+
+    const result = await chooseSourceScope()
+
+    expect(result.sourceMode).toBe('current')
+    expect(result.sourceRoots).toContain(tmpDir)
+  })
+
+  it('lists parent folders when user selects parent mode', async () => {
+    select.mockResolvedValue('parent')
+    const parentDir = path.dirname(tmpDir)
+    const siblingDir = path.join(parentDir, 'sibling-project')
+    fs.mkdirSync(siblingDir, { recursive: true })
+    fse.readdir.mockResolvedValue(['sibling-project'])
+
+    await chooseSourceScope()
+
+    expect(checkbox).toHaveBeenCalled()
+  })
+
+  it('falls back to current when no siblings found', async () => {
+    select.mockResolvedValue('parent')
+    fse.readdir.mockResolvedValue([])
+
+    const result = await chooseSourceScope()
+
+    expect(result.sourceMode).toBe('current')
+  })
+
+  it('falls back to current when no folders selected', async () => {
+    select.mockResolvedValue('parent')
+    checkbox.mockResolvedValue([])
+
+    const result = await chooseSourceScope()
+
+    expect(result.sourceMode).toBe('current')
+  })
+
+  it('lists child folders when user selects children mode', async () => {
+    select.mockResolvedValue('children')
+    fse.readdir.mockResolvedValue(['packages', 'apps'])
+    checkbox.mockResolvedValue([path.join(tmpDir, 'packages')])
+
+    const result = await chooseSourceScope()
+
+    expect(checkbox).toHaveBeenCalled()
+    expect(result.sourceMode).toBe('children-selected')
+    expect(result.sourceRoots).toContain(path.join(tmpDir, 'packages'))
+  })
+
+  it('falls back to current when no child folders found', async () => {
+    select.mockResolvedValue('children')
+    fse.readdir.mockResolvedValue([])
+
+    const result = await chooseSourceScope()
+
+    expect(result.sourceMode).toBe('current')
+  })
+
+  it('falls back to current when no child folders selected', async () => {
+    select.mockResolvedValue('children')
+    fse.readdir.mockResolvedValue(['packages'])
+    checkbox.mockResolvedValue([])
+
+    const result = await chooseSourceScope()
+
+    expect(result.sourceMode).toBe('current')
+  })
+})

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 = [];
+}