@orderful/droid 0.7.0 → 0.9.0

package/dist/skills/project/SKILL.md

@@ -1,11 +1,6 @@
 ---
 name: project
-description: >-
-  Manage project context files for persistent AI memory across sessions.
-  Load project context before working (/project {name}), update with
-  new learnings (/project update), or create new projects (/project create).
-  Use when working on multi-session features, refactors, or any work that
-  benefits from accumulated context.
+description: "Manage project context files for persistent AI memory across sessions. Load project context before working (/project {name}), update with new learnings (/project update), or create new projects (/project create). Use when working on multi-session features, refactors, or any work that benefits from accumulated context."
 globs:
   - "**/PROJECT.md"
 alwaysApply: false
@@ -32,31 +27,33 @@ Chat history disappears. Projects persist.
 
 ## Configuration
 
-Check `~/.droid/skills/project/overrides.yaml` for user settings:
+**IMPORTANT:** Before using any default paths, ALWAYS read `~/.droid/skills/project/overrides.yaml` first. If `projects_dir` is configured there, use that path. Only fall back to defaults if the file doesn't exist or lacks a `projects_dir` setting.
 
 | Setting | Default | Description |
 |---------|---------|-------------|
-| `projects_dir` | `~/{ai_tool}/projects` | Where projects are stored (varies by AI tool) |
+| `projects_dir` | (see below) | Where projects are stored |
 | `preset` | `markdown` | Output format: `markdown` or `obsidian` |
 
-Default `projects_dir` by AI tool:
+Default `projects_dir` by AI tool (only if not configured):
 - **claude-code**: `~/.claude/projects`
-- **opencode**: `~/.opencode/projects`
+- **opencode**: `~/.config/opencode/projects`
 
 ## Commands
 
 | Command | Action |
 |---------|--------|
 | `/project` | List and select a project |
-| `/project {keywords}` | Fuzzy-match and load |
-| `/project create {name}` | Create new project |
+| `/project {keywords}` | **Search** for existing project (fuzzy-match and load) |
+| `/project create {name}` | Create new project (requires `create` keyword) |
 | `/project update` | Update from conversation context |
 
+**IMPORTANT:** The default action for `/project {keywords}` is to **SEARCH** for existing projects, NOT create. Only use `/project create {name}` when the user explicitly wants to create a new project.
+
 ## Loading a Project
 
 **Trigger:** `/project {keywords}` or user asks to load/open a project
 
-**TLDR:** Fuzzy-match keywords against project folders, read PROJECT.md, summarize context.
+**TLDR:** Search for and load an existing project. Fuzzy-match keywords against project folders, read PROJECT.md, summarize context.
 
 Full procedure: `references/loading.md`
 

package/dist/skills/project/SKILL.yaml

@@ -1,11 +1,6 @@
 name: project
-description: >-
-  Manage project context files for persistent AI memory across sessions.
-  Load project context before working (/project {name}), update with
-  new learnings (/project update), or create new projects (/project create).
-  Use when working on multi-session features, refactors, or any work that
-  benefits from accumulated context.
-version: 0.1.0
+description: "Manage project context files for persistent AI memory across sessions. Load project context before working (/project {name}), update with new learnings (/project update), or create new projects (/project create). Use when working on multi-session features, refactors, or any work that benefits from accumulated context."
+version: 0.1.1
 status: beta
 dependencies: []
 provides_output: false

package/dist/skills/project/commands/project.md

@@ -1,6 +1,6 @@
 ---
 description: Manage project context files for persistent AI memory across sessions
-argument-hint: [name | update [name] | create [name]]
+argument-hint: "[{keywords} | update | create {name}]"
 allowed-tools: Read, Write, Edit, Glob, Bash(mkdir:*), Bash(ls:*)
 ---
 
@@ -12,20 +12,25 @@ Entry point for project context management. See the **project skill** for full b
 
 $ARGUMENTS
 
+## Default Behavior
+
+**IMPORTANT:** When given keywords (e.g., `/project audit-log`), the default action is to **SEARCH** for existing projects, NOT create. Only create a project when the `create` keyword is explicitly used.
+
 ## Usage
 
 ```
 /project                    # List and select a project
-/project {keywords}         # Fuzzy-match and load
+/project {keywords}         # SEARCH: Fuzzy-match and load existing project
 /project update             # Update from conversation context
 /project update {name}      # Update specific project
 /project create             # Create new project interactively
-/project create {name}      # Create with name
+/project create {name}      # Create with name (requires "create" keyword)
 ```
 
 ## Configuration
 
