@fitlab-ai/agent-infra 0.9.13 → 0.9.15

package/dist/lib/task/sections.js

@@ -1,3 +1,4 @@
+import { scanVisibleMarkdown } from './markdown.js';
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
 }
@@ -27,10 +28,13 @@ function parseTable(content, input) {
         throw new DocumentMutationError('TASK_DOCUMENT_INVALID', 'table section is ambiguous');
     }
     const section = sections[0];
+    const visibleStarts = new Set(scanVisibleMarkdown(content).lines.map((line) => line.start));
     const sectionLines = linesOf(content).filter((line) => line.start >= section.bodyStart && line.start < section.end);
     const firstContent = sectionLines.findIndex((line) => line.text.trim() !== '');
     const tables = [];
     for (let index = 0; index + 1 < sectionLines.length; index += 1) {
+        if (!visibleStarts.has(sectionLines[index].start) || !visibleStarts.has(sectionLines[index + 1].start))
+            continue;
         const header = splitTableCells(sectionLines[index].text);
         const separator = splitTableCells(sectionLines[index + 1].text);
         if (!header || !separator || header.cells.length !== input.columns.length)
@@ -97,22 +101,23 @@ function documentEol(content) {
 }
 function matchingSections(content, aliases) {
     const allowed = new Set(aliases);
-    return linesOf(content)
-        .map((line, index, lines) => {
-        const match = /^##\s+(.+?)\s*$/.exec(line.text);
-        if (!match?.[1] || !allowed.has(match[1]))
+    const lines = new Map(linesOf(content).map((line) => [line.start, line]));
+    const headings = scanVisibleMarkdown(content).headings.filter((heading) => heading.level === 2 || heading.level === 3);
+    return headings
+        .map((heading, index) => {
+        if (!allowed.has(heading.text))
             return null;
-        let end = content.length;
-        for (let i = index + 1; i < lines.length; i += 1) {
-            if (/^##\s+/.test(lines[i].text)) {
-                end = lines[i].start;
-                break;
-            }
-        }
-        return { heading: match[1], line, bodyStart: line.end, end };
+        const next = headings.slice(index + 1).find((candidate) => candidate.level <= heading.level);
+        const line = lines.get(heading.start);
+        return { heading: heading.text, level: heading.level, line, bodyStart: line.end, end: next?.start ?? content.length };
     })
         .filter((match) => match !== null);
 }
+// Select the first visible section at the requested level, in document order.
+function findSectionRange(content, aliases, level = 2) {
+    const section = matchingSections(content, aliases).find((candidate) => candidate.level === level);
+    return section ? { start: section.line.start, bodyStart: section.bodyStart, end: section.end, heading: section.heading } : null;
+}
 function validateAliases(aliases, name) {
     if (!Array.isArray(aliases) || aliases.length === 0 || aliases.some((alias) => !alias || /[\r\n]/.test(alias))) {
         throw new DocumentMutationError('MUTATION_INVALID', `${name} must contain headings`);
@@ -259,9 +264,12 @@ function mutateTableRow(content, mutation) {
         throw new DocumentMutationError(sections.length === 0 ? 'TABLE_NOT_FOUND' : 'TASK_DOCUMENT_INVALID', sections.length === 0 ? 'table section not found' : 'table section is ambiguous');
     }
     const section = sections[0];
+    const visibleStarts = new Set(scanVisibleMarkdown(content).lines.map((line) => line.start));
     const sectionLines = linesOf(content).filter((line) => line.start >= section.bodyStart && line.start < section.end);
     const tables = [];
     for (let i = 0; i + 1 < sectionLines.length; i += 1) {
+        if (!visibleStarts.has(sectionLines[i].start) || !visibleStarts.has(sectionLines[i + 1].start))
+            continue;
         const header = splitTableCells(sectionLines[i].text);
         const separator = splitTableCells(sectionLines[i + 1].text);
         if (!header || !separator || header.cells.length !== mutation.columns.length)
@@ -366,25 +374,12 @@ function mutateTableRow(content, mutation) {
  * lines are trimmed. Returns '' when no alias heading is present.
  */
 function extractSection(content, aliases) {
-    const lines = content.split('\n');
-    let start = -1;
-    for (let i = 0; i < lines.length; i += 1) {
-        const line = lines[i].trim();
-        if (aliases.some((alias) => new RegExp(`^##\\s+${escapeRegExp(alias)}\\s*$`).test(line))) {
-            start = i + 1;
-            break;
-        }
-    }
-    if (start === -1)
+    const headings = scanVisibleMarkdown(content).headings.filter((heading) => heading.level === 2);
+    const index = headings.findIndex((heading) => aliases.includes(heading.text));
+    if (index === -1)
         return '';
-    let end = lines.length;
-    for (let i = start; i < lines.length; i += 1) {
-        if (/^##\s+/.test(lines[i])) {
-            end = i;
-            break;
-        }
-    }
-    return lines.slice(start, end).join('\n').replace(/^\n+/, '').replace(/\n+$/, '');
+    return content.slice(headings[index].end + 1, headings[index + 1]?.start ?? content.length)
+        .replace(/^(?:\r?\n)+|(?:\r?\n)+$/g, '');
 }
 /**
  * Return the heading alias that actually appears as a `## {alias}` line, so a
@@ -392,11 +387,8 @@ function extractSection(content, aliases) {
  * alias when none is present.
  */
 function findSectionHeading(content, aliases) {
-    for (const alias of aliases) {
-        if (new RegExp(`^##\\s+${escapeRegExp(alias)}\\s*$`, 'm').test(content))
-            return alias;
-    }
-    return aliases[0];
+    const headings = scanVisibleMarkdown(content).headings.filter((heading) => heading.level === 2);
+    return aliases.find((alias) => headings.some((heading) => heading.text === alias)) ?? aliases[0];
 }
 /**
  * Return the body of the first `### {headingPrefix}` sub-section, from the
@@ -407,25 +399,13 @@ function findSectionHeading(content, aliases) {
  * and trailing blank lines are trimmed. Returns '' when no match is present.
  */
 function extractSubSection(content, headingPrefix) {
-    const lines = content.split('\n');
-    const headRe = new RegExp(`^###\\s+${escapeRegExp(headingPrefix)}(?![\\w-])`);
-    let start = -1;
-    for (let i = 0; i < lines.length; i += 1) {
-        if (headRe.test(lines[i].trim())) {
-            start = i;
-            break;
-        }
-    }
-    if (start === -1)
+    const headings = scanVisibleMarkdown(content).headings.filter((heading) => heading.level === 2 || heading.level === 3);
+    const headRe = new RegExp(`^${escapeRegExp(headingPrefix)}(?![\\w-])`);
+    const index = headings.findIndex((heading) => heading.level === 3 && headRe.test(heading.text));
+    if (index === -1)
         return '';
-    let end = lines.length;
-    for (let i = start + 1; i < lines.length; i += 1) {
-        if (/^###?\s+/.test(lines[i])) {
-            end = i;
-            break;
-        }
-    }
-    return lines.slice(start, end).join('\n').replace(/^\n+/, '').replace(/\n+$/, '');
+    return content.slice(headings[index].start, headings[index + 1]?.start ?? content.length)
+        .replace(/^(?:\r?\n)+|(?:\r?\n)+$/g, '');
 }
-export { extractSection, findSectionHeading, extractSubSection, parseTable, upsertSection, mutateTableRow, DocumentMutationError };
+export { findSectionRange, extractSection, findSectionHeading, extractSubSection, parseTable, upsertSection, mutateTableRow, DocumentMutationError };
 //# sourceMappingURL=sections.js.map

package/dist/lib/task/verification-config.js

@@ -1,5 +1,6 @@
 import fs from 'node:fs';
 import path from 'node:path';
+import { getArtifactSchema } from './artifact-schema.js';
 function loadVerificationConfig(repositoryRoot, skillName) {
     const verifyPath = path.join(repositoryRoot, '.agents', 'skills', skillName, 'config', 'verify.json');
     if (!fs.existsSync(verifyPath))
@@ -8,6 +9,16 @@ function loadVerificationConfig(repositoryRoot, skillName) {
     if (!parsed.checks || typeof parsed.checks !== 'object' || Array.isArray(parsed.checks)) {
         throw new Error(`config/verify.json has invalid checks for skill '${skillName}'`);
     }
+    const artifact = parsed.checks.artifact;
+    if (artifact && typeof artifact === 'object' && !Array.isArray(artifact)) {
+        const schema = artifact.schema;
+        if (schema !== undefined && (typeof schema !== 'string' || !getArtifactSchema(schema))) {
+            throw new Error(`config/verify.json has unknown artifact schema '${String(schema)}' for skill '${skillName}'`);
+        }
+        if (schema !== undefined && 'required_patterns' in artifact) {
+            throw new Error(`config/verify.json must not duplicate required_patterns for schema '${String(schema)}' in skill '${skillName}'`);
+        }
+    }
     return { ...(parsed.skill ? { skill: parsed.skill } : {}), checks: parsed.checks };
 }
 export { loadVerificationConfig };