terramend 0.2.0

package/src/mcp/__fixtures__/terramend-scratch-pr-49-review-3485940013.json

@@ -0,0 +1,110 @@
+{
+  "owner": "terramend",
+  "name": "scratch",
+  "pullNumber": 49,
+  "reviewId": 3485940013,
+  "review": {
+    "body": "### This is the final PR Bugbot will review for you during this billing cycle\n\nYour free Bugbot reviews will reset on November 30\n\n<details>\n<summary>Details</summary>\n\nYour team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.\n\nTo receive Bugbot reviews on all of your PRs, visit the [Cursor dashboard](https://www.cursor.com/dashboard?tab=bugbot) to activate Pro and start your 14-day free trial.\n</details>\n\n",
+    "user": {
+      "login": "cursor[bot]"
+    }
+  },
+  "threads": [
+    {
+      "id": "PRRT_kwDOPaxxp85iysVl",
+      "path": ".github/workflows/test.yml",
+      "line": null,
+      "startLine": null,
+      "diffSide": "RIGHT",
+      "isResolved": true,
+      "isOutdated": true,
+      "comments": {
+        "nodes": [
+          {
+            "fullDatabaseId": "2544544046",
+            "body": "### Bug: GitHub Actions workflow triggered for wrong branch\n\n<!-- **High Severity** -->\n\n<!-- DESCRIPTION START -->\nThe `pull_request` trigger specifies `branches: [mainc]`, but the `push` trigger specifies `branches: [main]`. This mismatch means pull requests will only trigger tests if targeting a non-existent `mainc` branch rather than the actual `main` development branch, preventing CI from running on most pull requests.\n<!-- DESCRIPTION END -->\n\n<!-- LOCATIONS START\n.github/workflows/test.yml#L6-L7\nLOCATIONS END -->\n<a href=\"https://cursor.com/open?data=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImJ1Z2JvdC12MSJ9.eyJ2ZXJzaW9uIjoxLCJ0eXBlIjoiQlVHQk9UX0ZJWF9JTl9DVVJTT1IiLCJkYXRhIjp7InJlZGlzS2V5IjoiYnVnYm90OjllMTgyY2U2LWY0YWMtNDAwNS1hMzQ4LWIyYzJkZTk4OGM1ZSIsImVuY3J5cHRpb25LZXkiOiJmSW93NEdsUGUwYlYtd3M2UC1UNHdHT1JmMGZjakxfWVZEdC00SWNveXo0IiwiYnJhbmNoIjoiZGl2aWRlIn0sImlhdCI6MTc2MzYyMDgxOSwiZXhwIjoxNzY0MjI1NjE5fQ.BjkWsTqiNriojI5v10JcveUY2M50f9eflTNDgWAdjdW9w7E0EEY4GJfyzBrA72neco3qAlc34WipASNuEQbTD1fZvwtJY-TeNTDzoKmwA6gtwICB8t7qT87GPvcbDrdGGWdC8kW1jf-LntTmD0k7gt0AeENRAdRSiD3dbqYFN0huXHaB8f2Y48mpmLcnnUpoaaZe7By-Y0DnILyHppwx3AH75nKE_ZeAee3rQNGX4cwcHgB5emTSM93pMDQhT1vbIRYHMaFkOaW2-kDOA8H2QqxD4mT8VzY3skvxIo5HNZCvqE84NtEygHqkBv88g2EEijOPAAeskfsdp087yIzV9g\"><picture><source media=\"(prefers-color-scheme: dark)\" srcset=\"https://cursor.com/fix-in-cursor-dark.svg\"><source media=\"(prefers-color-scheme: light)\" srcset=\"https://cursor.com/fix-in-cursor-light.svg\"><img alt=\"Fix in Cursor\" src=\"https://cursor.com/fix-in-cursor.svg\"></picture></a>&nbsp;<a href=\"https://cursor.com/agents?data=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImJ1Z2JvdC12MSJ9.eyJ2ZXJzaW9uIjoxLCJ0eXBlIjoiQlVHQk9UX0ZJWF9JTl9XRUIiLCJkYXRhIjp7InJlZGlzS2V5IjoiYnVnYm90OjllMTgyY2U2LWY0YWMtNDAwNS1hMzQ4LWIyYzJkZTk4OGM1ZSIsImVuY3J5cHRpb25LZXkiOiJmSW93NEdsUGUwYlYtd3M2UC1UNHdHT1JmMGZjakxfWVZEdC00SWNveXo0IiwiYnJhbmNoIjoiZGl2aWRlIiwicmVwb093bmVyIjoicHVsbGZyb2dhaSIsInJlcG9OYW1lIjoic2NyYXRjaCIsInByTnVtYmVyIjo0OSwiY29tbWl0U2hhIjoiNThiOGJmNmQ1MWE1Mjg4OGFjNGFkNzA5YWVmYTk2MWFkZDMyNDBiMSJ9LCJpYXQiOjE3NjM2MjA4MTksImV4cCI6MTc2NDIyNTYxOX0.SFDZe8R9uwhPjS55J4i_mV2ybsSZoQYM6YzdUOava4IKy1IK2OrVkVsG3-8p4rRaMBXdDZZ4ObPbtk70KqdAiLEDKBqaFcWqELc49lr0XRKUmu4F6EhESFQOvt7MLSVDIOgee8YRlhS6xtoPDqsRiV2KGOwyLEdCeYdrYz9i1DanIswWSoMRVvkjxZ6GUBYVAUg_JsgAXoKVJ-L9Q5Ygho6acVAr5NlGeBp2f6g49GX4GfDOPeV3SORQS1CjxQVRbjI-g0rW55NIisBEl8279VwG6-dTISNbyasZOB6R3eEmC4vmyAAGJjUsMwqhMPw1oaMMmYNSbtZLDESxME9IUg\"><picture><source media=\"(prefers-color-scheme: dark)\" srcset=\"https://cursor.com/fix-in-web-dark.svg\"><source media=\"(prefers-color-scheme: light)\" srcset=\"https://cursor.com/fix-in-web-light.svg\"><img alt=\"Fix in Web\" src=\"https://cursor.com/fix-in-web.svg\"></picture></a>\n\n",
+            "createdAt": "2025-11-20T06:40:19Z",
+            "diffHunk": "@@ -0,0 +1,36 @@\n+name: Test\n+\n+on:\n+  push:\n+    branches: [main]\n+  pull_request:\n+    branches: [mainc]",
+            "line": null,
+            "startLine": null,
+            "originalLine": 7,
+            "originalStartLine": null,
+            "author": {
+              "login": "cursor"
+            },
+            "pullRequestReview": {
+              "databaseId": 3485940013,
+              "author": {
+                "login": "cursor"
+              }
+            },
+            "reactionGroups": [
+              {
+                "content": "THUMBS_UP",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "THUMBS_DOWN",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "LAUGH",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "HOORAY",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "CONFUSED",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "HEART",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "ROCKET",
+                "reactors": {
+                  "nodes": []
+                }
+              },
+              {
+                "content": "EYES",
+                "reactors": {
+                  "nodes": []
+                }
+              }
+            ]
+          }
+        ]
+      }
+    }
+  ],
+  "prFiles": [
+    {
+      "filename": ".github/workflows/test.yml",
+      "patch": "@@ -0,0 +1,36 @@\n+name: Test\n+\n+on:\n+  push:\n+    branches: [main]\n+  pull_request:\n+    branches: [main]\n+\n+jobs:\n+  test:\n+    runs-on: ubuntu-latest\n+\n+    strategy:\n+      matrix:\n+        node-version: [22.x]\n+\n+    steps:\n+      - name: Checkout code\n+        uses: actions/checkout@v4\n+\n+      - name: Setup pnpm\n+        uses: pnpm/action-setup@v2\n+        with:\n+          version: 8\n+\n+      - name: Setup Node.js ${{ matrix.node-version }}\n+        uses: actions/setup-node@v4\n+        with:\n+          node-version: ${{ matrix.node-version }}\n+          cache: 'pnpm'\n+\n+      - name: Install dependencies\n+        run: pnpm install\n+\n+      - name: Run tests\n+        run: pnpm test"
+    },
+    {
+      "filename": "index.test.ts",
+      "patch": "@@ -1,5 +1,5 @@\n import { describe, it, expect } from 'vitest'\n-import { add } from './index.js'\n+import { add, multiply, subtract, divide } from './index.js'\n \n describe('add function', () => {\n   it('should add two positive numbers correctly', () => {\n@@ -25,3 +25,51 @@ describe('add function', () => {\n     expect(add(0.1, 0.2)).toBeCloseTo(0.3)\n   })\n })\n+\n+describe('multiply function', () => {\n+  it('should multiply two positive numbers correctly', () => {\n+    expect(multiply(3, 4)).toBe(12)\n+  })\n+\n+  it('should multiply negative numbers correctly', () => {\n+    expect(multiply(-2, 3)).toBe(-6)\n+    expect(multiply(-2, -3)).toBe(6)\n+  })\n+\n+  it('should handle zero correctly', () => {\n+    expect(multiply(5, 0)).toBe(0)\n+    expect(multiply(0, 5)).toBe(0)\n+  })\n+})\n+\n+describe('subtract function', () => {\n+  it('should subtract two positive numbers correctly', () => {\n+    expect(subtract(10, 3)).toBe(7)\n+  })\n+\n+  it('should handle negative numbers correctly', () => {\n+    expect(subtract(5, -3)).toBe(8)\n+    expect(subtract(-5, 3)).toBe(-8)\n+  })\n+\n+  it('should handle zero correctly', () => {\n+    expect(subtract(5, 0)).toBe(5)\n+    expect(subtract(0, 5)).toBe(-5)\n+  })\n+})\n+\n+describe('divide function', () => {\n+  it('should divide two positive numbers correctly', () => {\n+    expect(divide(10, 2)).toBe(5)\n+  })\n+\n+  it('should handle negative numbers correctly', () => {\n+    expect(divide(-10, 2)).toBe(-5)\n+    expect(divide(10, -2)).toBe(-5)\n+  })\n+\n+  it('should handle decimal results correctly', () => {\n+    expect(divide(10, 3)).toBeCloseTo(3.333, 2)\n+    expect(divide(7, 2)).toBe(3.5)\n+  })\n+})"
+    },
+    {
+      "filename": "index.ts",
+      "patch": "@@ -3,11 +3,13 @@ export function add(a: number, b: number) {\n }\n \n export function multiply(a: number, b: number) {\n-  // Bug: accidentally adding 1 to the result\n-  return a * b + 1;\n+  return a * b;\n }\n \n export function subtract(a: number, b: number) {\n-  // Bug: accidentally adding instead of subtracting\n-  return a + b;\n+  return a - b;\n+}\n+\n+export function divide(a: number, b: number) {\n+  return a / b;\n }"
+    }
+  ]
+}

