fix(claude): move model usage log to result message only

[louis-thorp-datacom]

Summary

• The 'modelUsage' debug log on line 597 of 'server/claude-sdk.js' ran on every streamed SDK message, but 'modelUsage' is only populated on 'result' messages per the SDK type definitions. This produced repeated '---> Model was sent using: []' console output for every non-result message during streaming.
• Moved the log inside the existing 'if (message.type === 'result')' block so it fires once with the actual model data.

Summary by CodeRabbit

• Refactor
  • Optimized debug logging by moving it to be more targeted and conditional, improving performance efficiency during result processing.

[coderabbitai]

📝 Walkthrough

Walkthrough

A debug log statement for modelUsage was relocated from the global message handler to the result message branch with a conditional guard to only log when modelUsage entries exist, refining debug output specificity.

Changes

Cohort / File(s)	Summary
Debug Logging Adjustment server/claude-sdk.js	Moved modelUsage debug log into the message.type === 'result' branch with a conditional guard for non-empty modelUsage, removing the unconditional log from the general message processing flow.

Possibly related PRs

• chore: upgrade @anthropic-ai/claude-agent-sdk to version 0.2.59 and add model usage logging #446: Also modifies modelUsage debug logging in server/claude-sdk.js, but adds a log earlier in the flow whereas this PR moves and guards the log within the result branch.

Suggested reviewers

• viper151

Poem

🐰 A log finds a new home today,
Guarded and wise, no longer at play,
In the result's embrace it shall stay,
Only when models have something to say! 📝

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: moving the model usage log from outside to inside the result message handling block.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}