@touchskyer/opc 0.2.0 → 0.2.2

package/README.md

@@ -35,6 +35,8 @@ git clone https://github.com/iamtouchskyer/opc.git
 ln -s $(pwd)/opc ~/.claude/skills/opc
 ```
 
+> **Note:** Symlink tracks the repo — `git pull` updates the skill immediately. Use `cp -r` instead if you want a stable snapshot.
+
 ### Use it
 
 ```bash
@@ -130,7 +132,13 @@ You: /opc review this PR
 6. Report     → Curated findings with severity, file:line references, and fix suggestions
 ```
 
-## Works better with memex
+## Requirements
+
+- [Claude Code](https://claude.ai/code) (CLI, desktop app, or IDE extension)
+- Node.js ≥ 18 (for npm install only — not needed if you install manually)
+- That's it. No runtime dependencies, no build step, no MCP server. Just markdown files.
+
+## Works better with memex (optional)
 
 OPC works standalone — but pair it with [memex](https://github.com/iamtouchskyer/memex) and it learns across sessions. Memex remembers which roles were useful, which findings were false positives, and your project-specific context. OPC doesn't need to know how memex works — memex drives itself.
 
@@ -138,11 +146,6 @@ OPC works standalone — but pair it with [memex](https://github.com/iamtouchsky
 npm install -g @touchskyer/memex
 ```
 
