Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 15, 2025

Pins the @playwright/mcp package to version 1.56.1 using a centralized constant and updates the dependency checker workflow to monitor NPM package updates.

Changes

Constants and version management

  • Added DefaultPlaywrightVersion = "1.56.1" in pkg/constants/constants.go
  • Replaced hardcoded "latest" strings with constant reference in:
    • pkg/workflow/mcp_servers.go - getPlaywrightDockerImageVersion()
    • pkg/workflow/mcp-config.go - JSON config rendering
    • pkg/workflow/mcp_renderer.go - TOML config rendering

Dependency tracking workflow

  • Enhanced .github/workflows/dependabot-go-checker.md to include NPM package checks
  • Added Phase 1.5: instructions for checking @playwright/mcp updates via NPM registry
  • Added example issue format for Playwright version updates

Test updates

  • Updated test assertions across 4 test files to expect "1.56.1" instead of "latest"
  • Regenerated 5 workflow .lock.yml files with pinned version

Example

Before:

playwrightDockerImageVersion := "latest" // hardcoded default

After:

playwrightDockerImageVersion := constants.DefaultPlaywrightVersion // centralized version

Compiled workflows now generate:

"args": ["@playwright/mcp@1.56.1", ...]
Original prompt

Pin playwright to 1.56.1 using a constant in constants.go.

Update cli dependency check agentic workflow to check for playwright updates as well.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Nov 15, 2025
- Add DefaultPlaywrightVersion constant in pkg/constants/constants.go
- Update getPlaywrightDockerImageVersion() to use the constant
- Update renderPlaywrightMCPConfig functions to use the constant
- Update all tests to expect 1.56.1 instead of latest
- Update dependabot-go-checker.md workflow to check for playwright updates
- Add Phase 1.5 for checking @playwright/mcp NPM package updates
- Add example issue format for playwright package updates

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Pin Playwright to version 1.56.1 in constants.go Pin Playwright to 1.56.1 and add NPM package update tracking Nov 15, 2025
Copilot AI requested a review from pelikhan November 15, 2025 23:03
Copilot finished work on behalf of pelikhan November 15, 2025 23:03
@pelikhan pelikhan marked this pull request as ready for review November 15, 2025 23:05
Copilot AI review requested due to automatic review settings November 15, 2025 23:05
Copilot finished reviewing on behalf of pelikhan November 15, 2025 23:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR pins the @playwright/mcp package to version 1.56.1 using a centralized constant and extends the dependency monitoring workflow to track NPM package updates. The change improves version consistency across the codebase and enables automated monitoring of the Playwright MCP package.

Key changes:

  • Introduced DefaultPlaywrightVersion constant in pkg/constants/constants.go for centralized version management
  • Replaced all hardcoded "latest" references with the constant across production code
  • Enhanced the Dependabot workflow to include NPM package update checks with detailed instructions and example output

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/constants/constants.go Added DefaultPlaywrightVersion = "1.56.1" constant with appropriate documentation
pkg/workflow/mcp_servers.go Updated getPlaywrightDockerImageVersion() to use the new constant instead of hardcoded "latest"
pkg/workflow/mcp-config.go Updated JSON config rendering to reference the constant in two locations
pkg/workflow/mcp_renderer.go Updated TOML config rendering to reference the constant
pkg/workflow/version_field_test.go Updated test assertions to expect "1.56.1" instead of "latest"
pkg/workflow/mcp_config_shared_test.go Updated test expectations for both test cases to verify "1.56.1" version
pkg/workflow/mcp_config_refactor_test.go Updated test assertions in JSON and TOML test cases
pkg/workflow/custom_engine_test.go Updated assertions in two integration tests for domain configuration
.github/workflows/dependabot-go-checker.md Added Phase 1.5 with NPM package checking instructions and Example 3 for Playwright updates
.github/workflows/dependabot-go-checker.lock.yml Regenerated lock file with updated workflow name and new NPM monitoring instructions
.github/workflows/unbloat-docs.lock.yml Regenerated with pinned Playwright version "1.56.1"
.github/workflows/daily-multi-device-docs-tester.lock.yml Regenerated with pinned Playwright version "1.56.1"
.github/workflows/cloclo.lock.yml Regenerated with pinned Playwright version "1.56.1"
.github/workflows/blog-auditor.lock.yml Regenerated with pinned Playwright version "1.56.1"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pelikhan pelikhan merged commit 2f9b604 into main Nov 15, 2025
96 checks passed
@pelikhan pelikhan deleted the copilot/pin-playwright-version-1-56-1 branch November 15, 2025 23:10
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