@ariso-ai/ivan 1.0.25 → 1.0.28

package/dist/services/review-executor.js

@@ -0,0 +1,159 @@
+import { execSync } from 'child_process';
+import chalk from 'chalk';
+import { randomUUID } from 'crypto';
+import { DatabaseManager } from '../database.js';
+import { ExecutorFactory } from './executor-factory.js';
+export class ReviewExecutor {
+    dbManager;
+    constructor() {
+        this.dbManager = new DatabaseManager();
+    }
+    async executeReviews(prNumbers) {
+        const workingDir = process.cwd();
+        console.log(chalk.blue.bold(`🔍 Reviewing ${prNumbers.length} PR(s)...`));
+        console.log('');
+        for (const prNumber of prNumbers) {
+            await this.reviewPR(prNumber, workingDir);
+            console.log('');
+        }
+        this.dbManager.close();
+        console.log(chalk.green.bold('✅ All reviews completed!'));
+        console.log(chalk.gray('Run "ivan web" to view reviews in the dashboard.'));
+    }
+    async reviewPR(prNumber, workingDir) {
+        const db = this.dbManager.getKysely();
+        const now = new Date().toISOString();
+        const reviewUuid = randomUUID();
+        console.log(chalk.blue(`📋 Reviewing PR #${prNumber}...`));
+        // Fetch PR metadata
+        let prUrl = null;
+        let prTitle = null;
+        let repositoryId = null;
+        try {
+            const prJson = execSync(`gh pr view ${prNumber} --json number,title,url`, { cwd: workingDir, encoding: 'utf-8' });
+            const pr = JSON.parse(prJson);
+            prUrl = pr.url;
+            prTitle = pr.title;
+            console.log(chalk.gray(`  Title: ${prTitle}`));
+            console.log(chalk.gray(`  URL: ${prUrl}`));
+        }
+        catch (error) {
+            console.error(chalk.red(`❌ Failed to fetch PR #${prNumber} metadata:`), error);
+        }
+        // Look up repository_id from the DB
+        try {
+            const remoteUrl = execSync('git config --get remote.origin.url', {
+                cwd: workingDir,
+                encoding: 'utf-8'
+            }).trim();
+            const repo = await db
+                .selectFrom('repositories')
+                .select('id')
+                .where('remote_url', '=', remoteUrl)
+                .executeTakeFirst();
+            if (repo)
+                repositoryId = repo.id;
+        }
+        catch {
+            // Ignore — repository may not be registered
+        }
+        // Create the review record as 'running'
+        await db
+            .insertInto('reviews')
+            .values({
+            uuid: reviewUuid,
+            pr_number: prNumber,
+            pr_url: prUrl,
+            pr_title: prTitle,
+            repository_id: repositoryId,
+            status: 'running',
+            review_content: null,
+            execution_log: null,
+            created_at: now,
+            updated_at: now
+        })
+            .execute();
+        // Build the review prompt using the PR diff
+        let prDiff = '';
+        let prFiles = '';
+        try {
+            prDiff = execSync(`gh pr diff ${prNumber}`, {
+                cwd: workingDir,
+                encoding: 'utf-8',
+                maxBuffer: 10 * 1024 * 1024
+            });
+        }
+        catch (error) {
+            console.warn(chalk.yellow(`  ⚠️  Could not fetch diff for PR #${prNumber}`));
+        }
+        try {
+            prFiles = execSync(`gh pr view ${prNumber} --json files --jq '.files[].path'`, { cwd: workingDir, encoding: 'utf-8' });
+        }
+        catch {
+            // Ignore
+        }
+        const prompt = `You are a senior software engineer performing a thorough code review of PR #${prNumber}.
+
+${prTitle ? `PR Title: ${prTitle}` : ''}
+${prUrl ? `PR URL: ${prUrl}` : ''}
+${prFiles ? `\nChanged files:\n${prFiles}` : ''}
+
+${prDiff ? `\nDiff:\n\`\`\`diff\n${prDiff}\n\`\`\`` : 'No diff available — review the PR branch directly.'}
+
+Please review this pull request and provide:
+1. A brief summary of what the PR does
+2. Potential bugs or logic errors
+3. Security concerns (if any)
+4. Code quality and maintainability issues
+5. Suggestions for improvement
+6. Overall verdict: Approve / Request Changes / Comment
+
+Format your review clearly with sections and be specific about file and line references where applicable.
+After completing your review, output the final review as plain text starting with the line: "## Code Review: PR #${prNumber}"`;
+        try {
+            const executor = ExecutorFactory.getExecutor();
+            executor.quietMode = false;
+            const { log, lastMessage } = await executor.executeTask(prompt, workingDir);
+            // Extract the review content from the last message or log
+            const reviewContent = this.extractReviewContent(lastMessage, log, prNumber);
+            await db
+                .updateTable('reviews')
+                .set({
+                status: 'completed',
+                review_content: reviewContent,
+                execution_log: log,
+                updated_at: new Date().toISOString()
+            })
+                .where('uuid', '=', reviewUuid)
+                .execute();
+            console.log(chalk.green(`  ✅ Review for PR #${prNumber} completed.`));
+        }
+        catch (error) {
+            const errorMsg = error instanceof Error ? error.message : String(error);
+            await db
+                .updateTable('reviews')
+                .set({
+                status: 'failed',
+                execution_log: errorMsg,
+                updated_at: new Date().toISOString()
+            })
+                .where('uuid', '=', reviewUuid)
+                .execute();
+            console.error(chalk.red(`  ❌ Review for PR #${prNumber} failed: ${errorMsg}`));
+        }
+    }
+    extractReviewContent(lastMessage, log, prNumber) {
+        const marker = `## Code Review: PR #${prNumber}`;
+        // Try to find the structured review in the last message first
+        const inLast = lastMessage.indexOf(marker);
+        if (inLast !== -1)
+            return lastMessage.slice(inLast);
+        // Fall back to scanning the execution log
+        const inLog = log.indexOf(marker);
+        if (inLog !== -1)
+            return log.slice(inLog);
+        // If Claude didn't use the marker, return the last message as-is
+        return lastMessage || log;
+    }
+}
+//# sourceMappingURL=review-executor.js.map

