josdejong/mathjs

TS regression: Narrowing of MathNodes by type

Aperta

#2810 aperta il 13 ott 2022

Vedi su GitHubGuida contributor
 (23 commenti) (0 reazioni) (0 assegnatari)JavaScript (1314 fork)batch import
bughelp wantedtypescript

Metriche repository

Star
 (15.065 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Describe the bug Prior to v11.3 (not sure how far back, but certainly at least 10), MathNode would be narrowed based on its type property:

const doStuffWithMathNode(node: MathNode) => {
  if (node.type === 'FunctionAssignmentNode) {
    // now TS knows that node is a FunctionAssignmentNode and lets you access associated props
    // like node.params
  }
  // back to being a wider MathNode
}

To Reproduce

Guida contributor

Direzione di ricerca
Indaga sulla regressione esaminando le definizioni di tipo per MathNode nel codice sorgente di mathjs, in particolare eventuali modifiche tra v11.2 e v11.3. Usa gli esempi di codesandbox forniti per comprendere lo scenario di fallimento. Cerca commit recenti che abbiano modificato il restringimento di tipo o i tipi MathNode. Rivedi i file di definizione dei tipi del progetto (ad es. src/type/) e testa il corretto restringimento utilizzando tipi condizionali.
Tech stack
typescript
Dominio
tooling
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
TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70