From b423b5c69503c58610699894a1a7dc7ef9a015bf Mon Sep 17 00:00:00 2001
From: Rich Harris <rich.harris@vercel.com>
Date: Sat, 14 Dec 2024 23:47:40 -0500
Subject: [PATCH] update errors

---
 .../98-reference/.generated/compile-errors.md  | 18 ++++++------------
 .../svelte/messages/compile-errors/script.md   | 12 ++++--------
 packages/svelte/src/compiler/errors.js         | 13 ++-----------
 .../2-analyze/visitors/CallExpression.js       |  2 +-
 4 files changed, 13 insertions(+), 32 deletions(-)

diff --git a/documentation/docs/98-reference/.generated/compile-errors.md b/documentation/docs/98-reference/.generated/compile-errors.md
index 45a525b714..c553c60c88 100644
--- a/documentation/docs/98-reference/.generated/compile-errors.md
+++ b/documentation/docs/98-reference/.generated/compile-errors.md
@@ -442,6 +442,12 @@ Expected whitespace
 Imports of `svelte/internal/*` are forbidden. It contains private runtime code which is subject to change without notice. If you're importing from `svelte/internal/*` to work around a limitation of Svelte, please open an issue at https://github.com/sveltejs/svelte and explain your use case
 ```
 
+### inspect_trace_invalid_placement
+
+```
+`$inspect.trace(...)` must be the first statement of a function body
+```
+
 ### invalid_arguments_usage
 
 ```
@@ -970,18 +976,6 @@ A `<textarea>` can have either a value attribute or (equivalently) child content
 `<title>` can only contain text and {tags}
 ```
 
-### trace_rune_invalid_argument
-
-```
-`$inspect.trace` requires a string argument for the trace name
-```
-
-### trace_rune_invalid_location
-
-```
-`$inspect.trace(...)` must be the first statement of a function body
-```
-
 ### transition_conflict
 
 ```
diff --git a/packages/svelte/messages/compile-errors/script.md b/packages/svelte/messages/compile-errors/script.md
index 7482a6c917..5a3f2802c9 100644
--- a/packages/svelte/messages/compile-errors/script.md
+++ b/packages/svelte/messages/compile-errors/script.md
@@ -54,6 +54,10 @@
 
 > Imports of `svelte/internal/*` are forbidden. It contains private runtime code which is subject to change without notice. If you're importing from `svelte/internal/*` to work around a limitation of Svelte, please open an issue at https://github.com/sveltejs/svelte and explain your use case
 
+## inspect_trace_invalid_placement
+
+> `$inspect.trace(...)` must be the first statement of a function body
+
 ## invalid_arguments_usage
 
 > The arguments keyword cannot be used within the template or at the top level of a component
@@ -186,14 +190,6 @@ It's possible to export a snippet from a `<script module>` block, but only if it
 
 Using a `$` prefix to refer to the value of a store is only possible inside `.svelte` files, where Svelte can automatically create subscriptions when a component is mounted and unsubscribe when the component is unmounted. Consider migrating to runes instead.
 
-## trace_rune_invalid_argument
-
-> `$inspect.trace` requires a string argument for the trace name
-
-## trace_rune_invalid_location
-
-> `$inspect.trace(...)` must be the first statement of a function body
-
 ## typescript_invalid_feature
 
 > TypeScript language features like %feature% are not natively supported, and their use is generally discouraged. Outside of `<script>` tags, these features are not supported. For use within `<script>` tags, you will need to use a preprocessor to convert it to JavaScript before it gets passed to the Svelte compiler. If you are using `vitePreprocess`, make sure to specifically enable preprocessing script tags (`vitePreprocess({ script: true })`)
diff --git a/packages/svelte/src/compiler/errors.js b/packages/svelte/src/compiler/errors.js
index 87e1c8bb33..3ba204e1d2 100644
--- a/packages/svelte/src/compiler/errors.js
+++ b/packages/svelte/src/compiler/errors.js
@@ -469,22 +469,13 @@ export function store_invalid_subscription_module(node) {
 	e(node, "store_invalid_subscription_module", `Cannot reference store value outside a \`.svelte\` file\nhttps://svelte.dev/e/store_invalid_subscription_module`);
 }
 
-/**
- * `$inspect.trace` requires a string argument for the trace name
- * @param {null | number | NodeLike} node
- * @returns {never}
- */
-export function trace_rune_invalid_argument(node) {
-	e(node, "trace_rune_invalid_argument", `\`$inspect.trace\` requires a string argument for the trace name\nhttps://svelte.dev/e/trace_rune_invalid_argument`);
-}
-
 /**
  * `$inspect.trace(...)` must be the first statement of a function body
  * @param {null | number | NodeLike} node
  * @returns {never}
  */
-export function trace_rune_invalid_location(node) {
-	e(node, "trace_rune_invalid_location", `\`$inspect.trace(...)\` must be the first statement of a function body\nhttps://svelte.dev/e/trace_rune_invalid_location`);
+export function inspect_trace_invalid_placement(node) {
+	e(node, "inspect_trace_invalid_placement", `\`$inspect.trace(...)\` must be the first statement of a function body\nhttps://svelte.dev/e/inspect_trace_invalid_placement`);
 }
 
 /**
diff --git a/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js b/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js
index 83d07816e9..c221cedb4a 100644
--- a/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js
+++ b/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js
@@ -155,7 +155,7 @@ export function CallExpression(node, context) {
 				) ||
 				grand_parent.body[0] !== parent
 			) {
-				e.trace_rune_invalid_location(node);
+				e.inspect_trace_invalid_placement(node);
 			}
 
 			if (dev) {