feat: implement enhanced authentication system for AI providers #943
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add feature flags for whisper-cuda and whisper-cpu in Cargo.toml - Create separate tauri.cuda.conf.json with Blinko(CUDA) product name - Implement conditional compilation for voice module based on features - Add is_cuda_available command for frontend CUDA detection - Update GitHub Actions workflows for matrix builds (cpu/cuda variants) - Add CUDA availability check in voice settings UI - Configure conditional voice module loading in lib.rs and setup.rs
Docs: update French translation of todo to Tâches instead of Procuration
- Add 250ms delay activation constraint for mouse and touch sensors - Consolidate all drag logic into dedicated useDragCard hook - Remove drag functionality from BlinkoCard component - Implement stable drag with placeholder and insertion line - Add multi-language support for dragging state - Fix text selection conflicts with long-press approach - Improve masonry layout stability during drag operations
Add comprehensive authentication support for custom AI providers, addressing limitations with the previous API key-only system. ### Key Features: - Multiple authentication types: Bearer token, API key, custom headers, no auth - Flexible header configuration for complex authentication scenarios - Backward compatibility with existing provider configurations - Enhanced UI with example configurations for common providers - Support for Zhipu AI, Moonshot AI, and other custom providers ### Changes: #### Backend: - `server/aiServer/authTypes.ts`: Authentication types and utilities - `server/aiServer/providers/EmbeddingProvider.ts`: Enhanced with custom auth - `server/aiServer/providers/LLMProvider.ts`: Enhanced with custom auth - `server/routerTrpc/ai.ts`: Updated test connection and API endpoints #### Frontend: - `app/src/components/BlinkoSettings/AiSetting/AuthenticationConfig.tsx`: New auth UI component - `app/src/components/BlinkoSettings/AiSetting/ProviderDialogContent.tsx`: Enhanced provider dialog - `app/src/store/aiSettingStore.tsx`: Enhanced store with auth support - `app/public/locales/en/translation.json`: Updated translations #### Documentation: - `docs/ENHANCED_AUTHENTICATION.md`: Comprehensive usage guide ### Technical Details: - Authentication config stored in provider.config.authConfig JSON field - Custom fetch functions inject authentication headers - Graceful fallback to legacy authentication for existing providers - Input validation and error handling for security 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
dosubot
bot
added
size:XXL
revaxe
reviewed
Oct 19, 2025
| updateConfig({ customHeaders: newHeaders }); | ||
| }; | ||
|
|
||
| const renderBearerConfig = () => ( |
Contributor
There was a problem hiding this comment.
I learned that rendering by function could create problems when rerendering.
Instead, the best practice is to create a React component.
Simply like this:
Suggested change
| const renderBearerConfig = () => ( | |
| const BearerConfig = <div... |
To then use it with:
{localConfig.type === 'bearer' && <BearerConfig/>}
revaxe
reviewed
Oct 19, 2025
| <Icon icon="hugeicons:settings-03" width="16" height="16" /> | ||
| <div> | ||
| <div className="font-medium">Custom Headers</div> | ||
| <div className="text-xs text-default-500">Full control over headers</div> |
Contributor
There was a problem hiding this comment.
Translation should certainly be provided for these descriptions and labels
revaxe
reviewed
Oct 19, 2025
| </div> | ||
| ))} | ||
|
|
||
| <div className="flex items-center gap-2"> |
Contributor
There was a problem hiding this comment.
I would have put a <form here, and passed the button to submit type
Suggested change
| <div className="flex items-center gap-2"> | |
| <form onSubmit={addCustomHeader} className="flex items-center gap-2"> |
revaxe
reviewed
Oct 19, 2025
Comment on lines
+28
to
+34
| const baseURL = config.baseURL; | ||
|
|
||
| switch (config.provider.toLowerCase()) { | ||
| case 'openai': | ||
| return createOpenAI({ | ||
| apiKey: config.apiKey, | ||
| baseURL: config.baseURL || undefined, | ||
| apiKey: apiKey, | ||
| baseURL: baseURL || undefined, |
Contributor
There was a problem hiding this comment.
I would also have added || undefined in the const.
const baseURL = config.baseURL || undefined;
switch (config.provider.toLowerCase()) {
case 'openai':
return createOpenAI({
apiKey,
baseURL,
Contributor
|
That's cool what you're suggesting, it will answer my request for improvement : #818 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add comprehensive authentication support for custom AI providers, addressing limitations with the previous API key-only system.
Key Features:
Changes:
Backend:
server/aiServer/authTypes.ts: Authentication types and utilitiesserver/aiServer/providers/EmbeddingProvider.ts: Enhanced with custom authserver/aiServer/providers/LLMProvider.ts: Enhanced with custom authserver/routerTrpc/ai.ts: Updated test connection and API endpointsFrontend:
app/src/components/BlinkoSettings/AiSetting/AuthenticationConfig.tsx: New auth UI componentapp/src/components/BlinkoSettings/AiSetting/ProviderDialogContent.tsx: Enhanced provider dialogapp/src/store/aiSettingStore.tsx: Enhanced store with auth supportapp/public/locales/en/translation.json: Updated translationsDocumentation:
docs/ENHANCED_AUTHENTICATION.md: Comprehensive usage guideTechnical Details:
🤖 Generated with Claude Code