Remove all the sdk retry code. I think retries should be at the level of individual llm calls, not the whole run, which could have unexpected behavior

Author: jahooma

cli/src/hooks/use-send-message.ts

@@ -373,11 +373,8 @@ export const useSendMessage = ({
           prompt: effectivePrompt,
           content: messageContent,
           previousRunState: previousRunStateRef.current,
-          abortController,
           agentDefinitions,
           eventHandlerState,
-          setIsRetrying,
-          setStreamStatus,
         })
 
         const runState = await client.run(runConfig)

cli/src/utils/create-run-config.ts

@@ -1,9 +1,3 @@
-import {
-  MAX_RETRIES_PER_MESSAGE,
-  RETRY_BACKOFF_BASE_DELAY_MS,
-  RETRY_BACKOFF_MAX_DELAY_MS,
-} from '@codebuff/sdk'
-
 import {
   createEventHandler,
   createStreamChunkHandler,
@@ -12,30 +6,15 @@ import {
 import type { EventHandlerState } from './sdk-event-handlers'
 import type { AgentDefinition, MessageContent, RunState } from '@codebuff/sdk'
 import type { Logger } from '@codebuff/common/types/contracts/logger'
-import type { StreamStatus } from '../hooks/use-message-queue'
 
 export type CreateRunConfigParams = {
   logger: Logger
   agent: AgentDefinition | string
   prompt: string
   content: MessageContent[] | undefined
   previousRunState: RunState | null
-  abortController: AbortController
   agentDefinitions: AgentDefinition[]
   eventHandlerState: EventHandlerState
-  setIsRetrying: (retrying: boolean) => void
-  setStreamStatus: (status: StreamStatus) => void
-}
-
-type RetryArgs = {
-  attempt: number
-  delayMs: number
-  statusCode?: number
-}
-
-type RetryExhaustedArgs = {
-  totalAttempts: number
-  statusCode?: number
 }
 
 export const createRunConfig = (params: CreateRunConfigParams) => {
@@ -45,11 +24,8 @@ export const createRunConfig = (params: CreateRunConfigParams) => {
     prompt,
     content,
     previousRunState,
-    abortController,
     agentDefinitions,
     eventHandlerState,
-    setIsRetrying,
-    setStreamStatus,
   } = params
 
   return {
@@ -58,26 +34,6 @@ export const createRunConfig = (params: CreateRunConfigParams) => {
     prompt,
     content,
     previousRun: previousRunState ?? undefined,
-    abortController,
-    retry: {
-      maxRetries: MAX_RETRIES_PER_MESSAGE,
-      backoffBaseMs: RETRY_BACKOFF_BASE_DELAY_MS,
-      backoffMaxMs: RETRY_BACKOFF_MAX_DELAY_MS,
-      onRetry: async ({ attempt, delayMs, statusCode }: RetryArgs) => {
-        logger.warn(
-          { sdkAttempt: attempt, delayMs, statusCode },
-          'SDK retrying after error',
-        )
-        setIsRetrying(true)
-        setStreamStatus('waiting')
-      },
-      onRetryExhausted: async ({
-        totalAttempts,
-        statusCode,
-      }: RetryExhaustedArgs) => {
-        logger.warn({ totalAttempts, statusCode }, 'SDK exhausted all retries')
-      },
-    },
     agentDefinitions,
     maxAgentSteps: 100,
     handleStreamChunk: createStreamChunkHandler(eventHandlerState),