Skip to content

[MCP] Remove MCP feature flag check (EnableMcpAccess)#7403

Open
onbuyuka wants to merge 2 commits intomainfrom
bugs/626506-remove-mcp-feature-flag
Open

[MCP] Remove MCP feature flag check (EnableMcpAccess)#7403
onbuyuka wants to merge 2 commits intomainfrom
bugs/626506-remove-mcp-feature-flag

Conversation

@onbuyuka
Copy link
Contributor

@onbuyuka onbuyuka commented Mar 26, 2026

Summary

  • Remove the EnableMcpAccess feature flag check that blocked the MCP Config List page when the feature key was not enabled
  • Remove the IsFeatureEnabled() procedure from MCPConfigImplementation
  • Remove the Feature Key dependency from the MCP app since it is no longer referenced

Fixes AB#626506

🤖 Generated with Claude Code

The EnableMcpAccess feature key is no longer used. Remove the
IsFeatureEnabled() procedure, the OnOpenPage guard, associated labels,
and the Feature Key dependency from app.json.

Fixes AB#626506

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@onbuyuka onbuyuka requested review from a team as code owners March 26, 2026 08:38
@onbuyuka onbuyuka enabled auto-merge (squash) March 26, 2026 08:39
@onbuyuka onbuyuka changed the title Remove MCP feature flag check (EnableMcpAccess) [MCP] Remove MCP feature flag check (EnableMcpAccess) Mar 26, 2026
@github-actions github-actions bot added this to the Version 29.0 milestone Mar 26, 2026
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

AL Documentation Audit

Documentation gaps were detected in the following apps:

  • MCP: 0% documentation coverage

To generate documentation, run /al-docs init or /al-docs update using GitHub Copilot CLI or Claude Code.
This review is for awareness to help keep documentation in sync with code changes. It is okay to dismiss this request.

Fixes AL0792 compiler errors from the feature flag removal.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant