address feedback

Author: trueadm

packages/svelte/src/compiler/errors.js

@@ -475,7 +475,7 @@ export function store_invalid_subscription_module(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");
+	e(node, "trace_rune_duplicate", `\`$inspect.trace\` must only be used once within the same block statement\nhttps://svelte.dev/e/trace_rune_duplicate`);
 }
 
 /**
@@ -484,7 +484,7 @@ export function trace_rune_duplicate(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");
+	e(node, "trace_rune_invalid_argument", `\`$inspect.trace\` requires a string argument for the trace name\nhttps://svelte.dev/e/trace_rune_invalid_argument`);
 }
 
 /**
@@ -493,7 +493,7 @@ export function trace_rune_invalid_argument(node) {
  * @returns {never}
  */
 export function trace_rune_invalid_location(node) {
-	e(node, "trace_rune_invalid_location", "`$inspect.trace` must be placed directly inside a block statement");
+	e(node, "trace_rune_invalid_location", `\`$inspect.trace\` must be placed directly inside a block statement\nhttps://svelte.dev/e/trace_rune_invalid_location`);
 }
 
 /**
@@ -1590,4 +1590,4 @@ export function unexpected_reserved_word(node, word) {
  */
 export function void_element_invalid_content(node) {
 	e(node, "void_element_invalid_content", `Void elements cannot have children or closing tags\nhttps://svelte.dev/e/void_element_invalid_content`);
-}
+}

packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js

@@ -143,7 +143,15 @@ export function CallExpression(node, context) {
 			if (node.arguments[0].type !== 'Literal' || typeof node.arguments[0].value !== 'string') {
 				e.trace_rune_invalid_argument(node);
 			}
-			if (parent.type !== 'ExpressionStatement' || context.path.at(-2)?.type !== 'BlockStatement') {
+			if (
+				parent.type !== 'ExpressionStatement' ||
+				context.path.at(-2)?.type !== 'BlockStatement' ||
+				!(
+					context.path.at(-3)?.type === 'FunctionDeclaration' ||
+					context.path.at(-3)?.type === 'FunctionExpression' ||
+					context.path.at(-3)?.type === 'ArrowFunctionExpression'
+				)
+			) {
 				e.trace_rune_invalid_location(node);
 			}
 			if (context.state.scope.tracing) {