scai 0.1.78 → 0.1.79
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/commands/ReviewCmd.js +1 -1
- package/dist/config.js +0 -2
- package/dist/github/github.js +2 -25
- package/dist/github/githubAuthCheck.js +1 -1
- package/dist/github/repo.js +32 -4
- package/package.json +1 -1
|
@@ -365,7 +365,7 @@ export async function reviewPullRequestCmd(branch = 'main', showAll = false) {
|
|
|
365
365
|
try {
|
|
366
366
|
const token = await ensureGitHubAuth();
|
|
367
367
|
const username = await getGitHubUsername(token);
|
|
368
|
-
const { owner, repo } = getRepoDetails();
|
|
368
|
+
const { owner, repo } = await getRepoDetails();
|
|
369
369
|
const prsWithReviewRequested = await getPullRequestsForReview(token, owner, repo, username, branch, !showAll);
|
|
370
370
|
if (prsWithReviewRequested.length === 0)
|
|
371
371
|
return;
|
package/dist/config.js
CHANGED
|
@@ -97,8 +97,6 @@ export const Config = {
|
|
|
97
97
|
await this.setRepoIndexDir(scaiRepoRoot, absPath); // Set the indexDir for the repo
|
|
98
98
|
// Ensure base folders exist
|
|
99
99
|
fs.mkdirSync(scaiRepoRoot, { recursive: true });
|
|
100
|
-
fs.mkdirSync(path.join(scaiRepoRoot, 'summaries'), { recursive: true });
|
|
101
|
-
fs.mkdirSync(path.join(scaiRepoRoot, 'metadata'), { recursive: true });
|
|
102
100
|
// Init DB if not exists
|
|
103
101
|
const dbPath = path.join(scaiRepoRoot, 'db.sqlite');
|
|
104
102
|
if (!fs.existsSync(dbPath)) {
|
package/dist/github/github.js
CHANGED
|
@@ -50,7 +50,7 @@ export async function fetchPullRequestDiff(pr, token) {
|
|
|
50
50
|
}
|
|
51
51
|
export async function submitReview(prNumber, body, event, comments) {
|
|
52
52
|
const token = await ensureGitHubAuth();
|
|
53
|
-
const { owner, repo } = getRepoDetails();
|
|
53
|
+
const { owner, repo } = await getRepoDetails();
|
|
54
54
|
const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/reviews`;
|
|
55
55
|
const res = await fetch(url, {
|
|
56
56
|
method: 'POST',
|
|
@@ -102,7 +102,7 @@ export async function submitReview(prNumber, body, event, comments) {
|
|
|
102
102
|
export async function postInlineComment(prNumber, commitId, path, body, line, side = 'RIGHT', reviewId = null // Associate with a review if available
|
|
103
103
|
) {
|
|
104
104
|
const token = await ensureGitHubAuth();
|
|
105
|
-
const { owner, repo } = getRepoDetails();
|
|
105
|
+
const { owner, repo } = await getRepoDetails();
|
|
106
106
|
const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/comments`;
|
|
107
107
|
const res = await fetch(url, {
|
|
108
108
|
method: 'POST',
|
|
@@ -125,26 +125,3 @@ export async function postInlineComment(prNumber, commitId, path, body, line, si
|
|
|
125
125
|
}
|
|
126
126
|
console.log(`💬 Posted inline comment on ${path}:${line}`);
|
|
127
127
|
}
|
|
128
|
-
export async function createReviewForPR(prNumber, body, event = 'COMMENT') {
|
|
129
|
-
const token = await ensureGitHubAuth();
|
|
130
|
-
const { owner, repo } = getRepoDetails();
|
|
131
|
-
const url = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/reviews`;
|
|
132
|
-
const res = await fetch(url, {
|
|
133
|
-
method: 'POST',
|
|
134
|
-
headers: {
|
|
135
|
-
Authorization: `token ${token}`,
|
|
136
|
-
Accept: 'application/vnd.github.v3+json',
|
|
137
|
-
},
|
|
138
|
-
body: JSON.stringify({
|
|
139
|
-
body,
|
|
140
|
-
event,
|
|
141
|
-
}),
|
|
142
|
-
});
|
|
143
|
-
if (!res.ok) {
|
|
144
|
-
const errorText = await res.text();
|
|
145
|
-
throw new Error(`Failed to create review: ${res.status} ${res.statusText} - ${errorText}`);
|
|
146
|
-
}
|
|
147
|
-
const review = await res.json();
|
|
148
|
-
console.log(`✅ Created review for PR #${prNumber}`);
|
|
149
|
-
return review.id; // Return the review ID to be used for inline comments
|
|
150
|
-
}
|
|
@@ -2,7 +2,7 @@ import { ensureGitHubAuth } from './auth.js';
|
|
|
2
2
|
import { getRepoDetails } from './repo.js';
|
|
3
3
|
export async function validateGitHubTokenAgainstRepo() {
|
|
4
4
|
const token = await ensureGitHubAuth();
|
|
5
|
-
const { owner, repo } = getRepoDetails();
|
|
5
|
+
const { owner, repo } = await getRepoDetails();
|
|
6
6
|
const response = await fetch(`https://api.github.com/repos/${owner}/${repo}`, {
|
|
7
7
|
headers: {
|
|
8
8
|
Authorization: `Bearer ${token}`,
|
package/dist/github/repo.js
CHANGED
|
@@ -1,6 +1,7 @@
|
|
|
1
1
|
import path from 'path';
|
|
2
2
|
import { execSync } from 'child_process';
|
|
3
3
|
import { Config } from '../config.js';
|
|
4
|
+
import { ensureGitHubAuth } from './auth.js';
|
|
4
5
|
/**
|
|
5
6
|
* Executes a Git command inside the specified working directory.
|
|
6
7
|
*/
|
|
@@ -41,17 +42,44 @@ function getRepoOwnerAndNameFromIndexDir(indexDir) {
|
|
|
41
42
|
* Get the GitHub repo details, always from the configured indexDir.
|
|
42
43
|
* Prefers Git config, falls back to parsing the path.
|
|
43
44
|
*/
|
|
44
|
-
export function getRepoDetails() {
|
|
45
|
-
|
|
45
|
+
export async function getRepoDetails() {
|
|
46
|
+
let indexDir = Config.getIndexDir(); // Fetch the current indexDir
|
|
47
|
+
const currentDir = process.cwd(); // Get the current working directory
|
|
46
48
|
if (!indexDir) {
|
|
47
49
|
throw new Error("❌ indexDir is not configured.");
|
|
48
50
|
}
|
|
49
51
|
console.log(`📦 Resolving GitHub repo info from indexDir: ${indexDir}`);
|
|
52
|
+
// Check if the current directory is the same as the index directory
|
|
53
|
+
if (currentDir !== indexDir) {
|
|
54
|
+
console.log(`🚧 Current directory is not the index directory. Setting up new index...`);
|
|
55
|
+
// Use the existing functionality to set up a new index directory
|
|
56
|
+
await setIndexDirForCurrentDir(currentDir);
|
|
57
|
+
// Now update indexDir to the newly set one
|
|
58
|
+
indexDir = Config.getIndexDir(); // Re-fetch the updated indexDir
|
|
59
|
+
}
|
|
60
|
+
// Ensure the repository authentication is available
|
|
61
|
+
ensureAuthCredentials();
|
|
62
|
+
// Proceed with the normal flow to fetch the repo details
|
|
50
63
|
try {
|
|
51
|
-
return getRepoOwnerAndNameFromGit(indexDir);
|
|
64
|
+
return getRepoOwnerAndNameFromGit(indexDir); // Try fetching from Git config
|
|
52
65
|
}
|
|
53
66
|
catch {
|
|
54
67
|
console.log("🔁 Falling back to extracting from indexDir path...");
|
|
55
|
-
return getRepoOwnerAndNameFromIndexDir(indexDir);
|
|
68
|
+
return getRepoOwnerAndNameFromIndexDir(indexDir); // Fallback if Git config fails
|
|
69
|
+
}
|
|
70
|
+
}
|
|
71
|
+
async function setIndexDirForCurrentDir(currentDir) {
|
|
72
|
+
console.log(`🔧 Setting up index directory for the current directory: ${currentDir}`);
|
|
73
|
+
// Use the existing scai index set functionality to set the index directory
|
|
74
|
+
await Config.setIndexDir(currentDir);
|
|
75
|
+
// After setting the index, verify the directory is set correctly
|
|
76
|
+
const indexDir = Config.getIndexDir();
|
|
77
|
+
console.log(`✅ Index directory set to: ${indexDir}`);
|
|
78
|
+
}
|
|
79
|
+
async function ensureAuthCredentials() {
|
|
80
|
+
const token = await ensureGitHubAuth();
|
|
81
|
+
if (!token) {
|
|
82
|
+
throw new Error('❌ GitHub authentication token not found. Please authenticate first.');
|
|
56
83
|
}
|
|
84
|
+
console.log('✅ GitHub authentication credentials are valid.');
|
|
57
85
|
}
|