Skip to content

deps: bump @playwright/cli from 0.1.14 to 0.1.15#43196

Merged
pelikhan merged 3 commits into
mainfrom
copilot/update-playwright-cli-version
Jul 4, 2026
Merged

deps: bump @playwright/cli from 0.1.14 to 0.1.15#43196
pelikhan merged 3 commits into
mainfrom
copilot/update-playwright-cli-version

Conversation

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Patch version bump for @playwright/cli. Updates the pinned DefaultPlaywrightCLIVersion constant and refreshes the cooldown-guard test metadata with the v0.1.15 npm publish timestamp.

Changes

  • pkg/constants/version_constants.goDefaultPlaywrightCLIVersion: "0.1.14""0.1.15"
  • pkg/constants/version_constants_test.go — update expectedVersion and publishedAtRFC3339 (2026-06-30T14:53:08.941Z) in TestDefaultPlaywrightCLIVersionOutsideCooldownWindow

The 72-hour npm release-age cooldown for v0.1.15 expires at 2026-07-03T14:53:08Z; the test will pass once CI runs after that point.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update @playwright/cli from 0.1.14 to 0.1.15 deps: bump @playwright/cli from 0.1.14 to 0.1.15 Jul 3, 2026
Copilot AI requested a review from pelikhan July 3, 2026 13:30
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Hey @Copilot 👋 — thanks for picking up this @playwright/cli patch bump from 0.1.14 → 0.1.15! The PR is well-scoped and the source issue (#43167) gives a clear implementation plan.

A couple of things to address before this is ready:

  • No changes committed yet — the PR is still [WIP] with an empty diff. The code edit to pkg/constants/version_constants.go and the test update still need to land.
  • TestsTestDefaultPlaywrightCLIVersionOutsideCooldownWindow enforces the npm release-age cooldown; the test file needs to be updated alongside the constant, and the test must pass before merging. Be sure to verify that v0.1.15 is old enough to satisfy the cooldown window.

When ready, here's a prompt you can assign to complete the work:

Complete the `@playwright/cli` version bump in the gh-aw repository:

1. Open `pkg/constants/version_constants.go` and change:
   const DefaultPlaywrightCLIVersion Version = "0.1.14"
   to:
   const DefaultPlaywrightCLIVersion Version = "0.1.15"

2. Update the expected version in `TestDefaultPlaywrightCLIVersionOutsideCooldownWindow` inside `pkg/constants/` to reference "0.1.15".

3. Verify the npm release age of `@playwright/cli` v0.1.15 exceeds the configured cooldown window before proceeding.

4. Run: go test ./pkg/constants/... -run TestDefaultPlaywrightCLIVersion

5. Run: make test

6. Commit the changes and remove the [WIP] prefix from the PR title.

Generated by ✅ Contribution Check · 116.2 AIC · ⌖ 16 AIC · ⊞ 6.3K ·

@pelikhan pelikhan marked this pull request as ready for review July 3, 2026 14:49
Copilot AI review requested due to automatic review settings July 3, 2026 14:49
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Design Decision Gate 🏗️ completed the design decision gate check.

No ADR enforcement needed: PR does not have the 'implementation' label and has ≤100 new lines of code in business logic directories (only 3 additions detected).

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Test Quality Sentinel completed test quality analysis.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

PR Code Quality Reviewer completed the code quality review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Patch version bump of the pinned @playwright/cli version used by gh-aw when Playwright is run in cli mode, along with updating the associated cooldown-guard test metadata.

Changes:

  • Bump DefaultPlaywrightCLIVersion from 0.1.14 to 0.1.15.
  • Update TestDefaultPlaywrightCLIVersionOutsideCooldownWindow to expect 0.1.15 and record the v0.1.15 npm publish timestamp.
Show a summary per file
File Description
pkg/constants/version_constants.go Updates the default pinned Playwright CLI npm version to 0.1.15.
pkg/constants/version_constants_test.go Refreshes cooldown-guard test metadata (expected version + publish timestamp) for 0.1.15.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Low

Comment on lines 135 to +139
// DefaultPlaywrightCLIVersion is the default version of the @playwright/cli package.
// Used when tools.playwright.mode is "cli" to install the CLI tool instead of the MCP server.
// Keep this version outside the default 3-day npm release-age cooldown window enforced by
// generated Playwright CLI install steps. See TestDefaultPlaywrightCLIVersionOutsideCooldownWindow.
const DefaultPlaywrightCLIVersion Version = "0.1.14"
const DefaultPlaywrightCLIVersion Version = "0.1.15"

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Skills-Based Review 🧠

Applied /tdd — changes are correct and well-documented. Approving.

📋 Key Themes & Highlights

Key Themes

  • Routine patch bump: DefaultPlaywrightCLIVersion 0.1.140.1.15, low risk.
  • Test correctly refreshed: expectedVersion and publishedAtRFC3339 updated to match npm publish timestamp for v0.1.15 (2026-06-30T14:53:08.941Z).
  • Timing note: The 72-hour cooldown expires at 2026-07-03T14:53:08Z. The PR description calls this out explicitly — no action needed.

Positive Highlights

  • TestDefaultPlaywrightCLIVersionOutsideCooldownWindow keeps the version honest; updating it alongside the constant is exactly the right pattern.
  • ✅ PR description is precise: includes the exact cooldown-expiry timestamp so CI reviewers know when the test will pass.
  • ✅ Minimal, surgical change — only the two necessary lines touched in each file.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · 24 AIC · ⌖ 8.98 AIC · ⊞ 6.6K
Comment /matt to run again

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Patch bump looks correct. Version constant and test metadata (expectedVersion + publishedAtRFC3339) are consistently updated. No security or correctness concerns.

Note: The 72-hour npm cooldown for v0.1.15 expires at 2026-07-03T14:53:08Z — approximately 2 minutes after this review was run. The test will pass once CI executes past that threshold (as called out in the PR body).

🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · 19.6 AIC · ⌖ 6.03 AIC · ⊞ 4.9K

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

🧪 Test Quality Sentinel Report

Test Quality Score: 100/100 — Excellent

Analyzed 1 test(s): 1 design, 0 implementation, 0 violation(s).

📊 Metrics (1 test)
Metric Value
Analyzed 1 (Go: 1, JS: 0)
✅ Design 1 (100%)
⚠️ Implementation 0 (0%)
Edge/error coverage 1 (100%)
Duplicate clusters 0
Inflation No
🚨 Violations 0
Test File Classification Issues
TestDefaultPlaywrightCLIVersionOutsideCooldownWindow pkg/constants/version_constants_test.go:10 design_test / behavioral_contract / high_value None

Verdict

Passed. 0% implementation tests (threshold: 30%). The modified test correctly updates the behavioral invariant: it verifies that the new pinned @playwright/cli@0.1.15 version is outside the 3-day npm release-age cooldown window. The test has a proper //go:build !integration build tag, descriptive t.Fatalf messages with full context, and covers both the exact-version-pinning check and the cooldown age guard.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

🧪 Test quality analysis by Test Quality Sentinel · 26 AIC · ⌖ 9.61 AIC · ⊞ 6.8K ·
Comment /review to run again

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Test Quality Sentinel: 100/100. 0% implementation tests (threshold: 30%).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review: Routine Patch Bump

The only concern — the 72 h npm release-age cooldown expiring at 2026-07-03T14:53:08 UTC — is already captured by an existing review comment and is self-enforced by TestDefaultPlaywrightCLIVersionOutsideCooldownWindow; CI will hard-block if this merges too early. No code-correctness issues in the diff.

💡 Minor observation (non-blocking)

publishedAtRFC3339 is manually entered with no automated verification that it matches the actual npm publish time for 0.1.15. A wrong timestamp could silently shorten or lengthen the effective cooldown window. Consider cross-checking against the npm registry (npm view @playwright/cli@0.1.15 time.created) when automating future bumps.

🔎 Code quality review by PR Code Quality Reviewer · 38.4 AIC · ⌖ 5.46 AIC · ⊞ 5.4K
Comment /review to run again

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, review the latest comments, and rerun checks if any follow-up changes are needed.

Generated by 👨‍🍳 PR Sous Chef · 10.6 AIC · ⌖ 3.18 AIC · ⊞ 6.6K ·
Comment /souschef to run again

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, review the latest comments, and rerun checks if any follow-up changes are needed.

...

Ran pr-finisher. Here's the status:

  • Reviews — Two github-actions[bot] approvals (Matt Pocock Skills Reviewer, Impeccable Skills Reviewer, Test Quality Sentinel). The one open review thread from copilot-pull-request-reviewer[bot] about the 72-hour cooldown is now outdated and moot — the cooldown expired at 2026-07-03T14:53:08Z and TestDefaultPlaywrightCLIVersionOutsideCooldownWindow passes locally.
  • Checks (local)make fmt, make lint, and make test-unit all pass. TestDefaultPlaywrightCLIVersionOutsideCooldownWindow passes with the current timestamp.
  • Checks (CI) — Stale after agent push; needs human re-trigger. The prior CI run predates cooldown expiry — a fresh re-trigger will pick up the now-passing test.
  • Mergeable — PR is open, not draft, no conflicts.

Actions taken: No code changes needed — the only blocker was time (cooldown window). Local validation confirmed all checks pass.

Hand-off: CI must be re-triggered by a maintainer (close/reopen PR, workflow_dispatch, or a push) before merge.

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Copilot AI requested a review from gh-aw-bot July 3, 2026 15:40
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

PR Triage

Field Value
Category chore (dep bump)
Risk Low
Score 28 (impact 10 + urgency 8 + quality 10)
Action batch_review

Breakdown: Patch bump @playwright/cli 0.1.14 -> 0.1.15. 17 add / 17 del. Updates version constant and test cooldown timestamp. Mechanical, low risk.

Batch: Can be reviewed alongside other dep/version bump PRs.

Generated by 🔧 PR Triage Agent · 86.6 AIC · ⌖ 10.9 AIC · ⊞ 5.5K ·

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.

[deps] Update @playwright/cli from 0.1.14 to 0.1.15

4 participants