@ottocode/server 0.1.234 → 0.1.236

package/src/runtime/stream/step-finish.ts

@@ -6,7 +6,6 @@ import type { RunOpts } from '../session/queue.ts';
 import type { ToolAdapterContext } from '../../tools/adapter.ts';
 import type { UsageData, ProviderMetadata } from '../session/db-operations.ts';
 import type { StepFinishEvent } from './types.ts';
-import { debugLog } from '../debug/index.ts';
 
 export function createStepFinishHandler(
 	opts: RunOpts,
@@ -45,11 +44,7 @@ export function createStepFinishHandler(
 					.set({ completedAt: finishedAt })
 					.where(eq(messageParts.id, currentPartId));
 			}
-		} catch (err) {
-			debugLog(
-				`[step-finish] Failed to update part completedAt: ${err instanceof Error ? err.message : String(err)}`,
-			);
-		}
+		} catch {}
 
 		if (step.usage) {
 			try {
@@ -59,11 +54,7 @@ export function createStepFinishHandler(
 					opts,
 					db,
 				);
-			} catch (err) {
-				debugLog(
-					`[step-finish] Failed to update session tokens: ${err instanceof Error ? err.message : String(err)}`,
-				);
-			}
+			} catch {}
 
 			try {
 				await updateMessageTokensIncrementalFn(
@@ -72,11 +63,7 @@ export function createStepFinishHandler(
 					opts,
 					db,
 				);
-			} catch (err) {
-				debugLog(
-					`[step-finish] Failed to update message tokens: ${err instanceof Error ? err.message : String(err)}`,
-				);
-			}
+			} catch {}
 		}
 
 		try {
@@ -97,21 +84,13 @@ export function createStepFinishHandler(
 					payload: { stepIndex, ...step.usage },
 				});
 			}
-		} catch (err) {
-			debugLog(
-				`[step-finish] Failed to publish finish-step: ${err instanceof Error ? err.message : String(err)}`,
-			);
-		}
+		} catch {}
 
 		try {
 			const newStepIndex = incrementStepIndex();
 			sharedCtx.stepIndex = newStepIndex;
 			updateCurrentPartId(null);
 			updateAccumulated('');
-		} catch (err) {
-			debugLog(
-				`[step-finish] Failed to increment step: ${err instanceof Error ? err.message : String(err)}`,
-			);
-		}
+		} catch {}
 	};
 }

package/src/runtime/tools/approval.ts

@@ -1,5 +1,4 @@
 import { publish } from '../../events/bus.ts';
-import { debugLog } from '../debug/index.ts';
 
 export type ToolApprovalMode = 'auto' | 'dangerous' | 'all';
 
