korekt-cli 0.9.0 → 0.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "AI-powered code review CLI - Keep your kode korekt",
   "main": "src/index.js",
   "bin": {

package/scripts/azure.sh

@@ -72,40 +72,20 @@ wrap_code_block() {
   printf "%s\n" "$content" | fold -s -w "$max_width"
 }
 
-# Function to fetch all pages of threads using pagination
-fetch_all_threads() {
+# Function to fetch threads (single request, no pagination)
+fetch_threads() {
   local url="$1"
-  local all_threads="[]"
-  local skip=0
-  local top=100
-  local has_more=true
-
-  while [ "$has_more" = true ]; do
-    local response
-    response=$(curl -s -X GET "${url}?\$top=${top}&\$skip=${skip}&api-version=6.0" \
-      -H "Authorization: Bearer $SYSTEM_ACCESSTOKEN" \
-      -H "Content-Type: application/json")
-
-    if ! echo "$response" | jq -e '.value' > /dev/null 2>&1; then
-      echo "[]"
-      return
-    fi
-
-    local page_threads
-    page_threads=$(echo "$response" | jq '.value')
-
-    local count
-    count=$(echo "$page_threads" | jq 'length')
+  local response
+  response=$(curl -s --max-time 30 -X GET "${url}?api-version=7.1" \
+    -H "Authorization: Bearer $SYSTEM_ACCESSTOKEN" \
+    -H "Content-Type: application/json")
 
-    if [ "$count" -eq 0 ]; then
-      has_more=false
-    else
-      all_threads=$(jq -s '.[0] + .[1]' <(echo "$all_threads") <(echo "$page_threads"))
-      skip=$((skip + top))
-    fi
-  done
+  if ! echo "$response" | jq -e '.value' > /dev/null 2>&1; then
+    echo "[]"
+    return
+  fi
 
-  echo "$all_threads"
+  echo "$response" | jq '.value'
 }
 
 # Function to delete old bot summary comments
@@ -114,7 +94,7 @@ delete_old_summary_comments() {
   local threads_url="${BASE_API_URL}/threads"
   local existing_threads
 
-  existing_threads=$(fetch_all_threads "$threads_url")
+  existing_threads=$(fetch_threads "$threads_url")
 
   if ! echo "$existing_threads" | jq -e . > /dev/null 2>&1; then
     echo "Warning: Could not fetch existing threads to delete old summaries."
@@ -153,7 +133,7 @@ populate_existing_comments_map() {
   local threads_url="${BASE_API_URL}/threads"
   local existing_threads
 
-  existing_threads=$(fetch_all_threads "$threads_url")
+  existing_threads=$(fetch_threads "$threads_url")
 
   if ! echo "$existing_threads" | jq -e . > /dev/null 2>&1; then
     echo "Warning: Could not fetch existing threads. Duplicate checking will be skipped."

package/src/index.test.js

@@ -451,7 +451,7 @@ describe('getPrUrl', () => {
     delete process.env.BITBUCKET_WORKSPACE;
     delete process.env.BITBUCKET_REPO_SLUG;
     delete process.env.BITBUCKET_PR_ID;
-    delete process.env.SYSTEM_COLLECTIONURI;
+    delete process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI;
     delete process.env.SYSTEM_TEAMPROJECT;
     delete process.env.BUILD_REPOSITORY_NAME;
     delete process.env.SYSTEM_PULLREQUEST_PULLREQUESTID;
@@ -477,7 +477,7 @@ describe('getPrUrl', () => {
   });
 
   it('should return Azure DevOps PR URL when Azure env vars are set', () => {
-    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI = 'https://dev.azure.com/myorg/';
     process.env.SYSTEM_TEAMPROJECT = 'myproject';
     process.env.BUILD_REPOSITORY_NAME = 'myrepo';
     process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
@@ -486,7 +486,7 @@ describe('getPrUrl', () => {
   });
 
   it('should strip trailing slash from Azure DevOps collection URI', () => {
-    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI = 'https://dev.azure.com/myorg/';
     process.env.SYSTEM_TEAMPROJECT = 'myproject';
     process.env.BUILD_REPOSITORY_NAME = 'myrepo';
     process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
@@ -497,7 +497,7 @@ describe('getPrUrl', () => {
   });
 
   it('should URL-encode spaces in Azure DevOps project and repo names', () => {
-    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI = 'https://dev.azure.com/myorg/';
     process.env.SYSTEM_TEAMPROJECT = 'My Project';
     process.env.BUILD_REPOSITORY_NAME = 'My Repo';
     process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
@@ -527,7 +527,7 @@ describe('getPrUrl', () => {
   });
 
   it('should return null when only partial Azure env vars are set', () => {
-    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI = 'https://dev.azure.com/myorg/';
     process.env.SYSTEM_TEAMPROJECT = 'myproject';
     // BUILD_REPOSITORY_NAME and SYSTEM_PULLREQUEST_PULLREQUESTID not set
 

package/src/utils.js

@@ -34,12 +34,12 @@ export function getPrUrl() {
   }
   // Azure DevOps Pipelines
   if (
-    process.env.SYSTEM_COLLECTIONURI &&
+    process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI &&
     process.env.SYSTEM_TEAMPROJECT &&
     process.env.BUILD_REPOSITORY_NAME &&
     process.env.SYSTEM_PULLREQUEST_PULLREQUESTID
   ) {
-    const collectionUri = process.env.SYSTEM_COLLECTIONURI.replace(/\/$/, '');
+    const collectionUri = process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI.replace(/\/$/, '');
     return `${collectionUri}/${encodeURIComponent(process.env.SYSTEM_TEAMPROJECT)}/_git/${encodeURIComponent(process.env.BUILD_REPOSITORY_NAME)}/pullrequest/${process.env.SYSTEM_PULLREQUEST_PULLREQUESTID}`;
   }
   return null;