Skip to content

feat(ai): konsole mcp#2357

Merged
ehayes2000 merged 1 commit intomainfrom
mcp-k
Apr 2, 2026
Merged

feat(ai): konsole mcp#2357
ehayes2000 merged 1 commit intomainfrom
mcp-k

Conversation

@ehayes2000
Copy link
Copy Markdown
Contributor

No description provided.

@ehayes2000 ehayes2000 requested a review from a team as a code owner April 2, 2026 21:49
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Summary by CodeRabbit

Release Notes

  • New Features

    • Introduced a global keyboard shortcut for quick access to MCP configuration setup
  • Improvements

    • Enhanced command search to index and include command tags, improving discoverability
  • Style

    • Updated empty state component styling for improved visual consistency

Walkthrough

The PR introduces a new MCP setup modal component with a global hotkey to access it, integrates it into the authenticated layout, enhances command search indexing with tags, and refactors AI chat empty state styling from macro-specific to generic utility classes.

Changes

Cohort / File(s) Summary
Global Hotkey & Modal Registration
js/app/packages/app/component/GlobalHotkeys.tsx, js/app/packages/app/component/Layout.tsx
Registered a new global hotkey (scopeId: 'global') that opens the MCP setup modal, with gear icon and MCP-related tags. Renders MacroMcpSetupModal in the authenticated layout section alongside other global modals.
MCP Setup Modal Component
js/app/packages/app/component/macro-mcp-setup-modal/MacroMcpSetupModal.tsx
New controlled modal component using SolidJS that manages open/close state via createControlledOpenSignal. Exports openMacroMcpSetupModal(), closeMacroMcpSetupModal(), and the MacroMcpSetupModal() component. Renders a dialog with fixed 760px width containing AiChatEmptyState.
Command Search Enhancement
js/app/packages/app/component/command/useCommandItems.ts
Enhanced search indexing by concatenating command tags into the searchText alongside the description, improving searchability for command menu items.
AI Chat Empty State Refactoring
js/app/packages/core/component/AI/component/AIChatEmptyState.tsx, js/app/packages/core/component/AI/component/message/EmptyChatState.tsx
Simplified AiChatEmptyState styling by removing macro-specific classes and replacing with generic utilities. Removed AiChatEmptyState usage from EmptyChatState in favor of a simpler standalone container.

Possibly related PRs

  • macro#2344: Implements the MCP empty-state UI directly in AiChatEmptyState and EmptyChatState, which directly overlaps with the styling refactoring performed in this PR.
🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset. Add a pull request description explaining the purpose, implementation details, and scope of the MCP setup modal feature being introduced.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title follows conventional commits format with 'feat(ai):' prefix and is under 72 characters, clearly describing the main feature addition of MCP setup modal integration.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

@ehayes2000 ehayes2000 merged commit df0f7fe into main Apr 2, 2026
22 of 23 checks passed
@ehayes2000 ehayes2000 deleted the mcp-k branch April 2, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant