Does not work with builtin objects with internal slots (URL, Map, Set, etc)

Métriques du dépôt

Stars

61 stars (61 stars)

Métriques de merge PR

Métriques PR en attente (Métriques PR en attente)

Description

    traverse({ url: new URL("https://example.com") }).map(function () {
      if (this.node instanceof URL) {
        this.update(this.node.toString());
      }
    });

TypeError: Receiver must be an instance of class URL
 ❯ Object.<anonymous> src/log-sanitizer.spec.ts:57:31
     55|     traverse({ url: new URL("https://example.com") }).map(function () {
     56|       if (this.node instanceof URL) {
     57|         this.update(this.node.toString());
       |                               ^
     58|       }
     59|     });
 ❯ walker node_modules/traverse/index.js:169:16
 ❯ node_modules/traverse/index.js:190:17
 ❯ forEach node_modules/traverse/index.js:19:30
 ❯ walker node_modules/traverse/index.js:185:4
 ❯ walk node_modules/traverse/index.js:208:3
 ❯ Traverse.map node_modules/traverse/index.js:251:9

Guide contributeur

Direction de recherche

Enquêtez sur la raison pour laquelle la bibliothèque traverse échoue sur les objets intégrés avec des emplacements internes (ex. URL, Map, Set). La méthode update tente probablement de définir une propriété directement sur l'objet, ce qui déclenche une TypeError car les emplacements internes ne peuvent pas être définis par affectation. Envisagez d'utiliser Object.getOwnPropertyDescriptor et Object.defineProperty pour gérer ces cas, ou vérifiez ces objets et sautez les ou clonez les de manière appropriée.

Stack technique

javascript

Domaine

developer experience

Type d'issue

Bug

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

2

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 heures

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Active

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

JavaScriptNode.js

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

60