msb-public
/
svelte
mirror of https://github.com/sveltejs/svelte
1
0
Code Issues Projects Releases Wiki Activity

Add compilation variables report option to allow getting all variables (even undeclared or internal) (#6192)

Browse Source 
* feat: add compilation variables report option

* test: add full var report test

* docs: document varsReport compile option
pull/6470/head
Maxime LUCE 4 years ago committed by GitHub
parent 487190fd93
commit aedf69c1cf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
12 changed files with 129 additions and 16 deletions
Show Stats Download Patch File Download Diff File

4
site/content/docs/04-compile-time.md
Unescape View File

@ -44,7 +44,8 @@ The following options can be passed to the compiler. None are required:
| `filename` | string | `null` | `filename` | string | `null`
| `name` | string | `"Component"` | `name` | string | `"Component"`
| `format` | `"esm"` or `"cjs"` | `"esm"` | `format` | `"esm"` or `"cjs"` | `"esm"`
| `generate` | `"dom"` or `"ssr"` | `"dom"` | `generate` | `"dom"` or `"ssr" or false` | `"dom"`
| `varsReport` | `"strict"` or `"full" or false` | `"strict"`
| `dev` | boolean | `false` | `dev` | boolean | `false`
| `immutable` | boolean | `false` | `immutable` | boolean | `false`
| `hydratable` | boolean | `false` | `hydratable` | boolean | `false`
@ -66,6 +67,7 @@ The following options can be passed to the compiler. None are required:
| `name` | `"Component"` | `string` that sets the name of the resulting JavaScript class (though the compiler will rename it if it would otherwise conflict with other variables in scope). It will normally be inferred from `filename`. | `name` | `"Component"` | `string` that sets the name of the resulting JavaScript class (though the compiler will rename it if it would otherwise conflict with other variables in scope). It will normally be inferred from `filename`.
| `format` | `"esm"` | If `"esm"`, creates a JavaScript module (with `import` and `export`). If `"cjs"`, creates a CommonJS module (with `require` and `module.exports`), which is useful in some server-side rendering situations or for testing. | `format` | `"esm"` | If `"esm"`, creates a JavaScript module (with `import` and `export`). If `"cjs"`, creates a CommonJS module (with `require` and `module.exports`), which is useful in some server-side rendering situations or for testing.
| `generate` | `"dom"` | If `"dom"`, Svelte emits a JavaScript class for mounting to the DOM. If `"ssr"`, Svelte emits an object with a `render` method suitable for server-side rendering. If `false`, no JavaScript or CSS is returned; just metadata. | `generate` | `"dom"` | If `"dom"`, Svelte emits a JavaScript class for mounting to the DOM. If `"ssr"`, Svelte emits an object with a `render` method suitable for server-side rendering. If `false`, no JavaScript or CSS is returned; just metadata.
| `varsReport` | `"strict"` | If `"strict"`, Svelte returns a variables report with only variables that are not globals nor internals. If `"full"`, Svelte returns a variables report with all detected variables. If `false`, no variables report is returned.
| `dev` | `false` | If `true`, causes extra code to be added to components that will perform runtime checks and provide debugging information during development. | `dev` | `false` | If `true`, causes extra code to be added to components that will perform runtime checks and provide debugging information during development.
| `immutable` | `false` | If `true`, tells the compiler that you promise not to mutate any objects. This allows it to be less conservative about checking whether values have changed. | `immutable` | `false` | If `true`, tells the compiler that you promise not to mutate any objects. This allows it to be less conservative about checking whether values have changed.
| `hydratable` | `false` | If `true` when generating DOM code, enables the `hydrate: true` runtime option, which allows a component to upgrade existing DOM rather than creating new DOM from scratch. When generating SSR code, this adds markers to `<head>` elements so that hydration knows which to replace. | `hydratable` | `false` | If `true` when generating DOM code, enables the `hydrate: true` runtime option, which allows a component to upgrade existing DOM rather than creating new DOM from scratch. When generating SSR code, this adds markers to `<head>` elements so that hydration knows which to replace.

47
src/compiler/compile/Component.ts
Unescape View File

@ -183,9 +183,12 @@ export default class Component {
this.stylesheet.warn_on_unused_selectors(this); this.stylesheet.warn_on_unused_selectors(this);
} }
add_var(variable: Var) { add_var(variable: Var, add_to_lookup = true) {
this.vars.push(variable); this.vars.push(variable);
this.var_lookup.set(variable.name, variable);
if (add_to_lookup) {
this.var_lookup.set(variable.name, variable);
}
} }
add_reference(name: string) { add_reference(name: string) {
@ -216,6 +219,10 @@ export default class Component {
variable.subscribable = true; variable.subscribable = true;
} }
} else { } else {
if (this.compile_options.varsReport === 'full') {
this.add_var({ name, referenced: true }, false);
}
this.used_names.add(name); this.used_names.add(name);
} }
} }
@ -340,19 +347,7 @@ export default class Component {
css, css,
ast: this.original_ast, ast: this.original_ast,
warnings: this.warnings, warnings: this.warnings,
vars: this.vars vars: this.get_vars_report(),
.filter(v => !v.global && !v.internal)
.map(v => ({
name: v.name,
export_name: v.export_name || null,
injected: v.injected || false,
module: v.module || false,
mutated: v.mutated || false,
reassigned: v.reassigned || false,
referenced: v.referenced || false,
writable: v.writable || false,
referenced_from_script: v.referenced_from_script || false
})),
stats: this.stats.render() stats: this.stats.render()
}; };
} }
@ -402,6 +397,28 @@ export default class Component {
}; };
} }
get_vars_report(): Var[] {
const { compile_options, vars } = this;
const vars_report = compile_options.varsReport === false
? []
: compile_options.varsReport === 'full'
? vars
: vars.filter(v => !v.global && !v.internal);
return vars_report.map(v => ({
name: v.name,
export_name: v.export_name || null,
injected: v.injected || false,
module: v.module || false,
mutated: v.mutated || false,
reassigned: v.reassigned || false,
referenced: v.referenced || false,
writable: v.writable || false,
referenced_from_script: v.referenced_from_script || false
}));
}
error( error(
pos: { pos: {
start: number; start: number;

1
src/compiler/compile/index.ts
Unescape View File

@ -14,6 +14,7 @@ const valid_options = [
'filename', 'filename',
'sourcemap', 'sourcemap',
'generate', 'generate',
'varsReport',
'outputFilename', 'outputFilename',
'cssOutputFilename', 'cssOutputFilename',
'sveltePath', 'sveltePath',

1
src/compiler/interfaces.ts
Unescape View File

@ -116,6 +116,7 @@ export interface CompileOptions {
name?: string; name?: string;
filename?: string; filename?: string;
generate?: 'dom' | 'ssr' | false; generate?: 'dom' | 'ssr' | false;
varsReport?: 'full' | 'strict' | false;
sourcemap?: object | string; sourcemap?: object | string;
outputFilename?: string; outputFilename?: string;

9
test/vars/samples/vars-report-false/_config.js
Unescape View File

@ -0,0 +1,9 @@
export default {
options: {
varsReport: false
},
test(assert, vars) {
assert.deepEqual(vars, []);
}
};

5
test/vars/samples/vars-report-false/input.svelte
Unescape View File

@ -0,0 +1,5 @@
<script>
export let foo = "bar";
</script>
{foo}

19
test/vars/samples/vars-report-full-noscript/_config.js
Unescape View File

@ -0,0 +1,19 @@
export default {
options: {
varsReport: 'full'
},
test(assert, vars) {
assert.deepEqual(vars, [{
name: 'foo',
export_name: null,
injected: false,
module: false,
mutated: false,
reassigned: false,
referenced: true,
referenced_from_script: false,
writable: false
}]);
}
};

1
test/vars/samples/vars-report-full-noscript/input.svelte
Unescape View File

@ -0,0 +1 @@
{foo}

31
test/vars/samples/vars-report-full-script/_config.js
Unescape View File

@ -0,0 +1,31 @@
export default {
options: {
varsReport: 'full'
},
test(assert, vars) {
assert.deepEqual(vars, [
{
name: 'foo',
export_name: 'foo',
injected: false,
module: false,
mutated: false,
reassigned: false,
referenced: true,
referenced_from_script: false,
writable: true
}, {
name: 'bar',
export_name: null,
injected: false,
module: false,
mutated: false,
reassigned: false,
referenced: true,
referenced_from_script: false,
writable: false
}
]);
}
};

5
test/vars/samples/vars-report-full-script/input.svelte
Unescape View File

@ -0,0 +1,5 @@
<script>
export let foo = "bar";
</script>
{foo} {bar}

19
test/vars/samples/vars-report-full/_config.js
Unescape View File

@ -0,0 +1,19 @@
export default {
options: {
varsReport: 'full'
},
test(assert, vars) {
assert.deepEqual(vars, [{
name: 'foo',
export_name: null,
injected: false,
module: false,
mutated: false,
reassigned: false,
referenced: true,
referenced_from_script: false,
writable: false
}]);
}
};

3
test/vars/samples/vars-report-full/input.svelte
Unescape View File

@ -0,0 +1,3 @@
<script></script>
{foo}
Write Preview
Loading…
Cancel
Save