SourceMapConsumer.fromSourceMap ignores its sourceMapUrl argument

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#544 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 5 jours

@sebxt y travaille déjà.

Depuis le 20/10/2026.

  • #545 par @sebxt — ouverte
  • #549 par @user-Param — ouverte
  • #553 par @user-Param — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
javascript
Domaine
devtools

Piste de recherche

Commencez à BasicSourceMapConsumer.fromSourceMap dans l’implémentation du consommateur de source maps et comparez l’appel au constructeur avec la signature documentée. Transmettez l’argument URL et vérifiez les tests existants pour fromSourceMap et la résolution des sources. Le travail est terminé lorsque la reproduction résout src/a.js en http://example.com/maps/src/a.js, conformément au résultat du constructeur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

SourceMapConsumer.fromSourceMap(generator, sourceMapUrl) is typed and documented to accept the URL the source map lives at, but the URL is dropped, so sources are never resolved against it. Passing the same map through the constructor with the same URL gives a different result.

Repro:

const { SourceMapConsumer, SourceMapGenerator } = require("source-map");

const g = new SourceMapGenerator({ file: "out.js" });
g.addMapping({ generated: { line: 1, column: 0 }, original: { line: 1, column: 0 }, source: "src/a.js" });

const url = "http://example.com/maps/out.js.map";
(await SourceMapConsumer.fromSourceMap(g, url)).sources;
// actual:   [ 'src/a.js' ]
(await new SourceMapConsumer(g.toString(), url)).sources;
// [ 'http://example.com/maps/src/a.js' ]

Expected: both return [ 'http://example.com/maps/src/a.js' ].

Root cause: BasicSourceMapConsumer.fromSourceMap(aSourceMap, aSourceMapURL) calls new BasicSourceMapConsumer(aSourceMap.toString()) without forwarding aSourceMapURL.

Langage dominant
JavaScript
Étoiles
3.7k
Forks
388
Merge moyen
4 j 18 h
PR mergées (30 j)
1

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de mozilla/source-map

Toutes les issues de mozilla/source-map

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.