Skip to content

chore(babel-config): forJest sentinel#2061

Merged
Tobbe merged 1 commit into
mainfrom
tobbe-vite-context-wrapping-plugin-use
Jul 8, 2026
Merged

chore(babel-config): forJest sentinel#2061
Tobbe merged 1 commit into
mainfrom
tobbe-vite-context-wrapping-plugin-use

Conversation

@Tobbe

@Tobbe Tobbe commented Jul 8, 2026

Copy link
Copy Markdown
Member

Match #2058 so that we have a consistent pattern going forward for babel -> vite plugin rewrites

@netlify

netlify Bot commented Jul 8, 2026

Copy link
Copy Markdown

Deploy Preview for cedarjs canceled.

Name Link
🔨 Latest commit e69149c
🔍 Latest deploy log https://app.netlify.com/projects/cedarjs/deploys/6a4e59bb60000100089a4a85

@github-actions github-actions Bot added this to the chore milestone Jul 8, 2026
@greptile-apps

greptile-apps Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR narrows the web Babel dev fatal error page transform to Jest. The main changes are:

  • The Babel plugin now runs only when forJest is true.
  • The comments now point prerender and Vite to their Rollup and Vite plugins.

Confidence Score: 4/5

The main build, Jest, Vite, and prerender paths look covered, but default Babel consumers can now keep the dev fatal error page import.

  • Jest still passes the new sentinel.
  • Vite and prerender have separate removal plugins.
  • Flagless Babel config callers no longer get the production-only removal transform.

packages/babel-config/src/web.ts

Important Files Changed

Filename Overview
packages/babel-config/src/web.ts Updates the production DevFatalErrorPage removal gate from non-Vite Babel consumers to the Jest sentinel path.

Reviews (1): Last reviewed commit: "chore(babel-config): `forJest` sentinel" | Re-trigger Greptile

Comment thread packages/babel-config/src/web.ts
@nx-cloud

nx-cloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit e69149c

Command Status Duration Result
nx run-many -t test --minWorkers=1 --maxWorkers=4 ✅ Succeeded 3m 4s View ↗
nx run-many -t build:pack --exclude create-ceda... ✅ Succeeded 24s View ↗
nx run-many -t build ✅ Succeeded 1m 26s View ↗
nx run-many -t test:types ✅ Succeeded 11s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-07-08 14:20:45 UTC

@Tobbe Tobbe merged commit ab2f66a into main Jul 8, 2026
50 checks passed
@Tobbe Tobbe deleted the tobbe-vite-context-wrapping-plugin-use branch July 8, 2026 14:25
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

The changes in this PR are now available on npm.

Try them out by running yarn cedar upgrade -t 5.0.0-canary.2605

Or try it in a new app with yarn dlx create-cedar-app@5.0.0-canary.2605

Tobbe added a commit that referenced this pull request Jul 8, 2026
Tobbe added a commit that referenced this pull request Jul 8, 2026
Tobbe added a commit that referenced this pull request Jul 8, 2026
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.

1 participant