pull-request-split-advisor 3.2.13 → 3.2.17

package/dist/cli.js

@@ -43,6 +43,23 @@ import { runConfigWizard, runConfigWithKey } from "./ai/config-wizard.js";
 function normalizeStoryNumber(value) {
     return value.trim().replace(/[^0-9]/g, "");
 }
+/**
+ * Convierte las reglas de scoring de una métrica en una cadena legible con los umbrales.
+ * Ejemplo: "≤2→5  ≤4→4  =5→3  ≤7→2  *→1"
+ */
+function formatThresholds(rules) {
+    return rules.map((r) => {
+        const cond = r.default ? "*"
+            : r.eq !== undefined ? `=${r.eq}`
+                : r.lte !== undefined ? `≤${r.lte}`
+                    : r.lt !== undefined ? `<${r.lt}`
+                        : r.gte !== undefined && r.lt !== undefined ? `${r.gte}–${r.lt}`
+                            : r.gte !== undefined ? `≥${r.gte}`
+                                : r.gt !== undefined ? `>${r.gt}`
+                                    : "?";
+        return `${cond}→${r.points}pts`;
+    }).join("  ");
+}
 /**
  * Pide al usuario el número de subtarea para cada commit del plan y rellena
  * `commit.ticketCode` y `commit.suggestedMessage` con el resultado.
@@ -585,8 +602,13 @@ async function main() {
         }
         const aheadCommits = localAheadCount(baseBranch);
         const commitCount = Math.max(aheadCommits, 1);
-        const filesPerCommit = Number((scoredFiles / commitCount).toFixed(2));
-        const avgLinesPerCommit = Math.round(scoredLines / commitCount);
+        // Filtrar commits chore/style/docs del denominador de M1.4 y M1.5,
+        // igual que el script bash original que usa filtered_commit_count.
+        const allAheadCommits = localAheadCommits(baseBranch);
+        const filteredCommits = allAheadCommits.filter((c) => !/^(chore|style|docs)(\(|!|:|\s)/i.test(c.subject));
+        const filteredCount = Math.max(filteredCommits.length, 1);
+        const filesPerCommit = Number((scoredFiles / filteredCount).toFixed(2));
+        const avgLinesPerCommit = Math.round(scoredLines / filteredCount);
         const result = scorePullRequest({ commitCount, filesPerCommit, avgLinesPerCommit, totalLinesChanged: scoredLines }, config);
         const warnThreshold = config.targetScore > 4 ? 4 : Math.max(0, config.targetScore - 1);
         const scoreColor = ui.scoreColor(result.complexity, config.targetScore);
@@ -629,10 +651,23 @@ async function main() {
             `Score:                  ${ui.score(result.complexity, config.targetScore)}`,
             `Score objetivo:         ${config.targetScore}`,
             "",
-            ...Object.values(result.metrics).map((m) => `  ${m.label}: valor=${m.rawValue}  pts=${m.points}  pond=${m.weightedScore}`),
-            "",
             `Recomendación:          ${result.recommendation}`
         ], scoreColor);
+        // ─── Tabla de métricas con umbrales ─────────────────────────────────
+        ui.section("DETALLE DE MÉTRICAS", "-");
+        ui.table(["Cód.", "Métrica", "Valor", "Pts", "Peso", "Aporte", "Umbrales"], Object.entries(result.metrics).map(([code, m]) => {
+            const def = config.metrics[code];
+            const thresholds = def ? formatThresholds(def.scoring) : "";
+            return [
+                code,
+                m.label,
+                String(m.rawValue),
+                `${m.points} / 5`,
+                `${(m.weight * 100).toFixed(0)}%`,
+                String(m.weightedScore),
+                thresholds
+            ];
+        }));
         closeReadlineInterface();
     });
     // ─── Subcomando: config ──────────────────────────────────────────────────

package/dist/config/default-config.js

@@ -94,10 +94,10 @@ export const defaultConfig = {
     // ── Métricas de scoring ───────────────────────────────────────────────────
     //
     // Orden posicional REQUERIDO (no cambiar el orden de las claves):
-    //   posición 0 → commitCount      (M1.3)
-    //   posición 1 → filesPerCommit   (M1.4)
-    //   posición 2 → avgLinesPerCommit (M1.5)
-    //   posición 3 → totalLinesChanged (M3.2)
+    //   posición 0 → commitCount       (M1.3 — total de commits)
+    //   posición 1 → filesPerCommit    (M1.4 — archivos / commits filtrados sin chore/style/docs)
+    //   posición 2 → avgLinesPerCommit (M1.5 — líneas / commits filtrados sin chore/style/docs)
+    //   posición 3 → totalLinesChanged (M3.2 — total de líneas)
     //
     metrics: {
         /**
@@ -131,7 +131,7 @@ export const defaultConfig = {
             ]
         },
         /**
-         * M1.5 — Promedio de líneas por commit (excluye chore/style/docs).
+         * M1.5 — Promedio de líneas por commit (excluye chore/style/docs del denominador).
          * Peso: 0.25  |  Contribución máxima: 1.25 puntos
          */
         "M1.5": {

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/core/scoring.js

@@ -15,9 +15,9 @@
  * El motor asigna valores brutos **posicionalmente** según el orden declarado
  * en `config.metrics`. Los 4 valores brutos, en orden obligatorio, son:
  *
- * 1. Cantidad de commits del PR
- * 2. Promedio de archivos por commit
- * 3. Promedio de líneas por commit
+ * 1. Cantidad de commits del PR (total, incluye chore/style/docs)
+ * 2. Promedio de archivos por commit (denominador: commits sin chore/style/docs)
+ * 3. Promedio de líneas por commit  (denominador: commits sin chore/style/docs)
  * 4. Total de líneas cambiadas
  *
  * Los códigos de cada métrica son libres («sólo etiquetas») y cada equipo
@@ -130,8 +130,8 @@ export function scorePullRequest(raw, config) {
     // Mapeo posicional: el orden declarado en config.metrics determina qué valor
     // bruto recibe cada métrica. Convención invariante (ver MetricCode en types.ts):
     //   posición 0 → commitCount
-    //   posición 1 → filesPerCommit
-    //   posición 2 → avgLinesPerCommit
+    //   posición 1 → filesPerCommit        (excl. chore/style/docs del denominador)
+    //   posición 2 → avgLinesPerCommit     (excl. chore/style/docs del denominador)
     //   posición 3 → totalLinesChanged
     const rawValues = [
         raw.commitCount,

package/package.json

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