helpers should not be considered dependencies — fixes #492

Author: Rich-Harris

src/generators/Generator.js

@@ -156,6 +156,8 @@ export default class Generator {
 		let scope = annotateWithScopes( expression );
 		const dependencies = [];
 
+		const generator = this; // can't use arrow functions, because of this.skip()
+
 		walk( expression, {
 			enter ( node, parent ) {
 				if ( node._scope ) {
@@ -165,7 +167,7 @@ export default class Generator {
 
 				if ( isReference( node, parent ) ) {
 					const { name } = flattenReference( node );
-					if ( scope.has( name ) ) return;
+					if ( scope.has( name ) || generator.helpers.has( name ) ) return;
 
 					if ( contextDependencies.has( name ) ) {
 						dependencies.push( ...contextDependencies.get( name ) );

test/runtime/samples/dev-warning-helper/_config.js

@@ -0,0 +1,13 @@
+export default {
+	solo: true,
+
+	dev: true,
+
+	data: {
+		bar: 1
+	},
+
+	html: '2',
+
+	warnings: []
+};

test/runtime/samples/dev-warning-helper/main.html

@@ -0,0 +1,11 @@
+{{foo(bar)}}
+
+<script>
+	export default {
+		helpers: {
+			foo ( bar ) {
+				return bar * 2;
+			}
+		}
+	};
+</script>