Add attributed MCP install buttons across SharePoint Embedded articles - #10953
Add attributed MCP install buttons across SharePoint Embedded articles#10953Gregory Joseph (gnjoseph) wants to merge 8 commits into
Conversation
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Adds a docs-to-devtools install-buttons include to the SharePoint Embedded MCP server Learn article (Install and configure section): a native 'Install in Visual Studio Code' next-step-action button plus one-click install links for VS Code Insiders, Visual Studio, and Cursor, and the claude/codex 'mcp add' CLI commands. Scoped to the MCP server page only for now. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
7eb3d37 to
eafc4a5
Compare
Gregory Joseph (gnjoseph)
changed the title
|
Learn Build status updates of commit eafc4a5: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Add Microsoft Learn source and campaign arguments to every supported install path, and surface the reusable install include on high-intent SharePoint Embedded build articles. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Gregory Joseph (gnjoseph)
changed the title
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 5dfce54: ✅ Validation status: passed
For more details, please refer to the build report. |
Keep ms.date unchanged while adding the reusable MCP install include. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Addressed the review follow-ups:
All decoded configurations contain |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 446e83d: ✅ Validation status: passed
For more details, please refer to the build report. |
Expand MCP on first use, use direct active-voice instructions, and simplify the reusable install text after reviewing the PR with the internal SPE documentation skills. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Internal
|
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 41e7a9c: ✅ Validation status: passed
For more details, please refer to the build report. |
Replace long editor-specific install URLs with validated, team-owned aka.ms aliases while preserving attributed Claude and Codex command examples. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Companion PR: microsoft/SharePoint-Embedded-MCP-Server#86. Eight team-owned |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit dff552a: ✅ Validation status: passed
For more details, please refer to the build report. |
Keep the reusable one-click links compact and move Claude Code and Codex CLI commands into a dedicated, copyable console section on the MCP article. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 194d363: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Diego’s UX feedback was addressed in commit |
List alternative clients as scannable links and keep Claude Code and Codex CLI commands free of visible attribution parameters. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit 1c7bded: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Verified the final client setup UX for head |
Give each SharePoint Embedded Learn placement its own bounded content label while retaining the Microsoft Learn source and docs install campaign. Attribute the Claude Code and Codex setup commands to the MCP server article. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit cd72c16: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Validation evidenceRetested the attributed install links and the Microsoft Learn preview for this PR. Automated checks
Microsoft Learn preview in EdgeAll eight changed files passed OpenPublishing validation and generated preview links:
The
Selecting Install in Visual Studio Code reaches the expected external-application handoff:
|
Validation evidenceRetested the attributed install links and the Microsoft Learn preview for this PR. Automated checks
Microsoft Learn preview in EdgeAll eight changed files passed OpenPublishing validation and generated preview links:
The
Selecting Install in Visual Studio Code reaches the expected external-application handoff:
|
Diego Luces (dluces)
left a comment
Diego Luces (dluces)
left a comment
There was a problem hiding this comment.
Wish you could pass parameters into the included blocks…
2 participants
Summary
Adds docs-to-devtools install buttons for the SharePoint Embedded MCP server across high-intent SharePoint Embedded build articles.
Every install path supplies bounded attribution arguments:
Each article uses its own published
contentvalue:sharepoint-embedded-mcp-serverquickstart-vscodecreate-container-typecreate-manage-containersThe MCP client stores these arguments in its server configuration and supplies them on each launch. The server adds the bounded labels to existing Graph and ARM request metadata; it does not open a separate telemetry channel.
Existing
ms.datevalues are intentionally unchanged.Install targets
Implementation status
@microsoft/spe-mcp@0.2.0-alpha.4latest: resolves to0.2.0-alpha.4Validation
--install-source,--install-content,--install-campaign, and--no-install-attribution.@microsoft/spe-mcp@latestto0.2.0-alpha.4.