fix binding to module-level variables (#4352)

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+* Fix binding to module-level variables ([#4086](https://github.com/sveltejs/svelte/issues/4086))
 * Disallow attribute/prop names from matching two-way-bound names or `{shorthand}` attribute/prop names ([#4325](https://github.com/sveltejs/svelte/issues/4325))
 
 ## 3.18.1

src/compiler/compile/render_dom/Renderer.ts

@@ -161,11 +161,14 @@ export default class Renderer {
 		}
 
 		if (
-			variable &&
+			variable && (
-			!variable.referenced &&
+				variable.module || (
-			!variable.is_reactive_dependency &&
+					!variable.referenced &&
-			!variable.export_name &&
+					!variable.is_reactive_dependency &&
-			!name.startsWith('$$')
+					!variable.export_name &&
+					!name.startsWith('$$')
+				)
+			)
 		) {
 			return value || name;
 		}

test/runtime/samples/module-context-bind/_config.js

@@ -0,0 +1,4 @@
+export default {
+	skip_if_ssr: true,
+	html: '<div>object</div>'
+};

test/runtime/samples/module-context-bind/main.svelte

@@ -0,0 +1,11 @@
+<script context='module'>
+	let foo;
+</script>
+
+<script>
+	import { onMount } from 'svelte';
+	let bar;
+	onMount(() => bar = foo);
+</script>
+
+<div bind:this={foo}>{typeof bar}</div>