-## Requirements
-
-- [Claude Code](https://claude.ai/code) (CLI, desktop app, or IDE extension)
-- That's it. No dependencies, no build step, no MCP server. Just markdown files.
-
 ## License
 
 MIT

package/bin/opc.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import { existsSync, mkdirSync, cpSync, rmSync, readFileSync } from "fs";
+import { existsSync, mkdirSync, cpSync, rmSync, readdirSync, readFileSync, lstatSync, readlinkSync, realpathSync } from "fs";
 import { join, dirname } from "path";
 import { homedir } from "os";
 import { fileURLToPath } from "url";
@@ -9,15 +9,27 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
 const SKILL_NAME = "opc";
 const skillsDir = join(homedir(), ".claude", "skills", SKILL_NAME);
 const srcDir = join(__dirname, "..");
-const entries = ["skill.md", "roles"];
+
+// Only these files/dirs are managed by OPC — custom roles are left alone
+const MANAGED_ENTRIES = ["skill.md", "roles"];
 
 const pkg = JSON.parse(readFileSync(join(srcDir, "package.json"), "utf8"));
 const command = process.argv[2];
 
 switch (command) {
   case "install": {
+    // If skillsDir is a symlink pointing to srcDir, it's already installed via symlink
+    if (existsSync(skillsDir) && lstatSync(skillsDir).isSymbolicLink()) {
+      const target = realpathSync(skillsDir);
+      const src = realpathSync(srcDir);
+      if (target === src) {
+        console.log(`✓ OPC v${pkg.version} already linked at ${skillsDir}`);
+        console.log(`  Use /opc in Claude Code to get started.`);
+        break;
+      }
+    }
     mkdirSync(skillsDir, { recursive: true });
-    for (const entry of entries) {
+    for (const entry of MANAGED_ENTRIES) {
       const src = join(srcDir, entry);
       if (!existsSync(src)) continue;
       cpSync(src, join(skillsDir, entry), { recursive: true, force: true });
@@ -28,12 +40,40 @@ switch (command) {
   }
 
   case "uninstall": {
-    if (existsSync(skillsDir)) {
-      rmSync(skillsDir, { recursive: true });
-      console.log(`✓ OPC removed from ${skillsDir}`);
-    } else {
+    if (!existsSync(skillsDir)) {
       console.log(`Nothing to remove — ${skillsDir} does not exist.`);
+      break;
+    }
+
+    // Only remove OPC-managed files, preserve custom roles
+    const rolesDir = join(skillsDir, "roles");
+    const managedRoles = readdirSync(join(srcDir, "roles"));
+
+    // Remove managed role files
+    if (existsSync(rolesDir)) {
+      for (const role of managedRoles) {
+        const rolePath = join(rolesDir, role);
+        if (existsSync(rolePath)) rmSync(rolePath);
+      }
+      // Remove roles dir only if empty
+      try {
+        const remaining = readdirSync(rolesDir);
+        if (remaining.length === 0) rmSync(rolesDir);
+        else console.log(`  Kept ${remaining.length} custom role(s) in ${rolesDir}`);
+      } catch {}
     }
+
+    // Remove skill.md
+    const skillFile = join(skillsDir, "skill.md");
+    if (existsSync(skillFile)) rmSync(skillFile);
+
+    // Remove dir only if empty
+    try {
+      const remaining = readdirSync(skillsDir);
+      if (remaining.length === 0) rmSync(skillsDir);
+    } catch {}
+
+    console.log(`✓ OPC removed from ${skillsDir}`);
     break;
   }
 
@@ -49,7 +89,7 @@ switch (command) {
     console.log();
     console.log("Usage:");
     console.log("  opc install     Install skill files to ~/.claude/skills/opc/");
-    console.log("  opc uninstall   Remove skill files");
+    console.log("  opc uninstall   Remove skill files (preserves custom roles)");
     console.log("  opc version     Show version");
     console.log();
     console.log("Once installed, use /opc in Claude Code.");

package/package.json

@@ -1,16 +1,21 @@
 {
   "name": "@touchskyer/opc",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "OPC — One Person Company. A full team in a single Claude Code skill.",
   "type": "module",
   "bin": {
     "opc": "bin/opc.mjs"
   },
+  "engines": {
+    "node": ">=18"
+  },
   "files": [
     "bin",
     "scripts",
     "skill.md",
-    "roles"
+    "roles",
+    "README.md",
+    "LICENSE"
   ],
   "scripts": {
     "postinstall": "node scripts/postinstall.mjs"

package/scripts/postinstall.mjs

@@ -1,31 +1,17 @@
 #!/usr/bin/env node
 
-import { existsSync, mkdirSync, cpSync, readFileSync } from "fs";
-import { join } from "path";
-import { homedir } from "os";
+// Delegate to bin/opc.mjs install
+import { execFileSync } from "child_process";
+import { join, dirname } from "path";
+import { fileURLToPath } from "url";
 
-const SKILL_NAME = "opc";
-const skillsDir = join(homedir(), ".claude", "skills", SKILL_NAME);
-const srcDir = join(import.meta.dirname, "..");
-
-// Files to copy
-const entries = ["skill.md", "roles"];
+const __dirname = dirname(fileURLToPath(import.meta.url));
 
 try {
-  mkdirSync(skillsDir, { recursive: true });
-
-  for (const entry of entries) {
-    const src = join(srcDir, entry);
-    if (!existsSync(src)) continue;
-    cpSync(src, join(skillsDir, entry), { recursive: true, force: true });
-  }
-
-  const pkg = JSON.parse(readFileSync(join(srcDir, "package.json"), "utf8"));
-  console.log(`✓ OPC v${pkg.version} installed to ${skillsDir}`);
-  console.log(`  Use /opc in Claude Code to get started.`);
+  execFileSync(process.execPath, [join(__dirname, "..", "bin", "opc.mjs"), "install"], {
+    stdio: "inherit",
+  });
 } catch (err) {
-  // Don't fail the install if copy fails (e.g., permissions)
-  console.warn(`⚠ Could not install OPC skill files to ${skillsDir}`);
-  console.warn(`  Run 'opc install' manually to retry.`);
+  console.warn(`⚠ Postinstall failed. Run 'opc install' manually.`);
   console.warn(`  Error: ${err.message}`);
 }

package/skill.md

@@ -1,6 +1,6 @@
 ---
 name: opc
-version: 0.2.0
+version: 0.2.2
 description: "OPC — One Person Company. A full team in a single skill: 11 specialist agents for review, analysis, execution, and brainstorming. /opc review, /opc -i, or /opc <role>."
 ---
 
@@ -143,6 +143,8 @@ For Modes B/C/D, light context gathering (read key files) is sufficient — no f
 
 ## Step 5: Dispatch Agents
 
+**Agent prompt = mode template + role file only.** Do NOT inject full skill.md into agents. Each agent gets: its mode-specific instruction block, the role's expertise + anti-patterns, project context, and task scope. The coordinator reads skill.md; agents do not.
+
 Launch agents in parallel. Each agent prompt is assembled from:
 
 ```
@@ -311,12 +313,11 @@ For findings that pass mechanical checks:
 2. **Challenge severity** — "Is this really 🔴? What's the concrete exploit path?"
 3. **Deduplicate** — multiple agents reporting same issue → keep best-articulated one
 
-### Effort Check (Mode A/B)
+### Effort Check
 
-- Count files agent referenced vs files in assigned scope
-- If scope ≤ 5 files: coverage must be 100%. If scope > 5 files: coverage ≥ 50%.
-- If output is suspiciously thin for scope complexity → RE-DISPATCH with explicit file list
-- Mode C: verify all files in scope were addressed. Mode D: check that key constraints were considered.
+- **Mode A/B:** Count files agent referenced vs files in assigned scope. If scope ≤ 5 files: coverage must be 100%. If scope > 5 files: coverage ≥ 50%. If suspiciously thin → RE-DISPATCH with explicit file list.
+- **Mode C:** Verify all files in scope were addressed in the modification list.
+- **Mode D:** Check that key constraints from the prompt were considered. No file-coverage metric.
 
 ### Coordinator Actions
 
@@ -389,7 +390,6 @@ Recommendation: {coordinator's pick with rationale}
 ## Notes
 
 - Agents run via the Agent tool with `subagent_type: "general-purpose"`.
-- **Agent prompt = mode template + role file only.** Do NOT inject full skill.md into agents. Each agent gets: its mode-specific instruction block, the role's expertise + anti-patterns, project context, and task scope. The coordinator reads skill.md; agents do not.
 - Agents are READ-ONLY by default. No code changes except in Mode C.
 - Scope each agent to specific files — don't let them scan everything.
 - If scope exceeds 20 files, split across multiple agents of the same role.