fix(converters): always allow skill tool when emitting skill stubs under from-commands

applyPermissions only scanned plugin.commands for allowedTools, so a
skill-only plugin (or any plugin whose commands don't list 'skill') would
produce permission.skill = "deny", immediately breaking every generated
stub. When hasSkillStubs is true, unconditionally add 'skill' to the
enabled set in from-commands mode.

Author: lucabattistini

src/converters/claude-to-opencode.ts

@@ -116,7 +116,7 @@ export function convertClaudeToOpenCode(
     mcp: mcp && Object.keys(mcp).length > 0 ? mcp : undefined,
   }
 
-  applyPermissions(config, plugin.commands, options.permissions)
+  applyPermissions(config, plugin.commands, options.permissions, skillStubs.length > 0)
 
   return {
     pluginName: plugin.manifest.name,
@@ -381,6 +381,7 @@ function applyPermissions(
   config: OpenCodeConfig,
   commands: ClaudeCommand[],
   mode: PermissionMode,
+  hasSkillStubs = false,
 ) {
   if (mode === "none") return
 
@@ -419,6 +420,12 @@ function applyPermissions(
         }
       }
     }
+    // Skill stubs require the `skill` tool to load the skill at invocation time.
+    // If we're emitting stubs, ensure `skill` is allowed even when no explicit
+    // command listed it in allowed-tools.
+    if (hasSkillStubs) {
+      enabled.add("skill")
+    }
   }
 
   const permission: Record<string, "allow" | "deny" | Record<string, "allow" | "deny">> = {}

tests/converter.test.ts

@@ -163,6 +163,30 @@ describe("convertClaudeToOpenCode", () => {
     expect(parsed.data.mode).toBe("subagent")
   })
 
+  test("from-commands mode: skill tool is allowed when skill stubs are emitted", async () => {
+    // Regression: applyPermissions only scanned plugin.commands for allowedTools,
+    // so a plugin with no explicit commands (or none listing "skill") would get
+    // permission.skill = "deny", causing every generated stub to fail immediately.
+    const plugin = await loadClaudePlugin(fixtureRoot)
+    // Build a minimal plugin with skills but no explicit commands.
+    const skillOnlyPlugin: ClaudePlugin = {
+      ...plugin,
+      commands: [],
+    }
+    const bundle = convertClaudeToOpenCode(skillOnlyPlugin, {
+      agentMode: "subagent",
+      inferTemperature: false,
+      permissions: "from-commands",
+    })
+
+    // Skill stubs should be emitted (skill-one is opencode-eligible)
+    expect(bundle.commandFiles.some((f) => f.name === "skill-one")).toBe(true)
+
+    // The skill tool must be allowed so the stubs can actually load the skill
+    const permission = bundle.config.permission as Record<string, string>
+    expect(permission.skill).toBe("allow")
+  })
+
   test("normalizes models and infers temperature", async () => {
     const plugin = await loadClaudePlugin(fixtureRoot)
     const bundle = convertClaudeToOpenCode(plugin, {