codeant-cli 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeant-cli",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Code review CLI tool",
   "type": "module",
   "bin": {

package/src/index.js

@@ -269,6 +269,25 @@ program
       }));
     });
 
+  pr.command('resolve')
+    .description('Resolve a conversation/comment thread on a PR')
+    .option('--name <repo>', 'Repository (owner/repo)')
+    .option('--remote <provider>', 'github, gitlab, bitbucket, azure')
+    .requiredOption('--pr-number <n>', 'PR number', parseInt)
+    .option('--comment-id <id>', 'Comment ID (GitHub, Bitbucket)', parseInt)
+    .option('--thread-id <id>', 'Thread/node ID (GitHub GraphQL, Azure)')
+    .option('--discussion-id <id>', 'Discussion ID (GitLab)')
+    .action((options) => {
+      const opts = resolveRepoOpts(options);
+      runCmd(() => scm.resolveConversation({
+        name: opts.name, remote: opts.remote,
+        prNumber: opts.prNumber,
+        commentId: opts.commentId,
+        threadId: opts.threadId,
+        discussionId: opts.discussionId,
+      }));
+    });
+
   // ─── Code review commands ───
   const codeReview = program.command('code-review').description('Code review tools');
 

package/src/scm/azure/index.js

@@ -4,3 +4,4 @@ export { listPullRequestComments } from './listPullRequestComments.js';
 export { listCodeReviews } from './listCodeReviews.js';
 export { getCodeReview } from './getCodeReview.js';
 export { searchComments } from './searchComments.js';
+export { resolveConversation } from './resolveConversation.js';

package/src/scm/azure/resolveConversation.js

@@ -0,0 +1,22 @@
+import { getConnection, splitRepo } from './client.js';
+
+/**
+ * Resolve a comment thread on an Azure DevOps pull request.
+ * Azure uses thread status: 1 = Active, 2 = Fixed (resolved), 3 = WontFix, 4 = Closed, 5 = ByDesign, 6 = Pending
+ */
+export async function resolveConversation({ name, prNumber, threadId }) {
+  if (!threadId) throw new Error('threadId is required to resolve an Azure DevOps thread');
+
+  const parsedThreadId = Number(threadId);
+  if (!Number.isInteger(parsedThreadId) || parsedThreadId <= 0) {
+    throw new Error('threadId must be a positive integer for Azure DevOps');
+  }
+
+  const connection = getConnection();
+  const gitApi = await connection.getGitApi();
+  const { project, repo } = splitRepo(name);
+
+  await gitApi.updateThread({ status: 2 }, repo, prNumber, parsedThreadId, project);
+
+  return { success: true };
+}

package/src/scm/bitbucket/index.js

@@ -4,3 +4,4 @@ export { listPullRequestComments } from './listPullRequestComments.js';
 export { listCodeReviews } from './listCodeReviews.js';
 export { getCodeReview } from './getCodeReview.js';
 export { searchComments } from './searchComments.js';
+export { resolveConversation } from './resolveConversation.js';

package/src/scm/bitbucket/resolveConversation.js

@@ -0,0 +1,21 @@
+import { getClient, splitRepo } from './client.js';
+
+/**
+ * Resolve a comment (conversation) on a Bitbucket pull request.
+ */
+export async function resolveConversation({ name, prNumber, commentId }) {
+  if (!commentId) throw new Error('commentId is required to resolve a Bitbucket comment');
+
+  const client = getClient();
+  const { workspace, repo_slug } = splitRepo(name);
+
+  await client.repositories.updatePullRequestComment({
+    workspace,
+    repo_slug,
+    pull_request_id: prNumber,
+    comment_id: commentId,
+    _body: { resolved: true },
+  });
+
+  return { success: true };
+}

package/src/scm/github/index.js

@@ -4,3 +4,4 @@ export { listPullRequestComments } from './listPullRequestComments.js';
 export { listCodeReviews } from './listCodeReviews.js';
 export { getCodeReview } from './getCodeReview.js';
 export { searchComments } from './searchComments.js';
+export { resolveConversation } from './resolveConversation.js';

package/src/scm/github/resolveConversation.js

