opencode-code-simplifier 1.3.0 → 1.5.0

package/README.md

@@ -4,16 +4,7 @@ Code simplifier plugin for [OpenCode](https://opencode.ai). Simplifies and refin
 
 ## Install
 
-```bash
-npm install opencode-code-simplifier --save-dev
-```
-
-This will automatically:
-1. Copy `skills/code-simplifier/SKILL.md` to `.opencode/skills/`
-2. Copy `rules/code-simplifier.md` to `.opencode/rules/`
-3. Copy `commands/simplify.md` to `.opencode/commands/`
-
-Then add to your `opencode.json`:
+Add to your `opencode.json`:
 
 ```json
 {
@@ -21,31 +12,31 @@ Then add to your `opencode.json`:
 }
 ```
 
-Restart OpenCode to activate.
-
-## Features
-
-- Reduces nesting and complexity
-- Removes debug artifacts (unconditional console.log, debugger, commented code)
-- Removes obvious comments
-- Preserves conditional logging (if debug, if isDev, etc.)
-- Converts nested ternaries to if/else chains
-- Applies language-specific best practices
+Restart OpenCode. The plugin will auto-create `skills/code-simplifier/SKILL.md` and `rules/code-simplifier.md` on first load.
 
 ## Usage
 
-### Command
+Ask OpenCode to simplify code using the tool:
 
 ```
-/simplify
+Use simplify-code tool on path/to/file.js
 ```
 
-### Manual Tool
+Or just describe what you want:
 
 ```
-simplify-code file=path/to/file.js
+Simplify the code in src/utils.js
 ```
 
+## Features
+
+- Reduces nesting and complexity
+- Removes debug artifacts (unconditional console.log, debugger, commented code)
+- Removes obvious comments
+- Preserves conditional logging (if debug, if isDev, etc.)
+- Converts nested ternaries to if/else chains
+- Applies language-specific best practices
+
 ## What Gets Removed
 
 | Type | Action |

package/dist/plugin.js

@@ -12334,8 +12334,8 @@ function tool(input) {
 }
 tool.schema = exports_external;
 // plugin.ts
-import { existsSync, mkdirSync, writeFileSync } from "fs";
-import { join } from "path";
+import { existsSync, mkdirSync, writeFileSync, readFileSync } from "fs";
+import { join, extname } from "path";
 import { homedir } from "os";
 var SKILL_CONTENT = `---
 name: code-simplifier
@@ -12399,7 +12399,7 @@ Simplify code for clarity while preserving exact functionality.
 
 ## Core Rules
 
-1. **Preserve Functionality** - Never change what code does, only how.
+1. **Preserve Functionality** - Never change what it does, only how.
 
 2. **Enhance Clarity**:
    - Reduce nesting and complexity
@@ -12439,31 +12439,113 @@ Simplify code for clarity while preserving exact functionality.
 Skip when:
 - User wants exact structure preserved
 - Code is already clean`;
-var COMMAND_CONTENT = `---
-description: Simplify code for clarity while preserving functionality
-agent: code
----
-
-Use code-simplifier skill to simplify recently modified code.
-
-Focus on:
-1. Reducing complexity and nesting
-2. Eliminating nested ternaries
-3. Improving readability with clear names
-4. Extracting reusable functions
-5. Removing debug artifacts (unconditional console.log, debugger, commented code)
-6. Removing obvious comments
-7. Preserving all functionality
-
-If no file specified, focus on recently modified code.
-
-User request: $ARGUMENTS`;
+var CODE_EXTENSIONS = new Set([
+  ".js",
+  ".jsx",
+  ".ts",
+  ".tsx",
+  ".mjs",
+  ".cjs",
+  ".py",
+  ".go",
+  ".rs",
+  ".java",
+  ".rb",
+  ".c",
+  ".cpp",
+  ".h",
+  ".hpp",
+  ".cs",
+  ".php",
+  ".swift",
+  ".kt",
+  ".lua",
+  ".zig"
+]);
+var SKIP_PATTERNS = [
+  "node_modules",
+  ".git",
+  "dist",
+  "build",
+  ".next",
+  "vendor",
+  "__pycache__",
+  ".cache",
+  "target"
+];
+var NESTING_THRESHOLD = 3;
+var REPEATED_THRESHOLD = 3;
+function isCodeFile(filePath) {
+  const ext = extname(filePath).toLowerCase();
+  if (!CODE_EXTENSIONS.has(ext))
+    return false;
+  return !SKIP_PATTERNS.some((p) => filePath.includes(`/${p}/`) || filePath.includes(`\\${p}\\`));
+}
+function getIndentLevel(line) {
+  let spaces = 0;
+  for (const ch of line) {
+    if (ch === " ")
+      spaces++;
+    else if (ch === "\t")
+      spaces += 4;
+    else
+      break;
+  }
+  return Math.floor(spaces / 2);
+}
+function checkNestingDepth(code) {
+  let maxDepth = 0;
+  for (const line of code.split(`
+`)) {
+    if (line.trim() === "")
+      continue;
+    const depth = getIndentLevel(line);
+    if (depth > maxDepth)
+      maxDepth = depth;
+  }
+  if (maxDepth >= NESTING_THRESHOLD) {
+    return {
+      rule: "nesting-depth",
+      detail: `Max nesting depth: ${maxDepth} (threshold: ${NESTING_THRESHOLD})`
+    };
+  }
+  return null;
+}
+function checkRepeatedExpressions(code) {
+  const issues = [];
+  const calls = new Map;
+  const pattern = /(\w+(?:\.\w+)*(?:\[.*?\])*(?:\(.*?\))*)/g;
+  let match;
+  while ((match = pattern.exec(code)) !== null) {
+    const expr = match[1];
+    if (expr.length < 5)
+      continue;
+    const base = expr.replace(/\[.*?\]/g, "[]");
+    calls.set(base, (calls.get(base) ?? 0) + 1);
+  }
+  for (const [expr, count] of calls) {
+    if (count >= REPEATED_THRESHOLD) {
+      issues.push({
+        rule: "repeated-expression",
+        detail: `Repeated expression "${expr}" appears ${count} times`
+      });
+    }
+  }
+  return issues;
+}
+function analyzeCode(code) {
+  const issues = [];
+  const nesting = checkNestingDepth(code);
+  if (nesting)
+    issues.push(nesting);
+  issues.push(...checkRepeatedExpressions(code));
+  return issues;
+}
 function ensureFilesExist() {
   const configDir = join(homedir(), ".config", "opencode");
   const files = [
     { path: "skills/code-simplifier/SKILL.md", content: SKILL_CONTENT },
-    { path: "rules/code-simplifier.md", content: RULES_CONTENT },
-    { path: "commands/simplify.md", content: COMMAND_CONTENT }
+    { path: "rules/code-simplifier.md", content: RULES_CONTENT }
   ];
   for (const file2 of files) {
     const fullPath = join(configDir, file2.path);
@@ -12474,40 +12556,40 @@ function ensureFilesExist() {
     }
   }
 }
+function readFileSafe(filePath) {
+  try {
+    return readFileSync(filePath, "utf-8");
+  } catch {
+    return null;
+  }
+}
 var CodeSimplifier = async ({ client }) => {
   ensureFilesExist();
-  await client.app.log({
-    body: {
-      service: "code-simplifier",
-      level: "info",
-      message: "Code Simplifier loaded. Use /simplify or the simplify-code tool."
-    }
-  });
   return {
-    event: async ({ event }) => {
-      if (event.type === "session.created") {
-        await client.app.log({
-          body: {
-            service: "code-simplifier",
-            level: "info",
-            message: "Code Simplifier active for this session."
-          }
-        });
-      }
-    },
-    "tool.execute.after": async (input, output) => {
-      if (input.tool === "write" || input.tool === "edit") {
-        const filePath = input?.args?.filePath || input?.args?.file_path;
-        if (filePath) {
-          await client.app.log({
-            body: {
-              service: "code-simplifier",
-              level: "debug",
-              message: `Code modified: ${filePath}`
-            }
-          });
+    async "tool.execute.after"(input, output) {
+      if (input.tool !== "write" && input.tool !== "edit")
+        return;
+      const filePath = input.args?.file ?? input.args?.filePath ?? input.args?.path;
+      if (!filePath || !isCodeFile(filePath))
+        return;
+      const code = readFileSafe(filePath);
+      if (!code)
+        return;
+      const issues = analyzeCode(code);
+      if (issues.length === 0)
+        return;
+      const fileName = filePath.split("/").pop() ?? filePath;
+      const lines = issues.map((i) => `  - ${i.detail}`).join(`
+`);
+      await client.event.emit({
+        type: "tui.toast.show",
+        properties: {
+          message: `${fileName} has potential issues:
+${lines}
+Use "simplify-code" to clean up.`,
+          variant: "warning"
         }
-      }
+      });
     },
     tool: {
       "simplify-code": tool({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-code-simplifier",
-  "version": "1.3.0",
+  "version": "1.5.0",
   "description": "Code simplifier plugin for OpenCode - simplifies and refines code for clarity while preserving functionality",
   "main": "dist/plugin.js",
   "types": "dist/plugin.d.ts",