chore(yarn): Fix build after yarn 4.14.1 upgrade#1721
Conversation
✅ Deploy Preview for cedarjs canceled.
|
Greptile SummaryThis PR adds a
Confidence Score: 4/5Safe to merge; the change is minimal and targeted, with a single question about whether @playwright/test needs to be included. The change correctly mirrors the approach used for user-facing project fixtures in #1673. The only open question is whether @playwright/test should be opted in; its absence is fine if CI invokes playwright install independently, but could silently break browser-based tests otherwise. Only package.json changed; verify whether @playwright/test install scripts run via a separate CI step before merging. Important Files Changed
Reviews (1): Last reviewed commit: "yarn.lock" | Re-trigger Greptile |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run-many -t build |
✅ Succeeded | 7s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-05-05 13:36:56 UTC
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run-many -t build:pack --exclude create-ceda... |
✅ Succeeded | 1s | View ↗ |
nx run-many -t build |
✅ Succeeded | 5s | View ↗ |
nx run-many -t test --minWorkers=1 --maxWorkers=4 |
✅ Succeeded | 1m 28s | View ↗ |
nx run-many -t test:types |
✅ Succeeded | 10s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-05-05 13:52:57 UTC
|
The changes in this PR are now available on npm. Try them out by running Or try it in a new app with |

#1673 disables build scripts for packages by default. I added overrides for packages we need to build for Cedar apps, but didn't realize I needed to add them for the framework itself too. This PR addresses that oversight.