reviewflow 3.42.2 → 3.43.1

package/CHANGELOG.md

@@ -5,12 +5,32 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.43.1](https://github.com/DGouron/review-flow/compare/reviewflow-v3.43.0...reviewflow-v3.43.1) (2026-07-15)
+
+
+### Fixed
+
+* **webhook:** soften the oversized-MR split message tone ([#342](https://github.com/DGouron/review-flow/issues/342)) ([27a967f](https://github.com/DGouron/review-flow/commit/27a967f6e129998e3d6049269770a221ebf0c68f))
+
+## [3.43.0](https://github.com/DGouron/review-flow/compare/reviewflow-v3.42.2...reviewflow-v3.43.0) (2026-07-14)
+
+
+### Added
+
+* **templates:** add review-advanced and followup-advanced skill templates ([#337](https://github.com/DGouron/review-flow/issues/337)) ([f4aa52e](https://github.com/DGouron/review-flow/commit/f4aa52e77555a13e93492010a8bd165a3bfc6bbd))
+
+
+### Fixed
+
+* [#339](https://github.com/DGouron/review-flow/issues/339) dashboard loading state stuck after successful fetch ([#340](https://github.com/DGouron/review-flow/issues/340)) ([45bce47](https://github.com/DGouron/review-flow/commit/45bce4789215cb24dd556800932734e3fa6dc0e8))
+
 ## [3.42.2](https://github.com/DGouron/review-flow/compare/reviewflow-v3.42.1...reviewflow-v3.42.2) (2026-07-13)
 
 
 ### Fixed
 
 * **claude-invocation:** strip ANSI codes before parsing --bg session id ([#334](https://github.com/DGouron/review-flow/issues/334)) ([88c66e3](https://github.com/DGouron/review-flow/commit/88c66e31b07988b3f275c803722474048e4e82a9))
+* **setup:** write the project review-config shape the review engine actually expects ([#333](https://github.com/DGouron/review-flow/issues/333)) ([34c6c8b](https://github.com/DGouron/review-flow/commit/34c6c8b7cd6478f7aa036aca3a8c750ea8865716))
 
 ## [3.42.1](https://github.com/DGouron/review-flow/compare/reviewflow-v3.42.0...reviewflow-v3.42.1) (2026-07-13)
 

package/dist/dashboard/index.html

@@ -1046,11 +1046,11 @@
         const response = await fetch(url);
         const data = await response.json();
         currentData.reviewFiles = data.reviews || [];
-        updateReviewFilesUI();
       } catch (error) {
         console.error('Error fetching review files:', error);
       } finally {
         setLoadingFlag('reviewFiles', false);
+        updateReviewFilesUI();
       }
     }
 
@@ -1990,11 +1990,11 @@
           currentData.pendingApproval = data.pendingApproval || [];
           currentData.merged = data.merged || [];
         }
-        updateMrTrackingUI();
       } catch (error) {
         console.error('Error fetching MR tracking:', error);
       } finally {
         setLoadingFlag('mrTracking', false);
+        updateMrTrackingUI();
       }
     }
 

package/dist/modules/platform-integration/usecases/guardDiffSize.usecase.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"guardDiffSize.usecase.d.ts","sourceRoot":"","sources":["../../../../src/modules/platform-integration/usecases/guardDiffSize.usecase.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,4EAA4E,CAAC;AAE3H,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qCAAqC,CAAC;AAEnE,UAAU,kBAAkB;IAC1B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,kBAAkB,EAAE,MAAM,CAAC;IAC3B,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,MAAM,oBAAoB,GAC5B;IAAE,IAAI,EAAE,SAAS,CAAA;CAAE,GACnB;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/E,UAAU,yBAAyB;IACjC,wBAAwB,EAAE,wBAAwB,CAAC;CACpD;AAYD,qBAAa,oBAAqB,YAAW,OAAO,CAAC,kBAAkB,EAAE,oBAAoB,CAAC;IAChF,OAAO,CAAC,QAAQ,CAAC,YAAY;gBAAZ,YAAY,EAAE,yBAAyB;IAEpE,OAAO,CAAC,KAAK,EAAE,kBAAkB,GAAG,oBAAoB;IAuBxD,OAAO,CAAC,iBAAiB;CAU1B"}
+{"version":3,"file":"guardDiffSize.usecase.d.ts","sourceRoot":"","sources":["../../../../src/modules/platform-integration/usecases/guardDiffSize.usecase.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,4EAA4E,CAAC;AAE3H,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qCAAqC,CAAC;AAEnE,UAAU,kBAAkB;IAC1B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,kBAAkB,EAAE,MAAM,CAAC;IAC3B,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,MAAM,oBAAoB,GAC5B;IAAE,IAAI,EAAE,SAAS,CAAA;CAAE,GACnB;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/E,UAAU,yBAAyB;IACjC,wBAAwB,EAAE,wBAAwB,CAAC;CACpD;AAeD,qBAAa,oBAAqB,YAAW,OAAO,CAAC,kBAAkB,EAAE,oBAAoB,CAAC;IAChF,OAAO,CAAC,QAAQ,CAAC,YAAY;gBAAZ,YAAY,EAAE,yBAAyB;IAEpE,OAAO,CAAC,KAAK,EAAE,kBAAkB,GAAG,oBAAoB;IAuBxD,OAAO,CAAC,iBAAiB;CAU1B"}

package/dist/modules/platform-integration/usecases/guardDiffSize.usecase.js

@@ -1,10 +1,13 @@
 import { evaluateDiffSizeGate } from '../../../modules/shared-kernel/entities/diffSizeGate/diffSizeGate.js';
 function buildSplitMessage(countedLines, budget) {
-    return (`Revue refusée : cette merge request fait ${countedLines} lignes comptées ` +
-        `(budget ${budget}). Pour faciliter la revue, découpez-la : ` +
+    return (`Petite pause avant la revue : cette merge request compte ${countedLines} lignes ` +
+        `(budget ${budget}), un peu trop pour une revue de qualité en une seule fois. ` +
+        'Découpée en plus petits morceaux, elle sera revue plus vite, plus finement, ' +
+        'et bien plus simple à corriger si besoin. Une piste pour la découper : ' +
         '1) séparez les refactorings des nouvelles fonctionnalités, ' +
         '2) extrayez les changements indépendants dans des MR dédiées, ' +
-        '3) limitez chaque MR à une seule intention.');
+        '3) limitez chaque MR à une seule intention. ' +
+        'Merci pour le travail, hâte de la relire découpée !');
 }
 export class GuardDiffSizeUseCase {
     dependencies;

package/dist/modules/platform-integration/usecases/guardDiffSize.usecase.js.map

@@ -1 +1 @@
-{"version":3,"file":"guardDiffSize.usecase.js","sourceRoot":"","sources":["../../../../src/modules/platform-integration/usecases/guardDiffSize.usecase.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,oBAAoB,EAAE,MAAM,+DAA+D,CAAC;AAiBrG,SAAS,iBAAiB,CAAC,YAAoB,EAAE,MAAc;IAC7D,OAAO,CACL,4CAA4C,YAAY,mBAAmB;QAC3E,WAAW,MAAM,4CAA4C;QAC7D,6DAA6D;QAC7D,gEAAgE;QAChE,6CAA6C,CAC9C,CAAC;AACJ,CAAC;AAED,MAAM,OAAO,oBAAoB;IACF;IAA7B,YAA6B,YAAuC;QAAvC,iBAAY,GAAZ,YAAY,CAA2B;IAAG,CAAC;IAExE,OAAO,CAAC,KAAyB;QAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;QAC5C,IAAI,KAAK,KAAK,IAAI,EAAE,CAAC;YACnB,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;QAC7B,CAAC;QAED,MAAM,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,EAAE,GAAG,oBAAoB,CAAC;YAC/D,KAAK;YACL,MAAM,EAAE,KAAK,CAAC,MAAM;SACrB,CAAC,CAAC;QAEH,IAAI,CAAC,SAAS,EAAE,CAAC;YACf,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;QAC7B,CAAC;QAED,OAAO;YACL,IAAI,EAAE,SAAS;YACf,YAAY;YACZ,MAAM;YACN,OAAO,EAAE,iBAAiB,CAAC,YAAY,EAAE,MAAM,CAAC;SACjD,CAAC;IACJ,CAAC;IAEO,iBAAiB,CAAC,KAAyB;QACjD,IAAI,CAAC;YACH,OAAO,IAAI,CAAC,YAAY,CAAC,wBAAwB,CAAC,iBAAiB,CACjE,KAAK,CAAC,iBAAiB,EACvB,KAAK,CAAC,kBAAkB,CACzB,CAAC;QACJ,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;CACF"}
+{"version":3,"file":"guardDiffSize.usecase.js","sourceRoot":"","sources":["../../../../src/modules/platform-integration/usecases/guardDiffSize.usecase.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,oBAAoB,EAAE,MAAM,+DAA+D,CAAC;AAiBrG,SAAS,iBAAiB,CAAC,YAAoB,EAAE,MAAc;IAC7D,OAAO,CACL,4DAA4D,YAAY,UAAU;QAClF,WAAW,MAAM,8DAA8D;QAC/E,8EAA8E;QAC9E,yEAAyE;QACzE,6DAA6D;QAC7D,gEAAgE;QAChE,8CAA8C;QAC9C,qDAAqD,CACtD,CAAC;AACJ,CAAC;AAED,MAAM,OAAO,oBAAoB;IACF;IAA7B,YAA6B,YAAuC;QAAvC,iBAAY,GAAZ,YAAY,CAA2B;IAAG,CAAC;IAExE,OAAO,CAAC,KAAyB;QAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;QAC5C,IAAI,KAAK,KAAK,IAAI,EAAE,CAAC;YACnB,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;QAC7B,CAAC;QAED,MAAM,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,EAAE,GAAG,oBAAoB,CAAC;YAC/D,KAAK;YACL,MAAM,EAAE,KAAK,CAAC,MAAM;SACrB,CAAC,CAAC;QAEH,IAAI,CAAC,SAAS,EAAE,CAAC;YACf,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;QAC7B,CAAC;QAED,OAAO;YACL,IAAI,EAAE,SAAS;YACf,YAAY;YACZ,MAAM;YACN,OAAO,EAAE,iBAAiB,CAAC,YAAY,EAAE,MAAM,CAAC;SACjD,CAAC;IACJ,CAAC;IAEO,iBAAiB,CAAC,KAAyB;QACjD,IAAI,CAAC;YACH,OAAO,IAAI,CAAC,YAAY,CAAC,wBAAwB,CAAC,iBAAiB,CACjE,KAAK,CAAC,iBAAiB,EACvB,KAAK,CAAC,kBAAkB,CACzB,CAAC;QACJ,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;CACF"}

package/dist/tests/units/dashboard/dashboardLoadingRace.test.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Regression test for issue #339 — Priority Lanes and Completed Reviews stuck
+ * on "Loading..." forever.
+ *
+ * Structural assertions on src/dashboard/index.html via raw-string + regex,
+ * matching the precedent of dashboardLayout.test.ts (no jsdom dependency).
+ */
+export {};
+//# sourceMappingURL=dashboardLoadingRace.test.d.ts.map

package/dist/tests/units/dashboard/dashboardLoadingRace.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"dashboardLoadingRace.test.d.ts","sourceRoot":"","sources":["../../../../src/tests/units/dashboard/dashboardLoadingRace.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG"}

package/dist/tests/units/dashboard/dashboardLoadingRace.test.js

@@ -0,0 +1,59 @@
+/**
+ * Regression test for issue #339 — Priority Lanes and Completed Reviews stuck
+ * on "Loading..." forever.
+ *
+ * Structural assertions on src/dashboard/index.html via raw-string + regex,
+ * matching the precedent of dashboardLayout.test.ts (no jsdom dependency).
+ */
+import { readFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { beforeAll, describe, expect, it } from 'vitest';
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const PROJECT_ROOT = join(__dirname, '..', '..', '..', '..');
+const INDEX_HTML_PATH = join(PROJECT_ROOT, 'src', 'dashboard', 'index.html');
+function extractFunctionBody(source, functionName) {
+    const signatureIndex = source.indexOf(`function ${functionName}(`);
+    if (signatureIndex === -1)
+        return '';
+    const firstBraceIndex = source.indexOf('{', signatureIndex);
+    if (firstBraceIndex === -1)
+        return '';
+    let depth = 1;
+    let cursor = firstBraceIndex + 1;
+    while (depth > 0 && cursor < source.length) {
+        const char = source[cursor];
+        if (char === '{')
+            depth += 1;
+        if (char === '}')
+            depth -= 1;
+        cursor += 1;
+    }
+    return source.substring(signatureIndex, cursor);
+}
+describe('Issue #339 — dashboard loading state race', () => {
+    let indexHtml;
+    beforeAll(() => {
+        indexHtml = readFileSync(INDEX_HTML_PATH, 'utf-8');
+    });
+    it('fetchMrTracking renders after clearing the mrTracking loading flag, not before', () => {
+        const functionBody = extractFunctionBody(indexHtml, 'fetchMrTracking');
+        expect(functionBody).not.toBe('');
+        const clearFlagIndex = functionBody.lastIndexOf("setLoadingFlag('mrTracking', false)");
+        const renderCallIndex = functionBody.lastIndexOf('updateMrTrackingUI()');
+        expect(clearFlagIndex).toBeGreaterThan(-1);
+        expect(renderCallIndex).toBeGreaterThan(-1);
+        expect(renderCallIndex).toBeGreaterThan(clearFlagIndex);
+    });
+    it('fetchReviewFiles renders after clearing the reviewFiles loading flag, not before', () => {
+        const functionBody = extractFunctionBody(indexHtml, 'fetchReviewFiles');
+        expect(functionBody).not.toBe('');
+        const clearFlagIndex = functionBody.indexOf("setLoadingFlag('reviewFiles', false)");
+        const renderCallIndex = functionBody.lastIndexOf('updateReviewFilesUI()');
+        expect(clearFlagIndex).toBeGreaterThan(-1);
+        expect(renderCallIndex).toBeGreaterThan(-1);
+        expect(renderCallIndex).toBeGreaterThan(clearFlagIndex);
+    });
+});
+//# sourceMappingURL=dashboardLoadingRace.test.js.map

package/dist/tests/units/dashboard/dashboardLoadingRace.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"dashboardLoadingRace.test.js","sourceRoot":"","sources":["../../../../src/tests/units/dashboard/dashboardLoadingRace.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACvC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAEzC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEzD,MAAM,UAAU,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AAClD,MAAM,SAAS,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC;AACtC,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC;AAC7D,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,CAAC,CAAC;AAE7E,SAAS,mBAAmB,CAAC,MAAc,EAAE,YAAoB;IAC/D,MAAM,cAAc,GAAG,MAAM,CAAC,OAAO,CAAC,YAAY,YAAY,GAAG,CAAC,CAAC;IACnE,IAAI,cAAc,KAAK,CAAC,CAAC;QAAE,OAAO,EAAE,CAAC;IACrC,MAAM,eAAe,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;IAC5D,IAAI,eAAe,KAAK,CAAC,CAAC;QAAE,OAAO,EAAE,CAAC;IACtC,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,MAAM,GAAG,eAAe,GAAG,CAAC,CAAC;IACjC,OAAO,KAAK,GAAG,CAAC,IAAI,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC;QAC3C,MAAM,IAAI,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC;QAC5B,IAAI,IAAI,KAAK,GAAG;YAAE,KAAK,IAAI,CAAC,CAAC;QAC7B,IAAI,IAAI,KAAK,GAAG;YAAE,KAAK,IAAI,CAAC,CAAC;QAC7B,MAAM,IAAI,CAAC,CAAC;IACd,CAAC;IACD,OAAO,MAAM,CAAC,SAAS,CAAC,cAAc,EAAE,MAAM,CAAC,CAAC;AAClD,CAAC;AAED,QAAQ,CAAC,2CAA2C,EAAE,GAAG,EAAE;IACzD,IAAI,SAAiB,CAAC;IAEtB,SAAS,CAAC,GAAG,EAAE;QACb,SAAS,GAAG,YAAY,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,YAAY,GAAG,mBAAmB,CAAC,SAAS,EAAE,iBAAiB,CAAC,CAAC;QACvE,MAAM,CAAC,YAAY,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAElC,MAAM,cAAc,GAAG,YAAY,CAAC,WAAW,CAAC,qCAAqC,CAAC,CAAC;QACvF,MAAM,eAAe,GAAG,YAAY,CAAC,WAAW,CAAC,sBAAsB,CAAC,CAAC;QAEzE,MAAM,CAAC,cAAc,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5C,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,cAAc,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kFAAkF,EAAE,GAAG,EAAE;QAC1F,MAAM,YAAY,GAAG,mBAAmB,CAAC,SAAS,EAAE,kBAAkB,CAAC,CAAC;QACxE,MAAM,CAAC,YAAY,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAElC,MAAM,cAAc,GAAG,YAAY,CAAC,OAAO,CAAC,sCAAsC,CAAC,CAAC;QACpF,MAAM,eAAe,GAAG,YAAY,CAAC,WAAW,CAAC,uBAAuB,CAAC,CAAC;QAE1E,MAAM,CAAC,cAAc,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAC5C,MAAM,CAAC,eAAe,CAAC,CAAC,eAAe,CAAC,cAAc,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/units/modules/platform-integration/usecases/guardDiffSize.usecase.test.js

@@ -33,7 +33,7 @@ describe('GuardDiffSizeUseCase', () => {
         });
         expect(verdict).toEqual({ kind: 'allowed' });
     });
-    it('posts a French split message mentioning actionable tips', () => {
+    it('posts a kind French split message mentioning actionable tips, not a harsh rejection', () => {
         const { gateway, useCase } = buildUseCase();
         gateway.setResponse(42, ChangedFilesFactory.list([{ path: 'src/big.ts', additions: 2100, deletions: 0 }]));
         const verdict = useCase.execute({
@@ -43,8 +43,9 @@ describe('GuardDiffSizeUseCase', () => {
         });
         expect(verdict.kind).toBe('blocked');
         if (verdict.kind === 'blocked') {
-            expect(verdict.message).toContain('Revue refusée');
-            expect(verdict.message).toContain('découpez');
+            expect(verdict.message).not.toContain('Revue refusée');
+            expect(verdict.message).toContain('découp');
+            expect(verdict.message).toContain('Merci');
         }
     });
     it('is fail-open when the gateway throws (processed normally, no message)', () => {

package/dist/tests/units/modules/platform-integration/usecases/guardDiffSize.usecase.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"guardDiffSize.usecase.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/platform-integration/usecases/guardDiffSize.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAE9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,kEAAkE,CAAC;AACxG,OAAO,EAAE,mBAAmB,EAAE,MAAM,2CAA2C,CAAC;AAChF,OAAO,EAAE,4BAA4B,EAAE,MAAM,yCAAyC,CAAC;AAEvF,SAAS,YAAY;IAInB,MAAM,OAAO,GAAG,IAAI,4BAA4B,EAAE,CAAC;IACnD,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,IAAI,oBAAoB,CAAC,EAAE,wBAAwB,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC;AAC/F,CAAC;AAED,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,GAAG,EAAE,CAAC,CAAC,CACpF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACxC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YAC1C,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAC5C,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC,CAC/E,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAClF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;YACnD,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QAChD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC;QAEvB,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,GAAG,EAAE;QACjF,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CAAC,EAAE,EAAE,IAAI,CAAC,CAAC;QAE9B,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,GAAG,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAC/E,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,GAAG;SACZ,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAChF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"guardDiffSize.usecase.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/platform-integration/usecases/guardDiffSize.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAE9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,kEAAkE,CAAC;AACxG,OAAO,EAAE,mBAAmB,EAAE,MAAM,2CAA2C,CAAC;AAChF,OAAO,EAAE,4BAA4B,EAAE,MAAM,yCAAyC,CAAC;AAEvF,SAAS,YAAY;IAInB,MAAM,OAAO,GAAG,IAAI,4BAA4B,EAAE,CAAC;IACnD,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,IAAI,oBAAoB,CAAC,EAAE,wBAAwB,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC;AAC/F,CAAC;AAED,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,GAAG,EAAE,CAAC,CAAC,CACpF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACxC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAClC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YAC1C,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAC5C,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC,CAC/E,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qFAAqF,EAAE,GAAG,EAAE;QAC7F,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAClF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;YACvD,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;YAC5C,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAC7C,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC;QAEvB,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,GAAG,EAAE;QACjF,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CAAC,EAAE,EAAE,IAAI,CAAC,CAAC;QAE9B,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,GAAG,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAC/E,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,GAAG;SACZ,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,YAAY,EAAE,CAAC;QAC5C,OAAO,CAAC,WAAW,CACjB,EAAE,EACF,mBAAmB,CAAC,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,EAAE,CAAC,CAAC,CAChF,CAAC;QAEF,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;YAC9B,iBAAiB,EAAE,eAAe;YAClC,kBAAkB,EAAE,EAAE;YACtB,MAAM,EAAE,IAAI;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reviewflow",
-  "version": "3.42.2",
+  "version": "3.43.1",
   "description": "AI-powered code review automation for GitLab/GitHub using Claude Code",
   "type": "module",
   "main": "dist/main/server.js",

package/templates/en/followup-advanced/README.md

@@ -0,0 +1,51 @@
+# followup-advanced
+
+A follow-up review skill that never trusts a commit message — it always re-reads the actual code before resolving a thread. Matching counterpart to `review-advanced`.
+
+## Overview
+
+This template provides:
+- The same context-file protocol as `followup-basic`
+- A hard rule: a commit message is a claim, never evidence — every thread is verified against the current code at its file:line
+- New issues found during follow-up cite a real source, same format as `review-advanced`
+
+## Installation
+
+1. Copy this folder to your project:
+   ```bash
+   cp -r templates/en/followup-advanced .claude/skills/my-followup
+   ```
+
+2. Rename the skill in `SKILL.md` frontmatter
+
+3. Configure it as the follow-up skill in `.claude/reviews/config.json`:
+   ```json
+   {
+     "reviewSkill": "my-review",
+     "reviewFollowupSkill": "my-followup"
+   }
+   ```
+
+## Why It Never Trusts the Commit Message
+
+A commit message ("fix: null check added") is an author's claim, not proof the code changed as described. Messages can be wrong, incomplete, or copy-pasted from an unrelated commit. This template mandates re-reading the current code at the exact file:line of every previous thread before marking it resolved. If the code can't be read for a thread, the thread stays open — resolving on assumption is never allowed.
+
+## Matching review-advanced
+
+Use this template alongside [review-advanced](../review-advanced/) if you want the initial review and its follow-up to share the same citation requirement for any new issue raised.
+
+## Markers Used
+
+| Marker | Purpose |
+|--------|---------|
+| `[PHASE:...]` | Track review phase |
+| `[PROGRESS:...:started/completed]` | Track context, verification, scan, threads, report |
+| `[THREAD_REPLY:...]` / `[THREAD_RESOLVE:...]` | Thread management (or via context-file actions) |
+| `[POST_COMMENT:...]` | Post final report |
+| `[REVIEW_STATS:...]` | Report statistics |
+
+## See Also
+
+- [review-advanced](../review-advanced/) — Matching initial review template with the same citation format
+- [followup-basic](../followup-basic/) — Lighter follow-up template without the code-verification rule
+- [Review Skills Guide](../../../docs/guide/review-skills.md)

package/templates/en/followup-advanced/SKILL.md

@@ -0,0 +1,389 @@
+---
+name: followup-advanced
+description: Follow-up review that never trusts a commit message — always re-reads the actual diff before resolving a thread. Cites a real source for new issues, matching review-advanced.
+---
+
+# Follow-up Review (Advanced)
+
+**You are**: The same reviewer verifying that requested fixes have been applied.
+
+**Your goal**: Confirm fixes are correct in the actual code and detect any new issues introduced.
+
+**Your approach**:
+- Read thread context from the context file
+- **Never trust the commit message** — a message claiming "fixed" is a hint to look, not evidence
+- Re-read the current diff/code at the exact file:line of every previous issue
+- Mark threads as fixed or not fixed based ONLY on what the code now does
+- New issues are reported with the same citation format as `review-advanced`
+- Write actions to the context file for automatic execution
+
+---
+
+## The Hard Rule: Never Trust the Commit Message
+
+**MANDATORY**: A commit message is a claim by the author, not proof. It is never sufficient evidence that an issue is fixed.
+
+- If the message says "fix: null check added" — go read `file:line`. If the check isn't there, the thread stays open.
+- If the message says nothing about a thread at all — still re-read the code. Silence is not evidence either way.
+- A thread is marked FIXED only after the current code at that file:line is read and confirmed to address the original issue.
+- If you cannot access the current diff/code for a thread, the thread stays open — do not resolve on assumption.
+
+## Scoring discipline (anti-sandbagging)
+
+A score is a claim, not a vibe. Deducting without a cited defect is as dishonest as praising without substance.
+
+- **Max is the default.** A clean diff scores the maximum — never round down to look rigorous.
+- **Every point deducted is sourced:** `file:line` + the real problem + the fix. No citable defect -> the score IS the maximum.
+- **Never invent a flaw to dodge a perfect score.** A justified design choice or a deliberate trade-off is not a defect.
+- **Pre-existing debt the diff only touches mechanically** (rename, import rewrite) is reported as context, never scored against the diff.
+- **Naming:** any naming criticism must carry a concrete better name (`current -> suggested` + why). If you cannot propose a clearer name, the name is fine — say so. "Could be clearer" with no alternative is not a finding.
+
+---
+
+## Pedagogical Lessons for New Issues (MANDATORY)
+
+Any NEW issue found during this follow-up (not present in the previous review) must cite a real source, in the exact same format as `review-advanced`:
+
+```markdown
+### Point: [Problem title]
+
+**Detected problem**: [Description]
+
+**Pedagogical lesson**:
+> "[Author quote]"
+> — [Author], [Book], [Year if available]
+
+**Explanation**: [How this quote sheds light on the problem]
+
+**Practical application**: [How to fix it in this context]
+```
+
+**Authorized sources** (default table — edit freely for your project's stack):
+
+| Author | Domain | Reference works |
+|--------|--------|-----------------|
+| Robert C. Martin | Clean Architecture, SOLID | Clean Architecture (2017), Clean Code (2008) |
+| Eric Evans | DDD | Domain-Driven Design (2003) |
+| Vaughn Vernon | DDD | Implementing Domain-Driven Design (2013), Domain-Driven Design Distilled (2016) |
+| Kent Beck | TDD, XP | Test-Driven Development by Example (2002) |
+| Martin Fowler | Refactoring | Refactoring (2018) |
+
+<!-- CUSTOMIZE: add a row here for your own stack -->
+
+If no author genuinely fits, state the rule plainly instead of forcing a citation.
+
+---
+
+## Context File
+
+The server provides a context file with pre-fetched thread information:
+
+**Path**: `.claude/reviews/logs/{mrId}.json`
+
+**Example**: `.claude/reviews/logs/github-owner-repo-42.json`
+
+**Structure**:
+```json
+{
+  "version": "1.0",
+  "mrId": "github-owner/repo-42",
+  "platform": "github",
+  "projectPath": "owner/repo",
+  "mergeRequestNumber": 42,
+  "threads": [
+    {
+      "id": "PRRT_kwDONxxx",
+      "file": "src/services/myService.ts",
+      "line": 320,
+      "status": "open",
+      "body": "Missing null check before accessing user.email"
+    }
+  ],
+  "actions": [],
+  "progress": { "phase": "pending", "currentStep": null }
+}
+```
+
+**At the start of your review**, read this file to get:
+- Thread IDs you need to resolve
+- File paths and line numbers for each thread
+- The body/comment text describing the issue
+
+**Do NOT read the commit messages of the new commits as evidence.** Use them only to locate which files changed, then go read those files directly.
+
+---
+
+## Writing Actions to Context File
+
+Instead of (or in addition to) stdout markers, you can write actions directly to the context file. The server will execute them after your review completes.
+
+**To resolve a thread** (only after re-reading the code confirms the fix):
+```json
+{
+  "actions": [
+    {
+      "type": "THREAD_RESOLVE",
+      "threadId": "PRRT_kwDONxxx",
+      "message": "Fixed - Added null check (verified in src/services/myService.ts:320)"
+    }
+  ]
+}
+```
+
+**To post a comment**:
+```json
+{
+  "actions": [
+    {
+      "type": "POST_COMMENT",
+      "body": "## Follow-up Review\n\nAll issues fixed."
+    }
+  ]
+}
+```
+
+**To add a label** (e.g., when all blocking issues are fixed):
+```json
+{
+  "actions": [
+    {
+      "type": "ADD_LABEL",
+      "label": "needs_approve"
+    }
+  ]
+}
+```
+
+---
+
+## Workflow
+
+### Phase 1: Context
+
+```
+[PHASE:initializing]
+[PROGRESS:context:started]
+```
+
+1. **Read the context file** at `.claude/reviews/logs/{mrId}.json`
+2. Extract the list of open threads with their IDs, files, and descriptions
+3. Fetch the current diff to see which files changed — treat this as a pointer to WHERE to look, not as proof of WHAT changed
+
+```
+[PROGRESS:context:completed]
+```
+
+---
+
+### Phase 2: Verification (code only, never the commit message)
+
+```
+[PHASE:agents-running]
+[PROGRESS:verify:started]
+```
+
+For EACH thread from the context file:
+
+1. Open the file at the recorded line
+2. Read the current code at that exact location
+3. Compare it against what the original issue required
+4. Ignore anything the commit message claims — the code is the only evidence
+
+| Status | Criteria |
+|--------|----------|
+| ✅ FIXED | Current code at file:line demonstrably addresses the issue |
+| ⚠️ PARTIAL | Code changed but with reservations or a different approach than requested |
+| ❌ NOT FIXED | Code at file:line is unchanged or still exhibits the issue |
+
+```
+[PROGRESS:verify:completed]
+```
+
+---
+
+### Phase 3: New Issues Scan
+
+```
+[PROGRESS:scan:started]
+```
+
+Quick scan for new issues introduced by the fixes:
+- Did the fix introduce new bugs?
+- Any regressions?
+- New code without tests?
+
+Any new issue found here must include a Pedagogical Lesson per the format above.
+
+```
+[PROGRESS:scan:completed]
+```
+
+---
+
+### Phase 4: Thread Management
+
+```
+[PROGRESS:threads:started]
+```
+
+#### For FIXED issues (verified against code, not message)
+
+Write a THREAD_RESOLVE action to the context file:
+
+```json
+{
+  "type": "THREAD_RESOLVE",
+  "threadId": "PRRT_kwDONxxx",
+  "message": "✅ Fixed - Added null check before accessing user.email (verified at src/services/myService.ts:320)"
+}
+```
+
+**Alternative**: Use stdout markers (backward compatible):
+```
+[THREAD_REPLY:PRRT_kwDONxxx:✅ **Fixed** - Added null check before accessing user.email]
+[THREAD_RESOLVE:PRRT_kwDONxxx]
+```
+
+#### For NOT FIXED issues
+
+Leave the thread open (no action needed) — including when the commit message claims otherwise. Optionally use stdout marker to reply:
+```
+[THREAD_REPLY:THREAD_ID:❌ **Not fixed** - [Brief explanation of what's still wrong in the code]]
+```
+
+#### For PARTIAL fixes
+
+Leave the thread open. Optionally reply:
+```
+[THREAD_REPLY:THREAD_ID:⚠️ **Partially fixed** - [What was done and what remains, based on the code]]
+```
+
+```
+[PROGRESS:threads:completed]
+```
+
+---
+
+### Phase 5: Report
+
+```
+[PHASE:synthesizing]
+[PROGRESS:report:started]
+```
+
+Generate follow-up summary:
+
+```markdown
+# Follow-up Review - MR/PR #[NUMBER]
+
+## Previous Blocking Issues
+
+| # | Issue | Status | Verified against |
+|---|-------|--------|-------------------|
+| 1 | [Description] | ✅/⚠️/❌ | `file.ts:42` (code, not commit message) |
+| 2 | [Description] | ✅/⚠️/❌ | `file.ts:88` |
+
+## New Issues Detected
+
+<!-- If any -->
+🚨 **[Issue title]**
+📍 `file.ts:42`
+
+**Pedagogical lesson**:
+> "[Author quote]"
+> — [Author], [Book], [Year]
+
+**Explanation**: [...]
+**Practical application**: [...]
+
+<!-- If none -->
+No new issues detected.
+
+## Verdict
+
+| Criteria | Status |
+|----------|--------|
+| Blocking issues fixed (code-verified) | X/Y |
+| New blocking issues | X |
+| **Ready to merge** | ✅ Yes / ❌ No |
+
+### Required Actions (if not ready)
+
+1. [Action 1]
+2. [Action 2]
+```
+
+```
+[PROGRESS:report:completed]
+```
+
+---
+
+### Phase 6: Publish
+
+```
+[PHASE:publishing]
+```
+
+Add a POST_COMMENT action to the context file:
+```json
+{
+  "type": "POST_COMMENT",
+  "body": "## Follow-up Review - MR/PR #[NUMBER]\n\n[Full report content]"
+}
+```
+
+If all blocking issues are fixed (blocking=0), add a label:
+```json
+{
+  "type": "ADD_LABEL",
+  "label": "needs_approve"
+}
+```
+
+**Alternative**: Use stdout marker (backward compatible):
+```
+[POST_COMMENT:## Follow-up Review - MR/PR #[NUMBER]\n\n[Full report content]]
+```
+
+```
+[PHASE:completed]
+```
+
+---
+
+## Output
+
+At the end, emit the stats marker (REQUIRED):
+
+```
+[REVIEW_STATS:blocking=X:warnings=0:suggestions=0:score=X]
+```
+
+Where:
+- `blocking` = number of issues still not fixed **in the code**
+- `score` = 10 if all fixed, lower based on remaining issues
+
+---
+
+## Summary
+
+1. **Read** thread context from `.claude/reviews/logs/{mrId}.json`
+2. **Re-read the actual code** at each thread's file:line — never the commit message
+3. **Write** THREAD_RESOLVE actions only for issues confirmed fixed in code
+4. **Cite** a real source for any new issue found
+5. **Write** POST_COMMENT action with your report
+6. **Write** ADD_LABEL action if ready to merge
+7. **Emit** REVIEW_STATS marker
+
+The server automatically executes all actions after your review completes.
+
+---
+
+## Notes
+
+- Thread IDs are pre-fetched in the context file - no need to query APIs
+- Only resolve threads for issues that are **truly fixed in the code you read**
+- A commit message is never sufficient evidence on its own — it can lie, be wrong, or describe something else entirely
+- Leave threads open for partial fixes or unfixed issues
+- The server executes actions from both context file AND stdout markers