WP4 Phase 2: Plugin Handler Updates for Hierarchical Skills

[Steffen025]

Summary

WP4 Phase 2: Update plugin handlers to support WP3's hierarchical skill structure.

Changes

Updated: plugins/handlers/skill-guard.ts

Problem: Skill guard used flat paths (skills/SkillName/) which broke after WP3 reorganization.

Solution: Added findSkillDir() function that searches both structures:

1. Flat: skills/SkillName/ (backward compatibility)
2. Hierarchical: skills/Category/SkillName/ (WP3)

Implementation:

function findSkillDir(skillName: string): string | null {
  // Try flat structure first
  const flatPath = path.join(skillsDir, skillName);
  if (fs.existsSync(flatPath)) return flatPath;
  
  // Search all categories
  for (const category of categories) {
    const nestedPath = path.join(skillsDir, category, skillName);
    if (fs.existsSync(nestedPath)) return nestedPath;
  }
  return null;
}

Functions Updated:

• extractTriggers() - Now uses findSkillDir()
• validateSkillInvocation() - Now uses findSkillDir()

Benefits

✅ Backward Compatibility: Still finds skills in old flat structure
✅ WP3 Support: Now finds skills in new hierarchical structure
✅ Zero Breaking Changes: Existing skills continue to work
✅ Better Error Messages: Indicates both structures were checked

Testing

// Test cases covered:
- Flat skill: skills/Research/ → found ✓
- Hierarchical: skills/Security/Recon/ → found ✓
- Missing skill: returns null with descriptive error

Files Changed

• .opencode/plugins/handlers/skill-guard.ts (+40 lines, -5 lines)

Related

• Part of WP4 Implementation Plan
• Follows WP3 category reorganization
• Enables proper skill validation with new structure

Next Steps

Phase 3 will add skill discovery validation scripts.