Handle Syntax Error
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
- Área
- developer-experience
Línea de trabajo
Comienza reproduciendo el SyntaxError del archivo generado e inspecciona cómo node-source-map-support procesa actualmente error.stack para las excepciones ordinarias. Sigue la primera línea del stack que contiene la ubicación generada y, después, verifica que las ubicaciones de SyntaxError se resuelvan en el código fuente original sin cambiar el manejo de excepciones existente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
First of all thank you for providing this project! It's a great help for my current project.
Regarding this issue: I would like to request that a special case be added that handles syntax errors so that the place of the original source is displayed when there is a syntax error in the generated file.
This is not very useful for projects like coffescript, typescript… which feature their own parsers (and thus do their own syntax error reporting); however it is useful for a use case like mine in which I just copy/past js source from one file into another where the input is not strictly tested for validity.
(I am building a tool that extracts examples from documentation and executes them as tests).
E.g. my errors are correctly resolved for normal exceptions, however when an (induced) syntax error happened, the output pasted below was the output. In there you can see the syntax error indicating the location in the generated file.
This could be solved by adding a special handler for syntax errors; extracting the generated location from the first line of error.stack…
/tmp/ferrum.doctest-ferrum.doctest-6d12dfba-331b-478f-94bf-862f4569da6/examples.test.js:10 [158/14653]
const foo = 42;
^^^^^
SyntaxError: Unexpected token 'const'
at wrapSafe (internal/modules/cjs/loader.js:1050:16)
at Module._compile (internal/modules/cjs/loader.js:1098:27)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
at Module.load (internal/modules/cjs/loader.js:983:32)
at Function.Module._load (internal/modules/cjs/loader.js:891:14)
at Module.require (internal/modules/cjs/loader.js:1023:19)
at require (internal/modules/cjs/helpers.js:72:18)
at /home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/mocha.js:334:36
at Array.forEach (<anonymous>)
at Mocha.loadFiles (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/mocha.js:331:14)
at Mocha.run (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/mocha.js:809:10)
at Object.exports.singleRun (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/cli/run-helpers.js:108:16)
at exports.runMocha (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/cli/run-helpers.js:142:13)
at Object.exports.handler (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/lib/cli/run.js:292:3)
at Object.runCommand (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/node_modules/yargs/lib/command.js:242:26)
at Object.parseArgs [as _parseArgs] (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/node_modules/yargs/yargs.js:1087:28)
at Object.parse (/home/karo/pr0j/adobe/ferrum.doctest/node_modules/mocha/node_modules/yargs/yargs.js:566:25)
- Lenguaje dominante
- JavaScript
- Estrellas
- 2.2k
- Forks
- 221
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de evanw/node-source-map-support
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
evanw/node-source-map-support#344 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
evanw/node-source-map-support#343 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
evanw/node-source-map-support#337 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
evanw/node-source-map-support#333 ·
-
Provide esm versionAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
evanw/node-source-map-support#330 · 2 comentarios · 9 reacciones ·
Todos los issues de evanw/node-source-map-support
Issues similares
-
feature request
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
docs hacktoberfest help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
pacocartones/free-llm-api-hub#347 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
cytoscape/cytoscape.js#3536 ·
Los mantenedores suelen responder en 15 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
cuttle-cards/cuttle#1440 ·
Los mantenedores suelen responder en 1 día
-
URL-encoded parser builds nested objects nothing readsPosiblemente ocupada @Senthil455 la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Janani-bn/CivicFix#168 · 1 comentario ·
Los mantenedores suelen responder en 1 día