oh-my-claudecode-opencode 0.6.5 → 0.6.7

package/dist/agents/index.d.ts

@@ -94,6 +94,38 @@ export declare const qaTesterAgent: AgentDefinition;
  * QA Tester High (Opus) - Comprehensive Production QA Specialist
  */
 export declare const qaTesterHighAgent: AgentDefinition;
+/**
+ * Build Fixer (Sonnet) - Build and TypeScript error resolution specialist
+ */
+export declare const buildFixerAgent: AgentDefinition;
+/**
+ * Build Fixer Low (Haiku) - Fast build error fixes
+ */
+export declare const buildFixerLowAgent: AgentDefinition;
+/**
+ * Code Reviewer (Opus) - Expert code review specialist
+ */
+export declare const codeReviewerAgent: AgentDefinition;
+/**
+ * Code Reviewer Low (Haiku) - Quick code checks
+ */
+export declare const codeReviewerLowAgent: AgentDefinition;
+/**
+ * TDD Guide (Sonnet) - Test-Driven Development specialist
+ */
+export declare const tddGuideAgent: AgentDefinition;
+/**
+ * TDD Guide Low (Haiku) - Quick test suggestions
+ */
+export declare const tddGuideLowAgent: AgentDefinition;
+/**
+ * Security Reviewer (Opus) - Security vulnerability detection specialist
+ */
+export declare const securityReviewerAgent: AgentDefinition;
+/**
+ * Security Reviewer Low (Haiku) - Quick security scans
+ */
+export declare const securityReviewerLowAgent: AgentDefinition;
 /**
  * Planner (Opus) - Strategic Planning Specialist
  */

package/dist/index.js