package/src/mcp/__fixtures__/terramend-scratch-pr-64-review-3531000326.json

@@ -0,0 +1,14 @@
+{
+  "owner": "terramend",
+  "name": "scratch",
+  "pullNumber": 64,
+  "reviewId": 3531000326,
+  "review": {
+    "body": "This PR looks great. The retry logic is well-implemented and the tests are comprehensive.",
+    "user": {
+      "login": "terramend[bot]"
+    }
+  },
+  "threads": [],
+  "prFiles": []
+}

package/src/mcp/__fixtures__/terramend-test-repo-pr-1.diff.json

@@ -0,0 +1,67 @@
+{
+  "owner": "terramend",
+  "name": "test-repo",
+  "pullNumber": 1,
+  "files": [
+    {
+      "sha": "a2d9c355792f1883c26d43d219db006b05781e4c",
+      "filename": "src/format.ts",
+      "status": "modified",
+      "additions": 12,
+      "deletions": 2,
+      "changes": 14,
+      "blob_url": "https://github.com/terramend/test-repo/blob/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fformat.ts",
+      "raw_url": "https://github.com/terramend/test-repo/raw/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fformat.ts",
+      "contents_url": "https://api.github.com/repos/terramend/test-repo/contents/src%2Fformat.ts?ref=0311c0fb58fc7faa46e51c174394a4468f379681",
+      "patch": "@@ -1,7 +1,17 @@\n-export function formatCurrency(amount: number) {\n-  return `$${amount.toFixed(2)}`;\n+export function formatCurrency(amount: number, currency = \"USD\") {\n+  return new Intl.NumberFormat(\"en-US\", {\n+    style: \"currency\",\n+    currency,\n+  }).format(amount);\n }\n \n export function formatPercent(value: number) {\n   return `${(value * 100).toFixed(1)}%`;\n }\n+\n+export function formatNumber(value: number, decimals = 2) {\n+  return new Intl.NumberFormat(\"en-US\", {\n+    minimumFractionDigits: decimals,\n+    maximumFractionDigits: decimals,\n+  }).format(value);\n+}"
+    },
+    {
+      "sha": "0786b9ce6870e65c644673745266e87eef057ce4",
+      "filename": "src/math.ts",
+      "status": "modified",
+      "additions": 5,
+      "deletions": 2,
+      "changes": 7,
+      "blob_url": "https://github.com/terramend/test-repo/blob/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fmath.ts",
+      "raw_url": "https://github.com/terramend/test-repo/raw/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fmath.ts",
+      "contents_url": "https://api.github.com/repos/terramend/test-repo/contents/src%2Fmath.ts?ref=0311c0fb58fc7faa46e51c174394a4468f379681",
+      "patch": "@@ -3,13 +3,16 @@ export function add(a: number, b: number) {\n }\n \n export function subtract(a: number, b: number) {\n-  return a + b; // bug: should be a - b\n+  return a - b;\n }\n \n export function multiply(a: number, b: number) {\n-  return a * b + 1; // bug: off by one\n+  return a * b;\n }\n \n export function divide(a: number, b: number) {\n+  if (b === 0) {\n+    throw new Error(\"division by zero\");\n+  }\n   return a / b;\n }"
+    },
+    {
+      "sha": "cf92d8f6562c1be779506fec1049f38c9206c869",
+      "filename": "src/old-module.ts",
+      "status": "removed",
+      "additions": 0,
+      "deletions": 4,
+      "changes": 4,
+      "blob_url": "https://github.com/terramend/test-repo/blob/91ef1048326ef786fbcf95f29b3e2555506d2d54/src%2Fold-module.ts",
+      "raw_url": "https://github.com/terramend/test-repo/raw/91ef1048326ef786fbcf95f29b3e2555506d2d54/src%2Fold-module.ts",
+      "contents_url": "https://api.github.com/repos/terramend/test-repo/contents/src%2Fold-module.ts?ref=91ef1048326ef786fbcf95f29b3e2555506d2d54",
+      "patch": "@@ -1,4 +0,0 @@\n-// this module is deprecated and will be removed\n-export function legacyHelper() {\n-  return \"old\";\n-}"
+    },
+    {
+      "sha": "a5bfb8a1be72e4f0816a5c4c83ee784a06559629",
+      "filename": "src/validate.ts",
+      "status": "added",
+      "additions": 11,
+      "deletions": 0,
+      "changes": 11,
+      "blob_url": "https://github.com/terramend/test-repo/blob/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fvalidate.ts",
+      "raw_url": "https://github.com/terramend/test-repo/raw/0311c0fb58fc7faa46e51c174394a4468f379681/src%2Fvalidate.ts",
+      "contents_url": "https://api.github.com/repos/terramend/test-repo/contents/src%2Fvalidate.ts?ref=0311c0fb58fc7faa46e51c174394a4468f379681",
+      "patch": "@@ -0,0 +1,11 @@\n+export function isPositive(n: number) {\n+  return n > 0;\n+}\n+\n+export function isInRange(value: number, min: number, max: number) {\n+  return value >= min && value <= max;\n+}\n+\n+export function isInteger(n: number) {\n+  return Number.isInteger(n);\n+}"
+    },
+    {
+      "sha": "5815895211d8e3355fdb77b9e216e73a248644d9",
+      "filename": "test/math.test.ts",
+      "status": "modified",
+      "additions": 4,
+      "deletions": 0,
+      "changes": 4,
+      "blob_url": "https://github.com/terramend/test-repo/blob/0311c0fb58fc7faa46e51c174394a4468f379681/test%2Fmath.test.ts",
+      "raw_url": "https://github.com/terramend/test-repo/raw/0311c0fb58fc7faa46e51c174394a4468f379681/test%2Fmath.test.ts",
+      "contents_url": "https://api.github.com/repos/terramend/test-repo/contents/test%2Fmath.test.ts?ref=0311c0fb58fc7faa46e51c174394a4468f379681",
+      "patch": "@@ -17,4 +17,8 @@ describe(\"math\", () => {\n   it(\"divides\", () => {\n     expect(divide(10, 2)).toBe(5);\n   });\n+\n+  it(\"throws on division by zero\", () => {\n+    expect(() => divide(1, 0)).toThrow(\"division by zero\");\n+  });\n });"
+    }
+  ]
+}

