@tea-agent/loop-agent 0.9.0 → 0.10.0-alpha.0

package/dist/worker/runner/run-ready.js

@@ -2,6 +2,7 @@ import { createHash } from "node:crypto";
 import { readFile, writeFile, mkdir } from "node:fs/promises";
 import path from "node:path";
 import YAML from "yaml";
+import { controllerIdentitiesMatch, controllerIdentityExpectationFailure, resolveControllerIdentity, } from "../loop-agent/loop-agent-client.js";
 import { deriveFailureRoute, deriveFailureRouteFromError, } from "../pool/failure-routing.js";
 import { findRunByWorkerRunId, getTaskPoolRoot, readAllTaskPoolStates, recordTaskPoolRun, writeTaskPoolState, } from "../pool/run-store.js";
 import { runTaskSpec, } from "../run-task/run-task.js";
@@ -9,7 +10,26 @@ import { formatDuration, noopProgressReporter, } from "../progress-reporter.js";
 import { computeReadyQueue } from "../task-graph/ready-queue.js";
 import { taskGraphSpecSchema } from "../task-graph/task-graph-schema.js";
 import { taskSpecSchema } from "../task-spec/schema.js";
+import { preflightTargetRepo } from "../preflight.js";
 export async function runReadyTasks(options) {
+    let controllerIdentity = resolveControllerIdentity(options.client, options.controllerIdentity);
+    const identityFailure = controllerIdentityExpectationFailure(controllerIdentity, options.controllerExpectation);
+    if (identityFailure) {
+        throw new Error(`${identityFailure.code}: ${identityFailure.message}`);
+    }
+    if (!options.skipTaskPreflight && !options.runTask) {
+        const preflight = await preflightTargetRepo({
+            repoRoot: options.repoRoot,
+            client: options.client,
+            runCheckRepo: options.runCheckRepo,
+            ...(options.checkRepoCommand ? { checkRepoCommand: options.checkRepoCommand } : {}),
+            ...(options.controllerExpectation ? { expectation: options.controllerExpectation } : {}),
+        });
+        if (!preflight.ok) {
+            throw new Error(`target repo preflight failed: ${preflight.code}: ${preflight.message}`);
+        }
+        controllerIdentity = preflight.controllerIdentity ?? controllerIdentity;
+    }
     const now = options.now ?? new Date();
     const startedAt = now.toISOString();
     const batchRunId = options.batchRunId ?? buildBatchRunId(now);
@@ -43,12 +63,19 @@ export async function runReadyTasks(options) {
         const taskSpecPath = path.join(options.featureDir, "tasks", node?.task ?? `${taskId}.yaml`);
         const taskSpec = await loadTaskSpec(taskSpecPath);
         const retryOfWorkerRunId = states[taskId]?.retryOfWorkerRunId;
-        const workerRunId = retryOfWorkerRunId
+        let workerRunId = retryOfWorkerRunId
             ? buildRetryWorkerRunId(taskId, retryOfWorkerRunId, now)
             : options.workerRunIdForTask?.(taskId, taskSpec) ??
-                buildStableWorkerRunId(taskId, taskSpec, now);
+                buildStableWorkerRunId(taskId, taskSpec, now, controllerIdentity);
         if (workerRunId) {
-            const existing = await findRunByWorkerRunId(options.repoRoot, workerRunId);
+            let existing = await findRunByWorkerRunId(options.repoRoot, workerRunId);
+            if (existing && !controllerRunCanBeReused(existing, controllerIdentity)) {
+                workerRunId = buildControllerScopedWorkerRunId(workerRunId, controllerIdentity);
+                existing = await findRunByWorkerRunId(options.repoRoot, workerRunId);
+                if (existing && !controllerRunCanBeReused(existing, controllerIdentity)) {
+                    throw new Error(`workerRunId collision has incompatible controller identity: ${workerRunId}`);
+                }
+            }
             if (existing) {
                 progress.task(`task ${index}/${total} ${taskId}: reuse existing run ${workerRunId}`);
                 emit(progress, {
@@ -66,6 +93,7 @@ export async function runReadyTasks(options) {
                     status: "reused",
                     runRecordPath: existing.runRecordPath,
                 });
+                await options.onTaskFinalized?.({ status: "reused", taskSpec, workerRunId, ...(existing.runRecordPath ? { runRecordPath: existing.runRecordPath } : {}) });
                 continue;
             }
         }
@@ -80,7 +108,7 @@ export async function runReadyTasks(options) {
             workerRunId,
             status: "running",
         });
-        progress.step(`preflight → materialize → dag-run-task → validate → run-dag → report`);
+        progress.step(`materialize → dag-run-task → validate → run-dag → report`);
         let result;
         try {
             await writeTaskPoolState(options.repoRoot, {
@@ -106,12 +134,11 @@ export async function runReadyTasks(options) {
                 client: options.client,
                 now,
                 workerRunId,
-                preflight: {
-                    runCheckRepo: options.runCheckRepo,
-                    ...(options.checkRepoCommand ? { checkRepoCommand: options.checkRepoCommand } : {}),
-                },
+                preflight: false,
                 progress,
                 ...(options.piModel ? { piModel: options.piModel } : {}),
+                ...(controllerIdentity ? { controllerIdentity } : {}),
+                ...(options.controllerExpectation ? { controllerExpectation: options.controllerExpectation } : {}),
             });
         }
         catch (error) {
@@ -128,6 +155,7 @@ export async function runReadyTasks(options) {
                 status: "failed",
                 message,
             });
+            let recorded = false;
             try {
                 await recordTaskPoolRun({
                     repoRoot: options.repoRoot,
@@ -142,18 +170,23 @@ export async function runReadyTasks(options) {
                         error: message,
                         failure,
                         ...(retryOfWorkerRunId ? { retryOfWorkerRunId } : {}),
+                        ...(controllerIdentity ? { controllerIdentity } : {}),
                     },
                 });
+                recorded = true;
             }
             catch (recordError) {
+                await options.onTaskFinalized?.({ status: "run-error", taskSpec, workerRunId });
                 tasks.push({
                     taskId,
                     workerRunId,
                     status: "record-error",
                     error: errorMessage(recordError),
                 });
-                continue;
+                break;
             }
+            if (recorded)
+                await options.onTaskFinalized?.({ status: "run-error", taskSpec, workerRunId });
             tasks.push({
                 taskId,
                 workerRunId,
@@ -206,9 +239,12 @@ export async function runReadyTasks(options) {
             ...(failure ? { failure } : {}),
             ...(retryOfWorkerRunId ? { retryOfWorkerRunId } : {}),
             ...(result.failureArtifacts ? { failureArtifacts: result.failureArtifacts } : {}),
+            ...(controllerIdentity ? { controllerIdentity } : {}),
         };
+        let recorded = false;
         try {
             await recordTaskPoolRun({ repoRoot: options.repoRoot, run });
+            recorded = true;
             tasks.push({
                 taskId,
                 workerRunId: result.workerRunId,
@@ -217,6 +253,7 @@ export async function runReadyTasks(options) {
             });
         }
         catch (error) {
+            await options.onTaskFinalized?.({ status: result.status, taskSpec, workerRunId: result.workerRunId, runRecordPath: result.runRecordPath });
             tasks.push({
                 taskId,
                 workerRunId: result.workerRunId,
@@ -224,7 +261,10 @@ export async function runReadyTasks(options) {
                 error: errorMessage(error),
                 runRecordPath: result.runRecordPath,
             });
+            break;
         }
+        if (recorded)
+            await options.onTaskFinalized?.({ status: result.status, taskSpec, workerRunId: result.workerRunId, runRecordPath: result.runRecordPath });
     }
     const summary = summarize(tasks);
     const batchRunPath = path.join(getTaskPoolRoot(options.repoRoot), "artifacts", batchRunId, "batch-run.json");
@@ -237,6 +277,7 @@ export async function runReadyTasks(options) {
         summary,
         tasks,
         batchRunPath,
+        ...(controllerIdentity ? { controllerIdentity } : {}),
     };
     await mkdir(path.dirname(batchRunPath), { recursive: true });
     await writeFile(batchRunPath, `${JSON.stringify(output, null, 2)}\n`, "utf-8");
@@ -260,18 +301,55 @@ function emit(progress, input) {
 export function buildBatchRunId(now) {
     return `batch-${now.toISOString().replace(/[-:.]/g, "").slice(0, 15)}`;
 }
-export function buildStableWorkerRunId(taskId, taskSpec, now) {
+export function buildStableWorkerRunId(taskId, taskSpec, now, controllerIdentity) {
     const date = now.toISOString().slice(0, 10).replace(/-/g, "");
     const slug = taskId
         .toLowerCase()
         .replace(/[^a-z0-9]+/g, "-")
         .replace(/^-+|-+$/g, "");
     const hash = createHash("sha256")
-        .update(`${taskSpec.feature_id}:${taskSpec.id}:${taskSpec.title}`)
+        .update([
+        taskSpec.feature_id,
+        taskSpec.id,
+        taskSpec.title,
+        ...(controllerIdentity
+            ? [
+                controllerIdentity.packageName,
+                controllerIdentity.packageVersion,
+                controllerIdentity.binarySha256,
+                controllerIdentity.packageFingerprint.value,
+            ]
+            : []),
+    ].join(":"))
         .digest("hex")
         .slice(0, 10);
     return `wr-${date}-${slug}-${hash}`;
 }
+function buildControllerScopedWorkerRunId(baseWorkerRunId, controllerIdentity) {
+    const hash = createHash("sha256")
+        .update(controllerIdentity
+        ? JSON.stringify({
+            packageName: controllerIdentity.packageName,
+            binName: controllerIdentity.binName,
+            packageVersion: controllerIdentity.packageVersion,
+            binarySha256: controllerIdentity.binarySha256,
+            packageFingerprint: controllerIdentity.packageFingerprint.value,
+            requested: controllerIdentity.requested,
+            entry: controllerIdentity.entry,
+            realEntry: controllerIdentity.realEntry,
+            launch: controllerIdentity.launch,
+            packageRoot: controllerIdentity.packageRoot,
+        })
+        : "legacy-unpinned-controller")
+        .digest("hex")
+        .slice(0, 10);
+    return `${baseWorkerRunId}-controller-${hash}`;
+}
+function controllerRunCanBeReused(existing, controllerIdentity) {
+    if (!existing.controllerIdentity && !controllerIdentity)
+        return true;
+    return controllerIdentitiesMatch(existing.controllerIdentity, controllerIdentity);
+}
 /** A retry is a distinct execution attempt and must not collide with failed evidence. */
 export function buildRetryWorkerRunId(taskId, previousWorkerRunId, now) {
     const date = now.toISOString().slice(0, 10).replace(/-/g, "");

package/dist/worker/task-spec/schema.js

@@ -97,7 +97,6 @@ export const taskSpecSchema = z
         profile_policy: z.literal("mapped"),
         strict_models: z.boolean().optional().default(true),
         strict_governance: z.boolean().optional().default(true),
-        no_cursor: z.boolean().optional().default(false),
         max_concurrent: z.number().int().positive().optional().default(1),
     })
         .strict(),

package/dist/workflows/dag/convergence/controller.js

@@ -5,7 +5,7 @@ import { freshNodeRecord } from "../dynamic-runtime/shared.js";
 const CONVERGENCE_CHAIN_NODE_IDS = [
     "process-supervisor-pi",
     "process-gate-shell",
-    "repair-cursor",
+    "repair-pi",
     "hard-verify-shell",
 ];
 const CONVERGENCE_NON_RETRY_FAILURES = new Set([

package/dist/workflows/dag/executor-registry.js

@@ -1,4 +1,3 @@
-import { executeDagCursorNode } from "../../executors/dag-cursor-executor.js";
 import { executeDagPiNode } from "../../executors/dag-pi-executor.js";
 import { executeDagShellNode } from "../../executors/shell-executor.js";
 import { executeDagStaticNode } from "../../executors/dag-static-executor.js";
@@ -9,7 +8,6 @@ function requireModel(input) {
     return input;
 }
 export const DEFAULT_DAG_EXECUTOR_REGISTRY = {
-    cursor: (input, meta) => executeDagCursorNode(requireModel(input), meta),
     pi: (input, meta) => executeDagPiNode(requireModel(input), meta),
     shell: executeDagShellNode,
     static: executeDagStaticNode,

package/dist/workflows/dag/governance-profile.js

@@ -147,6 +147,9 @@ function hasPostImplementationShellVerification(spec, ranks) {
     }
     return false;
 }
+function hasBackendTestTopology(spec) {
+    return spec.tasks.some((task) => /execute-backend-pytest|generate-backend-functional-cases|review-backend-cases|generate-backend-pytest|test-retrospect/i.test(task.id));
+}
 function hasSupervisedTopology(spec) {
     const hasSupervisor = spec.tasks.some((task) => task.role === "supervisor");
     const hasRepairWriter = spec.tasks.some((task) => isRepairNode(task) && isExclusiveWriter(task, spec));
@@ -302,6 +305,10 @@ function buildProfileReasons(profile, process, delivery, codeChange, supervisedT
         reasons.push("supervised topology detected (supervisor + repair + write-set/process/review gates)");
         return reasons;
     }
+    if (profile === "backend-test") {
+        reasons.push("backend-test topology detected (analyze → functional cases → review → pytest → execute → retrospect)");
+        return reasons;
+    }
     if (supervisedTopology) {
         reasons.push("partial supervised markers present but topology incomplete; profile downgraded");
     }
@@ -337,6 +344,9 @@ function buildProfileReasons(profile, process, delivery, codeChange, supervisedT
     return reasons;
 }
 function inferProfileName(spec, process, delivery, codeChange, ranks) {
+    if (hasBackendTestTopology(spec)) {
+        return "backend-test";
+    }
     if (hasSupervisedTopology(spec)) {
         return "supervised";
     }