From 7a3e5278b9f409bd5efa8b3ec0fad09f90757909 Mon Sep 17 00:00:00 2001
From: Jaakko Vanhala <jaakko.vanhala@kamk.fi>
Date: Fri, 10 Apr 2026 07:50:43 +0300
Subject: [PATCH] Review-korjausluuppi: DevOps tarkistaa korjaukset, max 2
 kierrosta
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Pipeline:
1. DevOps review → löytää virheitä
2. Koodari korjaa → päivittää files-objektin
3. DevOps review (kierros 2) → tarkistaa korjaukset
4. Jos yhä virheitä → Koodari korjaa uudelleen
5. LGTM tai max 2 kierrosta → eteenpäin

Terminaalissa näkyy kierrosnumero: "koodikatselmointi (kierros 2)"
LGTM merkitään vihreällä ✓-merkillä.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 network-poc/frontend/src/pages/index.astro | 41 +++++++++++++++-------
 1 file changed, 28 insertions(+), 13 deletions(-)

diff --git a/network-poc/frontend/src/pages/index.astro b/network-poc/frontend/src/pages/index.astro
index 50b9cea..f8b35d2 100644
--- a/network-poc/frontend/src/pages/index.astro
+++ b/network-poc/frontend/src/pages/index.astro
@@ -792,22 +792,37 @@ OUTPUT FORMAT:
         const allCode = Object.entries(files).map(([n,c]) => `--- ${n} ---\n${c}`).join('\n\n');
         let stepN = template.order.length + 1;
 
-        // DevOps/Testaaja: koodikatselmointi
+        // Review-korjausluuppi: max 2 kierrosta
         const tst = agents.tester || Object.values(agents)[4];
-        termLog(`\n<span style="color:var(--accent);font-weight:bold">[${stepN}] ${esc(tst.name)}</span> — koodikatselmointi`);
-        highlightAgent('tester');
-        explainStep('Koodikatselmointi', `${tst.name} tarkistaa importit, nimeämiset, virheenkäsittelyn ja tiedostojen yhteensopivuuden.`);
-        const tstPrompt = (tst.prompt ? tst.prompt+'\n\n' : '') + `Review this project:\n\n${allCode}`;
-        const review = await kpnRun(tst.model, tstPrompt);
-        stepN++;
+        const MAX_REVIEW_ROUNDS = 2;
 
-        // Korjausluuppi (jos tarpeen) — korjatut tiedostot päivitetään files-objektiin
-        if (review && !review.toLowerCase().includes('lgtm')) {
-            termLog(`\n<span style="color:#d29922;font-weight:bold">[${stepN}] ${esc(cdr.name)}</span> — korjaukset`);
+        for (let round = 0; round < MAX_REVIEW_ROUNDS; round++) {
+            const currentCode = Object.entries(files).map(([n,c]) => `--- ${n} ---\n${c}`).join('\n\n');
+
+            // DevOps review
+            termLog(`\n<span style="color:var(--accent);font-weight:bold">[${stepN}] ${esc(tst.name)}</span> — koodikatselmointi${round > 0 ? ' (kierros '+(round+1)+')' : ''}`);
+            highlightAgent('tester');
+            if (round === 0) explainStep('Koodikatselmointi', `${tst.name} analysoi koodin rivi riviltä: importit, nimeämiset, virheenkäsittely, tietoturva.`);
+            else explainStep('Uudelleentarkistus', `${tst.name} tarkistaa korjaukset.`);
+
+            const reviewPrompt = (tst.prompt ? tst.prompt+'\n\n' : '') + `Review this project:\n\n${currentCode}`;
+            const review = await kpnRun(tst.model, reviewPrompt);
+            stepN++;
+
+            // LGTM → ei korjauksia tarvita
+            if (!review || review.toLowerCase().includes('lgtm')) {
+                termLog(`  <span style="color:#3fb950">✓ ${esc(tst.name)}: LGTM</span>`);
+                break;
+            }
+
+            // Korjaukset
+            termLog(`\n<span style="color:#d29922;font-weight:bold">[${stepN}] ${esc(cdr.name)}</span> — korjaukset${round > 0 ? ' (kierros '+(round+1)+')' : ''}`);
             highlightAgent('coder');
-            explainStep('Korjausluuppi', `${tst.name} löysi ongelmia. ${cdr.name} korjaa ja palauttaa päivitetyt tiedostot.`);
-            const fixPrompt = `${cdr.prompt ? cdr.prompt+'\n\n' : ''}Fix these issues:\n${review}\n\nCurrent code:\n${allCode}\n\nWrite ALL corrected files. Start each file with: --- filename.py ---`;
+            explainStep('Korjaus', `${tst.name} löysi ongelmia. ${cdr.name} saa palautteen ja korjaa.`);
+
+            const fixPrompt = `${cdr.prompt ? cdr.prompt+'\n\n' : ''}Fix these issues:\n${review}\n\nCurrent code:\n${currentCode}\n\nWrite ALL corrected files. Start each file with: --- filename.py ---`;
             const fixedCode = await kpnRun(cdr.model, fixPrompt);
+
             // Parsitaan korjatut tiedostot takaisin files-objektiin
             if (fixedCode) {
                 const fixedParts = fixedCode.split(/^---\s*(\S+)\s*---$/m);
@@ -820,7 +835,7 @@ OUTPUT FORMAT:
                 }
             }
             stepN++;
-        }
+        } // for review round
 
         // Päivitetään allCode korjausten jälkeen
         const updatedCode = Object.entries(files).map(([n,c]) => `--- ${n} ---\n${c}`).join('\n\n');