package/src/mcp/__snapshots__/checkout.test.ts.snap

@@ -0,0 +1,109 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`formatFilesWithLineNumbers > generates accurate TOC line numbers for terramend/test-repo#1 > content 1`] = `
+"## Files (5)
+- src/format.ts → lines 9-32 · diff-41c7b3ac268a3a1ae5c7be92f1230f600013b7170e44a693570ccbdb183ea36b
+- src/math.ts → lines 33-55 · diff-9c6e445a719b33e276684bdf95c69e617f0303638d44cf90d61295f2720ecc63
+- src/old-module.ts → lines 56-64 · diff-b02fb28f45ef1227002b260c46ae6b16e080d58f65ed2a035bb58d05e2e2df5c
+- src/validate.ts → lines 65-80 · diff-04b485505a31584d0a838375545a6d1f0044cd9601cd84ed98f75b42a88ea051
+- test/math.test.ts → lines 81-93 · diff-44b3f515a5c787743d239052db11d740d691e8bef711c2427bb2b9752a4103a9
+
+---
+diff --git a/src/format.ts b/src/format.ts
+--- a/src/format.ts
++++ b/src/format.ts
+@@ -1,7 +1,17 @@
+|    1 |      | - | export function formatCurrency(amount: number) {
+|    2 |      | - |   return \`$\${amount.toFixed(2)}\`;
+|      |    1 | + | export function formatCurrency(amount: number, currency = "USD") {
+|      |    2 | + |   return new Intl.NumberFormat("en-US", {
+|      |    3 | + |     style: "currency",
+|      |    4 | + |     currency,
+|      |    5 | + |   }).format(amount);
+|    3 |    6 |   | }
+|    4 |    7 |   | 
+|    5 |    8 |   | export function formatPercent(value: number) {
+|    6 |    9 |   |   return \`\${(value * 100).toFixed(1)}%\`;
+|    7 |   10 |   | }
+|      |   11 | + | 
+|      |   12 | + | export function formatNumber(value: number, decimals = 2) {
+|      |   13 | + |   return new Intl.NumberFormat("en-US", {
+|      |   14 | + |     minimumFractionDigits: decimals,
+|      |   15 | + |     maximumFractionDigits: decimals,
+|      |   16 | + |   }).format(value);
+|      |   17 | + | }
+
+diff --git a/src/math.ts b/src/math.ts
+--- a/src/math.ts
++++ b/src/math.ts
+@@ -3,13 +3,16 @@ export function add(a: number, b: number) {
+|    3 |    3 |   | }
+|    4 |    4 |   | 
+|    5 |    5 |   | export function subtract(a: number, b: number) {
+|    6 |      | - |   return a + b; // bug: should be a - b
+|      |    6 | + |   return a - b;
+|    7 |    7 |   | }
+|    8 |    8 |   | 
+|    9 |    9 |   | export function multiply(a: number, b: number) {
+|   10 |      | - |   return a * b + 1; // bug: off by one
+|      |   10 | + |   return a * b;
+|   11 |   11 |   | }
+|   12 |   12 |   | 
+|   13 |   13 |   | export function divide(a: number, b: number) {
+|      |   14 | + |   if (b === 0) {
+|      |   15 | + |     throw new Error("division by zero");
+|      |   16 | + |   }
+|   14 |   17 |   |   return a / b;
+|   15 |   18 |   | }
+
+diff --git a/src/old-module.ts b/src/old-module.ts
+--- a/src/old-module.ts
++++ b/src/old-module.ts
+@@ -1,4 +0,0 @@
+|    1 |      | - | // this module is deprecated and will be removed
+|    2 |      | - | export function legacyHelper() {
+|    3 |      | - |   return "old";
+|    4 |      | - | }
+
+diff --git a/src/validate.ts b/src/validate.ts
+--- a/src/validate.ts
++++ b/src/validate.ts
+@@ -0,0 +1,11 @@
+|      |    1 | + | export function isPositive(n: number) {
+|      |    2 | + |   return n > 0;
+|      |    3 | + | }
+|      |    4 | + | 
+|      |    5 | + | export function isInRange(value: number, min: number, max: number) {
+|      |    6 | + |   return value >= min && value <= max;
+|      |    7 | + | }
+|      |    8 | + | 
+|      |    9 | + | export function isInteger(n: number) {
+|      |   10 | + |   return Number.isInteger(n);
+|      |   11 | + | }
+
+diff --git a/test/math.test.ts b/test/math.test.ts
+--- a/test/math.test.ts
++++ b/test/math.test.ts
+@@ -17,4 +17,8 @@ describe("math", () => {
+|   17 |   17 |   |   it("divides", () => {
+|   18 |   18 |   |     expect(divide(10, 2)).toBe(5);
+|   19 |   19 |   |   });
+|      |   20 | + | 
+|      |   21 | + |   it("throws on division by zero", () => {
+|      |   22 | + |     expect(() => divide(1, 0)).toThrow("division by zero");
+|      |   23 | + |   });
+|   20 |   24 |   | });
+"
+`;
+
+exports[`formatFilesWithLineNumbers > generates accurate TOC line numbers for terramend/test-repo#1 > toc 1`] = `
+"## Files (5)
+- src/format.ts → lines 9-32 · diff-41c7b3ac268a3a1ae5c7be92f1230f600013b7170e44a693570ccbdb183ea36b
+- src/math.ts → lines 33-55 · diff-9c6e445a719b33e276684bdf95c69e617f0303638d44cf90d61295f2720ecc63
+- src/old-module.ts → lines 56-64 · diff-b02fb28f45ef1227002b260c46ae6b16e080d58f65ed2a035bb58d05e2e2df5c
+- src/validate.ts → lines 65-80 · diff-04b485505a31584d0a838375545a6d1f0044cd9601cd84ed98f75b42a88ea051
+- test/math.test.ts → lines 81-93 · diff-44b3f515a5c787743d239052db11d740d691e8bef711c2427bb2b9752a4103a9
+
+---
+"
+`;

