Skip to content

chore: Bump remaining Playwright 1.58.2 references across workspace, templates, E2E actors, and Node 24 Docker images#3538

Merged
B4nan merged 7 commits intomasterfrom
copilot/update-playwright-dependency
Apr 8, 2026
Merged

chore: Bump remaining Playwright 1.58.2 references across workspace, templates, E2E actors, and Node 24 Docker images#3538
B4nan merged 7 commits intomasterfrom
copilot/update-playwright-dependency

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026 •
edited
Loading

  • Inspect the new camoufox Dockerfile review comment and compare the current file against the cited commit history
  • Revert only the unnecessary camoufox Dockerfile change while preserving the required Playwright and Node 24 updates
  • Run targeted validation for the affected templates/Dockerfile area (corepack yarn validate in packages/templates)
  • Run final review/security checks and reply to the PR comment

@B4nan B4nan marked this pull request as ready for review March 28, 2026 13:05
@github-actions github-actions bot added this to the 137th sprint - Tooling team milestone Mar 28, 2026
@github-actions github-actions bot added t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics. labels Mar 28, 2026
@B4nan B4nan added the adhoc Ad-hoc unplanned task added during the sprint. label Mar 28, 2026
@B4nan B4nan changed the title Bump Playwright to 1.58.2 across workspace, templates, E2E actors, and Docker images chore: Bump Playwright to 1.58.2 across workspace, templates, E2E actors, and Docker images Mar 28, 2026
@B4nan
Copy link
Copy Markdown
Member

B4nan commented Apr 8, 2026

@copilot use node 24 in all the dockerfile tags (e.g. apify/actor-node-playwright-chrome:24-1.58.2 instead of apify/actor-node-playwright-chrome:20-1.58.2)

renovate bot and others added 4 commits April 8, 2026 16:23
@renovate
chore(deps): update patch/minor dependencies (playwright 1.58.2, node…
cbef2de 
… 24.14.1) (#3391)

Renovate patch/minor dependency update with puppeteer excluded and a
yarn.lock fix for playwright type conflicts.

## Changes

- **Excluded**: `puppeteer` bump (`24.36.1` → `24.40.0`) — kept at
`24.36.1`
- **Updated**: `playwright` / `@playwright/browser-*` `1.58.1` →
`1.58.2`, `node` `24.13.0` → `24.14.1`,
`@apify/docusaurus-plugin-typedoc-api` `5.1.0` → `5.1.3`
- **Fixed**: yarn.lock had stale entries resolving `playwright@^1.52.0`
→ `1.58.1` and `playwright-core@^1.54.1` → `1.58.1` (used by
`@browserbasehq/stagehand`'s transitive deps) while the root used
`1.58.2`. TypeScript treated the two `playwright-core` copies as
structurally distinct, causing type errors in stagehand tests.
Consolidated all specifiers to `1.58.2`.

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
chore: bump playwright to 1.58.2
74f9937 
Agent-Logs-Url: https://github.com/apify/crawlee/sessions/fcae0d9a-3d5a-480c-99bb-ba24a499314d
chore: switch playwright docker tags to node 24
eaaa9a8
Merge remote-tracking branch 'origin/copilot/update-playwright-depend…
6e04551 
…ency'
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 8, 2026

@copilot use node 24 in all the dockerfile tags (e.g. apify/actor-node-playwright-chrome:24-1.58.2 instead of apify/actor-node-playwright-chrome:20-1.58.2)

Done in eaaa9a8. The Playwright-related template and E2E Dockerfiles now use Node 24 image tags, and I rebased the branch onto master before pushing the update.

Copilot AI changed the title chore: Bump Playwright to 1.58.2 across workspace, templates, E2E actors, and Docker images chore: Bump remaining Playwright 1.58.2 references across workspace, templates, E2E actors, and Node 24 Docker images Apr 8, 2026
Copilot finished work on behalf of B4nan April 8, 2026 14:32
Copilot AI requested a review from B4nan April 8, 2026 14:32
B4nan
B4nan reviewed Apr 8, 2026
View reviewed changes
packages/templates/templates/camoufox-ts/Dockerfile Outdated
# tree for debugging
RUN npm --quiet set progress=false \
&& npm install --omit=dev \
&& npm install --omit=dev --omit=optional \
Copy link
Copy Markdown
Member

@B4nan B4nan Apr 8, 2026

Choose a reason for hiding this comment

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

@copilot revert this unless its actually needed

Copy link
Copy Markdown
Contributor Author

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

Reverted in 34dad1f. I kept the Node 24 / Playwright 1.58.2 tags and restored that install step back to npm install --omit=dev.

@B4nan
chore: revert unnecessary camoufox optional omit
34dad1f 
Co-authored-by: B4nan <615580+B4nan@users.noreply.github.com>
Copilot finished work on behalf of B4nan April 8, 2026 14:41
Copilot AI requested a review from B4nan April 8, 2026 14:42
@B4nan B4nan merged commit 6785cc8 into master Apr 8, 2026
10 checks passed
@B4nan B4nan deleted the copilot/update-playwright-dependency branch April 8, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@B4nan B4nan Awaiting requested review from B4nan

Assignees

@B4nan B4nan

Copilot code review Copilot

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Milestone

137th sprint - Tooling team

Development

Successfully merging this pull request may close these issues.

3 participants

@B4nan @apify-service-account