Repository navigation
Update Excel plugin listings - #4481
Conversation
…b1f6d890fa58d300abaee01f4)
🔒 PR Risk Scan ResultsScanned 2 changed file(s).
|
🚦 Submission status: 👀 Ready for reviewRisk tier: Why this tier
Automated checks
Review
Commands
Updated for |
✅ External plugin PR checks passed
Per-plugin quality summary
Gate output detailsexcel-cli - spec compliance (✅ pass)excel-cli - vally lint (✅ pass)excel-cli - install smoke test (✅ pass)excel-cli - version match (pass)excel-cli - ref/sha consistency (pass)excel-cli - canvas structure (not_run)excel-mcp - spec compliance (✅ pass)excel-mcp - vally lint (✅ pass)excel-mcp - install smoke test (✅ pass)excel-mcp - version match (pass)excel-mcp - ref/sha consistency (pass)excel-mcp - canvas structure (not_run) |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The CLI update removes its general automation skill, and the PR description understates the upstream removals.
Review effort: Balanced
Findings: 1
Open (2)
What changed in this PR
Updates the external Excel plugins from v2.0.1 to v2.2.0.
Changes:
- Updates descriptions, versions, refs, and immutable SHAs.
- Regenerates matching marketplace entries.
| File | Description |
|---|---|
plugins/external.json |
Updates Excel plugin listings. |
.github/plugin/marketplace.json |
Mirrors regenerated listings. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…64232df9c7935e131cd1433e5)
|
/request-review |
|
🙋 Added |
|
/rerun-checks |
|
🔁 Re-running: The status comment updates when the checks finish. |


Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
Updates the existing
excel-cliandexcel-mcplistings from 2.0.1 to 2.2.1, pinned to releasev2.2.1and commit8941e5d59a068e421ddbddf40107c6f505f5acacinsbroenne/mcp-server-excel-plugins. This supersedes the earlier 2.2.0 proposal on the same PR. Only the two plugins' entries inplugins/external.jsonand the generated.github/plugin/marketplace.jsonchange here; the referenced plugin release has broader changes described below.From custom download scripts to npm/npx: the old plugin scripts queried GitHub Releases, downloaded and verified executable archives, and maintained their own runtime cache. The new launch paths use published npm packages, with npm handling package resolution and caching.
excel-clinpx -y @sbroenne/excelcli@latest, throughbin/start-cli.ps1excel-mcpbin/start-mcp.ps1and the custom downloadernpx -y @sbroenne/mcp-server-excel@latestinmcp.jsonThe retained CLI wrapper preserves quoted JSON arguments under Windows PowerShell; it is a small npm launcher, not the retired downloader. Plugin installation does not put the bare
excelclicommand on PATH. Neither plugin requires the old global-install helper or a separate standalone executable installation.The CLI discovery finding is addressed in 2.2.1. The published plugin now includes a small general
skills/excel-cli/SKILL.mdfor ordinary workbook requests. It tells the agent where to find the bundled launcher, to use that same launcher for subsequent commands, and to discover exact syntax through native--help. It also distinguishes plugin-bundled paths from standalone skill installations, which must locate the actual installed plugin. This fixes the missing launcher instructions without restoring the former command/reference catalog. The source fix was merged in sbroenne/mcp-server-excel#1049; the existing 2.2.0 tag was not rewritten.Removed or narrowed guidance:
plugins/excel-mcp/skills/excel-mcp/tree is removed, including itsSKILL.mdand bundled references. MCP tool schemas remain the runtime command contract.plugins/excel-mcp/skills/excel-mcp/references/calculation.mdis removed, not added. This corrects the original generated description.bin/download.ps1scripts and both global-install helpers are removed. MCP'sbin/start-mcp.ps1is also removed; the CLI npm wrapper remains.The guidance changes do not themselves remove Excel operations from the runtime. Requirements: Windows, installed desktop Microsoft Excel 2016 or later, and Node.js 18 or later with npm/npx on PATH. Users need their own Excel installation/license; the plugins are MIT-licensed. This is not a cross-platform or Excel-free implementation.
The plugin source is pinned to the reviewed release/commit. The runtime launchers intentionally use npm's
@latesttag, so the marketplace source SHA does not pin the runtime version.Reviewable sources:
Type of Contribution
Additional Notes
I maintain both plugins. The description was checked against the immutable published sources, including the new CLI discovery skill, rather than inferred from a changed-path list.
Validation for the corrected source/release:
@microsoft/vally0.12.0 structural validation passed for the three prepared skills, with no discovery errors.npm ci --ignore-scripts --no-audit --no-fund,npm run plugin:validate, andnpm run buildin a disposable, credential-free upstream checkout. The guarded writer updated this existing PR, and its final checks restrict the diff to the two listing files.Successful release/publication/update run. GitHub initially created the release draft but the immediate follow-up lookup failed; retrying only the failed job and its dependents succeeded without republishing the successful npm/NuGet/VS Code jobs.
Earlier installation and real-Excel E2E/smoke checks were performed against 2.2.0, not rerun against 2.2.1. No live natural-language Copilot evaluation of automatic skill selection was run. Structural tests establish that the launcher instructions are present and correctly packaged; they do not prove which skill a model will select.
Unchecked checklist items: this updates existing external-plugin listings rather than adding a new content file; no live Copilot selection session was run; and
npm startwas not invoked by that exact name. Upstream defines it asnpm run build, which the workflow passed, but the checkbox remains unchecked to record the actual command used.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.