fix: improve HTTP server UI - show Login instead of Restart, use Reconnect in Actions (#184)

- Hide Restart button for HTTP servers on server cards (HTTP servers don't
  have a process to restart - use Login for OAuth or Reconnect instead)
- Show "Reconnect" instead of "Restart" in Actions menu for HTTP servers
- Fix Login button not appearing for HTTP servers with oauth: null config
  (HTTP servers always support OAuth via autodiscovery)
- Refactor isHttpProtocol to shared computed property in both components

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: Dumbris

frontend/src/components/ServerCard.vue

@@ -77,8 +77,9 @@
           Unquarantine
         </button>
 
+        <!-- Restart button only for stdio servers (HTTP servers use Login/Reconnect) -->
         <button
-          v-if="!server.connected && server.enabled"
+          v-if="!server.connected && server.enabled && !isHttpProtocol"
           @click="restart"
           :disabled="loading"
           class="btn btn-sm btn-outline"
@@ -173,22 +174,27 @@ const systemStore = useSystemStore()
 const loading = ref(false)
 const showDeleteConfirmation = ref(false)
 
+const isHttpProtocol = computed(() => {
+  return props.server.protocol === 'http' || props.server.protocol === 'streamable-http'
+})
+
 const needsOAuth = computed(() => {
   // Don't show Login button if server is disabled
   if (!props.server.enabled) return false
 
   // Don't show Login button while connecting (let the connection attempt finish)
   if (props.server.connecting) return false
 
-  const isHttpProtocol = props.server.protocol === 'http' || props.server.protocol === 'streamable-http'
-  if (!isHttpProtocol) return false
+  if (!isHttpProtocol.value) return false
 
   // Don't show Login if already connected
   if (props.server.connected) return false
 
-  // Check if server has OAuth configuration
-  const hasOAuthConfig = props.server.oauth !== null && props.server.oauth !== undefined
-  if (!hasOAuthConfig) return false
+  // HTTP servers always support OAuth via autodiscovery, even without explicit oauth config
+  // oauth: null (new server) and oauth: {} (explicit empty config) should both allow Login
+  // Only require explicit oauth config for non-HTTP protocols (though they don't reach here)
+  const hasOAuthSupport = isHttpProtocol.value || (props.server.oauth !== null && props.server.oauth !== undefined)
+  if (!hasOAuthSupport) return false
 
   // Show Login if user explicitly logged out
   if (props.server.user_logged_out) {
@@ -227,8 +233,7 @@ const canLogout = computed(() => {
   // Don't show Logout if user already explicitly logged out
   if (props.server.user_logged_out) return false
 
-  const isHttpProtocol = props.server.protocol === 'http' || props.server.protocol === 'streamable-http'
-  if (!isHttpProtocol) return false
+  if (!isHttpProtocol.value) return false
 
   const hasToken = props.server.authenticated === true
   if (!hasToken) return false

frontend/src/views/ServerDetail.vue

@@ -77,7 +77,7 @@
               <li v-if="server.enabled">
                 <button @click="restartServer" :disabled="actionLoading">
                   <span v-if="actionLoading" class="loading loading-spinner loading-xs"></span>
-                  Restart
+                  {{ isHttpProtocol ? 'Reconnect' : 'Restart' }}
                 </button>
               </li>
               <li v-if="needsOAuth">
@@ -460,9 +460,13 @@ const logsError = ref<string | null>(null)
 const logTail = ref(100)
 
 // Computed
+const isHttpProtocol = computed(() => {
+  return server.value?.protocol === 'http' || server.value?.protocol === 'streamable-http'
+})
+
 const needsOAuth = computed(() => {
   return server.value &&
-         (server.value.protocol === 'http' || server.value.protocol === 'streamable-http') &&
+         isHttpProtocol.value &&
          !server.value.connected &&
          server.value.enabled &&
          server.value.last_error?.includes('authorization')