@zq-silk/yui 0.13.7 → 0.13.9

package/dist/executor/fileRoleLaunchPlanner.js

@@ -10,11 +10,6 @@ import { activeRoleAgentBinding } from "../role/role.js";
 import { writeTextFileAtomically } from "../storage/durableFile.js";
 import { compileRoleSessionContext, roleSessionKind } from "../context/roleSessionContext.js";
 import { materializeSessionBootstrap } from "../context/sessionBootstrapManifest.js";
-import { serializeRunBootstrapEnvelope, serializeRunHostRecoveryEnvelope } from "../context/runContextContract.js";
-import { serializeProviderRetryEnvelope } from "../run/providerRetry.js";
-import { serializeWorkflowOutcomeRequestEnvelope } from "../run/runControlRequest.js";
-import { prefixYuiTitleInput } from "../run/runIdentity.js";
-import { agentRunDeliveryReceiptId } from "../run/agentRun.js";
 import { resolveAgentAdapter } from "./agentAdapter.js";
 import { resolveTaskRoleSessionTitle } from "../runtime/sessionTitle.js";
 import { nativeSessionIdForLaunch } from "../runtime/preallocatedNativeSession.js";
@@ -302,7 +297,6 @@ export class FileRoleLaunchPlanner {
         if (runtimeIsolation !== undefined && (owner.scope !== "task"
             || runtimeIsolation.taskId !== owner.taskId
             || runtimeIsolation.workspace.root !== effectiveWorkspace
-            || runtimeIsolation.generation.runId !== input.runId
             || runtimeIsolation.generation.launchId !== input.launchId)) {
             throw new Error("Role launch does not match its Task runtime isolation descriptor.");
         }
@@ -478,41 +472,31 @@ export class FileRoleLaunchPlanner {
         if (owner.scope === "task" && (input.mode === "new" || input.mode === "resume")) {
             jobCallerKey = randomBytes(32).toString("hex");
         }
-        // Every managed launch for an unpushed Run owns that Run's first Provider
-        // write, whether it creates or resumes the native Conversation. Keeping
-        // resume on the same launch handshake prevents a durable AgentRun from
-        // existing without a corresponding Provider Turn.
+        // Session lifecycle and Turn submission are separate atomic operations.
+        // Planning only starts or restores the exact native Session; delivery
+        // submits the Run input after that Session fact is durable.
         if (managedControl && (managedRun === null || managedRun.status !== "active")) {
             throw new Error(`Managed Run is no longer active: ${input.runId}.`);
         }
-        const carriesInitialTurn = managedControl && (managedRun.pushedAt === undefined
-            || managedRun.providerRetry?.state === "dispatching"
-            || managedRun.controlRequest?.state === "dispatching");
         const providerAuthority = managedControl
             ? this.#providerAuthorityForLaunch(owner.taskId, role.name, input.launchId, input.mode)
             : undefined;
-        const providerOwnedTurn = managedControl && !carriesInitialTurn
+        const providerOwnedTurn = managedControl && input.mode === "resume"
             ? this.#providerOwnedTurnForLaunch(owner.taskId, role.name, input.runId)
             : undefined;
         const providerNativeSessionId = binding.adapterId === "claude"
             ? preallocatedNativeSessionId
             : resumeNativeSessionId;
-        const initialTurn = carriesInitialTurn
-            ? {
-                attemptId: agentRunDeliveryReceiptId(managedRun),
-                boundedText: managedRunLaunchEnvelope(managedRun, input.mode, sessionContext.sessionManifestPath, sessionTitle)
-            }
-            : undefined;
         const providerControl = !managedControl
             ? undefined
-            : initialTurn === undefined
+            : input.mode === "resume"
                 ? {
                     schemaVersion: 1,
                     adapterId: binding.adapterId,
                     transport: managedCompiled.transport,
-                    kind: "ensure",
+                    kind: "restore",
                     mode: "resume",
-                    nativeSessionId: requireText(providerNativeSessionId, "Managed Provider ensure native session id"),
+                    nativeSessionId: requireText(providerNativeSessionId, "Managed Provider restore native session id"),
                     ...(sessionTitle === undefined ? {} : { sessionTitle }),
                     ...(managedCompiled.codexThread === undefined
                         ? {}
@@ -520,44 +504,24 @@ export class FileRoleLaunchPlanner {
                     ...(providerOwnedTurn === undefined ? {} : { ownedTurn: providerOwnedTurn }),
                     authority: providerAuthority
                 }
-                : resumeNativeSessionId === undefined
-                    ? {
-                        schemaVersion: 1,
-                        adapterId: binding.adapterId,
-                        transport: managedCompiled.transport,
-                        kind: "new",
-                        mode: "new",
-                        ...(providerNativeSessionId === undefined
-                            ? {}
-                            : { nativeSessionId: providerNativeSessionId }),
-                        ...(sessionTitle === undefined ? {} : { sessionTitle }),
-                        ...(managedCompiled.codexThread === undefined
-                            ? {}
-                            : { codexThread: managedCompiled.codexThread }),
-                        authority: providerAuthority,
-                        initialTurn
-                    }
-                    : {
-                        schemaVersion: 1,
-                        adapterId: binding.adapterId,
-                        transport: managedCompiled.transport,
-                        kind: "resume",
-                        mode: "resume",
-                        nativeSessionId: requireText(providerNativeSessionId, "Managed Provider resume native session id"),
-                        ...(sessionTitle === undefined ? {} : { sessionTitle }),
-                        ...(managedCompiled.codexThread === undefined
-                            ? {}
-                            : { codexThread: managedCompiled.codexThread }),
-                        authority: providerAuthority,
-                        initialTurn
-                    };
+                : {
+                    schemaVersion: 1,
+                    adapterId: binding.adapterId,
+                    transport: managedCompiled.transport,
+                    kind: "start",
+                    mode: "new",
+                    ...(providerNativeSessionId === undefined
+                        ? {}
+                        : { nativeSessionId: providerNativeSessionId }),
+                    ...(sessionTitle === undefined ? {} : { sessionTitle }),
+                    ...(managedCompiled.codexThread === undefined
+                        ? {}
+                        : { codexThread: managedCompiled.codexThread }),
+                    authority: providerAuthority
+                };
         const launch = {
             command,
             args,
-            ...(managedRun?.providerRetry !== undefined
-                && managedRun.providerRetry.state !== "dispatching"
-                ? { deferProviderStart: true }
-                : {}),
             ...(providerControl === undefined ? {} : { providerControl }),
             env: {
                 ...launchEnvironment,
@@ -608,15 +572,11 @@ export class FileRoleLaunchPlanner {
             role: {
                 name: role.name,
                 workspace: effectiveWorkspace,
-                ...(agentWorkspace === effectiveWorkspace ? {} : { cwd: agentWorkspace }),
-                ...(owner.scope === "task" ? { status: role.status } : {})
+                ...(agentWorkspace === effectiveWorkspace ? {} : { cwd: agentWorkspace })
             },
             launch: ordinaryConversationLaunch,
             session,
-            ...(sessionTitle === undefined ? {} : { sessionTitle }),
-            ...(carriesInitialTurn && input.runId !== undefined
-                ? { initialTurnRunId: input.runId }
-                : {})
+            ...(sessionTitle === undefined ? {} : { sessionTitle })
         };
     }
     #providerAuthorityForLaunch(taskId, roleName, launchId, mode) {
@@ -659,7 +619,7 @@ export class FileRoleLaunchPlanner {
     }
     #providerOwnedTurnForLaunch(taskId, roleName, runId) {
         const binding = this.store.getTaskRoleSessionSet(taskId, roleName)?.providerBinding;
-        if (binding === null || binding === undefined || binding.runId !== runId)
+        if (binding === null || binding === undefined || binding.turn?.runId !== runId)
             return undefined;
         const turn = binding.turn;
         if (turn === null || !["accepted", "running"].includes(turn.status))
@@ -706,33 +666,6 @@ export class FileRoleLaunchPlanner {
         return selectEnvironment(source, names);
     }
 }
-function managedRunLaunchEnvelope(run, mode, sessionManifestPath, title) {
-    const body = run.providerRetry?.state === "dispatching"
-        ? serializeProviderRetryEnvelope({
-            taskId: run.taskId,
-            runId: run.id,
-            roleName: run.roleName,
-            retry: run.providerRetry
-        })
-        : run.controlRequest?.state === "dispatching"
-            ? serializeWorkflowOutcomeRequestEnvelope({
-                taskId: run.taskId,
-                runId: run.id,
-                roleName: run.roleName,
-                request: run.controlRequest
-            })
-            : mode === "resume" && run.pushedAt !== undefined
-                ? serializeRunHostRecoveryEnvelope(run.bootstrapEnvelope)
-                : serializeRunBootstrapEnvelope(run.bootstrapEnvelope);
-    const guided = [
-        `Yui Session Manifest: ${sessionManifestPath}`,
-        "Read the manifest and its matching Role Skill before using the yui CLI for this Task.",
-        body
-    ].join("\n");
-    return title === undefined
-        ? guided
-        : prefixYuiTitleInput(guided, title);
-}
 export function nativeAgentWorkspace(workspace) {
     return workspace.entries.length === 1
         ? workspace.entries[0].path
@@ -962,7 +895,7 @@ function readySession(agentId, adapterId, nativeSessionId, effective) {
         agentId,
         adapterId,
         nativeSessionId: requireText(nativeSessionId, "Native session id"),
-        status: "ready",
+        status: "active",
         effective
     };
 }

package/dist/executor/turnCompletion.js

@@ -22,7 +22,7 @@ export function createPendingTurnCompletion(input) {
         roleName: requireSafeIdentity(input.roleName, "Role name"),
         agentId: requireSafeIdentity(input.agentId, "Agent id"),
         nativeSessionId: requireText(input.nativeSessionId, "Native session id"),
-        turnId: requireSafeIdentity(input.turnId, "Turn id"),
+        turnId: requireText(input.turnId, "Turn id"),
         runId: requireSafeIdentity(input.runId, "Run id"),
         summary: requireText(input.summary, "Turn summary"),
         observedAt,
@@ -44,7 +44,7 @@ export function validatePendingTurnCompletion(value) {
         roleName: requireSafeIdentityValue(input.roleName, "Role name"),
         agentId: requireSafeIdentityValue(input.agentId, "Agent id"),
         nativeSessionId: requireTextValue(input.nativeSessionId, "Native session id"),
-        turnId: requireSafeIdentityValue(input.turnId, "Turn id"),
+        turnId: requireTextValue(input.turnId, "Turn id"),
         runId: requireSafeIdentityValue(input.runId, "Run id"),
         summary: requireTextValue(input.summary, "Turn summary"),
         observedAt,
@@ -58,7 +58,7 @@ export function validateRecentTurnIds(value, limit = DEFAULT_RECENT_TURN_ID_LIMI
     if (value.length > normalizedLimit) {
         throw new Error(`Recent Turn ids must not contain more than ${normalizedLimit} entries.`);
     }
-    const result = value.map((turnId) => requireSafeIdentityValue(turnId, "Recent Turn id"));
+    const result = value.map((turnId) => requireTextValue(turnId, "Recent Turn id"));
     if (new Set(result).size !== result.length) {
         throw new Error("Recent Turn ids must not contain duplicates.");
     }
@@ -67,7 +67,7 @@ export function validateRecentTurnIds(value, limit = DEFAULT_RECENT_TURN_ID_LIMI
 export function rememberRecentTurnId(recentTurnIds, turnId, limit = DEFAULT_RECENT_TURN_ID_LIMIT) {
     const normalizedLimit = requireLimit(limit);
     const existing = validateRecentTurnIds(recentTurnIds, normalizedLimit);
-    const normalizedTurnId = requireSafeIdentity(turnId, "Recent Turn id");
+    const normalizedTurnId = requireText(turnId, "Recent Turn id");
     return [
         ...existing.filter((candidate) => candidate !== normalizedTurnId),
         normalizedTurnId
@@ -75,7 +75,7 @@ export function rememberRecentTurnId(recentTurnIds, turnId, limit = DEFAULT_RECE
 }
 export function hasRecentTurnId(recentTurnIds, turnId) {
     const existing = validateRecentTurnIds(recentTurnIds);
-    return existing.includes(requireSafeIdentity(turnId, "Recent Turn id"));
+    return existing.includes(requireText(turnId, "Recent Turn id"));
 }
 function requireOrderedTimestamps(observedAt, dueAt) {
     if (Date.parse(dueAt) < Date.parse(observedAt)) {

package/dist/lifecycle/exactRunTerminalization.js

@@ -2,7 +2,6 @@ import { isDeepStrictEqual } from "node:util";
 import { completeProcessing } from "../coordination/workMailbox.js";
 import { settleExactWorkExecution } from "../coordination/workMailboxQueue.js";
 import { terminalizeTaskRoleRunSession } from "../executor/agentExecutor.js";
-import { updateRoleStatus } from "../role/role.js";
 import { finishReviewRound, updateReviewExecutionGroup } from "../review/reviewRound.js";
 import { reconcileReviewFindingsAfterReview } from "../review/reviewFindingLedger.js";
 import { agentRunDeliveryReceiptId, failAgentRun, withYieldReceipt, yieldAgentRun } from "../run/agentRun.js";
@@ -265,12 +264,12 @@ export function retireExactActiveAgentRun(store, input, now) {
     const session = sessions?.sessions[input.agentId];
     const providerBinding = sessions?.providerBinding;
     const providerTurn = providerBinding?.turn;
-    const providerSettled = providerBinding?.runId === current.id
+    const providerSettled = providerTurn?.runId === current.id
         && (providerTurn?.status === "completed"
             || providerTurn?.status === "failed"
             || providerTurn?.status === "cancelled"
             || providerTurn?.status === "rejected");
-    if (session?.status !== "stopped" && session?.status !== "broken" && !providerSettled) {
+    if (session?.status === "active" && !providerSettled) {
         return {
             disposition: "blocked",
             run: current,
@@ -450,7 +449,6 @@ export function terminalizeExactTaskRun(store, input, now) {
     else {
         store.clearActiveAgentRun(input.taskId, input.roleName);
     }
-    store.saveRole(input.taskId, updateRoleStatus(role, "idle", now));
     if (sessions !== null) {
         store.saveTaskRoleSessionSet(terminalizeTaskRoleRunSession(sessions, {
             agentId: input.agentId,

package/dist/observability/executionAudit.js

@@ -86,26 +86,6 @@ function roleBucket(roleName) {
     }
     return "other";
 }
-/** Maps a provider-retry lifecycle event to its last recorded decision. */
-function providerRetryDecision(event) {
-    if (event.type === "runtime.provider-retry-waiting")
-        return "waiting";
-    if (event.type === "runtime.provider-retry-resume")
-        return "resume";
-    if (event.type === "runtime.provider-retry-session-dead")
-        return "session-dead";
-    if (event.type === "runtime.provider-retry-budget-exhausted")
-        return "budget-exhausted";
-    if (event.payload.note === "native-turn-completion-durable")
-        return "suppressed:completion-durable";
-    if (event.payload.note === "retry-budget-exhausted")
-        return "budget-exhausted";
-    if (event.payload.wouldRetry === "true")
-        return "retrying";
-    if (event.payload.shadow === "true")
-        return "shadow";
-    return "not-retried";
-}
 function durationMs(run) {
     if (run.endedAt === undefined)
         return 0;
@@ -205,7 +185,7 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
             integrations: section,
             publications: section,
             events: section,
-            providerRetries: section,
+            agentErrors: section,
             workItems: section,
             orchestration: section,
             storage: section,
@@ -378,13 +358,13 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
                     const history = Array.isArray(set.history) ? set.history : [];
                     for (const session of [...history, ...Object.values(set.sessions)]) {
                         generations += 1;
-                        if (session.status === "broken")
+                        if (session.status === "ended" && session.endReason === "failed")
                             broken += 1;
-                        else if (session.status === "stopped")
+                        else if (session.status === "ended")
                             stopped += 1;
                         else
                             other += 1;
-                        if (session.status === "broken" || session.status === "stopped") {
+                        if (session.status === "ended") {
                             classifyTerminalSessionRunRelation(runs, set.owner.roleName, session, terminalByRunRelation);
                         }
                     }
@@ -608,51 +588,38 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
             return failed(error);
         }
     })();
-    const providerRetries = (() => {
+    const agentErrors = (() => {
         try {
-            const entries = new Map();
+            const entries = [];
+            const categories = new Map();
             for (const taskId of taskIds) {
                 for (const event of store.listEvents(taskId)) {
-                    if (!inWindow(event.createdAt, options))
-                        continue;
-                    if (!event.type.startsWith("runtime.provider-retry-"))
-                        continue;
-                    const runId = event.payload.runId;
-                    if (typeof runId !== "string")
+                    if (!inWindow(event.createdAt, options) || event.type !== "runtime.agent-error") {
                         continue;
-                    const key = `${taskId}/${runId}`;
-                    const previous = entries.get(key);
-                    const attempt = Number(event.payload.attempt ?? "0");
-                    const decision = providerRetryDecision(event);
-                    const terminal = decision === "session-dead" || decision === "budget-exhausted";
-                    const next = {
+                    }
+                    const category = event.payload.category ?? "unknown";
+                    categories.set(category, (categories.get(category) ?? 0) + 1);
+                    entries.push({
                         taskId,
-                        runId,
-                        roleName: typeof event.payload.roleName === "string"
-                            ? event.payload.roleName
-                            : previous?.roleName ?? "",
-                        attempts: Number.isFinite(attempt) ? Math.max(previous?.attempts ?? 0, attempt) : (previous?.attempts ?? 0),
-                        errorClass: typeof event.payload.errorClass === "string"
-                            ? event.payload.errorClass
-                            : previous?.errorClass ?? "",
-                        firstFailureAt: previous?.firstFailureAt ?? event.createdAt,
-                        lastFailureAt: event.createdAt,
-                        ...(typeof event.payload.nextAttemptAt === "string"
-                            ? { lastBackoffAt: event.payload.nextAttemptAt }
-                            : previous?.lastBackoffAt !== undefined
-                                ? { lastBackoffAt: previous.lastBackoffAt }
-                                : {}),
-                        decision,
-                        terminal: (previous?.terminal ?? false) || terminal
-                    };
-                    entries.set(key, next);
+                        eventId: event.id,
+                        runId: event.payload.runId ?? "",
+                        roleName: event.payload.roleName ?? "",
+                        source: event.payload.source ?? "unknown",
+                        phase: event.payload.phase ?? "unknown",
+                        category,
+                        code: event.payload.code ?? "unknown",
+                        message: event.payload.message ?? "",
+                        raw: event.payload.raw ?? "",
+                        inputDisposition: event.payload.inputDisposition ?? "unknown",
+                        sessionDisposition: event.payload.sessionDisposition ?? "unknown",
+                        createdAt: event.createdAt
+                    });
                 }
             }
-            const list = [...entries.values()];
             return ok({
-                total: list.length,
-                terminal: list.filter((entry) => entry.terminal).length,
-                entries: list.map(({ terminal: _terminal, ...entry }) => entry)
+                total: entries.length,
+                byCategory: Object.fromEntries(categories),
+                entries
             });
         }
         catch (error) {
@@ -748,16 +715,10 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
         try {
             const protocolVersions = new Map();
             const manifestDigests = new Set();
-            const retryStates = new Map();
+            const agentErrorCategories = new Map();
             const exitClassifications = new Map();
             const usageSemantics = new Map();
-            let activeRetryEpisodes = 0;
-            let activeConsecutiveFailures = 0;
-            let activeDispatchedRetries = 0;
-            let retryClassifiedEvents = 0;
-            let retryDispatchedEvents = 0;
-            let retryRecoveredEvents = 0;
-            let retryExhaustedEvents = 0;
+            let agentErrors = 0;
             let contextCapacityFailures = 0;
             let processExitObservations = 0;
             let compactionEvents = 0;
@@ -770,26 +731,17 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
                     if (run.effective.sessionManifestCompatibilityDigest !== undefined) {
                         manifestDigests.add(run.effective.sessionManifestCompatibilityDigest);
                     }
-                    if (run.providerRetry !== undefined) {
-                        activeRetryEpisodes += 1;
-                        retryStates.set(run.providerRetry.state, (retryStates.get(run.providerRetry.state) ?? 0) + 1);
-                        activeConsecutiveFailures += run.providerRetry.consecutiveFailures;
-                        activeDispatchedRetries += run.providerRetry.dispatchedRetries;
-                    }
                 }
                 for (const event of store.listEvents(taskId)) {
                     if (!inWindow(event.createdAt, options))
                         continue;
-                    if (event.type === "runtime.provider-retry-classified")
-                        retryClassifiedEvents += 1;
-                    else if (event.type === "runtime.provider-retry-dispatched")
-                        retryDispatchedEvents += 1;
-                    else if (event.type === "runtime.provider-retry-recovered")
-                        retryRecoveredEvents += 1;
-                    else if (event.type === "runtime.provider-retry-exhausted")
-                        retryExhaustedEvents += 1;
-                    else if (event.type === "runtime.context-capacity-failure")
-                        contextCapacityFailures += 1;
+                    if (event.type === "runtime.agent-error") {
+                        agentErrors += 1;
+                        const category = event.payload.category ?? "unknown";
+                        agentErrorCategories.set(category, (agentErrorCategories.get(category) ?? 0) + 1);
+                        if (category === "context")
+                            contextCapacityFailures += 1;
+                    }
                     else if (event.type === "runtime.process-exit-observed") {
                         processExitObservations += 1;
                         const classification = event.payload.classification ?? "unknown";
@@ -808,14 +760,8 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
             return ok({
                 contextProtocolVersions: Object.fromEntries(protocolVersions),
                 manifestCompatibilityDigests: manifestDigests.size,
-                activeRetryEpisodes,
-                activeRetryStates: Object.fromEntries(retryStates),
-                activeConsecutiveFailures,
-                activeDispatchedRetries,
-                retryClassifiedEvents,
-                retryDispatchedEvents,
-                retryRecoveredEvents,
-                retryExhaustedEvents,
+                agentErrors,
+                agentErrorCategories: Object.fromEntries(agentErrorCategories),
                 contextCapacityFailures,
                 processExitObservations,
                 processExitClassifications: Object.fromEntries(exitClassifications),
@@ -870,7 +816,7 @@ export function runExecutionAudit(home, options = {}, ports = createProductionEx
         integrations,
         publications,
         events,
-        providerRetries,
+        agentErrors,
         workItems,
         orchestration,
         storage,

package/dist/operator/operatorSessionHistory.js

@@ -25,7 +25,7 @@ export function projectOperatorStatus(sessions, activeAgentId, activeAdapterId)
     }
     return Object.freeze({
         writer: {
-            state: matchingActive.status === "stopped" || matchingActive.status === "broken" ? "inactive" : "active",
+            state: matchingActive.status === "ended" ? "inactive" : "active",
             agentId: matchingActive.agentId,
             adapterId: matchingActive.adapterId,
             sessionRef: activeRef,
@@ -42,7 +42,7 @@ export function listOperatorSessions(sessions) {
     if (sessions === null)
         return [];
     validateRoleSessionSet(sessions);
-    const current = Object.values(sessions.sessions).map((session) => (listItem(session, session.status === "stopped" || session.status === "broken"
+    const current = Object.values(sessions.sessions).map((session) => (listItem(session, session.status === "ended"
         ? "current"
         : "running")));
     const history = Object.values(sessions.history ?? {}).map((session) => (listItem(session, "history")));
@@ -92,7 +92,8 @@ export function prepareOperatorResumeSession(sessions, ref, now) {
     delete history[normalizedRef];
     nextSessions[selected.agentId] = {
         ...selected,
-        status: "stopped",
+        status: "ended",
+        endReason: "stopped",
         updatedAt: requireDate(now)
     };
     return validateRoleSessionSet({
@@ -104,12 +105,13 @@ export function prepareOperatorResumeSession(sessions, ref, now) {
     });
 }
 function archiveCurrent(history, session) {
-    if (session.status !== "stopped" && session.status !== "broken") {
+    if (session.status !== "ended") {
         throw new Error(`Cannot replace Operator session while its native process is ${session.status}.`);
     }
     const archived = {
         ...session,
-        status: "stopped"
+        status: "ended",
+        endReason: session.endReason ?? "stopped"
     };
     return {
         ...history,

package/dist/output/rolePresentation.js

@@ -23,7 +23,6 @@ export function renderRoleDetails(title, role, input) {
     const overview = [
         `  Kind             ${input.kind}`,
         `  Active Agent     ${role.activeAgentId}`,
-        ...("status" in role ? [`  Status           ${role.status}`] : []),
         `  Workspace        ${role.workspace}`,
         `  Desired launch   r${role.launchRevision}; Profile intent=${role.defaultAccess}`,
         `  Effective launch ${effective === undefined

package/dist/repository/taskWorkspacePreparer.js

@@ -2764,7 +2764,6 @@ function retireWorkspaceBoundSession(store, taskId, roleName, now) {
 }
 function canCorrectActiveWorkItemRoleWorkspaceHint(store, taskId, role, item, workspace) {
     if (role.taskId !== taskId
-        || role.status !== "running"
         || item.taskId !== taskId
         || item.assignee !== role.name
         || ["completed", "failed", "retired"].includes(item.status)

package/dist/role/role.js

@@ -19,19 +19,22 @@ export function createRole(taskId, name, bindings, activeAgentId, workspace, now
     const owner = createRoleOwner(name, bindings, activeAgentId, workspace, now, profile, defaultAccess);
     return validateTaskRole({
         ...owner,
-        taskId: requireSafeIdentity(taskId, "Task id"),
-        status: "idle"
+        schemaVersion: 4,
+        taskId: requireSafeIdentity(taskId, "Task id")
     });
 }
 export function createGlobalRole(name, bindings, activeAgentId, workspace, now, profile = {}, defaultAccess = "write") {
-    return createRoleOwner(name, bindings, activeAgentId, workspace, now, profile, defaultAccess);
+    return validateGlobalRole({
+        ...createRoleOwner(name, bindings, activeAgentId, workspace, now, profile, defaultAccess),
+        schemaVersion: 3
+    });
 }
 export function copyGlobalRoleToTaskRole(globalRole, taskId, now, name = globalRole.name) {
     validateGlobalRole(globalRole);
     const timestamp = now.toISOString();
     return validateTaskRole({
         ...cloneProfile(globalRole),
-        schemaVersion: 3,
+        schemaVersion: 4,
         launchRevision: 1,
         defaultAccess: globalRole.defaultAccess,
         taskId: requireSafeIdentity(taskId, "Task id"),
@@ -39,7 +42,6 @@ export function copyGlobalRoleToTaskRole(globalRole, taskId, now, name = globalR
         activeAgentId: globalRole.activeAgentId,
         agentBindings: cloneBindings(globalRole.agentBindings),
         workspace: globalRole.workspace,
-        status: "idle",
         createdAt: timestamp,
         updatedAt: timestamp
     });
@@ -76,12 +78,6 @@ export function updateRole(role, patch, now) {
         : role.launchRevision + 1;
     return validateTaskRole(updated);
 }
-export function updateRoleStatus(role, status, now) {
-    validateTaskRole(role);
-    if (!isRoleStatus(status))
-        throw new Error(`Role status is invalid: ${status}.`);
-    return { ...role, status, updatedAt: now.toISOString() };
-}
 export function switchActiveRoleAgent(role, sessions, targetAgentId, runtime, now) {
     validateRoleOwner(role);
     const normalizedTarget = requireSafeIdentity(targetAgentId, "Target Agent id");
@@ -134,7 +130,7 @@ export function unbindRoleAgent(role, sessions, agentId, now) {
             throw new Error(`Cannot unbind Role Agent with an active Run: ${normalizedAgentId}.`);
         }
         const targetSession = sessions.sessions[normalizedAgentId];
-        if (targetSession !== undefined && targetSession.status !== "stopped") {
+        if (targetSession !== undefined && targetSession.status === "active") {
             throw new Error(`Cannot unbind Role Agent while its native session is ${targetSession.status}: `
                 + `${normalizedAgentId}.`);
         }
@@ -175,8 +171,6 @@ export function validateGlobalRole(role) {
 }
 export function validateTaskRole(role) {
     requireSafeIdentity(role.taskId, "Task id");
-    if (!isRoleStatus(role.status))
-        throw new Error(`Role status is invalid: ${role.status}.`);
     return validateRoleOwner(role);
 }
 function createRoleOwner(name, bindings, activeAgentId, workspace, now, profile, defaultAccess) {
@@ -189,9 +183,8 @@ function createRoleOwner(name, bindings, activeAgentId, workspace, now, profile,
         mappedBindings[binding.agentId] = binding;
     }
     const timestamp = now.toISOString();
-    return validateGlobalRole({
+    return {
         ...cloneProfile(profile),
-        schemaVersion: 3,
         launchRevision: 1,
         defaultAccess,
         name: requireSafeIdentity(name, "Role name"),
@@ -200,11 +193,13 @@ function createRoleOwner(name, bindings, activeAgentId, workspace, now, profile,
         workspace: requireText(workspace, "Role workspace"),
         createdAt: timestamp,
         updatedAt: timestamp
-    });
+    };
 }
 function validateRoleOwner(role) {
-    if (role.schemaVersion !== 3)
+    const expectedSchemaVersion = "taskId" in role ? 4 : 3;
+    if (role.schemaVersion !== expectedSchemaVersion) {
         throw new Error("Role schema version is invalid.");
+    }
     if (!Number.isSafeInteger(role.launchRevision) || role.launchRevision < 1) {
         throw new Error("Role launch revision must be a positive integer.");
     }
@@ -333,6 +328,3 @@ function requireText(value, label) {
         throw new Error(`${label} is required.`);
     return normalized;
 }
-function isRoleStatus(value) {
-    return ["idle", "running", "detached", "exited", "failed"].includes(value);
-}