Skip to content

Fix OpenAI transcription zero token usage#704

Closed
Gujiassh wants to merge 3 commits intocrmne:mainfrom
Gujiassh:fix/openai-transcription-zero-tokens
Closed

Fix OpenAI transcription zero token usage#704
Gujiassh wants to merge 3 commits intocrmne:mainfrom
Gujiassh:fix/openai-transcription-zero-tokens

Conversation

@Gujiassh
Copy link
Copy Markdown

Summary

  • preserve explicit zero-valued transcription token usage fields instead of falling back to prompt/completion token keys
  • add focused provider coverage for both the zero-valued and fallback usage paths

Testing

  • bundle exec rspec spec/ruby_llm/providers/open_ai/transcription_spec.rb
  • bundle exec rubocop lib/ruby_llm/providers/openai/transcription.rb spec/ruby_llm/providers/open_ai/transcription_spec.rb

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@Gujiassh Gujiassh force-pushed the fix/openai-transcription-zero-tokens branch from a052d4a to 239d816 Compare March 29, 2026 10:58
…ion-zero-tokens-clean

# Conflicts:
#	spec/ruby_llm/generators/chat_ui_generator_spec.rb
@Gujiassh
Copy link
Copy Markdown
Author

Gujiassh commented Apr 6, 2026

Refreshed this branch onto the latest main and reverified the merged branch tip locally.

Local verification after the refresh:

  • bundle exec rspec spec/ruby_llm/providers/open_ai/transcription_spec.rb spec/ruby_llm/generators/chat_ui_generator_spec.rb → passed (22 examples, 0 failures)
  • bundle exec rubocop lib/ruby_llm/providers/openai/transcription.rb spec/ruby_llm/providers/open_ai/transcription_spec.rb spec/ruby_llm/generators/chat_ui_generator_spec.rb → passed

The zero-token OpenAI transcription handling is still intact on the refreshed branch.

@Gujiassh
Copy link
Copy Markdown
Author

Closing this stale radar PR to keep the queue clean: it has been open for weeks without maintainer response, and I am not going to keep it in the active follow-up set without renewed upstream interest.

@Gujiassh Gujiassh closed this Apr 11, 2026
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