@krotovm/gitlab-ai-review 1.0.28 → 1.0.29

package/README.md

@@ -23,7 +23,7 @@ stages: [review]
 
 ai_review:
   stage: review
-  image: node:20
+  image: node:24
   rules:
     - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
   script:
@@ -37,7 +37,7 @@ stages: [review]
 
 ai_review:
   stage: review
-  image: node:20
+  image: node:24
   rules:
     - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
   script:

package/dist/cli/ci-review.js

@@ -637,12 +637,14 @@ export async function reviewMergeRequestMultiPass(params) {
         });
     }
     const triageMap = new Map(triageResult.files.map((f) => [f.path, f.verdict]));
-    const reviewFiles = changes.filter((c) => triageMap.get(c.new_path) !== "SKIP");
+    let reviewFiles = changes.filter((c) => triageMap.get(c.new_path) !== "SKIP");
     const skippedCount = changes.length - reviewFiles.length;
-    logStep(`Triage: ${reviewFiles.length} file(s) to review, ${skippedCount} skipped. Summary: ${triageResult.summary.slice(0, 120)}...`);
     if (reviewFiles.length === 0) {
-        const DISCLAIMER = "This comment was generated by AI review bot.";
-        return `No confirmed bugs or high-value optimizations found.\n\n---\n_${DISCLAIMER}_`;
+        logStep(`Triage wanted to skip all ${changes.length} file(s) — overriding to review all. Summary: ${triageResult.summary.slice(0, 120)}...`);
+        reviewFiles = changes;
+    }
+    else {
+        logStep(`Triage: ${reviewFiles.length} file(s) to review, ${skippedCount} skipped. Summary: ${triageResult.summary.slice(0, 120)}...`);
     }
     logStep(`Pass 2/4: reviewing ${reviewFiles.length} file(s) (concurrency=${reviewConcurrency})`);
     const allChangedPaths = changes.map((c) => c.new_path);

package/dist/prompt/templates/triage-system.js

@@ -10,8 +10,12 @@ export const TRIAGE_SYSTEM_LINES = [
     "",
     "Rules:",
     "- When in doubt, verdict is NEEDS_REVIEW.",
+    "- Look beyond comments and JSDoc: if the diff changes function signatures, return types, control flow, variable assignments, or adds/removes code lines (not just comments), it is a logic change → NEEDS_REVIEW.",
+    "- If a file mixes doc/comment edits with actual code changes, verdict is NEEDS_REVIEW.",
+    "- Refactoring (splitting/merging functions, changing data structures, renaming with signature changes) is NEEDS_REVIEW.",
     "- Deleted files are SKIP unless the deletion could break dependents.",
     "- New files containing logic are NEEDS_REVIEW.",
-    "- Test-only files are SKIP unless they cover security-critical or complex logic.",
+    "- Test files that add, remove, or change assertions or expected values are NEEDS_REVIEW.",
+    "- Test-only files are SKIP only if changes are purely cosmetic (formatting, comments).",
     "- Config/CI/docs files are SKIP unless they modify build targets, env vars, or secrets.",
 ];

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "@krotovm/gitlab-ai-review",
-  "version": "1.0.28",
+  "version": "1.0.29",
   "description": "CLI tool to generate AI code reviews for GitLab merge requests.",
   "main": "dist/cli.js",
   "bin": {