opencode-onboard 0.4.3 → 0.4.4

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
+}

package/src/steps/metadata/index.js

@@ -1,62 +1,63 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import path from 'path'
-import { createRequire } from 'node:module'
-import { header, success, warn } from '../../utils/exec.js'
-
-const require = createRequire(import.meta.url)
-const { version: onboardVersion } = require('../../../package.json')
-
-async function detectOpencodeVersion() {
-  try {
-    const result = await execa('opencode', ['--version'], { reject: false })
-    if (result.exitCode !== 0) return null
-    const output = (result.stdout || result.stderr || '').trim()
-    return output || null
-  } catch {
-    return null
-  }
-}
-
-export async function writeOnboardConfig(data) {
-  header('Step 10, Writing onboarding metadata')
-
-  const opencodeVersion = await detectOpencodeVersion()
-  const target = path.join(process.cwd(), '.opencode', 'opencode-onboard.json')
-
-  const payload = {
-    schema: 1,
-    generatedAt: new Date().toISOString(),
-    onboardVersion,
-    opencodeVersion,
-    wizard: {
-      platform: data.platform,
-      sourceMode: data.sourceMode,
-      sourceRoots: data.sourceRoots,
-      maxConcurrentAgents: data.maxConcurrentAgents ?? 4,
-      preserved: {
-        design: !!data.hasDesign,
-        architecture: !!data.hasArchitecture,
-        openspec: !!data.hasOpenspec,
-      },
-      additionalSkillsProvider: data.additionalSkillsProvider,
-      models: {
-        plan: data.planModel,
-        build: data.buildModel,
-        fast: data.fastModel,
-      },
-      optionalTools: data.optionalTools ?? null,
-      cavemanGuidance: data.cavemanGuidance ?? null,
-    },
-    note: 'Informational file only. Editing this file does not change runtime behavior.',
-  }
-
-  try {
-    await fse.ensureDir(path.dirname(target))
-    await fse.writeJson(target, payload, { spaces: 2 })
-    success('Wrote .opencode/opencode-onboard.json')
-    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
-  } catch (err) {
-    warn(`Could not write onboarding metadata: ${err.message}`)
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import path from 'path'
+import { createRequire } from 'node:module'
+import { header, success, warn } from '../../utils/exec.js'
+
+const require = createRequire(import.meta.url)
+const { version: onboardVersion } = require('../../../package.json')
+
+async function detectOpencodeVersion() {
+  try {
+    const result = await execa('opencode', ['--version'], { reject: false })
+    if (result.exitCode !== 0) return null
+    const output = (result.stdout || result.stderr || '').trim()
+    return output || null
+  } catch {
+    return null
+  }
+}
+
+export async function writeOnboardConfig(data) {
+  header('Step 10, Writing onboarding metadata')
+
+  const opencodeVersion = await detectOpencodeVersion()
+  const target = path.join(process.cwd(), '.opencode', 'opencode-onboard.json')
+
+  const payload = {
+    schema: 1,
+    generatedAt: new Date().toISOString(),
+    onboardVersion,
+    opencodeVersion,
+    wizard: {
+      platform: data.platform,
+      sourceMode: data.sourceMode,
+      sourceRoots: data.sourceRoots,
+      maxConcurrentAgents: data.maxConcurrentAgents ?? 4,
+      preserved: {
+        design: !!data.hasDesign,
+        architecture: !!data.hasArchitecture,
+        openspec: !!data.hasOpenspec,
+      },
+      openspec: data.openspec,
+      additionalSkillsProvider: data.additionalSkillsProvider,
+      models: {
+        plan: data.planModel,
+        build: data.buildModel,
+        fast: data.fastModel,
+      },
+      optionalTools: data.optionalTools ?? null,
+      cavemanGuidance: data.cavemanGuidance ?? null,
+    },
+    note: 'Informational file only. Editing this file does not change runtime behavior.',
+  }
+
+  try {
+    await fse.ensureDir(path.dirname(target))
+    await fse.writeJson(target, payload, { spaces: 2 })
+    success('Wrote .opencode/opencode-onboard.json')
+    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
+  } catch (err) {
+    warn(`Could not write onboarding metadata: ${err.message}`)
+  }
+}

package/src/steps/models/format.js

@@ -1,60 +1,61 @@
-import { search } from '@inquirer/prompts'
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const MODELS_PRESET_PATH = path.resolve(__dirname, '../../presets/models.json');
-const modelsPreset = await fse.readJson(MODELS_PRESET_PATH);
-
-function costTier(input) {
-  if (input === undefined || input === null) return '';
-  const tier = modelsPreset.costTiers.find(t => t.max === undefined || input <= t.max);
-  return tier ? ` ${tier.label}` : '';
-}
-
-function costTierDisplay(cost, canonicalCost) {
-  return costTier(canonicalCost !== undefined ? canonicalCost : cost);
-}
-
-function formatPrice(price) {
-  if (price === undefined || price === null) return '?';
-  if (price === 0) return '$0 (subscription)';
-  return `$${price}/M`;
-}
-
-export function buildDisplayModels(rawModels) {
-  return rawModels.map(m => {
-    const priceStr = formatPrice(m.cost);
-    const canonicalNote = m.canonicalCost !== undefined
-      ? ` · official price: ${formatPrice(m.canonicalCost)}/M`
-      : '';
-    return {
-      ...m,
-      label: `${m.name}${costTierDisplay(m.cost, m.canonicalCost)}, ${m.id}`,
-      description: `${priceStr}${canonicalNote} · context: ${m.context ? (m.context / 1000) + 'k' : '?'}`,
-    };
-  });
-}
-
-export async function pickModel(message, models) {
-  return await search({
-    message,
-    source: (input) => {
-      const q = (input || '').toLowerCase();
-      const filtered = q
-        ? models.filter(m =>
-            m.label.toLowerCase().includes(q) ||
-            m.id.toLowerCase().includes(q)
-          )
-        : models;
-      return filtered.slice(0, 50).map(m => ({
-        name: m.label,
-        value: m.id,
-        description: m.description,
-      }));
-    },
-  });
-}
-
-export { modelsPreset };
+import { search } from '@inquirer/prompts'
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const MODELS_PRESET_PATH = path.resolve(__dirname, '../../presets/models.json');
+const modelsPreset = await fse.readJson(MODELS_PRESET_PATH);
+
+function costTier(input) {
+  if (input === undefined || input === null) return '';
+  const tier = modelsPreset.costTiers.find(t => t.max === undefined || input <= t.max);
+  return tier ? ` ${tier.label}` : '';
+}
+
+function costTierDisplay(cost, canonicalCost) {
+  return costTier(canonicalCost !== undefined ? canonicalCost : cost);
+}
+
+function formatPrice(price) {
+  if (price === undefined || price === null) return '?';
+  if (price === 0) return '$0 (subscription)';
+  return `$${price}/M`;
+}
+
+export function buildDisplayModels(rawModels) {
+  return rawModels.map(m => {
+    const priceStr = formatPrice(m.cost);
+    const canonicalNote = m.canonicalCost !== undefined
+      ? ` · official price: ${formatPrice(m.canonicalCost)}/M`
+      : '';
+    const context = m.context ? `${m.context / 1000}k` : '?';
+    return {
+      ...m,
+      label: `${m.name}${costTierDisplay(m.cost, m.canonicalCost)}, ${m.id}`,
+      description: `${priceStr}${canonicalNote} · context: ${context}`,
+    };
+  });
+}
+
+export function pickModel(message, models) {
+  return search({
+    message,
+    source: (input) => {
+      const q = (input || '').toLowerCase();
+      const filtered = q
+        ? models.filter(m =>
+            m.label.toLowerCase().includes(q) ||
+            m.id.toLowerCase().includes(q)
+          )
+        : models;
+      return filtered.slice(0, 50).map(m => ({
+        name: m.label,
+        value: m.id,
+        description: m.description,
+      }));
+    },
+  });
+}
+
+export { modelsPreset };