oh-my-copilot 4.11.6 → 4.11.8

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
     {
       "name": "oh-my-copilot",
       "description": "Copilot CLI native multi-agent orchestration with intelligent model routing, 18 agent variants, and 32 powerful skills. Zero learning curve. Maximum power.",
-      "version": "4.11.6",
+      "version": "4.11.8",
       "author": {
         "name": "Robin Norberg",
         "email": "github@platypus.se"
@@ -27,5 +27,5 @@
       ]
     }
   ],
-  "version": "4.11.6"
+  "version": "4.11.8"
 }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-copilot",
-  "version": "4.11.6",
+  "version": "4.11.8",
   "description": "Multi-agent orchestration system for Copilot CLI",
   "author": {
     "name": "Robin Norberg and oh-my-copilot contributors"

package/README.md

@@ -32,19 +32,19 @@
 ```bash
 # Step 1: Install
 /plugin marketplace add https://github.com/RobinNorberg/oh-my-copilot
-/plugin install oh-my-copilot@omg
+/plugin install oh-my-copilot@omcp
 # or
 npm i -g oh-my-copilot@latest
 
-
 # Step 2: Setup
 /omc-setup
 
-
 # Step 3: Build something
 /autopilot: build a todo-app
 # or
 autopilot: build a todo-app
+
+# If you enjoy the output, give the repo att ⭐ and tell a friend 
 ```
 
 ### Not Sure Where to Start?
@@ -86,32 +86,46 @@ Optional shortcuts for power users. Natural language works fine without them.
 | `c3g` | ![orchestration](https://img.shields.io/badge/orchestration-blue) | Quadri-model orchestration | `c3g review this PR` |
 | `omcp team` | ![orchestration](https://img.shields.io/badge/orchestration-blue) | tmux CLI workers (codex/gemini/copilot) | `omcp team 2:codex "security review"` |
 | `code review` | ![analysis](https://img.shields.io/badge/analysis-purple) | Code review mode | `code review the auth module` |
+| `critique` | ![analysis](https://img.shields.io/badge/analysis-purple) | Pre-push adversarial critique | `critique my changes` |
+| `debug`, `diagnose` | ![analysis](https://img.shields.io/badge/analysis-purple) | Session/repo diagnostics | `debug why hooks aren't firing` |
 | `deep-analyze` | ![analysis](https://img.shields.io/badge/analysis-purple) | Deep analysis mode | `deep-analyze why tests are failing` |
 | `deep-dive` | ![analysis](https://img.shields.io/badge/analysis-purple) | Trace → interview pipeline | `deep-dive why auth is slow` |
+| `deepinit` | ![analysis](https://img.shields.io/badge/analysis-purple) | Deep codebase init with AGENTS.md | `deepinit` |
 | `deep-interview` | ![analysis](https://img.shields.io/badge/analysis-purple) | Socratic requirements clarification | `deep-interview "vague idea"` |
 | `deep-review` | ![analysis](https://img.shields.io/badge/analysis-purple) | Multi-pass code review (4 passes) | `deep-review this PR` |
 | `deepsearch` | ![analysis](https://img.shields.io/badge/analysis-purple) | Codebase-focused search routing | `deepsearch for auth middleware` |
+| `deslop`, `anti-slop` | ![analysis](https://img.shields.io/badge/analysis-purple) | AI code slop cleanup | `deslop the auth module` |
 | `discover` | ![analysis](https://img.shields.io/badge/analysis-purple) | Parallel codebase quality scan | `discover src/hooks/` |
+| `external-context` | ![analysis](https://img.shields.io/badge/analysis-purple) | Parallel external doc/web search | `external-context React Server Components` |
+| `sciomc` | ![analysis](https://img.shields.io/badge/analysis-purple) | Parallel scientist orchestration | `sciomc analyze test failures` |
 | `security review` | ![analysis](https://img.shields.io/badge/analysis-purple) | Security review mode | `security review the API endpoints` |
 | `tdd`, `test first` | ![analysis](https://img.shields.io/badge/analysis-purple) | TDD workflow enforcement | `tdd: implement password validation` |
+| `trace` | ![analysis](https://img.shields.io/badge/analysis-purple) | Evidence-driven causal tracing | `trace why auth is broken` |
+| `verify` | ![analysis](https://img.shields.io/badge/analysis-purple) | Verify changes work before claiming done | `verify the fix` |
 | `ultrathink` | ![analysis](https://img.shields.io/badge/analysis-purple) | Deep reasoning mode | `ultrathink about this architecture` |
 | `ralplan` | ![planning](https://img.shields.io/badge/planning-orange) | Iterative planning consensus | `ralplan this feature` |
 | `autopilot` | ![execution](https://img.shields.io/badge/execution-green) | Full autonomous execution | `autopilot: build a todo app` |
+| `cancelomc`, `stopomc` | ![execution](https://img.shields.io/badge/execution-green) | Stop active OMC modes | `stopomc` |
 | `experiment` | ![execution](https://img.shields.io/badge/execution-green) | Hypothesis-driven experiment loop | `experiment: optimize API latency` |
 | `ralph` | ![execution](https://img.shields.io/badge/execution-green) | Persistence mode | `ralph: refactor auth` |
 | `ralphthon` | ![execution](https://img.shields.io/badge/execution-green) | Autonomous hackathon mode | `ralphthon: build MVP in 2 hours` |
+| `self-improve` | ![execution](https://img.shields.io/badge/execution-green) | Autonomous evolutionary code improvement | `self-improve the parser module` |
+| `skillify` | ![execution](https://img.shields.io/badge/execution-green) | Extract reusable skill from session | `skillify this workflow` |
 | `ulw` | ![execution](https://img.shields.io/badge/execution-green) | Maximum parallelism | `ulw fix all errors` |
 | `gh setup` | ![github](https://img.shields.io/badge/github-blue) | Configure GitHub integration | `gh setup` |
 | `gh triage` | ![github](https://img.shields.io/badge/github-blue) | GitHub issue/PR/CI triage | `gh triage` |
 | `gh review` | ![github](https://img.shields.io/badge/github-blue) | Interactive GitHub PR review | `gh review` |
 | `gh auto-review` | ![github](https://img.shields.io/badge/github-blue) | Automated code review via code-reviewer agent | `gh auto-review` |
 | `gh project` | ![github](https://img.shields.io/badge/github-blue) | Manage GitHub Projects (v2) boards | `gh project` |
+| `ado sprint` | ![devops](https://img.shields.io/badge/devops-gray) | Sprint planning and iteration management | `ado sprint` |
 | `ado setup` | ![devops](https://img.shields.io/badge/devops-gray) | Configure Azure DevOps integration | `ado setup` |
 | `ado triage` | ![devops](https://img.shields.io/badge/devops-gray) | Azure DevOps work item triage | `ado triage` |
 | `ado review` | ![devops](https://img.shields.io/badge/devops-gray) | Interactive Azure DevOps PR review | `ado review` |
 | `ado auto-review` | ![devops](https://img.shields.io/badge/devops-gray) | Automated code review via code-reviewer agent | `ado auto-review` |
-| `ado sprint` | ![devops](https://img.shields.io/badge/devops-gray) | Sprint planning and iteration management | `ado sprint` |
-| `cancelomc`, `stopomc` | ![control](https://img.shields.io/badge/control-gray) | Stop active OMC modes | `stopomc` |
+| `hud` | ![utility](https://img.shields.io/badge/utility-lightgray) | Configure status line display | `hud preset minimal` |
+| `learner` | ![utility](https://img.shields.io/badge/utility-lightgray) | Learn a skill from the current conversation | `learner` |
+| `remember` | ![utility](https://img.shields.io/badge/utility-lightgray) | Save reusable project knowledge | `remember this pattern` |
+| `wiki` | ![utility](https://img.shields.io/badge/utility-lightgray) | Persistent markdown knowledge base | `wiki add auth architecture notes` |
 
 **Notes:**
 

package/agents/analyst.agent.md

@@ -1,7 +1,7 @@
 ---
 name: analyst
 description: Pre-planning consultant for requirements analysis (Opus)
-model: claude-opus-4-6
+model: opus
 disallowedTools: Write, Edit
 ---
 

package/agents/architect.agent.md

@@ -1,7 +1,7 @@
 ---
 name: architect
 description: Strategic Architecture & Debugging Advisor (Opus, READ-ONLY)
-model: claude-opus-4-6
+model: opus
 disallowedTools: Write, Edit
 ---
 

package/agents/code-reviewer.agent.md

@@ -1,7 +1,7 @@
 ---
 name: code-reviewer
 description: Expert code review specialist with severity-rated feedback, logic defect detection, SOLID principle checks, style, performance, and quality strategy
-model: claude-opus-4-6
+model: opus
 disallowedTools: Write, Edit
 ---
 

package/agents/code-simplifier.agent.md

@@ -1,7 +1,7 @@
 ---
 name: code-simplifier
 description: Simplifies and refines code for clarity, consistency, and maintainability while preserving all functionality. Focuses on recently modified code unless instructed otherwise.
-model: claude-opus-4-6
+model: opus
 ---
 
 <Agent_Prompt>

package/agents/critic.agent.md

@@ -1,7 +1,7 @@
 ---
 name: critic
 description: Work plan and code review expert — thorough, structured, multi-perspective (Opus)
-model: claude-opus-4-6
+model: opus
 disallowedTools: Write, Edit
 ---
 

package/agents/debugger.agent.md

@@ -1,7 +1,7 @@
 ---
 name: debugger
 description: Root-cause analysis, regression isolation, stack trace analysis, build/compilation error resolution
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/designer.agent.md

@@ -1,7 +1,7 @@
 ---
 name: designer
 description: UI/UX Designer-Developer for stunning interfaces (Sonnet)
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/devils-advocate.agent.md

@@ -0,0 +1,219 @@
+---
+name: devils-advocate
+description: Independent pre-push critique — finds flaws in unpushed commits with adversarial skepticism
+model: opus
+disallowedTools: Write, Edit
+---
+
+<Agent_Prompt>
+  <Role>
+    You are Devil's Advocate — an adversarial pre-push reviewer. You exist because authors are blind to their own mistakes.
+
+    You are reviewing code that was written in the same agent session or by the same developer. Your job is to find flaws the author missed. Approach with skepticism — the author fills in gaps mentally and rationalizes decisions. You only see the artifact and the codebase.
+
+    Do not defer to the author's intent. If code is wrong, it is wrong regardless of what the author meant. You are the last line of defense before code reaches the remote.
+
+    You are responsible for finding what is wrong, suboptimal, insufficient, faulty, or broken in unpushed commits, and proposing concrete fixes for every issue found.
+    You are not responsible for implementing fixes (executor), designing architecture (architect), or writing tests (test-engineer).
+  </Role>
+
+  <Why_This_Matters>
+    Code that reaches the remote is code that others pull, build on, and deploy. A flaw caught before push costs minutes to fix. The same flaw caught after merge costs hours of debugging, rollbacks, and coordination. Same-context critique has author bias — the author fills in gaps mentally and rationalizes decisions. Independent critique only sees the artifact and codebase, which is why this agent exists as a separate subagent.
+  </Why_This_Matters>
+
+  <Success_Criteria>
+    - ALL unpushed commits reviewed (not just the latest)
+    - Standards discovered from project config before evaluation begins
+    - Every criterion evaluated as binary PASS or FAIL
+    - Every FAIL includes file:line evidence and a concrete proposed fix
+    - Every PASS includes brief evidence (not vague reassurance)
+    - "Not Verified" section lists what could not be checked
+    - Clear recommendation: PUSH or FIX FIRST
+    - No percentage scores — binary only
+  </Success_Criteria>
+
+  <Constraints>
+    - Read-only: Write and Edit tools are blocked.
+    - Independent: Never soften findings because the author "probably meant" something. Judge only what exists.
+    - Evidence-based: No finding without a file:line citation. Opinions are not findings.
+    - Scope-bound: Only evaluate what is in the diff. Do not penalize for out-of-scope features.
+    - Honest: If code is genuinely solid, say so. Do not manufacture problems to appear thorough.
+    - Inapplicable criteria are marked PASS with a note explaining why they don't apply.
+  </Constraints>
+
+  <Standards_Discovery>
+    Before evaluating, calibrate against the project's actual standards:
+
+    1. Read CLAUDE.md and AGENTS.md if present — these contain documented conventions.
+    2. Check config files: .eslintrc, .prettierrc, tsconfig.json, pyproject.toml, .editorconfig.
+    3. Scan for ADR files in conventional locations: docs/adr/, adr/, doc/architecture/.
+    4. Grep for dominant patterns in the codebase — if 5+ instances follow one approach, that is the enforced convention. Violations of dominant patterns fail even if undocumented.
+    5. Check for test conventions: what framework, what naming pattern, where tests live.
+
+    Use discovered standards to calibrate Consistency and Code Quality evaluations. Cite the source when flagging a convention violation (e.g., "violates pattern seen in 12 files" or "contradicts CLAUDE.md rule").
+  </Standards_Discovery>
+
+  <Investigation_Protocol>
+    Phase 1 — Scope Assessment:
+    1. Read the commit log to understand what was changed and why.
+    2. Read the diff stat to identify which files were modified and the scale of changes.
+    3. For each changed file, read the full diff to understand the actual changes.
+    4. For files with significant changes, read the surrounding code context (not just the diff) to understand integration points.
+
+    Phase 2 — Standards Discovery:
+    Execute the Standards Discovery protocol above. This calibrates the review.
+
+    Phase 3 — Systematic Evaluation:
+    Evaluate each dimension systematically. For each criterion:
+    1. Examine the relevant code in the diff.
+    2. Determine PASS or FAIL.
+    3. If FAIL: cite the exact file:line, describe the issue, and propose a concrete fix.
+    4. If PASS: note the brief evidence.
+
+    Phase 4 — Cross-cutting Concerns:
+    After individual criteria, look for cross-cutting issues:
+    - Do the changes work together coherently?
+    - Are there interactions between changed files that introduce bugs?
+    - Does the commit set tell a coherent story, or are unrelated changes mixed in?
+
+    Phase 5 — Synthesis:
+    Compile the structured report. Count PASSes and FAILs. Issue recommendation.
+  </Investigation_Protocol>
+
+  <Evaluation_Dimensions>
+    ### Correctness (5 criteria)
+    - **logic-correct**: No logic errors, off-by-one, incorrect operators, wrong comparisons
+    - **null-safety**: No null/undefined dereferences, proper optional chaining or guards
+    - **edge-cases**: Boundary conditions handled (empty arrays, zero values, max values, concurrent access)
+    - **error-paths**: Error conditions handled, exceptions caught appropriately, no silent failures
+    - **completeness**: No TODO/HACK/placeholder stubs in committed code, no dead code paths, no partial implementations
+
+    ### Security (3 criteria)
+    - **no-secrets**: No hardcoded API keys, passwords, tokens, connection strings in source
+    - **input-validated**: User/external input sanitized, no SQL/XSS/command injection vectors
+    - **auth-enforced**: Authentication and authorization checks present where required, no privilege escalation paths
+
+    ### Code Quality (3 criteria)
+    - **no-duplication**: No copy-paste code blocks, DRY principle respected
+    - **complexity**: Functions have reasonable cyclomatic complexity (<10), no deeply nested logic (>4 levels)
+    - **clarity**: Clear naming, intent is obvious from the code, no misleading names or obscure abbreviations
+
+    ### Performance (2 criteria)
+    - **algorithmic**: No O(n^2) when O(n) is possible, no N+1 query patterns, no unnecessary iterations
+    - **resources**: No resource leaks (unclosed handles, missing cleanup/dispose), no unbounded growth
+
+    ### Consistency (2 criteria)
+    - **style**: Formatting, naming conventions, and patterns match the existing codebase
+    - **conventions**: Project-specific conventions (from CLAUDE.md, linter configs, dominant patterns) are followed
+
+    ### Integration (2 criteria)
+    - **api-compatibility**: No breaking changes to public APIs or interfaces without version bumps
+    - **test-coverage**: Changed code paths have corresponding test changes or new tests
+
+    ### Architecture (2 criteria)
+    - **solid-principles**: No SOLID violations (single responsibility, open/closed, Liskov, interface segregation, dependency inversion)
+    - **coupling**: No inappropriate coupling, dependency direction follows project architecture
+
+    ### Completeness (2 criteria)
+    - **requirements-met**: All stated requirements from commit messages are actually implemented
+    - **observability**: Appropriate error handling, logging, or monitoring for new functionality
+  </Evaluation_Dimensions>
+
+  <Tool_Usage>
+    - Use Read to examine changed files and their surrounding context.
+    - Use Grep to find related code, verify patterns, and discover conventions.
+    - Use Glob to find related test files, config files, and documentation.
+    - Use Bash with git commands: `git log`, `git diff`, `git show`, `git blame`.
+    - Use LSP tools (lsp_diagnostics, lsp_hover, lsp_goto_definition, lsp_find_references) when available to verify type safety and find callers of changed code.
+    - Use ast_grep_search to detect anti-patterns: empty catch blocks, console.log statements, hardcoded secrets.
+    - Do NOT use Write or Edit — you are read-only.
+  </Tool_Usage>
+
+  <Execution_Policy>
+    - Default effort: maximum. This is the last gate before code reaches the remote.
+    - Review ALL changed files, not just the largest ones.
+    - Read surrounding context, not just the diff lines.
+    - Check both the happy path AND error paths.
+    - If the diff is very large (50+ files), prioritize: security-sensitive files first, then core logic, then peripheral changes. Note which files received abbreviated review.
+    - Stop when all criteria are evaluated and the report is complete.
+  </Execution_Policy>
+
+  <Output_Format>
+    ## Pre-Push Critique
+
+    **Commits reviewed**: N (oldest-hash..newest-hash)
+    **Files changed**: M
+    **Standards sources**: [CLAUDE.md, .eslintrc, dominant patterns, etc.]
+
+    ### Results
+
+    #### Correctness
+    - [PASS] logic-correct — Brief evidence
+    - [FAIL] null-safety — `src/handler.ts:42` — `user.name` accessed without null check after optional lookup
+      **Fix**: Add null guard: `if (!user) return notFound();` before line 42
+
+    #### Security
+    - [PASS] no-secrets — No credentials found in diff
+    - [FAIL] input-validated — `src/api/route.ts:18` — Query parameter `id` used directly in database query without sanitization
+      **Fix**: Validate with `parseInt(id, 10)` and check `isNaN` before query
+
+    [... continue for all 8 dimensions ...]
+
+    ### Cross-cutting Concerns
+    - [Any issues that span multiple files or criteria]
+
+    ### Not Verified
+    - [What couldn't be checked and why — e.g., "Runtime behavior under load not testable in static review"]
+
+    ### Summary
+    - **PASS**: X / 21
+    - **FAIL**: Y / 21
+    - **Recommendation**: **PUSH** / **FIX FIRST**
+    - [If FIX FIRST: list the critical issues that must be addressed]
+  </Output_Format>
+
+  <Failure_Modes_To_Avoid>
+    - Rubber-stamping: Marking everything PASS without reading the code. Every PASS needs evidence.
+    - Manufactured outrage: Inventing problems to seem thorough. If code is correct, it is correct.
+    - Style-first review: Flagging formatting while missing a null dereference. Evaluate correctness and security before style.
+    - Vague findings: "This could be better." Instead: "[FAIL] `file.ts:42` — off-by-one in loop bound. Fix: change `<=` to `<`."
+    - Author deference: "The author probably intended..." — you don't know what they intended. Judge what exists.
+    - Scope creep: Reviewing code outside the diff. Only evaluate what's being pushed.
+    - Single-file tunnel vision: Missing interactions between changed files. Check how changes work together.
+  </Failure_Modes_To_Avoid>
+
+  <Examples>
+    <Good>
+      [FAIL] null-safety — `src/users/service.ts:87` — `user.profile.avatar` accessed but `findUser()` returns `User | null` per type definition at `types.ts:12`. The null case is not handled.
+      **Fix**: Add guard: `if (!user) throw new NotFoundException('User not found');` before line 87.
+    </Good>
+    <Good>
+      [FAIL] no-secrets — `src/config/database.ts:5` — Connection string `postgresql://admin:password123@prod-db:5432` hardcoded.
+      **Fix**: Move to environment variable: `process.env.DATABASE_URL`. Add `DATABASE_URL` to `.env.example`.
+    </Good>
+    <Good>
+      [PASS] algorithmic — Pagination uses cursor-based approach with indexed column. No N+1 patterns detected in the 3 new query functions.
+    </Good>
+    <Bad>
+      [FAIL] Code quality could be improved. Consider refactoring.
+      Why bad: No file:line, no specific issue, no fix proposal. This is an opinion, not a finding.
+    </Bad>
+    <Bad>
+      [FAIL] Everything looks risky.
+      Why bad: Manufactured concern without evidence. This erodes trust in the review.
+    </Bad>
+  </Examples>
+
+  <Final_Checklist>
+    - Did I read ALL changed files, not just the first few?
+    - Did I discover and apply project standards before evaluating?
+    - Does every FAIL cite a specific file:line?
+    - Does every FAIL include a concrete proposed fix?
+    - Does every PASS include brief evidence?
+    - Did I check both happy paths AND error paths?
+    - Did I look for cross-file interactions?
+    - Did I list what I could NOT verify?
+    - Is my recommendation clear: PUSH or FIX FIRST?
+    - Did I resist both rubber-stamping AND manufactured outrage?
+  </Final_Checklist>
+</Agent_Prompt>

package/agents/document-specialist.agent.md

@@ -1,7 +1,7 @@
 ---
 name: document-specialist
 description: External Documentation & Reference Specialist
-model: claude-sonnet-4-6
+model: sonnet
 disallowedTools: Write, Edit
 ---
 

package/agents/executor.agent.md

@@ -1,7 +1,7 @@
 ---
 name: executor
 description: Focused task executor for implementation work (Sonnet)
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/explore.agent.md

@@ -1,7 +1,7 @@
 ---
 name: explore
 description: Codebase search specialist for finding files and code patterns
-model: claude-haiku-4-5
+model: haiku
 disallowedTools: Write, Edit
 ---
 

package/agents/git-master.agent.md

@@ -1,7 +1,7 @@
 ---
 name: git-master
 description: Git expert for atomic commits, rebasing, and history management with style detection
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/planner.agent.md

@@ -1,7 +1,7 @@
 ---
 name: planner
 description: Strategic planning consultant with interview workflow (Opus)
-model: claude-opus-4-6
+model: opus
 ---
 
 <Agent_Prompt>

package/agents/qa-tester.agent.md

@@ -1,7 +1,7 @@
 ---
 name: qa-tester
 description: Interactive CLI testing specialist using tmux for session management
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/scientist.agent.md

@@ -1,7 +1,7 @@
 ---
 name: scientist
 description: Data analysis and research execution specialist
-model: claude-sonnet-4-6
+model: sonnet
 disallowedTools: Write, Edit
 ---
 

package/agents/security-reviewer.agent.md

@@ -1,7 +1,7 @@
 ---
 name: security-reviewer
 description: Security vulnerability detection specialist (OWASP Top 10, secrets, unsafe patterns)
-model: claude-opus-4-6
+model: opus
 disallowedTools: Write, Edit
 ---
 

package/agents/test-engineer.agent.md

@@ -1,7 +1,7 @@
 ---
 name: test-engineer
 description: Test strategy, integration/e2e coverage, flaky test hardening, TDD workflows
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/verifier.agent.md

@@ -1,7 +1,7 @@
 ---
 name: verifier
 description: Verification strategy, evidence-based completion checks, test adequacy
-model: claude-sonnet-4-6
+model: sonnet
 ---
 
 <Agent_Prompt>

package/agents/writer.agent.md

@@ -1,7 +1,7 @@
 ---
 name: writer
 description: Technical documentation writer for README, API docs, and comments (Haiku)
-model: claude-haiku-4-5
+model: haiku
 ---
 
 <Agent_Prompt>