@@ -49,12 +48,6 @@ export async function requestApproval(
 	args: unknown,
 	timeoutMs = 120000,
 ): Promise<boolean> {
-	debugLog('[approval] requestApproval called', {
-		sessionId,
-		messageId,
-		callId,
-		toolName,
-	});
 	return new Promise((resolve) => {
 		const approval: PendingApproval = {
 			callId,
@@ -67,10 +60,6 @@ export async function requestApproval(
 		};
 
 		pendingApprovals.set(callId, approval);
-		debugLog(
-			'[approval] Added to pendingApprovals, count:',
-			pendingApprovals.size,
-		);
 
 		publish({
 			type: 'tool.approval.required',
@@ -106,15 +95,8 @@ export function resolveApproval(
 	callId: string,
 	approved: boolean,
 ): { ok: boolean; error?: string } {
-	debugLog('[approval] resolveApproval called', {
-		callId,
-		approved,
-		pendingCount: pendingApprovals.size,
-		pendingIds: [...pendingApprovals.keys()],
-	});
 	const approval = pendingApprovals.get(callId);
 	if (!approval) {
-		debugLog('[approval] No pending approval found for callId:', callId);
 		return { ok: false, error: 'No pending approval found for this callId' };
 	}
 

package/src/runtime/utils/token.ts

@@ -1,5 +1,4 @@
 import { catalog } from '@ottocode/sdk';
-import { debugLog } from '../debug/index.ts';
 import type { ProviderName } from '../provider/index.ts';
 
 /**
@@ -16,23 +15,15 @@ export function getMaxOutputTokens(
 	try {
 		const providerCatalog = catalog[provider];
 		if (!providerCatalog) {
-			debugLog(`[maxOutputTokens] No catalog found for provider: ${provider}`);
 			return undefined;
 		}
 		const modelInfo = providerCatalog.models.find((m) => m.id === modelId);
 		if (!modelInfo) {
-			debugLog(
-				`[maxOutputTokens] No model info found for: ${modelId} in provider: ${provider}`,
-			);
 			return undefined;
 		}
 		const outputLimit = modelInfo.limit?.output;
-		debugLog(
-			`[maxOutputTokens] Provider: ${provider}, Model: ${modelId}, Limit: ${outputLimit}`,
-		);
 		return outputLimit;
-	} catch (err) {
-		debugLog(`[maxOutputTokens] Error looking up limit: ${err}`);
+	} catch {
 		return undefined;
 	}
 }

package/src/tools/adapter.ts

@@ -2,7 +2,7 @@ import type { Tool } from 'ai';
 import { messageParts, sessions } from '@ottocode/database/schema';
 import { eq } from 'drizzle-orm';
 import { publish } from '../events/bus.ts';
-import type { DiscoveredTool } from '@ottocode/sdk';
+import { logger, type DiscoveredTool } from '@ottocode/sdk';
 import { getCwd, setCwd, joinRelative } from '../runtime/utils/cwd.ts';
 import type {
 	ToolAdapterContext,
@@ -18,7 +18,6 @@ import {
 	requestApproval,
 } from '../runtime/tools/approval.ts';
 import { guardToolCall } from '../runtime/tools/guards.ts';
-import { debugLog } from '../runtime/debug/index.ts';
 
 export type { ToolAdapterContext } from '../runtime/tools/context.ts';
 
@@ -63,17 +62,9 @@ function extractToolCallId(options: unknown): string | undefined {
 const DEFAULT_TRACED_TOOL_INPUTS = new Set(['write', 'apply_patch']);
 
 function shouldTraceToolInput(name: string): boolean {
-	const raw = process.env.OTTO_DEBUG_TOOL_INPUT?.trim();
-	if (!raw) return false;
-	const normalized = raw.toLowerCase();
-	if (['1', 'true', 'yes', 'on', 'all'].includes(normalized)) {
-		return DEFAULT_TRACED_TOOL_INPUTS.has(name);
-	}
-	const tokens = raw
-		.split(/[\s,]+/)
-		.map((token) => token.trim().toLowerCase())
-		.filter(Boolean);
-	return tokens.includes('all') || tokens.includes(name.toLowerCase());
+	void DEFAULT_TRACED_TOOL_INPUTS;
+	void name;
+	return false;
 }
 
 function summarizeTraceValue(value: unknown, max = 160): string {
@@ -239,9 +230,7 @@ export function adaptTools(
 					stepIndex: ctx.stepIndex,
 				});
 				if (shouldTraceToolInput(name)) {
-					debugLog(
-						`[TOOL_INPUT_TRACE][adapter] onInputStart tool=${name} callId=${sdkCallId ?? queue[queue.length - 1]?.callId ?? 'unknown'} step=${ctx.stepIndex}`,
-					);
+					void (sdkCallId ?? queue[queue.length - 1]?.callId ?? 'unknown');
 				}
 				if (typeof base.onInputStart === 'function')
 					// biome-ignore lint/suspicious/noExplicitAny: AI SDK types are complex
@@ -254,9 +243,8 @@ export function adaptTools(
 				const queue = pendingCalls.get(name);
 				const meta = queue?.length ? queue[queue.length - 1] : undefined;
 				if (shouldTraceToolInput(name)) {
-					debugLog(
-						`[TOOL_INPUT_TRACE][adapter] onInputDelta tool=${name} callId=${sdkCallId ?? meta?.callId ?? 'unknown'} step=${meta?.stepIndex ?? ctx.stepIndex} delta=${summarizeTraceValue(delta ?? '')}`,
-					);
+					void (sdkCallId ?? meta?.callId ?? 'unknown');
+					void summarizeTraceValue(delta ?? '');
 				}
 				// Stream tool argument deltas as events if needed
 				publish({
@@ -297,9 +285,8 @@ export function adaptTools(
 				const callPartId = crypto.randomUUID();
 				const startTs = meta.startTs;
 				if (shouldTraceToolInput(name)) {
-					debugLog(
-						`[TOOL_INPUT_TRACE][adapter] onInputAvailable tool=${name} callId=${callId} step=${ctx.stepIndex} input=${summarizeTraceValue(args)}`,
-					);
+					void callId;
+					void summarizeTraceValue(args);
 				}
 
 				if (
@@ -325,6 +312,13 @@ export function adaptTools(
 							messageId: ctx.messageId,
 						},
 					});
+					logger.debug(`[tools] call ${name}`, {
+						sessionId: ctx.sessionId,
+						messageId: ctx.messageId,
+						toolName: name,
+						callId,
+						stepIndex: ctx.stepIndex,
+					});
 					// Persist synchronously to maintain correct ordering
 					try {
 						const index = await ctx.nextIndex();
@@ -702,6 +696,13 @@ export function adaptTools(
 							sessionId: ctx.sessionId,
 							payload: { ...contentObj, stepIndex: stepIndexForEvent },
 						});
+						logger.debug(`[tools] result ${name}`, {
+							sessionId: ctx.sessionId,
+							messageId: ctx.messageId,
+							toolName: name,
+							callId,
+							stepIndex: stepIndexForEvent,
+						});
 						if (name === 'update_todos') {
 							try {
 								const resultValue = (contentObj as { result?: unknown })