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

fix: adjust `loc` property of `Program` nodes created from `<script>` elements (#17428)

Browse Source 
* fix: adjust program location to prevent corrupted sourcemaps

* update changeset

* go the other way
pull/17431/head
Rich Harris 3 weeks ago committed by GitHub
parent 41e8643814
commit 044dce9da5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
22 changed files with 40 additions and 28 deletions
Show Stats Download Patch File Download Diff File

5
.changeset/tall-phones-fix.md
Unescape View File

@ -0,0 +1,5 @@
---
'svelte': patch
---
fix: adjust `loc` property of `Program` nodes created from `<script>` elements

9
packages/svelte/src/compiler/phases/1-parse/read/script.js
Unescape View File

@ -6,6 +6,7 @@ import { regex_not_newline_characters } from '../../patterns.js';
import * as e from '../../../errors.js';
import * as w from '../../../warnings.js';
import { is_text_attribute } from '../../../utils/ast.js';
import { locator } from '../../../state.js';
const regex_closing_script_tag = /<\/script\s*>/;
const regex_starts_with_closing_script_tag = /^<\/script\s*>/;
@ -39,9 +40,15 @@ export function read_script(parser, start, attributes) {
parser.acorn_error(err);
}
// TODO is this necessary?
ast.start = script_start;
if (ast.loc) {
// Acorn always uses `0` as the start of a `Program`, but for sourcemap purposes
// we need it to be the start of the `<script>` contents
({ line: ast.loc.start.line, column: ast.loc.start.column } = locator(start));
({ line: ast.loc.end.line, column: ast.loc.end.column } = locator(parser.index));
}
/** @type {'default' | 'module'} */
let context = 'default';

2
packages/svelte/tests/parser-legacy/samples/binding-shorthand/output.json
Unescape View File

@ -63,7 +63,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/binding/output.json
Unescape View File

@ -73,7 +73,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/dynamic-import/output.json
Unescape View File

@ -21,7 +21,7 @@
},
"end": {
"line": 9,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/each-block-destructured/output.json
Unescape View File

@ -190,7 +190,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/generic-snippets/output.json
Unescape View File

@ -258,7 +258,7 @@
},
"end": {
"line": 2,
"column": 0
"column": 9
}
},
"body": [],

2
packages/svelte/tests/parser-legacy/samples/javascript-comments/output.json
Unescape View File

@ -298,7 +298,7 @@
},
"end": {
"line": 31,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/refs/output.json
Unescape View File

@ -73,7 +73,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/script-attribute-with-curly-braces/output.json
Unescape View File

@ -73,7 +73,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/script-comment-only/output.json
Unescape View File

@ -37,7 +37,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [],

6
packages/svelte/tests/parser-legacy/samples/script-context-module-unquoted/output.json
Unescape View File

@ -24,12 +24,12 @@
"end": 77,
"loc": {
"start": {
"line": 1,
"line": 5,
"column": 0
},
"end": {
"line": 7,
"column": 0
"column": 9
}
},
"body": [
@ -84,7 +84,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-legacy/samples/script/output.json
Unescape View File

@ -73,7 +73,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

4
packages/svelte/tests/parser-legacy/samples/whitespace-after-script-tag/output.json
Unescape View File

@ -72,8 +72,8 @@
"column": 0
},
"end": {
"line": 3,
"column": 0
"line": 8,
"column": 1
}
},
"body": [

2
packages/svelte/tests/parser-modern/samples/comment-before-function-binding/output.json
Unescape View File

@ -272,7 +272,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

4
packages/svelte/tests/parser-modern/samples/comment-before-script/output.json
Unescape View File

@ -34,12 +34,12 @@
"end": 67,
"loc": {
"start": {
"line": 1,
"line": 2,
"column": 0
},
"end": {
"line": 4,
"column": 0
"column": 9
}
},
"body": [

2
packages/svelte/tests/parser-modern/samples/generic-snippets/output.json
Unescape View File

@ -296,7 +296,7 @@
},
"end": {
"line": 2,
"column": 0
"column": 9
}
},
"body": [],

2
packages/svelte/tests/parser-modern/samples/loose-valid-each-as/output.json
Unescape View File

@ -227,7 +227,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

8
packages/svelte/tests/parser-modern/samples/options/output.json
Unescape View File

@ -111,12 +111,12 @@
"end": 93,
"loc": {
"start": {
"line": 1,
"line": 3,
"column": 0
},
"end": {
"line": 4,
"column": 0
"column": 9
}
},
"body": [],
@ -182,12 +182,12 @@
"end": 160,
"loc": {
"start": {
"line": 1,
"line": 6,
"column": 0
},
"end": {
"line": 7,
"column": 0
"column": 9
}
},
"body": [],

2
packages/svelte/tests/parser-modern/samples/script-style-no-markup/output.json
Unescape View File

@ -93,7 +93,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [],

2
packages/svelte/tests/parser-modern/samples/snippets/output.json
Unescape View File

@ -230,7 +230,7 @@
},
"end": {
"line": 1,
"column": 18
"column": 27
}
},
"body": [],

2
packages/svelte/tests/parser-modern/samples/typescript-in-event-handler/output.json
Unescape View File

@ -396,7 +396,7 @@
},
"end": {
"line": 3,
"column": 0
"column": 9
}
},
"body": [

Write Preview
Loading…
Cancel
Save