fix: honor tools overrides via permission migration

[KonaEspresso94]

Summary

• Fix agents.*.tools configuration being ignored in oh-my-opencode.json
• Add migrateAgentConfig() call in mergeAgentConfig() to convert tools → permission
• Add unit tests to verify tools migration works for agent overrides

Changes

• src/agents/utils.ts:

  • Import migrateAgentConfig from shared
  • Call migrateAgentConfig() on override before merging in mergeAgentConfig()

• src/agents/utils.test.ts:

  • Add test: tools: { x: false } is migrated to permission: { x: "deny" }
  • Add test: tools: { x: true } is migrated to permission: { x: "allow" }
  • Add test: tools config is removed after migration

Testing

Unit tests

bun run typecheck
bun test src/agents/utils.test.ts
bun test src/shared/permission-compat.test.ts

Manual Verification

# Test migration works
bun -e "
const { createBuiltinAgents } = require('./src/agents/utils');

async function test() {
  const agents = await createBuiltinAgents([], { 
    explore: { 
      tools: { 
        'jetbrains_get_*': true, 
        'jetbrains_create_*': false 
      } 
    } 
  }, undefined, 'anthropic/claude-opus-4-5');
  
  console.log('permission:', agents.explore.permission);
  console.log('tools:', agents.explore.tools);
}

test();
"

# Expected output:
# permission: { ..., "jetbrains_get_*": "allow", "jetbrains_create_*": "deny" }
# tools: undefined

Verified In Real OpenCode Environment

Pointed OpenCode to the locally built plugin output via file:// (instead of the npm-installed plugin), then confirmed the issue described in #1288 is resolved.

screenshots (Sensitive information masked.)
before :

after :

Related Issues

Fix #1288

---

Summary by cubic

Fix ignored agents.*.tools overrides by migrating tools → permission during config merge. Tools allow/deny flags in oh-my-opencode.json now take effect (fixes #1288).

^{Written for commit 8a5b131. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA. Thank you! ✅
_{Posted by the CLA Assistant Lite bot.}

[KonaEspresso94]

I have read the CLA Document and I hereby sign the CLA

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.