From d8c3f3d87e4f008ee902bb084f2820202d3f97c3 Mon Sep 17 00:00:00 2001
From: ComputerGuy <63362464+Ocean-OS@users.noreply.github.com>
Date: Tue, 5 Aug 2025 17:47:58 -0700
Subject: [PATCH] fix: emit `snippet_invalid_export` instead of
 `undefined_export` for exported snippets (#16539)

* fix: emit `snippet_invalid_export` instead of `undefined_export` for exported snippets

* try this

* this should work

* remove some junk (this never did anything)

* add test

---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>
---
 .changeset/thick-snakes-look.md                     |  5 +++++
 .../svelte/src/compiler/phases/2-analyze/index.js   | 13 +++++++++----
 .../phases/2-analyze/visitors/SnippetBlock.js       |  5 -----
 packages/svelte/src/compiler/phases/types.d.ts      |  1 -
 .../samples/snippet-invalid-export/_config.js       | 10 ++++++++++
 .../samples/snippet-invalid-export/main.svelte      | 11 +++++++++++
 6 files changed, 35 insertions(+), 10 deletions(-)
 create mode 100644 .changeset/thick-snakes-look.md
 create mode 100644 packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/_config.js
 create mode 100644 packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/main.svelte

diff --git a/.changeset/thick-snakes-look.md b/.changeset/thick-snakes-look.md
new file mode 100644
index 0000000000..f5f22e8a6c
--- /dev/null
+++ b/.changeset/thick-snakes-look.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: emit `snippet_invalid_export` instead of `undefined_export` for exported snippets
diff --git a/packages/svelte/src/compiler/phases/2-analyze/index.js b/packages/svelte/src/compiler/phases/2-analyze/index.js
index cd44fd998a..ecc843d2f4 100644
--- a/packages/svelte/src/compiler/phases/2-analyze/index.js
+++ b/packages/svelte/src/compiler/phases/2-analyze/index.js
@@ -526,7 +526,6 @@ export function analyze_component(root, source, options) {
 			has_global: false
 		},
 		source,
-		undefined_exports: new Map(),
 		snippet_renderers: new Map(),
 		snippets: new Set(),
 		async_deriveds: new Set()
@@ -787,9 +786,15 @@ export function analyze_component(root, source, options) {
 		if (node.type === 'ExportNamedDeclaration' && node.specifiers !== null && node.source == null) {
 			for (const specifier of node.specifiers) {
 				if (specifier.local.type !== 'Identifier') continue;
-
-				const binding = analysis.module.scope.get(specifier.local.name);
-				if (!binding) e.export_undefined(specifier, specifier.local.name);
+				const name = specifier.local.name;
+				const binding = analysis.module.scope.get(name);
+				if (!binding) {
+					if ([...analysis.snippets].find((snippet) => snippet.expression.name === name)) {
+						e.snippet_invalid_export(specifier);
+					} else {
+						e.export_undefined(specifier, name);
+					}
+				}
 			}
 		}
 	}
diff --git a/packages/svelte/src/compiler/phases/2-analyze/visitors/SnippetBlock.js b/packages/svelte/src/compiler/phases/2-analyze/visitors/SnippetBlock.js
index 2f6bbd785a..7930c2b1a7 100644
--- a/packages/svelte/src/compiler/phases/2-analyze/visitors/SnippetBlock.js
+++ b/packages/svelte/src/compiler/phases/2-analyze/visitors/SnippetBlock.js
@@ -35,11 +35,6 @@ export function SnippetBlock(node, context) {
 	if (can_hoist) {
 		const binding = /** @type {Binding} */ (context.state.scope.get(name));
 		context.state.analysis.module.scope.declarations.set(name, binding);
-	} else {
-		const undefined_export = context.state.analysis.undefined_exports.get(name);
-		if (undefined_export) {
-			e.snippet_invalid_export(undefined_export);
-		}
 	}
 
 	node.metadata.can_hoist = can_hoist;
diff --git a/packages/svelte/src/compiler/phases/types.d.ts b/packages/svelte/src/compiler/phases/types.d.ts
index 4cbbc362c7..4da78f1482 100644
--- a/packages/svelte/src/compiler/phases/types.d.ts
+++ b/packages/svelte/src/compiler/phases/types.d.ts
@@ -95,7 +95,6 @@ export interface ComponentAnalysis extends Analysis {
 	};
 	/** @deprecated use `source` from `state.js` instead */
 	source: string;
-	undefined_exports: Map<string, Node>;
 	/**
 	 * Every render tag/component, and whether it could be definitively resolved or not
 	 */
diff --git a/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/_config.js b/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/_config.js
new file mode 100644
index 0000000000..cc0dd388f4
--- /dev/null
+++ b/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/_config.js
@@ -0,0 +1,10 @@
+import { test } from '../../test';
+
+export default test({
+	error: {
+		code: 'snippet_invalid_export',
+		message:
+			'An exported snippet can only reference things declared in a `<script module>`, or other exportable snippets',
+		position: [26, 29]
+	}
+});
diff --git a/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/main.svelte b/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/main.svelte
new file mode 100644
index 0000000000..eb776944c3
--- /dev/null
+++ b/packages/svelte/tests/compiler-errors/samples/snippet-invalid-export/main.svelte
@@ -0,0 +1,11 @@
+<script module>
+	export { foo }
+</script>
+
+<script>
+	let x = 42;
+</script>
+
+{#snippet foo()}
+	{x}
+{/snippet}