opencode-onboard 0.4.2 → 0.4.4

package/src/presets/browser.json

@@ -1,18 +1,22 @@
-{
-  "installer": {
-    "command": "npx",
-    "args": ["@different-ai/opencode-browser", "install"]
-  },
-  "output": {
-    "showAfter": "To load the extension",
-    "hideAfter": "Press Enter when"
-  },
-  "autoAnswers": [
-    { "trigger": "Press Enter when", "response": "" },
-    { "trigger": "Choose config location", "response": "2" },
-    { "trigger": "Add plugin automatically?", "response": "y" },
-    { "trigger": "Create one?", "response": "y" },
-    { "trigger": "Add browser-automation skill", "response": "n" },
-    { "trigger": "Check broker", "response": "n" }
-  ]
-}
+{
+  "installer": {
+    "command": "npx",
+    "args": ["@different-ai/opencode-browser", "install"]
+  },
+  "output": {
+    "showAfter": "To load the extension",
+    "hideAfter": "Press Enter when"
+  },
+  "locationChoices": {
+    "local": "2",
+    "global": "1"
+  },
+  "autoAnswers": [
+    { "trigger": "Press Enter when", "response": "" },
+    { "trigger": "Choose config location", "response": "__LOCATION__" },
+    { "trigger": "Add plugin automatically?", "response": "y" },
+    { "trigger": "Create one?", "response": "y" },
+    { "trigger": "Add browser-automation skill", "response": "n" },
+    { "trigger": "Check broker", "response": "n" }
+  ]
+}

package/src/presets/optimization.json

@@ -1,22 +1,27 @@
-{
-  "message": "Enable tools:",
-  "info": "Choose which optimization tools to enable (recommended: all).",
-  "timeoutMs": 30000,
-  "choices": [
-    {
-      "name": "RTK check (recommended)",
-      "value": "rtk",
-      "checked": true
-    },
-    {
-      "name": "opencode-quota plugin (recommended)",
-      "value": "quota",
-      "checked": true
-    },
-    {
-      "name": "caveman concise mode (recommended)",
-      "value": "caveman",
-      "checked": true
-    }
-  ]
-}
+{
+  "message": "Enable tools:",
+  "info": "Choose which optimization tools to enable (recommended: all).",
+  "timeoutMs": 30000,
+  "choices": [
+    {
+      "name": "RTK check (recommended)",
+      "value": "rtk",
+      "checked": true
+    },
+    {
+      "name": "opencode-quota plugin (recommended)",
+      "value": "quota",
+      "checked": true
+    },
+    {
+      "name": "caveman concise mode (recommended)",
+      "value": "caveman",
+      "checked": true
+    },
+    {
+      "name": "codegraph semantic index (recommended)",
+      "value": "codegraph",
+      "checked": true
+    }
+  ]
+}

package/src/presets/source.json

@@ -11,7 +11,13 @@
       "name": "Select folders in parent (../)",
       "value": "parent",
       "description": "Use when this repo only contains agent config"
+    },
+    {
+      "name": "Select child folders (./*/)",
+      "value": "children",
+      "description": "Use when source code lives in subdirectories of this repo"
     }
   ],
-  "parentSelectionMessage": "Select source folders from parent directory:"
+  "parentSelectionMessage": "Select source folders from parent directory:",
+  "childrenSelectionMessage": "Select child folders to include as source:"
 }

package/src/steps/browser/browser.test.js

