Skip to content

fix: exclude embedding return_usage from provider transport options#471

Merged
mikehostetler merged 1 commit intomainfrom
fix/embedding-return-usage-transport-leak
Feb 27, 2026
Merged

fix: exclude embedding return_usage from provider transport options#471
mikehostetler merged 1 commit intomainfrom
fix/embedding-return-usage-transport-leak

Conversation

@mikehostetler
Copy link
Contributor

Summary

  • exclude from embedding schema used by provider option processing
  • keep as a high-level concern only
  • add regression test to ensure processed embedding transport opts never include

Why

CI on fails with from in embedding provider request paths.

Validation

  • Running ExUnit with seed: 195034, max_cases: 20
    Excluding tags: [:coverage, :integration]

.............................................................................................................................................................................................................................................................................................................................................................
Finished in 0.5 seconds (0.5s async, 0.04s sync)
349 tests, 0 failures

  • Running ExUnit with seed: 790361, max_cases: 20
    Excluding tags: [:coverage, :integration]

.....................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................*****.........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
Finished in 3.7 seconds (3.6s async, 0.1s sync)
2201 tests, 0 failures, 11 skipped (116 excluded)

@mikehostetler mikehostetler merged commit f816b1e into main Feb 27, 2026
6 checks passed
@mikehostetler mikehostetler deleted the fix/embedding-return-usage-transport-leak branch February 27, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant