Skip to content

Separate MCP connection settings - #120

Merged
insighta-cloud merged 1 commit into
mainfrom
feat/mcp-settings-section
Sep 19, 2026
Merged

insighta-cloud merged 1 commit into
mainfrom
feat/mcp-settings-section

Conversation

@forthfate

Copy link
Copy Markdown
Owner

Summary:

  • move MCP configuration into its own settings section with tooltip and summary
  • show registered server status and names before opening the JSON editor
  • edit JSON only in a modal, using the standard Save label
  • document reusable settings-section design rules in specs/product/settings-section-patterns.md

Validation:

  • pnpm --filter agent-improvement-console-ui run build
  • Playwright check of settings summary and edit modal

@forthfate
forthfate force-pushed the feat/mcp-settings-section branch 8 times, most recently from 3e635e6 to a54a8ad Compare September 19, 2026 02:48
@forthfate
forthfate force-pushed the feat/mcp-settings-section branch from a54a8ad to 87389f3 Compare September 19, 2026 03:10

@insighta-cloud insighta-cloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found. Approved!

@insighta-cloud
insighta-cloud merged commit 347f54f into main Sep 19, 2026
2 checks passed
@insighta-cloud
insighta-cloud deleted the feat/mcp-settings-section branch September 19, 2026 03:22
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