ira-review 3.0.1 → 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;
 }

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;
 }