@tea-agent/loop-agent 0.43.0-next.16 → 0.43.0-next.18

package/dist/task/contract/recover.js

@@ -70,6 +70,12 @@ export async function recoverTaskContract(input) {
             };
         }
         const txId = input.txId ?? unfinished[0];
+        if (txId === undefined) {
+            return {
+                action: "manual-required",
+                message: "no unfinished transactions to recover",
+            };
+        }
         if (!unfinished.includes(txId)) {
             return {
                 action: "manual-required",

package/dist/task/source-prepare/artifact-meta.js

@@ -17,7 +17,10 @@ function parseSimpleFrontmatter(block) {
         const match = /^([A-Za-z0-9_-]+)\s*:\s*(.+?)\s*$/u.exec(line);
         if (!match)
             continue;
-        out[match[1]] = parseYamlishScalar(match[2]);
+        const [, key, value] = match;
+        if (key === undefined || value === undefined)
+            continue;
+        out[key] = parseYamlishScalar(value);
     }
     return out;
 }
@@ -46,7 +49,11 @@ export function detectProductArtifactMeta(markdown) {
     if (!match) {
         return { artifactType: "unknown", recognized: false };
     }
-    const fields = parseSimpleFrontmatter(match[1]);
+    const [, frontmatter] = match;
+    if (frontmatter === undefined) {
+        return { artifactType: "unknown", recognized: false };
+    }
+    const fields = parseSimpleFrontmatter(frontmatter);
     const artifactType = normalizeArtifactType(fields.artifact_type);
     const recognized = artifactType !== "unknown" || Boolean(fields.artifact_version);
     return {

package/dist/task/source-prepare/fragment-inventory.js

@@ -122,7 +122,10 @@ function isAtxHeading(line) {
     const match = /^(#{1,6})\s+(.+?)\s*$/u.exec(line);
     if (!match)
         return null;
-    return { level: match[1].length, text: match[2].trim() };
+    const [, hashes, text] = match;
+    if (hashes === undefined || text === undefined)
+        return null;
+    return { level: hashes.length, text: text.trim() };
 }
 function isListItem(line) {
     return /^\s*(?:[-*+]|\d+[.)])\s+(?:\[(?: |x|X)\]\s+)?/.test(line);

package/dist/task/source-prepare/parse-intent.js

@@ -59,11 +59,21 @@ function stripFencedAndComments(text) {
         .replace(/<!--[\s\S]*?-->/g, "\n");
 }
 function normalizeAcceptanceId(raw) {
-    return raw
+    return (raw
         .trim()
         .toUpperCase()
         .replace(/_/g, "-")
-        .replace(/^(AC)(FE|BE|UI|API)(?=-|\d)/u, "$1-$2");
+        .replace(/^(AC)(FE|BE|UI|API)(?=-|\d)/u, "$1-$2")
+        // A PRD writes `AC1` as naturally as `AC-1` or `AC_1`. Only the two earlier
+        // rules canonicalized a separator, so a bare `AC1` escaped verbatim — and the
+        // generated DAG's requirement-id schema requires a hyphen after the prefix
+        // (`/^(?:REQ|BR|AC)-[A-Z0-9]+(?:-[A-Z0-9]+)*$/`). Other consumers already
+        // assume `AC-...` (e.g. the frontend-test checklist demands `AC-FE-001`).
+        // The result was a meaningless split: the same PRD style generated fine under
+        // `taskKind: standard` and failed DAG generation under
+        // `taskKind: frontend-implementation`. Canonicalize here, once, so every
+        // workflow reads the same ids.
+        .replace(/^AC(?=\d)/u, "AC-"));
 }
 function parseEmbeddedAcHeading(headingText) {
     const normalized = headingText
@@ -77,7 +87,10 @@ function parseEmbeddedAcHeading(headingText) {
     const match = EMBEDDED_AC_HEADING.exec(normalized);
     if (!match)
         return null;
-    const id = normalizeAcceptanceId(match[1]);
+    const [, rawId] = match;
+    if (rawId === undefined)
+        return null;
+    const id = normalizeAcceptanceId(rawId);
     // Bold-wrapped ids leave their closing marker inside the captured text
     // ("**AC-FE-INS-001**：xxx" captures "**：xxx"); strip emphasis remnants
     // and leading separators so the bound text is clean prose.
@@ -154,13 +167,17 @@ function isAtxHeading(line) {
     const match = /^(#{1,6})\s+(.+?)\s*$/u.exec(line);
     if (!match)
         return null;
-    return { level: match[1].length, text: match[2] };
+    const [, hashes, text] = match;
+    if (hashes === undefined || text === undefined)
+        return null;
+    return { level: hashes.length, text };
 }
 function parseListItem(line) {
     const match = /^\s*(?:[-*+]|\d+[.)])\s+(?:\[(?: |x|X)\]\s+)?(.+?)\s*$/u.exec(line);
     if (!match)
         return null;
-    return match[1].trim();
+    const [, item] = match;
+    return item === undefined ? null : item.trim();
 }
 function parseAcceptanceLine(line, nextId) {
     const checklist = /^\s*[-*+]\s+\[(?: |x|X)\]\s+(.+?)\s*$/u.exec(line);
@@ -173,14 +190,20 @@ function parseAcceptanceLine(line, nextId) {
         return embedded;
     const withId = /^(AC[-_]?\d+)\s*[:：]\s*(.+)$/iu.exec(body);
     if (withId) {
-        return { id: normalizeAcceptanceId(withId[1]), text: withId[2].trim() };
+        const [, rawId, rawText] = withId;
+        if (rawId !== undefined && rawText !== undefined) {
+            return { id: normalizeAcceptanceId(rawId), text: rawText.trim() };
+        }
     }
     const bareAc = /^(AC[-_]?\d+)\s+(.+)$/iu.exec(body);
     if (bareAc) {
-        return {
-            id: normalizeAcceptanceId(bareAc[1]),
-            text: bareAc[2].trim(),
-        };
+        const [, rawId, rawText] = bareAc;
+        if (rawId !== undefined && rawText !== undefined) {
+            return {
+                id: normalizeAcceptanceId(rawId),
+                text: rawText.trim(),
+            };
+        }
     }
     return { id: nextId(), text: body };
 }
@@ -395,7 +418,9 @@ export function mergeAcceptanceCriteria(parts) {
     for (const id of order) {
         const texts = [...(byId.get(id) ?? [])];
         if (texts.length === 1) {
-            items.push({ id, text: texts[0] });
+            const [onlyText] = texts;
+            if (onlyText !== undefined)
+                items.push({ id, text: onlyText });
         }
         else if (texts.length > 1) {
             conflicts.push({ id, texts });

package/dist/task/task-demand-routing.js

@@ -102,11 +102,14 @@ function stripOutOfScopeSections(markdown) {
     for (const line of markdown.split(/\r?\n/)) {
         const heading = /^(#{1,6})\s+(.+?)\s*$/.exec(line.trim());
         if (heading) {
-            const level = heading[1].length;
+            const [, hashes, title] = heading;
+            if (hashes === undefined || title === undefined)
+                continue;
+            const level = hashes.length;
             if (skippedHeadingLevel !== null && level <= skippedHeadingLevel) {
                 skippedHeadingLevel = null;
             }
-            if (OUT_OF_SCOPE_HEADING.test(heading[2].trim())) {
+            if (OUT_OF_SCOPE_HEADING.test(title.trim())) {
                 skippedHeadingLevel = level;
                 continue;
             }