Skip to content

Mcp ux improvement #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 53 commits into
base: og_mcp_aider_pr_quin
Choose a base branch
from
Open

Mcp ux improvement #2

wants to merge 53 commits into from

Conversation

arosov
Copy link
Owner

@arosov arosov commented May 28, 2025

based on Aider-AI#3937

Opened this mostly for review purposes, to be able to distinguish my changes from the original PR.

Since the original PR is maintained for now, some commits from aider's main branch made it through.

paul-gauthier and others added 30 commits May 25, 2025 16:00
fix: Include mcpservers-file in known server names

fix: Ensure 'all' MCP profile reflects current server configs

feat: Support dict format for MCP server config

fix: Fix 'all' profile update logic on startup

Fix keys for mcp config

refactor: Extract MCP server names from 'mcpServers' key
refactor: Remove redundant MCP tool preparation message

colors
fix: Parse Rich markup in tool output strings
…m' option

refactor: Create constants for MCP profile YAML keys

fix: Update cmd_mcp to display server names from list of dicts
refactor: Encapsulate server no_confirm check in profile manager
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.

2 participants