@seanmars/tospec 0.4.2

package/dist/core/validation/section-validator.d.ts

@@ -0,0 +1,19 @@
+import type { ValidationIssue } from './types.js';
+export interface SectionValidationRules {
+    requiredSections: string[];
+    minSectionLength: Record<string, number>;
+}
+export interface SectionValidationContext {
+    /** Reported as ValidationIssue.path (typically the file's relative path). */
+    path: string;
+    /** Shown in the "missing section" message so authors know where to look. */
+    templateRef?: string;
+}
+/**
+ * Validates a document's headings against schema-declared rules, for
+ * artifacts (ticket.md, design.md, task.md, ...) that have no Requirement
+ * grammar to check structurally. Rules live in schema.yaml so new document
+ * types don't require validator code changes.
+ */
+export declare function validateSections(content: string, rules: SectionValidationRules, context: SectionValidationContext): ValidationIssue[];
+//# sourceMappingURL=section-validator.d.ts.map

package/dist/core/validation/section-validator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"section-validator.d.ts","sourceRoot":"","sources":["../../../src/core/validation/section-validator.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAElD,MAAM,WAAW,sBAAsB;IACrC,gBAAgB,EAAE,MAAM,EAAE,CAAC;IAC3B,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CAC1C;AAED,MAAM,WAAW,wBAAwB;IACvC,6EAA6E;IAC7E,IAAI,EAAE,MAAM,CAAC;IACb,4EAA4E;IAC5E,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAgBD;;;;;GAKG;AACH,wBAAgB,gBAAgB,CAC9B,OAAO,EAAE,MAAM,EACf,KAAK,EAAE,sBAAsB,EAC7B,OAAO,EAAE,wBAAwB,GAChC,eAAe,EAAE,CAwCnB"}

package/dist/core/validation/section-validator.js

@@ -0,0 +1,59 @@
+import { MarkdownParser } from '../parsers/markdown-parser.js';
+function findSectionByTitle(sections, title) {
+    const target = title.trim().toLowerCase();
+    for (const section of sections) {
+        if (section.title.trim().toLowerCase() === target) {
+            return section;
+        }
+        const found = findSectionByTitle(section.children, title);
+        if (found) {
+            return found;
+        }
+    }
+    return undefined;
+}
+/**
+ * Validates a document's headings against schema-declared rules, for
+ * artifacts (ticket.md, design.md, task.md, ...) that have no Requirement
+ * grammar to check structurally. Rules live in schema.yaml so new document
+ * types don't require validator code changes.
+ */
+export function validateSections(content, rules, context) {
+    const issues = [];
+    const sections = new MarkdownParser(content).getSections();
+    for (const requiredTitle of rules.requiredSections) {
+        const section = findSectionByTitle(sections, requiredTitle);
+        if (!section) {
+            const hint = context.templateRef ? ` (see template: ${context.templateRef})` : '';
+            issues.push({
+                level: 'ERROR',
+                path: context.path,
+                message: `Missing required section "${requiredTitle}"${hint}`,
+            });
+            continue;
+        }
+        if (!section.content.trim()) {
+            issues.push({
+                level: 'WARNING',
+                path: context.path,
+                message: `Section "${requiredTitle}" is empty`,
+            });
+        }
+    }
+    for (const [sectionTitle, minLength] of Object.entries(rules.minSectionLength)) {
+        const section = findSectionByTitle(sections, sectionTitle);
+        if (!section) {
+            continue; // absence is already reported above when also required
+        }
+        const length = section.content.trim().length;
+        if (length > 0 && length < minLength) {
+            issues.push({
+                level: 'WARNING',
+                path: context.path,
+                message: `Section "${sectionTitle}" is shorter than the recommended ${minLength} characters (found ${length})`,
+            });
+        }
+    }
+    return issues;
+}
+//# sourceMappingURL=section-validator.js.map

