@planu/cli 4.14.1 → 5.0.0

package/dist/engine/validator/extractors.js

@@ -1,31 +1,29 @@
 // Planu — Validator: criteria extractors
 // Parses HU.md and FICHA-TECNICA.md to derive acceptance criteria lists.
 import { readFile } from 'node:fs/promises';
-import { stripFrontmatter } from '../frontmatter-parser.js';
+import { extractNormalizedAcceptanceCriteria, normalizeAcceptanceCriterionToken, } from '../spec-format/acceptance-criteria.js';
+import { extractListItems, extractSection } from '../spec-format/markdown-sections.js';
 import { readSpecTechnicalSection } from '../spec-format/read-technical-section.js';
+export { extractListItems, extractSection } from '../spec-format/markdown-sections.js';
 /**
  * Extract all acceptance criteria from a spec's source files.
  */
 export async function extractCriteria(spec) {
     const criteria = [];
+    const canonicalStableKeys = new Set();
+    const canonicalNormalizedTexts = new Set();
     // Extract from HU.md — look for acceptance criteria sections
     try {
         const rawHuContent = await readFile(spec.specPath, 'utf-8');
-        const huContent = stripFrontmatter(rawHuContent);
-        const acSection = extractSection(huContent, 'acceptance criteria', 'criterios de aceptaci');
-        if (acSection) {
-            const items = extractListItems(acSection);
-            criteria.push(...items);
-        }
-        // Also check for explicit "Given/When/Then" patterns at line start.
-        // Keep this anchored so narrative text containing "when" is not treated
-        // as an acceptance criterion.
-        const gwtMatches = huContent.matchAll(/^\s*(?:[-*+]\s+|\d+[.)]\s+)?(?:Given|Dado|When|Cuando|Then|Entonces)\s+(.+)$/gim);
-        for (const match of gwtMatches) {
-            /* v8 ignore next 3 -- defensive regex capture group guard */
-            if (match[1]) {
-                criteria.push(match[1].trim());
+        for (const criterion of extractNormalizedAcceptanceCriteria(rawHuContent)) {
+            if (canonicalStableKeys.has(criterion.stableKey)) {
+                continue;
             }
+            canonicalStableKeys.add(criterion.stableKey);
+            canonicalNormalizedTexts.add(criterion.normalizedText);
+            const oppositeKindAlias = `${criterion.id.startsWith('AC') ? 'ab' : 'ac'}${criterion.id.slice(2)}`;
+            const hasExplicitId = !criterion.aliases.includes(oppositeKindAlias);
+            criteria.push(hasExplicitId ? `${criterion.id}: ${criterion.text}` : criterion.text);
         }
     }
     catch {
@@ -38,8 +36,16 @@ export async function extractCriteria(spec) {
         const fichaContent = await readSpecTechnicalSection(spec);
         const reqSection = extractSection(fichaContent, 'requirements', 'requisitos', 'acceptance criteria', 'criterios de aceptaci', 'criterios de verificaci');
         if (reqSection) {
-            const items = extractListItems(reqSection);
-            criteria.push(...items);
+            const seenTechnicalTexts = new Set();
+            for (const item of extractListItems(reqSection)) {
+                const normalizedText = normalizeAcceptanceCriterionToken(item);
+                if (canonicalNormalizedTexts.has(normalizedText) ||
+                    seenTechnicalTexts.has(normalizedText)) {
+                    continue;
+                }
+                seenTechnicalTexts.add(normalizedText);
+                criteria.push(item);
+            }
         }
     }
     catch {
@@ -59,7 +65,7 @@ export async function extractCriteria(spec) {
         }
         criteria.push('No regression in existing tests');
     }
-    return [...new Set(criteria)]; // Deduplicate
+    return criteria;
 }
 /**
  * Parse verify blocks from HTML comments following acceptance criteria checkboxes.
@@ -97,55 +103,4 @@ export function extractVerifyBlocks(content) {
 /**
  * Extract the text content under a markdown heading that matches any of the given keywords.
  */
-export function extractSection(content, ...headingKeywords) {
-    const lines = content.split('\n');
-    let capturing = false;
-    let capturedLevel = 0;
-    const captured = [];
-    for (const line of lines) {
-        const headingMatch = /^(#{1,6})\s+(.+)/.exec(line);
-        if (headingMatch?.[1] !== undefined && headingMatch[2] !== undefined) {
-            const level = headingMatch[1].length;
-            const title = headingMatch[2].toLowerCase();
-            if (capturing && level <= capturedLevel) {
-                break; // End of section
-            }
-            if (!capturing && headingKeywords.some((kw) => title.includes(kw))) {
-                capturing = true;
-                capturedLevel = level;
-                continue;
-            }
-        }
-        if (capturing) {
-            captured.push(line);
-        }
-    }
-    return captured.length > 0 ? captured.join('\n') : null;
-}
-/**
- * Extract bullet/numbered list items from a markdown text block.
- */
-export function extractListItems(content) {
-    const items = [];
-    const lines = content.split('\n');
-    for (const line of lines) {
-        const listMatch = /^\s*[-*+]\s+\[?\s*[xX ]?\s*\]?\s*(.+)/.exec(line);
-        if (listMatch?.[1]) {
-            const item = listMatch[1].trim();
-            if (item.length > 5) {
-                // Skip trivially short items
-                items.push(item);
-            }
-        }
-        // Also handle numbered lists
-        const numberedMatch = /^\s*\d+[.)]\s+(.+)/.exec(line);
-        if (numberedMatch?.[1]) {
-            const item = numberedMatch[1].trim();
-            if (item.length > 5) {
-                items.push(item);
-            }
-        }
-    }
-    return items;
-}
 //# sourceMappingURL=extractors.js.map

package/dist/engine/validator/spec-compliance-runner.d.ts

@@ -3,11 +3,18 @@ export type ScenarioVerdict = 'pass' | 'fail' | 'missing';
 export interface ScenarioTestLink {
     path: string;
     line?: number;
+    runner?: string;
+    config?: string;
+}
+export interface ScenarioStep {
+    keyword: string;
+    text: string;
 }
 export interface FrontmatterScenario {
     title: string;
     done: boolean;
     tests?: ScenarioTestLink[];
+    steps?: ScenarioStep[];
 }
 export interface PerScenarioResult {
     title: string;
@@ -15,35 +22,16 @@ export interface PerScenarioResult {
     evidence: string[];
 }
 export interface SpecComplianceResult {
-    /** 0-100 */
     dimensionScore: number;
     perScenario: PerScenarioResult[];
-    /** The vitest command that was (or would have been) run */
     command: string;
 }
-/**
- * Parse scenarios from a raw spec.md content string.
- * Handles the multi-line YAML block that the simple KV parser cannot handle.
- *
- * Expected frontmatter shape:
- * ```yaml
- * scenarios:
- *   - title: "Scenario A"
- *     done: false
- *     tests:
- *       - path: tests/foo.test.ts
- *         line: 42
- * ```
- */
+/** A sanitized terminal adapter failure that cannot be mistaken for a failed test verdict. */
+export declare class ComplianceCommandTerminalError extends Error {
+    readonly code: string;
+    constructor(code: string);
+}
 export declare function parseFrontmatterScenarios(raw: string): FrontmatterScenario[];
-/**
- * Run vitest-backed spec-compliance for the given spec.
- * Each frontmatter scenario must have a `tests` array; missing links count as `fail`.
- */
 export declare function runSpecCompliance(spec: Spec, projectPath: string, signal?: AbortSignal, canonicalProjectId?: string): Promise<SpecComplianceResult>;
-/**
- * Returns a list of scenario titles that lack a `tests` array.
- * Used by check-readiness to block `approved` when scenarios are untested.
- */
 export declare function findScenariosWithoutTests(raw: string): string[];
 //# sourceMappingURL=spec-compliance-runner.d.ts.map