oh-my-claude-sisyphus 3.3.10 → 3.4.1

package/skills/code-review/SKILL.md

@@ -0,0 +1,179 @@
+---
+name: code-review
+description: Run a comprehensive code review
+---
+
+# Code Review Skill
+
+Conduct a thorough code review for quality, security, and maintainability with severity-rated feedback.
+
+## When to Use
+
+This skill activates when:
+- User requests "review this code", "code review"
+- Before merging a pull request
+- After implementing a major feature
+- User wants quality assessment
+
+## What It Does
+
+Delegates to the `code-reviewer` agent (Opus model) for deep analysis:
+
+1. **Identify Changes**
+   - Run `git diff` to find changed files
+   - Determine scope of review (specific files or entire PR)
+
+2. **Review Categories**
+   - **Security** - Hardcoded secrets, injection risks, XSS, CSRF
+   - **Code Quality** - Function size, complexity, nesting depth
+   - **Performance** - Algorithm efficiency, N+1 queries, caching
+   - **Best Practices** - Naming, documentation, error handling
+   - **Maintainability** - Duplication, coupling, testability
+
+3. **Severity Rating**
+   - **CRITICAL** - Security vulnerability (must fix before merge)
+   - **HIGH** - Bug or major code smell (should fix before merge)
+   - **MEDIUM** - Minor issue (fix when possible)
+   - **LOW** - Style/suggestion (consider fixing)
+
+4. **Specific Recommendations**
+   - File:line locations for each issue
+   - Concrete fix suggestions
+   - Code examples where applicable
+
+## Agent Delegation
+
+```
+Task(
+  subagent_type="oh-my-claudecode:code-reviewer",
+  model="opus",
+  prompt="CODE REVIEW TASK
+
+Review code changes for quality, security, and maintainability.
+
+Scope: [git diff or specific files]
+
+Review Checklist:
+- Security vulnerabilities (OWASP Top 10)
+- Code quality (complexity, duplication)
+- Performance issues (N+1, inefficient algorithms)
+- Best practices (naming, documentation, error handling)
+- Maintainability (coupling, testability)
+
+Output: Code review report with:
+- Files reviewed count
+- Issues by severity (CRITICAL, HIGH, MEDIUM, LOW)
+- Specific file:line locations
+- Fix recommendations
+- Approval recommendation (APPROVE / REQUEST CHANGES / COMMENT)"
+)
+```
+
+## Output Format
+
+```
+CODE REVIEW REPORT
+==================
+
+Files Reviewed: 8
+Total Issues: 15
+
+CRITICAL (0)
+-----------
+(none)
+
+HIGH (3)
+--------
+1. src/api/auth.ts:42
+   Issue: User input not sanitized before SQL query
+   Risk: SQL injection vulnerability
+   Fix: Use parameterized queries or ORM
+
+2. src/components/UserProfile.tsx:89
+   Issue: Password displayed in plain text in logs
+   Risk: Credential exposure
+   Fix: Remove password from log statements
+
+3. src/utils/validation.ts:15
+   Issue: Email regex allows invalid formats
+   Risk: Accepts malformed emails
+   Fix: Use proven email validation library
+
+MEDIUM (7)
+----------
+...
+
+LOW (5)
+-------
+...
+
+RECOMMENDATION: REQUEST CHANGES
+
+Critical security issues must be addressed before merge.
+```
+
+## Review Checklist
+
+The code-reviewer agent checks:
+
+### Security
+- [ ] No hardcoded secrets (API keys, passwords, tokens)
+- [ ] All user inputs sanitized
+- [ ] SQL/NoSQL injection prevention
+- [ ] XSS prevention (escaped outputs)
+- [ ] CSRF protection on state-changing operations
+- [ ] Authentication/authorization properly enforced
+
+### Code Quality
+- [ ] Functions < 50 lines (guideline)
+- [ ] Cyclomatic complexity < 10
+- [ ] No deeply nested code (> 4 levels)
+- [ ] No duplicate logic (DRY principle)
+- [ ] Clear, descriptive naming
+
+### Performance
+- [ ] No N+1 query patterns
+- [ ] Appropriate caching where applicable
+- [ ] Efficient algorithms (avoid O(n²) when O(n) possible)
+- [ ] No unnecessary re-renders (React/Vue)
+
+### Best Practices
+- [ ] Error handling present and appropriate
+- [ ] Logging at appropriate levels
+- [ ] Documentation for public APIs
+- [ ] Tests for critical paths
+- [ ] No commented-out code
+
+## Approval Criteria
+
+**APPROVE** - No CRITICAL or HIGH issues, minor improvements only
+**REQUEST CHANGES** - CRITICAL or HIGH issues present
+**COMMENT** - Only LOW/MEDIUM issues, no blocking concerns
+
+## Use with Other Skills
+
+**With Pipeline:**
+```
+/pipeline review "implement user authentication"
+```
+Includes code review as part of implementation workflow.
+
+**With Ralph:**
+```
+/ralph code-review then fix all issues
+```
+Review code, get feedback, fix until approved.
+
+**With Ultrawork:**
+```
+/ultrawork review all files in src/
+```
+Parallel code review across multiple files.
+
+## Best Practices
+
+- **Review early** - Catch issues before they compound
+- **Review often** - Small, frequent reviews better than huge ones
+- **Address CRITICAL/HIGH first** - Fix security and bugs immediately
+- **Consider context** - Some "issues" may be intentional trade-offs
+- **Learn from reviews** - Use feedback to improve coding practices

