@fission-ai/openspec 1.4.1 → 1.6.0-beta.1

package/dist/core/openers.js

@@ -0,0 +1,251 @@
+import * as nodeFs from 'node:fs';
+import * as path from 'node:path';
+import * as os from 'node:os';
+import { z } from 'zod';
+import { StoreError } from './store/errors.js';
+import { formatZodIssues } from './zod-issues.js';
+const DEFAULT_ATTACH_FLAG = '--add-dir';
+/**
+ * Temporary kill-switch (2026-06): worksets open only in IDE-style
+ * ('workspace-file') tools while the CLI-agent ('attach-dirs') open flow
+ * is reworked. The agents (Claude Code, codex) launch in a single primary
+ * cwd rather than a true combined multi-root view, which makes "where does
+ * my change land?" ambiguous. Default off; set
+ * OPENSPEC_ENABLE_CLI_AGENT_OPENERS=1 to restore them (internal rollback seam).
+ */
+export function isCliAgentOpenersEnabled() {
+    return process.env.OPENSPEC_ENABLE_CLI_AGENT_OPENERS === '1';
+}
+/** Whether a tool can be opened right now (CLI-agent styles are gated). */
+export function isOpenerEnabled(opener) {
+    return isCliAgentOpenersEnabled() || opener.style !== 'attach-dirs';
+}
+export const BUILTIN_OPENERS = [
+    {
+        id: 'code',
+        label: 'VS Code',
+        style: 'workspace-file',
+        command: 'code',
+        args: [],
+        attachFlag: DEFAULT_ATTACH_FLAG,
+    },
+    {
+        id: 'cursor',
+        label: 'Cursor',
+        style: 'workspace-file',
+        command: 'cursor',
+        args: [],
+        attachFlag: DEFAULT_ATTACH_FLAG,
+    },
+    {
+        id: 'claude',
+        label: 'Claude Code',
+        style: 'attach-dirs',
+        command: 'claude',
+        args: [],
+        attachFlag: DEFAULT_ATTACH_FLAG,
+    },
+    {
+        id: 'codex',
+        label: 'codex',
+        style: 'attach-dirs',
+        command: 'codex',
+        args: ['--sandbox', 'workspace-write'],
+        attachFlag: DEFAULT_ATTACH_FLAG,
+    },
+];
+const OPENER_STYLES = ['workspace-file', 'attach-dirs'];
+const OpenerConfigRowSchema = z
+    .object({
+    style: z.enum(OPENER_STYLES).optional(),
+    label: z.string().min(1).optional(),
+    command: z.string().min(1).optional(),
+    args: z.array(z.string()).optional(),
+    attach_flag: z.string().min(1).optional(),
+})
+    .strict();
+const OpenersConfigSchema = z.record(z.string(), OpenerConfigRowSchema);
+function invalidOpenerConfigError(message, configPath) {
+    return new StoreError(`Invalid openers config: ${message}`, 'invalid_opener_config', {
+        target: 'openers.config',
+        fix: `Each entry under "openers" in ${configPath} may set style ('workspace-file' or 'attach-dirs'), label, command, args, and attach_flag; new tools must set style.`,
+    });
+}
+/**
+ * Merges the global config file's raw `openers` value over the
+ * built-in table. A row keyed by a built-in id overrides only the
+ * fields it sets; a new id adds a tool (style required, command and
+ * label default to the id). Malformed rows fail typed - never
+ * silently ignored.
+ */
+function cloneOpener(opener) {
+    return { ...opener, args: [...opener.args] };
+}
+export function mergeOpenerTable(rawOpeners, configPath) {
+    if (rawOpeners === undefined || rawOpeners === null) {
+        return BUILTIN_OPENERS.map(cloneOpener);
+    }
+    const result = OpenersConfigSchema.safeParse(rawOpeners);
+    if (!result.success) {
+        throw invalidOpenerConfigError(formatZodIssues(result.error, 'openers'), configPath);
+    }
+    const table = BUILTIN_OPENERS.map(cloneOpener);
+    for (const [id, row] of Object.entries(result.data)) {
+        const builtinIndex = table.findIndex((opener) => opener.id === id);
+        if (builtinIndex >= 0) {
+            const builtin = table[builtinIndex];
+            table[builtinIndex] = {
+                ...builtin,
+                ...(row.style !== undefined ? { style: row.style } : {}),
+                ...(row.label !== undefined ? { label: row.label } : {}),
+                ...(row.command !== undefined ? { command: row.command } : {}),
+                ...(row.args !== undefined ? { args: row.args } : {}),
+                ...(row.attach_flag !== undefined
+                    ? { attachFlag: row.attach_flag }
+                    : {}),
+            };
+            continue;
+        }
+        if (row.style === undefined) {
+            throw invalidOpenerConfigError(`'${id}' adds a new tool and must set style ('workspace-file' or 'attach-dirs')`, configPath);
+        }
+        table.push({
+            id,
+            label: row.label ?? id,
+            style: row.style,
+            command: row.command ?? id,
+            args: row.args ?? [],
+            attachFlag: row.attach_flag ?? DEFAULT_ATTACH_FLAG,
+        });
+    }
+    return table;
+}
+function getPathValue(env) {
+    return env.PATH ?? env.Path ?? env.path ?? '';
+}
+function getPathExtensions(platform, env) {
+    if (platform !== 'win32') {
+        return [''];
+    }
+    return (env.PATHEXT ?? '.COM;.EXE;.BAT;.CMD')
+        .split(';')
+        .map((extension) => extension.trim())
+        .filter((extension) => extension.length > 0);
+}
+function defaultIsExecutableFile(candidatePath, platform) {
+    try {
+        if (!nodeFs.statSync(candidatePath).isFile()) {
+            return false;
+        }
+    }
+    catch {
+        return false;
+    }
+    if (platform === 'win32') {
+        return true;
+    }
+    try {
+        nodeFs.accessSync(candidatePath, nodeFs.constants.X_OK);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * PATH availability scan (ported from the deleted workspace openers
+ * at f858c19^, sharpened for injectability: the path module is keyed
+ * by the injected platform, and a command already carrying a known
+ * executable extension matches as-is).
+ */
+export function isOpenerCommandAvailable(command, options = {}) {
+    const env = options.env ?? process.env;
+    const platform = options.platform ?? os.platform();
+    const pathModule = platform === 'win32' ? path.win32 : path.posix;
+    const isExecutable = options.isExecutableFile ??
+        ((candidate) => defaultIsExecutableFile(candidate, platform));
+    const extensions = getPathExtensions(platform, env);
+    const lowerCommand = command.toLowerCase();
+    const carriesKnownExtension = extensions.some((extension) => extension.length > 0 && lowerCommand.endsWith(extension.toLowerCase()));
+    // One suffix policy: a command already carrying a known executable
+    // extension matches as-is and never gets a second extension appended
+    // - agreeing with spawn-time resolution.
+    const suffixes = carriesKnownExtension ? [''] : extensions;
+    if (/[\\/]/u.test(command)) {
+        // Direct paths additionally match bare even on win32 (the spawn
+        // call receives the literal path).
+        const directSuffixes = Array.from(new Set(['', ...suffixes]));
+        return directSuffixes.some((suffix) => isExecutable(command + suffix));
+    }
+    for (const directory of getPathValue(env).split(pathModule.delimiter)) {
+        if (directory.length === 0) {
+            continue;
+        }
+        if (suffixes.some((suffix) => isExecutable(pathModule.join(directory, command + suffix)))) {
+            return true;
+        }
+    }
+    return false;
+}
+/** Table order preserved, available tools first (stable sort). */
+export function listOpenerChoices(table, options = {}) {
+    return table
+        .filter((opener) => isOpenerEnabled(opener))
+        .map((opener) => {
+        const available = isOpenerCommandAvailable(opener.command, options);
+        return {
+            opener,
+            available,
+            note: available ? null : `(${opener.command} not found on PATH)`,
+        };
+    })
+        .sort((a, b) => {
+        if (a.available === b.available) {
+            return 0;
+        }
+        return a.available ? -1 : 1;
+    });
+}
+export function findOpener(table, id) {
+    return table.find((opener) => opener.id === id) ?? null;
+}
+/**
+ * Pure argv builder. workspace-file: pre-args + the generated file's
+ * absolute path (which also defuses the cursor shim's `agent`
+ * first-arg hijack). attach-dirs: pre-args + one attach flag + path
+ * pair per surviving member, the primary included (the locked "one
+ * attach flag per member"); never a trailing positional - both agent
+ * CLIs would read one as a starter prompt, which 7.1 locks out.
+ */
+export function buildLaunchCommand(opener, input) {
+    if (input.members.length === 0) {
+        throw new Error('buildLaunchCommand requires at least one member.');
+    }
+    // The no-hijack and no-positional guarantees lean on absolute paths
+    // (the child resolves relative argv against its own cwd) - keep the
+    // invariant local instead of three modules away.
+    if (!path.isAbsolute(input.codeWorkspacePath)) {
+        throw new Error(`buildLaunchCommand requires an absolute workspace-file path (got '${input.codeWorkspacePath}').`);
+    }
+    const cwd = input.members[0].path;
+    if (opener.style === 'workspace-file') {
+        return {
+            executable: opener.command,
+            args: [...opener.args, input.codeWorkspacePath],
+            cwd,
+            label: opener.label,
+            style: opener.style,
+        };
+    }
+    return {
+        executable: opener.command,
+        args: [
+            ...opener.args,
+            ...input.members.flatMap((member) => [opener.attachFlag, member.path]),
+        ],
+        cwd,
+        label: opener.label,
+        style: opener.style,
+    };
+}
+//# sourceMappingURL=openers.js.map

package/dist/core/openspec-root.d.ts

@@ -0,0 +1,45 @@
+import { type StoreDiagnostic } from './store/errors.js';
+export declare const OPENSPEC_ROOT_DIR = "openspec";
+export declare const OPENSPEC_CONFIG_YAML = "openspec/config.yaml";
+export declare const OPENSPEC_CONFIG_YML = "openspec/config.yml";
+export declare const OPENSPEC_SPECS_DIR = "openspec/specs";
+export declare const OPENSPEC_CHANGES_DIR = "openspec/changes";
+export declare const OPENSPEC_ARCHIVE_DIR = "openspec/changes/archive";
+export declare const DEFAULT_OPENSPEC_SCHEMA = "spec-driven";
+export declare const DIRECTORY_ANCHOR_FILE_NAME = ".gitkeep";
+export declare const ANCHORED_OPENSPEC_DIRS: readonly ["openspec/specs", "openspec/changes/archive"];
+export interface CreatedPathLedgerEntry {
+    relativePath: string;
+    absolutePath: string;
+    kind: 'directory' | 'file';
+}
+export interface OpenSpecRootInspection {
+    present: boolean | null;
+    config: {
+        present: boolean | null;
+        path?: string;
+    };
+    specs: {
+        present: boolean | null;
+    };
+    changes: {
+        present: boolean | null;
+    };
+    archive: {
+        present: boolean | null;
+    };
+    healthy: boolean;
+    diagnostics: StoreDiagnostic[];
+}
+export interface EnsureOpenSpecRootResult {
+    inspection: OpenSpecRootInspection;
+    createdArtifacts: string[];
+    createdPaths: CreatedPathLedgerEntry[];
+}
+export declare function inspectOpenSpecRoot(storeRoot: string): Promise<OpenSpecRootInspection>;
+export interface EnsureOpenSpecRootOptions {
+    anchorEmptyDirectories?: boolean;
+}
+export declare function ensureOpenSpecRoot(storeRoot: string, options?: EnsureOpenSpecRootOptions): Promise<EnsureOpenSpecRootResult>;
+export declare function rollbackCreatedPaths(entries: CreatedPathLedgerEntry[]): Promise<void>;
+//# sourceMappingURL=openspec-root.d.ts.map

package/dist/core/openspec-root.js

@@ -0,0 +1,195 @@
+import * as fs from 'node:fs/promises';
+import * as path from 'node:path';
+import { FileSystemUtils } from '../utils/file-system.js';
+import { serializeConfig } from './config-prompts.js';
+import { makeStoreDiagnostic, } from './store/errors.js';
+export const OPENSPEC_ROOT_DIR = 'openspec';
+export const OPENSPEC_CONFIG_YAML = 'openspec/config.yaml';
+export const OPENSPEC_CONFIG_YML = 'openspec/config.yml';
+export const OPENSPEC_SPECS_DIR = 'openspec/specs';
+export const OPENSPEC_CHANGES_DIR = 'openspec/changes';
+export const OPENSPEC_ARCHIVE_DIR = 'openspec/changes/archive';
+export const DEFAULT_OPENSPEC_SCHEMA = 'spec-driven';
+export const DIRECTORY_ANCHOR_FILE_NAME = '.gitkeep';
+// Git cannot track empty directories, so setup anchors otherwise-empty
+// conventional store directories for teammates who clone the repo later.
+export const ANCHORED_OPENSPEC_DIRS = [OPENSPEC_SPECS_DIR, OPENSPEC_ARCHIVE_DIR];
+async function pathKind(targetPath) {
+    try {
+        const stat = await fs.stat(targetPath);
+        if (stat.isDirectory())
+            return 'directory';
+        if (stat.isFile())
+            return 'file';
+        return 'other';
+    }
+    catch (error) {
+        if (typeof error === 'object' &&
+            error !== null &&
+            'code' in error &&
+            error.code === 'ENOENT') {
+            return 'missing';
+        }
+        throw error;
+    }
+}
+function relativeArtifact(relativePath, kind) {
+    const normalized = FileSystemUtils.toPosixPath(relativePath);
+    return kind === 'directory' ? `${normalized}/` : normalized;
+}
+function unresolvedInspection() {
+    return {
+        present: null,
+        config: { present: null },
+        specs: { present: null },
+        changes: { present: null },
+        archive: { present: null },
+        healthy: false,
+        diagnostics: [],
+    };
+}
+function missingDirectoryDiagnostic(code, message, target) {
+    return makeStoreDiagnostic('error', code, message, { target });
+}
+async function inspectOptionalPlanningDirectory(inspection, storeRoot, key, relativePath, notDirectoryCode, target) {
+    const kind = await pathKind(path.join(storeRoot, relativePath));
+    inspection[key] = { present: kind === 'directory' };
+    if (kind === 'directory' || kind === 'missing')
+        return kind;
+    inspection.diagnostics.push(missingDirectoryDiagnostic(notDirectoryCode, `${relativePath}/ exists but is not a directory.`, target));
+    return kind;
+}
+export async function inspectOpenSpecRoot(storeRoot) {
+    const rootKind = await pathKind(storeRoot);
+    const inspection = unresolvedInspection();
+    if (rootKind === 'missing') {
+        inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_store_root_missing', 'Store root does not exist.', 'store.root'));
+        return inspection;
+    }
+    if (rootKind !== 'directory') {
+        inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_store_root_not_directory', 'Store root is not a directory.', 'store.root'));
+        return inspection;
+    }
+    const openspecPath = path.join(storeRoot, OPENSPEC_ROOT_DIR);
+    const openspecKind = await pathKind(openspecPath);
+    inspection.present = openspecKind === 'directory';
+    if (openspecKind === 'missing') {
+        inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_root_missing', 'Missing openspec/ directory.', 'openspec.root'));
+        return inspection;
+    }
+    if (openspecKind !== 'directory') {
+        inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_root_not_directory', 'openspec/ exists but is not a directory.', 'openspec.root'));
+        return inspection;
+    }
+    const configYamlKind = await pathKind(path.join(storeRoot, OPENSPEC_CONFIG_YAML));
+    const configYmlKind = await pathKind(path.join(storeRoot, OPENSPEC_CONFIG_YML));
+    if (configYamlKind === 'file') {
+        inspection.config = { present: true, path: OPENSPEC_CONFIG_YAML };
+    }
+    else if (configYmlKind === 'file') {
+        inspection.config = { present: true, path: OPENSPEC_CONFIG_YML };
+    }
+    else {
+        inspection.config = { present: false };
+        if (configYamlKind !== 'missing' || configYmlKind !== 'missing') {
+            inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_config_not_file', 'OpenSpec config path exists but is not a file.', 'openspec.config'));
+        }
+        else {
+            inspection.diagnostics.push(missingDirectoryDiagnostic('openspec_config_missing', 'Missing openspec/config.yaml or openspec/config.yml.', 'openspec.config'));
+        }
+    }
+    await inspectOptionalPlanningDirectory(inspection, storeRoot, 'specs', OPENSPEC_SPECS_DIR, 'openspec_specs_not_directory', 'openspec.specs');
+    const changesKind = await inspectOptionalPlanningDirectory(inspection, storeRoot, 'changes', OPENSPEC_CHANGES_DIR, 'openspec_changes_not_directory', 'openspec.changes');
+    if (changesKind === 'directory') {
+        await inspectOptionalPlanningDirectory(inspection, storeRoot, 'archive', OPENSPEC_ARCHIVE_DIR, 'openspec_archive_not_directory', 'openspec.archive');
+    }
+    else {
+        inspection.archive = { present: false };
+    }
+    inspection.healthy =
+        inspection.present === true &&
+            inspection.config.present === true &&
+            inspection.diagnostics.length === 0;
+    return inspection;
+}
+async function ensureDirectory(storeRoot, relativePath, ledger) {
+    const absolutePath = path.join(storeRoot, relativePath);
+    const kind = await pathKind(absolutePath);
+    if (kind === 'directory')
+        return;
+    if (kind !== 'missing') {
+        throw new Error(`${relativePath}/ exists but is not a directory.`);
+    }
+    await fs.mkdir(absolutePath, { recursive: true });
+    ledger.push({
+        relativePath: relativeArtifact(relativePath, 'directory'),
+        absolutePath,
+        kind: 'directory',
+    });
+}
+async function ensureDefaultConfig(storeRoot, ledger) {
+    const configYamlPath = path.join(storeRoot, OPENSPEC_CONFIG_YAML);
+    const configYmlPath = path.join(storeRoot, OPENSPEC_CONFIG_YML);
+    const yamlKind = await pathKind(configYamlPath);
+    const ymlKind = await pathKind(configYmlPath);
+    if (yamlKind === 'file' || ymlKind === 'file')
+        return;
+    if (yamlKind !== 'missing' || ymlKind !== 'missing') {
+        throw new Error('OpenSpec config path exists but is not a file.');
+    }
+    await FileSystemUtils.writeFile(configYamlPath, serializeConfig({ schema: DEFAULT_OPENSPEC_SCHEMA }));
+    ledger.push({
+        relativePath: relativeArtifact(OPENSPEC_CONFIG_YAML, 'file'),
+        absolutePath: configYamlPath,
+        kind: 'file',
+    });
+}
+async function ensureDirectoryAnchor(storeRoot, relativeDir, ledger) {
+    const directory = path.join(storeRoot, relativeDir);
+    if ((await fs.readdir(directory)).length > 0)
+        return;
+    const relativePath = `${relativeDir}/${DIRECTORY_ANCHOR_FILE_NAME}`;
+    const absolutePath = path.join(directory, DIRECTORY_ANCHOR_FILE_NAME);
+    await fs.writeFile(absolutePath, '', 'utf-8');
+    ledger.push({
+        relativePath: relativeArtifact(relativePath, 'file'),
+        absolutePath,
+        kind: 'file',
+    });
+}
+export async function ensureOpenSpecRoot(storeRoot, options = {}) {
+    const ledger = [];
+    const rootKind = await pathKind(storeRoot);
+    if (rootKind === 'missing') {
+        await fs.mkdir(storeRoot, { recursive: true });
+    }
+    else if (rootKind !== 'directory') {
+        throw new Error('Store root is not a directory.');
+    }
+    await ensureDirectory(storeRoot, OPENSPEC_ROOT_DIR, ledger);
+    await ensureDirectory(storeRoot, OPENSPEC_SPECS_DIR, ledger);
+    await ensureDirectory(storeRoot, OPENSPEC_CHANGES_DIR, ledger);
+    await ensureDirectory(storeRoot, OPENSPEC_ARCHIVE_DIR, ledger);
+    await ensureDefaultConfig(storeRoot, ledger);
+    if (options.anchorEmptyDirectories) {
+        for (const relativeDir of ANCHORED_OPENSPEC_DIRS) {
+            await ensureDirectoryAnchor(storeRoot, relativeDir, ledger);
+        }
+    }
+    return {
+        inspection: await inspectOpenSpecRoot(storeRoot),
+        createdArtifacts: ledger.map((entry) => entry.relativePath),
+        createdPaths: ledger,
+    };
+}
+export async function rollbackCreatedPaths(entries) {
+    for (const entry of [...entries].reverse()) {
+        if (entry.kind === 'file') {
+            await fs.rm(entry.absolutePath, { force: true }).catch(() => undefined);
+        }
+        else {
+            await fs.rmdir(entry.absolutePath).catch(() => undefined);
+        }
+    }
+}
+//# sourceMappingURL=openspec-root.js.map

package/dist/core/parsers/change-parser.js

@@ -1,4 +1,5 @@
 import { MarkdownParser } from './markdown-parser.js';
+import { buildCodeFenceMask } from './requirement-text.js';
 import path from 'path';
 import { promises as fs } from 'fs';
 export class ChangeParser extends MarkdownParser {
@@ -148,7 +149,7 @@ export class ChangeParser extends MarkdownParser {
     parseSectionsFromContent(content) {
         const normalizedContent = ChangeParser.normalizeContent(content);
         const lines = normalizedContent.split('\n');
-        const codeFenceLineMask = ChangeParser.buildCodeFenceMask(lines);
+        const codeFenceLineMask = buildCodeFenceMask(lines);
         const sections = [];
         const stack = [];
         for (let i = 0; i < lines.length; i++) {

package/dist/core/parsers/markdown-parser.d.ts

@@ -11,9 +11,6 @@ export declare class MarkdownParser {
     private currentLine;
     constructor(content: string);
     protected static normalizeContent(content: string): string;
-    protected static buildCodeFenceMask(lines: string[]): boolean[];
-    private static getFenceMarker;
-    private static isClosingFence;
     parseSpec(name: string): Spec;
     parseChange(name: string): Change;
     protected parseSections(): Section[];

package/dist/core/parsers/markdown-parser.js

@@ -1,3 +1,4 @@
+import { buildCodeFenceMask, extractRequirementText } from './requirement-text.js';
 export class MarkdownParser {
     lines;
     codeFenceLineMask;
@@ -5,47 +6,12 @@ export class MarkdownParser {
     constructor(content) {
         const normalized = MarkdownParser.normalizeContent(content);
         this.lines = normalized.split('\n');
-        this.codeFenceLineMask = MarkdownParser.buildCodeFenceMask(this.lines);
+        this.codeFenceLineMask = buildCodeFenceMask(this.lines);
         this.currentLine = 0;
     }
     static normalizeContent(content) {
         return content.replace(/\r\n?/g, '\n');
     }
-    static buildCodeFenceMask(lines) {
-        const mask = new Array(lines.length).fill(false);
-        let activeFence = null;
-        for (let i = 0; i < lines.length; i++) {
-            const fence = MarkdownParser.getFenceMarker(lines[i]);
-            if (!activeFence) {
-                if (fence) {
-                    activeFence = fence;
-                    mask[i] = true;
-                }
-                continue;
-            }
-            mask[i] = true;
-            if (MarkdownParser.isClosingFence(lines[i], activeFence)) {
-                activeFence = null;
-            }
-        }
-        return mask;
-    }
-    static getFenceMarker(line) {
-        const fenceMatch = line.match(/^\s*(`{3,}|~{3,})/);
-        if (!fenceMatch) {
-            return null;
-        }
-        return {
-            marker: fenceMatch[1][0],
-            length: fenceMatch[1].length,
-        };
-    }
-    static isClosingFence(line, activeFence) {
-        const fenceMatch = line.match(/^\s*(`{3,}|~{3,})\s*$/);
-        return Boolean(fenceMatch &&
-            fenceMatch[1][0] === activeFence.marker &&
-            fenceMatch[1].length >= activeFence.length);
-    }
     parseSpec(name) {
         const sections = this.parseSections();
         const purpose = this.findSection(sections, 'Purpose')?.content || '';
@@ -149,29 +115,9 @@ export class MarkdownParser {
     parseRequirements(section) {
         const requirements = [];
         for (const child of section.children) {
-            // Extract requirement text from first non-empty content line, fall back to heading
-            let text = child.title;
-            // Get content before any child sections (scenarios)
-            if (child.content.trim()) {
-                // Split content into lines and find content before any child headers
-                const lines = child.content.split('\n');
-                const contentBeforeChildren = [];
-                for (const line of lines) {
-                    // Stop at child headers (scenarios start with ####)
-                    if (line.trim().startsWith('#')) {
-                        break;
-                    }
-                    contentBeforeChildren.push(line);
-                }
-                // Find first non-empty line
-                const directContent = contentBeforeChildren.join('\n').trim();
-                if (directContent) {
-                    const firstLine = directContent.split('\n').find(l => l.trim());
-                    if (firstLine) {
-                        text = firstLine.trim();
-                    }
-                }
-            }
+            // Read the requirement text via the shared reader (multi-line, fence- and
+            // metadata-aware, with the shared header-title fallback for empty bodies).
+            const text = extractRequirementText(child.title, child.content.split('\n'));
             const scenarios = this.parseScenarios(child);
             requirements.push({
                 text,

package/dist/core/parsers/requirement-blocks.d.ts

@@ -15,6 +15,16 @@ export declare function normalizeRequirementName(name: string): string;
  * Extracts the Requirements section from a spec file and parses requirement blocks.
  */
 export declare function extractRequirementsSection(content: string): RequirementsSectionParts;
+/**
+ * A level-3 header inside `## ADDED`/`## MODIFIED Requirements` that is not a
+ * canonical `### Requirement:` header, recorded at the moment the delta reader
+ * skips over it. Surfaced as an INFO note by `validate <change>` (#498).
+ */
+export interface SkippedHeader {
+    header: string;
+    section: string;
+    line: number;
+}
 export interface DeltaPlan {
     added: RequirementBlock[];
     modified: RequirementBlock[];
@@ -23,6 +33,7 @@ export interface DeltaPlan {
         from: string;
         to: string;
     }>;
+    skippedHeaders: SkippedHeader[];
     sectionPresence: {
         added: boolean;
         modified: boolean;