@krotovm/gitlab-ai-review 1.0.37 → 1.0.38

package/dist/cli/ci-review.js

@@ -827,6 +827,10 @@ export async function reviewMergeRequestMultiPass(params) {
     if (failedPaths.length === reviewFiles.length) {
         throw new Error(`All ${reviewFiles.length} per-file reviews failed (e.g. ${failedPaths[0]}). See log above.`);
     }
+    // Deterministic note header: title + triage summary. Prepended in code so
+    // every final-answer path gets it regardless of model formatting discipline.
+    const summaryText = triageResult.summary.trim().replace(/\s*\n\s*/g, " ");
+    const withHeader = (body) => `### 🤖 AI Code Review\n\n${summaryText === "" ? "" : `> ${summaryText}\n\n`}${body}`;
     logStep("Pass 3/4: consolidating findings");
     const consolidateMessages = buildConsolidatePrompt({
         perFileFindings,
@@ -836,7 +840,7 @@ export async function reviewMergeRequestMultiPass(params) {
     });
     if (consolidateMessages == null) {
         const DISCLAIMER = "This comment was generated by AI review bot.";
-        return `${NO_FINDINGS_SENTENCE}\n\n---\n_${DISCLAIMER}_`;
+        return withHeader(`${NO_FINDINGS_SENTENCE}\n\n---\n_${DISCLAIMER}_`);
     }
     try {
         const consolidateCompletion = await createCompletionWithDebug({
@@ -853,7 +857,7 @@ export async function reviewMergeRequestMultiPass(params) {
         });
         const consolidatedText = extractCompletionText(consolidateCompletion);
         if (consolidatedText == null || consolidatedText.trim() === "") {
-            return buildAnswer(consolidateCompletion);
+            return withHeader(buildAnswer(consolidateCompletion));
         }
         logStep("Pass 4/4: verifying consolidated findings (repo tools)");
         const verificationMessages = buildVerificationPrompt({
@@ -879,11 +883,11 @@ export async function reviewMergeRequestMultiPass(params) {
                 debugDumpFile,
                 debugRecordWriter,
             });
-            return buildAnswer(verificationCompletion);
+            return withHeader(buildAnswer(verificationCompletion));
         }
         catch (error) {
             logStep(`Verification failed: ${error?.message ?? error}. Returning consolidated findings.`);
-            return buildAnswer(consolidateCompletion);
+            return withHeader(buildAnswer(consolidateCompletion));
         }
     }
     catch (error) {
@@ -893,6 +897,6 @@ export async function reviewMergeRequestMultiPass(params) {
             .filter((f) => !isEmptyReviewBody(f.findings))
             .map((f) => f.findings)
             .join("\n");
-        return `${raw || NO_FINDINGS_SENTENCE}\n\n---\n_${DISCLAIMER}_`;
+        return withHeader(`${raw || NO_FINDINGS_SENTENCE}\n\n---\n_${DISCLAIMER}_`);
     }
 }

package/dist/prompt/utils.js

@@ -70,11 +70,16 @@ export function normalizeReviewFindingsMarkdown(input) {
     const lines = expanded.split("\n");
     const findings = [];
     const headerRe = /^\s*(?:[-*•]\s*)?\[(high|medium)\]\s+(.+?)\s*$/i;
+    // Already-rendered block (see renderer below) — parsing it too keeps this
+    // function idempotent when a model echoes pretty findings back verbatim.
+    const prettyHeaderRe = /^\s*(?:[-*•]\s*)?\*\*(?:🔴|🟠)?\s*(high|medium)\s*[—–-]\s*(.+?)\*\*\s*$/i;
+    const matchHeader = (line) => line.match(headerRe) ?? line.match(prettyHeaderRe);
     const fileRe = /^\s*[-*]?\s*File:\s*(.+?)\s*$/i;
     const lineRe = /^\s*[-*]?\s*Line:\s*(.+?)\s*$/i;
     const whyRe = /^\s*[-*]?\s*Why:\s*(.+?)\s*$/i;
+    const prettyFileLineRe = /^\s*`([^`\n]+?):(~?[^`\n]*)`\s*$/;
     for (let i = 0; i < lines.length; i += 1) {
-        const headerMatch = lines[i].match(headerRe);
+        const headerMatch = matchHeader(lines[i]);
         if (headerMatch == null)
             continue;
         const severity = headerMatch[1].toLowerCase();
@@ -84,9 +89,18 @@ export function normalizeReviewFindingsMarkdown(input) {
         let why = null;
         let j = i + 1;
         while (j < lines.length) {
-            const nextHeader = lines[j].match(headerRe);
+            const nextHeader = matchHeader(lines[j]);
             if (nextHeader != null)
                 break;
+            if (file == null && line == null) {
+                const m = lines[j].match(prettyFileLineRe);
+                if (m != null) {
+                    file = m[1].trim();
+                    line = m[2].trim();
+                    j += 1;
+                    continue;
+                }
+            }
             if (file == null) {
                 const m = lines[j].match(fileRe);
                 if (m != null) {
@@ -111,6 +125,15 @@ export function normalizeReviewFindingsMarkdown(input) {
                     continue;
                 }
             }
+            // Pretty blocks carry the why as an unlabeled line after `file:line`.
+            if (why == null && file != null && line != null) {
+                const text = lines[j].trim();
+                if (text !== "" && !/^(-{3,}|_)/.test(text)) {
+                    why = text;
+                    j += 1;
+                    continue;
+                }
+            }
             j += 1;
         }
         if (file != null && line != null && why != null) {
@@ -120,7 +143,11 @@ export function normalizeReviewFindingsMarkdown(input) {
     }
     if (findings.length === 0)
         return normalized;
+    const severityLabel = {
+        high: "🔴 High",
+        medium: "🟠 Medium",
+    };
     return findings
-        .map((f) => `- [${f.severity}] ${f.title}  \n  File: ${f.file}  \n  Line: ${f.line}  \n  Why: ${f.why}`)
+        .map((f) => `- **${severityLabel[f.severity]} — ${f.title}**  \n  \`${f.file}:${f.line}\`  \n  ${f.why}`)
         .join("\n\n");
 }

package/package.json

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