package/skills/deepinit/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: deepinit
 description: Deep codebase initialization with hierarchical AGENTS.md documentation
-user-invocable: true
 ---
 
 # Deep Init Skill

package/skills/deepsearch/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: deepsearch
 description: Thorough codebase search
-user-invocable: true
 ---
 
 # Deep Search Mode

package/skills/doctor/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: doctor
 description: Diagnose and fix oh-my-claudecode installation issues
-user-invocable: true
 ---
 
 # Doctor Skill

package/skills/ecomode/SKILL.md

@@ -0,0 +1,128 @@
+---
+name: ecomode
+description: Token-efficient parallel execution mode using Haiku and Sonnet agents
+---
+
+# Ecomode Skill
+
+Activates token-efficient parallel execution for pro-plan users who prioritize cost efficiency.
+
+## When Activated
+
+This skill enhances Claude's capabilities by:
+
+1. **Parallel Execution**: Running multiple agents simultaneously for independent tasks
+2. **Token-Conscious Routing**: Preferring Haiku and Sonnet agents, avoiding Opus
+3. **Background Operations**: Using `run_in_background: true` for long operations
+4. **Persistence Enforcement**: Never stopping until all tasks are verified complete
+5. **Cost Optimization**: Minimizing token usage while maintaining quality
+
+## Ecomode Routing Rules (CRITICAL)
+
+**ALWAYS prefer lower tiers. Only escalate when task genuinely requires it.**
+
+| Decision | Rule |
+|----------|------|
+| DEFAULT | Use LOW tier (Haiku) for all tasks |
+| UPGRADE | Use MEDIUM (Sonnet) only when task complexity warrants |
+| AVOID | HIGH tier (Opus) - only use for planning/critique if explicitly essential |
+
+## Smart Model Routing (PREFER LOW TIER)
+
+**Choose tier based on task complexity: LOW (haiku) preferred → MEDIUM (sonnet) fallback → HIGH (opus) AVOID**
+
+### Agent Routing Table
+
+| Domain | PREFERRED (Haiku) | FALLBACK (Sonnet) | AVOID (Opus) |
+|--------|-------------------|-------------------|--------------|
+| **Analysis** | `architect-low` | `architect-medium` | ~~`architect`~~ |
+| **Execution** | `executor-low` | `executor` | ~~`executor-high`~~ |
+| **Search** | `explore` | `explore-medium` | ~~`explore-high`~~ |
+| **Research** | `researcher-low` | `researcher` | - |
+| **Frontend** | `designer-low` | `designer` | ~~`designer-high`~~ |
+| **Docs** | `writer` | - | - |
+| **Visual** | - | `vision` | - |
+| **Planning** | - | - | `planner` (if essential) |
+| **Critique** | - | - | `critic` (if essential) |
+| **Testing** | - | `qa-tester` | ~~`qa-tester-high`~~ |
+| **Security** | `security-reviewer-low` | - | ~~`security-reviewer`~~ |
+| **Build** | `build-fixer-low` | `build-fixer` | - |
+| **TDD** | `tdd-guide-low` | `tdd-guide` | - |
+| **Code Review** | `code-reviewer-low` | - | ~~`code-reviewer`~~ |
+| **Data Science** | `scientist-low` | `scientist` | ~~`scientist-high`~~ |
+
+### Tier Selection Guide (Token-Conscious)
+
+| Task Complexity | Tier | Examples |
+|-----------------|------|----------|
+| Simple lookups | LOW | "What does this function return?", "Find where X is defined" |
+| Standard work | LOW first, MEDIUM if fails | "Add error handling", "Implement this feature" |
+| Complex analysis | MEDIUM | "Debug this issue", "Refactor this module" |
+| Planning only | HIGH (if essential) | "Design architecture for new system" |
+
+### Routing Examples
+
+**CRITICAL: Always pass `model` parameter explicitly - Claude Code does NOT auto-apply models from agent definitions!**
+
+```
+// Simple question → LOW tier (DEFAULT)
+Task(subagent_type="oh-my-claudecode:architect-low", model="haiku", prompt="What does this function return?")
+
+// Standard implementation → TRY LOW first
+Task(subagent_type="oh-my-claudecode:executor-low", model="haiku", prompt="Add validation to login form")
+
+// If LOW fails, escalate to MEDIUM
+Task(subagent_type="oh-my-claudecode:executor", model="sonnet", prompt="Add error handling to login")
+
+// File lookup → ALWAYS LOW
+Task(subagent_type="oh-my-claudecode:explore", model="haiku", prompt="Find where UserService is defined")
+
+// Only use MEDIUM for complex patterns
+Task(subagent_type="oh-my-claudecode:explore-medium", model="sonnet", prompt="Find all authentication patterns in the codebase")
+```
+
+## DELEGATION ENFORCEMENT (CRITICAL)
+
+**YOU ARE AN ORCHESTRATOR, NOT AN IMPLEMENTER.**
+
+| Action | YOU Do | DELEGATE |
+|--------|--------|----------|
+| Read files for context | ✓ | |
+| Track progress (TODO) | ✓ | |
+| Spawn parallel agents | ✓ | |
+| **ANY code change** | ✗ NEVER | executor-low/executor |
+| **UI work** | ✗ NEVER | designer-low/designer |
+| **Docs** | ✗ NEVER | writer |
+
+**Path Exception**: Only write to `.omc/`, `.claude/`, `CLAUDE.md`, `AGENTS.md`
+
+## Background Execution Rules
+
+**Run in Background** (set `run_in_background: true`):
+- Package installation: npm install, pip install, cargo build
+- Build processes: npm run build, make, tsc
+- Test suites: npm test, pytest, cargo test
+- Docker operations: docker build, docker pull
+
+**Run Blocking** (foreground):
+- Quick status checks: git status, ls, pwd
+- File reads (NOT edits - delegate edits to executor-low)
+- Simple commands
+
+## Verification Checklist
+
+Before stopping, verify:
+- [ ] TODO LIST: Zero pending/in_progress tasks
+- [ ] FUNCTIONALITY: All requested features work
+- [ ] TESTS: All tests pass (if applicable)
+- [ ] ERRORS: Zero unaddressed errors
+
+**If ANY checkbox is unchecked, CONTINUE WORKING.**
+
+## Token Savings Tips
+
+1. **Batch similar tasks** to one agent instead of spawning many
+2. **Use explore (haiku)** for file discovery, not architect
+3. **Prefer executor-low** for simple changes - only upgrade if it fails
+4. **Avoid opus agents** unless the task genuinely requires deep reasoning
+5. **Use writer (haiku)** for all documentation tasks

