`toStringWithSourceMap` behaves incorrectly if `SourceNode`s source is `null` or is an empty string
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- devtools
Línea de trabajo
El punto de entrada es SourceNode.toStringWithSourceMap; primero reproduce los casos de null y empty-string del fragmento del issue y sigue cómo cada fuente llega a la generación de mappings. Se considera terminado cuando ambas entradas tienen un comportamiento intencionado y coherente, sin perder mappings silenciosamente ni fallar más adelante con un error de invalid-mapping.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When source set to null, mapping does not generated.
When source set to empty string there is an error during mapping calculation (not during SourceNode creation as expected if empty string is forbidden).
This code snippet demonstrates both problems: https://runkit.com/embed/47lduu6k2173
Source code of the snippet:
let SourceNode = require('source-map').SourceNode;
function test(source) {
let node = new SourceNode(
null,
null,
source,
[
'prefix',
new SourceNode(1, 0, source, 'code'),
'suffix',
]
);
let { code, map } = node.toStringWithSourceMap();
console.log(map.toJSON(), code);
}
// {version: 3, sources: ["source"], names: [], mappings: "MAAA,I"}
test("source");// OK
// {version: 3, sources: [], names: [], mappings: ""}
test(null); // Empty mapping
// Error: Invalid mapping: {"generated":{"line":1,"column":6},"source":"","original":{"line":1,"column":0},"name":null}
test(""); // Error
- 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
-
Complexity: Small P-Feature: Projects page ready for merge team role: back end/devOps role: front end size: 0.25pt
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 67/100
bellingcat/toolkit#905 ·
-
self-care self-care:docs-build-time-investigator
Dificultad 2/5 Medio día Aptitud para principiantes 76/100
githubnext/gh-aw-cao#14191 ·
Los mantenedores suelen responder en 1 día
-
effort:low impact:medium RAG status: auto-triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
mastra-ai/mastra#25229 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
sugarlabs/musicblocks#8984 ·
Los mantenedores suelen responder en 1 día