From 8461ca9e0eedbbe8f0637aceb4c221f99fbea9a4 Mon Sep 17 00:00:00 2001
From: Rich Harris <rich.harris@vercel.com>
Date: Sat, 14 Dec 2024 18:39:23 -0500
Subject: [PATCH] remove unreachable trace_rune_duplicate error

---
 .../docs/98-reference/.generated/compile-errors.md       | 6 ------
 packages/svelte/messages/compile-errors/script.md        | 4 ----
 packages/svelte/src/compiler/errors.js                   | 9 ---------
 .../compiler/phases/2-analyze/visitors/CallExpression.js | 3 ---
 4 files changed, 22 deletions(-)

diff --git a/documentation/docs/98-reference/.generated/compile-errors.md b/documentation/docs/98-reference/.generated/compile-errors.md
index 83fbd4d382..21d2f60bb0 100644
--- a/documentation/docs/98-reference/.generated/compile-errors.md
+++ b/documentation/docs/98-reference/.generated/compile-errors.md
@@ -970,12 +970,6 @@ A `<textarea>` can have either a value attribute or (equivalently) child content
 `<title>` can only contain text and {tags}
 ```
 
-### trace_rune_duplicate
-
-```
-`$inspect.trace` must only be used once within the same block statement
-```
-
 ### trace_rune_invalid_argument
 
 ```
diff --git a/packages/svelte/messages/compile-errors/script.md b/packages/svelte/messages/compile-errors/script.md
index 8a5583fe91..831ce2e279 100644
--- a/packages/svelte/messages/compile-errors/script.md
+++ b/packages/svelte/messages/compile-errors/script.md
@@ -186,10 +186,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_duplicate
-
-> `$inspect.trace` must only be used once within the same block statement
-
 ## trace_rune_invalid_argument
 
 > `$inspect.trace` requires a string argument for the trace name
diff --git a/packages/svelte/src/compiler/errors.js b/packages/svelte/src/compiler/errors.js
index 2ae11a08b4..9ef0c8a766 100644
--- a/packages/svelte/src/compiler/errors.js
+++ b/packages/svelte/src/compiler/errors.js
@@ -469,15 +469,6 @@ 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` must only be used once within the same block statement
- * @param {null | number | NodeLike} node
- * @returns {never}
- */
-export function trace_rune_duplicate(node) {
-	e(node, "trace_rune_duplicate", `\`$inspect.trace\` must only be used once within the same block statement\nhttps://svelte.dev/e/trace_rune_duplicate`);
-}
-
 /**
  * `$inspect.trace` requires a string argument for the trace name
  * @param {null | number | NodeLike} node
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 53b82afe6f..3fabb842c0 100644
--- a/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js
+++ b/packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js
@@ -157,9 +157,6 @@ export function CallExpression(node, context) {
 			) {
 				e.trace_rune_invalid_location(node);
 			}
-			if (context.state.scope.tracing) {
-				e.trace_rune_duplicate(node);
-			}
 
 			if (dev) {
 				context.state.scope.tracing = node.arguments[0].value;