@@ -0,0 +1,85 @@
+import { getClient, splitRepo } from './client.js';
+
+/**
+ * Resolve a review conversation on GitHub.
+ * GitHub doesn't have a native "resolve conversation" API for individual comments.
+ * We use the GraphQL API to minimize (hide) the comment, which is the closest equivalent.
+ * For review comments that are part of a PR review thread, we resolve via GraphQL resolveReviewThread.
+ */
+export async function resolveConversation({ name, prNumber, commentId, threadId }) {
+  const octokit = getClient();
+  const { owner, repo } = splitRepo(name);
+
+  // If a threadId (graphql node_id of the review thread) is provided, resolve the thread
+  if (threadId) {
+    await octokit.graphql(
+      `mutation($threadId: ID!) {
+        resolveReviewThread(input: { threadId: $threadId }) {
+          thread { isResolved }
+        }
+      }`,
+      { threadId }
+    );
+    return { success: true };
+  }
+
+  // Otherwise, try to find the review comment and resolve its thread via node_id
+  if (commentId) {
+    // Fetch the review comment to get its node_id for the thread
+    const { data: comment } = await octokit.pulls.getReviewComment({
+      owner, repo, comment_id: commentId,
+    });
+
+    if (comment.node_id) {
+      // Attempt to resolve the thread this comment belongs to
+      // The comment's node_id is not the thread node_id, so we need to query for the thread
+      const { repository } = await octokit.graphql(
+        `query($owner: String!, $repo: String!, $prNumber: Int!) {
+          repository(owner: $owner, name: $repo) {
+            pullRequest(number: $prNumber) {
+              reviewThreads(first: 100) {
+                nodes {
+                  id
+                  isResolved
+                  comments(first: 1) {
+                    nodes { databaseId }
+                  }
+                }
+              }
+            }
+          }
+        }`,
+        { owner, repo, prNumber }
+      );
+
+      const threads = repository?.pullRequest?.reviewThreads?.nodes;
+      if (!threads) {
+        throw new Error(`Pull request ${prNumber} was not found or is not accessible`);
+      }
+
+      const thread = threads.find(
+        t => t.comments.nodes.some(c => c.databaseId === commentId)
+      );
+
+      if (thread && !thread.isResolved) {
+        await octokit.graphql(
+          `mutation($threadId: ID!) {
+            resolveReviewThread(input: { threadId: $threadId }) {
+              thread { isResolved }
+            }
+          }`,
+          { threadId: thread.id }
+        );
+        return { success: true };
+      }
+
+      if (thread?.isResolved) {
+        return { success: true, message: 'Thread already resolved' };
+      }
+    }
+
+    throw new Error(`Could not find a review thread for comment ${commentId}`);
+  }
+
+  throw new Error('Either commentId or threadId is required');
+}

package/src/scm/gitlab/index.js

@@ -4,3 +4,4 @@ export { listPullRequestComments } from './listPullRequestComments.js';
 export { listCodeReviews } from './listCodeReviews.js';
 export { getCodeReview } from './getCodeReview.js';
 export { searchComments } from './searchComments.js';
+export { resolveConversation } from './resolveConversation.js';

package/src/scm/gitlab/resolveConversation.js

@@ -0,0 +1,14 @@
+import { getClient, projectId } from './client.js';
+
+/**
+ * Resolve a discussion (conversation) on a GitLab merge request.
+ */
+export async function resolveConversation({ name, prNumber, discussionId }) {
+  if (!discussionId) throw new Error('discussionId is required to resolve a GitLab discussion');
+
+  const api = getClient();
+
+  await api.MergeRequestDiscussions.edit(projectId(name), prNumber, discussionId, { resolved: true });
+
+  return { success: true };
+}

package/src/scm/index.js

