Error creating a SourceMapGenerator from a SourceMapConsumer of an index map (part 2)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- tooling
Línea de trabajo
Comienza en lib/source-map-consumer.js, alrededor de la implementación de sourceContentFor, y sigue cómo lib/source-map-generator.js la invoca desde fromSourceMap. Ejecuta el reproductor de index-map proporcionado y confirma que se conserva una entrada vacía de sourcesContent en lugar de tratarla como ausente, sin cambiar el comportamiento para contenido null o undefined.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
;(async () => {
let mozilla = require('source-map')
let map;
map = {
version: 3,
sections: [
{
offset: { line: 0, column: 0 },
map: {
version: 3,
sources: ['123', '456'],
sourcesContent: ['a\nb\nc\n', ''],
mappings: 'AAAA;AACA;AACA;'
}
}
]
}
// this map works for some reason
/*map = {
version: 3,
sources: ['123', '456'],
sourcesContent: ['a\nb\nc\n', ''],
mappings: 'AAAA;AACA;AACA;'
}*/
let consumer = await new mozilla.SourceMapConsumer(map)
consumer.sourceRoot = null; // https://github.com/mozilla/source-map/issues/345
let generator = mozilla.SourceMapGenerator.fromSourceMap(consumer)
console.log(generator.toJSON())
})().catch(console.log)
This errors out:
Error: "456" is not in the SourceMap.
at IndexedSourceMapConsumer.sourceContentFor (/tmp/node_modules/source-map/lib/source-map-consumer.js:840:11)
at /tmp/node_modules/source-map/lib/source-map-generator.js:82:42
at Array.forEach (<anonymous>)
at Function.fromSourceMap (/tmp/node_modules/source-map/lib/source-map-generator.js:72:32)
at /tmp/123/test.js:29:46
The issue is in here somewhere:
Content is empty - '' - but it is still in the sourcemap. I believe it should be if (content == null) { check, i.e. null or undefined, letting empty string pass along.
this may be related to https://github.com/mozilla/source-map/issues/345, so tagging it just in case
- Lenguaje dominante
- JavaScript
- Estrellas
- 3.7k
- Forks
- 370
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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 mozilla/source-map
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
mozilla/source-map#530 ·
-
allow wasm to be bundledAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
mozilla/source-map#527 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
mozilla/source-map#524 · 6 comentarios ·
-
source map seems like not work, is there some hints from the perspective of source-map tool?Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
mozilla/source-map#516 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
mozilla/source-map#510 · 1 reacción ·
Todos los issues de mozilla/source-map
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
parse-community/parse-server#10699 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: dashboard bug perceived difficulty: 3
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Nitjsefnie-Harness-Commons/daedalus#1179 ·
Los mantenedores suelen responder en 1 día
-
Aframe enhancement javascript
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
webarkit/Aframe-nft#17 ·
-
spec-feedback
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100