@plainconceptsplatform/agent-harness 2.9.0 → 3.0.2

package/README.md

@@ -284,17 +284,15 @@ your-project/
 ├── AGENTS.md                        ← bootstrap mode, replaced after first "/repo-initialize"
 ├── ARCHITECTURE.md                  ← prompt for agents to fill in from your codebase
 ├── DESIGN.md                        ← prompt for agents to fill in from your codebase
+├── opencode.jsonc                   ← native OpenCode 2 config: agents, permissions, MCP servers
 ├── .opencode/
-│   ├── opencode.json                ← default model + plugin config
 │   ├── harness.json                 ← harness config: platform, models, maxConcurrentAgents
 │   ├── harness-managed.json         ← hashes of every managed file, so "update" spares your edits
 │   ├── agents/                      ← build.md + plan.md (generated primaries), fullstack-engineer,
 │   │                                   and user-created *-engineer files (all subagents)
-│   ├── tui.json                     ← registers the Subagents sidebar panel
-│   ├── tui/
-│   │   └── pc-subagents.tsx         ← TUI plugin: live Subagents panel in the sidebar
 │   └── plugins/
 │       ├── pc-subagent-monitor.js   ← server plugin: subagent state → .opencode/harness-run.json, and the wave cap
+│       ├── pc-subagent-tiers.js     ← server plugin: derives tier agents and the two primaries
 │       └── pc-system-reminders.js   ← server plugin: gates work on the agent's abilities, denies the never-rules
 └── .agents/
     └── skills/
@@ -304,6 +302,8 @@ your-project/
         └── browser-automation/
 ```
 
+On OpenCode 2 the terminal client is configured globally (`~/.config/opencode/cli.json`), so the harness ships no project-local `tui.json`; the native session sidebar shows subagents without a sidecar panel.
+
 Platform skills ship as suffixed variants (`pc-userstory-gh`, `pc-userstory-az`, `pc-userstory-jira`, `pc-userstory-browser`) and the installer copies only the matching one, renamed to its generic name. Platform operations (ship, review, backlog) are injected directly into the `/ops-*` command files from `cli/fragments/ops-*/` during onboarding. Source-roots metadata lands in `.opencode/source-roots.json`. The always-shipped reminder plugin holds editing, shell and spawning until every ability under the agent's `## Abilities` is loaded, guardrails first, and re-arms after compaction. Token-optimization guidance is injected into `pc-guardrails-generic` marker blocks during onboarding.
 
 ---
@@ -348,7 +348,7 @@ Long unattended agent sessions can consume significant tokens. Set these control
 
 2. **Route models by task type**: use a fast and cheap model (for example `haiku`, `gpt-4o-mini`) for orchestration and status loops; reserve expensive models (for example `sonnet`, `opus`, `gpt-4o`) for implementation tasks only.
 
