opencode-onboard 0.5.1 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-onboard",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Prepare any codebase for AI agent workflows using OpenCode, OpenSpec, and ensemble orchestration.",
   "keywords": [
     "opencode",

package/src/steps/metadata/index.js

@@ -24,8 +24,15 @@ export async function writeOnboardConfig(data) {
   const opencodeVersion = await detectOpencodeVersion()
   const cwd = data.cwd ?? process.cwd()
   const target = path.join(cwd, '.opencode', 'opencode-onboard.json')
-
-  const payload = {
+  const selectedModels = Object.fromEntries(
+    Object.entries({
+      plan: data.planModel,
+      build: data.buildModel,
+      fast: data.fastModel,
+    }).filter(([, value]) => value)
+  )
+
+  const payload = {
     schema: 1,
     generatedAt: new Date().toISOString(),
     onboardVersion,
@@ -39,17 +46,13 @@ export async function writeOnboardConfig(data) {
         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,
-    },
+      },
+      openspec: data.openspec,
+      additionalSkillsProvider: data.additionalSkillsProvider,
+      ...(Object.keys(selectedModels).length > 0 ? { models: selectedModels } : {}),
+      optionalTools: data.optionalTools ?? null,
+      cavemanGuidance: data.cavemanGuidance ?? null,
+    },
     note: 'Informational file only. Editing this file does not change runtime behavior.',
   }
 

package/src/steps/metadata/metadata.test.js

@@ -104,4 +104,14 @@ describe('writeOnboardConfig()', () => {
     const payload = call[1]
     expect(payload.wizard.platform).toBe('none')
   })
+
+  it('omits model metadata when no model is selected', async () => {
+    execa.mockResolvedValue({ exitCode: 0, stdout: '1', stderr: '' })
+
+    await writeOnboardConfig({ platform: 'github', sourceMode: 'current', sourceRoots: [], cwd: tmpDir })
+
+    const call = fse.writeJson.mock.calls[0]
+    const payload = call[1]
+    expect(payload.wizard.models).toBeUndefined()
+  })
 })

package/src/steps/models/format.js

@@ -23,8 +23,8 @@ function formatPrice(price) {
   return `$${price}/M`;
 }
 