@@ -21262,6 +21262,171 @@ tmux kill-session -t <name>
 5. **PRODUCTION-READY verdict** - Only give if ALL categories pass
 </Critical_Rules>`
 };
+var buildFixerSystemPrompt = `You are an expert build error resolution specialist focused on fixing TypeScript, compilation, and build errors quickly and efficiently.
+
+## Core Responsibilities
+- TypeScript Error Resolution - Fix type errors, inference issues, generic constraints
+- Build Error Fixing - Resolve compilation failures, module resolution
+- Dependency Issues - Fix import errors, missing packages, version conflicts
+- Configuration Errors - Resolve tsconfig.json, webpack, build config issues
+- Minimal Diffs - Make smallest possible changes to fix errors
+- No Architecture Changes - Only fix errors, don't refactor or redesign
+
+## Error Resolution Workflow
+1. Run full type check: npx tsc --noEmit --pretty
+2. Capture ALL errors, categorize by type
+3. For each error: Read error, find minimal fix, verify, run tsc again
+4. Track progress (X/Y errors fixed)
+
+## Minimal Diff Strategy
+DO: Add type annotations, null checks, fix imports, add missing dependencies
+DON'T: Refactor unrelated code, change architecture, rename variables, add features
+
+**Remember**: Fix errors quickly with minimal changes. Don't refactor, don't optimize, don't redesign.`;
+var buildFixerAgent = {
+  name: "build-fixer",
+  description: "Build and TypeScript error resolution specialist. Fixes build/type errors with minimal diffs, no architectural edits.",
+  model: "sonnet",
+  systemPrompt: buildFixerSystemPrompt
+};
+var buildFixerLowAgent = {
+  name: "build-fixer-low",
+  description: "Fast build error fixer for simple TypeScript and compilation errors",
+  model: "haiku",
+  systemPrompt: buildFixerSystemPrompt
+};
+var codeReviewerSystemPrompt = `You are a senior code reviewer ensuring high standards of code quality and security.
+
+## Two-Stage Review Process (MANDATORY)
+**Iron Law: Spec compliance BEFORE code quality. Both are LOOPS.**
+
+### Stage 1: Spec Compliance (FIRST - MUST PASS)
+| Check | Question |
+|-------|----------|
+| Completeness | Does implementation cover ALL requirements? |
+| Correctness | Does it solve the RIGHT problem? |
+| Nothing Missing | Are all requested features present? |
+| Nothing Extra | Is there unrequested functionality? |
+
+**Stage 1 Outcome:**
+- PASS \u2192 Proceed to Stage 2
+- FAIL \u2192 Document gaps \u2192 FIX \u2192 RE-REVIEW Stage 1 (loop)
+
+### Stage 2: Code Quality (ONLY after Stage 1 passes)
+- Security Checks (CRITICAL): Hardcoded credentials, SQL injection, XSS, input validation
+- Code Quality (HIGH): Large functions, deep nesting, missing error handling
+- Performance (MEDIUM): Inefficient algorithms, N+1 queries
+- Best Practices (LOW): TODO comments, missing JSDoc, magic numbers
+
+## Severity Levels
+| Severity | Action |
+|----------|--------|
+| CRITICAL | Must fix before merge |
+| HIGH | Should fix before merge |
+| MEDIUM | Fix when possible |
+| LOW | Consider fixing |
+
+## Approval Criteria
+- APPROVE: No CRITICAL or HIGH issues
+- REQUEST CHANGES: CRITICAL or HIGH issues found`;
+var codeReviewerAgent = {
+  name: "code-reviewer",
+  description: "Expert code review specialist. Reviews code for quality, security, and maintainability with severity-rated feedback.",
+  model: "opus",
+  readOnly: true,
+  systemPrompt: codeReviewerSystemPrompt
+};
+var codeReviewerLowAgent = {
+  name: "code-reviewer-low",
+  description: "Quick code reviewer for simple quality checks and obvious issues",
+  model: "haiku",
+  readOnly: true,
+  systemPrompt: codeReviewerSystemPrompt
+};
+var tddGuideSystemPrompt = `You are a Test-Driven Development (TDD) specialist who ensures all code is developed test-first with comprehensive coverage.
+
+## The Iron Law
+**NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST**
+
+Write code before test? DELETE IT. Start over.
+
+| Violation | Consequence |
+|-----------|-------------|
+| Code written before test | Delete the code. Write test first. |
+| "I'll add tests after" | No. Stop. Write test now. |
+| "Just this once" | No exceptions. Ever. |
+
+## TDD Workflow
+1. Write Test First (RED) - Start with a failing test
+2. Run Test - Verify it FAILS
+3. Write Minimal Implementation (GREEN) - Make test pass
+4. Run Test - Verify it PASSES
+5. Refactor (IMPROVE) - Clean up code
+6. Verify Coverage - Ensure 80%+
+
+## Test Types You Must Write
+1. Unit Tests (Mandatory) - Test individual functions
+2. Integration Tests (Mandatory) - Test API endpoints and database
+3. E2E Tests (For Critical Flows) - Test complete user journeys
+
+## Edge Cases You MUST Test
+Null/Undefined, Empty arrays/strings, Invalid types, Boundaries, Errors, Race conditions, Large data, Special characters
+
+**Remember**: No code without tests. Tests are not optional.`;
+var tddGuideAgent = {
+  name: "tdd-guide",
+  description: "Test-Driven Development specialist enforcing write-tests-first methodology. Ensures 80%+ test coverage.",
+  model: "sonnet",
+  systemPrompt: tddGuideSystemPrompt
+};
+var tddGuideLowAgent = {
+  name: "tdd-guide-low",
+  description: "Quick TDD guide for simple test suggestions and coverage checks",
+  model: "haiku",
+  systemPrompt: tddGuideSystemPrompt
+};
+var securityReviewerSystemPrompt = `You are an expert security specialist focused on identifying and remediating vulnerabilities in web applications.
+
+## Core Responsibilities
+1. Vulnerability Detection - Identify OWASP Top 10 and common security issues
+2. Secrets Detection - Find hardcoded API keys, passwords, tokens
+3. Input Validation - Ensure all user inputs are properly sanitized
+4. Authentication/Authorization - Verify proper access controls
+5. Dependency Security - Check for vulnerable npm packages
+
+## OWASP Top 10 Checklist
+1. Injection (SQL, NoSQL, Command) - Are queries parameterized?
+2. Broken Authentication - Are passwords hashed? Is JWT validated?
+3. Sensitive Data Exposure - Is HTTPS enforced? Secrets in env vars?
+4. XML External Entities (XXE) - Is external entity processing disabled?
+5. Broken Access Control - Is authorization checked on every route?
+6. Security Misconfiguration - Are security headers set?
+7. Cross-Site Scripting (XSS) - Is output escaped/sanitized?
+8. Insecure Deserialization - Is user input deserialized safely?
+9. Using Components with Known Vulnerabilities - Is npm audit clean?
+10. Insufficient Logging & Monitoring - Are security events logged?
+
+## Critical Patterns to Detect
+- Hardcoded Secrets: const apiKey = "sk-proj-xxxxx" \u2192 Use env vars
+- SQL Injection: \`SELECT * FROM users WHERE id = \${userId}\` \u2192 Parameterize
+- Command Injection: exec(\`ping \${userInput}\`) \u2192 Use libraries
+- XSS: element.innerHTML = userInput \u2192 Use textContent
+
+**Remember**: Security is not optional. One vulnerability can cost users real financial losses.`;
+var securityReviewerAgent = {
+  name: "security-reviewer",
+  description: "Security vulnerability detection specialist. Detects OWASP Top 10, secrets, and unsafe patterns.",
+  model: "opus",
+  readOnly: true,
+  systemPrompt: securityReviewerSystemPrompt
+};
+var securityReviewerLowAgent = {
+  name: "security-reviewer-low",
+  description: "Quick security scanner for obvious vulnerabilities and secret detection",
+  model: "haiku",
+  readOnly: true,
+  systemPrompt: securityReviewerSystemPrompt
+};
 var plannerAgent = {
   name: "planner",
   description: "Strategic planning specialist for creating comprehensive implementation plans and roadmaps",
@@ -21577,6 +21742,14 @@ var HARDCODED_AGENTS = {
   writer: writerAgent,
   "qa-tester": qaTesterAgent,
   "qa-tester-high": qaTesterHighAgent,
+  "build-fixer": buildFixerAgent,
+  "build-fixer-low": buildFixerLowAgent,
+  "code-reviewer": codeReviewerAgent,
+  "code-reviewer-low": codeReviewerLowAgent,
+  "tdd-guide": tddGuideAgent,
+  "tdd-guide-low": tddGuideLowAgent,
+  "security-reviewer": securityReviewerAgent,
+  "security-reviewer-low": securityReviewerLowAgent,
   planner: plannerAgent,
   analyst: analystAgent,
   critic: criticAgent,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-claudecode-opencode",
-  "version": "0.6.5",
+  "version": "0.6.7",
   "description": "OpenCode port of oh-my-claudecode - Multi-agent orchestration plugin (omco)",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -12,7 +12,8 @@
   "files": [
     "dist",
     "assets",
-    "bin"
+    "bin",
+    "scripts"
   ],
   "exports": {
     ".": {
@@ -26,6 +27,7 @@
     "build:watch": "bun build src/index.ts --outdir dist --target bun --format esm --watch",
     "clean": "rm -rf dist",
     "prepublishOnly": "bun run clean && bun run build",
+    "postinstall": "node scripts/postinstall.js || true",
     "typecheck": "tsc --noEmit",
     "test": "bun test",
     "test:e2e": "vitest run --config tests/e2e/vitest.config.ts",

package/scripts/install-local.sh

@@ -0,0 +1,25 @@
+#!/bin/bash
+# Install local development build to OpenCode plugin directory
+
+set -e
+
+PLUGIN_DIR="$HOME/.config/opencode/node_modules/oh-my-claudecode-opencode"
+
+echo "Building..."
+bun run build
+
+echo "Copying to $PLUGIN_DIR..."
+cp -r dist/ "$PLUGIN_DIR/dist/"
+cp -r assets/ "$PLUGIN_DIR/assets/"
+cp package.json "$PLUGIN_DIR/package.json"
+
+echo "Verifying..."
+SKILL_COUNT=$(ls "$PLUGIN_DIR/assets/skills/" | wc -l)
+VERSION=$(grep '"version"' "$PLUGIN_DIR/package.json" | head -1 | sed 's/.*: "\(.*\)".*/\1/')
+
+echo ""
+echo "=== Installation Complete ==="
+echo "Version: $VERSION"
+echo "Skills: $SKILL_COUNT"
+echo ""
+echo "Restart OpenCode to apply changes."

package/scripts/postinstall.js

@@ -0,0 +1,112 @@
+#!/usr/bin/env node
+
+/**
+ * OMCO Post-Install Script
+ * 
+ * Adds github-copilot provider to global opencode.json if not present.
+ * This ensures OMCO's tier-mapped agents (planner, critic, architect) work out of the box.
+ */
+
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
+import { join } from 'path';
+import { homedir } from 'os';
+
+const GITHUB_COPILOT_PROVIDER = {
+  "name": "GitHub Copilot",
+  "models": {
+    "claude-opus-4.5": {
+      "name": "Claude Opus 4.5",
+      "attachment": true,
+      "limit": {
+        "context": 200000,
+        "output": 32000
+      }
+    },
+    "claude-sonnet-4": {
+      "name": "Claude Sonnet 4",
+      "attachment": true,
+      "limit": {
+        "context": 200000,
+        "output": 16000
+      }
+    },
+    "claude-haiku-4": {
+      "name": "Claude Haiku 4",
+      "attachment": true,
+      "limit": {
+        "context": 200000,
+        "output": 8000
+      }
+    }
+  }
+};
+
+function getOpencodeConfigPath() {
+  return join(homedir(), '.config', 'opencode', 'opencode.json');
+}
+
+function ensureConfigDir() {
+  const configDir = join(homedir(), '.config', 'opencode');
+  if (!existsSync(configDir)) {
+    mkdirSync(configDir, { recursive: true });
+    console.log('[omco] Created config directory:', configDir);
+  }
+}
+
+function main() {
+  const configPath = getOpencodeConfigPath();
+  
+  try {
+    ensureConfigDir();
+    
+    let config = {};
+    let configExists = false;
+    
+    if (existsSync(configPath)) {
+      configExists = true;
+      const content = readFileSync(configPath, 'utf-8');
+      config = JSON.parse(content);
+    }
+    
+    // Initialize provider section if not present
+    if (!config.provider) {
+      config.provider = {};
+    }
+    
+    // Check if github-copilot already exists
+    if (config.provider['github-copilot']) {
+      console.log('[omco] github-copilot provider already configured in opencode.json');
+      return;
+    }
+    
+    // Add github-copilot provider
+    config.provider['github-copilot'] = GITHUB_COPILOT_PROVIDER;
+    
+    // Add plugin if not present
+    if (!config.plugin) {
+      config.plugin = [];
+    }
+    if (!config.plugin.includes('oh-my-claudecode-opencode')) {
+      config.plugin.push('oh-my-claudecode-opencode');
+    }
+    
+    // Write back
+    writeFileSync(configPath, JSON.stringify(config, null, 2));
+    
+    if (configExists) {
+      console.log('[omco] ✅ Added github-copilot provider to opencode.json');
+    } else {
+      console.log('[omco] ✅ Created opencode.json with github-copilot provider');
+    }
+    
+    console.log('[omco] Config path:', configPath);
+    console.log('[omco] Available models: claude-opus-4.5, claude-sonnet-4, claude-haiku-4');
+    
+  } catch (error) {
+    // Don't fail install on config errors - just warn
+    console.warn('[omco] ⚠️  Could not update opencode.json:', error.message);
+    console.warn('[omco] You may need to manually add github-copilot provider for tier mapping to work.');
+  }
+}
+
+main();