Skip to content

[WIP] Address feedback on MCP servers for dotnet-blazor plugin integration#723

Merged
javiercn merged 3 commits into
javiercn/add-mcp-servers-to-blazor-pluginfrom
copilot/wip-address-feedback-on-mcp-servers
Jun 4, 2026
Merged

[WIP] Address feedback on MCP servers for dotnet-blazor plugin integration#723
javiercn merged 3 commits into
javiercn/add-mcp-servers-to-blazor-pluginfrom
copilot/wip-address-feedback-on-mcp-servers

Conversation

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Thanks for the feedback on #703. I've created this new PR, which merges into #703, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #703
Triggering review: #703 (comment)

Hmm, interesting. We can probably do this; it'll be a bit challenging because playwright gets updated every month and you want the latest version because it's paired with the browser versions. The playwright docs themselves give you this snippet as the way to set it up, but we can do a fixed version with something to update it.

@copilot instead of using latest here. Create a github action that runs every week and updates plugin.json with the latest available version that has been published for more than 24h. The action should use npm to view the available versions and point only to stable versions. If a new version is available, the action will make the change and open a PR with the title [dotnet-blazor] <<YYYY-MM-DD>> - Update playwright MCP version to <<XXX>>. While you are at it, make sure you replace the version here too. (Make sure you use the script to validate it works).

Changes Made

  • plugins/dotnet-blazor/plugin.json: Replaced @playwright/mcp@latest with @playwright/mcp@0.0.75 (the current latest stable release) to satisfy SDL security guidelines.

  • .github/workflows/update-playwright-mcp-version.yml: New weekly GitHub Actions workflow (runs every Monday at 09:00 UTC, also supports workflow_dispatch) that:

    • Queries the npm registry for all @playwright/mcp versions and their publish timestamps
    • Selects the latest stable version (skips alpha/beta/rc/next/dev/canary pre-releases) published more than 24 hours ago
    • Skips if plugin.json is already on the latest eligible version
    • Updates plugin.json, validates the plugin with skill-validator check, then opens a PR titled [dotnet-blazor] YYYY-MM-DD - Update playwright MCP version to XXX (or updates the title of an existing open PR for the same branch)

Copilot AI changed the title [WIP] [WIP] Address feedback on MCP servers for dotnet-blazor plugin integration [WIP] Address feedback on MCP servers for dotnet-blazor plugin integration Jun 4, 2026
Copilot AI requested a review from javiercn June 4, 2026 07:52
@javiercn javiercn marked this pull request as ready for review June 4, 2026 08:01
@javiercn javiercn merged commit 8a00c7e into javiercn/add-mcp-servers-to-blazor-plugin Jun 4, 2026
31 checks passed
@javiercn javiercn deleted the copilot/wip-address-feedback-on-mcp-servers branch June 4, 2026 08:01
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Skill Coverage Report

Plugin Skill Covered Coverage
dotnet-blazor author-component 0/1 0%
dotnet-blazor collect-user-input 0/4 0%
dotnet-blazor configure-auth 0/3 0%
dotnet-blazor coordinate-components 0/2 0%
dotnet-blazor fetch-and-send-data 1/5 20%
dotnet-blazor plan-ui-change 0/1 0%
dotnet-blazor support-prerendering 0/3 0%
dotnet-blazor use-js-interop 0/2 0%
Uncovered: dotnet-blazor/author-component
  • [CodePattern] [Parameter] (line 31)
Uncovered: dotnet-blazor/collect-user-input
  • [CodePattern] [SupplyParameterFromForm] (line 31)
  • [CodePattern] [Range] (line 135)
  • [CodePattern] [Required] (line 135)
  • [CodePattern] [CascadingParameter] (line 162)
Uncovered: dotnet-blazor/configure-auth
  • [CodePattern] [ExcludeFromInteractiveRouting] (line 152)
  • [CodePattern] [CascadingParameter] (line 51)
  • [CodePattern] [Authorize] (line 101)
Uncovered: dotnet-blazor/coordinate-components
  • [CodePattern] readonly (line 137)
  • [CodePattern] [CascadingParameter] (line 63)
Uncovered: dotnet-blazor/fetch-and-send-data
  • [CodePattern] [Parameter] (line 159)
  • [CodePattern] [StreamRendering] (line 74)
  • [CodePattern] [SupplyParameterFromQuery] (line 159)
  • [CodePattern] [PersistentState] (line 84)
Uncovered: dotnet-blazor/plan-ui-change
  • [CodePattern] [Parameter] (line 64)
Uncovered: dotnet-blazor/support-prerendering
  • [CodePattern] [ExcludeFromInteractiveRouting] (line 148)
  • [CodePattern] [CascadingParameter] (line 159)
  • [CodePattern] [PersistentState] (line 39)
Uncovered: dotnet-blazor/use-js-interop
  • [CodePattern] readonly (line 118)
  • [CodePattern] sealed (line 118)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants