Skip to content

[MCP] Loosen API page restriction beyond APIV2#7726

Merged
onbuyuka merged 3 commits into
mainfrom
private/onbuyuka/mcp-loosen-api-restriction
Apr 24, 2026
Merged

[MCP] Loosen API page restriction beyond APIV2#7726
onbuyuka merged 3 commits into
mainfrom
private/onbuyuka/mcp-loosen-api-restriction

Conversation

@onbuyuka
Copy link
Copy Markdown
Contributor

@onbuyuka onbuyuka commented Apr 15, 2026

Fixes AB#631961

Summary

  • Replace blanket APIPublisher='microsoft' block with two simple rules: block Microsoft.API.V1 namespace and APIVersion = 'beta' pages
  • Opens up extension APIs (EDocument, Subscription Billing, Quality Management, PowerBI Reports, Intercompany, Cloud Migration, Automation APIs, etc.) for MCP tool configuration
  • ValidateAPIPublisher bypass preserved for internal code paths

Blocked

  • V1 namespace: APIV1 base (49 pages) + Sustainability (7 pages)
  • Beta version: API Reports Finance (9), ExpenseAgent (39), ALCodeCoverage (5)

Allowed (new)

  • BCApps: EDocument, EDocument Connectors, Subscription Billing, Quality Management, PowerBI Reports, Performance Toolkit, AI Test Toolkit
  • NAV: IRS Forms, VAT Group, Intercompany, Cloud Migration, Agent toolkit, Dataverse, Automate, Runtime, Admin, Automation (APIV2 + BaseApp), PowerBI Labels

Changes

  • MCPConfigImplementation: ValidateAPITool, LookupAPITools, GetAPIPublishers, AddToolsByAPIGroup updated to filter by V1 namespace + beta version
  • Error label renamed InvalidAPIVersionErrAPIToolNotSupportedErr

Follow-up

AboutText additions on extension API pages will be done separately.

🤖 Generated with Claude Code

Replace the blanket APIPublisher='microsoft' block with a targeted
blocklist of publisher+group combinations using FilterGroup(-1) for
OR logic. This opens up extension APIs (EDocument, Subscription Billing,
Quality Management, PowerBI Reports, API Reports Finance, IRS Forms,
VAT Group) while keeping infrastructure, internal, and legacy V1 APIs
blocked.

Also adds AboutText descriptions to all newly available extension API
pages following the APIV2 pattern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

onbuyuka and others added 2 commits April 21, 2026 14:13
Replaces the publisher+group blocklist with simpler two-rule filtering:
- Block pages in Microsoft.API.V1 namespace
- Block pages with APIVersion = 'beta'

All other API pages (including microsoft-published extension APIs,
automation APIs, etc.) now pass through. FilterGroup(-1) is no longer
needed since there's no cross-field conditional.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Removes AboutText from extension API pages. AboutText will be added
separately via a dedicated pass over APIV2 pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot removed the AL: Apps (W1) Add-on apps for W1 label Apr 21, 2026
@onbuyuka onbuyuka marked this pull request as ready for review April 21, 2026 13:09
@onbuyuka onbuyuka requested a review from a team as a code owner April 21, 2026 13:09
@onbuyuka onbuyuka changed the title [WIP][MCP] Loosen API page restriction beyond APIV2 [MCP] Loosen API page restriction beyond APIV2 Apr 21, 2026
Copy link
Copy Markdown
Contributor

@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.

@github-actions github-actions Bot added this to the Version 29.0 milestone Apr 21, 2026
@onbuyuka onbuyuka enabled auto-merge (squash) April 21, 2026 13:18
@onbuyuka onbuyuka merged commit 1b05b19 into main Apr 24, 2026
54 of 57 checks passed
@onbuyuka onbuyuka deleted the private/onbuyuka/mcp-loosen-api-restriction branch April 24, 2026 13:46
onbuyuka added a commit that referenced this pull request May 21, 2026
Fixes
[AB#631961](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/631961)

- Replace blanket `APIPublisher='microsoft'` block with two simple
rules: block `Microsoft.API.V1` namespace and `APIVersion = 'beta'`
pages
- Opens up extension APIs (EDocument, Subscription Billing, Quality
Management, PowerBI Reports, Intercompany, Cloud Migration, Automation
APIs, etc.) for MCP tool configuration
- `ValidateAPIPublisher` bypass preserved for internal code paths

- **V1 namespace**: APIV1 base (49 pages) + Sustainability (7 pages)
- **Beta version**: API Reports Finance (9), ExpenseAgent (39),
ALCodeCoverage (5)

- BCApps: EDocument, EDocument Connectors, Subscription Billing, Quality
Management, PowerBI Reports, Performance Toolkit, AI Test Toolkit
- NAV: IRS Forms, VAT Group, Intercompany, Cloud Migration, Agent
toolkit, Dataverse, Automate, Runtime, Admin, Automation (APIV2 +
BaseApp), PowerBI Labels

- **MCPConfigImplementation**: `ValidateAPITool`, `LookupAPITools`,
`GetAPIPublishers`, `AddToolsByAPIGroup` updated to filter by V1
namespace + beta version
- Error label renamed `InvalidAPIVersionErr` → `APIToolNotSupportedErr`

`AboutText` additions on extension API pages will be done separately.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

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.

3 participants