Skip to content

Commit 99c4da7

Browse files
smhananclaude
andcommitted
refactor: cleanup per code audit
- Remove unused _ensure_worker_created method (dead code) - Change tool call logging from debug to trace level for consistency 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent a5bd172 commit 99c4da7

File tree

2 files changed

+3
-30
lines changed

2 files changed

+3
-30
lines changed

ccproxy/llms/streaming/processors.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -424,7 +424,7 @@ async def _process_chunk(
424424
elif self.tool_calls and self.enable_tool_calls:
425425
# Send completed tool calls for both SSE and dict formats
426426
# Previous bug: Only sent for SSE format, causing dict format (SDK mode) to miss tool calls
427-
logger.debug(
427+
logger.trace(
428428
"openai_stream_sending_tool_calls",
429429
tool_count=len(self.tool_calls),
430430
output_format=self.output_format,
@@ -434,7 +434,7 @@ async def _process_chunk(
434434
for tool_call_index, (tool_call_id, tool_call) in enumerate(
435435
self.tool_calls.items()
436436
):
437-
logger.debug(
437+
logger.trace(
438438
"openai_stream_tool_call_yielding",
439439
tool_call_id=tool_call_id,
440440
tool_name=tool_call["name"],
@@ -462,7 +462,7 @@ async def _process_chunk(
462462
)
463463

464464
# Clear tool_calls after yielding to prevent duplicates
465-
logger.debug(
465+
logger.trace(
466466
"openai_stream_clearing_tool_calls",
467467
cleared_count=len(self.tool_calls),
468468
category="streaming_conversion",

ccproxy/plugins/claude_sdk/stream_handle.py

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -158,33 +158,6 @@ async def create_listener(self) -> AsyncIterator[Any]:
158158
# Check if we should trigger cleanup
159159
await self._check_cleanup()
160160

161-
async def _ensure_worker_created(self) -> None:
162-
"""Ensure the worker is created (but not started).
163-
164-
Note: Worker is now started in create_listener() AFTER listener
165-
registration to prevent race conditions with fast STDIO tools.
166-
"""
167-
async with self._worker_lock:
168-
if self._worker is None:
169-
# Create worker (but don't start - that happens after listener registration)
170-
worker_id = f"{self.handle_id}-worker"
171-
self._worker = StreamWorker(
172-
worker_id=worker_id,
173-
message_iterator=self._message_iterator,
174-
session_id=self.session_id,
175-
request_id=self.request_id,
176-
session_client=self._session_client,
177-
stream_handle=self, # Pass self for message tracking
178-
)
179-
180-
logger.debug(
181-
"stream_handle_worker_created",
182-
handle_id=self.handle_id,
183-
worker_id=worker_id,
184-
session_id=self.session_id,
185-
category="streaming",
186-
)
187-
188161
async def _remove_listener(self, listener_id: str) -> None:
189162
"""Remove a listener and clean it up.
190163

0 commit comments

Comments
 (0)