@ottocode/server 0.1.227 → 0.1.228

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ottocode/server",
-	"version": "0.1.227",
+	"version": "0.1.228",
 	"description": "HTTP API server for ottocode",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -49,8 +49,8 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"dependencies": {
-		"@ottocode/sdk": "0.1.227",
-		"@ottocode/database": "0.1.227",
+		"@ottocode/sdk": "0.1.228",
+		"@ottocode/database": "0.1.228",
 		"drizzle-orm": "^0.44.5",
 		"hono": "^4.9.9",
 		"zod": "^4.3.6"

package/src/runtime/agent/oauth-codex-continuation.ts

@@ -10,6 +10,7 @@ export type OauthCodexContinuationInput = {
 	firstToolSeen: boolean;
 	hasTrailingAssistantText: boolean;
 	endedWithToolActivity?: boolean;
+	lastToolName?: string;
 	droppedPseudoToolText: boolean;
 	lastAssistantText: string;
 };
@@ -54,7 +55,7 @@ const MAX_UNCLEAN_EOF_RETRIES = 1;
 
 function isUncleanEof(input: OauthCodexContinuationInput): boolean {
 	if (input.finishReason && input.finishReason !== 'unknown') return false;
-	if (input.firstToolSeen) return true;
+	if (isMissingAssistantSummary(input)) return true;
 	if (looksLikeIntermediateProgressText(input.lastAssistantText)) return true;
 	return false;
 }
@@ -82,6 +83,10 @@ export function decideOauthCodexContinuation(
 		return { shouldContinue: true, reason: 'truncated' };
 	}
 
+	if (input.lastToolName === 'finish') {
+		return { shouldContinue: false };
+	}
+
 	if (input.endedWithToolActivity) {
 		return { shouldContinue: true, reason: 'ended-on-tool-activity' };
 	}

package/src/runtime/agent/runner.ts

@@ -182,12 +182,20 @@ async function runAssistant(opts: RunOpts) {
 	let _finishObserved = false;
 	let _toolActivityObserved = false;
 	let _trailingAssistantTextAfterTool = false;
+	let _endedWithToolActivity = false;
+	let _lastToolName: string | undefined;
 	let _abortedByUser = false;
 	let titleGenerationTriggered = false;
 	const unsubscribeFinish = subscribe(opts.sessionId, (evt) => {
 		if (evt.type === 'tool.call' || evt.type === 'tool.result') {
 			_toolActivityObserved = true;
 			_trailingAssistantTextAfterTool = false;
+			_endedWithToolActivity = true;
+			try {
+				_lastToolName = (evt.payload as { name?: string } | undefined)?.name;
+			} catch {
+				_lastToolName = undefined;
+			}
 		}
 		if (evt.type === 'tool.call') {
 			triggerTitleGenerationWhenReady();
@@ -327,6 +335,7 @@ async function runAssistant(opts: RunOpts) {
 					(delta.trim().length > 0 && firstToolSeen())
 				) {
 					_trailingAssistantTextAfterTool = true;
+					_endedWithToolActivity = false;
 				}
 
 				if (!currentPartId && !accumulated.trim()) {
@@ -443,8 +452,6 @@ async function runAssistant(opts: RunOpts) {
 
 		const MAX_CONTINUATIONS = 6;
 		const continuationCount = opts.continuationCount ?? 0;
-		const endedWithToolActivity =
-			_toolActivityObserved && !_trailingAssistantTextAfterTool;
 		const continuationDecision = decideOauthCodexContinuation({
 			provider: opts.provider,
 			isOpenAIOAuth,
@@ -456,7 +463,8 @@ async function runAssistant(opts: RunOpts) {
 			rawFinishReason: streamRawFinishReason,
 			firstToolSeen: fs,
 			hasTrailingAssistantText: _trailingAssistantTextAfterTool,
-			endedWithToolActivity,
+			endedWithToolActivity: _endedWithToolActivity,
+			lastToolName: _lastToolName,
 			droppedPseudoToolText: oauthTextGuard?.dropped ?? false,
 			lastAssistantText: latestAssistantText,
 		});