fix: only treat instance context exports as accessors (#9500)

* failing test for #9499 * limit export handling to instance context * changeset --------- Co-authored-by: Rich Harris <[email protected]>
sveltejs · Nov 16, 2023 · c827b0c · c827b0c · vercel · Nov 16, 2023
1 parent ede1edd
commit c827b0c
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 1 deletion.
diff --git a/.changeset/sharp-gorillas-impress.md b/.changeset/sharp-gorillas-impress.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: only treat instance context exports as accessors
diff --git a/packages/svelte/src/compiler/phases/2-analyze/index.js b/packages/svelte/src/compiler/phases/2-analyze/index.js
@@ -680,7 +680,7 @@ const runes_scope_tweaker = {
 		});
 	},
 	ExportNamedDeclaration(node, { next, state }) {
-		if (!node.declaration) {
+		if (!node.declaration || state.ast_type !== 'instance') {
 			return next();
 		}
 

diff --git a/packages/svelte/tests/runtime-runes/samples/module-context-export/_config.js b/packages/svelte/tests/runtime-runes/samples/module-context-export/_config.js
@@ -0,0 +1,9 @@
+import { test } from '../../test';
+
+export default test({
+	html: `<p>42</p>`,
+
+	async test({ assert, target, window, component }) {
+		assert.equal(component.answer, undefined);
+	}
+});
diff --git a/packages/svelte/tests/runtime-runes/samples/module-context-export/main.svelte b/packages/svelte/tests/runtime-runes/samples/module-context-export/main.svelte
@@ -0,0 +1,7 @@
+<svelte:options runes />
+
+<script context="module">
+	export const answer = 42;
+</script>
+
+<p>{answer}</p>