-3. **Install the quota plugin**: the [`@slkiser/opencode-quota`](https://www.npmjs.com/package/@slkiser/opencode-quota) plugin adds `/quota` and `/quota_status` commands that surface real-time token usage inside OpenCode sessions.
+3. **Install the quota plugin**: the [`@slkiser/opencode-quota`](https://www.npmjs.com/package/@slkiser/opencode-quota) plugin adds `/quota` and `/quota_status` commands that surface real-time token usage inside OpenCode sessions. The harness installs it with popups, toasts and reset notifications switched off (`enableToast`, `resetNotifications`), slash reports kept in the chat (`tuiCommandDisplay: "inline"`), and the sidebar panel left on. Settings live in `opencode-quota/quota-toast.jsonc` at the repository root, which is where opencode-quota 5.x reads them.
 
 4. **Use `/quota` checkpoints**: run `/quota` before starting any `/plan-apply` session and after each agent wave. Pause at 75 percent consumed; stop at 90 percent.
 

package/cli/commands/join.js

@@ -50,11 +50,27 @@ export async function runJoin() {
   header('Step 2, Installing OpenCode plugins')
   const pkgPath = path.join(opencodeDir, 'package.json')
   if (await fse.pathExists(pkgPath)) {
+    // `reject: false` made the catch below unreachable: a failed install exited
+    // non-zero without throwing, so this reported success while the plugin's
+    // dependency was missing and every plugin silently refused to load. Let the
+    // call reject, and confirm the dependency is actually on disk before saying so.
+    const declared = Object.keys(
+      parseJsonc(await fse.readFile(pkgPath, 'utf8'))?.dependencies ?? {},
+    )
     try {
-      await execa('npm', ['install'], { cwd: opencodeDir, reject: false, stdio: 'pipe' })
-      success('OpenCode plugins installed')
-    } catch {
+      await execa('npm', ['install'], { cwd: opencodeDir, stdio: 'pipe' })
+    } catch (err) {
       warn('npm install failed in .opencode/: plugins may not load correctly')
+      if (err?.stderr) info(String(err.stderr).trim().split('\n').slice(-3).join('\n'))
+    }
+    const missing = declared.filter(
+      name => !fse.pathExistsSync(path.join(opencodeDir, 'node_modules', ...name.split('/'))),
+    )
+    if (missing.length === 0) {
+      success('OpenCode plugins installed')
+    } else {
+      warn(`These dependencies are still missing from .opencode/node_modules: ${missing.join(', ')}`)
+      warn('Plugins that import them will not load. Re-run `npm install` there once the registry is reachable.')
     }
   } else {
     warn('No .opencode/package.json found: skipping plugin install')

package/cli/presets/quota.json

@@ -1,5 +1,5 @@
 {
-  "plugin": "@slkiser/opencode-quota@4.2.0",
+  "plugin": "@slkiser/opencode-quota@5.0.1",
   "prompt": {
     "message": "Install opencode-quota with recommended defaults?",
     "default": true,
@@ -11,6 +11,10 @@
     "percentDisplayMode": "used",
     "showSessionTokens": true,
     "enableToast": false,
-    "tuiCommandDisplay": "inline"
+    "resetNotifications": { "enabled": false },
+    "tuiCommandDisplay": "inline",
+    "tuiCompactStatus": { "enabled": false },
+    "tuiPromptBar": { "enabled": false },
+    "tuiSidebarPanel": { "enabled": true }
   }
 }

package/cli/steps/copy/opencode-json.js

@@ -1,178 +1,303 @@
 import fse from 'fs-extra'
-import { applyEdits, modify, parse } from 'jsonc-parser'
+import { parse } from 'jsonc-parser'
 import path from 'node:path'
 import { success } from '../../utils/exec.js'
 
-// The reminder plugin loads every agent ability before work, including
-// user-installed skills outside the pc-* and openspec-* namespaces.
-const SHARED_PERMISSIONS = [
-  ['question', 'allow'],
-  ['todowrite', 'allow'],
-]
+// The harness targets OpenCode 2. This patcher migrates a repository's
+// opencode.jsonc from the V1 shape to the native V2 shape, preserving any keys
+// it does not know about. V2 reads V1 config in memory, but the harness wants a
+// file that is already native so behaviour does not depend on the compatibility
+// layer, and the CI merge step requires comment-free JSON.
+//
+// V1 -> V2 field map this handles:
+//   plugin[]            -> plugins[]
+//   permissions object  -> ordered permissions[] rules (bash->shell, task->subagent)
+//   agent{}             -> agents{} (prompt->system, permission->permissions)
+//   skills.paths[]      -> skills[]
+//   mcp.<name>          -> mcp.servers.<name> (enabled -> !disabled)
+//   compaction.prune    -> removed
+//   compaction.reserved -> compaction.buffer
+//   experimental.mcp_timeout -> mcp.timeout.{catalog,execution}
+//   instructions        -> removed (V2 loads AGENTS.md, not this field)
+
+const CONFIG_SCHEMA = 'https://opencode.ai/config.json'
 
-// agent-browser replaced the @different-ai/opencode-browser plugin; any entry
-// with that prefix in the plugin array is stale and gets stripped on update.
-const STALE_BROWSER_PLUGIN_PREFIX = '@different-ai/opencode-browser'
-const AGENT_BROWSER_MCP = {
-  type: 'local',
-  command: ['agent-browser', 'mcp', '--tools', 'core'],
-  enabled: true,
+// Third-party plugins the harness configures, and how they move to V2. The
+// notifier is dropped: V2 has native attention notifications and sounds.
+const OPENCODE_QUOTA_PACKAGE = '@slkiser/opencode-quota'
+const OPENCODE_QUOTA_VERSION = '5.0.1'
+const DROPPED_PLUGIN_PREFIXES = ['@opencode-ai/', '@mohak34/opencode-notifier', '@different-ai/opencode-browser']
+
+// V1 permission keys -> V2 permission actions.
+const ACTION_RENAMES = {
+  bash: 'shell',
+  task: 'subagent',
+  write: 'edit',
+  patch: 'edit',
 }
 
-function applyModify(text, jsonPath, value) {
-  const edits = modify(text, jsonPath, value, {
-    formattingOptions: { insertSpaces: true, tabSize: 2 },
-  })
-  return applyEdits(text, edits)
+function basePackageName(entry) {
+  if (typeof entry !== 'string') return ''
+  const at = entry.lastIndexOf('@')
+  return at > 0 ? entry.slice(0, at) : entry
 }
 
-export async function patchOpencodeJson(cwd = process.cwd()) {
-  const opencodePath = path.join(cwd, 'opencode.jsonc')
+// V1 permission keys that are not V2 permission actions. todowrite is a built-in
+// tool allowed by the base policy, so it needs no rule.
+const NON_ACTION_PERMISSION_KEYS = new Set(['todowrite'])
 
-  let text
-  if (await fse.pathExists(opencodePath)) {
-    text = await fse.readFile(opencodePath, 'utf-8')
-  } else {
-    text = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
+function permissionMapToRules(map) {
+  const rules = []
+  for (const [key, value] of Object.entries(map ?? {})) {
+    if (value !== 'allow' && value !== 'deny' && value !== 'ask') continue
+    if (NON_ACTION_PERMISSION_KEYS.has(key)) continue
+    rules.push({ action: ACTION_RENAMES[key] ?? key, resource: '*', effect: value })
   }
+  return rules
+}
 
-  const errors = []
-  const parsed = parse(text, errors)
-  if (errors.length > 0 || typeof parsed !== 'object') {
-    return { patched: false, reason: 'parse error' }
+function upsertRules(existing, additions) {
+  const merged = [...existing]
+  for (const rule of additions) {
+    const index = merged.findIndex(r => r.action === rule.action && r.resource === rule.resource)
+    if (index === -1) merged.push(rule)
+    else merged[index] = rule
   }
+  return merged
+}
 
-  // build and plan are overridden, not disabled. Earlier versions disabled them
-  // and pointed default_agent at fullstack-engineer; now they are the only two
-  // primaries and pc-subagent-tiers regenerates their agent files from
-  // fullstack-engineer.md. A repo patched by an earlier version still carries
-  // `disable: true`, which would hide both agents, so that flag is cleared here.
-  const hasStaleDisable = (
-    parsed?.agent?.build?.disable !== undefined ||
-    parsed?.agent?.plan?.disable !== undefined
-  )
-  const needsAgentOverride = hasStaleDisable || !(
-    parsed?.agent?.build?.mode === 'primary' &&
-    parsed?.agent?.plan?.mode === 'primary' &&
-    parsed?.agent?.plan?.permission?.edit === 'deny' &&
-    parsed?.agent?.plan?.permission?.task === 'deny'
-  )
-
-  // default_agent pointing at fullstack-engineer is now invalid: it became a
-  // subagent when build and plan took over as the only primaries, so opencode
-  // has no valid default to open a session with. A deliberate choice of build
-  // or plan is left alone; anything else is repointed at plan, so a session
-  // starts read-only.
-  const currentDefault = parsed?.default_agent
-  const needsDefaultAgent = currentDefault !== 'plan' && currentDefault !== 'build'
-  const needsSkillPermission = parsed?.permission?.skill !== 'allow'
-  const missingSharedPermissions = SHARED_PERMISSIONS.filter(
-    ([key, value]) => parsed?.permission?.[key] !== value,
-  )
-  // opencode only scans .opencode/skills/ by default, but we install to .agents/skills/
-  const needsSkillsPaths = !(
-    Array.isArray(parsed?.skills?.paths) &&
-    parsed.skills.paths.includes('.agents/skills')
-  )
-  const needsCompaction = !(
-    parsed?.compaction?.auto === true &&
-    parsed?.compaction?.prune === true
-  )
-  const staleBrowserPlugins = (Array.isArray(parsed?.plugin) ? parsed.plugin : [])
-    .filter(entry => typeof entry === 'string' && entry.startsWith(STALE_BROWSER_PLUGIN_PREFIX))
-  const needsBrowserMcp = JSON.stringify(parsed?.mcp?.['agent-browser']) !== JSON.stringify(AGENT_BROWSER_MCP)
-
-  if (!needsAgentOverride && !needsDefaultAgent && !needsSkillPermission && missingSharedPermissions.length === 0 && !needsSkillsPaths && !needsCompaction && staleBrowserPlugins.length === 0 && !needsBrowserMcp) {
-    return { patched: false }
+function migratePluginArray(obj) {
+  if (!Array.isArray(obj.plugin)) return false
+  const plugins = []
+  for (const entry of obj.plugin) {
+    if (typeof entry !== 'string') {
+      plugins.push(entry)
+      continue
+    }
+    const base = basePackageName(entry)
+    if (DROPPED_PLUGIN_PREFIXES.some(prefix => base.startsWith(prefix))) continue
+    if (base === OPENCODE_QUOTA_PACKAGE) {
+      plugins.push(`${OPENCODE_QUOTA_PACKAGE}@${OPENCODE_QUOTA_VERSION}`)
+      continue
+    }
+    plugins.push(entry)
   }
+  obj.plugins = plugins
+  delete obj.plugin
+  return true
+}
 
-  // Apply edits sequentially so offsets stay correct
-  if (needsAgentOverride) {
-    // undefined removes the key, clearing the disable left by earlier versions.
-    if (parsed?.agent?.build?.disable !== undefined) {
-      text = applyModify(text, ['agent', 'build', 'disable'], undefined)
+function migrateAgents(obj) {
+  let changed = false
+  const agents = { ...(obj.agents ?? {}) }
+
+  if (obj.agent && typeof obj.agent === 'object' && !Array.isArray(obj.agent)) {
+    for (const [id, raw] of Object.entries(obj.agent)) {
+      if (!raw || typeof raw !== 'object') continue
+      const agent = { ...raw }
+      if (agent.prompt !== undefined) {
+        agent.system = agent.prompt
+        delete agent.prompt
+      }
+      if (agent.disable !== undefined) {
+        agent.disabled = agent.disable
+        delete agent.disable
+      }
+      if (agent.permission && typeof agent.permission === 'object') {
+        agent.permissions = upsertRules(agent.permissions ?? [], permissionMapToRules(agent.permission))
+        delete agent.permission
+      }
+      delete agent.maxSteps
+      agents[id] = agent
     }
-    if (parsed?.agent?.plan?.disable !== undefined) {
-      text = applyModify(text, ['agent', 'plan', 'disable'], undefined)
+    delete obj.agent
+    changed = true
+  }
+
+  // build and plan are the only primaries; plan is read-only. Guarantee both
+  // even when the repo never had an explicit entry.
+  const build = { ...(agents.build ?? {}), mode: 'primary' }
+  delete build.disabled
+  delete build.disable
+  if (JSON.stringify(agents.build) !== JSON.stringify(build)) {
+    agents.build = build
+    changed = true
+  }
+  const plan = {
+    ...(agents.plan ?? {}),
+    mode: 'primary',
+    permissions: upsertRules(agents.plan?.permissions ?? [], [
+      { action: 'edit', resource: '*', effect: 'deny' },
+      { action: 'subagent', resource: '*', effect: 'deny' },
+    ]),
+  }
+  delete plan.disabled
+  delete plan.disable
+  if (JSON.stringify(agents.plan) !== JSON.stringify(plan)) {
+    agents.plan = plan
+    changed = true
+  }
+
+  if (changed) obj.agents = agents
+  return changed
+}
+
+function migrateMcp(obj) {
+  if (!obj.mcp || typeof obj.mcp !== 'object' || Array.isArray(obj.mcp)) return false
+  if (obj.mcp.servers) return false
+  const servers = {}
+  let inheritedTimeout = 300000
+  for (const [name, raw] of Object.entries(obj.mcp)) {
+    if (!raw || typeof raw !== 'object') continue
+    const server = { ...raw }
+    if ('enabled' in server) {
+      server.disabled = !server.enabled
+      delete server.enabled
     }
-    text = applyModify(text, ['agent', 'build', 'mode'], 'primary')
-    text = applyModify(text, ['agent', 'plan', 'mode'], 'primary')
-    text = applyModify(text, ['agent', 'plan', 'permission', 'edit'], 'deny')
-    // Read-only has to include spawning: a plan session that can call task()
-    // can have a build worker make the change for it.
-    text = applyModify(text, ['agent', 'plan', 'permission', 'task'], 'deny')
-  }
-  if (needsDefaultAgent) {
-    text = applyModify(text, ['default_agent'], 'plan')
-  }
-  if (needsSkillPermission) text = applyModify(text, ['permission', 'skill'], 'allow')
-  for (const [key, value] of missingSharedPermissions) {
-    text = applyModify(text, ['permission', key], value)
-  }
-
-  // Ensure .agents/skills is in skills.paths so opencode discovers installed skills
-  if (needsSkillsPaths) {
-    const existingPaths = Array.isArray(parsed?.skills?.paths) ? parsed.skills.paths : []
-    if (!existingPaths.includes('.agents/skills')) {
-      const updatedPaths = [...existingPaths, '.agents/skills']
-      text = applyModify(text, ['skills', 'paths'], updatedPaths)
+    if (typeof server.timeout === 'number') {
+      inheritedTimeout = server.timeout
+      server.timeout = { catalog: server.timeout, execution: server.timeout }
     }
+    servers[name] = server
+  }
+  obj.mcp = {
+    servers,
+    timeout: { catalog: inheritedTimeout, execution: inheritedTimeout },
   }
+  return true
+}
+
+/** Mutate `obj` in place to the native V2 shape; returns true when it changed. */
+function migrateToV2(obj) {
+  let changed = false
 
-  // Enable context pruning to reduce token costs in long agent sessions
-  if (needsCompaction) {
-    text = applyModify(text, ['compaction', 'auto'], true)
-    text = applyModify(text, ['compaction', 'prune'], true)
-    text = applyModify(text, ['compaction', 'reserved'], 10000)
+  if (obj.$schema !== CONFIG_SCHEMA) {
+    obj.$schema = CONFIG_SCHEMA
+    changed = true
   }
 
-  // Swap the retired opencode-browser plugin for the agent-browser MCP server
-  if (staleBrowserPlugins.length > 0) {
-    const keptPlugins = parsed.plugin.filter(entry => !staleBrowserPlugins.includes(entry))
-    text = applyModify(text, ['plugin'], keptPlugins)
+  if ('instructions' in obj) {
+    delete obj.instructions
+    changed = true
   }
-  if (needsBrowserMcp) {
-    text = applyModify(text, ['mcp', 'agent-browser'], AGENT_BROWSER_MCP)
+
+  if (obj.experimental && typeof obj.experimental === 'object') {
+    for (const key of ['openTelemetry', 'batch_tool', 'primary_tools', 'continue_loop_on_deny', 'mcp_timeout']) {
+      if (key in obj.experimental) {
+        delete obj.experimental[key]
+        changed = true
+      }
+    }
+    if (Object.keys(obj.experimental).length === 0) {
+      delete obj.experimental
+      changed = true
+    }
   }
 
-  await fse.writeFile(opencodePath, text, 'utf-8')
-  if (needsAgentOverride) {
-    success('Set build/plan as the primary agents in opencode.jsonc (plan is read-only)')
+  const compaction = obj.compaction && typeof obj.compaction === 'object' ? { ...obj.compaction } : {}
+  if ('prune' in compaction) {
+    delete compaction.prune
+    changed = true
+  }
+  if (compaction.auto !== true) {
+    compaction.auto = true
+    changed = true
+  }
+  if ('reserved' in compaction) {
+    compaction.buffer = compaction.reserved
+    delete compaction.reserved
+    changed = true
   }
-  if (needsDefaultAgent) {
-    success(`default_agent -> plan${currentDefault ? ` (was ${currentDefault})` : ''}`)
+  if (!Number.isFinite(compaction.buffer)) {
+    compaction.buffer = 10000
+    changed = true
   }
-  if (needsSkillPermission) {
-    success('Allowed skill loading in opencode.jsonc')
+  if (JSON.stringify(obj.compaction) !== JSON.stringify(compaction)) {
+    obj.compaction = compaction
+    changed = true
   }
-  if (missingSharedPermissions.length > 0) {
-    success('Allowed question/todowrite tools in opencode.jsonc')
+
+  const additions = []
+  if (obj.permission && typeof obj.permission === 'object' && !Array.isArray(obj.permission)) {
+    additions.push(...permissionMapToRules(obj.permission))
+    delete obj.permission
+    changed = true
   }
-  if (needsSkillsPaths) {
-    success('Added .agents/skills to skills.paths in opencode.jsonc')
+  if (additions.length > 0) {
+    const merged = upsertRules(Array.isArray(obj.permissions) ? obj.permissions : [], additions)
+    if (JSON.stringify(obj.permissions) !== JSON.stringify(merged)) {
+      obj.permissions = merged
+      changed = true
+    }
   }
-  if (needsCompaction) {
-    success('Enabled context pruning (compaction.prune) in opencode.jsonc')
+
+  if (obj.skills && typeof obj.skills === 'object' && !Array.isArray(obj.skills)) {
+    const paths = Array.isArray(obj.skills.paths) ? obj.skills.paths : []
+    obj.skills = paths.includes('.agents/skills') ? paths : [...paths, '.agents/skills']
+    changed = true
+  } else if (Array.isArray(obj.skills)) {
+    if (!obj.skills.includes('.agents/skills')) {
+      obj.skills = [...obj.skills, '.agents/skills']
+      changed = true
+    }
+  } else {
+    obj.skills = ['.agents/skills']
+    changed = true
+  }
+
+  if (migrateAgents(obj)) changed = true
+  if (migrateMcp(obj)) changed = true
+  if (migratePluginArray(obj)) changed = true
+
+  // A session starts read-only: fullstack-engineer is a subagent now, so a
+  // default_agent left pointing at it has no valid primary to open with.
+  if (obj.default_agent !== 'plan' && obj.default_agent !== 'build') {
+    obj.default_agent = 'plan'
+    changed = true
+  }
+
+  return changed
+}
+
+export async function patchOpencodeJson(cwd = process.cwd()) {
+  const opencodePath = path.join(cwd, 'opencode.jsonc')
+
+  let text
+  if (await fse.pathExists(opencodePath)) {
+    text = await fse.readFile(opencodePath, 'utf-8')
+  } else {
+    text = JSON.stringify({ $schema: CONFIG_SCHEMA }, null, 2)
   }
-  if (staleBrowserPlugins.length > 0) {
-    success(`Removed ${staleBrowserPlugins.length} stale opencode-browser plugin entr${staleBrowserPlugins.length === 1 ? 'y' : 'ies'} in opencode.jsonc`)
+
+  const errors = []
+  const parsed = parse(text, errors)
+  if (errors.length > 0 || typeof parsed !== 'object' || parsed === null) {
+    return { patched: false, reason: 'parse error' }
   }
-  if (needsBrowserMcp) {
-    success('Configured agent-browser MCP server in opencode.jsonc')
+
+  if (!migrateToV2(parsed)) {
+    return { patched: false }
   }
 
+  await fse.writeFile(opencodePath, `${JSON.stringify(parsed, null, 2)}\n`, 'utf-8')
+  success('Migrated opencode.jsonc to the native OpenCode 2 shape')
   return { patched: true }
 }
 
+// OpenCode 2 plugin packages. @opencode/plugin is the V2 plugin API; the V1
+// @opencode-ai/plugin and the retired TUI peers are removed on update.
 const OPENCODE_PACKAGE_DEPENDENCIES = {
-  '@opencode-ai/plugin': '1.18.19',
-  '@opentui/core': '0.5.6',
-  '@opentui/solid': '0.5.6',
-  'solid-js': '1.9.12',
+  '@opencode/plugin': '2.0.22',
 }
 
-// agent-browser replaced this plugin; remove it from consumer installs on update
-const STALE_PACKAGE_DEPENDENCIES = ['@different-ai/opencode-browser']
+const STALE_PACKAGE_DEPENDENCIES = [
+  '@opencode-ai/plugin',
+  '@mohak34/opencode-notifier',
+  '@different-ai/opencode-browser',
+  '@opentui/core',
+  '@opentui/solid',
+  'solid-js',
+]
 
 export async function patchOpencodePackage(cwd = process.cwd()) {
   const packagePath = path.join(cwd, '.opencode', 'package.json')
@@ -183,7 +308,7 @@ export async function patchOpencodePackage(cwd = process.cwd()) {
 
   let patched = false
   for (const [name, version] of Object.entries(OPENCODE_PACKAGE_DEPENDENCIES)) {
-    if (!(name in packageJson.dependencies) || packageJson.dependencies[name] === version) continue
+    if (packageJson.dependencies[name] === version) continue
     packageJson.dependencies[name] = version
     patched = true
   }

package/cli/steps/optimization/quota.js

@@ -16,11 +16,6 @@ export function splitPackageSpec(spec) {
   return { name: spec.slice(0, separator), version: spec.slice(separator + 1) }
 }
 
-function ensurePlugin(config) {
-  if (!Array.isArray(config.plugin)) config.plugin = []
-  if (!config.plugin.includes(PLUGIN)) config.plugin.push(PLUGIN)
-}
-
 export async function installQuota(options = {}) {
   if (!options.skipHeader) header('Installing opencode-quota')
 
@@ -57,9 +52,14 @@ export async function installQuota(options = {}) {
     const opencodeDir = path.join(process.cwd(), '.opencode')
     const opencodePath = path.join(process.cwd(), 'opencode.jsonc')
     const pkgPath = path.join(opencodeDir, 'package.json')
-    const tuiPath = path.join(opencodeDir, 'tui.json')
-    const quotaDir = path.join(opencodeDir, 'opencode-quota')
-    const quotaPath = path.join(quotaDir, 'quota-toast.json')
+
+    // opencode-quota 5.x reads its settings from `opencode-quota/quota-toast.jsonc`
+    // next to the OpenCode config (project root), not from `.opencode/`. The old
+    // `.opencode/opencode-quota/quota-toast.json` is the v4 location and is migrated
+    // below so the settings actually take effect.
+    const quotaDir = path.join(process.cwd(), 'opencode-quota')
+    const quotaPath = path.join(quotaDir, 'quota-toast.jsonc')
+    const legacyQuotaPath = path.join(opencodeDir, 'opencode-quota', 'quota-toast.json')
 
     let opencodeText = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
     let opencode
@@ -78,18 +78,20 @@ export async function installQuota(options = {}) {
       ? await fse.readJson(pkgPath)
       : {}
 
-    const tui = await fse.pathExists(tuiPath)
-      ? await fse.readJson(tuiPath)
-      : { $schema: 'https://opencode.ai/tui.json' }
-
-    const plugins = Array.isArray(opencode.plugin) ? opencode.plugin : []
+    // V2 configures server plugins under `plugins`. A V1 `plugin` array found
+    // here is carried over so the quota entry is not lost when the config
+    // patcher has not run first.
+    const plugins = Array.isArray(opencode.plugins)
+      ? [...opencode.plugins]
+      : Array.isArray(opencode.plugin)
+        ? [...opencode.plugin]
+        : []
     if (!plugins.includes(PLUGIN)) {
       plugins.push(PLUGIN)
-      opencodeText = applyEdits(opencodeText, modify(opencodeText, ['plugin'], plugins, {
+      opencodeText = applyEdits(opencodeText, modify(opencodeText, ['plugins'], plugins, {
         formattingOptions: { insertSpaces: true, tabSize: 2 },
       }))
     }
-    ensurePlugin(tui)
 
     if (!pkg.dependencies) pkg.dependencies = {}
     const { name: pkgName, version: pkgVersion } = splitPackageSpec(PLUGIN)
@@ -98,18 +100,32 @@ export async function installQuota(options = {}) {
     await fse.ensureDir(opencodeDir)
     await fse.writeFile(opencodePath, opencodeText, 'utf-8')
     await fse.writeJson(pkgPath, pkg, { spaces: 2 })
-    await fse.writeJson(tuiPath, tui, { spaces: 2 })
 
-    const quotaConfig = await fse.pathExists(quotaPath)
-      ? await fse.readJson(quotaPath)
-      : {}
+    // Carry over any existing settings (new location first, then the legacy
+    // v4 file) and layer the recommended defaults on top. The edge defaults
+    // disable toasts and reset notifications and keep slash reports inline.
+    let quotaConfig = {}
+    if (await fse.pathExists(quotaPath)) {
+      quotaConfig = await fse.readJson(quotaPath)
+    } else if (await fse.pathExists(legacyQuotaPath)) {
+      quotaConfig = await fse.readJson(legacyQuotaPath)
+    }
 
     Object.assign(quotaConfig, quotaPreset.defaults)
 
     await fse.ensureDir(quotaDir)
     await fse.writeJson(quotaPath, quotaConfig, { spaces: 2 })
 
-    success('opencode-quota configured (manual setup)')
+    if (await fse.pathExists(legacyQuotaPath)) {
+      await fse.remove(legacyQuotaPath)
+      const legacyDir = path.dirname(legacyQuotaPath)
+      if ((await fse.readdir(legacyDir)).length === 0) {
+        await fse.remove(legacyDir)
+      }
+      info('Moved opencode-quota settings to opencode-quota/quota-toast.jsonc')
+    }
+
+    success('opencode-quota configured (popups and notifications disabled)')
     info('Restart OpenCode and run /quota to verify')
     return { optedIn: true, installed: true }
   } catch (err) {

package/cli/utils/copy.js

@@ -40,7 +40,6 @@ function isTemplateTest(relativePath) {
 export function isManagedContentPath(relativePath) {
   const normalized = relativePath.split(path.sep).join('/')
   if (normalized.startsWith('.opencode/plugins/')) return true
-  if (normalized === '.opencode/tui/pc-subagents.tsx') return true
   if (normalized.startsWith('.opencode/commands/') && !MARKER_COMMANDS.has(normalized)) return true
   return normalized === 'openspec/specs/.gitkeep' || normalized === 'openspec/changes/archive/.gitkeep'
 }