@tea-agent/loop-agent 0.34.2 → 0.34.4

package/dist/worker/console/prd-intake-bridge.js

@@ -8,6 +8,7 @@ import path from "node:path";
 import { getTaskPaths } from "../../task/runtime.js";
 import { SEMANTIC_INTAKE_ARTIFACT_REL, } from "../../task/source-prepare/semantic-intake.js";
 import { extractRequirementFactsFromMarkdown } from "../../task/source-prepare/parse-intent.js";
+import { filterPlaceholderPaths, isNonPlaceholderPath, } from "../../task/source-prepare/placeholder-paths.js";
 import { emptyDraft } from "./interview/grill-me.js";
 import { normalizeConsoleWorkflowKind } from "./workflow-kinds.js";
 function asStringList(value) {
@@ -105,13 +106,13 @@ export function parseEngineeringBoundaryFromParams(p) {
         if (Array.isArray(value)) {
             return value
                 .map((item) => (typeof item === "string" ? item.trim() : ""))
-                .filter(Boolean);
+                .filter(isNonPlaceholderPath);
         }
         if (typeof value === "string") {
             return value
                 .split(/[\n,;，；]+/)
                 .map((s) => s.trim())
-                .filter(Boolean);
+                .filter(isNonPlaceholderPath);
         }
         return [];
     };
@@ -176,10 +177,10 @@ export function parseEngineeringBoundaryFromParams(p) {
     };
 }
 export function appendEngineeringCliArgs(args, boundary) {
-    for (const pathGlob of boundary.allowedPaths ?? []) {
+    for (const pathGlob of filterPlaceholderPaths(boundary.allowedPaths)) {
         args.push("--allowed-path", pathGlob);
     }
-    for (const pathGlob of boundary.forbiddenPaths ?? []) {
+    for (const pathGlob of filterPlaceholderPaths(boundary.forbiddenPaths)) {
         args.push("--forbidden-path", pathGlob);
     }
     for (const cmd of boundary.verifyCommands ?? []) {
@@ -187,8 +188,10 @@ export function appendEngineeringCliArgs(args, boundary) {
     }
 }
 export function applyEngineeringBoundaryToDraft(draft, boundary) {
-    if (!(boundary.allowedPaths?.length) &&
-        !(boundary.forbiddenPaths?.length) &&
+    const allowedPaths = filterPlaceholderPaths(boundary.allowedPaths);
+    const forbiddenPaths = filterPlaceholderPaths(boundary.forbiddenPaths);
+    if (allowedPaths.length === 0 &&
+        forbiddenPaths.length === 0 &&
         !(boundary.verifyCommands?.length)) {
         return draft;
     }
@@ -196,11 +199,11 @@ export function applyEngineeringBoundaryToDraft(draft, boundary) {
         ...draft,
         constraints: {
             invariants: draft.constraints?.invariants ?? [],
-            allowedPaths: boundary.allowedPaths?.length
-                ? boundary.allowedPaths
+            allowedPaths: allowedPaths.length > 0
+                ? allowedPaths
                 : (draft.constraints?.allowedPaths ?? []),
-            forbiddenPaths: boundary.forbiddenPaths?.length
-                ? boundary.forbiddenPaths
+            forbiddenPaths: forbiddenPaths.length > 0
+                ? forbiddenPaths
                 : (draft.constraints?.forbiddenPaths ?? []),
         },
         verification: {
@@ -253,15 +256,53 @@ export async function buildDraftFromTaskIntake(input) {
         const title = artifact?.draft?.title?.trim() ||
             artifact?.semantic?.title?.trim() ||
             input.title;
+        // Preserve semantic engineering boundary (paths/verify/taskKind) so the
+        // Console draft does not fall back to interview defaults and overwrite
+        // TX1 precise values on contractApply --from-draft.
+        const semDraftConstraints = artifact?.draft?.constraints;
+        const semDraftVerification = artifact?.draft?.verification;
+        const semTaskKind = artifact?.draft?.taskKind?.trim();
+        const mergedAllowed = filterPlaceholderPaths(semDraftConstraints?.allowedPaths);
+        const mergedForbidden = filterPlaceholderPaths(semDraftConstraints?.forbiddenPaths);
+        const mergedVerify = Array.isArray(semDraftVerification?.commands)
+            ? semDraftVerification.commands
+                .filter((cmd) => typeof cmd?.command === "string" && cmd.command.trim().length > 0)
+                .map((cmd) => ({
+                label: typeof cmd.label === "string" && cmd.label.trim()
+                    ? cmd.label.trim()
+                    : "verify",
+                command: cmd.command.trim(),
+                ...(typeof cmd.timeoutMs === "number"
+                    ? { timeoutMs: cmd.timeoutMs }
+                    : {}),
+            }))
+            : [];
         draft = {
             ...draft,
             title,
+            ...(semTaskKind
+                ? { taskKind: normalizeConsoleWorkflowKind(semTaskKind, taskKind) }
+                : {}),
             requirement: {
                 objective: semanticReq.objective.trim(),
                 scope: asStringList(semanticReq.scope),
                 nonGoals: asStringList(semanticReq.nonGoals),
                 acceptanceCriteria: asAcceptance(semanticReq.acceptanceCriteria),
             },
+            ...(mergedAllowed.length > 0 ||
+                mergedForbidden.length > 0 ||
+                Array.isArray(semDraftConstraints?.invariants)
+                ? {
+                    constraints: {
+                        invariants: asStringList(semDraftConstraints?.invariants),
+                        allowedPaths: mergedAllowed,
+                        forbiddenPaths: mergedForbidden,
+                    },
+                }
+                : {}),
+            ...(mergedVerify.length > 0
+                ? { verification: { commands: mergedVerify } }
+                : {}),
             openQuestions: asStringList(artifact?.draft?.openQuestions ?? artifact?.semantic?.openQuestions) || undefined,
             assumptions: asStringList(artifact?.draft?.assumptions ?? artifact?.semantic?.assumptions) || undefined,
             references: artifact?.draft?.references,

package/dist/worker/console/server.js

@@ -14,6 +14,7 @@ import { InterviewSessionStore } from "./interview/session.js";
 import { isLoopbackHost } from "./loopback.js";
 import { ChatSessionStore } from "./chat/session-store.js";
 import { createChatEventStore, createChatOperationLinker, } from "./chat/chat-event-store.js";
+import { createChatTurnExecutionRegistry } from "./chat/turn-execution-registry.js";
 import { ConsolePiRuntime } from "./chat/pi-runtime.js";
 import { handleChatRequest } from "./chat/routes.js";
 import { createOperationEventStore, } from "./operation-sse.js";
@@ -147,6 +148,7 @@ export async function createConsoleServer(options) {
     // than silently consume a different process' event namespace.
     const chatEventEpoch = randomBytes(12).toString("hex");
     const chatEvents = createChatEventStore({ appData, epoch: chatEventEpoch });
+    const chatTurnExecutions = createChatTurnExecutionRegistry();
     const chatOperationLinker = createChatOperationLinker({
         chatEvents,
         operations,
@@ -155,6 +157,7 @@ export async function createConsoleServer(options) {
     const chatRuntime = new ConsolePiRuntime({
         cwd: repoRoot,
         sessionDir: appData.chatSessions,
+        resourcePreferencesPath: appData.resourcePreferences,
         actionContext,
     });
     // chatRouteDeps is a plain object literal — it cannot throw. The previous
@@ -165,6 +168,7 @@ export async function createConsoleServer(options) {
         runtime: chatRuntime,
         store: chatStore,
         events: chatEvents,
+        turnExecutions: chatTurnExecutions,
         operationLinker: chatOperationLinker,
         appData,
         bootToken,