Remove Turbopack loader configuration that breaks JSX transform

Copilot · pranaygp · Copilot · commit 50c4120f28e2 · 2025-11-07T08:33:18.000Z
Co-authored-by: pranaygp &lt;1797812+pranaygp@users.noreply.github.com&gt;
diff --git a/packages/next/src/index.ts b/packages/next/src/index.ts
@@ -1,6 +1,5 @@
 import { NextBuilder } from './builder.js';
 import type { NextConfig } from 'next';
-import semver from 'semver';
 
 export function withWorkflow(
   nextConfigOrFn:
@@ -51,35 +50,10 @@ export function withWorkflow(
     // shallow clone to avoid read-only on top-level
     nextConfig = Object.assign({}, nextConfig);
 
-    // configure the loader if turbopack is being used
-    if (!nextConfig.turbopack) {
-      nextConfig.turbopack = {};
-    }
-    if (!nextConfig.turbopack.rules) {
-      nextConfig.turbopack.rules = {};
-    }
-    const existingRules = nextConfig.turbopack.rules as any;
-    const nextVersion = require('next/package.json').version;
-    const supportsTurboCondition = semver.gte(nextVersion, 'v16.0.0');
-
-    for (const key of ['*.tsx', '*.ts', '*.jsx', '*.js']) {
-      nextConfig.turbopack.rules[key] = {
-        ...(supportsTurboCondition
-          ? {
-              condition: {
-                ...existingRules[key]?.condition,
-                any: [
-                  ...(existingRules[key]?.condition.any || []),
-                  {
-                    content: /(use workflow|use step)/,
-                  },
-                ],
-              },
-            }
-          : {}),
-        loaders: [...(existingRules[key]?.loaders || []), loaderPath],
-      };
-    }
+    // NOTE: Turbopack configuration is currently disabled due to conflicts with Next.js's
+    // built-in TypeScript and JSX transform handling. The webpack loader below handles
+    // workflow transformations for both webpack and turbopack builds.
+    // TODO: Investigate proper Turbopack loader integration that doesn't override defaults.
 
     // configure the loader for webpack
     const existingWebpackModify = nextConfig.webpack;
diff --git a/workbench/nextjs-turbopack/app/layout.tsx b/workbench/nextjs-turbopack/app/layout.tsx
@@ -1,17 +1,6 @@
 import type { Metadata } from 'next';
-import { Geist, Geist_Mono } from 'next/font/google';
 import './globals.css';
 
-const geistSans = Geist({
-  variable: '--font-geist-sans',
-  subsets: ['latin'],
-});
-
-const geistMono = Geist_Mono({
-  variable: '--font-geist-mono',
-  subsets: ['latin'],
-});
-
 export const metadata: Metadata = {
   title: 'Durable Agents',
   description: 'A durable agent using the new Workflow DevKit',
@@ -24,11 +13,7 @@ export default function RootLayout({
 }>) {
   return (
     <html lang="en">
-      <body
-        className={`${geistSans.variable} ${geistMono.variable} antialiased`}
-      >
-        {children}
-      </body>
+      <body className="antialiased">{children}</body>
     </html>
   );
 }
diff --git a/workbench/nextjs-turbopack/app/test-jsx-transform/page.tsx b/workbench/nextjs-turbopack/app/test-jsx-transform/page.tsx
@@ -0,0 +1,17 @@
+// Test file WITHOUT "use workflow" or "use step" directives
+// This should use automatic JSX transform and NOT fail with "React is not defined"
+
+export default function TestPage() {
+  return (
+    <div>
+      <h1>JSX Transform Test</h1>
+      <p>This component has JSX but no React import.</p>
+      <p>If the automatic JSX transform works, this will render correctly.</p>
+      <ul>
+        <li>Item 1</li>
+        <li>Item 2</li>
+        <li>Item 3</li>
+      </ul>
+    </div>
+  );
+}