-export function buildDisplayModels(rawModels) {
-  return rawModels.map(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`
@@ -35,27 +35,37 @@ export function buildDisplayModels(rawModels) {
       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 function buildModelChoices(input, models) {
+  const q = (input || '').toLowerCase();
+  const filtered = q
+    ? models.filter(m =>
+        m.label.toLowerCase().includes(q) ||
+        m.id.toLowerCase().includes(q)
+      )
+    : models;
+
+  return [
+    {
+      name: 'None',
+      value: null,
+      description: 'Leave this model unset',
+    },
+    ...filtered.slice(0, 50).map(m => ({
+      name: m.label,
+      value: m.id,
+      description: m.description,
+    })),
+  ];
+}
+
+export function pickModel(message, models) {
+  return search({
+    message,
+    source: input => buildModelChoices(input, models),
+  });
+}
 
 export { modelsPreset };

package/src/steps/models/format.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest'
-import { buildDisplayModels } from './format.js'
+import { buildDisplayModels, buildModelChoices } from './format.js'
 
 describe('buildDisplayModels()', () => {
   it('adds cost tier label for cheap models', () => {
@@ -72,4 +72,30 @@ describe('buildDisplayModels()', () => {
     expect(result[1].id).toBe('cheap/model')
     expect(result[2].id).toBe('mid/model')
   })
-})
+
+  it('keeps None as the first choice with no search', () => {
+    const models = buildDisplayModels([
+      { id: 'openai/gpt-4.1', name: 'GPT-4.1', cost: 2, context: 128000 },
+    ])
+
+    const result = buildModelChoices('', models)
+
+    expect(result[0]).toEqual({
+      name: 'None',
+      value: null,
+      description: 'Leave this model unset',
+    })
+  })
+
+  it('keeps None as the first choice during search', () => {
+    const models = buildDisplayModels([
+      { id: 'openai/gpt-4.1', name: 'GPT-4.1', cost: 2, context: 128000 },
+      { id: 'anthropic/claude-3-5-sonnet', name: 'Claude 3.5 Sonnet', cost: 3, context: 200000 },
+    ])
+
+    const result = buildModelChoices('claude', models)
+
+    expect(result[0].name).toBe('None')
+    expect(result[1].value).toBe('anthropic/claude-3-5-sonnet')
+  })
+})

package/src/steps/models/write.js

@@ -2,12 +2,20 @@ import fse from 'fs-extra'
 import path from 'path'
 import { success } from '../../utils/exec.js'
 
+function updateFrontmatterModel(content, modelId) {
+  return content.replace(
+    /^(---\n)([\s\S]*?)(\n---)/m,
+    (_, start, body, end) => {
+      const withoutModel = body.replace(/^model:\s.*\n?/m, '').replace(/\n+$/, '')
+      const nextBody = modelId ? `${withoutModel}\nmodel: ${modelId}` : withoutModel
+      return `${start}${nextBody}${end}`
+    }
+  )
+}
+
 export async function writeModelToAgent(agentFile, modelId) {
   const content = await fse.readFile(agentFile, 'utf-8');
-  const updated = content.replace(
-    /^(---\n[\s\S]*?)\n---/m,
-    `$1\nmodel: ${modelId}\n---`
-  );
+  const updated = updateFrontmatterModel(content, modelId)
   await fse.writeFile(agentFile, updated, 'utf-8');
 }
 
@@ -29,7 +37,7 @@ export async function writeModelsToConfigs({ planModel, buildModel, fastModel, a
     const file = path.join(agentsDir, `${name}.md`);
     if (await fse.pathExists(file)) {
       await writeModelToAgent(file, buildModel);
-      success(`${name} → ${buildModel}`);
+      if (buildModel) success(`${name} → ${buildModel}`);
     }
   }
 
@@ -37,31 +45,37 @@ export async function writeModelsToConfigs({ planModel, buildModel, fastModel, a
     const file = path.join(agentsDir, `${name}.md`);
     if (await fse.pathExists(file)) {
       await writeModelToAgent(file, fastModel);
-      success(`${name} → ${fastModel}`);
+      if (fastModel) success(`${name} → ${fastModel}`);
     }
   }
 
   const opencodeJsonPath = path.join(cwd, '.opencode', 'opencode.json');
   if (await fse.pathExists(opencodeJsonPath)) {
     const config = await fse.readJson(opencodeJsonPath);
-    config.model = buildModel;
+    if (buildModel) config.model = buildModel;
+    else delete config.model;
     await fse.writeJson(opencodeJsonPath, config, { spaces: 2 });
-    success(`default model -> ${buildModel} (written to .opencode/opencode.json)`);
+    if (buildModel) success(`default model -> ${buildModel} (written to .opencode/opencode.json)`);
   }
 
   const ensembleJsonPath = path.join(cwd, '.opencode', 'ensemble.json');
   if (await fse.pathExists(ensembleJsonPath)) {
     const ensemble = await fse.readJson(ensembleJsonPath);
     delete ensemble.defaultModel;
-    ensemble.modelsByAgent = {
-      ...ensemble.modelsByAgent,
-      plan: planModel,
-      build: buildModel,
-      explore: fastModel,
-    };
+    const modelsByAgent = { ...ensemble.modelsByAgent }
+    if (planModel) modelsByAgent.plan = planModel
+    else delete modelsByAgent.plan
+    if (buildModel) modelsByAgent.build = buildModel
+    else delete modelsByAgent.build
+    if (fastModel) modelsByAgent.explore = fastModel
+    else delete modelsByAgent.explore
+
+    if (Object.keys(modelsByAgent).length > 0) ensemble.modelsByAgent = modelsByAgent
+    else delete ensemble.modelsByAgent
+
     await fse.writeJson(ensembleJsonPath, ensemble, { spaces: 2 });
-    success(`plan model -> ${planModel} (written to .opencode/ensemble.json)`);
-    success(`build model -> ${buildModel} (written to .opencode/ensemble.json)`);
-    success(`fast model -> ${fastModel} (written to .opencode/ensemble.json)`);
+    if (planModel) success(`plan model -> ${planModel} (written to .opencode/ensemble.json)`);
+    if (buildModel) success(`build model -> ${buildModel} (written to .opencode/ensemble.json)`);
+    if (fastModel) success(`fast model -> ${fastModel} (written to .opencode/ensemble.json)`);
   }
 }

package/src/steps/models/write.test.js

@@ -38,7 +38,7 @@ description: A test agent
     expect(updated).toContain('model: anthropic/claude-3-sonnet')
   })
 
-  it('preserves existing frontmatter fields', async () => {
+  it('preserves existing frontmatter fields', async () => {
     const filePath = path.join(tmpDir, 'test-agent.md')
     const original = `---
 name: Test Agent
@@ -54,10 +54,28 @@ custom_field: custom_value
     const updated = fs.readFileSync(filePath, 'utf-8')
     expect(updated).toContain('name: Test Agent')
     expect(updated).toContain('description: A test agent')
-    expect(updated).toContain('custom_field: custom_value')
-    expect(updated).toContain('model: test/model')
-  })
-})
+    expect(updated).toContain('custom_field: custom_value')
+    expect(updated).toContain('model: test/model')
+  })
+
+  it('removes model field when no model is selected', async () => {
+    const filePath = path.join(tmpDir, 'test-agent.md')
+    const original = `---
+name: Test Agent
+model: existing/model
+description: A test agent
+---
+
+# Test Agent`
+    fs.writeFileSync(filePath, original, 'utf-8')
+
+    await writeModelToAgent(filePath, null)
+
+    const updated = fs.readFileSync(filePath, 'utf-8')
+    expect(updated).not.toContain('model:')
+    expect(updated).toContain('description: A test agent')
+  })
+})
 
 describe('writeModelsToConfigs()', () => {
   let tmpDir, agentsDir, opencodeJsonPath
@@ -115,4 +133,27 @@ describe('writeModelsToConfigs()', () => {
 
     expect(success).toHaveBeenCalled()
   })
+
+  it('removes model config entries when None is selected', async () => {
+    const agentFile = path.join(agentsDir, 'basic-engineer.md')
+    const ensembleJsonPath = path.join(tmpDir, '.opencode', 'ensemble.json')
+
+    fs.writeFileSync(agentFile, '---\nname: Basic\nmodel: existing/model\n---', 'utf-8')
+    fs.writeFileSync(opencodeJsonPath, JSON.stringify({ model: 'existing/model', theme: 'dark' }, null, 2), 'utf-8')
+    fs.writeFileSync(ensembleJsonPath, JSON.stringify({ modelsByAgent: { plan: 'a', build: 'b', explore: 'c', keep: 'yes' } }, null, 2), 'utf-8')
+
+    await writeModelsToConfigs({
+      planModel: null,
+      buildModel: null,
+      fastModel: null,
+      agentsDir,
+      cwd: tmpDir,
+      preset: { roles: { build: { agents: ['basic-engineer'] }, fast: { agents: [] } } },
+    })
+
+    expect(fs.readFileSync(agentFile, 'utf-8')).not.toContain('model:')
+    expect(JSON.parse(fs.readFileSync(opencodeJsonPath, 'utf-8'))).toEqual({ theme: 'dark' })
+    expect(JSON.parse(fs.readFileSync(ensembleJsonPath, 'utf-8'))).toEqual({ modelsByAgent: { keep: 'yes' } })
+    expect(success).not.toHaveBeenCalled()
+  })
 })