@@ -1,81 +1,115 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest'
-
-vi.mock('../../utils/exec.js', () => ({
-  header: vi.fn(),
-  info: vi.fn(),
-  success: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-}))
-
-vi.mock('fs-extra', () => ({
-  default: {
-    readJson: vi.fn().mockResolvedValue({
-      installer: { command: 'npx', args: ['@different-ai/opencode-browser', 'install'] },
-      output: { showAfter: '===', hideAfter: '===' },
-      autoAnswers: [
-        { trigger: 'Install', response: 'y' },
-      ],
-    }),
-  },
-}))
-
-vi.mock('execa', () => ({
-  execa: vi.fn(),
-}))
-
-import fse from 'fs-extra'
-import { installBrowser } from './index.js'
-
-describe('installBrowser()', () => {
-  beforeEach(() => {
-    vi.clearAllMocks()
-  })
-
-  it('calls installer command from preset', async () => {
-    const { execa } = await import('execa')
-    const mockChild = {
-      stdout: { on: vi.fn() },
-      stderr: { on: vi.fn() },
-      stdin: { write: vi.fn() },
-      then: (cb) => cb({ exitCode: 0 }),
-    }
-    execa.mockReturnValue(mockChild)
-
-    await installBrowser()
-
-    expect(execa).toHaveBeenCalledWith('npx', expect.arrayContaining(['@different-ai/opencode-browser']), expect.any(Object))
-  })
-
-  it('logs success when exit code is 0', async () => {
-    const { execa } = await import('execa')
-    const mockChild = {
-      stdout: { on: vi.fn() },
-      stderr: { on: vi.fn() },
-      stdin: { write: vi.fn() },
-      then: (cb) => cb({ exitCode: 0 }),
-    }
-    execa.mockReturnValue(mockChild)
-    const { success } = await import('../../utils/exec.js')
-
-    await installBrowser()
-
-    expect(success).toHaveBeenCalledWith('opencode-browser installed')
-  })
-
-  it('logs warning when exit code is non-zero', async () => {
-    const { execa } = await import('execa')
-    const mockChild = {
-      stdout: { on: vi.fn() },
-      stderr: { on: vi.fn() },
-      stdin: { write: vi.fn() },
-      then: (cb) => cb({ exitCode: 1 }),
-    }
-    execa.mockReturnValue(mockChild)
-    const { warn } = await import('../../utils/exec.js')
-
-    await installBrowser()
-
-    expect(warn).toHaveBeenCalledWith('opencode-browser install exited with non-zero code')
-  })
-})
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { installBrowser } from './index.js'
+
+vi.mock('../../utils/exec.js', () => ({
+  header: vi.fn(),
+  info: vi.fn(),
+  success: vi.fn(),
+  warn: vi.fn(),
+  error: vi.fn(),
+}))
+
+vi.mock('fs-extra', () => ({
+  default: {
+    readJson: vi.fn().mockResolvedValue({
+      installer: { command: 'npx', args: ['@different-ai/opencode-browser', 'install'] },
+      output: { showAfter: '===', hideAfter: '===' },
+      locationChoices: { local: '2', global: '1' },
+      autoAnswers: [
+        { trigger: 'Install', response: 'y' },
+        { trigger: 'Choose config location', response: '__LOCATION__' },
+      ],
+    }),
+  },
+}))
+
+vi.mock('execa', () => ({
+  execa: vi.fn(),
+}))
+
+describe('installBrowser()', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
+  it('calls installer command from preset', async () => {
+    const { execa } = await import('execa')
+    const mockChild = {
+      stdout: { on: vi.fn() },
+      stderr: { on: vi.fn() },
+      stdin: { write: vi.fn() },
+      then: (cb) => cb({ exitCode: 0 }),
+    }
+    execa.mockReturnValue(mockChild)
+
+    await installBrowser()
+
+    expect(execa).toHaveBeenCalledWith('npx', expect.arrayContaining(['@different-ai/opencode-browser']), expect.any(Object))
+  })
+
+  it('logs success when exit code is 0', async () => {
+    const { execa } = await import('execa')
+    const mockChild = {
+      stdout: { on: vi.fn() },
+      stderr: { on: vi.fn() },
+      stdin: { write: vi.fn() },
+      then: (cb) => cb({ exitCode: 0 }),
+    }
+    execa.mockReturnValue(mockChild)
+    const { success } = await import('../../utils/exec.js')
+
+    await installBrowser()
+
+    expect(success).toHaveBeenCalledWith('opencode-browser installed')
+  })
+
+  it('logs warning when exit code is non-zero', async () => {
+    const { execa } = await import('execa')
+    const mockChild = {
+      stdout: { on: vi.fn() },
+      stderr: { on: vi.fn() },
+      stdin: { write: vi.fn() },
+      then: (cb) => cb({ exitCode: 1 }),
+    }
+    execa.mockReturnValue(mockChild)
+    const { warn } = await import('../../utils/exec.js')
+
+    await installBrowser()
+
+    expect(warn).toHaveBeenCalledWith('opencode-browser install exited with non-zero code')
+  })
+
+  it('resolves __LOCATION__ to local answer by default', async () => {
+    const { execa } = await import('execa')
+    let capturedTriggers = null
+    const mockChild = {
+      stdout: { on: vi.fn((_, cb) => { capturedTriggers = cb }) },
+      stderr: { on: vi.fn() },
+      stdin: { write: vi.fn() },
+      then: (cb) => cb({ exitCode: 0 }),
+    }
+    execa.mockReturnValue(mockChild)
+
+    await installBrowser()
+
+    if (capturedTriggers) capturedTriggers(Buffer.from('Choose config location'))
+    expect(mockChild.stdin.write).toHaveBeenCalledWith('2\n')
+  })
+
+  it('resolves __LOCATION__ to global answer when installScope is global', async () => {
+    const { execa } = await import('execa')
+    let capturedTriggers = null
+    const mockChild = {
+      stdout: { on: vi.fn((_, cb) => { capturedTriggers = cb }) },
+      stderr: { on: vi.fn() },
+      stdin: { write: vi.fn() },
+      then: (cb) => cb({ exitCode: 0 }),
+    }
+    execa.mockReturnValue(mockChild)
+
+    await installBrowser({ installScope: 'global' })
+
+    if (capturedTriggers) capturedTriggers(Buffer.from('Choose config location'))
+    expect(mockChild.stdin.write).toHaveBeenCalledWith('1\n')
+  })
+})