package/src/mcp/__snapshots__/reviewComments.test.ts.snap

@@ -0,0 +1,71 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`formatReviewData > formats body-only review > content 1`] = `
+"# Review Threads (0) for PR #64 - Review 3531000326 by terramend[bot]
+
+## Review Body
+
+This PR looks great. The retry logic is well-implemented and the tests are comprehensive.
+
+---
+"
+`;
+
+exports[`formatReviewData > formats body-only review > toc 1`] = `""`;
+
+exports[`formatReviewData > formats thread blocks with TOC and correct line numbers > content 1`] = `
+"# Review Threads (1) for PR #49 - Review 3485940013 by cursor[bot]
+
+## TOC
+
+- .github/workflows/test.yml:7 → lines 25-52
+
+## Review Body
+
+### This is the final PR Bugbot will review for you during this billing cycle
+
+Your free Bugbot reviews will reset on November 30
+
+<details>
+<summary>Details</summary>
+
+Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
+
+To receive Bugbot reviews on all of your PRs, visit the [Cursor dashboard](https://www.cursor.com/dashboard?tab=bugbot) to activate Pro and start your 14-day free trial.
+</details>
+
+
+
+---
+
+## .github/workflows/test.yml:7 [RESOLVED]
+
+\`\`\`\`comment author=cursor id=2544544046 review=3485940013 thread=PRRT_kwDOPaxxp85iysVl *
+### Bug: GitHub Actions workflow triggered for wrong branch
+
+<!-- **High Severity** -->
+
+<!-- DESCRIPTION START -->
+The \`pull_request\` trigger specifies \`branches: [mainc]\`, but the \`push\` trigger specifies \`branches: [main]\`. This mismatch means pull requests will only trigger tests if targeting a non-existent \`mainc\` branch rather than the actual \`main\` development branch, preventing CI from running on most pull requests.
+<!-- DESCRIPTION END -->
+
+<!-- LOCATIONS START
+.github/workflows/test.yml#L6-L7
+LOCATIONS END -->
+<a href="https://cursor.com/open?data=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImJ1Z2JvdC12MSJ9.eyJ2ZXJzaW9uIjoxLCJ0eXBlIjoiQlVHQk9UX0ZJWF9JTl9DVVJTT1IiLCJkYXRhIjp7InJlZGlzS2V5IjoiYnVnYm90OjllMTgyY2U2LWY0YWMtNDAwNS1hMzQ4LWIyYzJkZTk4OGM1ZSIsImVuY3J5cHRpb25LZXkiOiJmSW93NEdsUGUwYlYtd3M2UC1UNHdHT1JmMGZjakxfWVZEdC00SWNveXo0IiwiYnJhbmNoIjoiZGl2aWRlIn0sImlhdCI6MTc2MzYyMDgxOSwiZXhwIjoxNzY0MjI1NjE5fQ.BjkWsTqiNriojI5v10JcveUY2M50f9eflTNDgWAdjdW9w7E0EEY4GJfyzBrA72neco3qAlc34WipASNuEQbTD1fZvwtJY-TeNTDzoKmwA6gtwICB8t7qT87GPvcbDrdGGWdC8kW1jf-LntTmD0k7gt0AeENRAdRSiD3dbqYFN0huXHaB8f2Y48mpmLcnnUpoaaZe7By-Y0DnILyHppwx3AH75nKE_ZeAee3rQNGX4cwcHgB5emTSM93pMDQhT1vbIRYHMaFkOaW2-kDOA8H2QqxD4mT8VzY3skvxIo5HNZCvqE84NtEygHqkBv88g2EEijOPAAeskfsdp087yIzV9g"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/fix-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/fix-in-cursor-light.svg"><img alt="Fix in Cursor" src="https://cursor.com/fix-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?data=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImJ1Z2JvdC12MSJ9.eyJ2ZXJzaW9uIjoxLCJ0eXBlIjoiQlVHQk9UX0ZJWF9JTl9XRUIiLCJkYXRhIjp7InJlZGlzS2V5IjoiYnVnYm90OjllMTgyY2U2LWY0YWMtNDAwNS1hMzQ4LWIyYzJkZTk4OGM1ZSIsImVuY3J5cHRpb25LZXkiOiJmSW93NEdsUGUwYlYtd3M2UC1UNHdHT1JmMGZjakxfWVZEdC00SWNveXo0IiwiYnJhbmNoIjoiZGl2aWRlIiwicmVwb093bmVyIjoicHVsbGZyb2dhaSIsInJlcG9OYW1lIjoic2NyYXRjaCIsInByTnVtYmVyIjo0OSwiY29tbWl0U2hhIjoiNThiOGJmNmQ1MWE1Mjg4OGFjNGFkNzA5YWVmYTk2MWFkZDMyNDBiMSJ9LCJpYXQiOjE3NjM2MjA4MTksImV4cCI6MTc2NDIyNTYxOX0.SFDZe8R9uwhPjS55J4i_mV2ybsSZoQYM6YzdUOava4IKy1IK2OrVkVsG3-8p4rRaMBXdDZZ4ObPbtk70KqdAiLEDKBqaFcWqELc49lr0XRKUmu4F6EhESFQOvt7MLSVDIOgee8YRlhS6xtoPDqsRiV2KGOwyLEdCeYdrYz9i1DanIswWSoMRVvkjxZ6GUBYVAUg_JsgAXoKVJ-L9Q5Ygho6acVAr5NlGeBp2f6g49GX4GfDOPeV3SORQS1CjxQVRbjI-g0rW55NIisBEl8279VwG6-dTISNbyasZOB6R3eEmC4vmyAAGJjUsMwqhMPw1oaMMmYNSbtZLDESxME9IUg"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/fix-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/fix-in-web-light.svg"><img alt="Fix in Web" src="https://cursor.com/fix-in-web.svg"></picture></a>
+
+
+\`\`\`\`
+
+\`\`\`diff file=.github/workflows/test.yml lines=7 side=RIGHT
+@@ -0,0 +1,36 @@
+... (3 lines above) ...
++  push:
++    branches: [main]
++  pull_request:
++    branches: [main]
+\`\`\`
+"
+`;
+
+exports[`formatReviewData > formats thread blocks with TOC and correct line numbers > toc 1`] = `"- .github/workflows/test.yml:7 → lines 25-52"`;

