fix: response model would overwrite request model

[dinmukhamedm]

Important

Fix incorrect model usage in get_llm_usage_for_span in utils.rs and correct a typo in attributes.rs.

• Behavior:
  • Fix get_llm_usage_for_span in utils.rs to correctly use response_model instead of request_model for response calculations.
• Misc:
  • Fix typo in comment in attributes.rs.

^{This description was created by for b6e8384. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b6e8384 in 18 seconds

More details

• Looked at 35 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. app-server/src/traces/utils.rs:81

• Draft comment:
  The change from request_model to response_model is correct and fixes the bug where the response model was incorrectly set to the request model.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR fixes a bug where the response model was incorrectly set to the request model. The change is correct and aligns with the intended functionality.

Workflow ID: wflow_pJxRKCbAiMTEuX1c

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.