Comments are attached to wrong nodes as inner position
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, typescript
- Ambito
- compilers
Direzione di ricerca
Riprodurre il problema nella pagina di parsing dimostrativa di Esprima usando il JavaScript fornito e l’opzione “Attach comments”. Tracciare l’associazione dei commenti da quel punto di ingresso e verificare che i commenti non vengano duplicati o associati ai nodi sbagliati, mantenendo Comment 3 come commento interno del relativo blocco.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Steps to reproduce:
- Use Esprima demo page: https://esprima.org/demo/parse.html
- Copy paste this code inside the editor
function a() {
var b = 5;
/* Comment1 */
}
/* Comment2 */
{
/* Comment 3 */
}
- Check option (checkbox) "Attach comments"
Expected output:
All comments are trailing or leading with exception of the inner comment for "Comment 3"
Actual output:
{
"type": "Program",
"body": [
{
"type": "FunctionDeclaration",
"id": {
"type": "Identifier",
"name": "a"
},
"params": [],
"body": {
"type": "BlockStatement",
"body": [
{
"type": "VariableDeclaration",
"declarations": [
{
"type": "VariableDeclarator",
"id": {
"type": "Identifier",
"name": "b"
},
"init": {
"type": "Literal",
"value": 5,
"raw": "5"
}
}
],
"kind": "var",
"trailingComments": [
{
"type": "Block",
"value": " Comment1 ",
"range": [
32,
46
]
}
]
}
]
},
"generator": false,
"expression": false,
"async": false,
"trailingComments": [
{
"type": "Block",
"value": " Comment2 ",
"range": [
49,
63
]
}
]
},
{
"type": "BlockStatement",
"body": [],
"innerComments": [
{
"type": "Block",
"value": " Comment1 ",
"range": [
32,
46
]
},
{
"type": "Block",
"value": " Comment2 ",
"range": [
49,
63
]
},
{
"type": "Block",
"value": " Comment 3 ",
"range": [
69,
84
]
}
]
}
],
"sourceType": "script"
}
- Lingua principale
- TypeScript
- Stelle
- 7.1k
- Fork
- 772
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di jquery/esprima
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Tokenizer misidentifies Regex as Division after nested parentheses due to flat state trackingAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
-
Parse error in EsprimaAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Tutte le issue di jquery/esprima
Issue simili
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededForse già presa @github-actions l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
SSWConsulting/SSW.Rules#2809 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
automated-triage type/bug type/ci
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
📦 expo-router Android contributor: external needs review sdk-58 severity: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
expo/expo#53436 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Translate the READMEApertadocumentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
asasemahmed/openboard#68 ·
I maintainer di solito rispondono entro 1 giorno