@ouro.bot/cli 0.1.0-alpha.729 → 0.1.0-alpha.730

package/dist/senses/teams.js

@@ -72,6 +72,7 @@ const http = __importStar(require("http"));
 const path = __importStar(require("path"));
 const trust_gate_1 = require("./trust-gate");
 const pipeline_1 = require("./pipeline");
+const session_transaction_1 = require("../mind/session-transaction");
 const teamsFailoverStates = new Map();
 const pending_1 = require("../mind/pending");
 const continuity_1 = require("./continuity");
@@ -640,18 +641,11 @@ function handleTeamsSlashCommand(text, registry, friendId, conversationId, strea
 }
 /* v8 ignore stop */
 // Handle an incoming Teams message
-async function handleTeamsMessage(text, stream, conversationId, teamsContext, sendMessage, reactionOverrides) {
+async function handleTeamsMessage(text, stream, conversationId, teamsContext, sendMessage, reactionOverrides, runtimeOverrides) {
     const turnKey = teamsTurnKey(conversationId);
     // NOTE: Confirmation resolution is handled in the app.on("message") handler
     // BEFORE the conversation lock.  By the time we get here, any pending
     // confirmation has already been resolved and the reply consumed.
-    // Send first thinking phrase immediately so the user sees feedback
-    // before sync I/O (session load, trim) blocks the event loop.
-    // Skip for reaction signals — they should be processed quietly.
-    if (!reactionOverrides) {
-        stream.update((0, phrases_1.pickPhrase)((0, phrases_1.getPhrases)().thinking) + "...");
-    }
-    await new Promise(r => setImmediate(r));
     // Resolve identity provider early for friend resolution + slash command session path
     const store = getFriendStore();
     const provider = teamsContext?.aadObjectId ? "aad" : "teams-conversation";
@@ -675,167 +669,185 @@ async function handleTeamsMessage(text, stream, conversationId, teamsContext, se
     const sessPath = (0, config_2.sessionPath)(friendId, "teams", conversationId);
     const teamsCapabilities = (0, friends_1.getChannelCapabilities)("teams");
     const pendingDir = (0, pending_1.getPendingDir)((0, identity_1.getAgentName)(), friendId, "teams", conversationId);
-    // Build Teams-specific toolContext fields for injection into the pipeline
-    const teamsToolContext = teamsContext ? {
-        graphToken: teamsContext.graphToken,
-        adoToken: teamsContext.adoToken,
-        githubToken: teamsContext.githubToken,
-        signin: teamsContext.signin,
-        summarize: (0, core_1.createSummarize)("human"),
-        tenantId: teamsContext.tenantId,
-        botApi: teamsContext.botApi,
-    } : {};
-    let currentText = text;
-    const mcpManager = await (0, mcp_manager_1.getSharedMcpManager)() ?? undefined;
-    while (true) {
-        let drainedSteeringFollowUps = [];
-        // Build runAgentOptions with Teams-specific fields
-        const agentOptions = {
-            traceId,
-            toolContext: teamsToolContext,
-            mcpManager,
-            drainSteeringFollowUps: () => {
-                drainedSteeringFollowUps = _turnCoordinator.drainFollowUps(turnKey)
-                    .map(({ text: followUpText, effect }) => ({ text: followUpText, effect }));
-                return drainedSteeringFollowUps;
-            },
-            ...(reactionOverrides?.isReactionSignal ? { isReactionSignal: true } : {}),
-        };
-        // ── Call shared pipeline ──────────────────────────────────────────
-        // Capture terminal errors — failover message replaces the error card if it triggers
-        let capturedTerminalError = null;
-        const teamsFailoverState = (() => {
-            if (!teamsFailoverStates.has(conversationId)) {
-                teamsFailoverStates.set(conversationId, { pending: null });
+    const runWithLease = runtimeOverrides?._withSessionTurnLease ?? session_transaction_1.withSessionTurnLease;
+    try {
+        await runWithLease(sessPath, async (sessionTurnLease) => {
+            // Do not emit any activity until this turn owns the durable session.
+            if (!reactionOverrides) {
+                stream.update((0, phrases_1.pickPhrase)((0, phrases_1.getPhrases)().thinking) + "...");
             }
-            return teamsFailoverStates.get(conversationId);
-        })();
-        /* v8 ignore start -- failover-aware callback wrapper: tested via pipeline integration @preserve */
-        const failoverAwareCallbacks = {
-            ...callbacks,
-            onError: (error, severity) => {
-                if (severity === "terminal" && teamsFailoverState) {
-                    capturedTerminalError = error;
+            await new Promise(r => setImmediate(r));
+            // Build Teams-specific toolContext fields for injection into the pipeline
+            const teamsToolContext = teamsContext ? {
+                graphToken: teamsContext.graphToken,
+                adoToken: teamsContext.adoToken,
+                githubToken: teamsContext.githubToken,
+                signin: teamsContext.signin,
+                summarize: (0, core_1.createSummarize)("human"),
+                tenantId: teamsContext.tenantId,
+                botApi: teamsContext.botApi,
+            } : {};
+            let currentText = text;
+            const mcpManager = await (0, mcp_manager_1.getSharedMcpManager)() ?? undefined;
+            while (true) {
+                let drainedSteeringFollowUps = [];
+                // Build runAgentOptions with Teams-specific fields
+                const agentOptions = {
+                    traceId,
+                    toolContext: teamsToolContext,
+                    mcpManager,
+                    drainSteeringFollowUps: () => {
+                        drainedSteeringFollowUps = _turnCoordinator.drainFollowUps(turnKey)
+                            .map(({ text: followUpText, effect }) => ({ text: followUpText, effect }));
+                        return drainedSteeringFollowUps;
+                    },
+                    ...(reactionOverrides?.isReactionSignal ? { isReactionSignal: true } : {}),
+                };
+                // ── Call shared pipeline ──────────────────────────────────────────
+                // Capture terminal errors — failover message replaces the error card if it triggers
+                let capturedTerminalError = null;
+                const teamsFailoverState = (() => {
+                    if (!teamsFailoverStates.has(conversationId)) {
+                        teamsFailoverStates.set(conversationId, { pending: null });
+                    }
+                    return teamsFailoverStates.get(conversationId);
+                })();
+                /* v8 ignore start -- failover-aware callback wrapper: tested via pipeline integration @preserve */
+                const failoverAwareCallbacks = {
+                    ...callbacks,
+                    onError: (error, severity) => {
+                        if (severity === "terminal" && teamsFailoverState) {
+                            capturedTerminalError = error;
+                            return;
+                        }
+                        callbacks.onError(error, severity);
+                    },
+                };
+                /* v8 ignore stop */
+                const result = await (0, pipeline_1.handleInboundTurn)({
+                    channel: "teams",
+                    sessionKey: conversationId,
+                    capabilities: teamsCapabilities,
+                    messages: [{ role: "user", content: currentText }],
+                    continuityIngressTexts: [currentText],
+                    callbacks: failoverAwareCallbacks,
+                    sessionTurnLease,
+                    friendResolver: { resolve: () => Promise.resolve(resolvedContext) },
+                    sessionLoader: {
+                        loadOrCreate: async () => {
+                            const existing = (0, context_1.loadSession)(sessPath);
+                            const messages = existing?.messages && existing.messages.length > 0
+                                ? existing.messages
+                                : [{ role: "system", content: (0, prompt_1.flattenSystemPrompt)(await (0, prompt_1.buildSystem)("teams", {}, resolvedContext)) }];
+                            (0, core_1.repairOrphanedToolCalls)(messages);
+                            return {
+                                messages,
+                                sessionPath: sessPath,
+                                state: existing?.state,
+                                events: existing?.events,
+                                structuredOutputs: existing?.structuredOutputs,
+                            };
+                        },
+                    },
+                    pendingDir,
+                    friendStore: store,
+                    provider,
+                    externalId,
+                    tenantId: teamsContext?.tenantId,
+                    isGroupChat: false,
+                    groupHasFamilyMember: false,
+                    hasExistingGroupWithFamily: false,
+                    enforceTrustGate: trust_gate_1.enforceTrustGate,
+                    drainPending: pending_1.drainPending,
+                    drainDeferredReturns: (deferredFriendId) => (0, pending_1.drainDeferredReturns)((0, identity_1.getAgentName)(), deferredFriendId),
+                    runAgent: (msgs, cb, channel, sig, opts) => (0, core_1.runAgent)(msgs, cb, channel, sig, {
+                        ...opts,
+                        toolContext: {
+                            /* v8 ignore next -- default no-op signin; pipeline provides the real one @preserve */
+                            signin: async () => undefined,
+                            ...opts?.toolContext,
+                            summarize: teamsToolContext.summarize,
+                        },
+                    }),
+                    postTurn: (turnMessages, sessionPathArg, usage, hooks, state) => {
+                        const prepared = (0, context_1.postTurnTrim)(turnMessages, usage, hooks);
+                        return (0, context_1.deferPostTurnPersist)(sessionPathArg, prepared, usage, state);
+                    },
+                    accumulateFriendTokens: friends_1.accumulateFriendTokens,
+                    signal: controller.signal,
+                    runAgentOptions: agentOptions,
+                    failoverState: teamsFailoverState,
+                });
+                // ── Handle pipeline-intercepted commands ────────────────────────
+                if (result.turnOutcome === "command") {
+                    if (result.commandAction === "new") {
+                        (0, context_1.deleteSession)(sessPath);
+                        stream.emit("session cleared");
+                    }
+                    // For "response" commands: pipeline already emitted the response via onTextChunk
+                    await callbacks.flush();
                     return;
                 }
-                callbacks.onError(error, severity);
-            },
-        };
-        /* v8 ignore stop */
-        const result = await (0, pipeline_1.handleInboundTurn)({
-            channel: "teams",
-            sessionKey: conversationId,
-            capabilities: teamsCapabilities,
-            messages: [{ role: "user", content: currentText }],
-            continuityIngressTexts: [currentText],
-            callbacks: failoverAwareCallbacks,
-            friendResolver: { resolve: () => Promise.resolve(resolvedContext) },
-            sessionLoader: {
-                loadOrCreate: async () => {
-                    const existing = (0, context_1.loadSession)(sessPath);
-                    const messages = existing?.messages && existing.messages.length > 0
-                        ? existing.messages
-                        : [{ role: "system", content: (0, prompt_1.flattenSystemPrompt)(await (0, prompt_1.buildSystem)("teams", {}, resolvedContext)) }];
-                    (0, core_1.repairOrphanedToolCalls)(messages);
-                    return {
-                        messages,
-                        sessionPath: sessPath,
-                        state: existing?.state,
-                        events: existing?.events,
-                        structuredOutputs: existing?.structuredOutputs,
-                    };
-                },
-            },
-            pendingDir,
-            friendStore: store,
-            provider,
-            externalId,
-            tenantId: teamsContext?.tenantId,
-            isGroupChat: false,
-            groupHasFamilyMember: false,
-            hasExistingGroupWithFamily: false,
-            enforceTrustGate: trust_gate_1.enforceTrustGate,
-            drainPending: pending_1.drainPending,
-            drainDeferredReturns: (deferredFriendId) => (0, pending_1.drainDeferredReturns)((0, identity_1.getAgentName)(), deferredFriendId),
-            runAgent: (msgs, cb, channel, sig, opts) => (0, core_1.runAgent)(msgs, cb, channel, sig, {
-                ...opts,
-                toolContext: {
-                    /* v8 ignore next -- default no-op signin; pipeline provides the real one @preserve */
-                    signin: async () => undefined,
-                    ...opts?.toolContext,
-                    summarize: teamsToolContext.summarize,
-                },
-            }),
-            postTurn: (turnMessages, sessionPathArg, usage, hooks, state) => {
-                const prepared = (0, context_1.postTurnTrim)(turnMessages, usage, hooks);
-                (0, context_1.deferPostTurnPersist)(sessionPathArg, prepared, usage, state);
-            },
-            accumulateFriendTokens: friends_1.accumulateFriendTokens,
-            signal: controller.signal,
-            runAgentOptions: agentOptions,
-            failoverState: teamsFailoverState,
-        });
-        // ── Handle pipeline-intercepted commands ────────────────────────
-        if (result.turnOutcome === "command") {
-            if (result.commandAction === "new") {
-                (0, context_1.deleteSession)(sessPath);
-                stream.emit("session cleared");
-            }
-            // For "response" commands: pipeline already emitted the response via onTextChunk
-            await callbacks.flush();
-            return;
-        }
-        /* v8 ignore start -- failover display: tested via pipeline integration tests @preserve */
-        if (result.failoverMessage) {
-            stream.emit(result.failoverMessage);
-        }
-        else if (capturedTerminalError) {
-            callbacks.onError(capturedTerminalError, "terminal");
-        }
-        /* v8 ignore stop */
-        // ── Handle gate result ────────────────────────────────────────
-        if (!result.gateResult.allowed) {
-            if ("autoReply" in result.gateResult && result.gateResult.autoReply) {
-                stream.emit(result.gateResult.autoReply);
+                /* v8 ignore start -- failover display: tested via pipeline integration tests @preserve */
+                if (result.failoverMessage) {
+                    stream.emit(result.failoverMessage);
+                }
+                else if (capturedTerminalError) {
+                    callbacks.onError(capturedTerminalError, "terminal");
+                }
+                /* v8 ignore stop */
+                // ── Handle gate result ────────────────────────────────────────
+                if (!result.gateResult.allowed) {
+                    if ("autoReply" in result.gateResult && result.gateResult.autoReply) {
+                        stream.emit(result.gateResult.autoReply);
+                    }
+                    return;
+                }
+                // Flush any remaining accumulated text at end of turn
+                await callbacks.flush();
+                // After the agent loop, check if any tool returned AUTH_REQUIRED and trigger signin.
+                // This must happen after the stream is done so the OAuth card renders properly.
+                if (teamsContext && result.messages) {
+                    const allContent = result.messages.map(m => typeof m.content === "string" ? m.content : "").join("\n");
+                    if (allContent.includes("AUTH_REQUIRED:graph") && teamsContext.graphConnectionName)
+                        await teamsContext.signin(teamsContext.graphConnectionName);
+                    if (allContent.includes("AUTH_REQUIRED:ado") && teamsContext.adoConnectionName)
+                        await teamsContext.signin(teamsContext.adoConnectionName);
+                    if (allContent.includes("AUTH_REQUIRED:github") && teamsContext.githubConnectionName)
+                        await teamsContext.signin(teamsContext.githubConnectionName);
+                }
+                if (result.turnOutcome !== "superseded") {
+                    return;
+                }
+                const supersedingIndex = drainedSteeringFollowUps
+                    .map((followUp) => followUp.effect)
+                    .lastIndexOf("clear_and_supersede");
+                if (supersedingIndex < 0) {
+                    return;
+                }
+                const supersedingFollowUp = drainedSteeringFollowUps[supersedingIndex];
+                const replayTail = drainedSteeringFollowUps
+                    .slice(supersedingIndex + 1)
+                    .map((followUp) => followUp.text.trim())
+                    .filter((followUpText) => followUpText.length > 0)
+                    .join("\n");
+                if (replayTail) {
+                    currentText = replayTail;
+                    continue;
+                }
+                if (handleTeamsSlashCommand(supersedingFollowUp.text, createTeamsCommandRegistry(), friendId, conversationId, stream, false)) {
+                    return;
+                }
+                currentText = supersedingFollowUp.text;
             }
+        });
+    }
+    catch (error) {
+        if (error instanceof session_transaction_1.SessionTurnBusyError || (error instanceof Error && error.name === "SessionTurnBusyError")) {
+            stream.emit("this conversation is busy finishing another turn; please try again");
             return;
         }
-        // Flush any remaining accumulated text at end of turn
-        await callbacks.flush();
-        // After the agent loop, check if any tool returned AUTH_REQUIRED and trigger signin.
-        // This must happen after the stream is done so the OAuth card renders properly.
-        if (teamsContext && result.messages) {
-            const allContent = result.messages.map(m => typeof m.content === "string" ? m.content : "").join("\n");
-            if (allContent.includes("AUTH_REQUIRED:graph") && teamsContext.graphConnectionName)
-                await teamsContext.signin(teamsContext.graphConnectionName);
-            if (allContent.includes("AUTH_REQUIRED:ado") && teamsContext.adoConnectionName)
-                await teamsContext.signin(teamsContext.adoConnectionName);
-            if (allContent.includes("AUTH_REQUIRED:github") && teamsContext.githubConnectionName)
-                await teamsContext.signin(teamsContext.githubConnectionName);
-        }
-        if (result.turnOutcome !== "superseded") {
-            return;
-        }
-        const supersedingIndex = drainedSteeringFollowUps
-            .map((followUp) => followUp.effect)
-            .lastIndexOf("clear_and_supersede");
-        if (supersedingIndex < 0) {
-            return;
-        }
-        const supersedingFollowUp = drainedSteeringFollowUps[supersedingIndex];
-        const replayTail = drainedSteeringFollowUps
-            .slice(supersedingIndex + 1)
-            .map((followUp) => followUp.text.trim())
-            .filter((followUpText) => followUpText.length > 0)
-            .join("\n");
-        if (replayTail) {
-            currentText = replayTail;
-            continue;
-        }
-        if (handleTeamsSlashCommand(supersedingFollowUp.text, createTeamsCommandRegistry(), friendId, conversationId, stream, false)) {
-            return;
-        }
-        currentText = supersedingFollowUp.text;
+        throw error;
     }
 }
 // Internal port for the secondary bot App (not exposed externally).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouro.bot/cli",
-  "version": "0.1.0-alpha.729",
+  "version": "0.1.0-alpha.730",
   "engines": {
     "node": ">=22"
   },
@@ -65,6 +65,7 @@
     "@ouro.bot/friends": "0.1.0-alpha.7",
     "@types/react": "^17.0.91",
     "@types/ws": "^8.18.1",
+    "ajv": "^8.20.0",
     "better-sqlite3": "13.0.2",
     "fast-glob": "^3.3.3",
     "ink": "^3.2.0",