From 0aa7f1d8b44f452892cd6c6d18d41f3b2568b050 Mon Sep 17 00:00:00 2001
From: Andrei <168741329+andreiborza@users.noreply.github.com>
Date: Tue, 12 Nov 2024 12:52:36 +0100
Subject: [PATCH] fix(nuxt): Fix dynamic import rollup plugin to work with
 latest nitro (#14243)

With nitro 2.10.X the way we imported the server config file and
dynamically imported the nitro runtime broke. Instead of landing in the
server output entry file, the sdk init and dynamic import landed in the
nitro runtime itself.

Based on
https://github.com/nitrojs/nitro/commit/8b4a408231bdc222569a32ce109796a41eac4aa6#diff-e58102d2230f95ddeef2662957b48d847a6e891e354cfd0ae6e2e03ce848d1a2R142
we managed to get back to the original solution by using the `\0raw`
prefix for the entry file.

__Note for internal__: This does not play well when using symlinks (like
`yarn link`) to link the sdk into a project. Haven't tested how yalc
behaves, but tarballs work.
---
 packages/nuxt/src/vite/addServerConfig.ts | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/packages/nuxt/src/vite/addServerConfig.ts b/packages/nuxt/src/vite/addServerConfig.ts
index cf4b2a95473e..824b46781e0d 100644
--- a/packages/nuxt/src/vite/addServerConfig.ts
+++ b/packages/nuxt/src/vite/addServerConfig.ts
@@ -118,6 +118,12 @@ function wrapEntryWithDynamicImport({
   entrypointWrappedFunctions,
   debug,
 }: { resolvedSentryConfigPath: string; entrypointWrappedFunctions: string[]; debug?: boolean }): InputPluginOption {
+  // In order to correctly import the server config file
+  // and dynamically import the nitro runtime, we need to
+  // mark the resolutionId with '\0raw' to fall into the
+  // raw chunk group, c.f. https://github.com/nitrojs/nitro/commit/8b4a408231bdc222569a32ce109796a41eac4aa6#diff-e58102d2230f95ddeef2662957b48d847a6e891e354cfd0ae6e2e03ce848d1a2R142
+  const resolutionIdPrefix = '\0raw';
+
   return {
     name: 'sentry-wrap-entry-with-dynamic-import',
     async resolveId(source, importer, options) {
@@ -146,19 +152,19 @@ function wrapEntryWithDynamicImport({
         // The enclosing `if` already checks for the suffix in `source`, but a check in `resolution.id` is needed as well to prevent multiple attachment of the suffix
         return resolution.id.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)
           ? resolution.id
-          : resolution.id
+          : `${resolutionIdPrefix}${resolution.id
               // Concatenates the query params to mark the file (also attaches names of re-exports - this is needed for serverless functions to re-export the handler)
               .concat(SENTRY_WRAPPED_ENTRY)
               .concat(
                 constructWrappedFunctionExportQuery(moduleInfo.exportedBindings, entrypointWrappedFunctions, debug),
               )
-              .concat(QUERY_END_INDICATOR);
+              .concat(QUERY_END_INDICATOR)}`;
       }
       return null;
     },
     load(id: string) {
       if (id.includes(`.mjs${SENTRY_WRAPPED_ENTRY}`)) {
-        const entryId = removeSentryQueryFromPath(id);
+        const entryId = removeSentryQueryFromPath(id).slice(resolutionIdPrefix.length);
 
         // Mostly useful for serverless `handler` functions
         const reExportedFunctions =