-From `~/.droid/skills/project/overrides.yaml`:
+**ALWAYS read `~/.droid/skills/project/overrides.yaml` first.** Use configured values if present, only fall back to defaults if missing.
+
 - `projects_dir` - Where projects live (default varies by AI tool)
 - `preset` - Template format: `markdown` or `obsidian`
 

package/dist/skills/project/references/creating.md

@@ -4,21 +4,26 @@
 
 ## Procedure
 
-1. **Get project name**
+1. **Read config first**
+   - Read `~/.droid/skills/project/overrides.yaml`
+   - Use `projects_dir` if configured, otherwise use default for current AI tool
+   - Use `preset` if configured (markdown or obsidian)
+
+2. **Get project name**
    - Use provided name, or ask if not provided
    - Convert to kebab-case for folder name
    - Convert to Title Case for display name
 
-2. **Create project folder**
+3. **Create project folder**
    - Path: `{projects_dir}/{kebab-case-name}/`
    - Verify folder doesn't already exist
 
-3. **Create files from templates** (see `templates.md`)
+4. **Create files from templates** (see `templates.md`)
    - `PROJECT.md` - Main context file
    - `CHANGELOG.md` - Version history
    - Format varies by `preset` config (markdown vs obsidian)
 
-4. **Confirm creation**
+5. **Confirm creation**
    - Show created folder path
    - Offer to help fill in sections (Overview, Goals, Technical Details)
 

package/dist/skills/project/references/loading.md

@@ -2,25 +2,31 @@
 
 **Trigger:** `/project {keywords}` or user asks to load/open a project
 
+**IMPORTANT:** This is a SEARCH operation. When keywords are provided, search for existing projects - do NOT create a new project. Creating requires the explicit `/project create {name}` command.
+
 ## Procedure
 
-1. **List projects** in configured `projects_dir`
+1. **Read config first**
+   - Read `~/.droid/skills/project/overrides.yaml`
+   - Use `projects_dir` if configured, otherwise use default for current AI tool
+
+2. **List projects** in configured `projects_dir`
    - Each subfolder with a `PROJECT.md` is a project
 
-2. **If no name provided:**
+3. **If no name provided:**
    - Use AskUserQuestion to present available projects
    - Let user select which to load
 
-3. **If name/keywords provided:**
+4. **If name/keywords provided:**
    - Parse as space-separated keywords (e.g., "transaction templates" → `["transaction", "templates"]`)
    - Find folders where name contains ALL keywords (case-insensitive, hyphens as word separators)
 
-4. **Based on matches:**
+5. **Based on matches:**
    - **No matches**: List available projects, ask user to select
    - **One match**: Read `{folder}/PROJECT.md`
    - **Multiple matches**: Use AskUserQuestion to select from matches
 
-5. **After loading:**
+6. **After loading:**
    - Confirm which project was loaded
    - Summarize key context (2-3 sentences)
    - Use project contents for all subsequent work in the session

package/package.json

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

package/src/commands/setup.test.ts

