fix: only warn on context="module" in runes mode (#13332)

* fix: only warn on context="module" in runes mode

* wow

Author: Rich-Harris

.changeset/old-planets-kiss.md

@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: only warn on context="module" in runes mode

packages/svelte/src/compiler/phases/1-parse/read/script.js

@@ -74,8 +74,6 @@ export function read_script(parser, start, attributes) {
 				e.script_invalid_context(attribute);
 			}
 
-			w.script_context_deprecated(attribute);
-
 			context = 'module';
 		}
 	}

packages/svelte/src/compiler/phases/2-analyze/index.js

@@ -359,6 +359,13 @@ export function analyze_component(root, source, options) {
 
 	const runes = options.runes ?? Array.from(module.scope.references.keys()).some(is_rune);
 
+	if (runes && root.module) {
+		const context = root.module.attributes.find((attribute) => attribute.name === 'context');
+		if (context) {
+			w.script_context_deprecated(context);
+		}
+	}
+
 	// TODO remove all the ?? stuff, we don't need it now that we're validating the config
 	/** @type {ComponentAnalysis} */
 	const analysis = {

packages/svelte/tests/validator/samples/script-context-module-deprecated/input.svelte renamed to packages/svelte/tests/validator/samples/script-context-module-legacy/input.svelte

@@ -0,0 +1 @@
+[]

packages/svelte/tests/validator/samples/script-context-module-legacy/warnings.json

@@ -0,0 +1,5 @@
+<svelte:options runes />
+
+<script context="module">
+	let num = 2;
+</script>

packages/svelte/tests/validator/samples/script-context-module-runes-deprecated/input.svelte

@@ -4,11 +4,11 @@
 		"message": "`context=\"module\"` is deprecated, use the `module` attribute instead",
 		"start": {
 			"column": 8,
-			"line": 1
+			"line": 3
 		},
 		"end": {
 			"column": 24,
-			"line": 1
+			"line": 3
 		}
 	}
 ]