@@ -94,3 +94,4 @@ export function listPullRequestComments(opts) { return getProvider(opts.remote).
 export function listCodeReviews(opts) { return getProvider(opts.remote).listCodeReviews(opts); }
 export function getCodeReview(opts) { return getProvider(opts.remote).getCodeReview(opts); }
 export function searchComments(opts) { return getProvider(opts.remote).searchComments(opts); }
+export function resolveConversation(opts) { return getProvider(opts.remote).resolveConversation(opts); }

package/src/utils/reviewApiHelper.js

@@ -5,6 +5,7 @@ import { minimatch } from 'minimatch';
 
 const MAX_REVIEW_FILES = 10;
 const MAX_FILE_LINES = 5000;
+const MAX_FILE_SIZE_BYTES = 200 * 1024; // 200 KB
 
 const EXCLUDED_FILENAMES = new Set([
   'package-lock.json',
@@ -37,6 +38,10 @@ const EXCLUDED_EXTENSIONS = new Set([
   '.pb', '.proto.bin',
   // PDF / docs
   '.pdf', '.doc', '.docx', '.xls', '.xlsx', '.ppt', '.pptx',
+  // Data / config (often large, auto-generated)
+  '.csv', '.tsv', '.parquet', '.avro', '.ndjson',
+  // Database
+  '.sqlite', '.db',
 ]);
 
 function matchesPatterns(filePath, includePatterns, excludePatterns) {
@@ -111,10 +116,15 @@ class ReviewApiHelper extends CommonApiHelper {
           continue;
         }
 
-        // Skip files exceeding line limit to avoid blowing up the AI context window
+        // Skip files exceeding size or line limits to avoid blowing up the AI context window
         try {
           const fullPath = path.resolve(gitRoot, f);
           if (fs.existsSync(fullPath)) {
+            const stats = fs.statSync(fullPath);
+            if (stats.size > MAX_FILE_SIZE_BYTES) {
+              skipped.push({ file: f, reason: `too large (${(stats.size / 1024).toFixed(0)} KB, max ${MAX_FILE_SIZE_BYTES / 1024} KB)` });
+              continue;
+            }
             const content = fs.readFileSync(fullPath, 'utf-8');
             const lineCount = content.split('\n').length;
             if (lineCount > MAX_FILE_LINES) {
@@ -214,6 +224,12 @@ class ReviewApiHelper extends CommonApiHelper {
       const hunks = fileHunks.get(f);
       const content = hunks[0]?.head_file_str || '';
       if (content) {
+        const contentSize = Buffer.byteLength(content, 'utf-8');
+        if (contentSize > MAX_FILE_SIZE_BYTES) {
+          skippedLargeFiles.add(f);
+          dedupedSkipped.push({ file: f, reason: `too large (${(contentSize / 1024).toFixed(0)} KB, max ${MAX_FILE_SIZE_BYTES / 1024} KB)` });
+          continue;
+        }
         const lineCount = content.split('\n').length;
         if (lineCount > MAX_FILE_LINES) {
           skippedLargeFiles.add(f);

package/src/utils/secretsApiHelper.js

@@ -48,9 +48,14 @@ const EXCLUDED_EXTENSIONS = new Set([
   '.pb', '.proto.bin',
   // PDF / docs
   '.pdf', '.doc', '.docx', '.xls', '.xlsx', '.ppt', '.pptx',
+  // Data / config (often large, auto-generated)
+  '.parquet', '.avro',
+  // Database
+  '.sqlite', '.db',
 ]);
 
 const MAX_FILE_LINES = 5000;
+const MAX_FILE_SIZE_BYTES = 200 * 1024; // 200 KB
 
 function isScannableFile(filePath) {
   const basename = path.basename(filePath);
@@ -201,8 +206,13 @@ class SecretsApiHelper extends CommonApiHelper {
         return false;
       }
 
-      // Skip files exceeding line limit
+      // Skip files exceeding size or line limits
       if (fileObj.code) {
+        const contentSize = Buffer.byteLength(fileObj.code, 'utf-8');
+        if (contentSize > MAX_FILE_SIZE_BYTES) {
+          skipped.push({ file: filePath, reason: `too large (${(contentSize / 1024).toFixed(0)} KB, max ${MAX_FILE_SIZE_BYTES / 1024} KB)` });
+          return false;
+        }
         const lineCount = fileObj.code.split('\n').length;
         if (lineCount > MAX_FILE_LINES) {
           skipped.push({ file: filePath, reason: `too large (${lineCount} lines, max ${MAX_FILE_LINES})` });