pull-request-split-advisor 3.2.12 → 3.2.14

package/dist/cli.js

@@ -399,9 +399,18 @@ async function main() {
             ], "red");
             // ─── Estado de la cascada en terminal ───────────────────────────────────
             ui.section("ESTADO DE LA CASCADA", "#");
+            // Rama actual primero (#1)
+            ui.subsection(`#1  ${currentBranch}  ✔ rama actual`);
+            if (currentBranchCommits.length === 0) {
+                ui.muted("  Sin commits detectados.");
+            }
+            else {
+                ui.table(["SHA", "Mensaje"], currentBranchCommits.map((c) => [c.sha, c.subject]), true);
+            }
+            // Ramas hermanas a continuación (#2, #3, …)
             for (let i = 0; i < siblings.length; i++) {
                 const sibling = siblings[i];
-                ui.subsection(`#${i + 1}  ${sibling.name}`);
+                ui.subsection(`#${i + 2}  ${sibling.name}`);
                 if (sibling.commits.length === 0) {
                     ui.muted("  Sin commits detectados.");
                 }
@@ -409,13 +418,6 @@ async function main() {
                     ui.table(["SHA", "Mensaje"], sibling.commits.map((c) => [c.sha, c.subject]), true);
                 }
             }
-            ui.subsection(`#${siblings.length + 1}  ${currentBranch}  ✔ rama actual`);
-            if (currentBranchCommits.length === 0) {
-                ui.muted("  Sin commits detectados.");
-            }
-            else {
-                ui.table(["SHA", "Mensaje"], currentBranchCommits.map((c) => [c.sha, c.subject]), true);
-            }
         }
         // ─── Pipeline de análisis ────────────────────────────────────────────────
         ui.spinner.start("Analizando cambios del working tree...");

package/dist/core/blocks.js

@@ -132,6 +132,63 @@ export function buildBlocks(fileStats, config, deps) {
             depScore: 0
         });
     }
+    // ── Post-proceso: fusionar bloques de solo-tests con su bloque fuente ──────
+    // Cubre el caso en que test y fuente están en directorios distintos
+    // (ej: `tests/` vs `src/`) y getGroupKey los asigna a bloques distintos.
+    // Regla: un test no puede ir a una rama sin su fuente, salvo que la fuente
+    // no tenga cambios y no esté en el conjunto analizado.
+    {
+        const linesByPath = new Map(fileStats.map((f) => [f.path, f.lines]));
+        // Índice: nombre-base del fuente → bloque que lo contiene.
+        const sourceBlockByBaseName = new Map();
+        for (const block of blocks) {
+            for (const f of block.files) {
+                if (!isTestFile(f)) {
+                    const baseName = basename(f).replace(/\.[^.]+$/, "").replace(/\.?(test|spec)$/, "");
+                    if (!sourceBlockByBaseName.has(baseName)) {
+                        sourceBlockByBaseName.set(baseName, block);
+                    }
+                }
+            }
+        }
+        // Para cada bloque que contiene exclusivamente tests, intentar fusionarlo
+        // con el bloque de su archivo fuente si éste tiene cambios.
+        const blocksToDelete = new Set();
+        for (const block of blocks) {
+            if (blocksToDelete.has(block))
+                continue;
+            if (!block.files.every((f) => isTestFile(f)))
+                continue; // ya tiene fuente: OK
+            const moved = [];
+            for (const testFile of block.files) {
+                const testBase = basename(testFile)
+                    .replace(/\.[^.]+$/, "")
+                    .replace(/\.?(test|spec)$/, "")
+                    .replace(/^test_/, "")
+                    .replace(/_test$/, "");
+                const sourceBlock = sourceBlockByBaseName.get(testBase);
+                if (!sourceBlock || sourceBlock === block)
+                    continue;
+                // Fusionar el test en el bloque del fuente.
+                sourceBlock.files.push(testFile);
+                sourceBlock.lines += linesByPath.get(testFile) ?? 0;
+                sourceBlock.divisible = false; // test + fuente → siempre indivisible
+                moved.push(testFile);
+            }
+            if (moved.length === block.files.length) {
+                // Todos los tests fueron reasignados: eliminar este bloque vacío.
+                blocksToDelete.add(block);
+            }
+            else if (moved.length > 0) {
+                // Quitar solo los archivos que se movieron.
+                block.files = block.files.filter((f) => !moved.includes(f));
+                block.lines = block.files.reduce((sum, f) => sum + (linesByPath.get(f) ?? 0), 0);
+            }
+        }
+        if (blocksToDelete.size > 0) {
+            blocks.splice(0, blocks.length, ...blocks.filter((b) => !blocksToDelete.has(b)));
+        }
+    }
     // Calcular depScore con un mapa de frecuencias: O(n+m) en lugar de O(n×m).
     const fileEdgeCount = new Map();
     for (const { from, to } of deps) {

package/dist/output/report.js

@@ -506,8 +506,8 @@ export function renderHtmlReport(input) {
          Para mantener el orden correcto de PRs apilados, la nueva rama debe crearse desde esa rama.`
             : `No se encontraron otras ramas de la misma cascada. Crea la nueva rama directamente desde la base.`;
         const allBranches = [
-            ...cw.siblings,
-            { name: currentBranch, commits: cw.currentBranchCommits, isCurrent: true }
+            { name: currentBranch, commits: cw.currentBranchCommits, isCurrent: true },
+            ...cw.siblings
         ];
         const cascadeStateHtml = allBranches.map((branch, idx) => {
             const num = idx + 1;
@@ -521,7 +521,7 @@ export function renderHtmlReport(input) {
         <div style="display:flex;align-items:center;gap:6px;margin-bottom:4px">
           <span style="font-size:.75em;font-weight:700;opacity:.5;min-width:22px">#${num}</span>
           <code style="display:inline-block;background:#fff;color:#1e293b;font-size:.8em;font-weight:600;padding:1px 7px;border-radius:4px;letter-spacing:.01em">${esc(branch.name)}</code>
-          ${branch.isCurrent ? `<span style="font-size:.75em;color:#4ade80;font-weight:600">← rama actual</span>` : ""}
+          ${branch.isCurrent ? `<span style="font-size:.74em;background:#fff;color:#16a34a;font-weight:700;padding:1px 6px;border-radius:4px;border:1px solid #bbf7d0">← rama actual</span>` : ""}
         </div>
         ${commitsHtml}
       </div>`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pull-request-split-advisor",
-  "version": "3.2.12",
+  "version": "3.2.14",
   "description": "CLI that analyses your Git working tree and suggests how to split changes into smaller, reviewable pull requests and commits.",
   "keywords": [
     "git",