pull-request-split-advisor 3.2.13 → 3.2.14

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pull-request-split-advisor",
-  "version": "3.2.13",
+  "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",