package/src/steps/browser/index.js

@@ -1,54 +1,62 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import { header, info, success, warn, error } from '../../utils/exec.js'
-import os from 'os'
-import path from 'path'
-import { fileURLToPath } from 'url'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const BROWSER_PRESET_PATH = path.resolve(__dirname, '../../presets/browser.json')
-const browserPreset = await fse.readJson(BROWSER_PRESET_PATH)
-
-export async function installBrowser() {
-  header('Step 9, Installing opencode-browser')
-
-  try {
-    const child = execa(browserPreset.installer.command, browserPreset.installer.args, {
-      cwd: os.homedir(),
-      stdio: ['pipe', 'pipe', 'pipe'],
-      reject: false,
-    })
-
-    const pendingTriggers = [...browserPreset.autoAnswers]
-    let show = false
-
-    child.stdout.on('data', (chunk) => {
-      const text = chunk.toString()
-
-      if (text.includes(browserPreset.output.showAfter)) show = true
-      if (text.includes(browserPreset.output.hideAfter)) show = false
-
-      if (show) process.stdout.write(chunk)
-
-      for (let i = 0; i < pendingTriggers.length; i++) {
-        if (text.includes(pendingTriggers[i].trigger)) {
-          child.stdin.write(pendingTriggers[i].response + '\n')
-          pendingTriggers.splice(i, 1)
-          break
-        }
-      }
-    })
-
-    child.stderr.on('data', (chunk) => process.stderr.write(chunk))
-
-    const result = await child
-
-    if (result.exitCode === 0) {
-      success('opencode-browser installed')
-    } else {
-      warn('opencode-browser install exited with non-zero code')
-    }
-  } catch (err) {
-    error(`Failed to install opencode-browser: ${err.message}`)
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import { header, success, warn, error } from '../../utils/exec.js'
+import os from 'os'
+import path from 'path'
+import { fileURLToPath } from 'url'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const BROWSER_PRESET_PATH = path.resolve(__dirname, '../../presets/browser.json')
+const browserPreset = await fse.readJson(BROWSER_PRESET_PATH)
+
+export async function installBrowser(ctx = {}) {
+  header('Step 9, Installing opencode-browser')
+
+  const installScope = ctx.installScope || 'local'
+  const locationAnswer = browserPreset.locationChoices?.[installScope] ?? browserPreset.locationChoices?.local ?? '2'
+
+  const pendingTriggers = browserPreset.autoAnswers.map(a => ({
+    ...a,
+    response: a.response === '__LOCATION__' ? locationAnswer : a.response,
+  }))
+
+  try {
+    const child = execa(browserPreset.installer.command, browserPreset.installer.args, {
+      cwd: os.homedir(),
+      stdio: ['pipe', 'pipe', 'pipe'],
+      reject: false,
+    })
+
+    let show = false
+    const triggers = [...pendingTriggers]
+
+    child.stdout.on('data', (chunk) => {
+      const text = chunk.toString()
+
+      if (text.includes(browserPreset.output.showAfter)) show = true
+      if (text.includes(browserPreset.output.hideAfter)) show = false
+
+      if (show) process.stdout.write(chunk)
+
+      for (let i = 0; i < triggers.length; i++) {
+        if (text.includes(triggers[i].trigger)) {
+          child.stdin.write(`${triggers[i].response}\n`)
+          triggers.splice(i, 1)
+          break
+        }
+      }
+    })
+
+    child.stderr.on('data', (chunk) => process.stderr.write(chunk))
+
+    const result = await child
+
+    if (result.exitCode === 0) {
+      success('opencode-browser installed')
+    } else {
+      warn('opencode-browser install exited with non-zero code')
+    }
+  } catch (err) {
+    error(`Failed to install opencode-browser: ${err.message}`)
+  }
+}

package/src/steps/clean/index.js

@@ -1,107 +1,108 @@
-import { checkbox } from '@inquirer/prompts'
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { findAiFiles } from '../../utils/copy.js'
-import { header, info, success, warn } from '../../utils/exec.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const CLEAN_PRESET_PATH = path.resolve(__dirname, '../../presets/clean.json')
-const cleanPreset = await fse.readJson(CLEAN_PRESET_PATH)
-
-async function childrenExcludingPreserved(dir) {
-  const results = []
-  if (!await fse.pathExists(dir)) return results
-  const entries = await fse.readdir(dir)
-  for (const entry of entries) {
-    if (cleanPreset.preserveSubfolders.includes(entry)) continue
-    results.push(path.join(dir, entry))
-  }
-  return results
-}
-
-async function isPopulated(filePath) {
-  if (!await fse.pathExists(filePath)) return false
-  const content = await fse.readFile(filePath, 'utf-8')
-  const trimmed = content.trim()
-  if (!trimmed) return false
-  if (trimmed.startsWith('<!-- onboard-prompt')) return false
-  return true
-}
-
-async function hasOpenspecHistory(cwd) {
-  const changesDir = path.join(cwd, 'openspec', 'changes')
-  const archiveDir = path.join(cwd, 'openspec', 'archive')
-  if (await fse.pathExists(changesDir)) {
-    const entries = await fse.readdir(changesDir)
-    if (entries.length > 0) return true
-  }
-  if (await fse.pathExists(archiveDir)) {
-    const entries = await fse.readdir(archiveDir)
-    if (entries.length > 0) return true
-  }
-  return false
-}
-
-export async function cleanAiFiles() {
-  header('Step 2, Existing AI config files')
-
-  const cwd = process.cwd()
-  const ctx = {
-    hasDesign: await isPopulated(path.join(cwd, 'DESIGN.md')),
-    hasArchitecture: await isPopulated(path.join(cwd, 'ARCHITECTURE.md')),
-    hasOpenspec: await hasOpenspecHistory(cwd),
-  }
-
-  if (ctx.hasDesign) info('DESIGN.md exists and is populated, keeping it')
-  if (ctx.hasArchitecture) info('ARCHITECTURE.md exists and is populated, keeping it')
-  if (ctx.hasOpenspec) info('openspec/ history exists, keeping it')
-
-  const flatFiles = await findAiFiles(cwd, cleanPreset.detectFiles)
-  const dirTargets = cleanPreset.directoryTargets
-  const dirEntries = []
-  for (const dirName of dirTargets) {
-    const dirPath = path.join(cwd, dirName)
-    const children = await childrenExcludingPreserved(dirPath)
-    dirEntries.push(...children)
-  }
-
-  const filteredFlat = flatFiles.filter(f => {
-    const rel = path.relative(cwd, f)
-    if (dirTargets.includes(rel)) return false
-    if (cleanPreset.preserve.some(p => rel === p || rel.startsWith(p + path.sep))) return false
-    return true
-  })
-
-  const allToRemove = [...filteredFlat, ...dirEntries]
-
-  if (allToRemove.length === 0) {
-    success('No existing AI config files to remove')
-    return ctx
-  }
-
-  const choices = allToRemove.map(f => ({
-    name: path.relative(cwd, f).replace(/\\/g, '/'),
-    value: f,
-    checked: true,
-  }))
-
-  const selected = await checkbox({
-    message: cleanPreset.selectionMessage,
-    choices,
-  })
-
-  if (!selected || selected.length === 0) {
-    success('No AI config files selected for removal')
-    return ctx
-  }
-
-  warn('Removing selected AI config files:')
-  for (const f of selected) {
-    info('  ' + f.replace(cwd + path.sep, ''))
-    await fse.remove(f)
-  }
-  success('Removed existing AI config files')
-
-  return ctx
-}
+import { checkbox } from '@inquirer/prompts'
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+import { findAiFiles } from '../../utils/copy.js'
+import { header, info, success, warn } from '../../utils/exec.js'
+import { MARKERS } from '../../utils/terminal.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const CLEAN_PRESET_PATH = path.resolve(__dirname, '../../presets/clean.json')
+const cleanPreset = await fse.readJson(CLEAN_PRESET_PATH)
+
+async function childrenExcludingPreserved(dir) {
+  const results = []
+  if (!await fse.pathExists(dir)) return results
+  const entries = await fse.readdir(dir)
+  for (const entry of entries) {
+    if (cleanPreset.preserveSubfolders.includes(entry)) continue
+    results.push(path.join(dir, entry))
+  }
+  return results
+}
+
+async function isPopulated(filePath) {
+  if (!await fse.pathExists(filePath)) return false
+  const content = await fse.readFile(filePath, 'utf-8')
+  const trimmed = content.trim()
+  if (!trimmed) return false
+  if (trimmed.startsWith('<!-- onboard-prompt')) return false
+  return true
+}
+
+async function hasOpenspecHistory(cwd) {
+  const changesDir = path.join(cwd, 'openspec', 'changes')
+  const archiveDir = path.join(cwd, 'openspec', 'archive')
+  if (await fse.pathExists(changesDir)) {
+    const entries = await fse.readdir(changesDir)
+    if (entries.length > 0) return true
+  }
+  if (await fse.pathExists(archiveDir)) {
+    const entries = await fse.readdir(archiveDir)
+    if (entries.length > 0) return true
+  }
+  return false
+}
+
+export async function cleanAiFiles() {
+  header('Step 2, Existing AI config files')
+
+  const cwd = process.cwd()
+  const ctx = {
+    hasDesign: await isPopulated(path.join(cwd, 'DESIGN.md')),
+    hasArchitecture: await isPopulated(path.join(cwd, 'ARCHITECTURE.md')),
+    hasOpenspec: await hasOpenspecHistory(cwd),
+  }
+
+  if (ctx.hasDesign) info('DESIGN.md exists and is populated, keeping it')
+  if (ctx.hasArchitecture) info('ARCHITECTURE.md exists and is populated, keeping it')
+  if (ctx.hasOpenspec) info('openspec/ history exists, keeping it')
+
+  const flatFiles = await findAiFiles(cwd, cleanPreset.detectFiles)
+  const dirTargets = cleanPreset.directoryTargets
+  const dirEntries = []
+  for (const dirName of dirTargets) {
+    const dirPath = path.join(cwd, dirName)
+    const children = await childrenExcludingPreserved(dirPath)
+    dirEntries.push(...children)
+  }
+
+  const filteredFlat = flatFiles.filter(f => {
+    const rel = path.relative(cwd, f)
+    if (dirTargets.includes(rel)) return false
+    if (cleanPreset.preserve.some(p => rel === p || rel.startsWith(p + path.sep))) return false
+    return true
+  })
+
+  const allToRemove = [...filteredFlat, ...dirEntries]
+
+  if (allToRemove.length === 0) {
+    success('No existing AI config files to remove')
+    return ctx
+  }
+
+  const choices = allToRemove.map(f => ({
+    name: path.relative(cwd, f).replace(/\\/g, '/'),
+    value: f,
+    checked: true,
+  }))
+
+  const selected = await checkbox({
+    message: cleanPreset.selectionMessage,
+    choices,
+  })
+
+  if (!selected || selected.length === 0) {
+    success('No AI config files selected for removal')
+    return ctx
+  }
+
+  warn('Removing selected AI config files:')
+  for (const f of selected) {
+    info(`${MARKERS.EMPTY}${f.replace(cwd + path.sep, '')}`)
+    await fse.remove(f)
+  }
+  success('Removed existing AI config files')
+
+  return ctx
+}