package/skills/frontend-ui-ux/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: frontend-ui-ux
 description: Designer-turned-developer who crafts stunning UI/UX even without design mockups
-user-invocable: true
 ---
 
 # Frontend UI/UX Skill

package/skills/git-master/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: git-master
 description: Git expert for atomic commits, rebasing, and history management with style detection
-user-invocable: true
 ---
 
 # Git Master Skill

package/skills/help/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: help
 description: Guide on using oh-my-claudecode plugin
-user-invocable: true
 ---
 
 # How OMC Works
@@ -63,4 +62,4 @@ But now you don't NEED them - everything is automatic.
 
 ---
 
-*Version: 3.0.11*
+*Version: 3.4.0*

package/skills/hud/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: hud
 description: Configure HUD display options (layout, presets, display elements)
-user-invocable: true
 role: config-writer  # DOCUMENTATION ONLY - This skill writes to ~/.claude/ paths
 scope: ~/.claude/**  # DOCUMENTATION ONLY - Allowed write scope
 ---

package/skills/learner/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: learner
 description: Extract a learned skill from the current conversation
-user-invocable: true
 ---
 
 # Learner Skill

package/skills/mcp-setup/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: mcp-setup
 description: Configure popular MCP servers for enhanced agent capabilities
-user-invocable: true
 ---
 
 # MCP Setup
@@ -66,7 +65,7 @@ Read the current `~/.claude/settings.json` and add/update the `mcpServers` secti
   "mcpServers": {
     "context7": {
       "command": "npx",
-      "args": ["-y", "@context7/mcp"]
+      "args": ["-y", "@upstash/context7-mcp"]
     }
   }
 }
@@ -78,7 +77,7 @@ Read the current `~/.claude/settings.json` and add/update the `mcpServers` secti
   "mcpServers": {
     "exa": {
       "command": "npx",
-      "args": ["-y", "@anthropic/exa-mcp-server"],
+      "args": ["-y", "exa-mcp-server"],
       "env": {
         "EXA_API_KEY": "<user-provided-key>"
       }
@@ -93,7 +92,7 @@ Read the current `~/.claude/settings.json` and add/update the `mcpServers` secti
   "mcpServers": {
     "filesystem": {
       "command": "npx",
-      "args": ["-y", "@anthropic/mcp-server-filesystem", "<allowed-directories>"]
+      "args": ["-y", "@modelcontextprotocol/server-filesystem", "<allowed-directories>"]
     }
   }
 }
@@ -104,16 +103,18 @@ Read the current `~/.claude/settings.json` and add/update the `mcpServers` secti
 {
   "mcpServers": {
     "github": {
-      "command": "npx",
-      "args": ["-y", "@anthropic/github-mcp-server"],
+      "command": "docker",
+      "args": ["run", "-i", "--rm", "-e", "GITHUB_PERSONAL_ACCESS_TOKEN", "ghcr.io/github/github-mcp-server"],
       "env": {
-        "GITHUB_TOKEN": "<user-provided-token>"
+        "GITHUB_PERSONAL_ACCESS_TOKEN": "<user-provided-token>"
       }
     }
   }
 }
 ```
 
+> Note: GitHub MCP requires Docker. Alternatively, configure as a remote URL: https://api.githubcopilot.com/mcp/
+
 ## Step 4: Merge Configuration
 
 When updating settings.json:

package/skills/note/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: note
 description: Save notes to notepad.md for compaction resilience
-user-invocable: true
 ---
 
 # Note Skill

package/skills/omc-default/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: omc-default
 description: Configure OMC in local project (.claude/CLAUDE.md)
-user-invocable: true
 ---
 
 # OMC Default (Project-Scoped)

package/skills/omc-default-global/SKILL.md

@@ -1,7 +1,6 @@
 ---
 name: omc-default-global
 description: Configure OMC globally in ~/.claude/CLAUDE.md
-user-invocable: true
 ---
 
 # OMC Default Global