[chore] improve illegal declaration error message

sveltejs · Sep 11, 2021 · f64259c · f64259c
1 parent c040f13
commit f64259c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/compiler/compile/Component.ts b/src/compiler/compile/Component.ts
@@ -587,7 +587,7 @@ export default class Component {
 
 		scope.declarations.forEach((node, name) => {
 			if (name[0] === '$') {
-				return this.error(node as any, compiler_errors.illegal_declaration);
+				return this.error(node as any, compiler_errors.illegal_declaration(name));
 			}
 
 			const writable = node.type === 'VariableDeclaration' && (node.kind === 'var' || node.kind === 'let');
@@ -660,7 +660,7 @@ export default class Component {
 
 		instance_scope.declarations.forEach((node, name) => {
 			if (name[0] === '$') {
-				return this.error(node as any, compiler_errors.illegal_declaration);
+				return this.error(node as any, compiler_errors.illegal_declaration(name));
 			}
 
 			const writable = node.type === 'VariableDeclaration' && (node.kind === 'var' || node.kind === 'let');

diff --git a/src/compiler/compile/compiler_errors.ts b/src/compiler/compile/compiler_errors.ts
@@ -174,10 +174,10 @@ export default {
 		code: 'default-export',
 		message: 'A component cannot have a default export'
 	},
-	illegal_declaration: {
+	illegal_declaration: (name: string) => ({
 		code: 'illegal-declaration',
-		message: 'The $ prefix is reserved, and cannot be used for variable and import names'
-	},
+		message: `The $ prefix is reserved, and cannot be used for variable and import names, but found ${name}`
+	}),
 	illegal_subscription: {
 		code: 'illegal-subscription',
 		message: 'Cannot reference store value inside <script context="module">'