Skip to content

Conversation

@onbuyuka
Copy link
Contributor

@onbuyuka onbuyuka commented Dec 16, 2025

Summary

This PR enhances the MCP (Model Context Protocol) configuration management with validation capabilities, system tools display, and improved user experience in the configuration card.

Changes

Configuration Validation Framework

  • Added ValidateConfiguration procedure to detect configuration issues before activation
  • Introduced extensible warning system using interface pattern:
    • Interface: MCP Config Warning - defines WarningMessage, RecommendedAction, and ApplyRecommendedAction procedures
    • Enum: MCP Config Warning Type - implements the interface with warning types
    • Implementations: MCP Config Missing Object and MCP Config Missing Parent codeunits
  • New MCP Config Warning temporary table to store validation warnings
  • New MCP Config Warning List page to display and act on warnings
  • Validation automatically runs when activating a configuration with option to review warnings

System Tools Display

  • Added MCP System Tool temporary table to represent built-in MCP tools
  • Added MCP System Tool List page part to display system tools (bc_action_describe, bc_action_invoke, bc_action_search)
  • System tools are now visible in Dynamic Tool Mode on the configuration card

Configuration Card UX Improvements

  • Added Tool Mode description field explaining Static vs Dynamic tool modes
  • Added Validate action to manually validate configuration
  • Improved field editability:
    • Name and Description are now read-only when configuration is active
    • Tool list is read-only when configuration is active
    • Add Tools by API Group and Add All Standard APIs actions disabled when configuration is active
  • Removed sandbox-specific logic for create/update/delete permissions (now controlled by AllowProdChanges setting)

Work Item(s)

Fixes AB#615692

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants