fix: resolve more alias issues (#14514)

* fix: resolve more alias issues

- split up regexes to make them easier to reason about
- fix some more cases
- turn slow e2e test into unit test which can test more combinations easily

Fixes #14455

* also fixes #14480

* remove

Author: dummdidumm

.changeset/sour-bananas-tease.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/package': patch
+---
+
+fix: resolve more alias issues

packages/package/src/utils.js

@@ -22,7 +22,12 @@ export function resolve_aliases(input, file, content, aliases) {
 	 */
 	const replace_import_path = (match, quote, import_path) => {
 		for (const [alias, value] of Object.entries(aliases)) {
-			if (import_path !== alias && !import_path.startsWith(alias + '/')) continue;
+			if (
+				import_path !== alias &&
+				!import_path.startsWith(alias + (alias.endsWith('/') ? '' : '/'))
+			) {
+				continue;
+			}
 
 			const full_path = path.join(input, file);
 			const full_import_path = path.join(value, import_path.slice(alias.length));
@@ -33,24 +38,22 @@ export function resolve_aliases(input, file, content, aliases) {
 		return match;
 	};
 
-	// import/export ... from ...
+	// import/export (type) (xxx | xxx,) { ... } from ...
 	content = content.replace(
-		// Regex parts for import/export ... from ... statements:
-		// 1. \b(import|export)         - Match 'import' or 'export' at a word boundary
-		// 2. (?:\s+type)?              - Optionally match ' type'
-		// 3. \s+                       - At least one whitespace
-		// 4. (                         - Start of specifier group
-		//    (?:(?:\*\s+as\s+)?\p{L}[\p{L}0-9]*\s+)  - default import/export, e.g. 'name', or named star import/export, e.g. '* as name '
-		//    | (?:\*\s+)               - e.g. star import/export, e.g. '* '
-		//    | (?:(?:\p{L}[\p{L}0-9]*\s*,\s*)?\{[^}]*\}\s*)  - e.g. named imports/exports with optional default import/export, e.g. 'name, { ... }' or '{ ... }'
-		//   )
-		// 5. from\s*                   - Match 'from' with optional whitespace
-		// 6. (['"])                    - Capture quote
-		// 7. ([^'";]+)                 - Capture import path
-		// 8. \3                        - Match the same quote as before
-		/\b(import|export)(?:\s+type)?\s+((?:(?:\*\s+as\s+)?\p{L}[\p{L}0-9]*\s+)|(?:\*\s+)|(?:(?:\p{L}[\p{L}0-9]*\s*,\s*)?\{[^}]*\}\s*))from\s*(['"])([^'";]+)\3/gmu,
-		(match, _keyword, _specifier, quote, import_path) =>
-			replace_import_path(match, quote, import_path)
+		/\b(?:import|export)(?:\s+type)?(?:(?:\s+\p{L}[\p{L}0-9]*\s+)|(?:(?:\s+\p{L}[\p{L}0-9]*\s*,\s*)?\s*\{[^}]*\}\s*))from\s*(['"])([^'";]+)\1/gmu,
+		(match, quote, import_path) => replace_import_path(match, quote, import_path)
+	);
+
+	// import/export (type) * as xxx from ...
+	content = content.replace(
+		/\b(?:import|export)(?:\s+type)?\s*\*\s*as\s+\p{L}[\p{L}0-9]*\s+from\s*(['"])([^'";]+)\1/gmu,
+		(match, quote, import_path) => replace_import_path(match, quote, import_path)
+	);
+
+	// export (type) * from ...
+	content = content.replace(
+		/\b(?:export)(?:\s+type)?\s*\*\s*from\s*(['"])([^'";]+)\1/gmu,
+		(match, quote, import_path) => replace_import_path(match, quote, import_path)
 	);
 
 	// import(...)