@tea-agent/loop-agent 0.41.1-beta.0 → 0.41.2

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

@@ -175,11 +175,8 @@ export function splitDocumentIntoFragments(doc) {
         const explicitSectionAc = matchExplicitAcHeading(lastHeadingSegment);
         const sectionKind = classifySectionHeading(section.headingPath);
         const effectiveKind = explicitSectionAc ? "acceptance" : sectionKind;
-        // V4 显式 AC 标题行已在下方入口 emit（explicitSectionAc 分支）；正文循环
-        // 必须从标题行的下一行开始，否则同一标题行会被正文循环的 heading 分支
-        // 再次 emit，产生结构性 duplicate（每个 AC 标题式片段出现两次）。
         const bodyStart = section.headingPath
-            ? section.startLine + 1
+            ? section.startLine + (explicitSectionAc ? 0 : 1)
             : section.startLine;
         const bodyEnd = section.endLine;
         const bodyLines = lines.slice(Math.max(0, bodyStart - 1), Math.min(bodyEnd, lines.length));
@@ -441,17 +438,14 @@ export function buildFragmentInventory(input) {
             disposition = "non-goal";
         }
         else {
-            // Non-goal/constraint/acceptance/objective sections are classified
-            // above. An "other" section (背景 / 概述 / 说明 …) is contextual
-            // description, not an executable requirement: even when it mentions
-            // implementation verbs ("需要", "渲染", "组件"), it must not be
-            // promoted to a high-risk unresolved executable fragment — that
-            // blocks PRDs whose 背景 prose happens to use such words (merge
-            // regression: "需要一个小组件" was flagged executable via 需要).
-            // Explicit AC ids are already handled by the branches above, so
-            // this branch is always contextual.
-            disposition = "context-only";
-            reason = "context-only fragment; excluded from canonical requirement coverage";
+            const executable = EXECUTABLE_HINT_RE.test(text);
+            disposition = executable ? "unresolved" : "context-only";
+            if (disposition === "unresolved") {
+                reason = "executable fragment outside recognized requirement/acceptance section";
+            }
+            else {
+                reason = "context-only fragment; excluded from canonical requirement coverage";
+            }
         }
         const priorId = seenText.get(text);
         if (priorId) {

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

@@ -5,9 +5,6 @@ import { ContractMutationError, applyTaskContract, logicalRevisionForState, obse
 import { TASK_CONTRACT_DRAFT_SCHEMA_VERSION } from "../contract/constants.js";
 import { getTaskContractPaths } from "../contract/paths.js";
 import { loadTaskConfig } from "../runtime.js";
-import { discoverFrontendProjectCapability, resolveFrontendSpecRootAliases, } from "../frontend-project-capability.js";
-import { classifyTaskDemand } from "../task-demand-routing.js";
-import { DEFAULT_FRONTEND_SPEC_ROOTS, extractDeclaredFrontendSpecRoots, } from "../../shared/openspec-spec.js";
 import { buildPrepareDraft } from "./build-draft.js";
 import { hasBlockingGaps, hasBlockingRisks, isFreshLegacyUnversioned, listPrepareGaps, appendLedgerGaps, } from "./completeness.js";
 import { detectProductArtifactMeta, isIntakeSoftGapCode, } from "./artifact-meta.js";
@@ -684,7 +681,7 @@ export async function prepareTaskSource(input) {
             facts = importedFacts ?? mergedSupplemental;
         }
     }
-    let built = buildPrepareDraft({
+    const built = buildPrepareDraft({
         taskId,
         existingTaskConfig,
         baseDraft,
@@ -692,91 +689,6 @@ export async function prepareTaskSource(input) {
         flags: input.flags,
         references,
     });
-    // Resolve an implicit frontend workflow before the first managed contract is
-    // projected. Explicit taskKind (including explicit standard) remains a hard
-    // contract; only a standard task with strong source/path/project evidence may
-    // be promoted here. This keeps task.json, contract.json, and the first DAG
-    // generation on the same workflow without a later hash-changing re-projection.
-    if (built.draft.taskKind === "standard" &&
-        !built.draft.taskKindExplicit) {
-        const routingRequirement = [
-            ...(sourceIntegrity?.documents ?? [])
-                .map((document) => document.content)
-                .filter((content) => Boolean(content?.trim())),
-            ...(input.intent.kind === "facts" && input.intent.text
-                ? [input.intent.text]
-                : []),
-            built.draft.requirement.objective,
-            ...built.draft.requirement.scope,
-            ...built.draft.requirement.acceptanceCriteria.map((item) => item.text),
-        ].join("\n");
-        let frontendProject;
-        try {
-            frontendProject = await discoverFrontendProjectCapability(repoRoot);
-        }
-        catch {
-            // Capability discovery is advisory for routing. A failed probe must not
-            // turn an otherwise valid standard contract preparation into an error.
-        }
-        const classification = classifyTaskDemand({
-            title: built.draft.title,
-            requirementMarkdown: routingRequirement,
-            allowedPaths: built.draft.constraints.allowedPaths,
-            frontendProject: frontendProject
-                ? {
-                    framework: frontendProject.framework,
-                    evidencePaths: frontendProject.evidencePaths,
-                }
-                : undefined,
-        });
-        if (classification.kind === "frontend") {
-            built = buildPrepareDraft({
-                taskId,
-                existingTaskConfig,
-                baseDraft,
-                facts,
-                flags: {
-                    ...input.flags,
-                    taskKind: "frontend-implementation",
-                },
-                references,
-            });
-        }
-    }
-    // A user can explicitly name a non-default normative directory in the task
-    // prompt (for example “规范目录 `design-system`”). Freeze that root into
-    // the managed task config before the DAG exists; normal DAG selection consumes
-    // the already-resolved task kind instead of mutating task.json later.
-    if (built.draft.taskKind === "frontend-implementation") {
-        // An imported PRD is materialized under sourceIntegrity rather than copied
-        // into intent.text. Use both inputs so a `--prd` invocation can declare a
-        // task-owned spec root just like supplemental free-form text can.
-        const sourceText = [
-            ...(sourceIntegrity?.documents ?? []).flatMap((document) => document.content ? [document.content] : []),
-            ...(input.intent.kind === "facts" && input.intent.text
-                ? [input.intent.text]
-                : []),
-        ].join("\n");
-        const inferredRoots = extractDeclaredFrontendSpecRoots(sourceText);
-        if (inferredRoots.length > 0) {
-            const current = existingTaskConfig?.frontendOpenspec;
-            const configuredRoots = current?.specRoots ?? [...DEFAULT_FRONTEND_SPEC_ROOTS];
-            // Include roots declared by this source before resolving physical-target
-            // aliases. A newly declared root may itself be a symlink (for example
-            // `project-specs -> .../ai_doc`), so resolving aliases against only the
-            // previous config cannot map `ai_doc/rules.md` back to project-specs.
-            const provisionalRoots = [
-                ...new Set([...configuredRoots, ...inferredRoots]),
-            ].sort();
-            const aliases = await resolveFrontendSpecRootAliases(repoRoot, provisionalRoots);
-            const aliasMap = new Map(aliases.map(({ alias, logicalRoot }) => [alias, logicalRoot]));
-            built.draft.frontendOpenspec = {
-                policy: current?.policy ?? "cited",
-                specRoots: [...new Set([...configuredRoots, ...inferredRoots.map((root) => aliasMap.get(root) ?? root)])].sort(),
-                requiredReadPaths: current?.requiredReadPaths ?? [],
-            };
-        }
-    }
     const { gaps, risks } = listPrepareGaps({
         draft: built.draft,
         state,

package/dist/task/source-prepare/semantic-intake.js

@@ -457,12 +457,8 @@ async function buildPrompt(input) {
         "Do NOT invent product behavior that is not supported by the documents.",
         "You MAY recommend an optional engineering block when the repo layout hint supports it:",
         "- allowedPaths / forbiddenPaths as minimal precise globs (e.g. source/packages/**/*.test.ts)",
-        "- verifyCommands as real package scripts (e.g. pnpm -C source test); when acceptance criteria name a specific test file to run (e.g. 定向 vitest（test/.../xxx.test.ts）), emit the corresponding targeted command as its own entry (e.g. npx vitest run test/.../xxx.test.ts) instead of only the full-suite script — targeted commands are what a frontend implementation plan maps to behavior verification targets",
-        "- taskKind when clearly implied, choosing from {standard / frontend-implementation / frontend-test / backend-test / knowledge-sync}:",
-        "  - frontend-implementation: the deliverable is a frontend component/page/UI feature (even when it ships with its own tests)",
-        "  - frontend-test: the deliverable is a browser/UI automation test suite for EXISTING functionality (no implementation work)",
-        "  - backend-test: pytest/backend test suite; standard: general repo implementation",
-        "  A task that IMPLEMENTS a component and adds tests for it is frontend-implementation, never frontend-test.",
+        "- verifyCommands as real package scripts (e.g. pnpm -C source test)",
+        "- taskKind when clearly implied (backend-test / frontend-test / standard / …)",
         "Prefer smallest correct write boundary; do not invent product paths that are not in the layout hint.",
         "Do NOT rewrite or suggest editing the original PRD files.",
         "Prefer explicit text; use provenance derived for rephrasing; use inferred only when clearly implied — if you must invent ACs, leave acceptanceCriteria empty so the system can stop for clarification.",

package/dist/task/source-references.js

@@ -22,18 +22,6 @@ function slugifyName(value) {
         .replace(/-{2,}/g, "-");
     return slug || "requirement";
 }
-/** Stable identity of a manifest document for change detection. Timestamps and
- * byte sizes are excluded: identical content re-imported later must not appear
- * "changed" (the canonical hash includes manifest bytes). */
-function documentIdentity(doc) {
-    return `${doc.role}|${doc.name}|${doc.materializedPath}|${doc.sha256}`;
-}
-function sameDocumentSet(left, right) {
-    if (left.length !== right.length)
-        return false;
-    const rightIds = new Set(right.map(documentIdentity));
-    return left.every((doc) => rightIds.has(documentIdentity(doc)));
-}
 function ensureUniqueReferenceName(existing, baseName, extension) {
     const used = new Set(existing);
     const candidate = `${baseName}${extension}`;
@@ -161,19 +149,10 @@ export async function importPrdDocument(input) {
     const documents = [...(existingManifest?.documents ?? [])].filter((entry) => entry.materializedPath !== materializedRelativePath);
     documents.push(document);
     documents.sort((left, right) => left.materializedPath.localeCompare(right.materializedPath));
-    // Keep `updatedAt` stable across re-imports of the same content: the
-    // canonical contract hash includes the manifest bytes, so a fresh
-    // timestamp on every identical re-import would make the observed
-    // canonical hash drift and fail task verification. Only bump it when the
-    // document set actually changed (a new/changed materialized file).
-    const documentSetChanged = !existingManifest ||
-        !sameDocumentSet(existingManifest.documents, documents);
     const manifest = {
         schemaVersion: 1,
         taskId: input.taskId,
-        updatedAt: documentSetChanged
-            ? importedAt
-            : (existingManifest?.updatedAt ?? importedAt),
+        updatedAt: importedAt,
         documents,
     };
     await writeSourceManifest(paths.sourceDir, manifest);

package/dist/worker/console/chat/chat-ui-policy.js

@@ -19,7 +19,9 @@ export function shouldPlayCompletionSound(input) {
 export function copyTextForMessage(input) {
     return input.code ?? input.text;
 }
-/** Empty sandbox means scripts, same-origin access, navigation and popups are denied. */
+/** Project HTML may run its own UI scripts, but remains an opaque origin with
+ * forms, popups and top-level navigation denied by omission. */
+export const HTML_PREVIEW_SANDBOX = "allow-scripts";
 export function previewSandbox() {
-    return { sandbox: "" };
+    return { sandbox: HTML_PREVIEW_SANDBOX };
 }