Skip to content

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

Merged
Dumbris merged 1 commit into
mainfrom
fix/http-server-ui-improvements
Dec 8, 2025
Merged

fix: improve HTTP server UI - show Login instead of Restart, use Reconnect in Actions#184
Dumbris merged 1 commit into
mainfrom
fix/http-server-ui-improvements

Conversation

@Dumbris

@Dumbris Dumbris commented Dec 8, 2025

Copy link
Copy Markdown
Member

Summary

  • Hide Restart button for HTTP servers on server cards (HTTP servers don't have a process to restart)
  • Show "Reconnect" instead of "Restart" in Actions menu on server detail page for HTTP servers
  • Fix Login button not appearing for HTTP servers added via Web UI with oauth: null config

Changes

ServerCard.vue

  • Added isHttpProtocol computed property
  • Restart button now only shows for stdio servers (v-if="!isHttpProtocol")
  • HTTP servers show Login button for OAuth authentication instead

ServerDetail.vue

  • Added isHttpProtocol computed property
  • Actions menu shows "Reconnect" for HTTP servers, "Restart" for stdio servers

UX Rationale

  • HTTP servers: "Restart" doesn't make sense since there's no local process to restart. Users should use "Reconnect" or "Login" (for OAuth) instead
  • stdio servers: "Restart" makes sense since it kills and restarts the local command process

Test plan

  • Add HTTP server via Web UI, verify Login button appears (not Restart)
  • Open HTTP server detail page, verify Actions menu shows "Reconnect"
  • Add stdio server, verify Restart button appears on card
  • Open stdio server detail page, verify Actions menu shows "Restart"

🤖 Generated with Claude Code

…nnect in Actions

- 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>
@github-actions

github-actions Bot commented Dec 8, 2025

Copy link
Copy Markdown

📦 Build Artifacts

Workflow Run: View Run
Branch: fix/http-server-ui-improvements

Available Artifacts

  • archive-darwin-amd64 (23 MB)
  • archive-darwin-arm64 (20 MB)
  • archive-linux-amd64 (12 MB)
  • archive-linux-arm64 (11 MB)
  • archive-windows-amd64 (22 MB)
  • archive-windows-arm64 (20 MB)
  • frontend-dist-pr (0 MB)
  • installer-dmg-darwin-amd64 (25 MB)
  • installer-dmg-darwin-arm64 (23 MB)

How to Download

Option 1: GitHub Web UI (easiest)

  1. Go to the workflow run page linked above
  2. Scroll to the bottom "Artifacts" section
  3. Click on the artifact you want to download

Option 2: GitHub CLI

gh run download 20028915827 --repo smart-mcp-proxy/mcpproxy-go

Note: Artifacts expire in 14 days.

@Dumbris Dumbris merged commit 5df6310 into main Dec 8, 2025
23 checks passed
@Dumbris Dumbris deleted the fix/http-server-ui-improvements branch December 8, 2025 13:07
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