package/dist/core/validation/section-validator.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"section-validator.js","sourceRoot":"","sources":["../../../src/core/validation/section-validator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAgB,MAAM,+BAA+B,CAAC;AAe7E,SAAS,kBAAkB,CAAC,QAAmB,EAAE,KAAa;IAC5D,MAAM,MAAM,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC1C,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,IAAI,OAAO,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,KAAK,MAAM,EAAE,CAAC;YAClD,OAAO,OAAO,CAAC;QACjB,CAAC;QACD,MAAM,KAAK,GAAG,kBAAkB,CAAC,OAAO,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC;QAC1D,IAAI,KAAK,EAAE,CAAC;YACV,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,gBAAgB,CAC9B,OAAe,EACf,KAA6B,EAC7B,OAAiC;IAEjC,MAAM,MAAM,GAAsB,EAAE,CAAC;IACrC,MAAM,QAAQ,GAAG,IAAI,cAAc,CAAC,OAAO,CAAC,CAAC,WAAW,EAAE,CAAC;IAE3D,KAAK,MAAM,aAAa,IAAI,KAAK,CAAC,gBAAgB,EAAE,CAAC;QACnD,MAAM,OAAO,GAAG,kBAAkB,CAAC,QAAQ,EAAE,aAAa,CAAC,CAAC;QAC5D,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,MAAM,IAAI,GAAG,OAAO,CAAC,WAAW,CAAC,CAAC,CAAC,mBAAmB,OAAO,CAAC,WAAW,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;YAClF,MAAM,CAAC,IAAI,CAAC;gBACV,KAAK,EAAE,OAAO;gBACd,IAAI,EAAE,OAAO,CAAC,IAAI;gBAClB,OAAO,EAAE,6BAA6B,aAAa,IAAI,IAAI,EAAE;aAC9D,CAAC,CAAC;YACH,SAAS;QACX,CAAC;QACD,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;YAC5B,MAAM,CAAC,IAAI,CAAC;gBACV,KAAK,EAAE,SAAS;gBAChB,IAAI,EAAE,OAAO,CAAC,IAAI;gBAClB,OAAO,EAAE,YAAY,aAAa,YAAY;aAC/C,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,KAAK,MAAM,CAAC,YAAY,EAAE,SAAS,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,gBAAgB,CAAC,EAAE,CAAC;QAC/E,MAAM,OAAO,GAAG,kBAAkB,CAAC,QAAQ,EAAE,YAAY,CAAC,CAAC;QAC3D,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,SAAS,CAAC,uDAAuD;QACnE,CAAC;QACD,MAAM,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,MAAM,CAAC;QAC7C,IAAI,MAAM,GAAG,CAAC,IAAI,MAAM,GAAG,SAAS,EAAE,CAAC;YACrC,MAAM,CAAC,IAAI,CAAC;gBACV,KAAK,EAAE,SAAS;gBAChB,IAAI,EAAE,OAAO,CAAC,IAAI;gBAClB,OAAO,EAAE,YAAY,YAAY,qCAAqC,SAAS,sBAAsB,MAAM,GAAG;aAC/G,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/core/validation/types.d.ts

@@ -0,0 +1,18 @@
+export type ValidationLevel = 'ERROR' | 'WARNING' | 'INFO';
+export interface ValidationIssue {
+    level: ValidationLevel;
+    path: string;
+    message: string;
+    line?: number;
+    column?: number;
+}
+export interface ValidationReport {
+    valid: boolean;
+    issues: ValidationIssue[];
+    summary: {
+        errors: number;
+        warnings: number;
+        info: number;
+    };
+}
+//# sourceMappingURL=types.d.ts.map

package/dist/core/validation/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/core/validation/types.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,eAAe,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AAE3D,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,eAAe,CAAC;IACvB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,gBAAgB;IAC/B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,eAAe,EAAE,CAAC;IAC1B,OAAO,EAAE;QACP,MAAM,EAAE,MAAM,CAAC;QACf,QAAQ,EAAE,MAAM,CAAC;QACjB,IAAI,EAAE,MAAM,CAAC;KACd,CAAC;CACH"}

package/dist/core/validation/types.js

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=types.js.map

package/dist/core/validation/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../../src/core/validation/types.ts"],"names":[],"mappings":""}

package/dist/core/validation/validator.d.ts

@@ -0,0 +1,51 @@
+import { ValidationReport } from './types.js';
+export declare class Validator {
+    private strictMode;
+    constructor(strictMode?: boolean);
+    validateSpec(filePath: string): Promise<ValidationReport>;
+    /**
+     * Validate spec content from a string (used for pre-write validation of rebuilt specs)
+     */
+    validateSpecContent(specName: string, content: string): Promise<ValidationReport>;
+    validateChange(filePath: string): Promise<ValidationReport>;
+    /**
+     * Validate delta-formatted spec files under a change directory.
+     * Enforces:
+     * - At least one delta across all files
+     * - ADDED/MODIFIED: each requirement has SHALL/MUST and at least one scenario
+     * - REMOVED: names only; no scenario/description required
+     * - RENAMED: pairs well-formed
+     * - No duplicates within sections; no cross-section conflicts per spec
+     */
+    validateChangeDeltaSpecs(changeDir: string): Promise<ValidationReport>;
+    /**
+     * Recursively collect every delta `spec.md` under a change's specs directory,
+     * so both the one-level (specs/<capability>/spec.md) and nested multi-area
+     * (specs/<area>/<capability>/spec.md) layouts are discovered (#1182b).
+     * Returns absolute paths, sorted for deterministic issue ordering.
+     */
+    private findDeltaSpecFiles;
+    private convertZodErrors;
+    private applySpecRules;
+    private applyChangeRules;
+    private enrichTopLevelError;
+    private extractNameFromPath;
+    private createReport;
+    isValid(report: ValidationReport): boolean;
+    private extractRequirementText;
+    private containsShallOrMust;
+    /**
+     * Build an error message for a requirement block whose body lacks SHALL/MUST.
+     *
+     * When the SHALL/MUST keyword already appears in the requirement header (e.g.
+     * `### Requirement: The system SHALL ...`) the original generic error
+     * ("must contain SHALL or MUST") is confusing because the keyword is visibly
+     * present in the spec. Per the Tospec conventions the keyword has to live
+     * on the requirement body line (the line right after the header), so we point
+     * the author at that exact fix when the keyword is found in the header only.
+     */
+    private buildMissingShallOrMustMessage;
+    private countScenarios;
+    private formatSectionList;
+}
+//# sourceMappingURL=validator.d.ts.map

package/dist/core/validation/validator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validator.d.ts","sourceRoot":"","sources":["../../../src/core/validation/validator.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,gBAAgB,EAAoC,MAAM,YAAY,CAAC;AAehF,qBAAa,SAAS;IACpB,OAAO,CAAC,UAAU,CAAU;gBAEhB,UAAU,GAAE,OAAe;IAIjC,YAAY,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,CAAC;IA8B/D;;OAEG;IACG,mBAAmB,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAkBjF,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,CAAC;IA+BjE;;;;;;;;OAQG;IACG,wBAAwB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAkK5E;;;;;OAKG;YACW,kBAAkB;IAsBhC,OAAO,CAAC,gBAAgB;IAcxB,OAAO,CAAC,cAAc;IA2DtB,OAAO,CAAC,gBAAgB;IA2BxB,OAAO,CAAC,mBAAmB;IAc3B,OAAO,CAAC,mBAAmB;IAmB3B,OAAO,CAAC,YAAY;IAoBpB,OAAO,CAAC,MAAM,EAAE,gBAAgB,GAAG,OAAO;IAI1C,OAAO,CAAC,sBAAsB;IAU9B,OAAO,CAAC,mBAAmB;IAI3B;;;;;;;;;OASG;IACH,OAAO,CAAC,8BAA8B;IAQtC,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,iBAAiB;CAO1B"}

package/dist/core/validation/validator.js

@@ -0,0 +1,469 @@
+import { readFileSync, promises as fs } from 'fs';
+import path from 'path';
+import { SpecSchema, ChangeSchema } from '../schemas/index.js';
+import { MarkdownParser } from '../parsers/markdown-parser.js';
+import { ChangeParser } from '../parsers/change-parser.js';
+import { MIN_PURPOSE_LENGTH, MAX_REQUIREMENT_TEXT_LENGTH, VALIDATION_MESSAGES } from './constants.js';
+import { parseDeltaSpec, normalizeRequirementName, extractRequirementsSection } from '../parsers/requirement-blocks.js';
+import { extractRequirementBody as extractRequirementBodyShared, containsShallOrMust as containsShallOrMustShared, countScenarios as countScenariosShared, } from '../parsers/requirement-text.js';
+import { findMainSpecStructureIssues } from '../parsers/spec-structure.js';
+import { FileSystemUtils } from '../../utils/file-system.js';
+export class Validator {
+    strictMode;
+    constructor(strictMode = false) {
+        this.strictMode = strictMode;
+    }
+    async validateSpec(filePath) {
+        const issues = [];
+        const specName = this.extractNameFromPath(filePath);
+        try {
+            const content = readFileSync(filePath, 'utf-8');
+            const parser = new MarkdownParser(content);
+            const spec = parser.parseSpec(specName);
+            const result = SpecSchema.safeParse(spec);
+            if (!result.success) {
+                issues.push(...this.convertZodErrors(result.error));
+            }
+            issues.push(...this.applySpecRules(spec, content));
+        }
+        catch (error) {
+            const baseMessage = error instanceof Error ? error.message : 'Unknown error';
+            const enriched = this.enrichTopLevelError(specName, baseMessage);
+            issues.push({
+                level: 'ERROR',
+                path: 'file',
+                message: enriched,
+            });
+        }
+        return this.createReport(issues);
+    }
+    /**
+     * Validate spec content from a string (used for pre-write validation of rebuilt specs)
+     */
+    async validateSpecContent(specName, content) {
+        const issues = [];
+        try {
+            const parser = new MarkdownParser(content);
+            const spec = parser.parseSpec(specName);
+            const result = SpecSchema.safeParse(spec);
+            if (!result.success) {
+                issues.push(...this.convertZodErrors(result.error));
+            }
+            issues.push(...this.applySpecRules(spec, content));
+        }
+        catch (error) {
+            const baseMessage = error instanceof Error ? error.message : 'Unknown error';
+            const enriched = this.enrichTopLevelError(specName, baseMessage);
+            issues.push({ level: 'ERROR', path: 'file', message: enriched });
+        }
+        return this.createReport(issues);
+    }
+    async validateChange(filePath) {
+        const issues = [];
+        const changeName = this.extractNameFromPath(filePath);
+        try {
+            const content = readFileSync(filePath, 'utf-8');
+            const changeDir = path.dirname(filePath);
+            const parser = new ChangeParser(content, changeDir);
+            const change = await parser.parseChangeWithDeltas(changeName);
+            const result = ChangeSchema.safeParse(change);
+            if (!result.success) {
+                issues.push(...this.convertZodErrors(result.error));
+            }
+            issues.push(...this.applyChangeRules(change, content));
+        }
+        catch (error) {
+            const baseMessage = error instanceof Error ? error.message : 'Unknown error';
+            const enriched = this.enrichTopLevelError(changeName, baseMessage);
+            issues.push({
+                level: 'ERROR',
+                path: 'file',
+                message: enriched,
+            });
+        }
+        return this.createReport(issues);
+    }
+    /**
+     * Validate delta-formatted spec files under a change directory.
+     * Enforces:
+     * - At least one delta across all files
+     * - ADDED/MODIFIED: each requirement has SHALL/MUST and at least one scenario
+     * - REMOVED: names only; no scenario/description required
+     * - RENAMED: pairs well-formed
+     * - No duplicates within sections; no cross-section conflicts per spec
+     */
+    async validateChangeDeltaSpecs(changeDir) {
+        const issues = [];
+        const specsDir = path.join(changeDir, 'specs');
+        let totalDeltas = 0;
+        const missingHeaderSpecs = [];
+        const emptySectionSpecs = [];
+        try {
+            // Discover delta specs at any depth so the nested multi-area layout
+            // (specs/<area>/<capability>/spec.md) is validated, not just the
+            // one-level specs/<capability>/spec.md layout (#1182b). The sdd
+            // specs glob is specs/**/*.md; delta files are always named spec.md.
+            const specFiles = await this.findDeltaSpecFiles(specsDir);
+            for (const specFile of specFiles) {
+                let content;
+                try {
+                    content = await fs.readFile(specFile, 'utf-8');
+                }
+                catch {
+                    continue;
+                }
+                const plan = parseDeltaSpec(content);
+                const entryPath = FileSystemUtils.toPosixPath(path.relative(specsDir, specFile));
+                const sectionNames = [];
+                if (plan.sectionPresence.added)
+                    sectionNames.push('## ADDED Requirements');
+                if (plan.sectionPresence.modified)
+                    sectionNames.push('## MODIFIED Requirements');
+                if (plan.sectionPresence.removed)
+                    sectionNames.push('## REMOVED Requirements');
+                if (plan.sectionPresence.renamed)
+                    sectionNames.push('## RENAMED Requirements');
+                const hasSections = sectionNames.length > 0;
+                const hasEntries = plan.added.length + plan.modified.length + plan.removed.length + plan.renamed.length > 0;
+                if (!hasEntries) {
+                    if (hasSections)
+                        emptySectionSpecs.push({ path: entryPath, sections: sectionNames });
+                    else
+                        missingHeaderSpecs.push(entryPath);
+                }
+                const addedNames = new Set();
+                const modifiedNames = new Set();
+                const removedNames = new Set();
+                const renamedFrom = new Set();
+                const renamedTo = new Set();
+                // Validate ADDED
+                for (const block of plan.added) {
+                    const key = normalizeRequirementName(block.name);
+                    totalDeltas++;
+                    if (addedNames.has(key)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Duplicate requirement in ADDED: "${block.name}"` });
+                    }
+                    else {
+                        addedNames.add(key);
+                    }
+                    const requirementText = this.extractRequirementText(block.raw);
+                    if (!requirementText) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `ADDED "${block.name}" is missing requirement text` });
+                    }
+                    else if (!this.containsShallOrMust(requirementText)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: this.buildMissingShallOrMustMessage(`ADDED "${block.name}"`, block.name) });
+                    }
+                    const scenarioCount = this.countScenarios(block.raw);
+                    if (scenarioCount < 1) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `ADDED "${block.name}" must include at least one scenario` });
+                    }
+                }
+                // Validate MODIFIED
+                for (const block of plan.modified) {
+                    const key = normalizeRequirementName(block.name);
+                    totalDeltas++;
+                    if (modifiedNames.has(key)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Duplicate requirement in MODIFIED: "${block.name}"` });
+                    }
+                    else {
+                        modifiedNames.add(key);
+                    }
+                    const requirementText = this.extractRequirementText(block.raw);
+                    if (!requirementText) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `MODIFIED "${block.name}" is missing requirement text` });
+                    }
+                    else if (!this.containsShallOrMust(requirementText)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: this.buildMissingShallOrMustMessage(`MODIFIED "${block.name}"`, block.name) });
+                    }
+                    const scenarioCount = this.countScenarios(block.raw);
+                    if (scenarioCount < 1) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `MODIFIED "${block.name}" must include at least one scenario` });
+                    }
+                }
+                // Validate REMOVED (names only)
+                for (const name of plan.removed) {
+                    const key = normalizeRequirementName(name);
+                    totalDeltas++;
+                    if (removedNames.has(key)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Duplicate requirement in REMOVED: "${name}"` });
+                    }
+                    else {
+                        removedNames.add(key);
+                    }
+                }
+                // Validate RENAMED pairs
+                for (const { from, to } of plan.renamed) {
+                    const fromKey = normalizeRequirementName(from);
+                    const toKey = normalizeRequirementName(to);
+                    totalDeltas++;
+                    if (renamedFrom.has(fromKey)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Duplicate FROM in RENAMED: "${from}"` });
+                    }
+                    else {
+                        renamedFrom.add(fromKey);
+                    }
+                    if (renamedTo.has(toKey)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Duplicate TO in RENAMED: "${to}"` });
+                    }
+                    else {
+                        renamedTo.add(toKey);
+                    }
+                }
+                // Cross-section conflicts (within the same spec file)
+                for (const n of modifiedNames) {
+                    if (removedNames.has(n)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Requirement present in both MODIFIED and REMOVED: "${n}"` });
+                    }
+                    if (addedNames.has(n)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Requirement present in both MODIFIED and ADDED: "${n}"` });
+                    }
+                }
+                for (const n of addedNames) {
+                    if (removedNames.has(n)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `Requirement present in both ADDED and REMOVED: "${n}"` });
+                    }
+                }
+                for (const { from, to } of plan.renamed) {
+                    const fromKey = normalizeRequirementName(from);
+                    const toKey = normalizeRequirementName(to);
+                    if (modifiedNames.has(fromKey)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `MODIFIED references old name from RENAMED. Use new header for "${to}"` });
+                    }
+                    if (addedNames.has(toKey)) {
+                        issues.push({ level: 'ERROR', path: entryPath, message: `RENAMED TO collides with ADDED for "${to}"` });
+                    }
+                }
+            }
+        }
+        catch {
+            // If no specs dir, treat as no deltas
+        }
+        for (const { path: specPath, sections } of emptySectionSpecs) {
+            issues.push({
+                level: 'ERROR',
+                path: specPath,
+                message: `Delta sections ${this.formatSectionList(sections)} were found, but no requirement entries parsed. Ensure each section includes at least one "### Requirement:" block (REMOVED may use bullet list syntax).`,
+            });
+        }
+        for (const path of missingHeaderSpecs) {
+            issues.push({
+                level: 'ERROR',
+                path,
+                message: 'No delta sections found. Add headers such as "## ADDED Requirements" or move non-delta notes outside specs/.',
+            });
+        }
+        if (totalDeltas === 0) {
+            issues.push({ level: 'ERROR', path: 'file', message: this.enrichTopLevelError('change', VALIDATION_MESSAGES.CHANGE_NO_DELTAS) });
+        }
+        return this.createReport(issues);
+    }
+    /**
+     * Recursively collect every delta `spec.md` under a change's specs directory,
+     * so both the one-level (specs/<capability>/spec.md) and nested multi-area
+     * (specs/<area>/<capability>/spec.md) layouts are discovered (#1182b).
+     * Returns absolute paths, sorted for deterministic issue ordering.
+     */
+    async findDeltaSpecFiles(specsDir) {
+        const results = [];
+        const walk = async (dir) => {
+            let entries;
+            try {
+                entries = await fs.readdir(dir, { withFileTypes: true });
+            }
+            catch {
+                return;
+            }
+            for (const entry of entries) {
+                const full = path.join(dir, entry.name);
+                if (entry.isDirectory()) {
+                    await walk(full);
+                }
+                else if (entry.isFile() && entry.name === 'spec.md') {
+                    results.push(full);
+                }
+            }
+        };
+        await walk(specsDir);
+        return results.sort();
+    }
+    convertZodErrors(error) {
+        return error.issues.map(err => {
+            let message = err.message;
+            if (message === VALIDATION_MESSAGES.CHANGE_NO_DELTAS) {
+                message = `${message}. ${VALIDATION_MESSAGES.GUIDE_NO_DELTAS}`;
+            }
+            return {
+                level: 'ERROR',
+                path: err.path.join('.'),
+                message,
+            };
+        });
+    }
+    applySpecRules(spec, content) {
+        const issues = [];
+        for (const structuralIssue of findMainSpecStructureIssues(content)) {
+            issues.push({
+                level: 'ERROR',
+                path: 'file',
+                line: structuralIssue.line,
+                message: structuralIssue.message,
+            });
+        }
+        if (spec.overview.length < MIN_PURPOSE_LENGTH) {
+            issues.push({
+                level: 'WARNING',
+                path: 'overview',
+                message: VALIDATION_MESSAGES.PURPOSE_TOO_BRIEF,
+            });
+        }
+        spec.requirements.forEach((req, index) => {
+            if (req.text.length > MAX_REQUIREMENT_TEXT_LENGTH) {
+                issues.push({
+                    level: 'INFO',
+                    path: `requirements[${index}]`,
+                    message: VALIDATION_MESSAGES.REQUIREMENT_TOO_LONG,
+                });
+            }
+            if (req.scenarios.length === 0) {
+                issues.push({
+                    level: 'WARNING',
+                    path: `requirements[${index}].scenarios`,
+                    message: `${VALIDATION_MESSAGES.REQUIREMENT_NO_SCENARIOS}. ${VALIDATION_MESSAGES.GUIDE_SCENARIO_FORMAT}`,
+                });
+            }
+        });
+        // SHALL/MUST body-keyword enforcement for main specs (#1156). The main-spec
+        // parser collapses the requirement header into `text`, so we recover the
+        // header+body pairs here (the same source the delta path trusts) and reuse
+        // the delta detection: a body that omits the keyword errors, with the
+        // targeted "move it to the body line" hint when the keyword is in the header
+        // only and the generic message otherwise. Emitted exactly once per
+        // requirement (the Zod refine that used to emit a generic error is removed).
+        extractRequirementsSection(content).bodyBlocks.forEach((block, index) => {
+            const requirementText = this.extractRequirementText(block.raw);
+            if (!requirementText || !this.containsShallOrMust(requirementText)) {
+                issues.push({
+                    level: 'ERROR',
+                    path: `requirements[${index}]`,
+                    message: this.buildMissingShallOrMustMessage(`Requirement "${block.name}"`, block.name),
+                });
+            }
+        });
+        return issues;
+    }
+    applyChangeRules(change, content) {
+        const issues = [];
+        const MIN_DELTA_DESCRIPTION_LENGTH = 10;
+        change.deltas.forEach((delta, index) => {
+            if (!delta.description || delta.description.length < MIN_DELTA_DESCRIPTION_LENGTH) {
+                issues.push({
+                    level: 'WARNING',
+                    path: `deltas[${index}].description`,
+                    message: VALIDATION_MESSAGES.DELTA_DESCRIPTION_TOO_BRIEF,
+                });
+            }
+            if ((delta.operation === 'ADDED' || delta.operation === 'MODIFIED') &&
+                (!delta.requirements || delta.requirements.length === 0)) {
+                issues.push({
+                    level: 'WARNING',
+                    path: `deltas[${index}].requirements`,
+                    message: `${delta.operation} ${VALIDATION_MESSAGES.DELTA_MISSING_REQUIREMENTS}`,
+                });
+            }
+        });
+        return issues;
+    }
+    enrichTopLevelError(itemId, baseMessage) {
+        const msg = baseMessage.trim();
+        if (msg === VALIDATION_MESSAGES.CHANGE_NO_DELTAS) {
+            return `${msg}. ${VALIDATION_MESSAGES.GUIDE_NO_DELTAS}`;
+        }
+        if (msg.includes('Spec must have a Purpose section') || msg.includes('Spec must have a Requirements section')) {
+            return `${msg}. ${VALIDATION_MESSAGES.GUIDE_MISSING_SPEC_SECTIONS}`;
+        }
+        if (msg.includes('Change must have a Why section') || msg.includes('Change must have a What Changes section')) {
+            return `${msg}. ${VALIDATION_MESSAGES.GUIDE_MISSING_CHANGE_SECTIONS}`;
+        }
+        return msg;
+    }
+    extractNameFromPath(filePath) {
+        const normalizedPath = FileSystemUtils.toPosixPath(filePath);
+        const parts = normalizedPath.split('/');
+        // Look for the directory name after 'specs' or 'changes'
+        for (let i = parts.length - 1; i >= 0; i--) {
+            if (parts[i] === 'specs' || parts[i] === 'changes') {
+                if (i < parts.length - 1) {
+                    return parts[i + 1];
+                }
+            }
+        }
+        // Fallback to filename without extension if not in expected structure
+        const fileName = parts[parts.length - 1] ?? '';
+        const dotIndex = fileName.lastIndexOf('.');
+        return dotIndex > 0 ? fileName.slice(0, dotIndex) : fileName;
+    }
+    createReport(issues) {
+        const errors = issues.filter(i => i.level === 'ERROR').length;
+        const warnings = issues.filter(i => i.level === 'WARNING').length;
+        const info = issues.filter(i => i.level === 'INFO').length;
+        const valid = this.strictMode
+            ? errors === 0 && warnings === 0
+            : errors === 0;
+        return {
+            valid,
+            issues,
+            summary: {
+                errors,
+                warnings,
+                info,
+            },
+        };
+    }
+    isValid(report) {
+        return report.valid;
+    }
+    extractRequirementText(blockRaw) {
+        // Delegate to the shared, fence-/metadata-/multi-line-aware body reader.
+        // Validation intentionally does not use the parser/display header-title
+        // fallback for canonical `### Requirement:` blocks: a SHALL/MUST that
+        // appears only in the header must still receive the body-keyword hint.
+        // Line 0 is the `### Requirement: ...` header.
+        const [, ...bodyLines] = blockRaw.split('\n');
+        return extractRequirementBodyShared(bodyLines) || undefined;
+    }
+    containsShallOrMust(text) {
+        return containsShallOrMustShared(text);
+    }
+    /**
+     * Build an error message for a requirement block whose body lacks SHALL/MUST.
+     *
+     * When the SHALL/MUST keyword already appears in the requirement header (e.g.
+     * `### Requirement: The system SHALL ...`) the original generic error
+     * ("must contain SHALL or MUST") is confusing because the keyword is visibly
+     * present in the spec. Per the Tospec conventions the keyword has to live
+     * on the requirement body line (the line right after the header), so we point
+     * the author at that exact fix when the keyword is found in the header only.
+     */
+    buildMissingShallOrMustMessage(prefix, blockName) {
+        const base = `${prefix} must contain SHALL or MUST`;
+        if (this.containsShallOrMust(blockName)) {
+            return `${base} in the requirement body, not only in the header. Move the SHALL/MUST statement to the line immediately after the "### Requirement: ..." header.`;
+        }
+        return base;
+    }
+    countScenarios(blockRaw) {
+        // Fence-aware count via the shared reader: a `#### Scenario:` inside a fenced
+        // example is not a real scenario. Drop the header line (index 0).
+        return countScenariosShared(blockRaw.split('\n').slice(1));
+    }
+    formatSectionList(sections) {
+        if (sections.length === 0)
+            return '';
+        if (sections.length === 1)
+            return sections[0];
+        const head = sections.slice(0, -1);
+        const last = sections[sections.length - 1];
+        return `${head.join(', ')} and ${last}`;
+    }
+}
+//# sourceMappingURL=validator.js.map