opencode-onboard 0.4.1 → 0.4.3

package/src/steps/copy/agents.js

@@ -61,6 +61,34 @@ function replaceBetween(content, start, end, replacement) {
   return content.replace(pattern, `${start}\n${replacement.trim()}\n${end}`)
 }
 
+const CONCURRENCY_PLACEHOLDER = '{{MAX_CONCURRENT_AGENTS}}'
+const DEFAULT_MAX_CONCURRENT_AGENTS = 4
+
+export async function patchConcurrency(ctx) {
+  const maxAgents = String(ctx.maxConcurrentAgents ?? DEFAULT_MAX_CONCURRENT_AGENTS)
+  const cwd = process.cwd()
+
+  const filesToPatch = [
+    'AGENTS.md',
+    path.join('.opencode', 'commands', 'opsx-apply.md'),
+    path.join('.opencode', 'skills', 'openspec-apply-change', 'SKILL.md'),
+  ]
+
+  let patched = 0
+  for (const rel of filesToPatch) {
+    const abs = path.join(cwd, rel)
+    if (!await fse.pathExists(abs)) continue
+    const content = await fse.readFile(abs, 'utf-8')
+    if (!content.includes(CONCURRENCY_PLACEHOLDER)) continue
+    await fse.writeFile(abs, content.replaceAll(CONCURRENCY_PLACEHOLDER, maxAgents), 'utf-8')
+    patched++
+  }
+
+  if (patched > 0) {
+    success(`Concurrency limit set to ${maxAgents} agents in ${patched} file(s)`)
+  }
+}
+
 export async function patchAgentsMd(ctx) {
   const agentsMdPath = path.join(process.cwd(), 'AGENTS.md')
   if (!await fse.pathExists(agentsMdPath)) return

package/src/steps/copy/copy.test.js

@@ -12,6 +12,7 @@ vi.mock('../../utils/copy.js', () => ({
 
 vi.mock('./agents.js', () => ({
   patchAgentsMd: vi.fn(),
+  patchConcurrency: vi.fn(),
   patchDevopsManagerMd: vi.fn(),
 }))
 

package/src/steps/copy/index.js

@@ -3,7 +3,7 @@ import path from 'path'
 import { fileURLToPath } from 'url'
 import { copyContent } from '../../utils/copy.js'
 import { error, header, success } from '../../utils/exec.js'
-import { patchAgentsMd, patchDevopsManagerMd } from './agents.js'
+import { patchAgentsMd, patchConcurrency, patchDevopsManagerMd } from './agents.js'
 import { installSkills } from './skills.js'
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
@@ -24,6 +24,7 @@ export async function copyContentStep(platform, ctx = {}) {
     }, { spaces: 2 })
     await patchDevopsManagerMd(platform)
     await patchAgentsMd(ctx)
+    await patchConcurrency(ctx)
     await installSkills()
     success('Files copied to project root')
   } catch (err) {

package/src/steps/metadata/index.js

@@ -33,6 +33,7 @@ export async function writeOnboardConfig(data) {
       platform: data.platform,
       sourceMode: data.sourceMode,
       sourceRoots: data.sourceRoots,
+      maxConcurrentAgents: data.maxConcurrentAgents ?? 4,
       preserved: {
         design: !!data.hasDesign,
         architecture: !!data.hasArchitecture,

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

@@ -25,14 +25,17 @@ import fse from 'fs-extra'
 import { writeOnboardConfig } from './index.js'
 
 describe('writeOnboardConfig()', () => {
-  let tmpDir
+  let tmpDir, originalCwd
 
   beforeEach(() => {
+    vi.clearAllMocks()
     tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'metadata-test-'))
+    originalCwd = process.cwd()
     process.chdir(tmpDir)
   })
 
   afterEach(() => {
+    process.chdir(originalCwd)
     fs.rmSync(tmpDir, { recursive: true, force: true })
   })
 
@@ -57,7 +60,7 @@ describe('writeOnboardConfig()', () => {
     expect(fse.ensureDir).toHaveBeenCalled()
     expect(fse.writeJson).toHaveBeenCalled()
     const call = fse.writeJson.mock.calls[0]
-    const payload = call[0]
+    const payload = call[1]
     expect(payload.schema).toBe(1)
     expect(payload.wizard.platform).toBe('github')
     expect(payload.wizard.models.build).toBe('build-model')
@@ -70,7 +73,7 @@ describe('writeOnboardConfig()', () => {
     await writeOnboardConfig({ platform: 'github', sourceMode: 'current', sourceRoots: [] })
 
     const call = fse.writeJson.mock.calls[0]
-    const payload = call[0]
+    const payload = call[1]
     expect(payload.opencodeVersion).toBe('2.0.0')
   })
 
@@ -80,7 +83,7 @@ describe('writeOnboardConfig()', () => {
     await writeOnboardConfig({ platform: 'github', sourceMode: 'current', sourceRoots: [] })
 
     const call = fse.writeJson.mock.calls[0]
-    const payload = call[0]
+    const payload = call[1]
     expect(payload.opencodeVersion).toBe(null)
   })
 
@@ -90,7 +93,7 @@ describe('writeOnboardConfig()', () => {
     await writeOnboardConfig({ platform: 'github', sourceMode: 'current', sourceRoots: [] })
 
     const call = fse.writeJson.mock.calls[0]
-    const payload = call[0]
+    const payload = call[1]
     expect(payload.note).toContain('Informational file only')
   })
 })

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

@@ -7,7 +7,7 @@ describe('buildDisplayModels()', () => {
 
     const result = buildDisplayModels(raw)
 
-    expect(result[0].label).toContain('[$]')
+    expect(result[0].label).toContain('[$$]')
   })
 
   it('adds cost tier label for mid-range models', () => {
@@ -15,7 +15,7 @@ describe('buildDisplayModels()', () => {
 
     const result = buildDisplayModels(raw)
 
-    expect(result[0].label).toContain('[$$]')
+    expect(result[0].label).toContain('[$$$]')
   })
 
   it('adds cost tier label for expensive models', () => {
@@ -47,7 +47,8 @@ describe('buildDisplayModels()', () => {
 
     const result = buildDisplayModels(raw)
 
-    expect(result[0].description).toContain('cost: ?')
+    expect(result[0].description).toContain('?')
+    expect(result[0].label).not.toContain('[')
   })
 
   it('handles $0 subscription pricing', () => {
@@ -58,7 +59,7 @@ describe('buildDisplayModels()', () => {
     expect(result[0].description).toContain('$0 (subscription)')
   })
 
-  it('sorts models by cost ascending', () => {
+  it('preserves input order (sorting is done upstream by parseModels)', () => {
     const raw = [
       { id: 'expensive/model', name: 'Expensive', cost: 100, context: 1000 },
       { id: 'cheap/model', name: 'Cheap', cost: 1, context: 1000 },
@@ -67,8 +68,8 @@ describe('buildDisplayModels()', () => {
 
     const result = buildDisplayModels(raw)
 
-    expect(result[0].id).toBe('cheap/model')
-    expect(result[1].id).toBe('mid/model')
-    expect(result[2].id).toBe('expensive/model')
+    expect(result[0].id).toBe('expensive/model')
+    expect(result[1].id).toBe('cheap/model')
+    expect(result[2].id).toBe('mid/model')
   })
 })

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

@@ -7,16 +7,6 @@ vi.mock('../../utils/exec.js', () => ({
   success: vi.fn(),
 }))
 
-vi.mock('fs-extra', () => ({
-  default: {
-    readFile: vi.fn(),
-    writeFile: vi.fn(),
-    writeJson: vi.fn(),
-    pathExists: vi.fn(),
-  },
-}))
-
-import fse from 'fs-extra'
 import { success } from '../../utils/exec.js'
 import { writeModelToAgent, writeModelsToConfigs } from './write.js'
 
@@ -24,6 +14,7 @@ describe('writeModelToAgent()', () => {
   let tmpDir
 
   beforeEach(() => {
+    vi.clearAllMocks()
     tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'models-write-test-'))
   })
 
@@ -69,18 +60,22 @@ custom_field: custom_value
 })
 
 describe('writeModelsToConfigs()', () => {
-  let tmpDir, agentsDir, opencodeJsonPath, ensembleJsonPath
+  let tmpDir, agentsDir, opencodeJsonPath, ensembleJsonPath, originalCwd
 
   beforeEach(() => {
+    vi.clearAllMocks()
+    originalCwd = process.cwd()
     tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'models-config-test-'))
     agentsDir = path.join(tmpDir, '.agents', 'agents')
     fs.mkdirSync(agentsDir, { recursive: true })
     opencodeJsonPath = path.join(tmpDir, '.opencode', 'opencode.json')
     ensembleJsonPath = path.join(tmpDir, '.opencode', 'ensemble.json')
     fs.mkdirSync(path.dirname(opencodeJsonPath), { recursive: true })
+    process.chdir(tmpDir)
   })
 
   afterEach(() => {
+    process.chdir(originalCwd)
     fs.rmSync(tmpDir, { recursive: true, force: true })
   })
 
@@ -106,12 +101,15 @@ describe('writeModelsToConfigs()', () => {
   })
 
   it('reports success when writing configs', async () => {
+    const agentFile = path.join(agentsDir, 'back-engineer.md')
+    fs.writeFileSync(agentFile, '---\nname: Back\n---', 'utf-8')
+
     await writeModelsToConfigs({
       planModel: 'plan-model',
       buildModel: 'build-model',
       fastModel: 'fast-model',
-      agentsDir: '/nonexistent',
-      preset: { roles: { build: { agents: [] }, fast: { agents: [] } } },
+      agentsDir,
+      preset: { roles: { build: { agents: ['back-engineer'] }, fast: { agents: [] } } },
     })
 
     expect(success).toHaveBeenCalled()

package/src/steps/openspec/ensemble.js

@@ -36,7 +36,14 @@ export const ENSEMBLE_SECTION = `6. **Implement via ensemble team**
       DO NOT call team_claim yourself, only agents claim tasks.
       DO NOT proceed to 6d until team_tasks_add succeeds.
 
-   **Step 6d.** Discover relevant skills, then spawn specialists.
+   **Step 6d.** Discover relevant skills, then spawn specialists with an INITIAL BATCH of tasks.
+
+      **ROLLING BATCH MODEL:**
+      Agents do NOT receive all their tasks upfront. Instead:
+      - Assign each agent an initial batch of up to 3 unblocked tasks.
+      - When an agent completes its batch and messages back, the lead assigns the next batch of up to 3 unassigned tasks from the board that match the agent's domain.
+      - Repeat until no pending tasks remain on the board.
+      - Only shut down an agent when the board has no more tasks for its domain.
 
       Before spawning, scan \`.agents/skills/\` and read each \`SKILL.md\` description line.
       Match skills to agents by domain:
@@ -49,11 +56,11 @@ export const ENSEMBLE_SECTION = `6. **Implement via ensemble team**
 
       The spawn prompt must contain exactly:
       1. Their name and role on this team
-      2. Which tasks are theirs, include the LITERAL task IDs (e.g. "task-abc123") AND the task content for each. Copy them verbatim from the IDs returned by team_tasks_add. Do NOT paraphrase or omit IDs.
+      2. Their initial batch of tasks (up to 3): include the LITERAL task IDs (e.g. "task-abc123") AND the task content. Copy them verbatim from the IDs returned by team_tasks_add. Do NOT paraphrase or omit IDs.
       3. Key context they need (summarized from context files, do NOT tell them to read files themselves)
       4. The 6 OpenCode tools they have available (these are OpenCode tools, NOT shell commands, call them directly as tools, never via bash):
          team_claim, team_tasks_complete, team_tasks_list, team_tasks_add, team_message, team_broadcast
-      5. How to proceed: for EACH task ID listed above, call team_claim tool with that exact task_id before starting it, call team_tasks_complete tool with that task_id after finishing it, then move to the next task. When all tasks are done or blocked, call team_message to notify lead with results or blockers.
+      5. How to proceed: for EACH task ID listed, call team_claim tool with that exact task_id before starting it, call team_tasks_complete tool with that task_id after finishing it, then move to the next task. When all listed tasks are done, message lead with results. Lead may assign more tasks, do NOT shut down until lead confirms no more tasks.
       6. Which skills to load: list the skill names and paths they MUST read before implementing. Example: "Before starting, read \`.agents/skills/next-best-practices/SKILL.md\` and follow its rules for all Next.js code."
 
       Keep spawn prompts under 600 tokens. Do not describe team internals or how ensemble works.
@@ -86,12 +93,24 @@ export const ENSEMBLE_SECTION = `6. **Implement via ensemble team**
       - \`team_view member:"<name>"\` for a teammate live session
       - \`team_results from:"<name>"\` for full teammate report text
 
-   **Step 6f.** When a teammate messages back, you receive a ping only, the full content is NOT in the notification.
-      Call team_results to read the full message and mark it read. Then for each teammate: team_shutdown -> team_merge.
-      If team_merge blocks ("overlapping local changes"), commit or stash your local changes first, then retry.
-      Fix any other blockers reported.
+   **Step 6f.** When a teammate messages back (rolling re-assignment loop):
+      1. Call \`team_results from:"<name>"\` to read full message.
+      2. Call \`team_tasks_list\` to check remaining pending/unassigned tasks on the board.
+      3. **If there are more unassigned tasks matching this agent's domain:**
+         - Pick up to 3 unassigned, unblocked tasks for this agent's domain.
+         - Send them via \`team_message to:"<name>" text:"Next tasks: [task-<id1>] <desc>, [task-<id2>] <desc>. Claim each with team_claim before starting."\`
+         - Do NOT shut down the agent. Go back to waiting (step 6e).
+      4. **If no more tasks for this agent:**
+         - \`team_shutdown member:"<name>"\`
+         - \`team_merge member:"<name>"\`
+         - If team_merge blocks on local changes: \`git stash\`, retry merge, \`git stash pop\`.
+      5. **If ALL agents are shut down and tasks remain unassigned** (new domain, dependencies unblocked):
+         - Spawn new agents for the remaining tasks (back to step 6d).
+      6. **If ALL tasks are done:** proceed to step 7.
       If a teammate reports rate-limit/quota/token exhaustion, immediately shutdown that teammate and respawn with an available model.
 
+      **ZERO PENDING TASKS GUARANTEE:** Before proceeding to step 7, call \`team_tasks_list\` and verify EVERY task is either \`done\` or \`blocked\`. If any task is \`pending\` and unassigned, assign it to an agent or spawn a new one. Never leave pending tasks orphaned.
+
 7. **Quality check**
 
    Spawn quality engineer with worktree:false (read-only, no file edits):
@@ -123,13 +142,17 @@ export const ENSEMBLE_SECTION = `6. **Implement via ensemble team**
 - NEVER call team_spawn before team_tasks_add, tasks must exist before agents are spawned
 - NEVER poll team_results or team_status in a loop, wait for teammates to message you
 - NEVER call team_claim or team_tasks_complete as lead, only agents call these tools
+- NEVER leave pending tasks orphaned, always verify board is empty before proceeding to step 7
 - ALWAYS pass the LITERAL task IDs returned by team_tasks_add into each agent's spawn prompt, copy the exact IDs, never paraphrase
+- ALWAYS assign initial batch of up to 3 tasks per agent; re-assign next batch (up to 3) via team_message when agent reports done
+- ALWAYS call team_tasks_list after each agent reports done to check for remaining unassigned tasks
 - ALWAYS repeat the same literal task IDs in the team_message start trigger, never send a generic "claim your first task" without the actual IDs
 - NEVER send a start message that omits task IDs; if a task ID is missing from the start message, the agent cannot claim
 - NEVER edit files between team_spawn and team_merge, team_merge blocks on overlapping local changes
 - ALWAYS add every task to the board with team_tasks_add before spawning
 - ALWAYS spawn agents sequentially (wait for each team_spawn result before the next), then send start messages to all of them together
 - ALWAYS instruct agents to call team_claim before each task and team_tasks_complete after
+- ALWAYS shut down + merge agents only when no more tasks remain for their domain
 - If teammates are stuck, use team_message to resend tasks, then wait, never implement directly
 - Mark tasks complete in openspec AFTER specialists finish, not before
 - Pause on errors, blockers, or unclear requirements. Do not guess

package/src/steps/optimization/optimization.test.js

@@ -20,6 +20,21 @@ vi.mock('./caveman.js', () => ({ installCaveman: vi.fn() }))
 vi.mock('./caveman-guidance.js', () => ({ enableCavemanGuidance: vi.fn() }))
 vi.mock('./global.js', () => ({ configureObGlobal: vi.fn() }))
 
+vi.mock('fs-extra', () => ({
+  default: {
+    readJson: vi.fn().mockResolvedValue({
+      info: 'Token optimization info',
+      message: 'Select tools',
+      timeoutMs: 5000,
+      choices: [
+        { value: 'rtk', checked: false },
+        { value: 'quota', checked: false },
+        { value: 'caveman', checked: false },
+      ],
+    }),
+  },
+}))
+
 import { checkbox } from '@inquirer/prompts'
 import { commandExists, warn } from '../../utils/exec.js'
 import { installQuota } from './quota.js'
@@ -34,6 +49,9 @@ describe('tokenOptimizationStep()', () => {
   })
 
   it('runs all optimizations by default selection', async () => {
+    const originalIsTTY = process.stdin.isTTY
+    Object.defineProperty(process.stdin, 'isTTY', { value: true, configurable: true })
+
     checkbox.mockResolvedValue(['rtk', 'quota', 'caveman'])
     commandExists.mockResolvedValue(true)
     installQuota.mockResolvedValue({ optedIn: true, installed: true })
@@ -43,6 +61,8 @@ describe('tokenOptimizationStep()', () => {
 
     const result = await tokenOptimizationStep()
 
+    Object.defineProperty(process.stdin, 'isTTY', { value: originalIsTTY, configurable: true })
+
     expect(commandExists).toHaveBeenCalledWith('rtk')
     expect(installQuota).toHaveBeenCalledWith({ skipHeader: true, skipPrompt: true })
     expect(installCaveman).toHaveBeenCalledWith({ skipHeader: true, skipPrompt: true })

package/src/steps/platform/platform.test.js

@@ -54,7 +54,7 @@ describe('choosePlatform()', () => {
         await checkPlatform('github')
 
         expect(success).toHaveBeenCalledWith('GitHub CLI (gh) available')
-        expect(success).toHaveBeenCalledWith('GitHub CLI authenticated')
+        expect(success).toHaveBeenCalledWith('GitHub CLI (gh) authenticated')
       })
 
       it('warns when gh is installed but not authenticated', async () => {

package/src/steps/source/index.js

@@ -8,6 +8,26 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const SOURCE_PRESET_PATH = path.resolve(__dirname, '../../presets/source.json')
 const sourcePreset = await fse.readJson(SOURCE_PRESET_PATH)
 
+async function listChildFolders(cwd) {
+  const entries = await fse.readdir(cwd)
+  const dirs = []
+
+  for (const name of entries) {
+    if (name.startsWith('.')) continue
+    const abs = path.join(cwd, name)
+    try {
+      const stat = await fse.stat(abs)
+      if (!stat.isDirectory()) continue
+      dirs.push({ name, abs })
+    } catch {
+      // ignore invalid entries
+    }
+  }
+
+  dirs.sort((a, b) => a.name.localeCompare(b.name))
+  return dirs
+}
+
 async function listParentFolders(cwd) {
   const parent = path.resolve(cwd, '..')
   const entries = await fse.readdir(parent)
@@ -47,6 +67,34 @@ export async function chooseSourceScope() {
     return { sourceMode: 'current', sourceRoots: [cwd] }
   }
 
+  if (mode === 'children') {
+    const childFolders = await listChildFolders(cwd)
+    if (childFolders.length === 0) {
+      warn('No child folders found in current directory. Falling back to current folder.')
+      success(`Source scope: ${cwd}`)
+      return { sourceMode: 'current', sourceRoots: [cwd] }
+    }
+
+    const selected = await checkbox({
+      message: sourcePreset.childrenSelectionMessage,
+      choices: childFolders.map(d => ({
+        name: `./${d.name}`,
+        value: d.abs,
+        checked: true,
+      })),
+      required: true,
+    })
+
+    if (!selected || selected.length === 0) {
+      warn('No folders selected. Falling back to current folder.')
+      success(`Source scope: ${cwd}`)
+      return { sourceMode: 'current', sourceRoots: [cwd] }
+    }
+
+    success(`Source scope: ${selected.map(p => path.basename(p)).join(', ')}`)
+    return { sourceMode: 'children-selected', sourceRoots: selected }
+  }
+
   const parentFolders = await listParentFolders(cwd)
   if (parentFolders.length === 0) {
     warn('No sibling folders found in parent directory. Falling back to current folder.')

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

@@ -23,11 +23,13 @@ vi.mock('fs-extra', () => ({
       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(),
+    stat: vi.fn().mockResolvedValue({ isDirectory: () => true }),
   },
 }))
 
@@ -36,15 +38,17 @@ import fse from 'fs-extra'
 import { chooseSourceScope } from './index.js'
 
 describe('chooseSourceScope()', () => {
-  let tmpDir
+  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 })
   })
 
@@ -61,7 +65,7 @@ describe('chooseSourceScope()', () => {
     select.mockResolvedValue('parent')
     const parentDir = path.dirname(tmpDir)
     const siblingDir = path.join(parentDir, 'sibling-project')
-    fs.mkdirSync(siblingDir)
+    fs.mkdirSync(siblingDir, { recursive: true })
     fse.readdir.mockResolvedValue(['sibling-project'])
 
     await chooseSourceScope()
@@ -86,4 +90,35 @@ describe('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/models-pricing.test.js

@@ -75,7 +75,6 @@ describe('parseModels()', () => {
     const result = parseModels(data)
 
     expect(result[0].cost).toBeUndefined()
-    expect(result[0].description).toContain('cost: ?')
   })
 
   it('extracts context limit', () => {