oh-my-githubcopilot 1.4.1 → 1.8.0-alpha.f50f59a

package/dist/skills/setup.mjs

@@ -1,38 +1,50 @@
-// src/skills/setup.mts
-async function activate(input2) {
+// src/skills/omp-setup.mts
+import { fileURLToPath } from "url";
+import { dirname, join } from "path";
+var REQUIRED_COPILOT_EXPERIMENTAL_FEATURES = [
+  "STATUS_LINE",
+  "SHOW_FILE",
+  "EXTENSIONS",
+  "BACKGROUND_SESSIONS",
+  "CONFIGURE_COPILOT_AGENT",
+  "MULTI_TURN_AGENTS",
+  "SESSION_STORE"
+];
+function getPackageRoot() {
+  return join(dirname(fileURLToPath(import.meta.url)), "..", "..");
+}
+async function activate(input) {
   const { spawn } = await import("child_process");
-  const isNonInteractive = input2.args.includes("--non-interactive");
-  const isMcpOnly = input2.args.includes("--mcp-only");
-  const isSkipMcp = input2.args.includes("--skip-mcp");
-  const baseArgs = ["bin/omp.mjs", "setup"];
-  if (isMcpOnly) baseArgs.push("--mcp-only");
-  if (isSkipMcp) baseArgs.push("--skip-mcp");
-  if (isNonInteractive) baseArgs.push("--non-interactive");
+  const packageRoot = getPackageRoot();
+  const baseArgs = ["bin/omp.mjs", "setup", ...input.args];
   return new Promise((resolve) => {
-    const child = spawn("node", baseArgs, { stdio: "inherit" });
-    child.on("close", (code) => {
-      if (code === 0) {
-        resolve({ status: "ok", message: "OMP setup complete." });
-      } else {
-        resolve({ status: "error", message: `Setup exited with code ${code}` });
+    const child = spawn("node", baseArgs, {
+      cwd: packageRoot,
+      stdio: "inherit",
+      env: {
+        ...process.env,
+        OMP_COPILOT_REQUIRED_EXPERIMENTAL_FEATURES: process.env["OMP_COPILOT_REQUIRED_EXPERIMENTAL_FEATURES"] ?? REQUIRED_COPILOT_EXPERIMENTAL_FEATURES.join(","),
+        OMP_COPILOT_STATUS_LINE_COMMAND: process.env["OMP_COPILOT_STATUS_LINE_COMMAND"] ?? join(packageRoot, "bin", "omp-statusline.sh")
       }
     });
-    child.on("error", (err) => {
-      resolve({ status: "error", message: `Failed to spawn omp setup: ${err.message}` });
+    child.on("close", (code) => {
+      resolve({ status: code === 0 ? "ok" : "error", message: `Setup exited with code ${code}` });
     });
+    child.on("error", (err) => resolve({ status: "error", message: `Failed to spawn: ${err.message}` }));
   });
 }
-var input = JSON.parse(await readStdin());
-var output = await activate(input);
-console.log(JSON.stringify(output));
-async function readStdin() {
-  const chunks = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return chunks.join("");
+function deactivate() {
+}
+
+// src/skills/setup.mts
+async function activate2(input) {
+  return activate(input);
+}
+function deactivate2() {
+  deactivate();
 }
 export {
-  activate
+  activate2 as activate,
+  deactivate2 as deactivate
 };
 //# sourceMappingURL=setup.mjs.map

package/dist/skills/setup.mjs.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
-  "sources": ["../../src/skills/setup.mts"],
-  "sourcesContent": ["/**\n * setup skill\n *\n * Orchestrates both phases of the OMP setup wizard:\n *   Phase 1: Base OMP setup (SQLite DB, directory structure, first-run guidance)\n *   Phase 2: MCP server configuration\n *\n * Invoked via `/setup` or `setup:` keyword.\n * For MCP-only setup, use the `mcp-setup` skill instead.\n */\n\nexport interface SkillInput {\n  trigger: string;\n  args: string[];\n}\n\nexport interface SkillOutput {\n  status: \"ok\" | \"error\";\n  message: string;\n}\n\nexport async function activate(input: SkillInput): Promise<SkillOutput> {\n  const { spawn } = await import(\"child_process\");\n\n  const isNonInteractive = input.args.includes(\"--non-interactive\");\n  const isMcpOnly = input.args.includes(\"--mcp-only\");\n  const isSkipMcp = input.args.includes(\"--skip-mcp\");\n\n  const baseArgs = [\"bin/omp.mjs\", \"setup\"];\n  if (isMcpOnly) baseArgs.push(\"--mcp-only\");\n  if (isSkipMcp) baseArgs.push(\"--skip-mcp\");\n  if (isNonInteractive) baseArgs.push(\"--non-interactive\");\n\n  return new Promise((resolve) => {\n    const child = spawn(\"node\", baseArgs, { stdio: \"inherit\" });\n\n    child.on(\"close\", (code) => {\n      if (code === 0) {\n        resolve({ status: \"ok\", message: \"OMP setup complete.\" });\n      } else {\n        resolve({ status: \"error\", message: `Setup exited with code ${code}` });\n      }\n    });\n\n    child.on(\"error\", (err) => {\n      resolve({ status: \"error\", message: `Failed to spawn omp setup: ${err.message}` });\n    });\n  });\n}\n\n// Standalone entry point\nconst input: SkillInput = JSON.parse(await readStdin());\nconst output = await activate(input);\nconsole.log(JSON.stringify(output));\n\nasync function readStdin(): Promise<string> {\n  const chunks: string[] = [];\n  for await (const chunk of process.stdin) {\n    chunks.push(chunk);\n  }\n  return chunks.join(\"\");\n}\n"],
-  "mappings": ";AAqBA,eAAsB,SAASA,QAAyC;AACtE,QAAM,EAAE,MAAM,IAAI,MAAM,OAAO,eAAe;AAE9C,QAAM,mBAAmBA,OAAM,KAAK,SAAS,mBAAmB;AAChE,QAAM,YAAYA,OAAM,KAAK,SAAS,YAAY;AAClD,QAAM,YAAYA,OAAM,KAAK,SAAS,YAAY;AAElD,QAAM,WAAW,CAAC,eAAe,OAAO;AACxC,MAAI,UAAW,UAAS,KAAK,YAAY;AACzC,MAAI,UAAW,UAAS,KAAK,YAAY;AACzC,MAAI,iBAAkB,UAAS,KAAK,mBAAmB;AAEvD,SAAO,IAAI,QAAQ,CAAC,YAAY;AAC9B,UAAM,QAAQ,MAAM,QAAQ,UAAU,EAAE,OAAO,UAAU,CAAC;AAE1D,UAAM,GAAG,SAAS,CAAC,SAAS;AAC1B,UAAI,SAAS,GAAG;AACd,gBAAQ,EAAE,QAAQ,MAAM,SAAS,sBAAsB,CAAC;AAAA,MAC1D,OAAO;AACL,gBAAQ,EAAE,QAAQ,SAAS,SAAS,0BAA0B,IAAI,GAAG,CAAC;AAAA,MACxE;AAAA,IACF,CAAC;AAED,UAAM,GAAG,SAAS,CAAC,QAAQ;AACzB,cAAQ,EAAE,QAAQ,SAAS,SAAS,8BAA8B,IAAI,OAAO,GAAG,CAAC;AAAA,IACnF,CAAC;AAAA,EACH,CAAC;AACH;AAGA,IAAM,QAAoB,KAAK,MAAM,MAAM,UAAU,CAAC;AACtD,IAAM,SAAS,MAAM,SAAS,KAAK;AACnC,QAAQ,IAAI,KAAK,UAAU,MAAM,CAAC;AAElC,eAAe,YAA6B;AAC1C,QAAM,SAAmB,CAAC;AAC1B,mBAAiB,SAAS,QAAQ,OAAO;AACvC,WAAO,KAAK,KAAK;AAAA,EACnB;AACA,SAAO,OAAO,KAAK,EAAE;AACvB;",
-  "names": ["input"]
+  "sources": ["../../src/skills/omp-setup.mts", "../../src/skills/setup.mts"],
+  "sourcesContent": ["/**\n * omp-setup skill\n *\n * ID:       omp-setup\n * Keywords: setup:, /setup, /omp:setup\n * Tier:     developer tool\n *\n * Orchestrates the OMP setup wizard:\n *   Phase 1: Base OMP setup (directory structure, first-run guidance)\n *   Phase 2: MCP server configuration\n */\n\nimport { fileURLToPath } from \"url\";\nimport { dirname, join } from \"path\";\n\nexport interface SkillInput {\n  trigger: string;\n  args: string[];\n}\n\nexport interface SkillOutput {\n  status: \"ok\" | \"error\";\n  message: string;\n}\n\nconst REQUIRED_COPILOT_EXPERIMENTAL_FEATURES = [\n  \"STATUS_LINE\",\n  \"SHOW_FILE\",\n  \"EXTENSIONS\",\n  \"BACKGROUND_SESSIONS\",\n  \"CONFIGURE_COPILOT_AGENT\",\n  \"MULTI_TURN_AGENTS\",\n  \"SESSION_STORE\",\n] as const;\n\nfunction getPackageRoot(): string {\n  return join(dirname(fileURLToPath(import.meta.url)), \"..\", \"..\");\n}\n\nexport async function activate(input: SkillInput): Promise<SkillOutput> {\n  const { spawn } = await import(\"child_process\");\n  const packageRoot = getPackageRoot();\n  const baseArgs = [\"bin/omp.mjs\", \"setup\", ...input.args];\n  return new Promise((resolve) => {\n    const child = spawn(\"node\", baseArgs, {\n      cwd: packageRoot,\n      stdio: \"inherit\",\n      env: {\n        ...process.env,\n        OMP_COPILOT_REQUIRED_EXPERIMENTAL_FEATURES:\n          process.env[\"OMP_COPILOT_REQUIRED_EXPERIMENTAL_FEATURES\"] ??\n          REQUIRED_COPILOT_EXPERIMENTAL_FEATURES.join(\",\"),\n        OMP_COPILOT_STATUS_LINE_COMMAND:\n          process.env[\"OMP_COPILOT_STATUS_LINE_COMMAND\"] ??\n          join(packageRoot, \"bin\", \"omp-statusline.sh\"),\n      },\n    });\n    child.on(\"close\", (code) => {\n      resolve({ status: code === 0 ? \"ok\" : \"error\", message: `Setup exited with code ${code}` });\n    });\n    child.on(\"error\", (err) => resolve({ status: \"error\", message: `Failed to spawn: ${err.message}` }));\n  });\n}\n\nexport function deactivate(): void {\n  // No persistent resources to clean up\n}\n", "/**\n * setup skill\n *\n * Legacy alias for `omp-setup`.\n * Keep this wrapper while docs and manifests migrate to the `omp-*` namespace.\n */\n\nimport {\n  activate as activateOmpSetup,\n  deactivate as deactivateOmpSetup,\n  type SkillInput,\n  type SkillOutput,\n} from \"./omp-setup.mjs\";\n\nexport type { SkillInput, SkillOutput } from \"./omp-setup.mjs\";\n\nexport async function activate(input: SkillInput): Promise<SkillOutput> {\n  return activateOmpSetup(input);\n}\n\nexport function deactivate(): void {\n  deactivateOmpSetup();\n}\n"],
+  "mappings": ";AAYA,SAAS,qBAAqB;AAC9B,SAAS,SAAS,YAAY;AAY9B,IAAM,yCAAyC;AAAA,EAC7C;AAAA,EACA;AAAA,EACA;AAAA,EACA;AAAA,EACA;AAAA,EACA;AAAA,EACA;AACF;AAEA,SAAS,iBAAyB;AAChC,SAAO,KAAK,QAAQ,cAAc,YAAY,GAAG,CAAC,GAAG,MAAM,IAAI;AACjE;AAEA,eAAsB,SAAS,OAAyC;AACtE,QAAM,EAAE,MAAM,IAAI,MAAM,OAAO,eAAe;AAC9C,QAAM,cAAc,eAAe;AACnC,QAAM,WAAW,CAAC,eAAe,SAAS,GAAG,MAAM,IAAI;AACvD,SAAO,IAAI,QAAQ,CAAC,YAAY;AAC9B,UAAM,QAAQ,MAAM,QAAQ,UAAU;AAAA,MACpC,KAAK;AAAA,MACL,OAAO;AAAA,MACP,KAAK;AAAA,QACH,GAAG,QAAQ;AAAA,QACX,4CACE,QAAQ,IAAI,4CAA4C,KACxD,uCAAuC,KAAK,GAAG;AAAA,QACjD,iCACE,QAAQ,IAAI,iCAAiC,KAC7C,KAAK,aAAa,OAAO,mBAAmB;AAAA,MAChD;AAAA,IACF,CAAC;AACD,UAAM,GAAG,SAAS,CAAC,SAAS;AAC1B,cAAQ,EAAE,QAAQ,SAAS,IAAI,OAAO,SAAS,SAAS,0BAA0B,IAAI,GAAG,CAAC;AAAA,IAC5F,CAAC;AACD,UAAM,GAAG,SAAS,CAAC,QAAQ,QAAQ,EAAE,QAAQ,SAAS,SAAS,oBAAoB,IAAI,OAAO,GAAG,CAAC,CAAC;AAAA,EACrG,CAAC;AACH;AAEO,SAAS,aAAmB;AAEnC;;;AClDA,eAAsBA,UAAS,OAAyC;AACtE,SAAO,SAAiB,KAAK;AAC/B;AAEO,SAASC,cAAmB;AACjC,aAAmB;AACrB;",
+  "names": ["activate", "deactivate"]
 }

package/hooks/hooks.json

@@ -1,47 +1,41 @@
 {
-  "schemaVersion": "1.0",
-  "hooks": [
-    {
-      "id": "keyword-detector",
-      "entry": "./dist/hooks/keyword-detector.mjs",
-      "trigger": "UserPromptSubmitted",
-      "timeoutMs": 200,
-      "priority": 100
-    },
-    {
-      "id": "delegation-enforcer",
-      "entry": "./dist/hooks/delegation-enforcer.mjs",
-      "trigger": "PreToolUse",
-      "timeoutMs": 200,
-      "priority": 90
-    },
-    {
-      "id": "model-router",
-      "entry": "./dist/hooks/model-router.mjs",
-      "trigger": "PreToolUse",
-      "timeoutMs": 200,
-      "priority": 80
-    },
-    {
-      "id": "token-tracker",
-      "entry": "./dist/hooks/token-tracker.mjs",
-      "trigger": "PostToolUse",
-      "timeoutMs": 200,
-      "priority": 70
-    },
-    {
-      "id": "hud-emitter",
-      "entry": "./dist/hooks/hud-emitter.mjs",
-      "trigger": "PostToolUse",
-      "timeoutMs": 200,
-      "priority": 60
-    },
-    {
-      "id": "stop-continuation",
-      "entry": "./dist/hooks/stop-continuation.mjs",
-      "trigger": "PostToolUse",
-      "timeoutMs": 200,
-      "priority": 50
-    }
-  ]
+  "version": 1,
+  "hooks": {
+    "UserPromptSubmitted": [
+      {
+        "type": "command",
+        "bash": "node ./bin/omp.mjs hook keyword-detector",
+        "timeoutSec": 0.2
+      }
+    ],
+    "PreToolUse": [
+      {
+        "type": "command",
+        "bash": "node ./dist/hooks/delegation-enforcer.mjs",
+        "timeoutSec": 0.2
+      },
+      {
+        "type": "command",
+        "bash": "node ./dist/hooks/model-router.mjs",
+        "timeoutSec": 0.2
+      }
+    ],
+    "PostToolUse": [
+      {
+        "type": "command",
+        "bash": "node ./dist/hooks/token-tracker.mjs",
+        "timeoutSec": 0.2
+      },
+      {
+        "type": "command",
+        "bash": "node ./dist/hooks/hud-emitter.mjs",
+        "timeoutSec": 0.2
+      },
+      {
+        "type": "command",
+        "bash": "node ./dist/hooks/stop-continuation.mjs",
+        "timeoutSec": 0.2
+      }
+    ]
+  }
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "oh-my-githubcopilot",
-  "version": "1.4.1",
-  "description": "Multi-agent orchestration for GitHub Copilot CLI. 23 agents, 25 skills, parallel execution, HUD, PSM, SWE-bench.",
+  "version": "1.8.0-alpha.f50f59a",
+  "description": "Multi-agent orchestration for GitHub Copilot CLI. 23 agents, 39 skills, parallel execution, HUD, PSM, SWE-bench.",
   "author": {
     "name": "r3dlex"
   },
@@ -21,17 +21,21 @@
     "omp": "./bin/omp.mjs"
   },
   "files": [
+    "README.md",
     "dist/",
     "bin/",
+    "agents/",
     "skills/",
     "hooks/",
-    "src/agents/",
+    "plugin.json",
+    ".mcp.json",
     ".claude-plugin/",
     "AGENTS.md",
     "CHANGELOG.md",
     "LICENSE"
   ],
   "scripts": {
+    "hud:watch": "node --import tsx src/index.mts hud --watch",
     "build": "node --import tsx esbuild.config.mts",
     "dev": "node --import tsx --watch esbuild.config.mts",
     "test": "vitest run",
@@ -40,8 +44,9 @@
     "lint": "eslint src/",
     "format": "prettier --write src/",
     "typecheck": "tsc --noEmit",
-    "sync-claude-plugin": "cp .github/plugin/plugin.json .claude-plugin/plugin.json",
+    "sync-claude-plugin": "cp plugin.json .claude-plugin/plugin.json",
     "prepublishOnly": "npm run build && npm run sync-claude-plugin",
+    "postinstall": "[ -d dist ] || npm run build",
     "archgate:init": "npx archgate init --editor claude",
     "archgate:check": "npx archgate check"
   },

package/plugin.json

@@ -0,0 +1,71 @@
+{
+  "schemaVersion": "1.0",
+  "name": "oh-my-githubcopilot",
+  "version": "1.8.0",
+  "description": "Multi-agent orchestration for GitHub Copilot CLI. 23 agents, 39 skills, parallel execution, HUD, PSM, SWE-bench.",
+  "author": {
+    "name": "r3dlex"
+  },
+  "repository": "https://github.com/r3dlex/oh-my-githubcopilot",
+  "license": "Apache-2.0",
+  "keywords": [
+    "orchestration",
+    "multi-agent",
+    "autopilot",
+    "swe-bench",
+    "hud",
+    "copilot-cli"
+  ],
+  "agents": [
+    "./agents"
+  ],
+  "skills": [
+    "./skills/autopilot",
+    "./skills/ralph",
+    "./skills/ultrawork",
+    "./skills/team",
+    "./skills/ecomode",
+    "./skills/swarm",
+    "./skills/pipeline",
+    "./skills/deep-interview",
+    "./skills/omp-plan",
+    "./skills/omp-setup",
+    "./skills/hud",
+    "./skills/wiki",
+    "./skills/learner",
+    "./skills/note",
+    "./skills/trace",
+    "./skills/release",
+    "./skills/configure-notifications",
+    "./skills/psm",
+    "./skills/swe-bench",
+    "./skills/mcp-setup",
+    "./skills/setup",
+    "./skills/graphify",
+    "./skills/graphwiki",
+    "./skills/graph-provider",
+    "./skills/spending",
+    "./skills/ralplan",
+    "./skills/research",
+    "./skills/omp-doctor",
+    "./skills/omp-reference",
+    "./skills/ai-slop-cleaner",
+    "./skills/tdd",
+    "./skills/improve-codebase-architecture",
+    "./skills/skillify",
+    "./skills/interview",
+    "./skills/graph-context",
+    "./skills/interactive-menu",
+    "./skills/notifications",
+    "./skills/doctor",
+    "./skills/session"
+  ],
+  "hooks": "./hooks/hooks.json",
+  "mcp": "./.mcp.json",
+  "entryPoints": {
+    "cli": "./bin/omp.mjs",
+    "mcp": "./dist/mcp/server.mjs"
+  },
+  "agentFormat": ".agent.md",
+  "agentFilePattern": "*.agent.md"
+}

package/skills/ai-slop-cleaner/SKILL.md

@@ -0,0 +1,137 @@
+---
+name: ai-slop-cleaner
+description: Clean AI-generated code slop with a regression-safe, deletion-first workflow and optional reviewer-only mode
+level: 3
+triggers:
+  - deslop
+  - /omp:ai-slop-cleaner
+  - anti-slop
+---
+
+# AI Slop Cleaner
+
+Use this skill to clean AI-generated code slop without drifting scope or changing intended behavior. In OMP, this is the bounded cleanup workflow for code that works but feels bloated, repetitive, weakly tested, or over-abstracted.
+
+## When to Use
+
+Use this skill when:
+- the user explicitly says `deslop`, `anti-slop`, or `AI slop`
+- the request is to clean up or refactor code that feels noisy, repetitive, or overly abstract
+- follow-up implementation left duplicate logic, dead code, wrapper layers, boundary leaks, or weak regression coverage
+- the user wants a reviewer-only anti-slop pass via `--review`
+- the goal is simplification and cleanup, not new feature delivery
+
+## When Not to Use
+
+Do not use this skill when:
+- the task is mainly a new feature build or product change
+- the user wants a broad redesign instead of an incremental cleanup pass
+- the request is a generic refactor with no simplification or anti-slop intent
+- behavior is too unclear to protect with tests or a concrete verification plan
+
+## OMP Execution Posture
+
+- Preserve behavior unless the user explicitly asks for behavior changes.
+- Lock behavior with focused regression tests first whenever practical.
+- Write a cleanup plan before editing code.
+- Prefer deletion over addition.
+- Reuse existing utilities and patterns before introducing new ones.
+- Avoid new dependencies unless the user explicitly requests them.
+- Keep diffs small, reversible, and smell-focused.
+- Stay concise and evidence-dense: inspect, edit, verify, and report.
+- Treat new user instructions as local scope updates without dropping earlier non-conflicting constraints.
+
+## Scoped File-List Usage
+
+This skill can be bounded to an explicit file list or changed-file scope when the caller already knows the safe cleanup surface.
+
+- Good fit: `omp:ai-slop-cleaner skills/ralph/SKILL.md skills/ai-slop-cleaner/SKILL.md`
+- Good fit: a Ralph session handing off only the files changed in that session
+- Preserve the same regression-safe workflow even when the scope is a short file list
+- Do not silently expand a changed-file scope into broader cleanup work unless the user explicitly asks for it
+
+## Ralph Integration
+
+omp:ralph can invoke this skill as a bounded post-review cleanup pass.
+
+- In that workflow, the cleaner runs in standard mode (not `--review`)
+- The cleanup scope is the Ralph session's changed files only
+- After the cleanup pass, Ralph re-runs regression verification before completion
+- `--review` remains the reviewer-only follow-up mode, not the default Ralph integration path
+
+## Review Mode (`--review`)
+
+`--review` is a reviewer-only pass after cleanup work is drafted. It exists to preserve explicit writer/reviewer separation for anti-slop work.
+
+- **Writer pass**: make the cleanup changes with behavior locked by tests.
+- **Reviewer pass**: inspect the cleanup plan, changed files, and verification evidence.
+- The same pass must not both write and self-approve high-impact cleanup without a separate review step.
+
+In review mode:
+1. Do **not** start by editing files.
+2. Review the cleanup plan, changed files, and regression coverage.
+3. Check specifically for:
+   - leftover dead code or unused exports
+   - duplicate logic that should have been consolidated
+   - needless wrappers or abstractions that still blur boundaries
+   - missing tests or weak verification for preserved behavior
+   - cleanup that appears to have changed behavior without intent
+4. Produce a reviewer verdict with required follow-ups.
+5. Hand needed changes back to a separate writer pass instead of fixing and approving in one step.
+
+## Workflow
+
+1. **Protect current behavior first**
+   - Identify what must stay the same.
+   - Add or run the narrowest regression tests needed before editing.
+   - If tests cannot come first, record the verification plan explicitly before touching code.
+
+2. **Write a cleanup plan before code**
+   - Bound the pass to the requested files or feature area.
+   - List the concrete smells to remove.
+   - Order the work from safest deletion to riskier consolidation.
+
+3. **Classify the slop before editing**
+   - **Duplication** — repeated logic, copy-paste branches, redundant helpers
+   - **Dead code** — unused code, unreachable branches, stale flags, debug leftovers
+   - **Needless abstraction** — pass-through wrappers, speculative indirection, single-use helper layers
+   - **Boundary violations** — hidden coupling, misplaced responsibilities, wrong-layer imports or side effects
+   - **Missing tests** — behavior not locked, weak regression coverage, edge-case gaps
+
+4. **Run one smell-focused pass at a time**
+   - **Pass 1: Dead code deletion**
+   - **Pass 2: Duplicate removal**
+   - **Pass 3: Naming and error-handling cleanup**
+   - **Pass 4: Test reinforcement**
+   - Re-run targeted verification after each pass.
+   - Do not bundle unrelated refactors into the same edit set.
+
+5. **Run the quality gates**
+   - Keep regression tests green.
+   - Run the relevant lint, typecheck, and unit/integration tests for the touched area.
+   - Run existing static or security checks when available.
+   - If a gate fails, fix the issue or back out the risky cleanup instead of forcing it through.
+
+6. **Close with an evidence-dense report**
+   Always report:
+   - **Changed files**
+   - **Simplifications**
+   - **Behavior lock / verification run**
+   - **Remaining risks**
+
+## Usage
+
+- `/omp:ai-slop-cleaner <target>`
+- `/omp:ai-slop-cleaner <target> --review`
+- `/omp:ai-slop-cleaner <file-a> <file-b> <file-c>`
+- From omp:ralph: run the cleaner on the Ralph session's changed files only, then return to Ralph for post-cleanup regression verification
+
+## Good Fits
+
+**Good:** `deslop this module: too many wrappers, duplicate helpers, and dead code`
+
+**Good:** `cleanup the AI slop in src/auth and tighten boundaries without changing behavior`
+
+**Bad:** `refactor auth to support SSO`
+
+**Bad:** `clean up formatting`

package/skills/autopilot/SKILL.md

@@ -1,3 +1,9 @@
+---
+name: autopilot
+description: Autonomous end-to-end execution from idea to working code
+trigger: "autopilot:, /autopilot, /omp:autopilot"
+autoinvoke: false
+---
 # Skill: Autopilot
 
 ## Metadata

package/skills/configure-notifications/SKILL.md

@@ -1,3 +1,9 @@
+---
+name: configure-notifications
+description: Configure session notification settings
+trigger: "configure-notifications:, /configure-notifications, /omp:configure-notifications"
+autoinvoke: false
+---
 # Skill: Configure-Notifications
 
 ## Metadata

package/skills/deep-interview/SKILL.md

@@ -1,3 +1,9 @@
+---
+name: deep-interview
+description: Deep requirements interview workflow
+trigger: "deep interview:, /deep-interview, /omp:deep-interview"
+autoinvoke: false
+---
 # Skill: Deep-Interview
 
 ## Metadata

package/skills/doctor/SKILL.md

@@ -0,0 +1,188 @@
+---
+name: doctor
+description: Diagnose and fix common issues. Use for "debug", "fix this", "what's wrong", and "doctor".
+trigger: "doctor:, /doctor, /omp:doctor"
+autoinvoke: false
+---
+# Skill: Doctor
+
+## Metadata
+
+| Field | Value |
+|-------|-------|
+| **ID** | `doctor` |
+| **Keywords** | `doctor:`, `/doctor` |
+| **Tier** | Developer Tool |
+| **Source** | `src/skills/doctor.mts` |
+
+## Description
+
+Diagnose problems and prescribe solutions based on symptoms. A general-purpose Socratic diagnostic for any error, failure, or unexpected behavior — not limited to OMP installation issues.
+
+## Differentiation from omp-doctor
+
+| Aspect | doctor | omp-doctor |
+|--------|--------|------------|
+| **Scope** | Any problem: errors, performance, config, setup | OMP-specific: plugin version, skills dir, agents dir, plugin.json |
+| **Method** | Symptom collection -> hypothesis -> root cause -> prescription | Structured installation checks with known pass/fail criteria |
+| **Use when** | Something is broken and you don't know why | OMP itself is not working correctly after install |
+
+## Interface
+
+```typescript
+interface SkillInput {
+  trigger: string;
+  args: string[];
+}
+
+interface SkillOutput {
+  status: "ok" | "error";
+  message: string;
+}
+
+export async function activate(input: SkillInput): Promise<SkillOutput>
+export function deactivate(): void
+```
+
+## Implementation
+
+Spawns `bin/omp.mjs doctor [args]`. No persistent resources are maintained.
+
+## When to Use
+
+- Unexpected errors or failures
+- Degraded performance
+- Configuration issues
+- Setup problems
+- "Something is wrong"
+
+## Diagnosis Process
+
+### 1. Collect Symptoms
+- Error messages
+- Behavior observed
+- When it started
+- Recent changes
+
+### 2. Identify Possible Causes
+- Based on symptoms
+- Known patterns
+- Recent updates
+- Environmental changes
+
+### 3. Narrow Down
+- Test each hypothesis
+- Eliminate unlikely causes
+- Gather more data
+
+### 4. Prescribe Fix
+- Root cause identified
+- Solution determined
+- Steps to resolve
+- Prevention
+
+## Common Diagnoses
+
+### "Command not found"
+**Possible causes:**
+- PATH misconfiguration
+- Installation incomplete
+- Typo in command name
+- Shell not reloaded
+
+**Fix:**
+```bash
+# Check PATH
+echo $PATH
+
+# Verify installation
+which {command}
+
+# Reload shell
+source ~/.zshrc
+```
+
+### "Permission denied"
+**Possible causes:**
+- File ownership wrong
+- Missing execute bit
+- SELinux/AppArmor
+- Sandbox restrictions
+
+**Fix:**
+```bash
+# Fix ownership
+chown {user}:{group} {file}
+
+# Add execute
+chmod +x {file}
+```
+
+### "Module not found"
+**Possible causes:**
+- Not installed
+- Wrong environment
+- PYTHONPATH issue
+- Virtual env not activated
+
+**Fix:**
+```bash
+# Install if needed
+pip install {module}
+
+# Check environment
+which python
+
+# Set PYTHONPATH
+export PYTHONPATH={path}
+```
+
+## Output Format
+
+```
+## Doctor: {issue}
+
+### Symptoms Reported
+- {symptom 1}
+- {symptom 2}
+
+### Diagnosis
+
+#### Possible Causes
+| Cause | Likelihood | Evidence |
+|-------|------------|----------|
+| {cause 1} | High | {evidence} |
+| {cause 2} | Low | {evidence} |
+
+#### Root Cause
+**Identified:** {cause}
+**Confidence:** {percentage}
+
+### Prescription
+
+#### Fix Steps
+\`\`\`bash
+{step 1}
+{step 2}
+\`\`\`
+
+#### Verification
+\`\`\`bash
+{verify command}
+\`\`\`
+
+### Prevention
+{how to prevent recurrence}
+
+### Outcome
+**Status:** {resolved|partial|unresolved}
+**Follow-up:** {if needed}
+```
+
+## Constraints
+
+- Collect evidence before diagnosing
+- Start with common causes
+- Verify fixes work
+- Document solutions
+- Build pattern library