-
Notifications
You must be signed in to change notification settings - Fork 478
Deps: Update Playwright #7623
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deps: Update Playwright #7623
Conversation
|
Warning Rate limit exceeded@DrJKL has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 17 minutes and 5 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdates the pnpm workspace configuration file to define workspace package globs, upgrade the Playwright test dependency, enable catalog cleanup, extend built dependency lists with additional packages, add dependency overrides, and remove duplicate configuration blocks. Changes
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
🎭 Playwright Test Results❌ Some tests failed ⏰ Completed at: 12/18/2025, 10:05:15 PM UTC 📈 Summary
📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 12/18/2025, 09:54:13 PM UTC 🔗 Links🎉 Your Storybook is ready for review! |
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 3.21 MB (baseline 3.21 MB) • ⚪ 0 BMain entry bundles and manifests
Graph Workspace — 996 kB (baseline 996 kB) • ⚪ 0 BGraph editor runtime, canvas, workflow orchestration
Views & Navigation — 6.54 kB (baseline 6.54 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Panels & Settings — 297 kB (baseline 297 kB) • ⚪ 0 BConfiguration panels, inspectors, and settings screens
UI Components — 184 kB (baseline 184 kB) • ⚪ 0 BReusable component library chunks
Data & Services — 12.5 kB (baseline 12.5 kB) • ⚪ 0 BStores, services, APIs, and repositories
Utilities & Hooks — 1.86 kB (baseline 1.86 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Vendor & Third-Party — 8.46 MB (baseline 8.46 MB) • ⚪ 0 BExternal libraries and shared vendor chunks
Other — 3.44 MB (baseline 3.44 MB) • ⚪ 0 BBundles that do not match a named category
|
benceruleanlu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can see the screenshots in this pr optionally, but I think it is blocking other prs
Yeah, I want to do the screenshots separately so that it doesn't look (at a glance) like the Playwright update caused the changes. |
|
|
|
|
Let's see if this lets it run. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7623-Deps-Update-Playwright-2cd6d73d36508155a156dec2ea7aaec1) by [Unito](https://www.unito.io)
Let's see if this lets it run. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7623-Deps-Update-Playwright-2cd6d73d36508155a156dec2ea7aaec1) by [Unito](https://www.unito.io)
## Summary Backport of #7623 to core/1.35. Original PR: #7623 🤖 Generated with [Claude Code](https://claude.com/claude-code) ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7670-backport-core-1-35-Deps-Update-Playwright-2cf6d73d365081a1b82dd84f230e9a10) by [Unito](https://www.unito.io) Co-authored-by: Alexander Brown <drjkl@comfy.org>
Backport of #7623 Cherry-picked merge commit 2044d14 to cloud/1.35. **Conflicts resolved:** - `pnpm-lock.yaml` - regenerated with pnpm install ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7669-backport-cloud-1-35-Deps-Update-Playwright-2cf6d73d365081eba108c20eed45b685) by [Unito](https://www.unito.io) Co-authored-by: Alexander Brown <drjkl@comfy.org>
## Summary Let's see if this lets it run. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7623-Deps-Update-Playwright-2cd6d73d36508155a156dec2ea7aaec1) by [Unito](https://www.unito.io)
Summary
Let's see if this lets it run.
┆Issue is synchronized with this Notion page by Unito