Merge pull request #2126 from sveltejs/repl-props

Some fixes for REPL props editor

site/static/workers/compiler.js

@@ -37,7 +37,7 @@ function compile({ source, options, entry }) {
 		);
 
 		const props = entry
-			? (vars || stats.vars).map(v => v.export_name).filter(Boolean) // TODO remove stats post-launch
+			? (vars || stats.vars).map(v => v.writable && v.export_name).filter(Boolean) // TODO remove stats post-launch
 			: null;
 
 		return { js: js.code, css: css.code, props };

src/compile/Component.ts

@@ -330,7 +330,7 @@ export default class Component {
 			css,
 			ast: this.ast,
 			warnings: this.warnings,
-			vars: this.vars.filter(v => !v.global && !v.implicit && !v.internal).map(v => ({
+			vars: this.vars.filter(v => !v.global && !v.internal).map(v => ({
 				name: v.name,
 				export_name: v.export_name || null,
 				injected: v.injected || false,

test/vars/samples/implicit/_config.js

@@ -1,5 +1,16 @@
 export default {
 	test(assert, vars) {
-		assert.deepEqual(vars, []);
+		assert.deepEqual(vars, [
+			{
+				export_name: 'foo',
+				injected: false,
+				module: false,
+				mutated: false,
+				name: 'foo',
+				reassigned: false,
+				referenced: true,
+				writable: true,
+			},
+		]);
 	},
 };

test/vars/samples/template-references/_config.js

@@ -1,5 +1,36 @@
 export default {
 	test(assert, vars) {
-		assert.deepEqual(vars, []);
+		assert.deepEqual(vars, [
+			{
+				export_name: 'foo',
+				injected: false,
+				module: false,
+				mutated: false,
+				name: 'foo',
+				reassigned: false,
+				referenced: true,
+				writable: true,
+			},
+			{
+				export_name: 'Bar',
+				injected: false,
+				module: false,
+				mutated: false,
+				name: 'Bar',
+				reassigned: false,
+				referenced: true,
+				writable: true,
+			},
+			{
+				export_name: 'baz',
+				injected: false,
+				module: false,
+				mutated: false,
+				name: 'baz',
+				reassigned: false,
+				referenced: true,
+				writable: true,
+			},
+		]);
 	},
 };