fix[angular-gen2]: only use the edge runtime when the edge build is imported (#4152)

sidmohanty11 · web-flow · commit f4584fa0431a · 2025-09-25T08:49:45.000Z
## Description As we are skipping hydration due to limitations with how we generate dynamic components we have to run eval twice - once on the server and once on the browser. The edge build was working and edge-runtime was being used but on the browser load it was fallbacking to use the browser runtime which was leading to the unsafe-eval errors. This PR fixes the issue by using edge-runtime both in the server and the browser - we will only use the edge build when users import it using: ```ts import { Content, type BuilderContent } from "@builder.io/sdk-angular/bundle/edge"; ``` Jira https://builder-io.atlassian.net/browse/ENG-10322 Screenshot <img width="2555" height="690" alt="image" src="https://github.com/user-attachments/assets/618bc9b2-e530-4547-b504-16c4be1ff369" />
diff --git a/.changeset/moody-ants-act.md b/.changeset/moody-ants-act.md
@@ -0,0 +1,5 @@
+---
+"@builder.io/sdk-angular": patch
+---
+
+fix: only use the edge runtime when the edge build is imported
diff --git a/packages/sdks/output/angular/scripts/multi-build.mjs b/packages/sdks/output/angular/scripts/multi-build.mjs
@@ -62,6 +62,30 @@ const transformFile = (filePath, replaceValue, revert = false) => {
                 hasReplacedImport = true;
               }
             },
+            /**
+             * When building EDGE bundle, force choose-eval to always use the edge evaluator
+             * and skip any browser fallback logic.
+             * 
+             * converts chooseBrowserOrServerEval = (args) => evaluator(args)
+             */
+            ExportNamedDeclaration(path) {
+              if (process.env.SDK_ENV !== 'edge' || revert) return;
+              const decl = path.node.declaration;
+              if (!decl || decl.type !== 'VariableDeclaration') return;
+              const first = decl.declarations && decl.declarations[0];
+              if (!first || first.type !== 'VariableDeclarator') return;
+              if (first.id.type !== 'Identifier') return;
+              if (first.id.name !== 'chooseBrowserOrServerEval') return;
+
+              // Replace the entire initializer with a simple arrow function
+              first.init = babel.types.arrowFunctionExpression(
+                [babel.types.identifier('args')],
+                babel.types.callExpression(
+                  babel.types.identifier('evaluator'),
+                  [babel.types.identifier('args')]
+                )
+              );
+            },
           },
         };
       },

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@builder.io/sdk-angular": patch
 +---
++
 +fix: only use the edge runtime when the edge build is imported