chore: tidy up post-#10481 (#10503)

* simplify

* replace snapshot test with runtime test

* this is overkill

---------

Co-authored-by: Rich Harris <[email protected]>

Author: Rich-Harris

packages/svelte/src/compiler/phases/3-transform/server/transform-server.js

@@ -362,25 +362,6 @@ function is_store_name(name) {
 	return name[0] === '$' && /[A-Za-z_]/.test(name[1]);
 }
 
-/**
- *
- * @param {Iterable<import('#compiler').Binding>} bindings
- */
-function store_sub_exist(bindings) {
-	for (const binding of bindings) {
-		if (binding.kind === 'store_sub') {
-			for (const reference of binding.references) {
-				const node = reference.path.at(-1);
-
-				// hacky way to ensure the sub is not in a directive e.g. use:$store as it is unneeded
-				if (node?.type !== 'RegularElement') {
-					return true;
-				}
-			}
-		}
-	}
-}
-
 /**
  * @param {import('estree').AssignmentExpression} node
  * @param {import('zimmerframe').Context<import('#compiler').SvelteNode, import('./types').ServerTransformState>} context
@@ -2108,7 +2089,11 @@ export function server_component(analysis, options) {
 		];
 	}
 
-	if (store_sub_exist(analysis.instance.scope.declarations.values())) {
+	if (
+		[...analysis.instance.scope.declarations.values()].some(
+			(binding) => binding.kind === 'store_sub'
+		)
+	) {
 		instance.body.unshift(b.const('$$store_subs', b.object([])));
 		template.body.push(b.stmt(b.call('$.unsubscribe_stores', b.id('$$store_subs'))));
 	}

packages/svelte/src/compiler/phases/scope.js

@@ -331,15 +331,10 @@ export function create_scopes(ast, root, allow_reactive_declarations, parent) {
 	}
 
 	/**
-	 * Reference store in transion:, use:, animate: directives
 	 * @type {import('zimmerframe').Visitor<import('#compiler').Directive, State, import('#compiler').SvelteNode>}
 	 */
 	const SvelteDirective = (node, context) => {
-		const name = node.name;
-
-		if (name[0] === '$') {
-			context.state.scope.reference(b.id(name), context.path);
-		}
+		context.state.scope.reference(b.id(node.name), context.path);
 	};
 
 	walk(ast, state, {

packages/svelte/tests/runtime-runes/samples/store-directive/_config.js

@@ -0,0 +1,6 @@
+import { test } from '../../test';
+
+export default test({
+	html: `<div>mounted</div>`,
+	ssrHtml: `<div>hello</div>`
+});

packages/svelte/tests/runtime-runes/samples/store-directive/main.svelte

@@ -0,0 +1,9 @@
+<script>
+	import { writable } from 'svelte/store';
+
+	let action = writable((node) => {
+		node.textContent = 'mounted';
+	});
+</script>
+
+<div use:$action>hello</div>