package/src/mcp/arkConfig.ts

@@ -0,0 +1,7 @@
+import { configure } from "arktype/config";
+
+configure({
+  toJsonSchema: {
+    dialect: null,
+  },
+});

package/src/mcp/checkSuite.test.ts

@@ -0,0 +1,245 @@
+import { mkdirSync, writeFileSync } from "node:fs";
+import { join } from "node:path";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { analyzeLog, GetCheckSuiteLogsTool } from "#app/mcp/checkSuite";
+import type { ToolContext } from "#app/mcp/server";
+
+vi.mock("node:fs", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("node:fs")>();
+  return { ...actual, default: actual, mkdirSync: vi.fn(), writeFileSync: vi.fn() };
+});
+
+type ToolResultShape = { content: [{ type: "text"; text: string }]; isError?: boolean };
+
+async function runTool(t: { execute?: unknown }, params: unknown): Promise<ToolResultShape> {
+  const exec = t.execute as (p: unknown, c: unknown) => Promise<ToolResultShape>;
+  return exec(params, {});
+}
+
+type WorkflowRun = { id: number; conclusion: string };
+type Job = {
+  id: number;
+  name: string;
+  conclusion: string;
+  html_url?: string | null;
+  steps?: { number: number; name: string; conclusion: string }[];
+};
+
+function makeCtx(runs: WorkflowRun[], jobs: Job[]) {
+  const rest = {
+    actions: {
+      listWorkflowRunsForRepo: { endpoint: "runs" },
+      listJobsForWorkflowRun: { endpoint: "jobs" },
+      downloadJobLogsForWorkflowRun: vi.fn(async (_p: unknown) => ({
+        url: "https://logs.example/job",
+      })),
+    },
+  };
+  const paginate = vi.fn(async (endpoint: unknown) => {
+    if (endpoint === rest.actions.listWorkflowRunsForRepo) return runs;
+    if (endpoint === rest.actions.listJobsForWorkflowRun) return jobs;
+    throw new Error("unexpected paginate endpoint");
+  });
+  const octokit = { rest, paginate };
+  const ctx = {
+    octokit,
+    repo: { owner: "octo", name: "repo" },
+  } as unknown as ToolContext;
+  return { ctx, octokit };
+}
+
+beforeEach(() => {
+  vi.clearAllMocks();
+  vi.stubEnv("TERRAMEND_TEMP_DIR", join("/tmp", "terramend-test"));
+  vi.stubGlobal(
+    "fetch",
+    vi.fn(async () => ({ ok: true, text: async () => "##[error] boom" })),
+  );
+});
+
+afterEach(() => {
+  vi.unstubAllEnvs();
+  vi.unstubAllGlobals();
+});
+
+describe("analyzeLog", () => {
+  it("indexes error, warning, failure, and trace lines with 1-based line numbers", () => {
+    const logs = [
+      "setup ok",
+      "##[error]Process completed with exit code 1.",
+      "##[warning]node version is old",
+      "Tests: 2 failed, 5 passed",
+      "    at Object.<anonymous> (test.ts:1:1)",
+    ].join("\n");
+    const { index } = analyzeLog(logs);
+    expect(index).toEqual([
+      { line: 2, content: "##[error]Process completed with exit code 1.", type: "error" },
+      { line: 3, content: "##[warning]node version is old", type: "warning" },
+      { line: 4, content: "Tests: 2 failed, 5 passed", type: "failure" },
+      { line: 5, content: "at Object.<anonymous> (test.ts:1:1)", type: "trace" },
+    ]);
+  });
+
+  it("strips ANSI escape codes before matching", () => {
+    const logs = "\x1b[31mError: red alert\x1b[0m";
+    const { index } = analyzeLog(logs);
+    expect(index).toEqual([{ line: 1, content: "Error: red alert", type: "error" }]);
+  });
+
+  it("skips apt/dpkg WARN noise but keeps real warnings", () => {
+    const logs = ["WARN apt does not have a stable CLI", "WARN deprecated dependency"].join("\n");
+    const { index } = analyzeLog(logs);
+    expect(index).toEqual([{ line: 2, content: "WARN deprecated dependency", type: "warning" }]);
+  });
+
+  it("dedupes consecutive stack-trace lines", () => {
+    const logs = [
+      "Error: kaput",
+      "    at first (a.ts:1:1)",
+      "    at second (b.ts:2:2)",
+      "FAIL src/x.test.ts",
+      "    at third (c.ts:3:3)",
+    ].join("\n");
+    const { index } = analyzeLog(logs);
+    const traces = index.filter((l) => l.type === "trace");
+    expect(traces).toHaveLength(2);
+    expect(traces[0]?.content).toContain("first");
+    expect(traces[1]?.content).toContain("third");
+  });
+
+  it("truncates indexed lines longer than 120 characters", () => {
+    const long = `Error: ${"x".repeat(150)}`;
+    const { index } = analyzeLog(long);
+    expect(index[0]?.content.length).toBeLessThanOrEqual(120);
+    expect(index[0]?.content.endsWith("...")).toBe(true);
+  });
+
+  it("centers the excerpt on the LAST ##[error] line", () => {
+    const lines = Array.from({ length: 200 }, (_, i) => `line ${i + 1}`);
+    lines[49] = "##[error]first failure";
+    lines[149] = "##[error]second failure";
+    const { excerpt } = analyzeLog(lines.join("\n"), 80);
+    // last error at index 149 → start = 149 - 75 = 74 (line 75), end = 149 + 5 = 154
+    expect(excerpt.startLine).toBe(75);
+    expect(excerpt.endLine).toBe(154);
+    expect(excerpt.content).toContain("##[error]second failure");
+    expect(excerpt.content).not.toContain("##[error]first failure");
+  });
+
+  it("falls back to the tail window when no ##[error] exists", () => {
+    const lines = Array.from({ length: 100 }, (_, i) => `line ${i + 1}`);
+    const result = analyzeLog(lines.join("\n"), 10);
+    expect(result.totalLines).toBe(100);
+    expect(result.excerpt.startLine).toBe(91);
+    expect(result.excerpt.endLine).toBe(100);
+    expect(result.excerpt.content).toContain("line 100");
+  });
+
+  it("clamps the excerpt start for an error near the top of the log", () => {
+    const logs = ["##[error]early", "after"].join("\n");
+    const { excerpt } = analyzeLog(logs, 80);
+    expect(excerpt.startLine).toBe(1);
+  });
+});
+
+describe("GetCheckSuiteLogsTool", () => {
+  it("returns a no-failures message when no workflow run failed", async () => {
+    const { ctx, octokit } = makeCtx([{ id: 1, conclusion: "success" }], []);
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBeUndefined();
+    expect(result.content[0].text).toContain("no failed workflow runs found");
+    expect(octokit.paginate).toHaveBeenCalledTimes(1);
+    expect(mkdirSync).not.toHaveBeenCalled();
+  });
+
+  it("errors when TERRAMEND_TEMP_DIR is not set", async () => {
+    vi.stubEnv("TERRAMEND_TEMP_DIR", "");
+    const { ctx } = makeCtx([{ id: 1, conclusion: "failure" }], []);
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBe(true);
+    expect(result.content[0].text).toContain("TERRAMEND_TEMP_DIR not set");
+  });
+
+  it("downloads, persists, and analyzes the logs of each failed job", async () => {
+    const { ctx, octokit } = makeCtx(
+      [
+        { id: 1, conclusion: "failure" },
+        { id: 2, conclusion: "success" },
+      ],
+      [
+        {
+          id: 11,
+          name: "build",
+          conclusion: "failure",
+          html_url: "https://gh/job/11",
+          steps: [
+            { number: 1, name: "checkout", conclusion: "success" },
+            { number: 2, name: "test", conclusion: "failure" },
+          ],
+        },
+        { id: 12, name: "lint", conclusion: "success" },
+      ],
+    );
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBeUndefined();
+    expect(octokit.rest.actions.downloadJobLogsForWorkflowRun).toHaveBeenCalledTimes(1);
+    expect(octokit.rest.actions.downloadJobLogsForWorkflowRun).toHaveBeenCalledWith(
+      expect.objectContaining({ owner: "octo", repo: "repo", job_id: 11 }),
+    );
+    const logPath = join(join("/tmp", "terramend-test"), "ci-logs", "job-11.log");
+    expect(mkdirSync).toHaveBeenCalledWith(join(join("/tmp", "terramend-test"), "ci-logs"), {
+      recursive: true,
+    });
+    expect(writeFileSync).toHaveBeenCalledWith(logPath, "##[error] boom");
+    const text = result.content[0].text;
+    expect(text).toContain("build");
+    expect(text).toContain("Step 2: test");
+    expect(text).not.toContain("Step 1: checkout");
+    expect(text).toContain("job-11.log");
+    expect(text).toContain("##[error] boom");
+  });
+
+  it("defaults html_url and failed_steps when the job omits them", async () => {
+    const { ctx } = makeCtx(
+      [{ id: 1, conclusion: "failure" }],
+      [{ id: 11, name: "build", conclusion: "failure", html_url: null }],
+    );
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBeUndefined();
+    expect(result.content[0].text).toContain("failed_steps: []");
+  });
+
+  it("skips a job whose log fetch returns a non-OK response", async () => {
+    vi.stubGlobal(
+      "fetch",
+      vi.fn(async () => ({ ok: false, status: 410, statusText: "Gone", text: async () => "" })),
+    );
+    const { ctx } = makeCtx(
+      [{ id: 1, conclusion: "failure" }],
+      [{ id: 11, name: "build", conclusion: "failure" }],
+    );
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBeUndefined();
+    expect(result.content[0].text).toContain("failed_jobs: []");
+    expect(writeFileSync).not.toHaveBeenCalled();
+  });
+
+  it("skips a job whose log-URL request rejects", async () => {
+    const { ctx, octokit } = makeCtx(
+      [{ id: 1, conclusion: "failure" }],
+      [{ id: 11, name: "build", conclusion: "failure" }],
+    );
+    octokit.rest.actions.downloadJobLogsForWorkflowRun.mockRejectedValueOnce(
+      new Error("api exploded"),
+    );
+    const result = await runTool(GetCheckSuiteLogsTool(ctx), { check_suite_id: 77 });
+
+    expect(result.isError).toBeUndefined();
+    expect(result.content[0].text).toContain("failed_jobs: []");
+  });
+});