Revert "switch from regex check to svelteCompiler.parse"

This reverts commit 731cdc0.

Author: Lms24

packages/svelte/rollup.npm.config.js

@@ -3,6 +3,6 @@ import { makeBaseNPMConfig, makeNPMConfigVariants } from '../../rollup/index.js'
 export default makeNPMConfigVariants(
   makeBaseNPMConfig({
     // Prevent 'svelte/internal' stuff from being included in the built JS
-    packageSpecificConfig: { external: ['svelte/internal', 'svelte/compiler'] },
+    packageSpecificConfig: { external: ['svelte/internal'] },
   }),
 );

packages/svelte/src/preprocessors.ts

@@ -1,5 +1,4 @@
 import MagicString from 'magic-string';
-import * as svelteCompiler from 'svelte/compiler';
 import { PreprocessorGroup } from 'svelte/types/compiler/preprocess';
 
 import { ComponentTrackingInitOptions, SentryPreprocessorGroup, TrackComponentOptions } from './types';
@@ -124,8 +123,16 @@ function getBaseName(filename: string): string {
 }
 
 function hasScriptTag(content: string): boolean {
-  const ast = svelteCompiler.parse(content);
-  // ast.instance holds whatever is defined in <script>
-  // if there is no script tag, it's undefined.
-  return ast.instance !== undefined;
+  // This regex is taken from the Svelte compiler code.
+  // They use this regex to find matching script tags that are passed to the `script` preprocessor hook:
+  // https://github.com/sveltejs/svelte/blob/bb83eddfc623437528f24e9fe210885b446e72fa/src/compiler/preprocess/index.ts#L144
+  // However, we remove the first part of the regex to not match HTML comments
+  const scriptTagRegex = /<script(\s[^]*?)?(?:>([^]*?)<\/script>|\/>)/gi;
+
+  // Regex testing is not a super safe way of checking for the presence of a <script> tag in the Svelte
+  // component file but I think we can use it as a start.
+  // A case that is not covered by regex-testing HTML is e.g. nested <script> tags but I cannot
+  // think of why one would do this in Svelte components. For instance, the Svelte compiler errors
+  // when there's more than one top-level script tag.
+  return scriptTagRegex.test(content);
 }