@@ -0,0 +1,276 @@
+import { describe, it, expect, beforeEach, afterEach, mock, spyOn } from 'bun:test';
+import { existsSync, mkdirSync, rmSync, readFileSync, writeFileSync } from 'fs';
+import { join } from 'path';
+import { tmpdir } from 'os';
+import { AITool } from '../lib/types.js';
+
+// 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('configureAIToolPermissions', () => {
+  beforeEach(() => {
+    testHomeDir = join(tmpdir(), `droid-setup-test-${Date.now()}`);
+    mkdirSync(testHomeDir, { recursive: true });
+  });
+
+  afterEach(() => {
+    if (existsSync(testHomeDir)) {
+      rmSync(testHomeDir, { recursive: true });
+    }
+  });
+
+  describe('OpenCode plugin configuration', () => {
+    it('should add opencode-skills plugin to empty 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');
+
+      writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
+
+      const read = JSON.parse(readFileSync(configPath, 'utf-8'));
+      expect(read.plugin).toContain('opencode-skills');
+    });
+
+    it('should add opencode-skills plugin to config with existing plugins', () => {
+      const configDir = join(testHomeDir, '.config', 'opencode');
+      const configPath = join(configDir, 'opencode.json');
+
+      mkdirSync(configDir, { recursive: true });
+
+      const existingConfig = {
+        plugin: ['some-other-plugin'],
+        theme: 'dark',
+      };
+      writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
+
+      // Read and modify (simulating what the function does)
+      const config = JSON.parse(readFileSync(configPath, 'utf-8'));
+      if (!config.plugin.includes('opencode-skills')) {
+        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.plugin).toContain('some-other-plugin');
+      expect(read.theme).toBe('dark');
+    });
+
+    it('should not duplicate opencode-skills if already present', () => {
+      const configDir = join(testHomeDir, '.config', 'opencode');
+      const configPath = join(configDir, 'opencode.json');
+
+      mkdirSync(configDir, { recursive: true });
+
+      const existingConfig = {
+        plugin: ['opencode-skills'],
+      };
+      writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
+
+      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);
+    });
+
+    it('should handle config without plugin key', () => {
+      const configDir = join(testHomeDir, '.config', 'opencode');
+      const configPath = join(configDir, 'opencode.json');
+
+      mkdirSync(configDir, { recursive: true });
+
+      const existingConfig = {
+        theme: 'dark',
+        model: 'claude-sonnet',
+      };
+      writeFileSync(configPath, JSON.stringify(existingConfig, null, 2), 'utf-8');
+
+      const config = JSON.parse(readFileSync(configPath, 'utf-8'));
+      if (!Array.isArray(config.plugin)) {
+        config.plugin = [];
+      }
+      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');
+    });
+
+    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);
+
+      const configPath = join(nestedDir, 'opencode.json');
+      const config = { plugin: ['opencode-skills'] };
+      writeFileSync(configPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
+
+      expect(existsSync(configPath)).toBe(true);
+    });
+
+    it('should handle corrupted JSON gracefully', () => {
+      const configDir = join(testHomeDir, '.config', 'opencode');
+      const configPath = join(configDir, 'opencode.json');
+
+      mkdirSync(configDir, { recursive: true });
+      writeFileSync(configPath, '{ invalid json }}}', 'utf-8');
+
+      let config: { plugin?: string[] } = {};
+      try {
+        config = JSON.parse(readFileSync(configPath, 'utf-8'));
+      } catch {
+        // Invalid JSON, start fresh
+        config = {};
+      }
+
+      if (!Array.isArray(config.plugin)) {
+        config.plugin = [];
+      }
+      config.plugin.push('opencode-skills');
+
+      expect(config.plugin).toContain('opencode-skills');
+    });
+  });
+
+  describe('Claude Code permissions configuration', () => {
+    it('should add permissions to empty settings', () => {
+      const claudeDir = join(testHomeDir, '.claude');
+      const settingsPath = join(claudeDir, 'settings.json');
+
+      mkdirSync(claudeDir, { recursive: true });
+
+      const settings: { permissions?: { allow?: string[] } } = {};
+      if (!settings.permissions) {
+        settings.permissions = {};
+      }
+      if (!Array.isArray(settings.permissions.allow)) {
+        settings.permissions.allow = [];
+      }
+
+      const permissions = ['Read(~/.droid/**)', 'Write(~/.droid/**)'];
+      for (const perm of permissions) {
+        if (!settings.permissions.allow.includes(perm)) {
+          settings.permissions.allow.push(perm);
+        }
+      }
+
+      writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n', 'utf-8');
+
+      const read = JSON.parse(readFileSync(settingsPath, 'utf-8'));
+      expect(read.permissions.allow).toContain('Read(~/.droid/**)');
+      expect(read.permissions.allow).toContain('Write(~/.droid/**)');
+    });
+
+    it('should preserve existing settings when adding permissions', () => {
+      const claudeDir = join(testHomeDir, '.claude');
+      const settingsPath = join(claudeDir, 'settings.json');
+
+      mkdirSync(claudeDir, { recursive: true });
+
+      const existingSettings = {
+        permissions: {
+          allow: ['Bash(git:*)'],
+          deny: ['Bash(rm -rf /*)'],
+        },
+        other_setting: true,
+      };
+      writeFileSync(settingsPath, JSON.stringify(existingSettings, null, 2), 'utf-8');
+
+      const settings = JSON.parse(readFileSync(settingsPath, 'utf-8'));
+      settings.permissions.allow.push('Read(~/.droid/**)');
+      writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n', 'utf-8');
+
+      const read = JSON.parse(readFileSync(settingsPath, 'utf-8'));
+      expect(read.permissions.allow).toContain('Bash(git:*)');
+      expect(read.permissions.allow).toContain('Read(~/.droid/**)');
+      expect(read.permissions.deny).toContain('Bash(rm -rf /*)');
+      expect(read.other_setting).toBe(true);
+    });
+
+    it('should handle corrupted JSON gracefully', () => {
+      const claudeDir = join(testHomeDir, '.claude');
+      const settingsPath = join(claudeDir, 'settings.json');
+
+      mkdirSync(claudeDir, { recursive: true });
+      writeFileSync(settingsPath, '{ not valid json {{{{', 'utf-8');
+
+      let settings: { permissions?: { allow?: string[] } } = {};
+      try {
+        settings = JSON.parse(readFileSync(settingsPath, 'utf-8'));
+      } catch {
+        // Invalid JSON, start fresh
+        settings = {};
+      }
+
+      if (!settings.permissions) {
+        settings.permissions = {};
+      }
+      if (!Array.isArray(settings.permissions.allow)) {
+        settings.permissions.allow = [];
+      }
+      settings.permissions.allow.push('Read(~/.droid/**)');
+
+      expect(settings.permissions.allow).toContain('Read(~/.droid/**)');
+    });
+  });
+
+  describe('Return value validation', () => {
+    it('should return added items when new permissions are added', () => {
+      const added: string[] = [];
+      const permissions = ['Read(~/.droid/**)', 'Write(~/.droid/**)'];
+      const existingAllow: string[] = [];
+
+      for (const perm of permissions) {
+        if (!existingAllow.includes(perm)) {
+          existingAllow.push(perm);
+          added.push(perm);
+        }
+      }
+
+      expect(added).toHaveLength(2);
+      expect(added).toContain('Read(~/.droid/**)');
+      expect(added).toContain('Write(~/.droid/**)');
+    });
+
+    it('should return alreadyPresent=true when no new permissions needed', () => {
+      const added: string[] = [];
+      const permissions = ['Read(~/.droid/**)'];
+      const existingAllow = ['Read(~/.droid/**)', 'Write(~/.droid/**)'];
+
+      for (const perm of permissions) {
+        if (!existingAllow.includes(perm)) {
+          existingAllow.push(perm);
+          added.push(perm);
+        }
+      }
+
+      const alreadyPresent = added.length === 0;
+      expect(alreadyPresent).toBe(true);
+    });
+
+    it('should return error when write fails', () => {
+      // Simulate what the function returns on write error
+      const result = { added: [], alreadyPresent: false, error: 'Failed to update OpenCode config: EACCES' };
+
+      expect(result.error).toBeDefined();
+      expect(result.error).toContain('Failed to update');
+      expect(result.added).toHaveLength(0);
+    });
+  });
+});

