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

Metriche repository

Star

61 star (61 star)

Metriche merge PR

Metriche PR in attesa (Metriche PR in attesa)

Descrizione

    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

Guida contributor

Direzione di ricerca

Indaga il motivo per cui la libreria traverse fallisce sugli oggetti incorporati con slot interni (es. URL, Map, Set). Probabilmente il metodo update tenta di impostare una proprietà direttamente sull'oggetto, causando un TypeError perché gli slot interni non possono essere impostati tramite assegnazione. Considera l'utilizzo di Object.getOwnPropertyDescriptor e Object.defineProperty per gestire questi casi, oppure verifica la presenza di tali oggetti e saltali o clonali opportunamente.

Tech stack

javascript

Dominio

developer experience

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

2

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-3 ore

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

JavaScriptNode.js

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

60