package/dist/services/review-executor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-executor.js","sourceRoot":"","sources":["../../src/services/review-executor.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AACzC,OAAO,KAAK,MAAM,OAAO,CAAC;AAC1B,OAAO,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AACpC,OAAO,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAC;AACjD,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAExD,MAAM,OAAO,cAAc;IACjB,SAAS,CAAkB;IAEnC;QACE,IAAI,CAAC,SAAS,GAAG,IAAI,eAAe,EAAE,CAAC;IACzC,CAAC;IAED,KAAK,CAAC,cAAc,CAAC,SAAmB;QACtC,MAAM,UAAU,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;QAEjC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,gBAAgB,SAAS,CAAC,MAAM,WAAW,CAAC,CAAC,CAAC;QAC1E,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAEhB,KAAK,MAAM,QAAQ,IAAI,SAAS,EAAE,CAAC;YACjC,MAAM,IAAI,CAAC,QAAQ,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;YAC1C,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAClB,CAAC;QAED,IAAI,CAAC,SAAS,CAAC,KAAK,EAAE,CAAC;QACvB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC,CAAC;QAC1D,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,kDAAkD,CAAC,CAAC,CAAC;IAC9E,CAAC;IAEO,KAAK,CAAC,QAAQ,CAAC,QAAgB,EAAE,UAAkB;QACzD,MAAM,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,CAAC;QACtC,MAAM,GAAG,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QACrC,MAAM,UAAU,GAAG,UAAU,EAAE,CAAC;QAEhC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,oBAAoB,QAAQ,KAAK,CAAC,CAAC,CAAC;QAE3D,oBAAoB;QACpB,IAAI,KAAK,GAAkB,IAAI,CAAC;QAChC,IAAI,OAAO,GAAkB,IAAI,CAAC;QAClC,IAAI,YAAY,GAAkB,IAAI,CAAC;QAEvC,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,QAAQ,CACrB,cAAc,QAAQ,0BAA0B,EAChD,EAAE,GAAG,EAAE,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,CACvC,CAAC;YACF,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;YAC9B,KAAK,GAAG,EAAE,CAAC,GAAG,CAAC;YACf,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC;YACnB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,YAAY,OAAO,EAAE,CAAC,CAAC,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,UAAU,KAAK,EAAE,CAAC,CAAC,CAAC;QAC7C,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,yBAAyB,QAAQ,YAAY,CAAC,EAAE,KAAK,CAAC,CAAC;QACjF,CAAC;QAED,oCAAoC;QACpC,IAAI,CAAC;YACH,MAAM,SAAS,GAAG,QAAQ,CAAC,oCAAoC,EAAE;gBAC/D,GAAG,EAAE,UAAU;gBACf,QAAQ,EAAE,OAAO;aAClB,CAAC,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,GAAG,MAAM,EAAE;iBAClB,UAAU,CAAC,cAAc,CAAC;iBAC1B,MAAM,CAAC,IAAI,CAAC;iBACZ,KAAK,CAAC,YAAY,EAAE,GAAG,EAAE,SAAS,CAAC;iBACnC,gBAAgB,EAAE,CAAC;YACtB,IAAI,IAAI;gBAAE,YAAY,GAAG,IAAI,CAAC,EAAE,CAAC;QACnC,CAAC;QAAC,MAAM,CAAC;YACP,4CAA4C;QAC9C,CAAC;QAED,wCAAwC;QACxC,MAAM,EAAE;aACL,UAAU,CAAC,SAAS,CAAC;aACrB,MAAM,CAAC;YACN,IAAI,EAAE,UAAU;YAChB,SAAS,EAAE,QAAQ;YACnB,MAAM,EAAE,KAAK;YACb,QAAQ,EAAE,OAAO;YACjB,aAAa,EAAE,YAAY;YAC3B,MAAM,EAAE,SAAS;YACjB,cAAc,EAAE,IAAI;YACpB,aAAa,EAAE,IAAI;YACnB,UAAU,EAAE,GAAG;YACf,UAAU,EAAE,GAAG;SAChB,CAAC;aACD,OAAO,EAAE,CAAC;QAEb,4CAA4C;QAC5C,IAAI,MAAM,GAAG,EAAE,CAAC;QAChB,IAAI,OAAO,GAAG,EAAE,CAAC;QACjB,IAAI,CAAC;YACH,MAAM,GAAG,QAAQ,CAAC,cAAc,QAAQ,EAAE,EAAE;gBAC1C,GAAG,EAAE,UAAU;gBACf,QAAQ,EAAE,OAAO;gBACjB,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI;aAC5B,CAAC,CAAC;QACL,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,sCAAsC,QAAQ,EAAE,CAAC,CAAC,CAAC;QAC/E,CAAC;QAED,IAAI,CAAC;YACH,OAAO,GAAG,QAAQ,CAChB,cAAc,QAAQ,oCAAoC,EAC1D,EAAE,GAAG,EAAE,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,CACvC,CAAC;QACJ,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QAED,MAAM,MAAM,GAAG,+EAA+E,QAAQ;;EAExG,OAAO,CAAC,CAAC,CAAC,aAAa,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE;EACrC,KAAK,CAAC,CAAC,CAAC,WAAW,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE;EAC/B,OAAO,CAAC,CAAC,CAAC,qBAAqB,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE;;EAE7C,MAAM,CAAC,CAAC,CAAC,wBAAwB,MAAM,UAAU,CAAC,CAAC,CAAC,oDAAoD;;;;;;;;;;;mHAWS,QAAQ,GAAG,CAAC;QAE3H,IAAI,CAAC;YACH,MAAM,QAAQ,GAAG,eAAe,CAAC,WAAW,EAAE,CAAC;YAC/C,QAAQ,CAAC,SAAS,GAAG,KAAK,CAAC;YAE3B,MAAM,EAAE,GAAG,EAAE,WAAW,EAAE,GAAG,MAAM,QAAQ,CAAC,WAAW,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;YAE5E,0DAA0D;YAC1D,MAAM,aAAa,GAAG,IAAI,CAAC,oBAAoB,CAAC,WAAW,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;YAE5E,MAAM,EAAE;iBACL,WAAW,CAAC,SAAS,CAAC;iBACtB,GAAG,CAAC;gBACH,MAAM,EAAE,WAAW;gBACnB,cAAc,EAAE,aAAa;gBAC7B,aAAa,EAAE,GAAG;gBAClB,UAAU,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;aACrC,CAAC;iBACD,KAAK,CAAC,MAAM,EAAE,GAAG,EAAE,UAAU,CAAC;iBAC9B,OAAO,EAAE,CAAC;YAEb,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,sBAAsB,QAAQ,aAAa,CAAC,CAAC,CAAC;QACxE,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,QAAQ,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;YACxE,MAAM,EAAE;iBACL,WAAW,CAAC,SAAS,CAAC;iBACtB,GAAG,CAAC;gBACH,MAAM,EAAE,QAAQ;gBAChB,aAAa,EAAE,QAAQ;gBACvB,UAAU,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;aACrC,CAAC;iBACD,KAAK,CAAC,MAAM,EAAE,GAAG,EAAE,UAAU,CAAC;iBAC9B,OAAO,EAAE,CAAC;YAEb,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,sBAAsB,QAAQ,YAAY,QAAQ,EAAE,CAAC,CAAC,CAAC;QACjF,CAAC;IACH,CAAC;IAEO,oBAAoB,CAAC,WAAmB,EAAE,GAAW,EAAE,QAAgB;QAC7E,MAAM,MAAM,GAAG,uBAAuB,QAAQ,EAAE,CAAC;QAEjD,8DAA8D;QAC9D,MAAM,MAAM,GAAG,WAAW,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAC3C,IAAI,MAAM,KAAK,CAAC,CAAC;YAAE,OAAO,WAAW,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;QAEpD,0CAA0C;QAC1C,MAAM,KAAK,GAAG,GAAG,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAClC,IAAI,KAAK,KAAK,CAAC,CAAC;YAAE,OAAO,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAE1C,iEAAiE;QACjE,OAAO,WAAW,IAAI,GAAG,CAAC;IAC5B,CAAC;CACF"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ariso-ai/ivan",
-  "version": "1.0.25",
+  "version": "1.0.28",
   "type": "module",
   "main": "dist/index.js",
   "bin": {
@@ -73,6 +73,6 @@
     "kysely": "^0.28.5",
     "openai": "^5.16.0",
     "ora": "^8.2.0",
-    "sqlite-vec": "^0.1.7"
+    "sqlite-vec": "^0.1.9"
   }
 }