package/src/commands/setup.ts

@@ -51,10 +51,17 @@ function detectGitUsername(): string {
   }
 }
 
+/**
+ * The opencode-skills plugin name for OpenCode
+ * This plugin enables Claude Code-style skills in OpenCode
+ * @see https://github.com/malhashemi/opencode-skills
+ */
+const OPENCODE_SKILLS_PLUGIN = 'opencode-skills';
+
 /**
  * Configure AI tool permissions for droid
  */
-export function configureAIToolPermissions(aiTool: AITool): { added: string[]; alreadyPresent: boolean } {
+export function configureAIToolPermissions(aiTool: AITool): { added: string[]; alreadyPresent: boolean; error?: string } {
   const added: string[] = [];
 
   if (aiTool === AITool.ClaudeCode) {
@@ -72,7 +79,7 @@ export function configureAIToolPermissions(aiTool: AITool): { added: string[]; a
       try {
         settings = JSON.parse(readFileSync(settingsPath, 'utf-8'));
       } catch {
-        // Invalid JSON, start fresh
+        console.warn(chalk.yellow('⚠ Claude Code settings.json appears corrupted, resetting permissions'));
         settings = {};
       }
     }
@@ -95,13 +102,63 @@ export function configureAIToolPermissions(aiTool: AITool): { added: string[]; a
 
     // Save if we added anything
     if (added.length > 0) {
-      writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n', 'utf-8');
+      try {
+        writeFileSync(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}` };
+      }
+    }
+
+    return { added, alreadyPresent: added.length === 0 };
+  }
+
+  if (aiTool === AITool.OpenCode) {
+    // OpenCode uses opencode.json for config
+    // Check global config location: ~/.config/opencode/opencode.json
+    const globalConfigDir = join(homedir(), '.config', 'opencode');
+    const globalConfigPath = join(globalConfigDir, 'opencode.json');
+
+    // Ensure config directory exists
+    if (!existsSync(globalConfigDir)) {
+      mkdirSync(globalConfigDir, { recursive: true });
+    }
+
+    // Load or create config
+    let config: { plugin?: string[] } = {};
+    if (existsSync(globalConfigPath)) {
+      try {
+        config = JSON.parse(readFileSync(globalConfigPath, 'utf-8'));
+      } catch {
+        console.warn(chalk.yellow('⚠ OpenCode config appears corrupted, resetting'));
+        config = {};
+      }
+    }
+
+    // Ensure plugin array exists
+    if (!Array.isArray(config.plugin)) {
+      config.plugin = [];
+    }
+
+    // Add opencode-skills plugin if not present
+    if (!config.plugin.includes(OPENCODE_SKILLS_PLUGIN)) {
+      config.plugin.push(OPENCODE_SKILLS_PLUGIN);
+      added.push(OPENCODE_SKILLS_PLUGIN);
+    }
+
+    // Save if we added anything
+    if (added.length > 0) {
+      try {
+        writeFileSync(globalConfigPath, JSON.stringify(config, null, 2) + '\n', 'utf-8');
+      } catch (e) {
+        const message = e instanceof Error ? e.message : 'Unknown error';
+        return { added: [], alreadyPresent: false, error: `Failed to update OpenCode config: ${message}` };
+      }
     }
 
     return { added, alreadyPresent: added.length === 0 };
   }
 
-  // OpenCode - TODO: implement when we know the settings path
   return { added: [], alreadyPresent: true };
 }
 
@@ -214,12 +271,25 @@ export async function setupCommand(): Promise<void> {
 
   console.log(chalk.green('\n✓ Config saved to ~/.droid/config.yaml'));
 
-  // Configure AI tool permissions
-  const { added, alreadyPresent } = configureAIToolPermissions(answers.ai_tool);
-  if (added.length > 0) {
-    console.log(chalk.green(`✓ Added droid permissions to ${answers.ai_tool} settings`));
-  } else if (alreadyPresent) {
-    console.log(chalk.gray(`  Droid permissions already configured in ${answers.ai_tool}`));
+  // Configure AI tool permissions/plugins
+  const { added, alreadyPresent, error } = configureAIToolPermissions(answers.ai_tool);
+  if (error) {
+    console.log(chalk.red(`✗ ${error}`));
+    console.log(chalk.yellow('  You may need to manually configure your AI tool'));
+  } else if (answers.ai_tool === AITool.ClaudeCode) {
+    if (added.length > 0) {
+      console.log(chalk.green(`✓ Added droid permissions to Claude Code settings`));
+    } else if (alreadyPresent) {
+      console.log(chalk.gray(`  Droid permissions already configured in Claude Code`));
+    }
+  } else if (answers.ai_tool === AITool.OpenCode) {
+    if (added.length > 0) {
+      console.log(chalk.green(`✓ Added opencode-skills plugin to OpenCode config`));
+      console.log(chalk.gray(`  This enables Claude Code-style skills in OpenCode`));
+      console.log(chalk.gray(`  Restart OpenCode to activate the plugin`));
+    } else if (alreadyPresent) {
+      console.log(chalk.gray(`  opencode-skills plugin already configured in OpenCode`));
+    }
   }
 
   console.log(chalk.gray('\nRun `droid skills` to browse and install skills.'));