From 97ca6f722ef93a7b474ff6e2111f0f59b8fbd723 Mon Sep 17 00:00:00 2001
From: justinvdm <justinvderm@gmail.com>
Date: Tue, 17 Jun 2025 04:26:18 +0200
Subject: [PATCH] fix: Include 'module' import condition in resolve.conditions

---
 sdk/src/vite/configPlugin.mts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/sdk/src/vite/configPlugin.mts b/sdk/src/vite/configPlugin.mts
index dae2d55d..cd82d714 100644
--- a/sdk/src/vite/configPlugin.mts
+++ b/sdk/src/vite/configPlugin.mts
@@ -76,11 +76,15 @@ export const configPlugin = ({
               },
             },
           },
+          resolve: {
+            conditions: ["browser", "module"],
+          },
         },
         ssr: {
           resolve: {
             conditions: [
               "workerd",
+              "module",
               // context(justinvdm, 11 Jun 2025): Some packages meant for cloudflare workers, yet
               // their deps have only node import conditions, e.g. `agents` package (meant for CF),
               // has `pkce-challenge` package as a dep, which has only node import conditions.
@@ -106,7 +110,6 @@ export const configPlugin = ({
             esbuildOptions: {
               jsx: "automatic",
               jsxImportSource: "react",
-              conditions: ["workerd"],
               plugins: [],
             },
           },
@@ -129,6 +132,7 @@ export const configPlugin = ({
             conditions: [
               "workerd",
               "react-server",
+              "module",
               // context(justinvdm, 11 Jun 2025): Some packages meant for cloudflare workers, yet
               // their deps have only node import conditions, e.g. `agents` package (meant for CF),
               // has `pkce-challenge` package as a dep, which has only node import conditions.