@triedotdev/mcp 1.0.7 → 1.0.8

package/dist/index.js

@@ -5,12 +5,14 @@ import {
   parseDocument
 } from "./chunk-EYNAGEQK.js";
 import {
+  CRITICAL_REVIEW_CHECKLIST,
+  SuperReviewerAgent,
   TrieFixTool,
   TrieScanTool,
   getAgentRegistry,
   getPrompt,
   getSystemPrompt
-} from "./chunk-EDWYG3KK.js";
+} from "./chunk-Z3GM6UVW.js";
 import "./chunk-3CS6Z2SL.js";
 import "./chunk-MR755QGT.js";
 import "./chunk-6NLHFIYA.js";
@@ -25,8 +27,8 @@ import {
   ListResourcesRequestSchema,
   ReadResourceRequestSchema
 } from "@modelcontextprotocol/sdk/types.js";
-import { readdir as readdir2, readFile as readFile6 } from "fs/promises";
-import { join as join6 } from "path";
+import { readdir as readdir2, readFile as readFile7 } from "fs/promises";
+import { join as join7 } from "path";
 
 // src/tools/explain.ts
 import { readFile } from "fs/promises";
@@ -2066,7 +2068,8 @@ var AGENT_TO_AI_TYPE = {
   "security": "security",
   "privacy": "privacy",
   "legal": "legal",
-  "design-engineer": "accessibility",
+  "accessibility": "accessibility",
+  "design-engineer": "design-engineer",
   "software-architect": "architecture",
   "bug-finding": "bugs",
   "user-testing": "ux",
@@ -2074,7 +2077,8 @@ var AGENT_TO_AI_TYPE = {
   "devops": "devops",
   "comprehension": "explain",
   "test": "test",
-  "trie_clean": "vibe"
+  "trie_clean": "vibe",
+  "super-reviewer": "pr_review"
 };
 var TrieAgentTool = class {
   agentRegistry = getAgentRegistry();
@@ -2300,7 +2304,8 @@ trie_scan                        # Full scan with smart triaging
       "security": "trie_security",
       "privacy": "trie_privacy",
       "legal": "trie_legal",
-      "design-engineer": "trie_accessibility",
+      "accessibility": "trie_accessibility",
+      "design-engineer": "trie_design",
       "software-architect": "trie_architecture",
       "bug-finding": "trie_bugs",
       "user-testing": "trie_ux",
@@ -2422,7 +2427,8 @@ ${issue.fix}
       "security": "\u{1F512}",
       "privacy": "\u{1F464}",
       "legal": "\u2696\uFE0F",
-      "design-engineer": "\u267F",
+      "accessibility": "\u267F",
+      "design-engineer": "\u{1F3A8}",
       "software-architect": "\u{1F3D7}\uFE0F",
       "bug-finding": "\u{1F41B}",
       "user-testing": "\u{1F3AF}",
@@ -2935,7 +2941,7 @@ var TrieListAgentsTool = class {
         { name: "legal", displayName: "Legal Agent", category: "compliance", isCustom: false },
         { name: "typecheck", displayName: "TypeCheck Agent", category: "quality", isCustom: false },
         { name: "comprehension", displayName: "Comprehension Agent", category: "communication", isCustom: false },
-        { name: "design-engineer", displayName: "Design Engineer Agent", category: "accessibility", isCustom: false },
+        { name: "accessibility", displayName: "Accessibility Agent", category: "accessibility", isCustom: false },
         { name: "test", displayName: "Test Agent", category: "testing", isCustom: false },
         { name: "software-architect", displayName: "Software Architect Agent", category: "architecture", isCustom: false },
         { name: "devops", displayName: "DevOps Agent", category: "devops", isCustom: false },
@@ -3007,6 +3013,402 @@ var TrieListAgentsTool = class {
   }
 };
 
+// src/tools/pr-review.ts
+import { readFile as readFile5 } from "fs/promises";
+import { existsSync as existsSync6 } from "fs";
+import { join as join5, basename as basename6, resolve as resolve4, isAbsolute as isAbsolute4 } from "path";
+import { execSync } from "child_process";
+var TriePRReviewTool = class {
+  agent = new SuperReviewerAgent();
+  async execute(args) {
+    const {
+      pr,
+      worktree,
+      mode,
+      files: specificFiles
+    } = args;
+    try {
+      const prInfo = await this.getPRInfo(pr, worktree);
+      if (!prInfo.success) {
+        return {
+          content: [{
+            type: "text",
+            text: `\u274C ${prInfo.error}
+
+Usage:
+- \`pr_review 12345\` \u2014 review specific PR
+- \`pr_review\` \u2014 review PR for current branch
+- \`pr_review ../worktree\` \u2014 review worktree changes`
+          }]
+        };
+      }
+      const changes = await this.getChanges(prInfo);
+      if (changes.files.length === 0) {
+        return {
+          content: [{
+            type: "text",
+            text: `\u2139\uFE0F No changes found to review.`
+          }]
+        };
+      }
+      const designDocs = await this.findDesignDocs(changes.files, prInfo);
+      const workflow = await this.agent.buildReviewWorkflow(
+        changes.files,
+        {
+          prNumber: prInfo.number,
+          prTitle: prInfo.title,
+          prAuthor: prInfo.author,
+          baseBranch: prInfo.baseBranch,
+          headBranch: prInfo.headBranch,
+          mode: mode || "own",
+          designDocs
+        }
+      );
+      const fileContents = await this.preloadFiles(changes.files.map((f) => f.path));
+      const reviewPrompt = this.generateReviewPrompt(workflow, changes, fileContents);
+      return {
+        content: [{
+          type: "text",
+          text: reviewPrompt
+        }]
+      };
+    } catch (error) {
+      return {
+        content: [{
+          type: "text",
+          text: `\u274C Error: ${error instanceof Error ? error.message : String(error)}`
+        }]
+      };
+    }
+  }
+  /**
+   * Get PR information from GitHub or local worktree
+   */
+  async getPRInfo(pr, worktree) {
+    if (worktree) {
+      const worktreePath = isAbsolute4(worktree) ? worktree : resolve4(process.cwd(), worktree);
+      if (!existsSync6(worktreePath)) {
+        return { success: false, error: `Worktree not found: ${worktreePath}` };
+      }
+      return {
+        success: true,
+        type: "worktree",
+        path: worktreePath,
+        title: `Local changes in ${basename6(worktreePath)}`,
+        author: this.getGitUser(),
+        baseBranch: "HEAD~1",
+        headBranch: "HEAD"
+      };
+    }
+    if (pr) {
+      return this.getPRFromGitHub(pr);
+    }
+    try {
+      const branch = execSync("git branch --show-current", { encoding: "utf-8" }).trim();
+      const prJson = execSync(`gh pr view --json number,title,author,headRefName,baseRefName`, {
+        encoding: "utf-8",
+        stdio: ["pipe", "pipe", "pipe"]
+      }).trim();
+      const prData = JSON.parse(prJson);
+      return {
+        success: true,
+        type: "github",
+        number: String(prData.number),
+        title: prData.title,
+        author: prData.author?.login || "unknown",
+        baseBranch: prData.baseRefName,
+        headBranch: prData.headRefName
+      };
+    } catch {
+      return {
+        success: true,
+        type: "local",
+        title: "Local uncommitted changes",
+        author: this.getGitUser(),
+        baseBranch: "HEAD",
+        headBranch: "working tree"
+      };
+    }
+  }
+  /**
+   * Get PR info from GitHub CLI
+   */
+  async getPRFromGitHub(prNumber) {
+    try {
+      const prJson = execSync(`gh pr view ${prNumber} --json number,title,author,headRefName,baseRefName`, {
+        encoding: "utf-8",
+        stdio: ["pipe", "pipe", "pipe"]
+      }).trim();
+      const prData = JSON.parse(prJson);
+      return {
+        success: true,
+        type: "github",
+        number: String(prData.number),
+        title: prData.title,
+        author: prData.author?.login || "unknown",
+        baseBranch: prData.baseRefName,
+        headBranch: prData.headRefName
+      };
+    } catch (error) {
+      return {
+        success: false,
+        error: `Failed to get PR #${prNumber}. Is \`gh\` CLI installed and authenticated?`
+      };
+    }
+  }
+  /**
+   * Get changes (diff) for the PR or local changes
+   */
+  async getChanges(prInfo) {
+    let diffOutput;
+    try {
+      if (prInfo.type === "github" && prInfo.number) {
+        diffOutput = execSync(`gh pr diff ${prInfo.number}`, {
+          encoding: "utf-8",
+          maxBuffer: 50 * 1024 * 1024
+          // 50MB buffer for large PRs
+        });
+      } else if (prInfo.type === "worktree" && prInfo.path) {
+        diffOutput = execSync(`git diff HEAD`, {
+          encoding: "utf-8",
+          cwd: prInfo.path,
+          maxBuffer: 50 * 1024 * 1024
+        });
+      } else {
+        diffOutput = execSync(`git diff HEAD`, {
+          encoding: "utf-8",
+          maxBuffer: 50 * 1024 * 1024
+        });
+        if (!diffOutput.trim()) {
+          diffOutput = execSync(`git diff --cached`, {
+            encoding: "utf-8",
+            maxBuffer: 50 * 1024 * 1024
+          });
+        }
+      }
+    } catch {
+      diffOutput = "";
+    }
+    const files = this.parseDiff(diffOutput);
+    return { files, fullDiff: diffOutput };
+  }
+  /**
+   * Parse git diff output into structured file changes
+   */
+  parseDiff(diffOutput) {
+    const files = [];
+    const fileDiffs = diffOutput.split(/^diff --git /m).slice(1);
+    for (const fileDiff of fileDiffs) {
+      const lines = fileDiff.split("\n");
+      const headerLine = lines[0] || "";
+      const pathMatch = headerLine.match(/a\/(.+?) b\/(.+)/);
+      if (!pathMatch) continue;
+      const path = pathMatch[2];
+      const diff = `diff --git ${fileDiff}`;
+      let additions = 0;
+      let deletions = 0;
+      for (const line of lines) {
+        if (line.startsWith("+") && !line.startsWith("+++")) {
+          additions++;
+        } else if (line.startsWith("-") && !line.startsWith("---")) {
+          deletions++;
+        }
+      }
+      files.push({ path, diff, additions, deletions });
+    }
+    return files;
+  }
+  /**
+   * Find related design docs
+   */
+  async findDesignDocs(files, prInfo) {
+    const designDocs = [];
+    const cwd = prInfo.path || process.cwd();
+    for (const file of files) {
+      if (file.path.includes("design_docs/") || file.path.includes("docs/design/") || file.path.includes("rfcs/")) {
+        designDocs.push(file.path);
+      }
+    }
+    const designDocPaths = [
+      "design_docs",
+      "docs/design",
+      "docs/rfcs",
+      "rfcs"
+    ];
+    for (const docPath of designDocPaths) {
+      const fullPath = join5(cwd, docPath);
+      if (existsSync6(fullPath)) {
+      }
+    }
+    return designDocs;
+  }
+  /**
+   * Pre-load all changed files for instant responses during review
+   */
+  async preloadFiles(filePaths) {
+    const contents = /* @__PURE__ */ new Map();
+    const cwd = process.cwd();
+    await Promise.all(filePaths.map(async (filePath) => {
+      try {
+        const fullPath = isAbsolute4(filePath) ? filePath : join5(cwd, filePath);
+        if (existsSync6(fullPath)) {
+          const content = await readFile5(fullPath, "utf-8");
+          contents.set(filePath, content);
+        }
+      } catch {
+      }
+    }));
+    return contents;
+  }
+  /**
+   * Get git user name
+   */
+  getGitUser() {
+    try {
+      return execSync("git config user.name", { encoding: "utf-8" }).trim();
+    } catch {
+      return "unknown";
+    }
+  }
+  /**
+   * Generate the full review prompt for Claude to execute
+   */
+  generateReviewPrompt(workflow, changes, fileContents) {
+    const { metadata, fileOrder, reviewInstructions } = workflow;
+    let prompt = `# \u{1F50D} Super Reviewer \u2014 Interactive PR Review
+
+`;
+    if (metadata.prNumber) {
+      prompt += `## PR #${metadata.prNumber}: ${metadata.prTitle}
+
+`;
+    } else {
+      prompt += `## ${metadata.prTitle}
+
+`;
+    }
+    prompt += `**Author:** ${metadata.prAuthor}
+`;
+    prompt += `**Branch:** \`${metadata.headBranch}\` \u2192 \`${metadata.baseBranch}\`
+`;
+    prompt += `**Scope:** ${metadata.totalFiles} files, +${metadata.totalAdditions}/-${metadata.totalDeletions} lines
+
+`;
+    prompt += `---
+
+`;
+    prompt += `## Review Mode
+
+`;
+    prompt += `**Current Mode:** ${reviewInstructions.mode === "own" ? "1" : "2"}. ${reviewInstructions.description}
+
+`;
+    prompt += `> To switch modes, say "mode 1" for your own PR or "mode 2" for someone else's
+
+`;
+    prompt += `---
+
+`;
+    prompt += `## Files to Review (ordered for understanding)
+
+`;
+    prompt += `| # | File | Why this order |
+`;
+    prompt += `|---|------|----------------|
+`;
+    for (const file of fileOrder) {
+      const stats = `+${changes.files.find((f) => f.path === file.path)?.additions || 0}/-${changes.files.find((f) => f.path === file.path)?.deletions || 0}`;
+      prompt += `| ${file.index} | \`${file.path}\` (${stats}) | ${file.reason} |
+`;
+    }
+    prompt += `
+---
+
+`;
+    prompt += `## \u{1F3AF} Critical Review Mindset
+
+`;
+    prompt += `As we go through each file, I'll actively look for:
+
+`;
+    prompt += `**State & Lifecycle:**
+`;
+    for (const check of CRITICAL_REVIEW_CHECKLIST.stateAndLifecycle) {
+      prompt += `- ${check}
+`;
+    }
+    prompt += `
+**Edge Cases & Races:**
+`;
+    for (const check of CRITICAL_REVIEW_CHECKLIST.edgeCasesAndRaces) {
+      prompt += `- ${check}
+`;
+    }
+    prompt += `
+**Missing Pieces:**
+`;
+    for (const check of CRITICAL_REVIEW_CHECKLIST.missingPieces) {
+      prompt += `- ${check}
+`;
+    }
+    prompt += `
+---
+
+`;
+    prompt += `## File Diffs (pre-loaded for instant review)
+
+`;
+    prompt += `<details>
+<summary>\u{1F4C1} All file diffs (click to expand)</summary>
+
+`;
+    for (const file of fileOrder) {
+      const fileChange = changes.files.find((f) => f.path === file.path);
+      if (fileChange) {
+        prompt += `### ${file.path}
+
+`;
+        prompt += `\`\`\`diff
+${fileChange.diff}
+\`\`\`
+
+`;
+      }
+    }
+    prompt += `</details>
+
+`;
+    prompt += `---
+
+`;
+    prompt += `## Ready to Begin
+
+`;
+    prompt += `I'll walk you through each file, explaining what changed and why. After each file, I'll pause for your questions.
+
+`;
+    prompt += `**Commands during review:**
+`;
+    prompt += `- \`yes\` or \`y\` \u2014 continue to next file
+`;
+    prompt += `- \`skip to [filename]\` \u2014 jump to specific file
+`;
+    prompt += `- \`questions?\` \u2014 ask about current file
+`;
+    prompt += `- \`reorder\` \u2014 change the file order
+`;
+    prompt += `- \`done\` \u2014 end review and show summary
+
+`;
+    prompt += `**Ready for File 1?** (\`${fileOrder[0]?.path || "no files"}\`)
+
+`;
+    prompt += `*(say "yes" to start, or ask me anything)*
+`;
+    return prompt;
+  }
+};
+
 // src/utils/ai-tool-detector.ts
 function detectAITool() {
   if (process.env.CLAUDE_CODE_VERSION || process.env.CLAUDE_CODE) {
@@ -3075,8 +3477,8 @@ function detectAITool() {
 }
 
 // src/config/loader.ts
-import { readFile as readFile5 } from "fs/promises";
-import { join as join5 } from "path";
+import { readFile as readFile6 } from "fs/promises";
+import { join as join6 } from "path";
 
 // src/config/defaults.ts
 var DEFAULT_CONFIG = {
@@ -3095,7 +3497,7 @@ var DEFAULT_CONFIG = {
       security: { enabled: true },
       privacy: { enabled: true },
       legal: { enabled: true },
-      "design-engineer": { enabled: true },
+      "accessibility": { enabled: true },
       "software-architect": { enabled: true },
       comprehension: { enabled: true },
       devops: { enabled: true },
@@ -3131,8 +3533,8 @@ var DEFAULT_CONFIG = {
 // src/config/loader.ts
 async function loadConfig() {
   try {
-    const configPath = join5(process.cwd(), ".trie", "config.json");
-    const configFile = await readFile5(configPath, "utf-8");
+    const configPath = join6(process.cwd(), ".trie", "config.json");
+    const configFile = await readFile6(configPath, "utf-8");
     const userConfig = JSON.parse(configFile);
     return mergeConfig(DEFAULT_CONFIG, userConfig);
   } catch (error) {
@@ -3179,6 +3581,7 @@ var agentTool = new TrieAgentTool();
 var createAgentTool = new TrieCreateAgentTool();
 var saveAgentTool = new TrieSaveAgentTool();
 var listAgentsTool = new TrieListAgentsTool();
+var prReviewTool = new TriePRReviewTool();
 var tools = [
   {
     name: "scan",
@@ -3422,6 +3825,22 @@ var tools = [
       }
     }
   },
+  {
+    name: "design",
+    description: "\u{1F3A8} Run design engineer agent: Awwwards-level polish, design systems, motion design, creative CSS",
+    inputSchema: {
+      type: "object",
+      properties: {
+        files: { type: "array", items: { type: "string" }, description: "Files to scan" },
+        directory: { type: "string", description: "Directory to scan" },
+        output: {
+          type: "string",
+          enum: ["summary", "full"],
+          description: "summary = concise (default), full = includes AI prompt/code (large output)"
+        }
+      }
+    }
+  },
   {
     name: "architecture",
     description: "Run architecture agent: code organization, SOLID principles, N+1 queries, scalability",
@@ -3625,6 +4044,34 @@ var tools = [
         }
       }
     }
+  },
+  // PR Review — Interactive AI-guided code review
+  {
+    name: "pr_review",
+    description: "\u{1F50D} Interactive PR review: walks through changes file-by-file, explains each chunk, pauses for cross-examination. Makes large PR reviews a delight.",
+    inputSchema: {
+      type: "object",
+      properties: {
+        pr: {
+          type: "string",
+          description: 'PR number to review (e.g., "12345"). If omitted, reviews current branch PR or local changes.'
+        },
+        worktree: {
+          type: "string",
+          description: "Path to worktree directory to review local changes"
+        },
+        mode: {
+          type: "string",
+          enum: ["own", "others"],
+          description: `Review mode: "own" = your PR (I explain, you verify), "others" = someone else's PR (I flag issues, you comment)`
+        },
+        files: {
+          type: "array",
+          items: { type: "string" },
+          description: "Specific files to review (optional, defaults to all changed files)"
+        }
+      }
+    }
   }
 ];
 server.setRequestHandler(ListToolsRequestSchema, async () => {
@@ -3653,6 +4100,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 "## Trie Menu",
                 "",
                 "- `trie_scan` or `scan`: Full intelligent scan (auto triage agents)",
+                "- `pr_review`: \u{1F50D} Interactive PR review \u2014 walks through changes file-by-file",
                 "- `fix`: Apply high-confidence fixes",
                 "- `explain`: Explain code/issues/risks",
                 "- `watch`: Watch mode (scan on change)",
@@ -3685,6 +4133,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case "legal":
         return await agentTool.execute({ ...args, agent: "legal" });
       case "accessibility":
+        return await agentTool.execute({ ...args, agent: "accessibility" });
+      case "design":
         return await agentTool.execute({ ...args, agent: "design-engineer" });
       case "architecture":
         return await agentTool.execute({ ...args, agent: "software-architect" });
@@ -3707,6 +4157,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         return await saveAgentTool.execute(args);
       case "list_agents":
         return await listAgentsTool.execute(args);
+      case "pr_review":
+        return await prReviewTool.execute(args);
       default:
         throw new Error(`Unknown tool: ${name}`);
     }
@@ -3749,7 +4201,7 @@ async function getAvailableResources() {
     mimeType: "application/json"
   });
   try {
-    const reportsDir = join6(process.cwd(), "trie-reports");
+    const reportsDir = join7(process.cwd(), "trie-reports");
     const files = await readdir2(reportsDir);
     const reportFiles = files.filter((f) => f.endsWith(".txt") || f.endsWith(".json"));
     for (const file of reportFiles.slice(0, 10)) {
@@ -3826,8 +4278,8 @@ async function readResourceContent(uri) {
   }
   if (parsedUri === "cache/stats") {
     try {
-      const cachePath = join6(process.cwd(), ".trie", ".trie-cache.json");
-      const cacheContent = await readFile6(cachePath, "utf-8");
+      const cachePath = join7(process.cwd(), ".trie", ".trie-cache.json");
+      const cacheContent = await readFile7(cachePath, "utf-8");
       const cache = JSON.parse(cacheContent);
       const fileCount = Object.keys(cache.files || {}).length;
       const totalVulns = Object.values(cache.files || {}).reduce((acc, file) => {
@@ -3879,9 +4331,9 @@ async function readResourceContent(uri) {
   }
   if (parsedUri.startsWith("reports/")) {
     const fileName = parsedUri.replace("reports/", "");
-    const reportPath = join6(process.cwd(), "trie-reports", fileName);
+    const reportPath = join7(process.cwd(), "trie-reports", fileName);
     try {
-      const content = await readFile6(reportPath, "utf-8");
+      const content = await readFile7(reportPath, "utf-8");
       return {
         contents: [{
           uri,