deconflict `value` parameter in contextual bindings (#4452)

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+* Deconflict `value` parameter name used in contextual bindings ([#4445](https://github.com/sveltejs/svelte/issues/4445))
 * Fix dev mode validation of `{#each}` blocks using strings ([#4450](https://github.com/sveltejs/svelte/issues/4450))
 
 ## 3.19.0

src/compiler/compile/render_dom/wrappers/InlineComponent/index.ts

@@ -332,8 +332,7 @@ export default class InlineComponentWrapper extends Wrapper {
 				contextual_dependencies.push(object.name, property.name);
 			}
 
-			const value = block.get_unique_name('value');
-			const params: any[] = [value];
+			const params = [x`#value`];
 			if (contextual_dependencies.length > 0) {
 				const args = [];
 
@@ -349,23 +348,23 @@ export default class InlineComponentWrapper extends Wrapper {
 
 
 				block.chunks.init.push(b`
-					function ${id}(${value}) {
-						${callee}.call(null, ${value}, ${args});
+					function ${id}(#value) {
+						${callee}.call(null, #value, ${args});
 					}
 				`);
 
 				block.maintain_context = true; // TODO put this somewhere more logical
 			} else {
 				block.chunks.init.push(b`
-					function ${id}(${value}) {
-						${callee}.call(null, ${value});
+					function ${id}(#value) {
+						${callee}.call(null, #value);
 					}
 				`);
 			}
 
 			const body = b`
 				function ${id}(${params}) {
-					${lhs} = ${value};
+					${lhs} = #value;
 					${renderer.invalidate(dependencies[0])};
 				}
 			`;

test/runtime/samples/deconflict-contextual-bind/Widget.svelte

@@ -0,0 +1,3 @@
+<script>
+	export let prop;
+</script>

test/runtime/samples/deconflict-contextual-bind/_config.js

@@ -0,0 +1,3 @@
+export default {
+	preserveIdentifiers: true
+};

test/runtime/samples/deconflict-contextual-bind/main.svelte

@@ -0,0 +1,8 @@
+<script>
+	import Widget from './Widget.svelte';
+	const values = ['foo', 'bar'];
+</script>
+
+{#each values as value}
+	<Widget bind:value/>
+{/each}