ira-review 3.0.0 → 3.0.2

package/README.github.md

@@ -182,7 +182,7 @@ Each rule has a `message` (what to tell the developer), a `severity` (BLOCKER, C
 }
 ```
 
-Rules without `paths` apply to all files. Rules with `paths` are only checked against matching files. The file is validated at load time: invalid severity values and missing required fields are skipped with a warning. Maximum 100 rules per file. IRA rules are for nuanced, context-dependent standards that linters cannot express. Deterministic checks (naming conventions, import order, formatting) belong in ESLint.
+Rules without `paths` apply to all files. Rules with `paths` are only checked against matching files. The file is validated at load time: invalid severity values and missing required fields are skipped with a warning. There is no hard cap on the number of rules; a soft warning is logged above 500 since large rulesets can inflate the AI prompt. IRA rules are for nuanced, context-dependent standards that linters cannot express. Deterministic checks (naming conventions, import order, formatting) belong in ESLint.
 
 Rules are enforced in all review surfaces (CLI, CI/CD, VS Code extension) with no license gating. In the VS Code extension, run `IRA: Init Rules File` from the command palette to scaffold an empty `.ira-rules.json`. The extension ships a JSON Schema for the file, so you get autocomplete and validation as you edit.
 
@@ -279,10 +279,10 @@ Suggested Fix: Use parameterized queries:
 
 1. Install from the [VS Code Marketplace](https://marketplace.visualstudio.com/items?itemName=ira-review.ira-review-vscode)
 2. Open a project with a GitHub or Bitbucket remote
-3. `Cmd+Shift+P` > `IRA: Review Current PR`
-4. Enter your PR number
+3. `Cmd+Shift+P` > `IRA: Quick Start` (auto-detects SCM, walks you through tokens, optionally sets up JIRA)
+4. `Cmd+Shift+P` > `IRA: Review Current PR` and enter your PR number
 
-If you have GitHub Copilot, that is all you need. No API keys, no configuration. Alternatively, set the AI provider to `amp` if you have the AMP CLI installed (`amp login`).
+If you have GitHub Copilot, Quick Start finishes in seconds — no API keys, no configuration. Alternatively, set the AI provider to `amp` if you have the AMP CLI installed (`amp login`). For Bitbucket Server / Data Center or JIRA, Quick Start gives you the right token-creation links inline.
 
 ### CLI
 

package/README.md

@@ -78,7 +78,7 @@ Commit a `.ira-rules.json` to your repo root. Rules are injected into the AI pro
 **Rules:**
 - `message` + `severity` required. `bad`/`good` examples and `paths` are optional.
 - Rules without `paths` apply to all files. Rules with `paths` match only those directories.
-- Maximum 100 rules. Deterministic checks (naming, formatting) belong in ESLint.
+- No hard cap on rules (soft warning above 500). Deterministic checks (naming, formatting) belong in ESLint.
 - Invalid rules are skipped with a warning, not a crash.
 - No license gating. Works in CLI, CI/CD, and VS Code extension.
 

package/README.npm.md

@@ -78,7 +78,7 @@ Commit a `.ira-rules.json` to your repo root. Rules are injected into the AI pro
 **Rules:**
 - `message` + `severity` required. `bad`/`good` examples and `paths` are optional.
 - Rules without `paths` apply to all files. Rules with `paths` match only those directories.
-- Maximum 100 rules. Deterministic checks (naming, formatting) belong in ESLint.
+- No hard cap on rules (soft warning above 500). Deterministic checks (naming, formatting) belong in ESLint.
 - Invalid rules are skipped with a warning, not a crash.
 - No license gating. Works in CLI, CI/CD, and VS Code extension.
 

package/dist/cli.js

@@ -513,7 +513,7 @@ function annotateDiffWithLineNumbers(diff) {
 import { readFileSync as readFileSync4, existsSync as existsSync6 } from "fs";
 import { resolve } from "path";
 var VALID_SEVERITIES = ["BLOCKER", "CRITICAL", "MAJOR", "MINOR"];
-var MAX_RULES = 100;
+var RULES_SOFT_WARN_THRESHOLD = 500;
 function loadRawRulesFile(cwd) {
   const dir = cwd ?? process.cwd();
   const filePath = resolve(dir, ".ira-rules.json");
@@ -569,9 +569,8 @@ function loadRulesFile(cwd) {
       ...typeof rule.createdAt === "string" && { createdAt: rule.createdAt }
     });
   }
-  if (valid.length > MAX_RULES) {
-    console.warn(`IRA: .ira-rules.json has more than ${MAX_RULES} rules. Only the first ${MAX_RULES} will be enforced. Tip: Move deterministic rules to ESLint and keep only nuanced, context-dependent rules in IRA.`);
-    return valid.slice(0, MAX_RULES);
+  if (valid.length > RULES_SOFT_WARN_THRESHOLD) {
+    console.warn(`IRA: .ira-rules.json has ${valid.length} rules (>${RULES_SOFT_WARN_THRESHOLD}). All will be enforced, but large rulesets can inflate the AI prompt and risk hitting model context limits. Tip: Move deterministic rules to ESLint and keep only nuanced, context-dependent rules in IRA.`);
   }
   return valid;
 }
@@ -1006,12 +1005,15 @@ var CommentTracker = class {
   provider;
   baseUrl;
   headers;
-  // Bitbucket
+  // Bitbucket Cloud
   workspace;
   repoSlug;
   // GitHub
   owner;
   repo;
+  // Bitbucket Server
+  project;
+  bbServerRepoSlug;
   constructor(config, provider = "bitbucket") {
     this.provider = provider;
     if (provider === "github") {
@@ -1024,6 +1026,15 @@ var CommentTracker = class {
         Accept: "application/vnd.github.v3+json",
         "Content-Type": "application/json"
       };
+    } else if (provider === "bitbucket-server") {
+      const bbs = config;
+      this.baseUrl = bbs.baseUrl.replace(/\/+$/, "");
+      this.project = bbs.project;
+      this.bbServerRepoSlug = bbs.repoSlug;
+      this.headers = {
+        Authorization: `Bearer ${bbs.token}`,
+        "Content-Type": "application/json"
+      };
     } else {
       const bb = config;
       this.baseUrl = (bb.baseUrl ?? "https://api.bitbucket.org/2.0").replace(/\/+$/, "");
@@ -1039,6 +1050,9 @@ var CommentTracker = class {
     if (this.provider === "github") {
       return this.getGitHubIraComments(pullRequestId);
     }
+    if (this.provider === "bitbucket-server") {
+      return this.getBitbucketServerIraComments(pullRequestId);
+    }
     return this.getBitbucketIraComments(pullRequestId);
   }
   async getBitbucketIraComments(pullRequestId) {
@@ -1098,6 +1112,37 @@ var CommentTracker = class {
     }
     return keys;
   }
+  async getBitbucketServerIraComments(pullRequestId) {
+    const keys = /* @__PURE__ */ new Set();
+    let start = 0;
+    while (true) {
+      const url = `${this.baseUrl}/rest/api/1.0/projects/${this.project}/repos/${this.bbServerRepoSlug}/pull-requests/${pullRequestId}/comments?start=${start}&limit=100`;
+      const data = await this.fetchBitbucketServerPage(url);
+      for (const comment of data.values) {
+        if (!comment.text.includes(IRA_MARKER)) continue;
+        const meta = comment.text.match(IRA_META_RE);
+        if (meta) {
+          keys.add(`${meta[1]}:${meta[2]}:${meta[3]}`);
+        }
+      }
+      if (data.isLastPage) break;
+      start = data.nextPageStart ?? start + 100;
+    }
+    return keys;
+  }
+  async fetchBitbucketServerPage(url) {
+    return withRetry(async () => {
+      const response = await fetchWithTimeout(url, { headers: this.headers });
+      if (!response.ok) {
+        const body = await response.text();
+        throw new RetryableError(
+          parseApiError(response.status, body, "Bitbucket Server"),
+          response.status
+        );
+      }
+      return await response.json();
+    });
+  }
   async fetchBitbucketPage(url) {
     return withRetry(async () => {
       const response = await fetchWithTimeout(url, { headers: this.headers });

package/dist/index.cjs

@@ -747,7 +747,7 @@ function annotateDiffWithLineNumbers(diff) {
 var import_node_fs6 = require("fs");
 var import_node_path6 = require("path");
 var VALID_SEVERITIES = ["BLOCKER", "CRITICAL", "MAJOR", "MINOR"];
-var MAX_RULES = 100;
+var RULES_SOFT_WARN_THRESHOLD = 500;
 function loadRawRulesFile(cwd) {
   const dir = cwd ?? process.cwd();
   const filePath = (0, import_node_path6.resolve)(dir, ".ira-rules.json");
@@ -803,9 +803,8 @@ function loadRulesFile(cwd) {
       ...typeof rule.createdAt === "string" && { createdAt: rule.createdAt }
     });
   }
-  if (valid.length > MAX_RULES) {
-    console.warn(`IRA: .ira-rules.json has more than ${MAX_RULES} rules. Only the first ${MAX_RULES} will be enforced. Tip: Move deterministic rules to ESLint and keep only nuanced, context-dependent rules in IRA.`);
-    return valid.slice(0, MAX_RULES);
+  if (valid.length > RULES_SOFT_WARN_THRESHOLD) {
+    console.warn(`IRA: .ira-rules.json has ${valid.length} rules (>${RULES_SOFT_WARN_THRESHOLD}). All will be enforced, but large rulesets can inflate the AI prompt and risk hitting model context limits. Tip: Move deterministic rules to ESLint and keep only nuanced, context-dependent rules in IRA.`);
   }
   return valid;
 }
@@ -1796,12 +1795,15 @@ var CommentTracker = class {
   provider;
   baseUrl;
   headers;
-  // Bitbucket
+  // Bitbucket Cloud
   workspace;
   repoSlug;
   // GitHub
   owner;
   repo;
+  // Bitbucket Server
+  project;
+  bbServerRepoSlug;
   constructor(config, provider = "bitbucket") {
     this.provider = provider;
     if (provider === "github") {
@@ -1814,6 +1816,15 @@ var CommentTracker = class {
         Accept: "application/vnd.github.v3+json",
         "Content-Type": "application/json"
       };
+    } else if (provider === "bitbucket-server") {
+      const bbs = config;
+      this.baseUrl = bbs.baseUrl.replace(/\/+$/, "");
+      this.project = bbs.project;
+      this.bbServerRepoSlug = bbs.repoSlug;
+      this.headers = {
+        Authorization: `Bearer ${bbs.token}`,
+        "Content-Type": "application/json"
+      };
     } else {
       const bb = config;
       this.baseUrl = (bb.baseUrl ?? "https://api.bitbucket.org/2.0").replace(/\/+$/, "");
@@ -1829,6 +1840,9 @@ var CommentTracker = class {
     if (this.provider === "github") {
       return this.getGitHubIraComments(pullRequestId);
     }
+    if (this.provider === "bitbucket-server") {
+      return this.getBitbucketServerIraComments(pullRequestId);
+    }
     return this.getBitbucketIraComments(pullRequestId);
   }
   async getBitbucketIraComments(pullRequestId) {
@@ -1888,6 +1902,37 @@ var CommentTracker = class {
     }
     return keys;
   }
+  async getBitbucketServerIraComments(pullRequestId) {
+    const keys = /* @__PURE__ */ new Set();
+    let start = 0;
+    while (true) {
+      const url = `${this.baseUrl}/rest/api/1.0/projects/${this.project}/repos/${this.bbServerRepoSlug}/pull-requests/${pullRequestId}/comments?start=${start}&limit=100`;
+      const data = await this.fetchBitbucketServerPage(url);
+      for (const comment of data.values) {
+        if (!comment.text.includes(IRA_MARKER)) continue;
+        const meta = comment.text.match(IRA_META_RE);
+        if (meta) {
+          keys.add(`${meta[1]}:${meta[2]}:${meta[3]}`);
+        }
+      }
+      if (data.isLastPage) break;
+      start = data.nextPageStart ?? start + 100;
+    }
+    return keys;
+  }
+  async fetchBitbucketServerPage(url) {
+    return withRetry(async () => {
+      const response = await fetchWithTimeout(url, { headers: this.headers });
+      if (!response.ok) {
+        const body = await response.text();
+        throw new RetryableError(
+          parseApiError(response.status, body, "Bitbucket Server"),
+          response.status
+        );
+      }
+      return await response.json();
+    });
+  }
   async fetchBitbucketPage(url) {
     return withRetry(async () => {
       const response = await fetchWithTimeout(url, { headers: this.headers });