scai 0.1.77 → 0.1.79

package/README.md

@@ -139,7 +139,7 @@ SCAI supports an integrated review flow for GitHub pull requests. To get started
    scai git review
    ```
 
-   Use `-a` to list all PRs that require your review:
+   Use `-a` to list all PRs that require a review:
 
    ```sh
    scai git review -a

package/dist/commands/ReviewCmd.js

@@ -365,7 +365,7 @@ export async function reviewPullRequestCmd(branch = 'main', showAll = false) {
     try {
         const token = await ensureGitHubAuth();
         const username = await getGitHubUsername(token);
-        const { owner, repo } = getRepoDetails();
+        const { owner, repo } = await getRepoDetails();
         const prsWithReviewRequested = await getPullRequestsForReview(token, owner, repo, username, branch, !showAll);
         if (prsWithReviewRequested.length === 0)
             return;

package/dist/config.js

@@ -97,8 +97,6 @@ export const Config = {
         await this.setRepoIndexDir(scaiRepoRoot, absPath); // Set the indexDir for the repo
         // Ensure base folders exist
         fs.mkdirSync(scaiRepoRoot, { recursive: true });
-        fs.mkdirSync(path.join(scaiRepoRoot, 'summaries'), { recursive: true });
-        fs.mkdirSync(path.join(scaiRepoRoot, 'metadata'), { recursive: true });
         // Init DB if not exists
         const dbPath = path.join(scaiRepoRoot, 'db.sqlite');
         if (!fs.existsSync(dbPath)) {

package/dist/github/github.js

@@ -50,7 +50,7 @@ export async function fetchPullRequestDiff(pr, token) {
 }
 export async function submitReview(prNumber, body, event, comments) {
     const token = await ensureGitHubAuth();
-    const { owner, repo } = getRepoDetails();
+    const { owner, repo } = await getRepoDetails();
     const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/reviews`;
     const res = await fetch(url, {
         method: 'POST',
@@ -61,11 +61,40 @@ export async function submitReview(prNumber, body, event, comments) {
         body: JSON.stringify({
             body,
             event,
-            comments
+            comments,
         }),
     });
     if (!res.ok) {
         const errorText = await res.text();
+        // Attempt to parse error body
+        let parsed = {};
+        try {
+            parsed = JSON.parse(errorText);
+        }
+        catch (_) {
+            // leave as raw text if parsing fails
+        }
+        const knownErrors = Array.isArray(parsed.errors) ? parsed.errors.join('; ') : '';
+        // Handle known error cases
+        if (res.status === 422) {
+            if (knownErrors.includes('Can not approve your own pull request')) {
+                console.warn(`⚠️ Skipping approval: You cannot approve your own pull request.`);
+                return;
+            }
+            if (knownErrors.includes('Comments may only be specified on pull requests with a diff')) {
+                console.warn(`⚠️ Cannot post comments: PR has no diff.`);
+                return;
+            }
+            if (knownErrors.includes('path is missing') || knownErrors.includes('line is missing')) {
+                console.warn(`⚠️ Some inline comments are missing a path or line number. Skipping review.`);
+                return;
+            }
+            if (knownErrors.includes('Position is invalid') || knownErrors.includes('line must be part of the diff')) {
+                console.warn(`⚠️ One or more comment positions are invalid — probably outside the diff. Skipping review.`);
+                return;
+            }
+        }
+        // Unknown error
         throw new Error(`Failed to submit review: ${res.status} ${res.statusText} - ${errorText}`);
     }
     console.log(`✅ Submitted ${event} review for PR #${prNumber}`);
@@ -73,7 +102,7 @@ export async function submitReview(prNumber, body, event, comments) {
 export async function postInlineComment(prNumber, commitId, path, body, line, side = 'RIGHT', reviewId = null // Associate with a review if available
 ) {
     const token = await ensureGitHubAuth();
-    const { owner, repo } = getRepoDetails();
+    const { owner, repo } = await getRepoDetails();
     const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/comments`;
     const res = await fetch(url, {
         method: 'POST',
@@ -96,26 +125,3 @@ export async function postInlineComment(prNumber, commitId, path, body, line, si
     }
     console.log(`💬 Posted inline comment on ${path}:${line}`);
 }
-export async function createReviewForPR(prNumber, body, event = 'COMMENT') {
-    const token = await ensureGitHubAuth();
-    const { owner, repo } = getRepoDetails();
-    const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/reviews`;
-    const res = await fetch(url, {
-        method: 'POST',
-        headers: {
-            Authorization: `token ${token}`,
-            Accept: 'application/vnd.github.v3+json',
-        },
-        body: JSON.stringify({
-            body,
-            event,
-        }),
-    });
-    if (!res.ok) {
-        const errorText = await res.text();
-        throw new Error(`Failed to create review: ${res.status} ${res.statusText} - ${errorText}`);
-    }
-    const review = await res.json();
-    console.log(`✅ Created review for PR #${prNumber}`);
-    return review.id; // Return the review ID to be used for inline comments
-}

package/dist/github/githubAuthCheck.js

@@ -2,7 +2,7 @@ import { ensureGitHubAuth } from './auth.js';
 import { getRepoDetails } from './repo.js';
 export async function validateGitHubTokenAgainstRepo() {
     const token = await ensureGitHubAuth();
-    const { owner, repo } = getRepoDetails();
+    const { owner, repo } = await getRepoDetails();
     const response = await fetch(`https://api.github.com/repos/${owner}/${repo}`, {
         headers: {
             Authorization: `Bearer ${token}`,

package/dist/github/repo.js

@@ -1,6 +1,7 @@
 import path from 'path';
 import { execSync } from 'child_process';
 import { Config } from '../config.js';
+import { ensureGitHubAuth } from './auth.js';
 /**
  * Executes a Git command inside the specified working directory.
  */
@@ -41,17 +42,44 @@ function getRepoOwnerAndNameFromIndexDir(indexDir) {
  * Get the GitHub repo details, always from the configured indexDir.
  * Prefers Git config, falls back to parsing the path.
  */
-export function getRepoDetails() {
-    const indexDir = Config.getIndexDir();
+export async function getRepoDetails() {
+    let indexDir = Config.getIndexDir(); // Fetch the current indexDir
+    const currentDir = process.cwd(); // Get the current working directory
     if (!indexDir) {
         throw new Error("❌ indexDir is not configured.");
     }
     console.log(`📦 Resolving GitHub repo info from indexDir: ${indexDir}`);
+    // Check if the current directory is the same as the index directory
+    if (currentDir !== indexDir) {
+        console.log(`🚧 Current directory is not the index directory. Setting up new index...`);
+        // Use the existing functionality to set up a new index directory
+        await setIndexDirForCurrentDir(currentDir);
+        // Now update indexDir to the newly set one
+        indexDir = Config.getIndexDir(); // Re-fetch the updated indexDir
+    }
+    // Ensure the repository authentication is available
+    ensureAuthCredentials();
+    // Proceed with the normal flow to fetch the repo details
     try {
-        return getRepoOwnerAndNameFromGit(indexDir);
+        return getRepoOwnerAndNameFromGit(indexDir); // Try fetching from Git config
     }
     catch {
         console.log("🔁 Falling back to extracting from indexDir path...");
-        return getRepoOwnerAndNameFromIndexDir(indexDir);
+        return getRepoOwnerAndNameFromIndexDir(indexDir); // Fallback if Git config fails
+    }
+}
+async function setIndexDirForCurrentDir(currentDir) {
+    console.log(`🔧 Setting up index directory for the current directory: ${currentDir}`);
+    // Use the existing scai index set functionality to set the index directory
+    await Config.setIndexDir(currentDir);
+    // After setting the index, verify the directory is set correctly
+    const indexDir = Config.getIndexDir();
+    console.log(`✅ Index directory set to: ${indexDir}`);
+}
+async function ensureAuthCredentials() {
+    const token = await ensureGitHubAuth();
+    if (!token) {
+        throw new Error('❌ GitHub authentication token not found. Please authenticate first.');
     }
+    console.log('✅ GitHub authentication credentials are valid.');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scai",
-  "version": "0.1.77",
+  "version": "0.1.79",
   "type": "module",
   "bin": {
     "scai": "./dist/index.js"