josdejong/mathjs

TS regression: Narrowing of MathNodes by type

Open

#2.810 aberto em 13 de out. de 2022

Ver no GitHub
 (23 comments) (0 reactions) (0 assignees)JavaScript (1.298 forks)batch import
bughelp wantedtypescript

Métricas do repositório

Stars
 (13.832 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Investigue a regressão examinando as definições de tipo para MathNode no código fonte do mathjs, particularmente quaisquer alterações entre v11.2 e v11.3. Use os exemplos do codesandbox fornecidos para entender o cenário de falha. Procure por commits recentes que modificaram o estreitamento de tipo ou os tipos MathNode. Revise os arquivos de definição de tipo do projeto (ex.: src/type/) e teste o estreitamento adequado usando tipos condicionais.
Pilha de tecnologia
typescript
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70