@orderful/droid 0.35.0 → 0.35.1

package/.claude-plugin/marketplace.json

@@ -61,7 +61,7 @@
     {
       "name": "droid-codex",
       "description": "Shared organizational knowledge - PRDs, tech designs, domains, proposals, patterns, and explored topics. Use when loading project context, searching codex, capturing decisions, or creating new entries.",
-      "version": "0.2.0",
+      "version": "0.2.1",
       "source": {
         "source": "github",
         "repo": "orderful/droid",

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @orderful/droid
 
+## 0.35.1
+
+### Patch Changes
+
+- [#211](https://github.com/Orderful/droid/pull/211) [`6df8a25`](https://github.com/Orderful/droid/commit/6df8a2511a1680bec65618d0f5e6e684811cd922) Thanks [@frytyler](https://github.com/frytyler)! - Bump codex tool version to force reinstall with fixed subdirectory copying
+
+  Users who installed codex 0.2.0 before the fix would be missing references/review/ subdirectory. Bumping to 0.2.1 ensures they get the complete installation with all reference files.
+
+- [#211](https://github.com/Orderful/droid/pull/211) [`6df8a25`](https://github.com/Orderful/droid/commit/6df8a2511a1680bec65618d0f5e6e684811cd922) Thanks [@frytyler](https://github.com/frytyler)! - Fix skill installation not copying reference and script subdirectories
+
+  The installSkill() function was only copying top-level files from references/ and scripts/ directories, causing subdirectories to be skipped. This blocked the /codex review feature which relies on references/review/workflow.md.
+
+  Added a generic copyDirectoryRecursive() helper that preserves directory structure while maintaining the existing file type filters (.md for references, .ts/.js/.py for scripts). Applied to both references/ and scripts/ copying.
+
 ## 0.35.0
 
 ### Minor Changes

package/dist/bin/droid.js

@@ -1305,6 +1305,22 @@ function updateAllSkills() {
   }
   return result;
 }
+function copyDirectoryRecursive(source, target, fileFilter) {
+  if (!existsSync6(target)) {
+    mkdirSync4(target, { recursive: true });
+  }
+  const entries = readdirSync5(source, { withFileTypes: true });
+  for (const entry of entries) {
+    const sourcePath = join7(source, entry.name);
+    const targetPath = join7(target, entry.name);
+    if (entry.isDirectory()) {
+      copyDirectoryRecursive(sourcePath, targetPath, fileFilter);
+    } else if (entry.isFile() && fileFilter(entry.name)) {
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
+    }
+  }
+}
 function installSkill(skillName) {
   const config = loadConfig();
   const skillPath = findSkillPath(skillName);
@@ -1418,34 +1434,20 @@ function installSkill(skillName) {
   const referencesSource = join7(skillDir, "references");
   if (existsSync6(referencesSource)) {
     const targetReferencesDir = join7(targetSkillDir, "references");
-    if (!existsSync6(targetReferencesDir)) {
-      mkdirSync4(targetReferencesDir, { recursive: true });
-    }
-    const referenceFiles = readdirSync5(referencesSource).filter(
+    copyDirectoryRecursive(
+      referencesSource,
+      targetReferencesDir,
       (f) => f.endsWith(".md")
     );
-    for (const file of referenceFiles) {
-      const sourcePath = join7(referencesSource, file);
-      const targetPath = join7(targetReferencesDir, file);
-      const content = readFileSync6(sourcePath, "utf-8");
-      writeFileSync4(targetPath, content);
-    }
   }
   const scriptsSource = join7(skillDir, "scripts");
   if (existsSync6(scriptsSource)) {
     const targetScriptsDir = join7(targetSkillDir, "scripts");
-    if (!existsSync6(targetScriptsDir)) {
-      mkdirSync4(targetScriptsDir, { recursive: true });
-    }
-    const scriptFiles = readdirSync5(scriptsSource).filter(
+    copyDirectoryRecursive(
+      scriptsSource,
+      targetScriptsDir,
       (f) => f.endsWith(".ts") || f.endsWith(".js") || f.endsWith(".py")
     );
-    for (const file of scriptFiles) {
-      const sourcePath = join7(scriptsSource, file);
-      const targetPath = join7(targetScriptsDir, file);
-      const content = readFileSync6(sourcePath, "utf-8");
-      writeFileSync4(targetPath, content);
-    }
   }
   const activePlatforms = getActivePlatforms(config);
   const targetPlatforms = activePlatforms.length > 0 ? activePlatforms : [config.platform];

package/dist/index.js

@@ -1262,6 +1262,22 @@ function updateAllSkills() {
   }
   return result;
 }
+function copyDirectoryRecursive(source, target, fileFilter) {
+  if (!existsSync6(target)) {
+    mkdirSync4(target, { recursive: true });
+  }
+  const entries = readdirSync5(source, { withFileTypes: true });
+  for (const entry of entries) {
+    const sourcePath = join7(source, entry.name);
+    const targetPath = join7(target, entry.name);
+    if (entry.isDirectory()) {
+      copyDirectoryRecursive(sourcePath, targetPath, fileFilter);
+    } else if (entry.isFile() && fileFilter(entry.name)) {
+      const content = readFileSync6(sourcePath, "utf-8");
+      writeFileSync4(targetPath, content);
+    }
+  }
+}
 function installSkill(skillName) {
   const config = loadConfig();
   const skillPath = findSkillPath(skillName);
@@ -1375,34 +1391,20 @@ function installSkill(skillName) {
   const referencesSource = join7(skillDir, "references");
   if (existsSync6(referencesSource)) {
     const targetReferencesDir = join7(targetSkillDir, "references");
-    if (!existsSync6(targetReferencesDir)) {
-      mkdirSync4(targetReferencesDir, { recursive: true });
-    }
-    const referenceFiles = readdirSync5(referencesSource).filter(
+    copyDirectoryRecursive(
+      referencesSource,
+      targetReferencesDir,
       (f) => f.endsWith(".md")
     );
-    for (const file of referenceFiles) {
-      const sourcePath = join7(referencesSource, file);
-      const targetPath = join7(targetReferencesDir, file);
-      const content = readFileSync6(sourcePath, "utf-8");
-      writeFileSync4(targetPath, content);
-    }
   }
   const scriptsSource = join7(skillDir, "scripts");
   if (existsSync6(scriptsSource)) {
     const targetScriptsDir = join7(targetSkillDir, "scripts");
-    if (!existsSync6(targetScriptsDir)) {
-      mkdirSync4(targetScriptsDir, { recursive: true });
-    }
-    const scriptFiles = readdirSync5(scriptsSource).filter(
+    copyDirectoryRecursive(
+      scriptsSource,
+      targetScriptsDir,
       (f) => f.endsWith(".ts") || f.endsWith(".js") || f.endsWith(".py")
     );
-    for (const file of scriptFiles) {
-      const sourcePath = join7(scriptsSource, file);
-      const targetPath = join7(targetScriptsDir, file);
-      const content = readFileSync6(sourcePath, "utf-8");
-      writeFileSync4(targetPath, content);
-    }
   }
   const activePlatforms = getActivePlatforms(config);
   const targetPlatforms = activePlatforms.length > 0 ? activePlatforms : [config.platform];

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

@@ -1 +1 @@
-{"version":3,"file":"skills.d.ts","sourceRoot":"","sources":["../../src/lib/skills.ts"],"names":[],"mappings":"AAYA,OAAO,EACL,QAAQ,EACR,WAAW,EACX,KAAK,aAAa,EAClB,KAAK,cAAc,EAGpB,MAAM,SAAS,CAAC;AAuBjB;;GAEG;AACH,wBAAgB,mBAAmB,IAAI,MAAM,CAE5C;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAE/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAEjE;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAEhE;AAED;;GAEG;AACH,wBAAgB,0BAA0B,CACxC,QAAQ,EAAE,QAAQ,EAClB,eAAe,EAAE,MAAM,EAAE,GACxB,IAAI,CAyCN;AAwBD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,aAAa,GAAG,IAAI,CA2BxE;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAwB9C;AAED;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,aAAa,EAAE,CA4BlD;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAI3D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,SAAS,EAAE,MAAM,GAAG,cAAc,GAAG,IAAI,CAI1E;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG;IACvD,SAAS,EAAE,OAAO,CAAC;IACnB,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B,CAkBA;AAED;;GAEG;AACH,wBAAgB,oBAAoB,IAAI,KAAK,CAAC;IAC5C,IAAI,EAAE,MAAM,CAAC;IACb,gBAAgB,EAAE,MAAM,CAAC;IACzB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC,CAqBD;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,SAAS,EAAE,MAAM,GAAG;IAC9C,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CA+BA;AAED;;GAEG;AACH,wBAAgB,eAAe,IAAI;IACjC,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC3D,MAAM,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC/C,QAAQ,EAAE,MAAM,CAAC;CAClB,CAiCA;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG;IAC/C,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CA+RA;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG;IACjD,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CAmFA;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,MAAM,CAUlE;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAChC,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB,OAAO,CAkBT;AAED;;GAEG;AACH,wBAAgB,cAAc,CAC5B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAqDvC;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAC9B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAwCvC"}
+{"version":3,"file":"skills.d.ts","sourceRoot":"","sources":["../../src/lib/skills.ts"],"names":[],"mappings":"AAYA,OAAO,EACL,QAAQ,EACR,WAAW,EACX,KAAK,aAAa,EAClB,KAAK,cAAc,EAGpB,MAAM,SAAS,CAAC;AAuBjB;;GAEG;AACH,wBAAgB,mBAAmB,IAAI,MAAM,CAE5C;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAE/D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAEjE;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAEhE;AAED;;GAEG;AACH,wBAAgB,0BAA0B,CACxC,QAAQ,EAAE,QAAQ,EAClB,eAAe,EAAE,MAAM,EAAE,GACxB,IAAI,CAyCN;AAwBD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,aAAa,GAAG,IAAI,CA2BxE;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAC3B,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAwB9C;AAED;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,aAAa,EAAE,CA4BlD;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAI3D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,SAAS,EAAE,MAAM,GAAG,cAAc,GAAG,IAAI,CAI1E;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG;IACvD,SAAS,EAAE,OAAO,CAAC;IACnB,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B,CAkBA;AAED;;GAEG;AACH,wBAAgB,oBAAoB,IAAI,KAAK,CAAC;IAC5C,IAAI,EAAE,MAAM,CAAC;IACb,gBAAgB,EAAE,MAAM,CAAC;IACzB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC,CAqBD;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,SAAS,EAAE,MAAM,GAAG;IAC9C,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CA+BA;AAED;;GAEG;AACH,wBAAgB,eAAe,IAAI;IACjC,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC3D,MAAM,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC/C,QAAQ,EAAE,MAAM,CAAC;CAClB,CAiCA;AAmCD;;GAEG;AACH,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG;IAC/C,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CA+QA;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG;IACjD,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;CACjB,CAmFA;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,MAAM,CAUlE;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAChC,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB,OAAO,CAkBT;AAED;;GAEG;AACH,wBAAgB,cAAc,CAC5B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAqDvC;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAC9B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,GAChB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAwCvC"}

package/dist/tools/codex/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "droid-codex",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Shared organizational knowledge - PRDs, tech designs, domains, proposals, patterns, and explored topics. Use when loading project context, searching codex, capturing decisions, or creating new entries.",
   "author": {
     "name": "Orderful",

package/dist/tools/codex/TOOL.yaml

@@ -1,6 +1,6 @@
 name: codex
 description: "Shared organizational knowledge - PRDs, tech designs, domains, proposals, patterns, and explored topics. Use when loading project context, searching codex, capturing decisions, or creating new entries."
-version: 0.2.0
+version: 0.2.1
 status: beta
 
 includes:

package/package.json

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

package/src/lib/skills.ts

@@ -393,6 +393,39 @@ export function updateAllSkills(): {
   return result;
 }
 
+/**
+ * Recursively copy a directory and all its contents
+ * @param source Source directory path
+ * @param target Target directory path
+ * @param fileFilter Function that returns true if the file should be copied
+ */
+function copyDirectoryRecursive(
+  source: string,
+  target: string,
+  fileFilter: (filename: string) => boolean,
+): void {
+  if (!existsSync(target)) {
+    mkdirSync(target, { recursive: true });
+  }
+
+  const entries = readdirSync(source, { withFileTypes: true });
+
+  for (const entry of entries) {
+    const sourcePath = join(source, entry.name);
+    const targetPath = join(target, entry.name);
+
+    if (entry.isDirectory()) {
+      // Recursively copy subdirectory
+      copyDirectoryRecursive(sourcePath, targetPath, fileFilter);
+    } else if (entry.isFile() && fileFilter(entry.name)) {
+      // Copy file if it passes the filter
+      const content = readFileSync(sourcePath, 'utf-8');
+      writeFileSync(targetPath, content);
+    }
+    // Ignore files that don't pass the filter
+  }
+}
+
 /**
  * Install a skill
  */
@@ -553,36 +586,20 @@ export function installSkill(skillName: string): {
   const referencesSource = join(skillDir, 'references');
   if (existsSync(referencesSource)) {
     const targetReferencesDir = join(targetSkillDir, 'references');
-    if (!existsSync(targetReferencesDir)) {
-      mkdirSync(targetReferencesDir, { recursive: true });
-    }
-    const referenceFiles = readdirSync(referencesSource).filter((f) =>
+    copyDirectoryRecursive(referencesSource, targetReferencesDir, (f) =>
       f.endsWith('.md'),
     );
-    for (const file of referenceFiles) {
-      const sourcePath = join(referencesSource, file);
-      const targetPath = join(targetReferencesDir, file);
-      const content = readFileSync(sourcePath, 'utf-8');
-      writeFileSync(targetPath, content);
-    }
   }
 
   // Copy scripts if present (deterministic CLI scripts for the skill)
   const scriptsSource = join(skillDir, 'scripts');
   if (existsSync(scriptsSource)) {
     const targetScriptsDir = join(targetSkillDir, 'scripts');
-    if (!existsSync(targetScriptsDir)) {
-      mkdirSync(targetScriptsDir, { recursive: true });
-    }
-    const scriptFiles = readdirSync(scriptsSource).filter(
+    copyDirectoryRecursive(
+      scriptsSource,
+      targetScriptsDir,
       (f) => f.endsWith('.ts') || f.endsWith('.js') || f.endsWith('.py'),
     );
-    for (const file of scriptFiles) {
-      const sourcePath = join(scriptsSource, file);
-      const targetPath = join(targetScriptsDir, file);
-      const content = readFileSync(sourcePath, 'utf-8');
-      writeFileSync(targetPath, content);
-    }
   }
 
   // Get active platforms for multi-platform install

package/src/tools/codex/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "droid-codex",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Shared organizational knowledge - PRDs, tech designs, domains, proposals, patterns, and explored topics. Use when loading project context, searching codex, capturing decisions, or creating new entries.",
   "author": {
     "name": "Orderful",

package/src/tools/codex/TOOL.yaml

@@ -1,6 +1,6 @@
 name: codex
 description: "Shared organizational knowledge - PRDs, tech designs, domains, proposals, patterns, and explored topics. Use when loading project context, searching codex, capturing decisions, or creating new entries."
-version: 0.2.0
+version: 0.2.1
 status: beta
 
 includes: