Skip to content

feat: add Venice transcription provider#24

Closed
Da7-Tech wants to merge 1 commit into
nibzard:mainfrom
Da7-Tech:add-venice-transcription-provider
Closed

feat: add Venice transcription provider#24
Da7-Tech wants to merge 1 commit into
nibzard:mainfrom
Da7-Tech:add-venice-transcription-provider

Conversation

@Da7-Tech
Copy link
Copy Markdown

Summary

  • add a Venice transcription provider using the multipart audio transcription endpoint
  • wire --api venice into the CLI provider selection
  • document the required Venice environment variables and usage example
  • add focused unit coverage for the request payload and missing API key handling

Verification

  • PYTHONPATH=src python3 -m unittest tests.test_venice -v
  • python3 -m py_compile src/sapat/script.py src/sapat/transcription/venice.py tests/test_venice.py
  • git diff --check

No secrets are included. The provider reads credentials from VENICE_API_KEY.

@nibzard
Copy link
Copy Markdown
Owner

nibzard commented May 28, 2026

Thank you for this contribution! Your implementation was reviewed as part of a batch review of all incoming provider PRs.

We've decided to take a different approach: instead of merging individual provider PRs (which all targeted a different code architecture than what exists on main), we've refactored the entire codebase into a proper provider plugin architecture with auto-discovery and shared mixins. Your provider's API integration patterns were used as reference when building the Venice provider in the new architecture.

The refactor is on branch feat/provider-plugin-architecture and includes 29 providers, 176 tests, and a clean plugin system. We appreciate your contribution — it helped shape the final design. Closing this in favor of the unified approach.

@nibzard nibzard closed this May 28, 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.

3 participants