From 2cb77f2dfd8376905e76cdd951830d24f2d29bbf Mon Sep 17 00:00:00 2001
From: kaitranntt <kaitran.ntt@gmail.com>
Date: Thu, 22 Jan 2026 14:24:37 -0500
Subject: [PATCH] feat(ui): add Ollama logo and make API key optional

- Add Ollama SVG logo to ui/public/icons/
- Add icon property to both ollama and ollama-cloud presets
- Make API key field optional for local Ollama (show input with hint)
- Update placeholder: "Optional - only if auth is enabled"
- Allows power users with Ollama auth configured to provide key
---
 ui/public/icons/ollama.svg                    |  1 +
 .../profiles/profile-create-dialog.tsx        | 91 ++++++++++---------
 ui/src/lib/provider-presets.ts                |  2 +
 3 files changed, 49 insertions(+), 45 deletions(-)
 create mode 100644 ui/public/icons/ollama.svg

diff --git a/ui/public/icons/ollama.svg b/ui/public/icons/ollama.svg
new file mode 100644
index 00000000..cc887e3d
--- /dev/null
+++ b/ui/public/icons/ollama.svg
@@ -0,0 +1 @@
+<svg fill="currentColor" fill-rule="evenodd" height="1em" style="flex:none;line-height:1" viewBox="0 0 24 24" width="1em" xmlns="http://www.w3.org/2000/svg"><title>Ollama</title><path d="M7.905 1.09c.216.085.411.225.588.41.295.306.544.744.734 1.263.191.522.315 1.1.362 1.68a5.054 5.054 0 012.049-.636l.051-.004c.87-.07 1.73.087 2.48.474.101.053.2.11.297.17.05-.569.172-1.134.36-1.644.19-.52.439-.957.733-1.264a1.67 1.67 0 01.589-.41c.257-.1.53-.118.796-.042.401.114.745.368 1.016.737.248.337.434.769.561 1.287.23.934.27 2.163.115 3.645l.053.04.026.019c.757.576 1.284 1.397 1.563 2.35.435 1.487.216 3.155-.534 4.088l-.018.021.002.003c.417.762.67 1.567.724 2.4l.002.03c.064 1.065-.2 2.137-.814 3.19l-.007.01.01.024c.472 1.157.62 2.322.438 3.486l-.006.039a.651.651 0 01-.747.536.648.648 0 01-.54-.742c.167-1.033.01-2.069-.48-3.123a.643.643 0 01.04-.617l.004-.006c.604-.924.854-1.83.8-2.72-.046-.779-.325-1.544-.8-2.273a.644.644 0 01.18-.886l.009-.006c.243-.159.467-.565.58-1.12a4.229 4.229 0 00-.095-1.974c-.205-.7-.58-1.284-1.105-1.683-.595-.454-1.383-.673-2.38-.61a.653.653 0 01-.632-.371c-.314-.665-.772-1.141-1.343-1.436a3.288 3.288 0 00-1.772-.332c-1.245.099-2.343.801-2.67 1.686a.652.652 0 01-.61.425c-1.067.002-1.893.252-2.497.703-.522.39-.878.935-1.066 1.588a4.07 4.07 0 00-.068 1.886c.112.558.331 1.02.582 1.269l.008.007c.212.207.257.53.109.785-.36.622-.629 1.549-.673 2.44-.05 1.018.186 1.902.719 2.536l.016.019a.643.643 0 01.095.69c-.576 1.236-.753 2.252-.562 3.052a.652.652 0 01-1.269.298c-.243-1.018-.078-2.184.473-3.498l.014-.035-.008-.012a4.339 4.339 0 01-.598-1.309l-.005-.019a5.764 5.764 0 01-.177-1.785c.044-.91.278-1.842.622-2.59l.012-.026-.002-.002c-.293-.418-.51-.953-.63-1.545l-.005-.024a5.352 5.352 0 01.093-2.49c.262-.915.777-1.701 1.536-2.269.06-.045.123-.09.186-.132-.159-1.493-.119-2.73.112-3.67.127-.518.314-.95.562-1.287.27-.368.614-.622 1.015-.737.266-.076.54-.059.797.042zm4.116 9.09c.936 0 1.8.313 2.446.855.63.527 1.005 1.235 1.005 1.94 0 .888-.406 1.58-1.133 2.022-.62.375-1.451.557-2.403.557-1.009 0-1.871-.259-2.493-.734-.617-.47-.963-1.13-.963-1.845 0-.707.398-1.417 1.056-1.946.668-.537 1.55-.849 2.485-.849zm0 .896a3.07 3.07 0 00-1.916.65c-.461.37-.722.835-.722 1.25 0 .428.21.829.61 1.134.455.347 1.124.548 1.943.548.799 0 1.473-.147 1.932-.426.463-.28.7-.686.7-1.257 0-.423-.246-.89-.683-1.256-.484-.405-1.14-.643-1.864-.643zm.662 1.21l.004.004c.12.151.095.37-.056.49l-.292.23v.446a.375.375 0 01-.376.373.375.375 0 01-.376-.373v-.46l-.271-.218a.347.347 0 01-.052-.49.353.353 0 01.494-.051l.215.172.22-.174a.353.353 0 01.49.051zm-5.04-1.919c.478 0 .867.39.867.871a.87.87 0 01-.868.871.87.87 0 01-.867-.87.87.87 0 01.867-.872zm8.706 0c.48 0 .868.39.868.871a.87.87 0 01-.868.871.87.87 0 01-.867-.87.87.87 0 01.867-.872zM7.44 2.3l-.003.002a.659.659 0 00-.285.238l-.005.006c-.138.189-.258.467-.348.832-.17.692-.216 1.631-.124 2.782.43-.128.899-.208 1.404-.237l.01-.001.019-.034c.046-.082.095-.161.148-.239.123-.771.022-1.692-.253-2.444-.134-.364-.297-.65-.453-.813a.628.628 0 00-.107-.09L7.44 2.3zm9.174.04l-.002.001a.628.628 0 00-.107.09c-.156.163-.32.45-.453.814-.29.794-.387 1.776-.23 2.572l.058.097.008.014h.03a5.184 5.184 0 011.466.212c.086-1.124.038-2.043-.128-2.722-.09-.365-.21-.643-.349-.832l-.004-.006a.659.659 0 00-.285-.239h-.004z"></path></svg>
\ No newline at end of file
diff --git a/ui/src/components/profiles/profile-create-dialog.tsx b/ui/src/components/profiles/profile-create-dialog.tsx
index 0d7fff12..2c3d283a 100644
--- a/ui/src/components/profiles/profile-create-dialog.tsx
+++ b/ui/src/components/profiles/profile-create-dialog.tsx
@@ -215,8 +215,8 @@ export function ProfileCreateDialog({
     // Use user-provided baseUrl (allows customization of preset URLs)
     const finalData = {
       ...data,
-      // For presets that don't require API key, use empty string
-      apiKey: currentPreset?.requiresApiKey === false ? '' : data.apiKey,
+      // Use provided API key, or empty string if not provided (for optional auth providers)
+      apiKey: data.apiKey || '',
     };
     try {
       await createMutation.mutateAsync(finalData);
@@ -374,51 +374,52 @@ export function ProfileCreateDialog({
                   )}
                 </div>
 
-                {/* API Key - hidden for presets that don't require it */}
-                {currentPreset?.requiresApiKey !== false ? (
-                  <div className="space-y-1.5">
-                    <Label htmlFor="apiKey">
-                      API Key <span className="text-destructive">*</span>
-                    </Label>
-                    <div className="relative">
-                      <Input
-                        id="apiKey"
-                        type={showApiKey ? 'text' : 'password'}
-                        {...register('apiKey')}
-                        placeholder={currentPreset?.apiKeyPlaceholder ?? 'sk-...'}
-                        className="pr-10"
-                      />
-                      <Button
-                        type="button"
-                        variant="ghost"
-                        size="icon"
-                        className="absolute right-0 top-0 h-9 w-9 text-muted-foreground hover:text-foreground"
-                        onClick={() => setShowApiKey(!showApiKey)}
-                        tabIndex={-1}
-                      >
-                        {showApiKey ? <EyeOff className="h-4 w-4" /> : <Eye className="h-4 w-4" />}
-                      </Button>
-                    </div>
-                    {errors.apiKey ? (
-                      <p className="text-xs text-destructive">{errors.apiKey.message}</p>
-                    ) : (
-                      currentPreset?.apiKeyHint && (
-                        <p className="text-xs text-muted-foreground">{currentPreset.apiKeyHint}</p>
-                      )
+                {/* API Key - optional for presets that don't require it */}
+                <div className="space-y-1.5">
+                  <Label htmlFor="apiKey">
+                    API Key{' '}
+                    {currentPreset?.requiresApiKey !== false && (
+                      <span className="text-destructive">*</span>
                     )}
+                    {currentPreset?.requiresApiKey === false && (
+                      <span className="text-muted-foreground text-xs font-normal">(optional)</span>
+                    )}
+                  </Label>
+                  <div className="relative">
+                    <Input
+                      id="apiKey"
+                      type={showApiKey ? 'text' : 'password'}
+                      {...register('apiKey')}
+                      placeholder={
+                        currentPreset?.requiresApiKey === false
+                          ? 'Optional - only if auth is enabled'
+                          : (currentPreset?.apiKeyPlaceholder ?? 'sk-...')
+                      }
+                      className="pr-10"
+                    />
+                    <Button
+                      type="button"
+                      variant="ghost"
+                      size="icon"
+                      className="absolute right-0 top-0 h-9 w-9 text-muted-foreground hover:text-foreground"
+                      onClick={() => setShowApiKey(!showApiKey)}
+                      tabIndex={-1}
+                    >
+                      {showApiKey ? <EyeOff className="h-4 w-4" /> : <Eye className="h-4 w-4" />}
+                    </Button>
                   </div>
-                ) : (
-                  <div className="flex items-start gap-3 p-3 bg-green-50 dark:bg-green-950/20 text-green-800 dark:text-green-300 rounded-md text-sm border border-green-100 dark:border-green-900/30">
-                    <Info className="w-5 h-5 shrink-0 mt-0.5" />
-                    <div>
-                      <p className="font-medium">No API Key Required</p>
-                      <p className="text-xs opacity-90">
-                        {currentPreset?.apiKeyHint ||
-                          'This provider runs locally and does not require authentication.'}
-                      </p>
-                    </div>
-                  </div>
-                )}
+                  {errors.apiKey ? (
+                    <p className="text-xs text-destructive">{errors.apiKey.message}</p>
+                  ) : currentPreset?.requiresApiKey === false ? (
+                    <p className="text-xs text-muted-foreground">
+                      Only needed if you have configured Ollama authentication
+                    </p>
+                  ) : (
+                    currentPreset?.apiKeyHint && (
+                      <p className="text-xs text-muted-foreground">{currentPreset.apiKeyHint}</p>
+                    )
+                  )}
+                </div>
               </TabsContent>
 
               <TabsContent value="models" className="p-6 mt-0 space-y-4">
diff --git a/ui/src/lib/provider-presets.ts b/ui/src/lib/provider-presets.ts
index 2cd68f18..4b900e4a 100644
--- a/ui/src/lib/provider-presets.ts
+++ b/ui/src/lib/provider-presets.ts
@@ -53,6 +53,7 @@ export const PROVIDER_PRESETS: ProviderPreset[] = [
     defaultProfileName: 'ollama',
     badge: 'Local',
     featured: true,
+    icon: '/icons/ollama.svg',
     defaultModel: 'qwen3-coder',
     requiresApiKey: false,
     apiKeyPlaceholder: '',
@@ -158,6 +159,7 @@ export const PROVIDER_PRESETS: ProviderPreset[] = [
     baseUrl: 'https://ollama.com',
     defaultProfileName: 'ollama-cloud',
     badge: 'Cloud',
+    icon: '/icons/ollama.svg',
     defaultModel: 'glm-4.7:cloud',
     requiresApiKey: true,
     apiKeyPlaceholder: 'YOUR_OLLAMA_CLOUD_API_KEY',