fix a few stats glitches

src/Stats.ts

@@ -99,7 +99,7 @@ export default class Stats {
 		return {
 			timings,
 			warnings: this.warnings,
-			vars: component.vars.map(variable => ({
+			vars: component.vars.filter(variable => variable.kind !== 'global').map(variable => ({
 				name: variable.name,
 				kind: variable.kind,
 				import_name: variable.import_name || null,

src/compile/Component.ts

@@ -603,10 +603,18 @@ export default class Component {
 		globals.forEach(name => {
 			if (this.module_scope && this.module_scope.declarations.has(name)) return;
 
-			this.add_var({
+			if (name[0] === '$') {
-				name,
+				this.add_var({
-				kind: 'global'
+					name,
-			});
+					kind: 'injected',
+					mutated: true
+				});
+			} else {
+				this.add_var({
+					name,
+					kind: 'global'
+				});
+			}
 		});
 
 		this.extract_imports(script.content, false);
@@ -682,9 +690,6 @@ export default class Component {
 
 					if (name[0] === '$' && !scope.has(name)) {
 						component.warn_if_undefined(object, null);
-
-						// cheeky hack
-						component.add_reference(name);
 					}
 				}
 			},

src/compile/nodes/shared/Expression.ts

@@ -195,7 +195,7 @@ export default class Expression {
 			if (this.template_scope.is_let(name)) return true;
 
 			const variable = this.component.var_lookup.get(name);
-			return variable.mutated;
+			return variable && variable.mutated;
 		});
 	}
 
@@ -479,7 +479,7 @@ function isContextual(component: Component, scope: TemplateScope, name: string)
 	const variable = component.var_lookup.get(name);
 
 	// hoistables, module declarations, and imports are non-contextual
-	if (variable.hoistable) return false;
+	if (!variable || variable.hoistable) return false;
 
 	// assume contextual
 	return true;

test/stats/samples/store-referenced/_config.js

@@ -0,0 +1,26 @@
+export default {
+	test(assert, stats) {
+		assert.deepEqual(stats.vars, [
+			{
+				name: 'foo',
+				kind: 'let',
+				import_name: null,
+				export_name: null,
+				source: null,
+				module: false,
+				mutated: false,
+				referenced: false
+			},
+			{
+				name: '$foo',
+				kind: 'injected',
+				import_name: null,
+				export_name: null,
+				source: null,
+				module: false,
+				mutated: true,
+				referenced: true
+			}
+		]);
+	}
+};

test/stats/samples/store-referenced/input.html

@@ -0,0 +1,5 @@
+<script>
+	let foo;
+</script>
+
+{$foo}

test/stats/samples/store-unreferenced/_config.js

@@ -0,0 +1,26 @@
+export default {
+	test(assert, stats) {
+		assert.deepEqual(stats.vars, [
+			{
+				name: 'foo',
+				kind: 'let',
+				import_name: null,
+				export_name: null,
+				source: null,
+				module: false,
+				mutated: false,
+				referenced: false
+			},
+			{
+				name: '$foo',
+				kind: 'injected',
+				import_name: null,
+				export_name: null,
+				source: null,
+				module: false,
+				mutated: true,
+				referenced: false
+			}
+		]);
+	}
+};

test/stats/samples/store-unreferenced/input.html

@@ -0,0 +1,4 @@
+<script>
+	let foo;
+	console.log($foo);
+</script>

test/stats/samples/undeclared/_config.js

@@ -0,0 +1,5 @@
+export default {
+	test(assert, stats) {
+		assert.deepEqual(stats.vars, []);
+	},
+};

test/stats/samples/undeclared/input.html

@@ -0,0 +1,3 @@
+<script></script>
+
+{foo}