oroute-cli 0.6.0 → 0.6.1

package/dist/oroute.cjs

@@ -241941,7 +241941,9 @@ var ALL_COMMANDS = [
   // Code analysis (Features 7, 8, 10)
   "/test-gen",
   "/dead-code",
-  "/deps"
+  "/deps",
+  // Intelligence features (#25, #30)
+  "/review-score"
 ];
 var COMMAND_DESCRIPTIONS = {
   "/help": "Show all available commands",
@@ -242036,7 +242038,8 @@ var COMMAND_DESCRIPTIONS = {
   "/find": "Semantic code search using AI",
   "/test-gen": "Generate tests for current file context",
   "/dead-code": "Detect unused imports/exports",
-  "/deps": "Show dependency graph for a file"
+  "/deps": "Show dependency graph for a file",
+  "/review-score": "Risk score for latest commit changes"
 };
 var KNOWN_COMMANDS = new Set(ALL_COMMANDS);
 function shell(cmd, cwd) {
@@ -242558,6 +242561,9 @@ function handleSlashCommand(input, ctx) {
     case "/deps":
       handleDeps(arg, ctx);
       return "handled";
+    case "/review-score":
+      handleReviewScore(ctx);
+      return "handled";
     // -----------------------------------------------------------------------
     // Misc
     // -----------------------------------------------------------------------
@@ -243088,6 +243094,7 @@ function printHelp() {
   console.log(`${GREEN}  /verify${RESET}                 Verify changes (build + typecheck)`);
   console.log(`${GREEN}  /tdd${RESET}                    Test-driven development mode`);
   console.log(`${GREEN}  /e2e${RESET}                    E2E testing`);
+  console.log(`${GREEN}  /review-score${RESET}           Risk score for latest commit changes`);
   console.log();
   console.log(`${BOLD}  File / Code${RESET}`);
   console.log(`${GRAY}  ${"\u2500".repeat(50)}${RESET}`);
@@ -243339,6 +243346,20 @@ function handleCommit(arg, ctx) {
       return;
     }
   }
+  const reviewPrompt = buildCommitReviewPrompt(ctx.cwd);
+  if (reviewPrompt) {
+    console.log(`${CYAN}  Auto-reviewing staged changes...${RESET}`);
+    ctx.messages.push({ role: "user", content: reviewPrompt });
+    ctx.setMessages(ctx.messages);
+    console.log(`${GRAY}  Review prompt injected. AI will review your changes in the next turn.${RESET}`);
+    console.log(`${GRAY}  After review, run /commit again to proceed with the commit.${RESET}`);
+    const lastMsg = ctx.messages[ctx.messages.length - 2];
+    const isReEntry = lastMsg && lastMsg.role === "assistant" && typeof lastMsg.content === "string" && (lastMsg.content.includes("LGTM") || lastMsg.content.includes("review"));
+    if (!isReEntry) {
+      return;
+    }
+    console.log(`${GREEN}  Review complete. Proceeding with commit...${RESET}`);
+  }
   if (arg) {
     const output2 = shell(`git commit -m "${arg.replace(/"/g, '\\"')}"`, ctx.cwd);
     console.log(output2);
@@ -243514,6 +243535,77 @@ function handleDeps(arg, ctx) {
   }
   console.log();
 }
+function handleReviewScore(ctx) {
+  console.log();
+  console.log(`${BOLD}  Code Review Risk Score${RESET}`);
+  console.log(`${GRAY}  ${"\u2500".repeat(40)}${RESET}`);
+  const diffStat = shell("git diff HEAD~1 --stat", ctx.cwd);
+  if (!diffStat || diffStat.startsWith("Command failed")) {
+    console.log(`${YELLOW}  No previous commit to compare against.${RESET}`);
+    return;
+  }
+  const diffContent = shell("git diff HEAD~1", ctx.cwd);
+  const fileLines = diffStat.split("\n").filter((l) => l.includes("|"));
+  const filesChanged = fileLines.length;
+  const summaryLine = diffStat.split("\n").pop() ?? "";
+  const insertMatch = summaryLine.match(/(\d+)\s+insertion/);
+  const deleteMatch = summaryLine.match(/(\d+)\s+deletion/);
+  const insertions = insertMatch ? parseInt(insertMatch[1], 10) : 0;
+  const deletions = deleteMatch ? parseInt(deleteMatch[1], 10) : 0;
+  const totalLines = insertions + deletions;
+  const newFunctions = (diffContent.match(/^\+.*(?:function\s+\w+|(?:const|let)\s+\w+\s*=\s*(?:async\s+)?(?:\([^)]*\)|[^=])\s*=>)/gm) ?? []).length;
+  const deletedFunctions = (diffContent.match(/^-.*(?:function\s+\w+|(?:const|let)\s+\w+\s*=\s*(?:async\s+)?(?:\([^)]*\)|[^=])\s*=>)/gm) ?? []).length;
+  const modifiedImports = (diffContent.match(/^[+-].*import\s+/gm) ?? []).length;
+  let riskLevel;
+  let riskColor;
+  if (filesChanged <= 3 && totalLines < 50) {
+    riskLevel = "LOW";
+    riskColor = GREEN;
+  } else if (filesChanged <= 10 && totalLines <= 200) {
+    riskLevel = "MEDIUM";
+    riskColor = YELLOW;
+  } else {
+    riskLevel = "HIGH";
+    riskColor = RED;
+  }
+  console.log(`${riskColor}  Risk: ${riskLevel}${RESET}`);
+  console.log();
+  console.log(`${WHITE}  Files changed:      ${filesChanged}${RESET}`);
+  console.log(`${WHITE}  Lines changed:      ${totalLines} (+${insertions} -${deletions})${RESET}`);
+  console.log(`${WHITE}  New functions:      ${newFunctions}${RESET}`);
+  console.log(`${WHITE}  Deleted functions:  ${deletedFunctions}${RESET}`);
+  console.log(`${WHITE}  Modified imports:   ${modifiedImports}${RESET}`);
+  console.log();
+  if (fileLines.length > 0) {
+    console.log(`${GRAY}  Changed files:${RESET}`);
+    for (const line of fileLines.slice(0, 15)) {
+      console.log(`${GRAY}    ${line.trim()}${RESET}`);
+    }
+    if (fileLines.length > 15) {
+      console.log(`${GRAY}    ... and ${fileLines.length - 15} more${RESET}`);
+    }
+  }
+  console.log();
+}
+function buildCommitReviewPrompt(cwd) {
+  const diff = shell("git diff --cached", cwd);
+  if (!diff || diff.startsWith("Command failed")) return "";
+  const truncated = diff.length > 3e3 ? diff.slice(0, 3e3) + "\n... (truncated)" : diff;
+  return [
+    "Review these staged changes for bugs, security issues, and style problems before I commit:",
+    "",
+    "```diff",
+    truncated,
+    "```",
+    "",
+    "Provide a brief review (3-5 bullet points max). Flag any:",
+    "- Potential bugs or logic errors",
+    "- Security concerns (exposed secrets, injection risks)",
+    "- Style issues (unused imports, inconsistent naming)",
+    "- Missing error handling",
+    'If everything looks good, say "LGTM" with a one-line summary.'
+  ].join("\n");
+}
 
 // src/hooks.ts
 var fs16 = __toESM(require("node:fs"), 1);
@@ -243598,6 +243690,65 @@ function substituteEnvVars(command, env) {
 
 // src/agent.ts
 init_mcpLoader();
+
+// src/errorMapper.ts
+var ERROR_PATTERNS = [
+  // TypeScript: src/foo.ts(10,5): error TS2345
+  /(?<file>[^\s()]+\.(?:ts|tsx|js|jsx|mjs|cjs))\((?<line>\d+),(?<col>\d+)\):\s*error\s+TS/g,
+  // Node.js stack trace: at /path/to/file.ts:42:15
+  /at\s+(?:.*?\s+\()?(?<file>[^\s()]+\.(?:ts|tsx|js|jsx|mjs|cjs|py|rb|go|rs|java|kt)):(?<line>\d+):(?<col>\d+)\)?/g,
+  // ESLint / many linters: /path/to/file.ts:42:15
+  /^(?<file>(?:\/|\.\.?\/|[A-Z]:\\)[^\s:]+\.(?:ts|tsx|js|jsx|mjs|cjs)):(?<line>\d+):(?<col>\d+)/gm,
+  // Python traceback: File "/path/to/file.py", line 42
+  /File\s+"(?<file>[^"]+)",\s+line\s+(?<line>\d+)/g,
+  // Rust/Go: src/main.rs:42:15
+  /(?<file>[^\s:]+\.(?:rs|go)):(?<line>\d+):(?<col>\d+)/g,
+  // Generic: Error in src/auth.ts (line 42)
+  /(?:Error|error|ERROR)\s+in\s+(?<file>[^\s(]+)\s+\(line\s+(?<line>\d+)\)/g,
+  // Webpack/Vite: ERROR in ./src/App.tsx 42:15
+  /ERROR\s+in\s+\.?(?<file>[^\s]+\.(?:ts|tsx|js|jsx))\s+(?<line>\d+):(?<col>\d+)/g,
+  // Jest: at Object.<anonymous> (src/test.ts:42:15)
+  /\((?<file>[^\s()]+\.(?:ts|tsx|js|jsx|test\.[tj]s)):(?<line>\d+):(?<col>\d+)\)/g,
+  // Simple line reference: file.ts:42
+  /(?<file>[^\s:]+\.(?:ts|tsx|js|jsx|mjs|cjs)):(?<line>\d+)(?::(?<col>\d+))?/g
+];
+function extractFileReferences(errorText) {
+  const seen = /* @__PURE__ */ new Set();
+  const results = [];
+  for (const pattern of ERROR_PATTERNS) {
+    pattern.lastIndex = 0;
+    let match;
+    while ((match = pattern.exec(errorText)) !== null) {
+      const groups = match.groups;
+      if (!groups) continue;
+      const file = groups["file"] ?? "";
+      const lineStr = groups["line"] ?? "";
+      const colStr = groups["col"];
+      if (!file || !lineStr) continue;
+      const line = parseInt(lineStr, 10);
+      if (isNaN(line) || line <= 0) continue;
+      const key = `${file}:${line}`;
+      if (seen.has(key)) continue;
+      seen.add(key);
+      const ref = colStr ? { file, line, column: parseInt(colStr, 10) } : { file, line };
+      results.push(ref);
+    }
+  }
+  return results;
+}
+function buildErrorContextHint(refs) {
+  if (refs.length === 0) return "";
+  const lines = refs.slice(0, 10).map(
+    (r) => `  - ${r.file}:${r.line}${r.column ? `:${r.column}` : ""}`
+  );
+  return [
+    "\n[Auto-detected error locations \u2014 read these files to understand and fix the errors]:",
+    ...lines,
+    refs.length > 10 ? `  ... and ${refs.length - 10} more` : ""
+  ].filter(Boolean).join("\n");
+}
+
+// src/agent.ts
 var keepAliveHttpAgent = new http.Agent({ keepAlive: true });
 var keepAliveHttpsAgent = new https.Agent({ keepAlive: true });
 var TOOL_DEFINITIONS = [
@@ -244085,7 +244236,30 @@ When you finish a task, mentally verify:
 2. Are there any files I modified that need updated imports?
 3. Would this change break anything in related files?
 4. Should I suggest running tests or type checking?
-5. Is there a logical next step the user might want?`;
+5. Is there a logical next step the user might want?
+
+# Import Auto-Cleanup (#27)
+After editing TypeScript files, check for unused imports at the top of the file and remove them.
+You can detect unused imports by checking if the imported name appears elsewhere in the file.
+- For named imports like \`import { Foo, Bar } from './mod'\`, check if Foo and Bar each appear at least once outside the import line.
+- If an import name is only used in the import statement itself, remove it.
+- If all names from an import are unused, remove the entire import line.` + loadProjectPromptOverride(cwd);
+}
+function loadProjectPromptOverride(cwd) {
+  const promptPath = path16.join(cwd, ".oroute", "prompt.md");
+  try {
+    if (fs17.existsSync(promptPath)) {
+      const content = fs17.readFileSync(promptPath, "utf-8").trim();
+      if (content) {
+        return `
+
+# Project-Specific Instructions (from .oroute/prompt.md)
+${content}`;
+      }
+    }
+  } catch {
+  }
+  return "";
 }
 function truncateResult(result) {
   if (result.length <= MAX_TOOL_RESULT_CHARS) return result;
@@ -244219,7 +244393,16 @@ async function executeToolInner(tool, cwd, confirmWrite, confirmExec, config) {
       if (result.stderr) console.log(`${RED}${result.stderr}${RESET}`);
       if (result.exitCode === 0) printSuccess("Command completed");
       else console.log(`${YELLOW}  Exit code: ${result.exitCode}${RESET}`);
-      return truncateResult(JSON.stringify(result));
+      const resultJson = JSON.stringify(result);
+      if (result.exitCode !== 0) {
+        const errorText = (result.stderr ?? "") + "\n" + (result.stdout ?? "");
+        const refs = extractFileReferences(errorText);
+        if (refs.length > 0) {
+          const hint = buildErrorContextHint(refs);
+          return truncateResult(resultJson + hint);
+        }
+      }
+      return truncateResult(resultJson);
     }
     case "edit_file": {
       const input = tool.input;
@@ -244638,7 +244821,7 @@ ${mcpSummary}
     turnRollbackBuffer.clear();
     let continueLoop = true;
     let retryCount = 0;
-    const maxRetries = 3;
+    const maxRetries = 5;
     let toolIterations = 0;
     const MAX_TOOL_ITERATIONS = 20;
     const MAX_TOOL_HARD_CAP = 25;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oroute-cli",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "O'Route CLI — AI API auto-routing for 13 models from your terminal",
   "type": "module",
   "bin": {