From c25c276e00b9bedd41ca0c1ea60b44294eb1c78a Mon Sep 17 00:00:00 2001 From: SK Akram Date: Mon, 16 Feb 2026 12:09:22 +0000 Subject: [PATCH] refactor: remove unnecessary optional chaining from agent meta usage in reply and cron modules --- src/auto-reply/reply/agent-runner.ts | 18 +++++++++--------- src/auto-reply/reply/followup-runner.ts | 10 +++++----- src/cron/isolated-agent/run.ts | 10 +++++----- 3 files changed, 19 insertions(+), 19 deletions(-) diff --git a/src/auto-reply/reply/agent-runner.ts b/src/auto-reply/reply/agent-runner.ts index aaa153a3a0e..3ab99f7f7d9 100644 --- a/src/auto-reply/reply/agent-runner.ts +++ b/src/auto-reply/reply/agent-runner.ts @@ -409,13 +409,13 @@ export async function runReplyAgent(params: { await Promise.allSettled(pendingToolTasks); } - const usage = runResult?.meta?.agentMeta?.usage; - const promptTokens = runResult?.meta?.agentMeta?.promptTokens; - const modelUsed = runResult?.meta?.agentMeta?.model ?? fallbackModel ?? defaultModel; + const usage = runResult.meta?.agentMeta?.usage; + const promptTokens = runResult.meta?.agentMeta?.promptTokens; + const modelUsed = runResult.meta?.agentMeta?.model ?? fallbackModel ?? defaultModel; const providerUsed = - runResult?.meta?.agentMeta?.provider ?? fallbackProvider ?? followupRun.run.provider; + runResult.meta?.agentMeta?.provider ?? fallbackProvider ?? followupRun.run.provider; const cliSessionId = isCliProvider(providerUsed, cfg) - ? runResult?.meta?.agentMeta?.sessionId?.trim() + ? runResult.meta?.agentMeta?.sessionId?.trim() : undefined; const contextTokensUsed = agentCfgContextTokens ?? @@ -427,12 +427,12 @@ export async function runReplyAgent(params: { storePath, sessionKey, usage, - lastCallUsage: runResult?.meta?.agentMeta?.lastCallUsage, + lastCallUsage: runResult.meta?.agentMeta?.lastCallUsage, promptTokens, modelUsed, providerUsed, contextTokensUsed, - systemPromptReport: runResult?.meta?.systemPromptReport, + systemPromptReport: runResult.meta?.systemPromptReport, cliSessionId, }); @@ -509,7 +509,7 @@ export async function runReplyAgent(params: { promptTokens, total: totalTokens, }, - lastCallUsage: runResult?.meta?.agentMeta?.lastCallUsage, + lastCallUsage: runResult.meta?.agentMeta?.lastCallUsage, context: { limit: contextTokensUsed, used: totalTokens, @@ -555,7 +555,7 @@ export async function runReplyAgent(params: { sessionStore: activeSessionStore, sessionKey, storePath, - lastCallUsage: runResult?.meta?.agentMeta?.lastCallUsage, + lastCallUsage: runResult.meta?.agentMeta?.lastCallUsage, contextTokensUsed, }); diff --git a/src/auto-reply/reply/followup-runner.ts b/src/auto-reply/reply/followup-runner.ts index d842bf7c837..cefb8cd21b1 100644 --- a/src/auto-reply/reply/followup-runner.ts +++ b/src/auto-reply/reply/followup-runner.ts @@ -194,9 +194,9 @@ export function createFollowupRunner(params: { return; } - const usage = runResult?.meta?.agentMeta?.usage; - const promptTokens = runResult?.meta?.agentMeta?.promptTokens; - const modelUsed = runResult?.meta?.agentMeta?.model ?? fallbackModel ?? defaultModel; + const usage = runResult.meta?.agentMeta?.usage; + const promptTokens = runResult.meta?.agentMeta?.promptTokens; + const modelUsed = runResult.meta?.agentMeta?.model ?? fallbackModel ?? defaultModel; const contextTokensUsed = agentCfgContextTokens ?? lookupContextTokens(modelUsed) ?? @@ -208,7 +208,7 @@ export function createFollowupRunner(params: { storePath, sessionKey, usage, - lastCallUsage: runResult?.meta?.agentMeta?.lastCallUsage, + lastCallUsage: runResult.meta?.agentMeta?.lastCallUsage, promptTokens, modelUsed, providerUsed: fallbackProvider, @@ -275,7 +275,7 @@ export function createFollowupRunner(params: { sessionStore, sessionKey, storePath, - lastCallUsage: runResult?.meta?.agentMeta?.lastCallUsage, + lastCallUsage: runResult.meta?.agentMeta?.lastCallUsage, contextTokensUsed, }); if (queued.run.verboseLevel && queued.run.verboseLevel !== "off") { diff --git a/src/cron/isolated-agent/run.ts b/src/cron/isolated-agent/run.ts index 37b0a178b84..70a2afa7d43 100644 --- a/src/cron/isolated-agent/run.ts +++ b/src/cron/isolated-agent/run.ts @@ -500,10 +500,10 @@ export async function runCronIsolatedAgentTurn(params: { } | undefined; { - const usage = runResult?.meta?.agentMeta?.usage; - const promptTokens = runResult?.meta?.agentMeta?.promptTokens; - const modelUsed = runResult?.meta?.agentMeta?.model ?? fallbackModel ?? model; - const providerUsed = runResult?.meta?.agentMeta?.provider ?? fallbackProvider ?? provider; + const usage = runResult.meta?.agentMeta?.usage; + const promptTokens = runResult.meta?.agentMeta?.promptTokens; + const modelUsed = runResult.meta?.agentMeta?.model ?? fallbackModel ?? model; + const providerUsed = runResult.meta?.agentMeta?.provider ?? fallbackProvider ?? provider; const contextTokens = agentCfg?.contextTokens ?? lookupContextTokens(modelUsed) ?? DEFAULT_CONTEXT_TOKENS; @@ -511,7 +511,7 @@ export async function runCronIsolatedAgentTurn(params: { cronSession.sessionEntry.model = modelUsed; cronSession.sessionEntry.contextTokens = contextTokens; if (isCliProvider(providerUsed, cfgWithAgentDefaults)) { - const cliSessionId = runResult?.meta?.agentMeta?.sessionId?.trim(); + const cliSessionId = runResult.meta?.agentMeta?.sessionId?.trim(); if (cliSessionId) { setCliSessionId(cronSession.sessionEntry, providerUsed, cliSessionId); }