Refactor three files to pass logger as parameter

Refactored batch-str-replace.ts, stream-parser.ts, and api/agents.ts to pass logger as a parameter instead of importing it directly. Updated all callers including run-agent-step.ts and test files.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>

Author: charleslien

backend/src/__tests__/malformed-tool-call.test.ts

@@ -131,6 +131,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     // Should have tool result errors in the final message history
@@ -187,6 +188,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     // Should have multiple error tool results
@@ -233,6 +235,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     // Should have error in both toolResults and message history
@@ -283,6 +286,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     const toolMessages = result.state.messages.filter(
@@ -335,6 +339,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     const toolMessages = result.state.messages.filter(
@@ -389,6 +394,7 @@ describe('malformed tool call error handling', () => {
       agentContext: {},
       onResponseChunk,
       fullResponse: '',
+      logger,
     })
 
     const toolMessages = result.state.messages.filter(

backend/src/__tests__/process-str-replace.test.ts

@@ -1,14 +1,14 @@
-import {
-  describe,
-  expect,
-  it,
-  spyOn,
-  beforeEach,
-  afterEach,
-  mock,
-} from 'bun:test'
+import { describe, expect, it, mock } from 'bun:test'
 import { applyPatch } from 'diff'
 
+import { processStrReplace } from '../process-str-replace'
+import {
+  executeBatchStrReplaces,
+  benchifyCanFixLanguage,
+} from '../tools/batch-str-replace'
+
+import type { Logger } from '@codebuff/types/logger'
+
 // Mock the benchify module to simulate missing API key
 mock.module('benchify', () => ({
   Benchify: class MockBenchify {
@@ -19,15 +19,6 @@ mock.module('benchify', () => ({
   },
 }))
 
-import { processStrReplace } from '../process-str-replace'
-import { mockFileContext } from './test-utils'
-import {
-  executeBatchStrReplaces,
-  benchifyCanFixLanguage,
-} from '../tools/batch-str-replace'
-
-import type { Logger } from '@codebuff/types/logger'
-
 const logger: Logger = {
   debug: () => {},
   info: () => {},
@@ -283,12 +274,12 @@ describe('processStrReplace', () => {
       const oldStr = 'const x'
       const newStr = 'let x'
 
-    const result = await processStrReplace({
-      path: 'test.ts',
-      replacements: [{ old: oldStr, new: newStr, allowMultiple: false }],
-      initialContentPromise: Promise.resolve(initialContent),
-      logger,
-    })
+      const result = await processStrReplace({
+        path: 'test.ts',
+        replacements: [{ old: oldStr, new: newStr, allowMultiple: false }],
+        initialContentPromise: Promise.resolve(initialContent),
+        logger,
+      })
 
       expect(result).not.toBeNull()
       expect('error' in result).toBe(true)
@@ -563,6 +554,7 @@ describe('Benchify resilience', () => {
         onResponseChunk: () => {},
         state: { messages: [] },
         userId: 'test-user',
+        logger,
       })
 
       // Should complete without error even when Benchify is unavailable
@@ -590,6 +582,7 @@ describe('Benchify resilience', () => {
           onResponseChunk: () => {},
           state: { messages: [] },
           userId: 'test-user',
+          logger,
         }),
       ).resolves.toBeUndefined() // Should complete without throwing
     })
@@ -627,6 +620,7 @@ describe('Benchify resilience', () => {
       onResponseChunk: () => {},
       state: { messages: [] },
       userId: 'test-user',
+      logger,
     })
 
     // Should complete without throwing an error

backend/src/run-agent-step.ts

@@ -306,6 +306,7 @@ export const runAgentStep = async (
     agentContext,
     onResponseChunk,
     fullResponse,
+    logger,
   })
   toolResults.push(...newToolResults)