Improve quarantine UX: auto-approve, dashboard visibility, doctor stats, log noise#332
Merged
Improve quarantine UX: auto-approve, dashboard visibility, doctor stats, log noise#332
Conversation
…to servers API Two quarantine improvements: 1. Auto-approve tools from trusted servers: When a new tool is discovered and the server is not quarantined (skip_quarantine=true or global quarantine disabled), the tool is immediately auto-approved with ApprovedBy="auto" instead of being marked "pending". This prevents existing users from having all tools blocked on upgrade. Changed tools (rug pulls) are still blocked regardless of trust level. 2. Add quarantine stats to GET /api/v1/servers: Each server in the response now includes a "quarantine" field with pending_count and changed_count, enabling the frontend to show quarantine badges. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add QuarantineStats interface and quarantine field to Server type so the ServerCard displays how many tools are pending approval without requiring users to click into each server's detail page. The badge appears in the Tools stat area with a warning icon when pending_count or changed_count is non-zero. Also updates v-memo keys in Servers.vue so cards re-render when quarantine stats change. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add prominent quarantine tool notifications so users can quickly see which servers have tools pending approval, instead of discovering them only through server detail pages. Dashboard changes: - Add warning banner showing "X tools pending approval across Y servers" - Lists each server with pending count, links to server detail page - Fetches tool approvals for all enabled servers in parallel on load - Auto-refreshes every 30 seconds alongside other dashboard data CLI doctor changes: - Add "Tools Pending Quarantine Approval" section to doctor output - Queries each enabled server's tool approvals via the existing API - Shows per-server breakdown with new vs changed tool counts - Includes remediation hints for Web UI and CLI approval commands - Quarantine data included in JSON output format Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…level The "Server state changed" message was logging at INFO every ~3 seconds due to server connection state fluctuations, flooding the log file. Changed to DEBUG since this is routine polling status, not actionable information. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Deploying mcpproxy-docs with
|
| Latest commit: |
89e671a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://64c6a16e.mcpproxy-docs.pages.dev |
| Branch Preview URL: | https://quarantine-ux-improvements.mcpproxy-docs.pages.dev |
📦 Build ArtifactsWorkflow Run: View Run Available Artifacts
How to DownloadOption 1: GitHub Web UI (easiest)
Option 2: GitHub CLI gh run download 22905567639 --repo smart-mcp-proxy/mcpproxy-go
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
Summary
/api/v1/serversnow includesquarantinefield withpending_countandchanged_countper server.mcpproxy doctor: Now shows quarantine statistics per server when pending/changed tools are detected.Test plan
go test ./internal/runtime/... -run Quarantine— auto-approve tests passgo test ./cmd/mcpproxy/... -run Doctor— doctor quarantine tests passgo test ./internal/httpapi/...— API tests passmake build— frontend + backend build clean./scripts/test-api-e2e.sh— 60/71 pass (same as main, 11 pre-existing failures)mcpproxy doctorshows quarantine stats🤖 Generated with Claude Code