fix(v9/svelte): Do not insert preprocess code in script module in Svelte 5 (#17124)

Backport of #17114

---------

Co-authored-by: Richard Jelínek <[email protected]>

Author: Lms24

CHANGELOG.md

@@ -4,6 +4,8 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+Work in this release was contributed by @richardjelinek-fastest. Thank you for your contribution!
+
 ## 9.40.0
 
 ### Important Changes

packages/svelte/src/preprocessors.ts

@@ -97,7 +97,7 @@ function shouldInjectFunction(
   // because the code inside is not executed when the component is instantiated but
   // when the module is first imported.
   // see: https://svelte.dev/docs#component-format-script-context-module
-  if (attributes.context === 'module') {
+  if (attributes.module || attributes.context === 'module') {
     return false;
   }
 

packages/svelte/test/preprocessors.test.ts

@@ -170,6 +170,26 @@ describe('componentTrackingPreprocessor', () => {
 
       expect(processedComponent.newCode).toEqual(processedComponent.originalCode);
     });
+
+    it('doesnt inject the function call to a module context script block with Svelte 5 module attribute', () => {
+      const preProc = componentTrackingPreprocessor();
+      const component = {
+        originalCode: 'console.log(cmp2)',
+        filename: 'lib/Cmp2.svelte',
+        name: 'Cmp2',
+      };
+
+      const res: any = preProc.script?.({
+        content: component.originalCode,
+        filename: component.filename,
+        attributes: { module: true },
+        markup: '',
+      });
+
+      const processedComponent = { ...component, newCode: res.code, map: res.map };
+
+      expect(processedComponent.newCode).toEqual(processedComponent.originalCode);
+    });
   });
 
   describe('markup hook', () => {