@orderful/droid 0.29.1 → 0.29.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @orderful/droid
 
+## 0.29.2
+
+### Patch Changes
+
+- [#176](https://github.com/Orderful/droid/pull/176) [`8451100`](https://github.com/Orderful/droid/commit/845110083082198213b68f34579beebd845b608f) Thanks [@frytyler](https://github.com/frytyler)! - Add migration to remove stale opencode-skills plugin from opencode.json. This plugin was required before OpenCode got native skills support, but now causes errors because it looks for `skills/` (plural) while native OpenCode looks for `skill/` (singular).
+
 ## 0.29.1
 
 ### Patch Changes

package/dist/bin/droid.js

@@ -7,9 +7,9 @@ import { program } from "commander";
 import inquirer from "inquirer";
 import chalk2 from "chalk";
 import { execSync as execSync2 } from "child_process";
-import { existsSync as existsSync6, readFileSync as readFileSync6, writeFileSync as writeFileSync4, mkdirSync as mkdirSync5 } from "fs";
+import { existsSync as existsSync6, readFileSync as readFileSync7, writeFileSync as writeFileSync5, mkdirSync as mkdirSync5 } from "fs";
 import { join as join8 } from "path";
-import { homedir as homedir3 } from "os";
+import { homedir as homedir4 } from "os";
 
 // src/lib/config.ts
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "fs";
@@ -182,9 +182,9 @@ function setAutoUpdateConfig(updates) {
 import {
   existsSync as existsSync5,
   readdirSync as readdirSync4,
-  readFileSync as readFileSync5,
+  readFileSync as readFileSync6,
   mkdirSync as mkdirSync4,
-  writeFileSync as writeFileSync3,
+  writeFileSync as writeFileSync4,
   rmSync as rmSync2
 } from "fs";
 import { join as join7, dirname as dirname5, basename } from "path";
@@ -525,9 +525,12 @@ import {
   mkdirSync as mkdirSync3,
   renameSync,
   rmSync,
-  readdirSync as readdirSync3
+  readdirSync as readdirSync3,
+  readFileSync as readFileSync5,
+  writeFileSync as writeFileSync3
 } from "fs";
 import { join as join6, dirname as dirname4 } from "path";
+import { homedir as homedir3 } from "os";
 var MIGRATIONS_LOG_FILE = ".migrations.log";
 function getMigrationsLogPath() {
   return join6(getConfigDir(), MIGRATIONS_LOG_FILE);
@@ -728,13 +731,57 @@ function createClaudeCodeCommandCleanupMigration(version2) {
     }
   };
 }
+function createOpenCodePluginCleanupMigration(version2) {
+  return {
+    version: version2,
+    description: "Remove opencode-skills plugin from opencode.json",
+    up: () => {
+      const opencodeConfigPath = join6(
+        homedir3(),
+        ".config",
+        "opencode",
+        "opencode.json"
+      );
+      if (!existsSync4(opencodeConfigPath)) {
+        return;
+      }
+      let config;
+      try {
+        config = JSON.parse(readFileSync5(opencodeConfigPath, "utf-8"));
+      } catch {
+        return;
+      }
+      if (!Array.isArray(config.plugin)) {
+        return;
+      }
+      const pluginIndex = config.plugin.indexOf("opencode-skills");
+      if (pluginIndex === -1) {
+        return;
+      }
+      config.plugin.splice(pluginIndex, 1);
+      if (config.plugin.length === 0) {
+        delete config.plugin;
+      }
+      try {
+        writeFileSync3(
+          opencodeConfigPath,
+          JSON.stringify(config, null, 2) + "\n",
+          "utf-8"
+        );
+      } catch (error) {
+        console.warn(`Warning: Could not update opencode.json: ${error}`);
+      }
+    }
+  };
+}
 var PACKAGE_MIGRATIONS = [
   createPlatformSyncMigration("0.25.0"),
   createConfigSkillNameMigration("0.27.2"),
   createOpenCodeSkillsPathMigration("0.28.0"),
   createClaudeCodeCommandCleanupMigration("0.28.0"),
   // Retry: 0.28.0 migration had platform check that prevented running after platform switch
-  createClaudeCodeCommandCleanupMigration("0.28.1")
+  createClaudeCodeCommandCleanupMigration("0.28.1"),
+  createOpenCodePluginCleanupMigration("0.29.2")
 ];
 var TOOL_MIGRATIONS = {
   brain: [createConfigDirMigration("droid-brain", "0.2.3")],
@@ -873,7 +920,7 @@ function updatePlatformConfigSkills(platform, installedSkills) {
   const configPath = getPlatformConfigPath(platform);
   let content = "";
   if (existsSync5(configPath)) {
-    content = readFileSync5(configPath, "utf-8");
+    content = readFileSync6(configPath, "utf-8");
   }
   const skillLines = installedSkills.map((name) => {
     const relativePath = `skills/${name}/SKILL.md`;
@@ -895,7 +942,7 @@ ${DROID_SKILLS_END}` : "";
   if (!existsSync5(configDir)) {
     mkdirSync4(configDir, { recursive: true });
   }
-  writeFileSync3(configPath, content, "utf-8");
+  writeFileSync4(configPath, content, "utf-8");
 }
 function parseSkillFrontmatter(content) {
   const trimmed = content.trimStart();
@@ -918,7 +965,7 @@ function loadSkillManifest(skillDir) {
   if (!existsSync5(skillMdPath)) {
     return null;
   }
-  const content = readFileSync5(skillMdPath, "utf-8");
+  const content = readFileSync6(skillMdPath, "utf-8");
   const frontmatter = parseSkillFrontmatter(content);
   if (!frontmatter || !frontmatter.name) {
     return null;
@@ -1161,8 +1208,8 @@ function installSkill(skillName) {
       mkdirSync4(targetSkillDir, { recursive: true });
     }
     const skillMdTarget = join7(targetSkillDir, "SKILL.md");
-    const content = readFileSync5(skillMdSource, "utf-8");
-    writeFileSync3(skillMdTarget, content);
+    const content = readFileSync6(skillMdSource, "utf-8");
+    writeFileSync4(skillMdTarget, content);
   }
   const referencesSource = join7(skillDir, "references");
   if (existsSync5(referencesSource)) {
@@ -1176,8 +1223,8 @@ function installSkill(skillName) {
     for (const file of referenceFiles) {
       const sourcePath = join7(referencesSource, file);
       const targetPath = join7(targetReferencesDir, file);
-      const content = readFileSync5(sourcePath, "utf-8");
-      writeFileSync3(targetPath, content);
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
     }
   }
   const scriptsSource = join7(skillDir, "scripts");
@@ -1192,8 +1239,8 @@ function installSkill(skillName) {
     for (const file of scriptFiles) {
       const sourcePath = join7(scriptsSource, file);
       const targetPath = join7(targetScriptsDir, file);
-      const content = readFileSync5(sourcePath, "utf-8");
-      writeFileSync3(targetPath, content);
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
     }
   }
   if (existsSync5(commandsSource)) {
@@ -1213,8 +1260,8 @@ function installSkill(skillName) {
       if (shouldInstall) {
         const sourcePath = join7(commandsSource, file);
         const targetPath = join7(commandsPath, file);
-        const content = readFileSync5(sourcePath, "utf-8");
-        writeFileSync3(targetPath, content);
+        const content = readFileSync6(sourcePath, "utf-8");
+        writeFileSync4(targetPath, content);
       }
     }
   }
@@ -1334,15 +1381,15 @@ function detectGitUsername() {
 function configurePlatformPermissions(platform) {
   const added = [];
   if (platform === "claude-code" /* ClaudeCode */) {
-    const settingsPath = join8(homedir3(), ".claude", "settings.json");
-    const claudeDir = join8(homedir3(), ".claude");
+    const settingsPath = join8(homedir4(), ".claude", "settings.json");
+    const claudeDir = join8(homedir4(), ".claude");
     if (!existsSync6(claudeDir)) {
       mkdirSync5(claudeDir, { recursive: true });
     }
     let settings = {};
     if (existsSync6(settingsPath)) {
       try {
-        settings = JSON.parse(readFileSync6(settingsPath, "utf-8"));
+        settings = JSON.parse(readFileSync7(settingsPath, "utf-8"));
       } catch {
         console.warn(chalk2.yellow("\u26A0 Claude Code settings.json appears corrupted, resetting permissions"));
         settings = {};
@@ -1362,7 +1409,7 @@ function configurePlatformPermissions(platform) {
     }
     if (added.length > 0) {
       try {
-        writeFileSync4(settingsPath, JSON.stringify(settings, null, 2) + "\n", "utf-8");
+        writeFileSync5(settingsPath, JSON.stringify(settings, null, 2) + "\n", "utf-8");
       } catch (e) {
         const message = e instanceof Error ? e.message : "Unknown error";
         return { added: [], alreadyPresent: false, error: `Failed to update Claude Code settings: ${message}` };
@@ -1371,7 +1418,7 @@ function configurePlatformPermissions(platform) {
     return { added, alreadyPresent: added.length === 0 };
   }
   if (platform === "opencode" /* OpenCode */) {
-    const globalConfigDir = join8(homedir3(), ".config", "opencode");
+    const globalConfigDir = join8(homedir4(), ".config", "opencode");
     if (!existsSync6(globalConfigDir)) {
       mkdirSync5(globalConfigDir, { recursive: true });
     }
@@ -2728,7 +2775,7 @@ function ReadmeViewer({ title, content, onClose }) {
 // src/commands/tui/views/ToolExplorer.tsx
 import { Box as Box10, Text as Text11, useInput as useInput5 } from "ink";
 import { useState as useState5, useMemo as useMemo3 } from "react";
-import { existsSync as existsSync7, readFileSync as readFileSync7 } from "fs";
+import { existsSync as existsSync7, readFileSync as readFileSync8 } from "fs";
 import { join as join9 } from "path";
 import { jsx as jsx11, jsxs as jsxs10 } from "react/jsx-runtime";
 function ToolExplorer({ tool, onViewSource, onClose }) {
@@ -2773,12 +2820,12 @@ function ToolExplorer({ tool, onViewSource, onClose }) {
     if (key.return && items.length > 0) {
       const item = items[selectedIndex];
       if (existsSync7(item.path)) {
-        const content = readFileSync7(item.path, "utf-8");
+        const content = readFileSync8(item.path, "utf-8");
         onViewSource(`${tool.name} / ${item.name}`, content);
       } else {
         const yamlPath = item.path.replace(".md", ".yaml");
         if (existsSync7(yamlPath)) {
-          const content = readFileSync7(yamlPath, "utf-8");
+          const content = readFileSync8(yamlPath, "utf-8");
           onViewSource(`${tool.name} / ${item.name}`, content);
         }
       }

package/dist/index.js

@@ -190,9 +190,9 @@ function setAutoUpdateConfig(updates) {
 import {
   existsSync as existsSync5,
   readdirSync as readdirSync4,
-  readFileSync as readFileSync5,
+  readFileSync as readFileSync6,
   mkdirSync as mkdirSync4,
-  writeFileSync as writeFileSync3,
+  writeFileSync as writeFileSync4,
   rmSync as rmSync2
 } from "fs";
 import { join as join7, dirname as dirname5, basename } from "path";
@@ -498,9 +498,12 @@ import {
   mkdirSync as mkdirSync3,
   renameSync,
   rmSync,
-  readdirSync as readdirSync3
+  readdirSync as readdirSync3,
+  readFileSync as readFileSync5,
+  writeFileSync as writeFileSync3
 } from "fs";
 import { join as join6, dirname as dirname4 } from "path";
+import { homedir as homedir3 } from "os";
 var MIGRATIONS_LOG_FILE = ".migrations.log";
 function getMigrationsLogPath() {
   return join6(getConfigDir(), MIGRATIONS_LOG_FILE);
@@ -701,13 +704,57 @@ function createClaudeCodeCommandCleanupMigration(version) {
     }
   };
 }
+function createOpenCodePluginCleanupMigration(version) {
+  return {
+    version,
+    description: "Remove opencode-skills plugin from opencode.json",
+    up: () => {
+      const opencodeConfigPath = join6(
+        homedir3(),
+        ".config",
+        "opencode",
+        "opencode.json"
+      );
+      if (!existsSync4(opencodeConfigPath)) {
+        return;
+      }
+      let config;
+      try {
+        config = JSON.parse(readFileSync5(opencodeConfigPath, "utf-8"));
+      } catch {
+        return;
+      }
+      if (!Array.isArray(config.plugin)) {
+        return;
+      }
+      const pluginIndex = config.plugin.indexOf("opencode-skills");
+      if (pluginIndex === -1) {
+        return;
+      }
+      config.plugin.splice(pluginIndex, 1);
+      if (config.plugin.length === 0) {
+        delete config.plugin;
+      }
+      try {
+        writeFileSync3(
+          opencodeConfigPath,
+          JSON.stringify(config, null, 2) + "\n",
+          "utf-8"
+        );
+      } catch (error) {
+        console.warn(`Warning: Could not update opencode.json: ${error}`);
+      }
+    }
+  };
+}
 var PACKAGE_MIGRATIONS = [
   createPlatformSyncMigration("0.25.0"),
   createConfigSkillNameMigration("0.27.2"),
   createOpenCodeSkillsPathMigration("0.28.0"),
   createClaudeCodeCommandCleanupMigration("0.28.0"),
   // Retry: 0.28.0 migration had platform check that prevented running after platform switch
-  createClaudeCodeCommandCleanupMigration("0.28.1")
+  createClaudeCodeCommandCleanupMigration("0.28.1"),
+  createOpenCodePluginCleanupMigration("0.29.2")
 ];
 var TOOL_MIGRATIONS = {
   brain: [createConfigDirMigration("droid-brain", "0.2.3")],
@@ -805,7 +852,7 @@ function updatePlatformConfigSkills(platform, installedSkills) {
   const configPath = getPlatformConfigPath(platform);
   let content = "";
   if (existsSync5(configPath)) {
-    content = readFileSync5(configPath, "utf-8");
+    content = readFileSync6(configPath, "utf-8");
   }
   const skillLines = installedSkills.map((name) => {
     const relativePath = `skills/${name}/SKILL.md`;
@@ -827,7 +874,7 @@ ${DROID_SKILLS_END}` : "";
   if (!existsSync5(configDir)) {
     mkdirSync4(configDir, { recursive: true });
   }
-  writeFileSync3(configPath, content, "utf-8");
+  writeFileSync4(configPath, content, "utf-8");
 }
 function parseSkillFrontmatter(content) {
   const trimmed = content.trimStart();
@@ -850,7 +897,7 @@ function loadSkillManifest(skillDir) {
   if (!existsSync5(skillMdPath)) {
     return null;
   }
-  const content = readFileSync5(skillMdPath, "utf-8");
+  const content = readFileSync6(skillMdPath, "utf-8");
   const frontmatter = parseSkillFrontmatter(content);
   if (!frontmatter || !frontmatter.name) {
     return null;
@@ -1109,8 +1156,8 @@ function installSkill(skillName) {
       mkdirSync4(targetSkillDir, { recursive: true });
     }
     const skillMdTarget = join7(targetSkillDir, "SKILL.md");
-    const content = readFileSync5(skillMdSource, "utf-8");
-    writeFileSync3(skillMdTarget, content);
+    const content = readFileSync6(skillMdSource, "utf-8");
+    writeFileSync4(skillMdTarget, content);
   }
   const referencesSource = join7(skillDir, "references");
   if (existsSync5(referencesSource)) {
@@ -1124,8 +1171,8 @@ function installSkill(skillName) {
     for (const file of referenceFiles) {
       const sourcePath = join7(referencesSource, file);
       const targetPath = join7(targetReferencesDir, file);
-      const content = readFileSync5(sourcePath, "utf-8");
-      writeFileSync3(targetPath, content);
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
     }
   }
   const scriptsSource = join7(skillDir, "scripts");
@@ -1140,8 +1187,8 @@ function installSkill(skillName) {
     for (const file of scriptFiles) {
       const sourcePath = join7(scriptsSource, file);
       const targetPath = join7(targetScriptsDir, file);
-      const content = readFileSync5(sourcePath, "utf-8");
-      writeFileSync3(targetPath, content);
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
     }
   }
   if (existsSync5(commandsSource)) {
@@ -1161,8 +1208,8 @@ function installSkill(skillName) {
       if (shouldInstall) {
         const sourcePath = join7(commandsSource, file);
         const targetPath = join7(commandsPath, file);
-        const content = readFileSync5(sourcePath, "utf-8");
-        writeFileSync3(targetPath, content);
+        const content = readFileSync6(sourcePath, "utf-8");
+        writeFileSync4(targetPath, content);
       }
     }
   }
@@ -1305,8 +1352,8 @@ function installCommand(commandName, skillName) {
   const actualSourcePath = join7(commandsDir, sourceFile);
   const targetPath = join7(commandsPath, sourceFile);
   try {
-    const content = readFileSync5(actualSourcePath, "utf-8");
-    writeFileSync3(targetPath, content);
+    const content = readFileSync6(actualSourcePath, "utf-8");
+    writeFileSync4(targetPath, content);
     return { success: true, message: `Installed /${commandName}` };
   } catch (error) {
     return { success: false, message: `Failed to install command: ${error}` };

package/dist/lib/migrations.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"migrations.d.ts","sourceRoot":"","sources":["../../src/lib/migrations.ts"],"names":[],"mappings":"AAUA,OAAO,EACL,KAAK,SAAS,EAIf,MAAM,SAAS,CAAC;AA4WjB;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,SAAS,EAAE,CAE/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAc/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,MAAM,GACd,IAAI,CAmBN;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CA2CtC;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,MAAM,EAChB,gBAAgB,EAAE,MAAM,GACvB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAStC;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,cAAc,EAAE,MAAM,GAAG;IAC5D,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB,CAyDA"}
+{"version":3,"file":"migrations.d.ts","sourceRoot":"","sources":["../../src/lib/migrations.ts"],"names":[],"mappings":"AAaA,OAAO,EACL,KAAK,SAAS,EAIf,MAAM,SAAS,CAAC;AA2ajB;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,SAAS,EAAE,CAE/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAc/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,MAAM,GACd,IAAI,CAmBN;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CA2CtC;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,MAAM,EAChB,gBAAgB,EAAE,MAAM,GACvB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAStC;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,cAAc,EAAE,MAAM,GAAG;IAC5D,OAAO,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB,CAyDA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orderful/droid",
-  "version": "0.29.1",
+  "version": "0.29.2",
   "description": "AI workflow toolkit for sharing skills, commands, and agents across the team",
   "type": "module",
   "bin": {

package/src/commands/setup.test.ts

@@ -1,16 +1,12 @@
-import { describe, it, expect, beforeEach, afterEach, mock, spyOn } from 'bun:test';
+import { describe, it, expect, beforeEach, afterEach } from 'bun:test';
 import { existsSync, mkdirSync, rmSync, readFileSync, writeFileSync } from 'fs';
 import { join } from 'path';
 import { tmpdir } from 'os';
-import { Platform } from '../lib/types';
 
 // We need to mock homedir() before importing the module
 // Create a test directory that will act as our fake home
 let testHomeDir: string;
 
-// Mock the os module's homedir function
-const originalHomedir = await import('os').then(m => m.homedir);
-
 describe('configurePlatformPermissions', () => {
   beforeEach(() => {
     testHomeDir = join(tmpdir(), `droid-setup-test-${Date.now()}`);
@@ -23,66 +19,80 @@ describe('configurePlatformPermissions', () => {
     }
   });
 
-  describe('OpenCode plugin configuration', () => {
-    it('should add opencode-skills plugin to empty config', () => {
+  describe('OpenCode plugin cleanup migration', () => {
+    it('should remove opencode-skills plugin from config', () => {
       const configDir = join(testHomeDir, '.config', 'opencode');
       const configPath = join(configDir, 'opencode.json');
 
       mkdirSync(configDir, { recursive: true });
 
-      const config: { plugin?: string[] } = {};
-      if (!Array.isArray(config.plugin)) {
-        config.plugin = [];
-      }
-      config.plugin.push('opencode-skills');
+      // Config with the stale plugin
+      const existingConfig = {
+        plugin: ['opencode-skills', 'some-other-plugin'],
+        theme: 'dark',
+      };
+      writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
 
+      // Simulate migration logic
+      const config = JSON.parse(readFileSync(configPath, 'utf-8'));
+      const pluginIndex = config.plugin.indexOf('opencode-skills');
+      if (pluginIndex !== -1) {
+        config.plugin.splice(pluginIndex, 1);
+      }
       writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
 
       const read = JSON.parse(readFileSync(configPath, 'utf-8'));
-      expect(read.plugin).toContain('opencode-skills');
+      expect(read.plugin).not.toContain('opencode-skills');
+      expect(read.plugin).toContain('some-other-plugin');
+      expect(read.theme).toBe('dark');
     });
 
-    it('should add opencode-skills plugin to config with existing plugins', () => {
+    it('should remove empty plugin array after cleanup', () => {
       const configDir = join(testHomeDir, '.config', 'opencode');
       const configPath = join(configDir, 'opencode.json');
 
       mkdirSync(configDir, { recursive: true });
 
+      // Config with only the stale plugin
       const existingConfig = {
-        plugin: ['some-other-plugin'],
+        plugin: ['opencode-skills'],
         theme: 'dark',
       };
       writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
 
-      // Read and modify (simulating what the function does)
+      // Simulate migration logic
       const config = JSON.parse(readFileSync(configPath, 'utf-8'));
-      if (!config.plugin.includes('opencode-skills')) {
-        config.plugin.push('opencode-skills');
+      const pluginIndex = config.plugin.indexOf('opencode-skills');
+      if (pluginIndex !== -1) {
+        config.plugin.splice(pluginIndex, 1);
+      }
+      if (config.plugin.length === 0) {
+        delete config.plugin;
       }
       writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
 
       const read = JSON.parse(readFileSync(configPath, 'utf-8'));
-      expect(read.plugin).toContain('opencode-skills');
-      expect(read.plugin).toContain('some-other-plugin');
+      expect(read.plugin).toBeUndefined();
       expect(read.theme).toBe('dark');
     });
 
-    it('should not duplicate opencode-skills if already present', () => {
+    it('should do nothing if plugin not present', () => {
       const configDir = join(testHomeDir, '.config', 'opencode');
       const configPath = join(configDir, 'opencode.json');
 
       mkdirSync(configDir, { recursive: true });
 
       const existingConfig = {
-        plugin: ['opencode-skills'],
+        plugin: ['some-other-plugin'],
+        theme: 'dark',
       };
       writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
 
+      // Simulate migration logic
       const config = JSON.parse(readFileSync(configPath, 'utf-8'));
-      const alreadyPresent = config.plugin.includes('opencode-skills');
-
-      expect(alreadyPresent).toBe(true);
-      expect(config.plugin.length).toBe(1);
+      const pluginIndex = config.plugin.indexOf('opencode-skills');
+      expect(pluginIndex).toBe(-1);
+      expect(config.plugin).toHaveLength(1);
     });
 
     it('should handle config without plugin key', () => {
@@ -97,31 +107,32 @@ describe('configurePlatformPermissions', () => {
       };
       writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
 
+      // Simulate migration logic - should not throw
       const config = JSON.parse(readFileSync(configPath, 'utf-8'));
-      if (!Array.isArray(config.plugin)) {
-        config.plugin = [];
+      if (Array.isArray(config.plugin)) {
+        const pluginIndex = config.plugin.indexOf('opencode-skills');
+        if (pluginIndex !== -1) {
+          config.plugin.splice(pluginIndex, 1);
+        }
       }
-      config.plugin.push('opencode-skills');
-      writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
 
-      const read = JSON.parse(readFileSync(configPath, 'utf-8'));
-      expect(read.plugin).toContain('opencode-skills');
-      expect(read.theme).toBe('dark');
-      expect(read.model).toBe('claude-sonnet');
+      expect(config.theme).toBe('dark');
+      expect(config.model).toBe('claude-sonnet');
     });
 
-    it('should create config directory if it does not exist', () => {
-      const nestedDir = join(testHomeDir, '.config', 'opencode');
-      expect(existsSync(nestedDir)).toBe(false);
-
-      mkdirSync(nestedDir, { recursive: true });
-      expect(existsSync(nestedDir)).toBe(true);
+    it('should handle missing config file gracefully', () => {
+      const configDir = join(testHomeDir, '.config', 'opencode');
+      const configPath = join(configDir, 'opencode.json');
 
-      const configPath = join(nestedDir, 'opencode.json');
-      const config = { plugin: ['opencode-skills'] };
-      writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
+      // Don't create the file - simulate missing config
+      expect(existsSync(configPath)).toBe(false);
 
-      expect(existsSync(configPath)).toBe(true);
+      // Migration should not throw
+      if (existsSync(configPath)) {
+        // Would process file here
+      }
+      // No error thrown = success
+      expect(true).toBe(true);
     });
 
     it('should handle corrupted JSON gracefully', () => {
@@ -131,20 +142,17 @@ describe('configurePlatformPermissions', () => {
       mkdirSync(configDir, { recursive: true });
       writeFileSync(configPath, '{ invalid json }}}', 'utf-8');
 
-      let config: { plugin?: string[] } = {};
+      // Simulate migration logic - should not throw
+      let processed = false;
       try {
-        config = JSON.parse(readFileSync(configPath, 'utf-8'));
+        JSON.parse(readFileSync(configPath, 'utf-8'));
+        processed = true;
       } catch {
-        // Invalid JSON, start fresh
-        config = {};
-      }
-
-      if (!Array.isArray(config.plugin)) {
-        config.plugin = [];
+        // Invalid JSON - migration skips this file
+        processed = false;
       }
-      config.plugin.push('opencode-skills');
 
-      expect(config.plugin).toContain('opencode-skills');
+      expect(processed).toBe(false);
     });
   });
 

package/src/lib/migrations.ts

@@ -5,8 +5,11 @@ import {
   renameSync,
   rmSync,
   readdirSync,
+  readFileSync,
+  writeFileSync,
 } from 'fs';
 import { join, dirname } from 'path';
+import { homedir } from 'os';
 import { loadConfig, saveConfig, getConfigDir } from './config';
 import {
   type Migration,
@@ -344,6 +347,68 @@ function createClaudeCodeCommandCleanupMigration(version: string): Migration {
   };
 }
 
+/**
+ * Migration: Remove opencode-skills plugin from opencode.json
+ *
+ * The opencode-skills plugin was required before OpenCode got native skills support.
+ * Now it causes errors because it looks for `skills/` (plural) while native OpenCode
+ * looks for `skill/` (singular). This migration removes the stale plugin entry.
+ */
+function createOpenCodePluginCleanupMigration(version: string): Migration {
+  return {
+    version,
+    description: 'Remove opencode-skills plugin from opencode.json',
+    up: () => {
+      const opencodeConfigPath = join(
+        homedir(),
+        '.config',
+        'opencode',
+        'opencode.json',
+      );
+
+      if (!existsSync(opencodeConfigPath)) {
+        return;
+      }
+
+      let config: { plugin?: string[]; [key: string]: unknown };
+      try {
+        config = JSON.parse(readFileSync(opencodeConfigPath, 'utf-8'));
+      } catch {
+        // Invalid JSON or read error - skip
+        return;
+      }
+
+      if (!Array.isArray(config.plugin)) {
+        return;
+      }
+
+      const pluginIndex = config.plugin.indexOf('opencode-skills');
+      if (pluginIndex === -1) {
+        return;
+      }
+
+      // Remove the plugin
+      config.plugin.splice(pluginIndex, 1);
+
+      // Clean up empty plugin array
+      if (config.plugin.length === 0) {
+        delete config.plugin;
+      }
+
+      try {
+        writeFileSync(
+          opencodeConfigPath,
+          JSON.stringify(config, null, 2) + '\n',
+          'utf-8',
+        );
+      } catch (error) {
+        // Non-fatal: Log warning but continue
+        console.warn(`Warning: Could not update opencode.json: ${error}`);
+      }
+    },
+  };
+}
+
 /**
  * Registry of package-level migrations
  * These run when the @orderful/droid npm package updates
@@ -357,6 +422,7 @@ const PACKAGE_MIGRATIONS: Migration[] = [
   createClaudeCodeCommandCleanupMigration('0.28.0'),
   // Retry: 0.28.0 migration had platform check that prevented running after platform switch
   createClaudeCodeCommandCleanupMigration('0.28.1'),
+  createOpenCodePluginCleanupMigration('0.29.2'),
 ];
 
 /**